Fix CI detection in tests

Author: SteveSandersonMS

nodejs/test/e2e/harness/sdkTestContext.ts

@@ -13,6 +13,9 @@ import { CopilotClient } from "../../../src";
 import { CapiProxy } from "./CapiProxy";
 import { retry } from "./sdkTestHelper";
 
+// Unfortunately the VS Code Vitest extension sets CI=true in all non-debug runs, so we have to exclude that
+export const isCI = process.env.CI === "true" && !process.env.VITEST_VSCODE;
+
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 const SNAPSHOTS_DIR = resolve(__dirname, "../../../../test/snapshots");
@@ -42,7 +45,7 @@ export async function createSdkTestContext({
         logLevel: logLevel || "error",
         cliPath: process.env.COPILOT_CLI_PATH,
         // Use fake token in CI to allow cached responses without real auth
-        githubToken: process.env.CI === "true" ? "fake-token-for-e2e-tests" : undefined,
+        githubToken: isCI ? "fake-token-for-e2e-tests" : undefined,
     });
 
     const harness = { homeDir, workDir, openAiEndpoint, copilotClient, env };

nodejs/test/e2e/session.test.ts

@@ -2,7 +2,7 @@ import { rm } from "fs/promises";
 import { describe, expect, it, onTestFinished } from "vitest";
 import { ParsedHttpExchange } from "../../../test/harness/replayingCapiProxy.js";
 import { CopilotClient, approveAll } from "../../src/index.js";
-import { createSdkTestContext } from "./harness/sdkTestContext.js";
+import { createSdkTestContext, isCI } from "./harness/sdkTestContext.js";
 import { getFinalAssistantMessage, getNextEventOfType } from "./harness/sdkTestHelper.js";
 
 describe("Sessions", async () => {
@@ -187,7 +187,7 @@ describe("Sessions", async () => {
         // Resume using a new client
         const newClient = new CopilotClient({
             env,
-            githubToken: process.env.CI === "true" ? "fake-token-for-e2e-tests" : undefined,
+            githubToken: isCI ? "fake-token-for-e2e-tests" : undefined,
         });
 
         onTestFinished(() => newClient.forceStop());

test/harness/replayingCapiProxy.ts

@@ -311,7 +311,9 @@ export class ReplayingCapiProxy extends CapturingHttpProxy {
 
         // Fallback to normal proxying if no cached response found
         // This implicitly captures the new exchange too
-        if (process.env.CI === "true") {
+        // The VS Code Vitest extension always sets CI=true in non-debug mode, so we have to exclude that case
+        const isCI = process.env.CI === "true" && !process.env.VITEST_VSCODE;
+        if (isCI) {
           await exitWithNoMatchingRequestError(
             options,
             state.testInfo,