Pin electron to 35.2.1 and split CI test jobs

- Pin electron to exact version to stop Dependabot churn; CI
  tests minimum (Electron 32) and latest via npx separately
- Split CI into extension tests (Electron matrix) and webview
  tests (own job) since Electron's patched fs.readFileSync
  crashes vitest's forks pool with jsdom
- Add version mapping reference (ewanharris/vscode-versions)

Author: EhabY

.github/workflows/ci.yaml

@@ -34,13 +34,14 @@ jobs:
 
       - run: pnpm build
 
-  test-unit:
-    name: Unit Test (Electron ${{ matrix.electron-version }})
+  test-extension:
+    name: Extension Test (Electron ${{ matrix.electron-version }})
     runs-on: ubuntu-22.04
     strategy:
       fail-fast: false
       matrix:
-        # Minimum supported version: VS Code 1.95  (Oct 2024) -> Electron 32 -> Node 20
+        # Minimum supported version: VS Code 1.95 (Oct 2024) -> Electron 32 -> Node 20
+        # See https://github.com/ewanharris/vscode-versions for version mapping
         electron-version: ["32", "latest"]
 
     steps:
@@ -55,11 +56,32 @@ jobs:
 
       - run: pnpm install
 
-      - name: Run tests with Electron ${{ matrix.electron-version }}
+      - name: Run extension tests with Electron ${{ matrix.electron-version }}
         run: ./scripts/test-electron.sh ${{ matrix.electron-version }}
         env:
           CI: true
 
+  test-webview:
+    name: Webview Test
+    runs-on: ubuntu-22.04
+
+    steps:
+      - uses: actions/checkout@v6
+
+      - uses: pnpm/action-setup@v5
+
+      - uses: actions/setup-node@v6
+        with:
+          node-version: "22"
+          cache: "pnpm"
+
+      - run: pnpm install
+
+      - name: Run webview tests
+        run: pnpm test:webview
+        env:
+          CI: true
+
   test-integration:
     name: Integration Test (VS Code ${{ matrix.vscode-version }})
     runs-on: ubuntu-22.04
@@ -88,7 +110,7 @@ jobs:
   package:
     name: Package
     runs-on: ubuntu-22.04
-    needs: [lint, test-unit, test-integration]
+    needs: [lint, test-extension, test-webview, test-integration]
     steps:
       - uses: actions/checkout@v6
 

.vscode-test.mjs

@@ -1,8 +1,9 @@
 import { defineConfig } from "@vscode/test-cli";
 
 // VS Code to Electron/Node version mapping:
-//   VS Code 1.95  (Oct 2024) -> Node 20 - Minimum supported
+//   VS Code 1.95  (Oct 2024) -> Electron 32, Node 20 - Minimum supported
 //   VS Code stable           -> Latest
+// See https://github.com/ewanharris/vscode-versions for version mapping
 const versions = ["1.95.0", "stable"];
 
 const baseConfig = {

package.json

@@ -549,7 +549,7 @@
 		"coder": "catalog:",
 		"concurrently": "^9.2.1",
 		"dayjs": "^1.11.20",
-		"electron": "^41.0.3",
+		"electron": "35.2.1",
 		"esbuild": "^0.27.4",
 		"eslint": "^10.0.3",
 		"eslint-config-prettier": "^10.1.8",

pnpm-lock.yaml

@@ -1,7 +1,11 @@
 #!/bin/bash
-# Run tests inside a specific Electron version's runtime.
+# Run extension unit tests inside a specific Electron version's runtime.
 #
-# Requires Electron 32+ due to Vitest ESM requirements.
+# Only runs the "extension" project because Electron's patched fs.readFileSync
+# causes stack overflows in vitest's forks pool with jsdom (webview tests).
+# Webview tests should be run separately via `pnpm test:webview`.
+#
+# See https://github.com/ewanharris/vscode-versions for version mapping.
 #
 # Usage: ./scripts/test-electron.sh <electron-version>
 # Examples:
@@ -12,7 +16,7 @@ set -e
 
 ELECTRON_VERSION="${1:?Usage: $0 <electron-version>}"
 
-echo "Running tests with Electron $ELECTRON_VERSION..."
-# --experimental-require-module needed for Electron 32-33 (Node 20.18), harmless for 34+
-ELECTRON_RUN_AS_NODE=1 NODE_OPTIONS="--experimental-require-module" \
-  npx --yes "electron@$ELECTRON_VERSION" node_modules/vitest/vitest.mjs
+echo "Running extension tests with Electron $ELECTRON_VERSION..."
+# --experimental-require-module needed for Electron 32-34 (Node 20.18), harmless for 35+ (Node 22+)
+ELECTRON_RUN_AS_NODE=1 NODE_OPTIONS="--experimental-require-module --disable-warning=ExperimentalWarning" \
+  npx --yes "electron@$ELECTRON_VERSION" node_modules/vitest/vitest.mjs --project extension