fix: improve e2e test CI environment compatibility

- Add --no-sandbox and --disable-dev-shm-usage Chrome flags for CI environments
- Use RUNNER_TEMP for better GitHub Actions temp directory handling
- Add better error logging and environment debugging to CI workflow
- Improve temporary directory path handling for cross-platform compatibility

Fixes GitHub Actions e2e test failures by ensuring proper browser
launch configuration and temporary file handling in CI environments.

Author: Kyle Walker

.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

packages/apex-ls/test/__mocks__/worker.mock.ts

@@ -1,4 +1,10 @@
-// Mock for worker files to avoid import.meta issues in Jest Node.js environment
+/*
+ * Copyright (c) 2025, salesforce.com, inc.
+ * All rights reserved.
+ * Licensed under the BSD 3-Clause license.
+ * For full license text, see LICENSE.txt file in the
+ * repo root or https://opensource.org/licenses/BSD-3-Clause
+ */
 export function createSimpleWebWorkerLanguageServer() {
   // Mock implementation for testing
   return Promise.resolve();

packages/apex-ls/test/communication/CoreBridge.test.ts

@@ -106,16 +106,16 @@ describe('CoreBridge', () => {
     it('should handle transport messages', () => {
       const reader = createTransportMessageReader(mockTransport);
       const mockCallback = jest.fn();
-      
+
       reader.listen(mockCallback);
-      
+
       expect(mockTransport.listen).toHaveBeenCalled();
-      
+
       // Simulate message from transport by calling the handler that was passed to listen
       const messageHandler = mockTransport.listen.mock.calls[0][0];
       const testMessage = { jsonrpc: '2.0', method: 'test' };
       messageHandler(testMessage);
-      
+
       expect(mockCallback).toHaveBeenCalledWith(testMessage);
     });
 
@@ -124,14 +124,14 @@ describe('CoreBridge', () => {
       const mockErrorCallback = jest.fn();
 
       reader.onError(mockErrorCallback);
-      
+
       expect(mockTransport.onError).toHaveBeenCalled();
-      
+
       // Simulate error from transport
       const errorHandler = mockTransport.onError.mock.calls[0][0];
       const testError = new Error('Transport error');
       errorHandler(testError);
-      
+
       expect(mockErrorCallback).toHaveBeenCalledWith(testError);
     });
   });
@@ -188,7 +188,7 @@ describe('CoreBridge', () => {
       // Simulate malformed message
       const messageHandler = mockTransport.listen.mock.calls[0][0];
       messageHandler('invalid json');
-      
+
       // Should not crash and should still call the callback
       expect(mockCallback).toHaveBeenCalledWith('invalid json');
     });
@@ -198,10 +198,10 @@ describe('CoreBridge', () => {
       const largeMessage = {
         jsonrpc: '2.0',
         method: 'test',
-        params: { data: 'x'.repeat(10000) }
+        params: { data: 'x'.repeat(10000) },
       };
 
       await expect(writer.write(largeMessage)).resolves.not.toThrow();
     });
   });
-});
+});

packages/apex-ls/test/communication/MessageTransports.test.ts

@@ -26,7 +26,6 @@ class MockWorkerScope {
   removeEventListener = jest.fn();
 }
 
