agentic-orchestrator 0.1.6 → 0.1.7

package/apps/control-plane/test/batch-operations.spec.ts

@@ -8,8 +8,8 @@ const specResolverResolveMock = vi.hoisted(() => vi.fn());
 
 vi.mock('node:fs/promises', () => ({
   default: {
-    readFile: fsReadFileMock
-  }
+    readFile: fsReadFileMock,
+  },
 }));
 
 vi.mock('../src/cli/spec-input-resolver.js', () => ({
@@ -17,12 +17,14 @@ vi.mock('../src/cli/spec-input-resolver.js', () => ({
     async resolve(...args: unknown[]) {
       return await specResolverResolveMock(...args);
     }
-  }
+  },
 }));
 
 import { RunCommandHandler } from '../src/cli/run-command-handler.js';
 
-function makeContext(toolCallImpl = vi.fn(async () => ({ ok: true, data: {} }))): RunCommandContext {
+function makeContext(
+  toolCallImpl = vi.fn(async () => ({ ok: true, data: {} })),
+): RunCommandContext {
   return {
     repoRoot: '/repo',
     env: {},
@@ -30,7 +32,7 @@ function makeContext(toolCallImpl = vi.fn(async () => ({ ok: true, data: {} })))
     transport: 'inprocess' as const,
     options: { command: 'run', batch: true, folder_input: '/specs' },
     kernel: { getAgentsConfig: () => ({}) } as never,
-    toolClient: { call: toolCallImpl } as never
+    toolClient: { call: toolCallImpl } as never,
   };
 }
 
@@ -42,7 +44,7 @@ describe('BatchOperations', () => {
   it('GIVEN_folder_with_specs_WHEN_batch_run_all_new_THEN_all_created', async () => {
     specResolverResolveMock.mockResolvedValue([
       '/repo/.aop/features/feature_a/spec.md',
-      '/repo/.aop/features/feature_b/spec.md'
+      '/repo/.aop/features/feature_b/spec.md',
     ]);
     fsReadFileMock.mockResolvedValue(JSON.stringify({ active: [], blocked: [] }));
 
@@ -52,24 +54,24 @@ describe('BatchOperations', () => {
 
     expect(result).toMatchObject({
       ok: true,
-      data: { command: 'batch-run', created: 2, skipped: 0, failed: 0 }
+      data: { command: 'batch-run', created: 2, skipped: 0, failed: 0 },
     });
     expect(toolCallMock).toHaveBeenCalledWith(
       TOOLS.FEATURE_INIT,
       expect.objectContaining({ feature_id: 'feature_a' }),
-      expect.any(Object)
+      expect.any(Object),
     );
     expect(toolCallMock).toHaveBeenCalledWith(
       TOOLS.FEATURE_INIT,
       expect.objectContaining({ feature_id: 'feature_b' }),
-      expect.any(Object)
+      expect.any(Object),
     );
   });
 
   it('GIVEN_folder_with_specs_WHEN_batch_run_some_active_THEN_active_skipped', async () => {
     specResolverResolveMock.mockResolvedValue([
       '/repo/.aop/features/feature_a/spec.md',
-      '/repo/.aop/features/feature_b/spec.md'
+      '/repo/.aop/features/feature_b/spec.md',
     ]);
     fsReadFileMock.mockResolvedValue(JSON.stringify({ active: ['feature_a'], blocked: [] }));
 
@@ -79,12 +81,12 @@ describe('BatchOperations', () => {
 
     expect(result).toMatchObject({
       ok: true,
-      data: { command: 'batch-run', created: 1, skipped: 1, failed: 0 }
+      data: { command: 'batch-run', created: 1, skipped: 1, failed: 0 },
     });
     expect(toolCallMock).not.toHaveBeenCalledWith(
       TOOLS.FEATURE_INIT,
       expect.objectContaining({ feature_id: 'feature_a' }),
-      expect.any(Object)
+      expect.any(Object),
     );
   });
 
@@ -94,9 +96,9 @@ describe('BatchOperations', () => {
       data: {
         features: [
           { feature_id: 'feature_a', status: 'planning' },
-          { feature_id: 'feature_b', status: 'building' }
-        ]
-      }
+          { feature_id: 'feature_b', status: 'building' },
+        ],
+      },
     }));
     const handler = new StatusCommandHandler({ call: dashboardCall } as never, 'run:test');
     const result = await handler.execute({ summary: true });
@@ -106,9 +108,9 @@ describe('BatchOperations', () => {
       data: {
         summary: [
           { feature_id: 'feature_a', status: 'planning', activity: 'unknown' },
-          { feature_id: 'feature_b', status: 'building', activity: 'unknown' }
-        ]
-      }
+          { feature_id: 'feature_b', status: 'building', activity: 'unknown' },
+        ],
+      },
     });
   });
 });

