chore: wip

Author: chrisbbreuer

test/plugin-architecture.test.ts

@@ -10,15 +10,17 @@ describe('Integration Ecosystem & Plugin Architecture', () => {
   let mockContext: SetupContext
   let originalEnv: Record<string, string | undefined>
   let originalCwd: string
+  let tempDir: string
 
   beforeEach(() => {
     // Store original working directory
     originalCwd = process.cwd()
-    
+
     // Change to a temporary directory to avoid interference with project files
-    const tempDir = fs.mkdtempSync(path.join(require('node:os').tmpdir(), 'plugin-test-'))
+    // eslint-disable-next-line ts/no-require-imports
+    tempDir = fs.mkdtempSync(path.join(require('node:os').tmpdir(), 'plugin-test-'))
     process.chdir(tempDir)
-    
+
     pluginManager = new PluginManager()
     mockContext = {
       step: 'setup_complete',
@@ -93,13 +95,13 @@ describe('Integration Ecosystem & Plugin Architecture', () => {
         fs.rmSync(file, { force: true })
       }
     })
-    
+
     // Restore original working directory and clean up temp directory
-    const tempDir = process.cwd()
     process.chdir(originalCwd)
     try {
       fs.rmSync(tempDir, { recursive: true, force: true })
-    } catch {
+    }
+    catch {
       // Ignore cleanup errors
     }
 
@@ -266,7 +268,20 @@ describe('Integration Ecosystem & Plugin Architecture', () => {
         }
 
         const customPluginPath = path.join('.buddy/plugins', 'custom.json')
-        fs.writeFileSync(customPluginPath, JSON.stringify(customPlugin))
+
+        // Ensure no composer.json exists in the test directory
+        const composerPath = path.join(process.cwd(), 'composer.json')
+        if (fs.existsSync(composerPath)) {
+          fs.rmSync(composerPath, { force: true })
+        }
+
+        // Write the file and ensure it's synced
+        const pluginContent = JSON.stringify(customPlugin)
+        fs.writeFileSync(customPluginPath, pluginContent)
+        fs.fsyncSync(fs.openSync(customPluginPath, 'r'))
+
+        // Small delay to ensure file system operations complete
+        await new Promise(resolve => setTimeout(resolve, 10))
 
         // Verify the file was written correctly
         expect(fs.existsSync(customPluginPath)).toBe(true)

test/respect-latest.test.ts

@@ -14,6 +14,13 @@ describe('respectLatest functionality', () => {
     // Create temporary test directory in a more isolated location
     // eslint-disable-next-line ts/no-require-imports
     testDir = fs.mkdtempSync(path.join(require('node:os').tmpdir(), 'buddy-test-'))
+
+    // Ensure the directory is truly isolated by checking it's empty
+    const files = fs.readdirSync(testDir)
+    if (files.length > 0) {
+      console.log('🚨 Test directory not empty:', files)
+      files.forEach(file => fs.rmSync(path.join(testDir, file), { recursive: true, force: true }))
+    }
   })
 
   afterEach(() => {
@@ -87,7 +94,19 @@ devDependencies:
 `
 
     const depsPath = path.join(testDir, 'deps.yaml')
+
+    // Ensure no composer.json exists in the test directory
+    const composerPath = path.join(testDir, 'composer.json')
+    if (fs.existsSync(composerPath)) {
+      fs.rmSync(composerPath, { force: true })
+    }
+
+    // Write the file and ensure it's synced
     fs.writeFileSync(depsPath, depsYaml)
+    fs.fsyncSync(fs.openSync(depsPath, 'r'))
+
+    // Small delay to ensure file system operations complete
+    await new Promise(resolve => setTimeout(resolve, 10))
 
     // Ensure the file was written correctly and verify content
     const actualFileContent = fs.readFileSync(depsPath, 'utf-8')