fix IPC handle is longer than 103 chars error for CI

.github/workflows/pr-check.yml

@@ -152,10 +152,7 @@ jobs:
         run: npm run compile-tests
 
       - name: Configure Test Settings
-        run: |
-          mkdir -p .vscode-test/user-data/User
-          echo '{"python.useEnvironmentsExtension": true}' > .vscode-test/user-data/User/settings.json
-        shell: bash
+        run: node build/setup-test-user-data.mjs
 
       - name: Run Smoke Tests (Linux)
         if: runner.os == 'Linux'
@@ -245,10 +242,7 @@ jobs:
         run: npm run compile-tests
 
       - name: Configure Test Settings
-        run: |
-          mkdir -p .vscode-test/user-data/User
-          echo '{"python.useEnvironmentsExtension": true}' > .vscode-test/user-data/User/settings.json
-        shell: bash
+        run: node build/setup-test-user-data.mjs
 
       - name: Run E2E Tests (Linux)
         if: runner.os == 'Linux'
@@ -338,10 +332,7 @@ jobs:
         run: npm run compile-tests
 
       - name: Configure Test Settings
-        run: |
-          mkdir -p .vscode-test/user-data/User
-          echo '{"python.useEnvironmentsExtension": true}' > .vscode-test/user-data/User/settings.json
-        shell: bash
+        run: node build/setup-test-user-data.mjs
 
       - name: Run Integration Tests (Linux)
         if: runner.os == 'Linux'
@@ -388,10 +379,7 @@ jobs:
         run: npm run compile-tests
 
       - name: Configure Test Settings
-        run: |
-          mkdir -p .vscode-test/user-data/User
-          echo '{"python.useEnvironmentsExtension": true}' > .vscode-test/user-data/User/settings.json
-        shell: bash
+        run: node build/setup-test-user-data.mjs
 
       - name: Run Integration Tests Multi-Root (Linux)
         if: runner.os == 'Linux'

.github/workflows/push-check.yml

@@ -153,10 +153,7 @@ jobs:
         run: npm run compile-tests
 
       - name: Configure Test Settings
-        run: |
-          mkdir -p .vscode-test/user-data/User
-          echo '{"python.useEnvironmentsExtension": true}' > .vscode-test/user-data/User/settings.json
-        shell: bash
+        run: node build/setup-test-user-data.mjs
 
       - name: Run Smoke Tests (Linux)
         if: runner.os == 'Linux'
@@ -246,10 +243,7 @@ jobs:
         run: npm run compile-tests
 
       - name: Configure Test Settings
-        run: |
-          mkdir -p .vscode-test/user-data/User
-          echo '{"python.useEnvironmentsExtension": true}' > .vscode-test/user-data/User/settings.json
-        shell: bash
+        run: node build/setup-test-user-data.mjs
 
       - name: Run E2E Tests (Linux)
         if: runner.os == 'Linux'
@@ -339,10 +333,7 @@ jobs:
         run: npm run compile-tests
 
       - name: Configure Test Settings
-        run: |
-          mkdir -p .vscode-test/user-data/User
-          echo '{"python.useEnvironmentsExtension": true}' > .vscode-test/user-data/User/settings.json
-        shell: bash
+        run: node build/setup-test-user-data.mjs
 
       - name: Run Integration Tests (Linux)
         if: runner.os == 'Linux'

.vscode-test.mjs

@@ -1,8 +1,12 @@
 import { defineConfig } from '@vscode/test-cli';
+import * as os from 'os';
 import * as path from 'path';
 
