chore: wip

Author: chrisbbreuer

packages/launchpad/src/binary-downloader.ts

@@ -800,7 +800,11 @@ Thanks for helping us make Launchpad better! 🙏
 
         // First, on macOS fix absolute Homebrew dylib references to Launchpad-managed libs
         try {
-          await this.fixMacOSDylibs(packageDir, launchpadLibraryPaths)
+          // Best-effort only; if this fails due to headerpad limits, skip silently (shims set DYLD paths)
+          try {
+            await this.fixMacOSDylibs(packageDir, launchpadLibraryPaths)
+          }
+          catch {}
         }
         catch (err) {
           console.warn(`⚠️ Could not fix macOS dylib references: ${err instanceof Error ? err.message : String(err)}`)

packages/launchpad/src/dev/dump.ts

@@ -285,6 +285,30 @@ async function executepostSetup(projectDir: string, commands: PostSetupCommand[]
       }
       // Small grace delay after ready
       await new Promise(r => setTimeout(r, 250))
+
+      // If still not listening, wait a bit longer with additional probes instead of restarting
+      const readyAfterFirstPass = await new Promise<boolean>((resolve) => {
+        // eslint-disable-next-line ts/no-require-imports
+        const { spawn } = require('node:child_process')
+        const p = spawn(pgIsReady, ['-h', host, '-p', port], { stdio: 'ignore' })
+        p.on('close', (code: number) => resolve(code === 0))
+        p.on('error', () => resolve(false))
+      })
+      if (!readyAfterFirstPass) {
+        for (let i = 0; i < 12; i++) {
+          const ok = await new Promise<boolean>((resolve) => {
+            // eslint-disable-next-line ts/no-require-imports
+            const { spawn } = require('node:child_process')
+            const p = spawn(pgIsReady, ['-h', host, '-p', port], { stdio: 'ignore' })
+            p.on('close', (code: number) => resolve(code === 0))
+            p.on('error', () => resolve(false))
+          })
+          if (ok)
+            break
+          await new Promise(r => setTimeout(r, Math.min(750 + i * 250, 3000)))
+        }
+        await new Promise(r => setTimeout(r, 250))
+      }
     }
     catch {}
   }
@@ -815,10 +839,20 @@ export async function dump(dir: string, options: DumpOptions = {}): Promise<void
     // Regular path for non-shell integration calls
     if (localPackages.length > 0 || globalPackages.length > 0) {
       await installPackagesOptimized(localPackages, globalPackages, envDir, globalEnvDir, dryrun, quiet)
+      // Visual separator after dependency install list
+      try { console.log() }
+      catch {}
     }
 
     // Auto-start services for any project that has services configuration
+    // Suppress interstitial processing messages during service startup phase
+    const prevProcessing = process.env.LAUNCHPAD_PROCESSING
+    process.env.LAUNCHPAD_PROCESSING = '0'
     await setupProjectServices(projectDir, sniffResult, !effectiveQuiet)
+    if (prevProcessing === undefined)
+      delete process.env.LAUNCHPAD_PROCESSING
+    else
+      process.env.LAUNCHPAD_PROCESSING = prevProcessing
 
     // Ensure php.ini and Laravel post-setup runs (regular path)
     await ensureProjectPhpIni(projectDir, envDir)
@@ -845,9 +879,12 @@ export async function dump(dir: string, options: DumpOptions = {}): Promise<void
 
       outputShellCode(dir, envBinPath, envSbinPath, projectHash, sniffResult, globalBinPath, globalSbinPath)
     }