-
 describe('MessageTransports', () => {
   describe('WorkerMessageTransport', () => {
     let mockWorker: MockWorker;
@@ -41,7 +40,7 @@ describe('MessageTransports', () => {
       it('should send messages to worker', async () => {
         const testMessage = { jsonrpc: '2.0', method: 'test' };
         await transport.send(testMessage);
-        
+
         expect(mockWorker.postMessage).toHaveBeenCalledWith(testMessage);
       });
 
@@ -51,30 +50,33 @@ describe('MessageTransports', () => {
         });
 
         const testMessage = { jsonrpc: '2.0', method: 'test' };
-        await expect(transport.send(testMessage)).rejects.toThrow('PostMessage failed');
+        await expect(transport.send(testMessage)).rejects.toThrow(
+          'PostMessage failed',
+        );
       });
     });
 
     describe('Message Listening', () => {
       it('should set up message listeners', () => {
         const mockCallback = jest.fn();
         const disposable = transport.listen(mockCallback);
-        
+
         expect(mockWorker.addEventListener).toHaveBeenCalledWith(
           'message',
-          expect.any(Function)
+          expect.any(Function),
         );
         expect(disposable.dispose).toBeDefined();
       });
 
       it('should handle incoming messages', () => {
         const mockCallback = jest.fn();
         transport.listen(mockCallback);
-        
+
         // Get the message handler
-        const messageHandler = mockWorker.addEventListener.mock.calls
-          .find(([event]) => event === 'message')?.[1];
-        
+        const messageHandler = mockWorker.addEventListener.mock.calls.find(
+          ([event]) => event === 'message',
+        )?.[1];
+
         if (messageHandler) {
           const testMessage = { data: { jsonrpc: '2.0', method: 'test' } };
           messageHandler(testMessage);
@@ -85,12 +87,12 @@ describe('MessageTransports', () => {
       it('should clean up listeners when disposed', () => {
         const mockCallback = jest.fn();
         const disposable = transport.listen(mockCallback);
-        
+
         disposable.dispose();
-        
+
         expect(mockWorker.removeEventListener).toHaveBeenCalledWith(
           'message',
-          expect.any(Function)
+          expect.any(Function),
         );
       });
     });
@@ -99,21 +101,22 @@ describe('MessageTransports', () => {
       it('should set up error listeners', () => {
         const mockCallback = jest.fn();
         const disposable = transport.onError(mockCallback);
-        
+
         expect(mockWorker.addEventListener).toHaveBeenCalledWith(
           'error',
-          expect.any(Function)
+          expect.any(Function),
         );
         expect(disposable.dispose).toBeDefined();
       });
 
       it('should handle worker errors', () => {
         const mockCallback = jest.fn();
         transport.onError(mockCallback);
-        
-        const errorHandler = mockWorker.addEventListener.mock.calls
-          .find(([event]) => event === 'error')?.[1];
-        
+
+        const errorHandler = mockWorker.addEventListener.mock.calls.find(
+          ([event]) => event === 'error',
+        )?.[1];
+
         if (errorHandler) {
           const errorEvent = { message: 'Worker error' };
           errorHandler(errorEvent);
@@ -136,21 +139,21 @@ describe('MessageTransports', () => {
       it('should send messages to self', async () => {
         const testMessage = { jsonrpc: '2.0', method: 'test' };
         await transport.send(testMessage);
-        
+
         expect(mockSelf.postMessage).toHaveBeenCalledWith(testMessage);
       });
 
       it('should work with global self when no scope provided', async () => {
         // Mock global self
         const globalSelf = new MockWorkerScope();
         (global as any).self = globalSelf;
-        
+
         const globalTransport = new SelfMessageTransport();
         const testMessage = { jsonrpc: '2.0', method: 'test' };
         await globalTransport.send(testMessage);
-        
+
         expect(globalSelf.postMessage).toHaveBeenCalledWith(testMessage);
-        
+
         // Cleanup
         delete (global as any).self;
       });
@@ -160,21 +163,22 @@ describe('MessageTransports', () => {
       it('should set up message listeners on self', () => {
         const mockCallback = jest.fn();
         const disposable = transport.listen(mockCallback);
-        
+
         expect(mockSelf.addEventListener).toHaveBeenCalledWith(
           'message',
-          expect.any(Function)
+          expect.any(Function),
         );
         expect(disposable.dispose).toBeDefined();
       });
 
       it('should handle messages from main thread', () => {
         const mockCallback = jest.fn();
         transport.listen(mockCallback);
-        
-        const messageHandler = mockSelf.addEventListener.mock.calls
-          .find(([event]) => event === 'message')?.[1];
-        
+
+        const messageHandler = mockSelf.addEventListener.mock.calls.find(
+          ([event]) => event === 'message',
+        )?.[1];
+
         if (messageHandler) {
           const testMessage = { data: { jsonrpc: '2.0', method: 'test' } };
           messageHandler(testMessage);
@@ -183,5 +187,4 @@ describe('MessageTransports', () => {
       });
     });
   });
-
-});
+});

packages/apex-ls/test/communication/NodeClient.test.ts

@@ -50,9 +50,11 @@ describe('Client', () => {
     });
 
     it('should initialize connection on construction', () => {
-      const { NodeMessageBridge } = require('../../src/communication/NodeBridge');
+      const {
+        NodeMessageBridge,
+      } = require('../../src/communication/NodeBridge');
       new Client({ logger: mockLogger });
-      
+
       expect(NodeMessageBridge.createConnection).toHaveBeenCalledWith({
         mode: 'stdio',
         logger: mockLogger,
@@ -85,7 +87,7 @@ describe('Client', () => {
     beforeEach(() => {
       client = new Client({ logger: mockLogger });
       // Allow connection to initialize
-      return new Promise(resolve => setTimeout(resolve, 0));
+      return new Promise((resolve) => setTimeout(resolve, 0));
     });
 
     it('should throw error when sending notification after disposal', () => {
@@ -104,21 +106,24 @@ describe('Client', () => {
     });
   });
 
-
   describe('Async Operations', () => {
     beforeEach(async () => {
       client = new Client({ logger: mockLogger });
       // Wait for connection to be established
-      await new Promise(resolve => setTimeout(resolve, 10));
+      await new Promise((resolve) => setTimeout(resolve, 10));
     });
 
     it('should handle async request operations', async () => {
       const mockResult = { result: 'test' };
       mockConnection.sendRequest.mockResolvedValue(mockResult);
 
-      const result = await client.sendRequest('test/method', { param: 'value' });
+      const result = await client.sendRequest('test/method', {
+        param: 'value',
+      });
       expect(result).toBe(mockResult);
-      expect(mockConnection.sendRequest).toHaveBeenCalledWith('test/method', { param: 'value' });
+      expect(mockConnection.sendRequest).toHaveBeenCalledWith('test/method', {
+        param: 'value',
+      });
     });
 
     it('should handle initialize request', async () => {
@@ -127,9 +132,12 @@ describe('Client', () => {
 
       const params = { processId: 1234, rootUri: '/test' };
       const result = await client.initialize(params);
-      
+
       expect(result).toBe(mockInitializeResult);
-      expect(mockConnection.sendRequest).toHaveBeenCalledWith('initialize', params);
+      expect(mockConnection.sendRequest).toHaveBeenCalledWith(
+        'initialize',
+        params,
+      );
     });
   });
-});
+});

packages/apex-ls/test/integration/split-architecture.test.ts

@@ -363,7 +363,9 @@ describe('Split Architecture Regression Tests', () => {
 
     it('should verify browser module exports correct interfaces', async () => {
       // Import BrowserStorageFactory directly from its source after removing re-exports
-      const storageModule = await import('../../src/storage/StorageImplementations');
+      const storageModule = await import(
+        '../../src/storage/StorageImplementations'
+      );
 
       // Browser storage should be available directly from StorageImplementations
       expect(storageModule.BrowserStorageFactory).toBeDefined();

packages/apex-ls/test/utils/BrowserLogNotificationHandler.test.ts

@@ -6,7 +6,7 @@
  * repo root or https://opensource.org/licenses/BSD-3-Clause
  */
 
-import { Connection, MessageType } from 'vscode-languageserver/browser';
+import { Connection } from 'vscode-languageserver/browser';
 import {
   LogMessageParams,
   type LogMessageType,