chore: wip

Author: chrisbbreuer

packages/launchpad/src/services/manager.ts

@@ -708,8 +708,10 @@ async function executePostStartCommands(service: ServiceInstance): Promise<void>
 
   // Wait for the service to be fully ready, especially for databases
   if (definition.name === 'postgres' || definition.name === 'mysql') {
-    // For databases, wait longer and check if they're actually ready
-    await new Promise(resolve => setTimeout(resolve, 5000))
+    // For databases, wait longer in CI environments and check if they're actually ready
+    const isCI = process.env.CI === 'true' || process.env.GITHUB_ACTIONS === 'true'
+    const waitTime = isCI ? 10000 : 5000 // 10s in CI, 5s locally
+    await new Promise(resolve => setTimeout(resolve, waitTime))
 
     // Try to verify the service is responding before running post-start commands
     if (definition.healthCheck) {

packages/launchpad/test/environment-isolation.test.ts

@@ -65,13 +65,13 @@ describe('Environment Isolation', () => {
       '/usr/local/bin',
     ].filter(Boolean)
 
-    // Add any missing bun paths to PATH
-    let enhancedPath = currentPath
-    for (const bunPath of bunPaths) {
-      if (!enhancedPath.includes(bunPath)) {
-        enhancedPath = `${bunPath}:${enhancedPath}`
+          // Add any missing bun paths to PATH
+      let enhancedPath = currentPath
+      for (const bunPath of bunPaths) {
+        if (bunPath && !enhancedPath.includes(bunPath)) {
+          enhancedPath = `${bunPath}:${enhancedPath}`
+        }
       }
-    }
 
     return {
       ...process.env,

packages/launchpad/test/services.test.ts

@@ -452,7 +452,7 @@ describe('Service Management', () => {
       const stopResults = await Promise.all(stopPromises)
 
       stopResults.forEach(result => expect(result).toBe(true))
-    }, 10000) // Set explicit 10-second timeout for this test
+    }, 30000) // Set explicit 30-second timeout for GitHub Actions
 
     it('should prevent starting already running service', async () => {
       if (!isPlatformSupported()) {
@@ -570,7 +570,7 @@ describe('Service Management', () => {
           expect((error as Error).message).toContain('not supported')
         }
       }
-    }, 10000) // Set explicit 10-second timeout for this test
+    }, 30000) // Set explicit 30-second timeout for GitHub Actions
 
     it('should handle service definition not found', async () => {
       // The service manager catches errors and returns false instead of throwing