@auto-engineer/job-graph-processor 1.26.1 → 1.27.0

package/src/graph-processor.ts

@@ -39,16 +39,16 @@ export function createGraphProcessor(messageBus: MessageBus, options?: { dispatc
     const { graphId, jobs, failurePolicy } = command.data;
 
     if (graphs.has(graphId)) {
-      return { type: 'graph.failed', data: { graphId, reason: `Graph ${graphId} already submitted` } };
+      return { type: 'GraphFailed', data: { graphId, reason: `Graph ${graphId} already submitted` } };
     }
 
     if (!Array.isArray(jobs)) {
-      return { type: 'graph.failed', data: { graphId, reason: 'jobs is required and must be an array' } };
+      return { type: 'GraphFailed', data: { graphId, reason: 'jobs is required and must be an array' } };
     }
 
     const validation = validateGraph(jobs);
     if (!validation.valid) {
-      return { type: 'graph.failed', data: { graphId, reason: validation.error } };
+      return { type: 'GraphFailed', data: { graphId, reason: validation.error } };
     }
 
     let state = evolve(initialState(), {
@@ -81,7 +81,7 @@ export function createGraphProcessor(messageBus: MessageBus, options?: { dispatc
       dispatch({ type: d.target, data: d.payload, correlationId: d.correlationId }).catch(() => {});
     }
 
-    return { type: 'graph.dispatching', data: { graphId, dispatchedJobs: dispatched } };
+    return { type: 'GraphDispatched', data: { graphId, dispatchedJobs: dispatched } };
   }
 
   function onJobEvent(graphId: string, event: Event): void {
@@ -113,7 +113,7 @@ export function createGraphProcessor(messageBus: MessageBus, options?: { dispatc
 
     if (isGraphComplete(state)) {
       graphs.delete(graphId);
-      messageBus.publishEvent({ type: 'graph.completed', data: { graphId } });
+      messageBus.publishEvent({ type: 'GraphProcessed', data: { graphId } });
     }
   }
 

package/src/process-graph.specs.ts

@@ -2,7 +2,7 @@ import { describe, expect, it } from 'vitest';
 import { handleProcessGraph } from './process-graph';
 
 describe('handleProcessGraph', () => {
-  it('returns graph.failed when graph validation fails', async () => {
+  it('returns GraphFailed when graph validation fails', async () => {
     const result = await handleProcessGraph({
       type: 'ProcessGraph',
       data: {
@@ -13,7 +13,7 @@ describe('handleProcessGraph', () => {
     });
 
     expect(result).toEqual({
-      type: 'graph.failed',
+      type: 'GraphFailed',
       data: {
         graphId: 'g1',
         reason: 'Graph must contain at least one job',
@@ -36,7 +36,7 @@ describe('handleProcessGraph', () => {
 
     expect(result).toEqual([
       {
-        type: 'job.dispatched',
+        type: 'JobDispatched',
         data: { graphId: 'g1', jobId: 'a', target: 'build', payload: { src: './app' } },
       },
     ]);

package/src/process-graph.ts

@@ -19,7 +19,7 @@ export async function handleProcessGraph(command: ProcessGraphCommand): Promise<
   const validation = validateGraph(jobs);
   if (!validation.valid) {
     return {
-      type: 'graph.failed',
+      type: 'GraphFailed',
       data: { graphId, reason: validation.error },
     };
   }
@@ -36,7 +36,7 @@ export async function handleProcessGraph(command: ProcessGraphCommand): Promise<
 
   const ready = getReadyJobs(state);
   return ready.map((jobId) => ({
-    type: 'job.dispatched',
+    type: 'JobDispatched',
     data: { graphId, jobId, target: jobById[jobId].target, payload: jobById[jobId].payload },
   }));
 }

package/src/process-job-graph.e2e.specs.ts

@@ -48,7 +48,7 @@ describe('ProcessJobGraph E2E', () => {
     await server.stop();
   });
 
-  it('command dispatch returns ack and produces graph.dispatching event', async () => {
+  it('command dispatch returns ack and produces GraphDispatched event', async () => {
     const server = new PipelineServer({ port: 0 });
     server.registerCommandHandlers(COMMANDS);
     await server.start();
@@ -73,18 +73,18 @@ describe('ProcessJobGraph E2E', () => {
     const messages = await fetchJson<StoredMessage[]>(`http://localhost:${server.port}/messages`);
     const eventTypes = messages.filter((m) => m.messageType === 'event').map((m) => m.message.type);
 
-    expect(eventTypes).toContain('graph.dispatching');
+    expect(eventTypes).toContain('GraphDispatched');
 
     await server.stop();
   });
 
-  it('full lifecycle produces graph.completed via correlation', async () => {
+  it('full lifecycle produces GraphProcessed via correlation', async () => {
     const server = new PipelineServer({ port: 0 });
     server.registerCommandHandlers(COMMANDS);
     await server.start();
 
     const completed: Array<{ type: string; data: Record<string, unknown> }> = [];
-    server.getMessageBus().subscribeToEvent('graph.completed', {
+    server.getMessageBus().subscribeToEvent('GraphProcessed', {
       name: 'e2e-completion-tracker',
       handle: (event) => {
         completed.push(event);
@@ -114,7 +114,7 @@ describe('ProcessJobGraph E2E', () => {
 
     await new Promise((r) => setTimeout(r, 100));
 
-    expect(completed).toEqual([{ type: 'graph.completed', data: { graphId: 'e2e-g2' } }]);
+    expect(completed).toEqual([{ type: 'GraphProcessed', data: { graphId: 'e2e-g2' } }]);
 
     await server.stop();
   });

package/ketchup-plan.md

@@ -1,31 +0,0 @@
-# Ketchup Plan: ProcessJobGraph Command Handler + E2E Tests
-
-## TODO
-
-(empty)
-
-## DONE
-
-### Bottle: Defensive input validation
-
-- [x] Burst 1: guard submit against missing jobs array (ecd82695)
-
-### Bottle: Dispatch target commands via dispatch callback
-
-- [x] Burst 1: submit sends target commands for initial ready jobs (caa7abf9)
-- [x] Burst 2: onJobEvent sends target commands for newly ready dependent jobs (0b4be016)
-- [x] Burst 3: accept dispatch callback instead of using messageBus.sendCommand (2505b01e)
-- [x] Burst 4: PipelineContext.sendCommand accepts optional correlationId override (0d858127)
-- [x] Burst 5: process-job-graph handler wires ctx.sendCommand as dispatch (41f044c5)
-
-### Bottle: Pipeline Command Handler + E2E Tests
-
-- [x] Burst 1: Command handler + test for graph.dispatching (4e876184)
-- [x] Burst 2: Test handler returns graph.failed when messageBus missing (4e876184)
-- [x] Burst 3: Test handler returns graph.failed for invalid graph (4e876184)
-- [x] Burst 4: Export COMMANDS array (772f16a4)
-- [x] Burst 5: E2E — registry shows ProcessJobGraph (8da16e8a)
-- [x] Burst 6: E2E — command callable, produces graph.dispatching (8da16e8a)
-- [x] Burst 7: E2E — full lifecycle, graph.completed via correlation (8da16e8a)
-- [x] Burst 8: Wire plugin into typical example (30ab6969)
-- [x] Burst 9: Update README (c2a955a6)