chore: wip

Author: chrisbbreuer

packages/launchpad/src/dev/dump.ts

@@ -380,7 +380,17 @@ export async function dump(dir: string, options: DumpOptions = {}): Promise<void
 
     // Parse dependency file and separate global vs local dependencies
     const { default: sniff } = await import('./sniff')
-    const sniffResult = await sniff({ string: projectDir })
+    let sniffResult: { pkgs: any[], env: Record<string, string> }
+
+    try {
+      sniffResult = await sniff({ string: projectDir })
+    } catch (error) {
+      // Handle malformed dependency files gracefully
+      if (config.verbose) {
+        console.warn(`Failed to parse dependency file: ${error instanceof Error ? error.message : String(error)}`)
+      }
+      sniffResult = { pkgs: [], env: {} }
+    }
 
     // Only check for global dependencies when not in shell mode or when global env doesn't exist, and not skipping global
     const globalSniffResults: Array<{ pkgs: any[], env: Record<string, string> }> = []

packages/launchpad/test/dev.test.ts

@@ -645,8 +645,9 @@ describe('Dev Commands', () => {
 
     it('should prioritize local environment over global and system', async () => {
       // Create a fake local environment
-      const projectHash = 'test-project-hash'
-      const localEnvDir = path.join(os.homedir(), '.local', 'share', 'launchpad', `${path.basename(tempDir)}_${projectHash}`)
+      const crypto = require('node:crypto')
+      const projectHash = `${path.basename(tempDir)}_${crypto.createHash('md5').update(tempDir).digest('hex').slice(0, 8)}`
+      const localEnvDir = path.join(os.homedir(), '.local', 'share', 'launchpad', projectHash)
       const localBinDir = path.join(localEnvDir, 'bin')
       const localPkgsDir = path.join(localEnvDir, 'pkgs', 'bun.sh', 'v1.2.18')
 

packages/launchpad/test/library-paths.test.ts

@@ -60,6 +60,18 @@ describe('Library Path Management', () => {
       fs.rmSync(tempDir, { recursive: true, force: true })
     }
 
+    // Clean up test environment directories
+    const launchpadEnvsDir = path.join(process.env.HOME || '', '.local', 'share', 'launchpad')
+    if (fs.existsSync(launchpadEnvsDir)) {
+      const entries = fs.readdirSync(launchpadEnvsDir)
+      for (const entry of entries) {
+        if (entry.startsWith('test-project_')) {
+          const entryPath = path.join(launchpadEnvsDir, entry)
+          fs.rmSync(entryPath, { recursive: true, force: true })
+        }
+      }
+    }
+
     // Restore original fetch
     globalThis.fetch = originalFetch
   })
@@ -143,15 +155,20 @@ describe('Library Path Management', () => {
 
   describe('Shell Environment Setup', () => {
     it('should include library paths in shell output', async () => {
-      // Create mock packages with library directories
-      const { libDir: nodeLibDir } = createMockPackageWithLibs('nodejs.org', '20.0.0', testInstallPath)
-      const { libDir: zlibLibDir } = createMockPackageWithLibs('zlib.net', '1.3.1', testInstallPath)
-
-      // Create project directory with dependencies
+      // Create project directory with dependencies first
       const projectDir = path.join(tempDir, 'test-project')
       fs.mkdirSync(projectDir, { recursive: true })
       createDepsYaml(projectDir, ['nodejs.org@20', 'zlib.net@1.3'])
 
+      // Calculate the correct environment directory where dev command will look
+      const crypto = require('node:crypto')
+      const projectHash = `test-project_${crypto.createHash('md5').update(projectDir).digest('hex').slice(0, 8)}`
+      const envDir = path.join(process.env.HOME || '', '.local', 'share', 'launchpad', projectHash)
+
+      // Create mock packages with library directories in the correct location
+      const { libDir: nodeLibDir } = createMockPackageWithLibs('nodejs.org', '20.0.0', envDir)
+      const { libDir: zlibLibDir } = createMockPackageWithLibs('zlib.net', '1.3.1', envDir)
+
       // Run dev command with shell output
       const result = await runCLI(['dev', projectDir, '--shell'])
 

packages/launchpad/test/smart-constraint-checking.test.ts

@@ -9,8 +9,11 @@ import { dump } from '../src/dev/dump'
 describe('Smart Constraint Checking', () => {
   const testBaseDir = path.join(homedir(), '.local', 'share', 'launchpad-test')
   const testProjectDir = path.join(testBaseDir, 'test-project')
-  const testLocalEnvDir = path.join(testBaseDir, 'test-project_12345678')
-  const testGlobalEnvDir = path.join(testBaseDir, 'global')
+  // Calculate the actual project hash that would be generated
+  const crypto = require('node:crypto')
+  const testProjectHash = `test-project_${crypto.createHash('md5').update(testProjectDir).digest('hex').slice(0, 8)}`
+  const testLocalEnvDir = path.join(homedir(), '.local', 'share', 'launchpad', testProjectHash)
+  const testGlobalEnvDir = path.join(homedir(), '.local', 'share', 'launchpad', 'global')
 
   beforeAll(async () => {
     // Clean up any existing test directories