-// Explicit user data directory - ensures VS Code reads our settings.json
-const userDataDir = path.resolve('.vscode-test/user-data');
+// Keep this path short: macOS caps Unix-domain socket paths at 103 chars and
+// VS Code creates `<userDataDir>/<x.y>-main.sock`. An in-workspace location
+// (e.g. /Users/runner/work/<repo>/<repo>/.vscode-test/user-data) overflows.
+// Must match build/setup-test-user-data.mjs.
+const userDataDir = path.join(os.tmpdir(), 'vsct-ud');
 
 export default defineConfig([
     {

build/setup-test-user-data.mjs

@@ -0,0 +1,19 @@
+// Writes the VS Code test user-data settings.json to a short path under
+// os.tmpdir(). Used by CI to seed `python.useEnvironmentsExtension=true`.
+// `.vscode-test.mjs` MUST compute the same `userDataDir` so both sides agree.
+//
+// Why os.tmpdir(): macOS Unix-domain socket paths are capped at 103 chars.
+// VS Code creates `<userDataDir>/<x.y>-main.sock`, so an in-workspace path
+// like `/Users/runner/work/<repo>/<repo>/.vscode-test/user-data/...` overflows.
+import * as fs from 'fs';
+import * as os from 'os';
+import * as path from 'path';
+
+const userDataDir = path.join(os.tmpdir(), 'vsct-ud');
+const userDir = path.join(userDataDir, 'User');
+fs.mkdirSync(userDir, { recursive: true });
+fs.writeFileSync(
+    path.join(userDir, 'settings.json'),
+    JSON.stringify({ 'python.useEnvironmentsExtension': true }) + '\n',
+);
+console.log(userDataDir);

src/managers/common/fastPath.ts

@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
 
+import * as fs from 'fs';
 import { Uri } from 'vscode';
 import { GetEnvironmentScope, PythonEnvironment, PythonEnvironmentApi } from '../../api';
 import { traceError, traceVerbose, traceWarn } from '../../common/logging';
@@ -48,6 +49,28 @@ export function getProjectFsPathForScope(api: Pick<PythonEnvironmentApi, 'getPyt
     return api.getPythonProject(scope)?.uri.fsPath ?? scope.fsPath;
 }
 
+/**
+ * Cheap local existence check on the persisted path. If the cached interpreter has
+ * been deleted/renamed/moved we can short-circuit before spawning PET (which holds
+ * the cache mutex and can take up to the spawn timeout to fail).
+ *
+ * Returns true if the path exists (or if the check itself errors — in which case we
+ * fall through to the normal resolve so we don't introduce a new failure mode).
+ */
+async function persistedPathExists(persistedPath: string): Promise<boolean> {
+    try {
+        await fs.promises.access(persistedPath);
+        return true;
+    } catch (err) {
+        const code = (err as NodeJS.ErrnoException)?.code;
+        if (code === 'ENOENT' || code === 'ENOTDIR') {
+            return false;
+        }
+        // Unknown error (e.g. EACCES) — don't treat as missing; let resolve() decide.
+        return true;
+    }
+}
+
 /**
  * Attempts fast-path resolution for manager.get(): if full initialization hasn't completed yet
  * and there's a persisted environment for the workspace, resolve it directly via nativeFinder
@@ -100,6 +123,18 @@ export async function tryFastPathGet(opts: FastPathOptions): Promise<FastPathRes
         const persistedPath = await getGlobalPersistedPath();
 
         if (persistedPath) {
+            // Cheap local existence check — avoids spawning PET (and waiting on the
+            // bounded timeout) when the cached interpreter has been removed.
+            if (!(await persistedPathExists(persistedPath))) {
+                sendTelemetryEvent(EventNames.GLOBAL_ENV_CACHE, cacheStopWatch.elapsedTime, {
+                    managerLabel: opts.label,
+                    result: 'stale',
+                });
+                traceVerbose(
+                    `[${opts.label}] Fast path: persisted path '${persistedPath}' does not exist, falling through to slow path`,
+                );
+                return undefined;
+            }
             try {
                 const resolved = await opts.resolve(persistedPath);
                 if (resolved) {
@@ -136,6 +171,14 @@ export async function tryFastPathGet(opts: FastPathOptions): Promise<FastPathRes
         const persistedPath = await opts.getPersistedPath(opts.getProjectFsPath(scope));
 
         if (persistedPath) {
+            // Cheap local existence check — avoids spawning PET (and waiting on the
+            // bounded timeout) when the cached interpreter has been removed.
+            if (!(await persistedPathExists(persistedPath))) {
+                traceVerbose(
+                    `[${opts.label}] Fast path: persisted path '${persistedPath}' does not exist, falling through to slow path`,
+                );
+                return undefined;
+            }
             try {
                 const resolved = await opts.resolve(persistedPath);
                 if (resolved) {

src/test/managers/common/fastPath.unit.test.ts

@@ -31,7 +31,7 @@ interface FastPathTestOptions {
 
 function createOpts(overrides?: Partial<FastPathOptions>): FastPathTestOptions {
     const setInitialized = sinon.stub();
-    const persistedPath = path.resolve('persisted', 'path');
+    const persistedPath = __filename;
     return {
         opts: {
             initialized: undefined,
@@ -77,7 +77,7 @@ suite('tryFastPathGet', () => {
     });
 
     test('returns resolved env for global scope when getGlobalPersistedPath returns a path', async () => {
-        const globalPath = path.resolve('usr', 'bin', 'python3');
+        const globalPath = __filename;
         const resolve = sinon.stub().resolves(createMockEnv(globalPath));
         const { opts } = createOpts({
             scope: undefined,
@@ -116,7 +116,7 @@ suite('tryFastPathGet', () => {
     });
 
     test('reports stale when global cached path resolves to undefined', async () => {
-        const globalPath = path.resolve('usr', 'bin', 'python3');
+        const globalPath = __filename;
         const { opts } = createOpts({
             scope: undefined,
             getGlobalPersistedPath: sinon.stub().resolves(globalPath),
@@ -132,7 +132,7 @@ suite('tryFastPathGet', () => {
     });
 
     test('returns undefined for global scope when cached path resolve fails', async () => {
-        const globalPath = path.resolve('usr', 'bin', 'python3');
+        const globalPath = __filename;
         const { opts } = createOpts({
             scope: undefined,
             getGlobalPersistedPath: sinon.stub().resolves(globalPath),
@@ -150,7 +150,7 @@ suite('tryFastPathGet', () => {
     });
 
     test('global scope fast path starts background init when initialized is undefined', async () => {
-        const globalPath = path.resolve('usr', 'bin', 'python3');
+        const globalPath = __filename;
         const startBackgroundInit = sinon.stub().resolves();
         const { opts, setInitialized } = createOpts({
             scope: undefined,
@@ -201,6 +201,37 @@ suite('tryFastPathGet', () => {
         assert.strictEqual(result, undefined);
     });
 
+    test('skips resolve and falls through when workspace persisted path no longer exists on disk', async () => {
+        const missingPath = path.resolve('does', 'not', 'exist', 'python-missing');
+        const resolve = sinon.stub().resolves(createMockEnv(missingPath));
+        const { opts } = createOpts({
+            getPersistedPath: sinon.stub().resolves(missingPath),
+            resolve,
+        });
+        const result = await tryFastPathGet(opts);
+
+        assert.strictEqual(result, undefined, 'Should fall through when cached path is missing');
+        assert.ok(resolve.notCalled, 'Should not invoke resolve (and thus PET) when path is missing');
+    });
+
+    test('skips resolve and reports stale when global persisted path no longer exists on disk', async () => {
+        const missingPath = path.resolve('does', 'not', 'exist', 'python-missing');
+        const resolve = sinon.stub().resolves(createMockEnv(missingPath));
+        const { opts } = createOpts({
+            scope: undefined,
+            getGlobalPersistedPath: sinon.stub().resolves(missingPath),
+            resolve,
+        });
+        const result = await tryFastPathGet(opts);
+
+        assert.strictEqual(result, undefined, 'Should fall through when cached global path is missing');
+        assert.ok(resolve.notCalled, 'Should not invoke resolve (and thus PET) when path is missing');
+        assert.ok(sendTelemetryStub.calledOnce, 'Should send telemetry for stale global cache');
+        const [eventName, , props] = sendTelemetryStub.firstCall.args;
+        assert.strictEqual(eventName, EventNames.GLOBAL_ENV_CACHE);
+        assert.strictEqual(props.result, 'stale');
+    });
+
     test('calls getProjectFsPath with the scope Uri', async () => {
         const scope = Uri.file(path.resolve('my', 'project'));
         const getProjectFsPath = sinon.stub().returns(scope.fsPath);
@@ -214,7 +245,7 @@ suite('tryFastPathGet', () => {
     test('passes project fsPath to getPersistedPath', async () => {
         const projectPath = path.resolve('project', 'path');
         const getProjectFsPath = sinon.stub().returns(projectPath);
-        const getPersistedPath = sinon.stub().resolves(path.resolve('persisted'));
+        const getPersistedPath = sinon.stub().resolves(__filename);
         const { opts } = createOpts({
             getProjectFsPath,
             getPersistedPath,
@@ -266,7 +297,7 @@ suite('tryFastPathGet', () => {
         const getPersistedPath = sinon.stub().callsFake(
             () =>
                 new Promise<string | undefined>((resolve) => {
-                    releasePersistedRead = () => resolve(path.resolve('persisted', 'path'));
+                    releasePersistedRead = () => resolve(__filename);
                 }),
         );
         const { opts, setInitialized } = createOpts({ getPersistedPath });