sofia-cli 0.1.2 → 0.1.4

package/tests/integration/ralphLoopFlow.spec.ts

@@ -12,7 +12,7 @@ import { tmpdir } from 'node:os';
 import { createRequire } from 'node:module';
 
 import { RalphLoop } from '../../src/develop/ralphLoop.js';
-import { PocScaffolder } from '../../src/develop/pocScaffolder.js';
+import { generateDynamicScaffold } from '../../src/develop/dynamicScaffolder.js';
 import { TestRunner } from '../../src/develop/testRunner.js';
 import type { WorkshopSession } from '../../src/shared/schemas/session.js';
 import type { LoopIO } from '../../src/loop/conversationLoop.js';
@@ -42,14 +42,19 @@ vi.mock('node:child_process', async (importOriginal) => {
 });
 
 // Mock validatePocOutput to always pass in integration tests
-vi.mock('../../src/develop/pocScaffolder.js', async (importOriginal) => {
-  const actual = await importOriginal<typeof import('../../src/develop/pocScaffolder.js')>();
+vi.mock('../../src/develop/pocUtils.js', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('../../src/develop/pocUtils.js')>();
   return {
     ...actual,
     validatePocOutput: vi.fn().mockResolvedValue({ valid: true, missingFiles: [], errors: [] }),
   };
 });
 
+// Mock generateDynamicScaffold to create minimal files for tests
+vi.mock('../../src/develop/dynamicScaffolder.js', () => ({
+  generateDynamicScaffold: vi.fn(),
+}));
+
 const require = createRequire(import.meta.url);
 const fixtureSession: WorkshopSession =
   require('../fixtures/completedSession.json') as WorkshopSession;
@@ -68,48 +73,36 @@ function makeIo(): LoopIO {
   };
 }
 
