agentic-orchestrator 0.1.6 → 0.1.8

package/apps/control-plane/test/qa-index-service.spec.ts

@@ -30,7 +30,7 @@ describe('QaIndexService', () => {
     const validateSchema = vi.fn(async () => ({ valid: true }));
     const service = new QaIndexService({
       qaIndexPath: () => path.join(tempDir, 'qa_test_index.json'),
-      validateSchema
+      validateSchema,
     });
 
     const result = await service.qaTestIndexGet('feature_qa');
@@ -38,14 +38,14 @@ describe('QaIndexService', () => {
     expect(result.data.index).toMatchObject({
       feature_id: 'feature_qa',
       version: 1,
-      items: []
+      items: [],
     });
     expect(result.data.summary).toEqual({
       pending: 0,
       running: 0,
       passed: 0,
       failed: 0,
-      waived: 0
+      waived: 0,
     });
     expect(result.data.pending_required_tests).toEqual({});
     expect(validateSchema).toHaveBeenCalledWith('qa_test_index.schema.json', expect.any(Object));
@@ -57,15 +57,15 @@ describe('QaIndexService', () => {
 
     const service = new QaIndexService({
       qaIndexPath: () => path.join(tempDir, 'qa_test_index.json'),
-      validateSchema: async () => ({ valid: false, errors: [{ message: 'invalid_schema' }] })
+      validateSchema: async () => ({ valid: false, errors: [{ message: 'invalid_schema' }] }),
     });
 
     await expect(service.qaTestIndexGet('feature_invalid')).rejects.toMatchObject({
       normalizedResponse: {
         error: {
-          code: ERROR_CODES.INTERNAL_ERROR
-        }
-      }
+          code: ERROR_CODES.INTERNAL_ERROR,
+        },
+      },
     });
   });
 
@@ -75,17 +75,15 @@ describe('QaIndexService', () => {
 
     const service = new QaIndexService({
       qaIndexPath: () => path.join(tempDir, 'qa_test_index.json'),
-      validateSchema: async () => ({ valid: true })
+      validateSchema: async () => ({ valid: true }),
     });
 
-    await expect(
-      service.qaTestIndexUpdate('feature_missing', 1, [], [])
-    ).rejects.toMatchObject({
+    await expect(service.qaTestIndexUpdate('feature_missing', 1, [], [])).rejects.toMatchObject({
       normalizedResponse: {
         error: {
-          code: ERROR_CODES.FILE_NOT_FOUND
-        }
-      }
+          code: ERROR_CODES.FILE_NOT_FOUND,
+        },
+      },
     });
   });
 
@@ -103,15 +101,15 @@ describe('QaIndexService', () => {
           path: 'src/sample.ts',
           hunks: [{ start_line: 1, end_line: 2, change_type: 'modify' }],
           required_tests: ['default:fast:lint'],
-          status: 'pending'
-        }
-      ]
+          status: 'pending',
+        },
+      ],
     };
     await writeJson(qaIndexPath, initialIndex);
 
     const service = new QaIndexService({
       qaIndexPath: () => qaIndexPath,
-      validateSchema: async () => ({ valid: false, errors: [{ message: 'schema_failed' }] })
+      validateSchema: async () => ({ valid: false, errors: [{ message: 'schema_failed' }] }),
     });
 
     await expect(
@@ -122,17 +120,17 @@ describe('QaIndexService', () => {
           {
             path: 'src/sample.ts',
             hunk: { start_line: 1, end_line: 2, change_type: 'modify' },
-            status: 'passed'
-          }
+            status: 'passed',
+          },
         ],
-        ['evidence://run-1']
-      )
+        ['evidence://run-1'],
+      ),
     ).rejects.toMatchObject({
       normalizedResponse: {
         error: {
-          code: ERROR_CODES.INTERNAL_ERROR
-        }
-      }
+          code: ERROR_CODES.INTERNAL_ERROR,
+        },
+      },
     });
   });
 
@@ -150,14 +148,14 @@ describe('QaIndexService', () => {
           path: 'src/sample.ts',
           hunks: [{ start_line: 3, end_line: 4, change_type: 'modify' }],
           required_tests: ['default:fast:lint'],
-          status: 'pending'
-        }
-      ]
+          status: 'pending',
+        },
+      ],
     });
 
     const service = new QaIndexService({
       qaIndexPath: () => qaIndexPath,
-      validateSchema: async () => ({ valid: true })
+      validateSchema: async () => ({ valid: true }),
     });
 
     const result = await service.qaTestIndexUpdate(
@@ -167,15 +165,15 @@ describe('QaIndexService', () => {
         {
           path: 'src/sample.ts',
           hunk: { start_line: 3, end_line: 4, change_type: 'modify' },
-          status: 'passed'
-        }
+          status: 'passed',
+        },
       ],
-      ['evidence://run-ok']
+      ['evidence://run-ok'],
     );
 
     expect(result.data).toEqual({
       feature_id: 'feature_update_ok',
-      version: 2
+      version: 2,
     });
 
     const persisted = JSON.parse(await fs.readFile(qaIndexPath, 'utf8'));