package/apps/control-plane/test/bootstrap-attach.spec.ts

@@ -13,20 +13,32 @@ vi.mock('node:readline/promises', () => ({
   default: {
     createInterface: vi.fn(() => ({
       question: vi.fn(async () => ''),
-      close: vi.fn()
-    }))
-  }
+      close: vi.fn(),
+    })),
+  },
 }));
 
 vi.mock('../src/core/kernel.js', () => {
   class AopKernel {
     private readonly repoRoot: string;
-    constructor(repoRoot: string) { this.repoRoot = repoRoot; }
-    async ensureLoaded() { await ensureLoadedMock(); }
-    getRepoRoot() { return this.repoRoot; }
-    getAgentsConfig() { return {}; }
-    getRbacPolicy() { return {}; }
-    getPolicySnapshot() { return {}; }
+    constructor(repoRoot: string) {
+      this.repoRoot = repoRoot;
+    }
+    async ensureLoaded() {
+      await ensureLoadedMock();
+    }
+    getRepoRoot() {
+      return this.repoRoot;
+    }
+    getAgentsConfig() {
+      return {};
+    }
+    getRbacPolicy() {
+      return {};
+    }
+    getPolicySnapshot() {
+      return {};
+    }
     setProvider() {}
   }
   return { AopKernel };
@@ -34,15 +46,25 @@ vi.mock('../src/core/kernel.js', () => {
 
 vi.mock('../src/mcp/runtime-factory.js', () => ({
   createToolingRuntime: createToolingRuntimeMock,
-  resolveToolClient: resolveToolClientMock
+  resolveToolClient: resolveToolClientMock,
 }));
 
 vi.mock('../src/cli/init-command-handler.js', () => ({
   InitCommandHandler: class InitCommandHandler {
     async execute() {
-      return { ok: true, data: { command: 'init', created: [], updated: [], skipped: [], validation_warnings: [], next_steps: [] } };
+      return {
+        ok: true,
+        data: {
+          command: 'init',
+          created: [],
+          updated: [],
+          skipped: [],
+          validation_warnings: [],
+          next_steps: [],
+        },
+      };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/dashboard-command-handler.js', () => ({
@@ -50,7 +72,7 @@ vi.mock('../src/cli/dashboard-command-handler.js', () => ({
     async execute() {
       return { ok: true, data: { message: 'started', port: 3000 } };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/run-command-handler.js', () => ({
@@ -58,7 +80,7 @@ vi.mock('../src/cli/run-command-handler.js', () => ({
     async execute() {
       return { ok: true, data: { status: 'running' } };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/resume-command-handler.js', () => ({
@@ -66,7 +88,7 @@ vi.mock('../src/cli/resume-command-handler.js', () => ({
     async execute() {
       return { ok: true, data: { resumed: false } };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/attach-command-handler.js', () => ({
@@ -74,10 +96,17 @@ vi.mock('../src/cli/attach-command-handler.js', () => ({
     async execute() {
       return {
         ok: true,
-        data: { command: 'attach', feature_id: 'feat_x', status: 'building', role: 'builder', session_id: 'sess-1', attached: true }
+        data: {
+          command: 'attach',
+          feature_id: 'feat_x',
+          status: 'building',
+          role: 'builder',
+          session_id: 'sess-1',
+          attached: true,
+        },
       };
     }
-  }
+  },
 }));
 
 describe('bootstrap runCli attach success path', () => {
@@ -100,12 +129,16 @@ describe('bootstrap runCli attach success path', () => {
     const { runCli } = await import('../src/interfaces/cli/bootstrap.js');
     const code = await runCli(['attach', '--feature-id', 'feat_x'], {
       cwd: '/tmp',
-      env: { ...process.env, AOP_AGENT_PROVIDER: 'custom' }
+      env: { ...process.env, AOP_AGENT_PROVIDER: 'custom' },
     });
     // Attach handler mock returns successfully → printPayload → return 0
     expect(code).toBe(0);
     const writes = stdoutSpy.mock.calls.flatMap((call) => {
-      try { return [JSON.parse(String(call[0]))]; } catch { return []; }
+      try {
+        return [JSON.parse(String(call[0]))];
+      } catch {
+        return [];
+      }
     });
     expect(writes[0]).toMatchObject({ ok: true, data: { command: 'attach' } });
   });

package/apps/control-plane/test/bootstrap-edge-cases.spec.ts

@@ -16,27 +16,29 @@ const toolCallMock = vi.hoisted(() => vi.fn(async () => ({ ok: true, data: {} })
 
 /** Allows per-test control of getPolicySnapshot return value */
 const policySnapshotOverride = vi.hoisted(() => ({
-  value: {} as Record<string, unknown>
+  value: {} as Record<string, unknown>,
 }));
 
 /** Allows per-test control of whether ensureLoaded throws */
 const kernelThrowOverride = vi.hoisted(() => ({
-  value: null as Error | null
+  value: null as Error | null,
 }));
 
 vi.mock('node:readline/promises', () => ({
   default: {
     createInterface: vi.fn(() => ({
       question: vi.fn(async () => ''),
-      close: vi.fn()
-    }))
-  }
+      close: vi.fn(),
+    })),
+  },
 }));
 
 vi.mock('../src/core/kernel.js', () => {
   class AopKernel {
     private readonly repoRoot: string;
-    constructor(repoRoot: string) { this.repoRoot = repoRoot; }
+    constructor(repoRoot: string) {
+      this.repoRoot = repoRoot;
+    }
 
     async ensureLoaded() {
       await ensureLoadedMock();
@@ -45,9 +47,15 @@ vi.mock('../src/core/kernel.js', () => {
       }
     }
 
-    getRepoRoot() { return this.repoRoot; }
-    getAgentsConfig() { return {}; }
-    getRbacPolicy() { return {}; }
+    getRepoRoot() {
+      return this.repoRoot;
+    }
+    getAgentsConfig() {
+      return {};
+    }
+    getRbacPolicy() {
+      return {};
+    }
 
     getPolicySnapshot() {
       return policySnapshotOverride.value;
@@ -61,15 +69,25 @@ vi.mock('../src/core/kernel.js', () => {
 
 vi.mock('../src/mcp/runtime-factory.js', () => ({
   createToolingRuntime: createToolingRuntimeMock,
-  resolveToolClient: resolveToolClientMock
+  resolveToolClient: resolveToolClientMock,
 }));
 
 vi.mock('../src/cli/init-command-handler.js', () => ({
   InitCommandHandler: class InitCommandHandler {
     async execute() {
-      return { ok: true, data: { command: 'init', created: [], updated: [], skipped: [], validation_warnings: [], next_steps: [] } };
+      return {
+        ok: true,
+        data: {
+          command: 'init',
+          created: [],
+          updated: [],
+          skipped: [],
+          validation_warnings: [],
+          next_steps: [],
+        },
+      };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/dashboard-command-handler.js', () => ({
@@ -77,7 +95,7 @@ vi.mock('../src/cli/dashboard-command-handler.js', () => ({
     async execute() {
       return { ok: true, data: { message: 'started', port: 3000 } };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/run-command-handler.js', () => ({
@@ -85,7 +103,7 @@ vi.mock('../src/cli/run-command-handler.js', () => ({
     async execute() {
       return { ok: true, data: { status: 'running' } };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/resume-command-handler.js', () => ({
@@ -93,7 +111,7 @@ vi.mock('../src/cli/resume-command-handler.js', () => ({
     async execute() {
       return { ok: true, data: { resumed: false } };
     }
-  }
+  },
 }));
 
 vi.mock('../src/cli/attach-command-handler.js', () => ({
@@ -101,22 +119,28 @@ vi.mock('../src/cli/attach-command-handler.js', () => ({
     async execute() {
       return { ok: true, data: { command: 'attach' } };
     }
-  }
+  },
 }));
 
 // ── helpers ───────────────────────────────────────────────────────────────────
 
 async function writeMultiProjectYaml(dir: string, content: string): Promise<void> {
-  const configDir = path.join(dir, 'agentic', 'orchestrator');
+  const configDir = path.join(dir, 'config', 'agentic', 'orchestrator');
   await fs.mkdir(configDir, { recursive: true });
   await fs.writeFile(path.join(configDir, 'multi-project.yaml'), content, 'utf8');
 }
 
 async function writeSchemaFile(dir: string): Promise<void> {
-  const schemasDir = path.join(dir, 'agentic', 'orchestrator', 'schemas');
+  const schemasDir = path.join(dir, 'config', 'agentic', 'orchestrator', 'schemas');
   await fs.mkdir(schemasDir, { recursive: true });
   const repoRoot = path.resolve(import.meta.dirname, '../../../');
-  const srcSchema = path.join(repoRoot, 'agentic', 'orchestrator', 'schemas', 'multi-project.schema.json');
+  const srcSchema = path.join(
+    repoRoot,
+    'agentic',
+    'orchestrator',
+    'schemas',
+    'multi-project.schema.json',
+  );
   const destSchema = path.join(schemasDir, 'multi-project.schema.json');
   await fs.copyFile(srcSchema, destSchema);
 }
@@ -150,7 +174,7 @@ describe('bootstrap edge-case branch coverage', () => {
     await fs.mkdir(proj1sub, { recursive: true });
     await writeMultiProjectYaml(
       tmpDir,
-      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\n`
+      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\n`,
     );
 
     const { runCli } = await import('../src/interfaces/cli/bootstrap.js');
@@ -167,7 +191,7 @@ describe('bootstrap edge-case branch coverage', () => {
     await fs.mkdir(other, { recursive: true });
     await writeMultiProjectYaml(
       tmpDir,
-      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\ndefaults:\n  default_project: proj1\n`
+      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\ndefaults:\n  default_project: proj1\n`,
     );
 
     const { runCli } = await import('../src/interfaces/cli/bootstrap.js');
@@ -179,7 +203,7 @@ describe('bootstrap edge-case branch coverage', () => {
   it('GIVEN_cleanup_with_policy_having_grace_period_and_auto_after_merge_WHEN_cleanup_THEN_uses_policy_values', async () => {
     // Covers lines 291 and 292: cleanup?.grace_period_seconds non-null and auto_after_merge=true
     policySnapshotOverride.value = {
-      cleanup: { grace_period_seconds: 120, auto_after_merge: true }
+      cleanup: { grace_period_seconds: 120, auto_after_merge: true },
     };
     toolCallMock.mockResolvedValue({ ok: true, data: { cleaned: [], would_clean: [] } });
 
@@ -205,12 +229,15 @@ describe('bootstrap edge-case branch coverage', () => {
     await fs.mkdir(proj1, { recursive: true });
     await writeMultiProjectYaml(
       tmpDir,
-      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\ndefaults:\n  max_active_features: 5\n  max_parallel_gate_runs: 2\n`
+      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\ndefaults:\n  max_active_features: 5\n  max_parallel_gate_runs: 2\n`,
     );
     toolCallMock.mockResolvedValue({ ok: true, data: { status: 'running' } });
 
     const { runCli } = await import('../src/interfaces/cli/bootstrap.js');
-    const code = await runCli(['run', '--feature-id', 'feat_test'], { cwd: tmpDir, env: process.env });
+    const code = await runCli(['run', '--feature-id', 'feat_test'], {
+      cwd: tmpDir,
+      env: process.env,
+    });
     expect(code).toBe(0);
   });
 
@@ -220,7 +247,7 @@ describe('bootstrap edge-case branch coverage', () => {
     await fs.mkdir(proj1, { recursive: true });
     await writeMultiProjectYaml(
       tmpDir,
-      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\n    dashboard_port: 4000\n`
+      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\n    dashboard_port: 4000\n`,
     );
 
     const { runCli } = await import('../src/interfaces/cli/bootstrap.js');
@@ -237,7 +264,7 @@ describe('bootstrap edge-case branch coverage', () => {
     await fs.mkdir(proj2, { recursive: true });
     await writeMultiProjectYaml(
       tmpDir,
-      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\n  - name: proj2\n    path: ${proj2}\n`
+      `version: "1.0"\nprojects:\n  - name: proj1\n    path: ${proj1}\n  - name: proj2\n    path: ${proj2}\n`,
     );
 
     // proj2 will throw an error without .code property
@@ -255,7 +282,11 @@ describe('bootstrap edge-case branch coverage', () => {
     const code = await runCli(['status', '--all'], { cwd: tmpDir, env: process.env });
     expect(code).toBe(0);
     const writes = stdoutSpy.mock.calls.flatMap((call) => {
-      try { return [JSON.parse(String(call[0]))]; } catch { return []; }
+      try {
+        return [JSON.parse(String(call[0]))];
+      } catch {
+        return [];
+      }
     });
     expect(writes.some((w) => w.ok === true)).toBe(true);
   });