-function makeFakeScaffolder(outputDir: string): PocScaffolder {
-  return {
-    scaffold: vi.fn().mockImplementation(async () => {
-      const { writeFile, mkdir } = await import('node:fs/promises');
-      await mkdir(join(outputDir, 'src'), { recursive: true });
-      await mkdir(join(outputDir, 'tests'), { recursive: true });
-      await writeFile(
-        join(outputDir, 'package.json'),
-        JSON.stringify({
-          name: 'route-optimizer-poc',
-          scripts: { test: 'vitest run' },
-          dependencies: {},
-          devDependencies: { vitest: '^3.0.0' },
-        }),
-        'utf-8',
-      );
-      await writeFile(
-        join(outputDir, 'src', 'index.ts'),
-        '// TODO: implement\nexport function optimize() { return []; }',
-        'utf-8',
-      );
-      await writeFile(
-        join(outputDir, 'tests', 'index.test.ts'),
-        'import { describe, it, expect } from "vitest";\nimport { optimize } from "../src/index.js";\ndescribe("optimizer", () => { it("should return stops", () => { expect(optimize().length).toBeGreaterThan(0); }); });',
-        'utf-8',
-      );
-      return {
-        createdFiles: ['package.json', 'src/index.ts', 'tests/index.test.ts'],
-        skippedFiles: [],
-        context: {
-          projectName: 'route-optimizer-poc',
-          ideaTitle: 'AI-Powered Route Optimizer',
-          ideaDescription: 'Optimize routes',
-          techStack: { language: 'TypeScript', runtime: 'Node.js 20', testRunner: 'npm test' },
-          planSummary: 'Route optimization',
-          sessionId: fixtureSession.sessionId,
-          outputDir,
-        },
-      };
-    }),
-    getTemplateFiles: () => ['package.json', 'src/index.ts', 'tests/index.test.ts'],
-  } as unknown as PocScaffolder;
+function setupDynamicScaffoldMock(outputDir: string): void {
+  vi.mocked(generateDynamicScaffold).mockImplementation(async () => {
+    const { writeFile, mkdir } = await import('node:fs/promises');
+    await mkdir(join(outputDir, 'src'), { recursive: true });
+    await mkdir(join(outputDir, 'tests'), { recursive: true });
+    await writeFile(
+      join(outputDir, 'package.json'),
+      JSON.stringify({
+        name: 'route-optimizer-poc',
+        scripts: { test: 'vitest run' },
+        dependencies: {},
+        devDependencies: { vitest: '^3.0.0' },
+      }),
+      'utf-8',
+    );
+    await writeFile(
+      join(outputDir, 'src', 'index.ts'),
+      '// TODO: implement\nexport function optimize() { return []; }',
+      'utf-8',
+    );
+    await writeFile(
+      join(outputDir, 'tests', 'index.test.ts'),
+      'import { describe, it, expect } from "vitest";\nimport { optimize } from "../src/index.js";\ndescribe("optimizer", () => { it("should return stops", () => { expect(optimize().length).toBeGreaterThan(0); }); });',
+      'utf-8',
+    );
+    return {
+      createdFiles: ['package.json', 'src/index.ts', 'tests/index.test.ts'],
+      techStack: { language: 'TypeScript', runtime: 'Node.js 20', testRunner: 'npm test' },
+    };
+  });
 }
 
 // ── SC-002-003: Iterative refinement test ────────────────────────────────────
@@ -128,7 +121,7 @@ describe('RalphLoop integration — iterative refinement (SC-002-003)', () => {
 
   it('scaffold → fail tests → LLM fix → tests pass → success', async () => {
     const io = makeIo();
-    const scaffolder = makeFakeScaffolder(tmpDir);
+    setupDynamicScaffoldMock(tmpDir);
 
     // Test runner: fails first, passes second
     let testCallCount = 0;
@@ -200,7 +193,6 @@ describe('RalphLoop integration — iterative refinement (SC-002-003)', () => {
       outputDir: tmpDir,
       maxIterations: 5,
       testRunner,
-      scaffolder,
       onSessionUpdate: async (session) => {
         sessionUpdates.push({ ...session });
       },
@@ -231,7 +223,7 @@ describe('RalphLoop integration — iterative refinement (SC-002-003)', () => {
 
   it('verifies failing tests are passed to LLM in iteration prompt (SC-002-003)', async () => {
     const io = makeIo();
-    const scaffolder = makeFakeScaffolder(tmpDir);
+    setupDynamicScaffoldMock(tmpDir);
 
     let testCallCount = 0;
     const testRunner: TestRunner = {
@@ -287,7 +279,6 @@ describe('RalphLoop integration — iterative refinement (SC-002-003)', () => {
       outputDir: tmpDir,
       maxIterations: 3,
       testRunner,
-      scaffolder,
     });
 
     await ralph.run();
@@ -314,9 +305,8 @@ describe('TODO tracking integration (T074)', () => {
   it('writes TODO counts to .sofia-metadata.json during scaffold and updates after iteration', async () => {
     const { writeFile, mkdir } = await import('node:fs/promises');
 
-    // Create a scaffolder that writes files with TODO markers
-    const todoScaffolder: PocScaffolder = {
-      scaffold: vi.fn().mockImplementation(async () => {
+    function setupTodoScaffoldMock(): void {
+      vi.mocked(generateDynamicScaffold).mockImplementation(async () => {
         await mkdir(join(tmpDir, 'src'), { recursive: true });
         await mkdir(join(tmpDir, 'tests'), { recursive: true });
         await writeFile(
@@ -344,20 +334,11 @@ describe('TODO tracking integration (T074)', () => {
         );
         return {
           createdFiles: ['package.json', 'src/index.ts', '.sofia-metadata.json'],
-          skippedFiles: [],
-          context: {
-            projectName: 'todo-test-poc',
-            ideaTitle: 'Test',
-            ideaDescription: 'Test',
-            techStack: { language: 'TypeScript', runtime: 'Node.js 20', testRunner: 'npm test' },
-            planSummary: 'Test',
-            sessionId: fixtureSession.sessionId,
-            outputDir: tmpDir,
-          },
+          techStack: { language: 'TypeScript', runtime: 'Node.js 20', testRunner: 'npm test' },
         };
-      }),
-      getTemplateFiles: () => ['package.json', 'src/index.ts'],
-    } as unknown as PocScaffolder;
+      });
+    }
+    setupTodoScaffoldMock();
 
     // Test runner that fails on first call (triggering TODO rescan), then passes
     let runCount = 0;
@@ -412,7 +393,6 @@ describe('TODO tracking integration (T074)', () => {
       outputDir: tmpDir,
       maxIterations: 3,
       testRunner: failThenPassRunner,
-      scaffolder: todoScaffolder,
     });
 
     await ralph.run();

package/tests/integration/ralphLoopPartial.spec.ts

@@ -13,7 +13,7 @@ import { tmpdir } from 'node:os';
 import { createRequire } from 'node:module';
 
 import { RalphLoop } from '../../src/develop/ralphLoop.js';
-import { PocScaffolder } from '../../src/develop/pocScaffolder.js';
+import { generateDynamicScaffold } from '../../src/develop/dynamicScaffolder.js';
 import { TestRunner } from '../../src/develop/testRunner.js';
 import type { WorkshopSession } from '../../src/shared/schemas/session.js';
 import type { LoopIO } from '../../src/loop/conversationLoop.js';
@@ -42,8 +42,14 @@ vi.mock('node:child_process', async (importOriginal) => {
   };
 });
 
+// Mock generateDynamicScaffold
+vi.mock('../../src/develop/dynamicScaffolder.js', () => ({
+  generateDynamicScaffold: vi.fn(),
+}));
+
 const require = createRequire(import.meta.url);
-const fixtureSession: WorkshopSession = require('../fixtures/completedSession.json') as WorkshopSession;
+const fixtureSession: WorkshopSession =
+  require('../fixtures/completedSession.json') as WorkshopSession;
 
 function makeIo(): LoopIO {
   return {
@@ -57,34 +63,26 @@ function makeIo(): LoopIO {
   };
 }
 
-function makeFakeScaffolder(outputDir: string): PocScaffolder {
-  return {
-    scaffold: vi.fn().mockImplementation(async () => {
-      const { writeFile, mkdir } = await import('node:fs/promises');
-      await mkdir(join(outputDir, 'src'), { recursive: true });
-      await writeFile(join(outputDir, 'package.json'), JSON.stringify({
+function setupDynamicScaffoldMock(outputDir: string): void {
+  vi.mocked(generateDynamicScaffold).mockImplementation(async () => {
+    const { writeFile, mkdir } = await import('node:fs/promises');
+    await mkdir(join(outputDir, 'src'), { recursive: true });
+    await writeFile(
+      join(outputDir, 'package.json'),
+      JSON.stringify({
         name: 'test-poc',
         scripts: { test: 'vitest run' },
         dependencies: {},
         devDependencies: {},
-      }), 'utf-8');
-      await writeFile(join(outputDir, 'src', 'index.ts'), 'export function main() {}', 'utf-8');
-      return {
-        createdFiles: ['package.json', 'src/index.ts'],
-        skippedFiles: [],
-        context: {
-          projectName: 'test-poc',
-          ideaTitle: 'Test',
-          ideaDescription: 'Test',
-          techStack: { language: 'TypeScript', runtime: 'Node.js 20', testRunner: 'npm test' },
-          planSummary: 'Test',
-          sessionId: fixtureSession.sessionId,
-          outputDir,
-        },
-      };
-    }),
-    getTemplateFiles: () => [],
-  } as unknown as PocScaffolder;
+      }),
+      'utf-8',
+    );
+    await writeFile(join(outputDir, 'src', 'index.ts'), 'export function main() {}', 'utf-8');
+    return {
+      createdFiles: ['package.json', 'src/index.ts'],
+      techStack: { language: 'TypeScript', runtime: 'Node.js 20', testRunner: 'npm test' },
+    };
+  });
 }
 
 function makeClient(): CopilotClient {
@@ -119,7 +117,7 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
   it('sets finalStatus=partial when some tests pass at max-iterations', async () => {
     const io = makeIo();
     const client = makeClient();
-    const scaffolder = makeFakeScaffolder(tmpDir);
+    setupDynamicScaffoldMock(tmpDir);
 
     const testRunner: TestRunner = {
       run: vi.fn().mockResolvedValue({
@@ -140,7 +138,6 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
       outputDir: tmpDir,
       maxIterations: 2,
       testRunner,
-      scaffolder,
     });
 
     const result = await ralph.run();
@@ -154,7 +151,7 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
   it('sets finalStatus=failed when no tests pass at max-iterations', async () => {
     const io = makeIo();
     const client = makeClient();
-    const scaffolder = makeFakeScaffolder(tmpDir);
+    setupDynamicScaffoldMock(tmpDir);
 
     const testRunner: TestRunner = {
       run: vi.fn().mockResolvedValue({
@@ -178,7 +175,6 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
       outputDir: tmpDir,
       maxIterations: 2,
       testRunner,
-      scaffolder,
     });
 
     const result = await ralph.run();
@@ -189,7 +185,7 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
 
   it('records error iteration when LLM returns empty response, continues loop', async () => {
     const io = makeIo();
-    const scaffolder = makeFakeScaffolder(tmpDir);
+    setupDynamicScaffoldMock(tmpDir);
 
     let testCallCount = 0;
     const testRunner: TestRunner = {
@@ -254,7 +250,6 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
       outputDir: tmpDir,
       maxIterations: 5,
       testRunner,
-      scaffolder,
     });
 
     const result = await ralph.run();
@@ -272,7 +267,7 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
   it('records terminationReason in session poc state', async () => {
     const io = makeIo();
     const client = makeClient();
-    const scaffolder = makeFakeScaffolder(tmpDir);
+    setupDynamicScaffoldMock(tmpDir);
 
     const testRunner: TestRunner = {
       run: vi.fn().mockResolvedValue({
@@ -293,7 +288,6 @@ describe('RalphLoop integration — partial/failed outcomes', () => {
       outputDir: tmpDir,
       maxIterations: 2,
       testRunner,
-      scaffolder,
     });
 
     const result = await ralph.run();
@@ -381,7 +375,7 @@ describe('resume from interrupted session', () => {
       }),
     } as unknown as TestRunner;
 
-    const scaffolder = makeFakeScaffolder(tmpDir);
+    setupDynamicScaffoldMock(tmpDir);
 
     const ralph = new RalphLoop({
       client: passingClient,
@@ -390,7 +384,6 @@ describe('resume from interrupted session', () => {
       outputDir: tmpDir,
       maxIterations: 10,
       testRunner,
-      scaffolder,
       checkpoint: {
         hasPriorRun: true,
         completedIterations: 2,
@@ -405,7 +398,7 @@ describe('resume from interrupted session', () => {
     const result = await ralph.run();
 
     // Should have seeded from prior iterations and continued
-    // Note: finalStatus may be 'partial' since makeFakeScaffolder doesn't create all
+    // Note: finalStatus may be 'partial' since setupDynamicScaffoldMock doesn't create all
     // files required by validatePocOutput (README.md, tsconfig.json, etc.)
     expect(['success', 'partial']).toContain(result.finalStatus);
     // Total iterations should include the 2 prior + scaffold + tests-passing

package/tests/unit/develop/githubMcpAdapter.spec.ts

@@ -3,7 +3,6 @@
  *
  * Verifies:
  * - isAvailable() checks McpManager.isAvailable('github')
- * - createRepository() calls MCP tool
  * - pushFiles() commits and pushes
  * - Graceful fallback returns { available: false } when MCP unavailable
  */
@@ -48,82 +47,6 @@ describe('GitHubMcpAdapter', () => {
     });
   });
 
-  describe('createRepository()', () => {
-    it('calls mcpManager.callTool with create_repository and returns parsed result', async () => {
-      const callTool = vi.fn().mockResolvedValue({
-        html_url: 'https://github.com/acme/poc-route-optimizer',
-        name: 'poc-route-optimizer',
-      });
-      const mgr = makeMcpManager(true, callTool);
-      const adapter = new GitHubMcpAdapter(mgr);
-      const result = await adapter.createRepository({
-        name: 'poc-route-optimizer',
-        description: 'AI route optimization PoC',
-      });
-
-      expect(result.available).toBe(true);
-      if (result.available) {
-        expect(result.repoUrl).toBe('https://github.com/acme/poc-route-optimizer');
-        expect(result.repoName).toBe('poc-route-optimizer');
-      }
-      expect(callTool).toHaveBeenCalledWith(
-        'github',
-        'create_repository',
-        {
-          name: 'poc-route-optimizer',
-          description: 'AI route optimization PoC',
-          private: true,
-        },
-        { timeoutMs: 60_000 },
-      );
-    });
-
-    it('returns { available: false } when GitHub MCP is unavailable', async () => {
-      const adapter = new GitHubMcpAdapter(makeMcpManager(false));
-      const result = await adapter.createRepository({
-        name: 'poc-route-optimizer',
-      });
-
-      expect(result.available).toBe(false);
-      if (!result.available) {
-        expect(result.reason).toBeDefined();
-      }
-    });
-
-    it('returns { available: false } when callTool throws', async () => {
-      const callTool = vi.fn().mockRejectedValue(new Error('MCP callTool not yet wired'));
-      const adapter = new GitHubMcpAdapter(makeMcpManager(true, callTool));
-      const result = await adapter.createRepository({ name: 'poc-fail' });
-
-      expect(result.available).toBe(false);
-      if (!result.available) {
-        expect(result.reason).toContain('not yet wired');
-      }
-    });
-
-    it('stores repo URL after successful creation', async () => {
-      const callTool = vi.fn().mockResolvedValue({
-        html_url: 'https://github.com/acme/my-poc',
-        name: 'my-poc',
-      });
-      const adapter = new GitHubMcpAdapter(makeMcpManager(true, callTool));
-      await adapter.createRepository({ name: 'my-poc' });
-
-      expect(adapter.getRepoUrl()).toBe('https://github.com/acme/my-poc');
-    });
-
-    it('returns { available: false } when response has no URL', async () => {
-      const callTool = vi.fn().mockResolvedValue({ id: 123 });
-      const adapter = new GitHubMcpAdapter(makeMcpManager(true, callTool));
-      const result = await adapter.createRepository({ name: 'no-url' });
-
-      expect(result.available).toBe(false);
-      if (!result.available) {
-        expect(result.reason).toContain('missing repository URL');
-      }
-    });
-  });
-
   describe('pushFiles()', () => {
     it('calls mcpManager.callTool with push_files and returns commitSha', async () => {
       const callTool = vi.fn().mockResolvedValue({ sha: 'abc123ff' });
@@ -203,63 +126,6 @@ describe('GitHubMcpAdapter', () => {
       const adapter = new GitHubMcpAdapter(makeMcpManager(true));
       expect(adapter.getRepoUrl()).toBeUndefined();
     });
-
-    it('returns URL after successful createRepository', async () => {
-      const callTool = vi.fn().mockResolvedValue({
-        html_url: 'https://github.com/acme/test-poc',
-        name: 'test-poc',
-      });
-      const adapter = new GitHubMcpAdapter(makeMcpManager(true, callTool));
-      await adapter.createRepository({ name: 'test-poc' });
-      expect(adapter.getRepoUrl()).toBe('https://github.com/acme/test-poc');
-    });
-  });
-
-  // ── T008: Contract tests per contracts/github-adapter.md ────────────────
-
-  describe('createRepository() — contract: URL fallback chain', () => {
-    it('falls back to response.url when html_url is missing', async () => {
-      const callTool = vi.fn().mockResolvedValue({
-        url: 'https://api.github.com/repos/acme/poc',
-        name: 'poc',
-      });
-      const adapter = new GitHubMcpAdapter(makeMcpManager(true, callTool));
-      const result = await adapter.createRepository({ name: 'poc' });
-
-      expect(result.available).toBe(true);
-      if (result.available) {
-        expect(result.repoUrl).toBe('https://api.github.com/repos/acme/poc');
-      }
-    });
-
-    it('falls back to response.clone_url when html_url and url are missing', async () => {
-      const callTool = vi.fn().mockResolvedValue({
-        clone_url: 'https://github.com/acme/poc.git',
-        name: 'poc',
-      });
-      const adapter = new GitHubMcpAdapter(makeMcpManager(true, callTool));
-      const result = await adapter.createRepository({ name: 'poc' });
-
-      expect(result.available).toBe(true);
-      if (result.available) {
-        expect(result.repoUrl).toBe('https://github.com/acme/poc.git');
-      }
-    });
-
-    it('extracts repoName from response.full_name as fallback', async () => {
-      const callTool = vi.fn().mockResolvedValue({
-        html_url: 'https://github.com/acme/poc',
-        full_name: 'acme/poc',
-      });
-      const adapter = new GitHubMcpAdapter(makeMcpManager(true, callTool));
-      const result = await adapter.createRepository({ name: 'poc' });
-
-      expect(result.available).toBe(true);
-      if (result.available) {
-        // name not in response → full_name fallback returns 'acme/poc'
-        expect(result.repoName).toBe('acme/poc');
-      }
-    });
   });
 
   describe('pushFiles() — contract: commitSha extraction', () => {

package/tests/unit/develop/outputValidator.spec.ts

@@ -8,7 +8,7 @@ import { mkdtemp, rm, writeFile, mkdir } from 'node:fs/promises';
 import { join } from 'node:path';
 import { tmpdir } from 'node:os';
 
-import { validatePocOutput } from '../../../src/develop/pocScaffolder.js';
+import { validatePocOutput } from '../../../src/develop/pocUtils.js';
 
 describe('validatePocOutput', () => {
   let tmpDir: string;
@@ -24,24 +24,44 @@ describe('validatePocOutput', () => {
   async function createCompleteScaffold(): Promise<void> {
     await mkdir(join(tmpDir, 'src'), { recursive: true });
     await mkdir(join(tmpDir, 'tests'), { recursive: true });
-    await writeFile(join(tmpDir, 'package.json'), JSON.stringify({
-      name: 'test-poc',
-      version: '0.1.0',
-      scripts: { test: 'vitest run', build: 'tsc', start: 'node dist/index.js' },
-      dependencies: {},
-      devDependencies: { vitest: '^3.0.0', typescript: '^5.0.0' },
-    }), 'utf-8');
+    await writeFile(
+      join(tmpDir, 'package.json'),
+      JSON.stringify({
+        name: 'test-poc',
+        version: '0.1.0',
+        scripts: { test: 'vitest run', build: 'tsc', start: 'node dist/index.js' },
+        dependencies: {},
+        devDependencies: { vitest: '^3.0.0', typescript: '^5.0.0' },
+      }),
+      'utf-8',
+    );
     await writeFile(join(tmpDir, 'README.md'), '# Test PoC\n\nA test proof of concept.', 'utf-8');
-    await writeFile(join(tmpDir, 'tsconfig.json'), JSON.stringify({
-      compilerOptions: { target: 'ES2022', module: 'Node16', strict: true },
-    }), 'utf-8');
+    await writeFile(
+      join(tmpDir, 'tsconfig.json'),
+      JSON.stringify({
+        compilerOptions: { target: 'ES2022', module: 'Node16', strict: true },
+      }),
+      'utf-8',
+    );
     await writeFile(join(tmpDir, '.gitignore'), 'node_modules/\ndist/\ncoverage/\n', 'utf-8');
-    await writeFile(join(tmpDir, '.sofia-metadata.json'), JSON.stringify({
-      sessionId: 'test-001',
-      featureSpec: '002-poc-generation',
-    }), 'utf-8');
-    await writeFile(join(tmpDir, 'src', 'index.ts'), 'export function main() { return "ok"; }', 'utf-8');
-    await writeFile(join(tmpDir, 'tests', 'index.test.ts'), 'import { test, expect } from "vitest"; test("works", () => { expect(true).toBe(true); });', 'utf-8');
+    await writeFile(
+      join(tmpDir, '.sofia-metadata.json'),
+      JSON.stringify({
+        sessionId: 'test-001',
+        featureSpec: '002-poc-generation',
+      }),
+      'utf-8',
+    );
+    await writeFile(
+      join(tmpDir, 'src', 'index.ts'),
+      'export function main() { return "ok"; }',
+      'utf-8',
+    );
+    await writeFile(
+      join(tmpDir, 'tests', 'index.test.ts'),
+      'import { test, expect } from "vitest"; test("works", () => { expect(true).toBe(true); });',
+      'utf-8',
+    );
   }
 
   it('1. validates package.json exists and has test script', async () => {
@@ -115,10 +135,14 @@ describe('validatePocOutput', () => {
 
   it('reports error when package.json has no test script', async () => {
     await createCompleteScaffold();
-    await writeFile(join(tmpDir, 'package.json'), JSON.stringify({
-      name: 'test-poc',
-      scripts: { build: 'tsc' }, // no test script
-    }), 'utf-8');
+    await writeFile(
+      join(tmpDir, 'package.json'),
+      JSON.stringify({
+        name: 'test-poc',
+        scripts: { build: 'tsc' }, // no test script
+      }),
+      'utf-8',
+    );
     const result = await validatePocOutput(tmpDir);
     expect(result.valid).toBe(false);
     expect(result.errors).toContain('package.json is missing "test" script');