package/apps/control-plane/test/qa-index.spec.ts

@@ -4,7 +4,7 @@ import {
   buildQaIndex,
   makeRequiredTests,
   summarizeQaIndex,
-  type QaIndex
+  type QaIndex,
 } from '../src/core/qa-index.js';
 import type { ParsedDiffFile } from '../src/core/patch.js';
 
@@ -13,14 +13,11 @@ describe('qa-index helpers', () => {
     const required = makeRequiredTests('default', {
       modes: {
         fast: [{ name: 'lint' }],
-        full: [{ name: 'test_with_coverage' }]
-      }
+        full: [{ name: 'test_with_coverage' }],
+      },
     });
 
-    expect(required).toEqual([
-      'default:fast:lint',
-      'default:full:test_with_coverage'
-    ]);
+    expect(required).toEqual(['default:fast:lint', 'default:full:test_with_coverage']);
   });
 
   it('GIVEN_previous_index_WHEN_building_new_index_THEN_preserves_known_item_state', () => {
@@ -29,14 +26,14 @@ describe('qa-index helpers', () => {
         oldPath: 'src/a.ts',
         newPath: 'src/a.ts',
         changeType: 'modify',
-        hunks: [{ start_line: 10, end_line: 12, change_type: 'modify' }]
+        hunks: [{ start_line: 10, end_line: 12, change_type: 'modify' }],
       },
       {
         oldPath: 'src/removed.ts',
         newPath: '/dev/null',
         changeType: 'delete',
-        hunks: []
-      }
+        hunks: [],
+      },
     ];
 
     const previous: QaIndex = {
@@ -50,9 +47,9 @@ describe('qa-index helpers', () => {
           required_tests: ['old'],
           status: 'passed',
           last_run_at: '2025-01-01T00:00:00.000Z',
-          evidence_refs: ['evidence-a']
-        }
-      ]
+          evidence_refs: ['evidence-a'],
+        },
+      ],
     };
 
     const next = buildQaIndex('feature_a', parsedDiff, ['default:fast:lint'], previous);
@@ -63,16 +60,16 @@ describe('qa-index helpers', () => {
       status: 'passed',
       last_run_at: '2025-01-01T00:00:00.000Z',
       evidence_refs: ['evidence-a'],
-      required_tests: ['default:fast:lint']
+      required_tests: ['default:fast:lint'],
     });
     expect(next.items[1]).toMatchObject({
       path: 'src/removed.ts',
-      status: 'pending'
+      status: 'pending',
     });
     expect(next.items[1].hunks[0]).toEqual({
       start_line: 1,
       end_line: 1,
-      change_type: 'delete'
+      change_type: 'delete',
     });
   });
 
@@ -86,15 +83,15 @@ describe('qa-index helpers', () => {
           path: 'src/a.ts',
           hunks: [{ start_line: 1, end_line: 2, change_type: 'modify' }],
           required_tests: ['fast'],
-          status: 'pending'
+          status: 'pending',
         },
         {
           path: 'src/b.ts',
           hunks: [{ start_line: 5, end_line: 6, change_type: 'add' }],
           required_tests: ['full'],
-          status: 'failed'
-        }
-      ]
+          status: 'failed',
+        },
+      ],
     };
 
     const summary = summarizeQaIndex(index);
@@ -103,15 +100,15 @@ describe('qa-index helpers', () => {
       running: 0,
       passed: 0,
       failed: 1,
-      waived: 0
+      waived: 0,
     });
     expect(summary.pending_by_file).toEqual({
       'src/a.ts': [
         {
           hunks: [{ start_line: 1, end_line: 2, change_type: 'modify' }],
-          required_tests: ['fast']
-        }
-      ]
+          required_tests: ['fast'],
+        },
+      ],
     });
   });
 
@@ -126,9 +123,9 @@ describe('qa-index helpers', () => {
           hunks: [{ start_line: 1, end_line: 3, change_type: 'modify' }],
           required_tests: ['fast'],
           status: 'pending',
-          evidence_refs: ['existing']
-        }
-      ]
+          evidence_refs: ['existing'],
+        },
+      ],
     };
 
     const next = applyQaUpdates(
@@ -138,10 +135,10 @@ describe('qa-index helpers', () => {
         {
           path: 'src/a.ts',
           hunk: { start_line: 1, end_line: 3, change_type: 'modify' },
-          status: 'passed'
-        }
+          status: 'passed',
+        },
       ],
-      ['new-evidence', 'existing']
+      ['new-evidence', 'existing'],
     );
 
     expect(next.version).toBe(3);
