fix: improve e2e test CI environment compatibility

Author: kylewalke

.cursor/rules/ts-rules.mdc

@@ -110,6 +110,7 @@ The application we are working on uses the following tech stack:
 - Include elaborate details in the body of the commit message
 - Always follow the conventional commit message format
 - Add two newlines after the commit message title
+- Do not ever commit and push without asking for consent from the user
 
 ## When to create a class
 

.github/workflows/e2e-tests.yml

@@ -59,7 +59,13 @@ jobs:
         run: npx playwright install --with-deps chromium
 
       - name: Run E2E tests
-        run: npm run test:e2e
+        run: |
+          echo "🧪 Starting e2e tests..."
+          echo "Environment variables:"
+          echo "  CI: $CI"
+          echo "  RUNNER_TEMP: $RUNNER_TEMP"
+          echo "  TMPDIR: $TMPDIR"
+          npm run test:e2e
         env:
           CI: true
           DEBUG: pw:webserver

e2e-tests/playwright.config.ts

@@ -42,7 +42,7 @@ export default defineConfig({
             '--enable-logging=stderr',
             '--log-level=0',
             '--v=1',
-            ...(process.env.DEBUG_MODE
+            ...(process.env.CI || process.env.DEBUG_MODE
               ? ['--no-sandbox', '--disable-dev-shm-usage']
               : []),
           ],

e2e-tests/test-server.js

@@ -17,7 +17,10 @@ async function startTestServer() {
       '../packages/apex-lsp-vscode-extension',
     );
     const workspacePath = process.env.CI
-      ? path.join(process.env.TMPDIR || '/tmp', 'apex-e2e-workspace')
+      ? path.join(
+          process.env.RUNNER_TEMP || process.env.TMPDIR || '/tmp',
+          'apex-e2e-workspace',
+        )
       : path.resolve(__dirname, './test-workspace');
 
     // Verify paths exist

e2e-tests/utils/setup.ts

@@ -46,7 +46,10 @@ export async function setupTestWorkspace(
   const workspacePath =
     customWorkspacePath ||
     (process.env.CI
-      ? path.join(process.env.TMPDIR || '/tmp', 'apex-e2e-workspace')
+      ? path.join(
+          process.env.RUNNER_TEMP || process.env.TMPDIR || '/tmp',
+          'apex-e2e-workspace',
+        )
       : path.resolve(__dirname, '../test-workspace'));
 
   // Ensure workspace directory exists