review-followups-4: @vscode/test-electron integration tests (R17)

.github/workflows/ci.yml

@@ -63,6 +63,17 @@ jobs:
         # asserts activate/deactivate are exported.
         run: npm run smoke
 
+      - name: Integration tests (real VS Code)
+        # @vscode/test-electron boots a real extension host and runs
+        # the mocha suite under src/test/integration/. Verifies the
+        # activation / command / view contracts that unit tests can
+        # only approximate. Linux-only — VS Code needs an X server
+        # (xvfb-run) on headless runners; Windows/macOS in this
+        # matrix already exercise the platform-specific paths via the
+        # unit suite + bundle smoke.
+        if: matrix.os == 'ubuntu-latest'
+        run: xvfb-run -a npm run test:integration
+
       - name: npm audit (prod deps, high+)
         # Network-bound and platform-independent; one run is enough.
         if: matrix.os == 'ubuntu-latest'

.gitignore

@@ -77,6 +77,7 @@ web_modules/
 # Next.js build output
 .next
 out
+out-test
 
 # Nuxt.js build / generate output
 .nuxt

.vscode-test.mjs

@@ -0,0 +1,25 @@
+// Configuration for `@vscode/test-cli`, which boots a real VS Code
+// extension host so we can verify the contracts that unit tests can
+// only approximate: that activation actually fires, commands really
+// register, the view appears in the activity bar. The compiled tests
+// live in `out-test/` (separate from the esbuild bundle in `dist/`)
+// and use mocha-bdd-ui — same convention every official VS Code
+// extension uses.
+
+import { defineConfig } from "@vscode/test-cli";
+
+export default defineConfig({
+  // Compiled test files; `tsconfig.integration.json` emits them.
+  files: "out-test/test/integration/**/*.test.js",
+  // Open the deliberately-vulnerable sample workflow as the workspace
+  // root so activation events fire (workspaceContains:**/.github/workflows/*).
+  workspaceFolder: "test-fixtures/sample-workflow",
+  // Tests need the extension's commands and views registered before
+  // they run; without a sane timeout, mocha may fail before the
+  // extension finishes activating in slow CI environments.
+  mocha: {
+    ui: "bdd",
+    timeout: 20000,
+    color: true,
+  },
+});

.vscodeignore

@@ -6,10 +6,14 @@ test-fixtures/**
 scripts/**
 docs/**
 out/**
+out-test/**
 ROADMAP.md
 .gitignore
 .eslintrc.json
+.vscode-test.mjs
 tsconfig.json
+tsconfig.integration.json
+vitest.config.ts
 **/*.map
 **/*.ts
 **/tsconfig.json