@@ -160,9 +157,9 @@ describe('qa-index helpers', () => {
           path: 'src/a.ts',
           hunks: [{ start_line: 1, end_line: 1, change_type: 'modify' }],
           required_tests: ['fast'],
-          status: 'pending'
-        }
-      ]
+          status: 'pending',
+        },
+      ],
     };
 
     expect(() =>
@@ -170,9 +167,9 @@ describe('qa-index helpers', () => {
         {
           path: '',
           hunk: { start_line: 1, end_line: 1, change_type: 'modify' },
-          status: 'failed'
-        }
-      ])
+          status: 'failed',
+        },
+      ]),
     ).toThrow('invalid_argument');
 
     expect(() =>
@@ -180,9 +177,9 @@ describe('qa-index helpers', () => {
         {
           path: 'src/missing.ts',
           hunk: { start_line: 1, end_line: 1, change_type: 'modify' },
-          status: 'failed'
-        }
-      ])
+          status: 'failed',
+        },
+      ]),
     ).toThrow('invalid_argument');
   });
 
@@ -191,7 +188,7 @@ describe('qa-index helpers', () => {
       feature_id: 'feature_a',
       version: 5,
       source_diff_ref: 'ref',
-      items: []
+      items: [],
     };
 
     expect(() => applyQaUpdates(index, 4, [])).toThrow('qa_index_version_conflict');
@@ -205,11 +202,11 @@ describe('qa-index helpers', () => {
           oldPath: '',
           newPath: '',
           changeType: 'modify',
-          hunks: []
-        }
+          hunks: [],
+        },
       ],
       ['default:fast:lint'],
-      null
+      null,
     );
 
     expect(next.items).toEqual([]);
@@ -219,7 +216,7 @@ describe('qa-index helpers', () => {
     const summary = summarizeQaIndex({
       feature_id: 'feature_sparse',
       version: 1,
-      source_diff_ref: 'ref'
+      source_diff_ref: 'ref',
     } as unknown as QaIndex);
 
     expect(summary.summary).toEqual({
@@ -227,7 +224,7 @@ describe('qa-index helpers', () => {
       running: 0,
       passed: 0,
       failed: 0,
-      waived: 0
+      waived: 0,
     });
     expect(summary.pending_by_file).toEqual({});
   });
@@ -244,16 +241,16 @@ describe('qa-index helpers', () => {
           oldPath: 'src/a.ts',
           newPath: 'src/a.ts',
           changeType: 'modify',
-          hunks: [{ start_line: 1, end_line: 2, change_type: 'modify' }]
-        }
+          hunks: [{ start_line: 1, end_line: 2, change_type: 'modify' }],
+        },
       ],
       ['default:fast:lint'],
       {
         feature_id: 'feature_sparse',
         version: 1,
         source_diff_ref: 'old',
-        items: [{ path: 'src/a.ts', status: 'passed' } as any]
-      } as QaIndex
+        items: [{ path: 'src/a.ts', status: 'passed' } as any],
+      } as QaIndex,
     );
     expect(rebuilt.items[0]?.status).toBe('pending');
 
@@ -266,12 +263,12 @@ describe('qa-index helpers', () => {
           path: 'src/unknown.ts',
           hunks: [{ start_line: 1, end_line: 1, change_type: 'modify' }],
           required_tests: ['fast'],
-          status: 'custom' as any
-        }
-      ]
+          status: 'custom' as any,
+        },
+      ],
     });
     expect(summary.summary).toMatchObject({
-      custom: 1
+      custom: 1,
     });
   });
 
@@ -281,10 +278,10 @@ describe('qa-index helpers', () => {
         feature_id: 'feature_sparse_apply',
         version: 3,
         source_diff_ref: 'ref',
-        items: [{ path: 'src/a.ts', required_tests: ['fast'], status: 'pending' } as any]
+        items: [{ path: 'src/a.ts', required_tests: ['fast'], status: 'pending' } as any],
       } as QaIndex,
       3,
-      []
+      [],
     );
     expect(sparse.version).toBe(4);
 
@@ -298,19 +295,19 @@ describe('qa-index helpers', () => {
             path: 'src/a.ts',
             hunks: [{ start_line: 2, end_line: 3, change_type: 'modify' }],
             required_tests: ['fast'],
-            status: 'pending'
-          }
-        ]
+            status: 'pending',
+          },
+        ],
       },
       1,
       [
         {
           path: 'src/a.ts',
           hunk: { start_line: 2, end_line: 3, change_type: 'modify' },
-          status: 'passed'
-        }
+          status: 'passed',
+        },
       ],
-      ['evidence-1']
+      ['evidence-1'],
     );
     expect(updated.items[0]?.evidence_refs).toEqual(['evidence-1']);
   });