-    else if (!effectiveQuiet) {
-      // Final success summary for non-shell runs (used by tests)
-      console.log('Successfully set up environment')
+    else {
+      // Always print a final activation message, even in quiet mode
+      const { config } = await import('../config')
+      const activation = (config.shellActivationMessage || '✅ Environment activated for {path}')
+        .replace('{path}', process.cwd())
+      console.log(activation)
     }
   }
   catch (error) {
@@ -1384,9 +1421,8 @@ async function setupProjectServices(projectDir: string, sniffResult: any, showMe
 
           // Special handling for PostgreSQL: wait for readiness and create project database
           if ((serviceName === 'postgres' || serviceName === 'postgresql') && hasPostgresInDeps) {
-            if (showMessages) {
+            if (showMessages)
               console.log('⏳ Verifying PostgreSQL readiness...')
-            }
             // Ensure postgres is actually accepting connections
             try {
               const { findBinaryInPath } = await import('../utils')
@@ -1407,9 +1443,8 @@ async function setupProjectServices(projectDir: string, sniffResult: any, showMe
             }
             catch {}
 
-            if (showMessages) {
+            if (showMessages)
               console.log('🔧 Creating project PostgreSQL database...')
-            }
             const projectName = path.basename(projectDir).replace(/\W/g, '_')
             try {
               await createProjectDatabase(projectName, {

packages/launchpad/src/logging.ts

@@ -127,35 +127,6 @@ export function logUniqueMessage(message: string, forceLog = false): void {
     console.log(message)
   }
 
-  // Show temporary processing message immediately after package success messages (not the final environment message)
-  // Use a simple static message instead of animated spinner to avoid conflicts with download progress
-  // But don't show it for the final package completion or summary messages
-  if (!config.verbose && message.startsWith('✅')
-    && !message.includes('Environment activated')
-    && !message.includes('Successfully set up environment')
-    && !message.includes('Installed')
-    && !message.includes('packages')
-    && !message.includes('(v')) { // Don't show processing message after individual package success messages
-    // Add a small delay to make the success message visible before showing processing message
-    setTimeout(() => {
-      // Only show processing message if we haven't completed all packages
-      if (!hasTemporaryProcessingMessage) {
-        const processingMsg = `🔄 Processing next dependency...`
-
-        if (process.env.LAUNCHPAD_SHELL_INTEGRATION === '1') {
-          process.stderr.write(`${processingMsg}\n`)
-          if (process.stderr.isTTY) {
-            fs.writeSync(process.stderr.fd, '')
-          }
-        }
-        else {
-          process.stdout.write(`${processingMsg}\n`)
-        }
-
-        hasTemporaryProcessingMessage = true
-      }
-    }, 50) // Small delay to ensure success message is visible
-  }
 }
 
 export function clearMessageCache(): void {

packages/launchpad/src/services/manager.ts

@@ -110,14 +110,27 @@ export async function startService(serviceName: string): Promise<boolean> {
     const service = await getOrCreateServiceInstance(serviceName)
 
     if (service.status === 'running') {
-      console.log(`✅ Service ${serviceName} is already running`)
-      operation.result = 'success'
-      operation.duration = 0
-      manager.operations.push(operation)
-      return true
+      // Verify actual health with a couple retries to avoid false negatives
+      let healthy = await checkServiceHealth(service)
+      if (!healthy) {
+        await new Promise(r => setTimeout(r, 500))
+        healthy = await checkServiceHealth(service)
+      }
+      if (healthy) {
+        logUniqueMessage(`✅ Service ${serviceName} is already running`, true)
+        operation.result = 'success'
+        operation.duration = 0
+        manager.operations.push(operation)
+        return true
+      }
+      else {
+        logUniqueMessage(`⚠️  ${service.definition?.displayName || serviceName} reported running but appears unhealthy. Attempting gentle restart...`, true)
+        await stopService(serviceName)
+        // fall through to start flow
+      }
     }
 
-    console.warn(`🚀 Starting ${service.definition?.displayName || serviceName}...`)
+    logUniqueMessage(`🚀 Starting ${service.definition?.displayName || serviceName}...`, true)
 
     // Update status to starting
     service.status = 'starting'
@@ -171,10 +184,18 @@ export async function startService(serviceName: string): Promise<boolean> {
       service.startedAt = new Date()
       service.pid = await getServicePid(service)
 
-      console.log(`✅ ${service.definition?.displayName || serviceName} started successfully`)
+      logUniqueMessage(`✅ ${service.definition?.displayName || serviceName} started successfully`, true)
 
       // Execute post-start setup commands
       await executePostStartCommands(service)
+      // For databases, perform a final readiness handshake to avoid races with immediate consumers
+      if (service.definition?.name === 'postgres' && process.platform === 'darwin') {
+        try {
+          // quick extra wait to ensure socket/listen transition is complete
+          await new Promise(r => setTimeout(r, 300))
+        }
+        catch {}
+      }
 
       // Health check after starting
       setTimeout(() => {
@@ -1167,7 +1188,7 @@ async function executePostStartCommands(service: ServiceInstance): Promise<void>
     }
 
     // Announce post-start setup phase for databases
-    logUniqueMessage(`🔧 Running ${definition.displayName} post-start setup...`, true)
+    logUniqueMessage(`🔧 ${definition.displayName} post-start setup...`, true)
   }
   else {
     // For other services, use the original wait time