test: add electron launch smoke test

.github/workflows/test-electron.yml

@@ -0,0 +1,62 @@
+name: Test Electron Launch
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: npm
+          cache-dependency-path: |
+            package-lock.json
+            web/package-lock.json
+            native/package-lock.json
+
+      - name: Install dependencies
+        run: |
+          npm ci --ignore-scripts
+          cd web && npm ci
+
+      - name: Setup Rust toolchain
+        uses: dtolnay/rust-toolchain@stable
+
+      - name: Install native addon dependencies
+        working-directory: native
+        run: npm ci
+
+      - name: Build native addon
+        working-directory: native
+        run: npm run build
+
+      - name: Build web + tsc
+        run: npm run build
+
+      - name: Install Playwright browsers
+        run: npx playwright install --with-deps chromium
+
+      - name: Bundle Electron
+        working-directory: packages/electron
+        run: node electron/build.mjs
+
+      - name: Prepare pack (copy resources)
+        working-directory: packages/electron
+        run: node electron/prepare-pack.mjs
+
+      - name: Test electron-builder
+        working-directory: packages/electron
+        run: npx electron-builder --config electron-builder.yml --linux --dir --publish never
+
+      - name: Run Playwright smoke test
+        run: xvfb-run -a npx vitest run packages/electron/__tests__/launch.test.ts
+        env:
+          CI: true

packages/electron/__tests__/launch.test.ts

@@ -0,0 +1,66 @@
+import { describe, it, expect, beforeAll, afterAll } from 'vitest';
+import { _electron as electron, ElectronApplication } from 'playwright';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const rootDir = path.resolve(__dirname, '..');
+
+describe.runIf(process.env.CI || process.env.RUN_E2E_SMOKE)('Electron Launch Smoke Test', () => {
+  let app: ElectronApplication;
+
+  beforeAll(async () => {
+    // We pass an empty string to use the built executable
+    let executablePath = '';
+    if (process.platform === 'win32') {
+      executablePath = path.join(rootDir, 'release', 'win-unpacked', 'Codex Proxy.exe');
+    } else if (process.platform === 'darwin') {
+      executablePath = path.join(rootDir, 'release', 'mac', 'Codex Proxy.app', 'Contents', 'MacOS', 'Codex Proxy');
+    } else {
+      executablePath = path.join(rootDir, 'release', 'linux-unpacked', '@codex-proxyelectron');
+    }
+
+    // We need to bypass native transport errors by either disabling or supplying curl-cli in local.yaml
+    const userDataDir = path.join(rootDir, 'release', '.test-user-data');
+
+    app = await electron.launch({
+      executablePath,
+      args: [
+        '--no-sandbox',
+        '--disable-gpu',
+        '--disable-dev-shm-usage',
+        `--user-data-dir=${userDataDir}`,
+      ],
+      env: {
+        ...process.env,
+        NODE_ENV: 'test',
+        DISABLE_NATIVE_TRANSPORT: '1',
+        ELECTRON_ENABLE_LOGGING: '1', // Important for seeing errors
+      },
+    });
+
+    app.process().stdout?.on('data', (data) => console.log(`stdout: ${data.toString()}`));
+    app.process().stderr?.on('data', (data) => console.error(`stderr: ${data.toString()}`));
+  }, 60000); // Allow time to launch
+
+  afterAll(async () => {
+    if (app) {
+      await app.close();
+    }
+  }, 20000);
+
+  it('launches successfully and opens the main window', async () => {
+    const window = await app.firstWindow();
+    expect(window).toBeTruthy();
+
+    // Check if the window is visible and not crashed
+    const isClosed = window.isClosed();
+    expect(isClosed).toBe(false);
+
+    // Optionally wait for some title or element to be sure the frontend is loaded,
+    // but simply verifying the window isn't crashed satisfies the smoke test requirements.
+    const title = await window.title();
+    expect(title).toBeTypeOf('string'); // The app has some title
+  }, 30000);
+});

packages/electron/package.json

@@ -19,8 +19,10 @@
     "electron-updater": "^6.3.0"
   },
   "devDependencies": {
+    "@playwright/test": "^1.59.1",
     "electron": "^35.7.5",
     "electron-builder": "^26.0.0",
-    "esbuild": "^0.25.12"
+    "esbuild": "^0.25.12",
+    "playwright": "^1.59.1"
   }
 }