Preserve runtime diagnostics on failure

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 6e902b9a-5527-4a32-a5a3-e0bf5bfef3f7

Author: stephentoub

.github/workflows/nodejs-sdk-tests.yml

@@ -10,19 +10,19 @@ on:
   pull_request:
     types: [opened, synchronize, reopened, ready_for_review]
     paths:
-      - 'nodejs/**'
-      - 'test/**'
-      - '.github/workflows/nodejs-sdk-tests.yml'
-      - '!nodejs/scripts/**'
-      - '!**/*.md'
-      - '!**/LICENSE*'
-      - '!**/.gitignore'
-      - '!**/.editorconfig'
-      - '!**/*.png'
-      - '!**/*.jpg'
-      - '!**/*.jpeg'
-      - '!**/*.gif'
-      - '!**/*.svg'
+      - "nodejs/**"
+      - "test/**"
+      - ".github/workflows/nodejs-sdk-tests.yml"
+      - "!nodejs/scripts/**"
+      - "!**/*.md"
+      - "!**/LICENSE*"
+      - "!**/.gitignore"
+      - "!**/.editorconfig"
+      - "!**/*.png"
+      - "!**/*.jpg"
+      - "!**/*.jpeg"
+      - "!**/*.gif"
+      - "!**/*.svg"
   workflow_dispatch:
   merge_group:
 
@@ -85,6 +85,7 @@ jobs:
       - name: Run Node.js SDK tests
         env:
           COPILOT_HMAC_KEY: ${{ secrets.COPILOT_DEVELOPER_CLI_INTEGRATION_HMAC_KEY }}
+          COPILOT_SDK_PRESERVE_TEST_HOME: "1"
         run: |
           pwsh.exe -NoProfile -Command '
             while ($true) {
@@ -98,3 +99,19 @@ jobs:
           monitor_pid=$!
           trap 'kill "$monitor_pid"' EXIT
           npm test
+
+      - name: Collect runtime diagnostics
+        if: failure()
+        shell: pwsh
+        run: |
+          $paths = Get-ChildItem $env:TEMP -Directory -Filter "copilot-test-home-*"
+          if ($paths) {
+            Compress-Archive -Path $paths.FullName -DestinationPath runtime-diagnostics.zip
+          }
+
+      - name: Upload runtime diagnostics
+        if: failure()
+        uses: actions/upload-artifact@v4
+        with:
+          name: nodejs-runtime-diagnostics
+          path: nodejs/runtime-diagnostics.zip

nodejs/test/e2e/harness/sdkTestContext.ts

@@ -230,7 +230,9 @@ export async function createSdkTestContext({
             // beforeEach below), so the worker inherits it; passing a per-client env here
             // would have no effect (and is rejected by the in-process transport).
             env: effectiveInProcess ? undefined : mergedEnv,
-            logLevel: logLevel ?? "error",
+            logLevel:
+                logLevel ??
+                (process.env.COPILOT_SDK_PRESERVE_TEST_HOME === "1" ? "debug" : "error"),
             connection: effectiveConnection,
             gitHubToken: authTokenToUse,
             ...rest,
@@ -308,9 +310,13 @@ export async function createSdkTestContext({
     afterAll(async () => {
         await runDiagnosticPhase("client.stop", () => copilotClient.stop());
         await runDiagnosticPhase("proxy.stop", () => openAiEndpoint.stop(anyTestFailed));
-        await runDiagnosticPhase("remove copilotHomeDir", () =>
-            rmDir("remove e2e test copilotHomeDir", copilotHomeDir)
-        );
+        if (anyTestFailed && process.env.COPILOT_SDK_PRESERVE_TEST_HOME === "1") {
+            console.error(`[sdk-test-diagnostic pid=${process.pid}] preserved ${copilotHomeDir}`);
+        } else {
+            await runDiagnosticPhase("remove copilotHomeDir", () =>
+                rmDir("remove e2e test copilotHomeDir", copilotHomeDir)
+            );
+        }
         await runDiagnosticPhase("remove homeDir", () => rmDir("remove e2e test homeDir", homeDir));
         await runDiagnosticPhase("remove workDir", () => rmDir("remove e2e test workDir", workDir));
     });