@auto-engineer/pipeline 1.131.0 → 1.134.0

package/package.json

@@ -15,8 +15,8 @@
     "jose": "^5.9.6",
     "nanoid": "^5.0.0",
     "uuid": "^11.0.0",
-    "@auto-engineer/file-store": "1.131.0",
-    "@auto-engineer/message-bus": "1.131.0"
+    "@auto-engineer/file-store": "1.134.0",
+    "@auto-engineer/message-bus": "1.134.0"
   },
   "devDependencies": {
     "@types/cors": "^2.8.17",
@@ -25,7 +25,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "1.131.0",
+  "version": "1.134.0",
   "scripts": {
     "build": "tsc && tsx ../../scripts/fix-esm-imports.ts",
     "test": "vitest run --reporter=dot",

package/src/engine/workflows/await-workflow.ts

@@ -21,7 +21,7 @@ export function initialState(): AwaitState {
   };
 }
 
-export function decide(input: AwaitInput, state: AwaitState): AwaitOutput | AwaitOutput[] {
+export function decide(_input: AwaitInput, state: AwaitState): AwaitOutput | AwaitOutput[] {
   if (state.status !== 'waiting' || state.pendingKeys.length > 0) {
     return [];
   }

package/src/engine/workflows/settled-workflow.ts

@@ -84,7 +84,7 @@ export function evolve(state: SettledState, event: SettledEvent): SettledState {
   }
 }
 
-export function decide(input: SettledInput, state: SettledState): SettledOutput | SettledOutput[] {
+export function decide(_input: SettledInput, state: SettledState): SettledOutput | SettledOutput[] {
   if (state.status !== 'waiting') {
     return [];
   }

package/src/server/phased-bridge.ts

@@ -84,7 +84,7 @@ export function createPhasedBridge(config: PhasedBridgeConfig) {
         const key = handler.completion.itemKey({ type: event.type, data });
         const phase = handler.classifier(item);
         itemRecords.push({ key, phase, original: item });
-        itemToExecution.set(key, correlationId + '|' + handler.eventType);
+        itemToExecution.set(key, `${correlationId}|${handler.eventType}`);
       }
 
       const execution: PhasedExecution = {
@@ -94,7 +94,7 @@ export function createPhasedBridge(config: PhasedBridgeConfig) {
         items: itemRecords,
         state: initialState(),
       };
-      executions.set(correlationId + '|' + handler.eventType, execution);
+      executions.set(`${correlationId}|${handler.eventType}`, execution);
 
       const startInput: PhasedInput = {
         type: 'StartPhased',

package/src/server/pipeline-server-v2.ts

@@ -29,7 +29,7 @@ export async function createPipelineServerV2(config?: { port?: number }): Promis
     res.json({ commands: engine.registeredCommands() });
   });
 
-  app.get('/events', (req, res) => {
+  app.get('/events', (_req, res) => {
     res.writeHead(200, {
       'Content-Type': 'text/event-stream',
       'Cache-Control': 'no-cache',

package/src/server/pipeline-server.specs.ts

@@ -351,6 +351,24 @@ describe('PipelineServer', () => {
       await server.stop();
     });
 
+    it('should use displayName for dynamically-added command nodes', async () => {
+      const handler = {
+        name: 'DynamicCmd',
+        displayName: 'Dynamic Command',
+        events: ['DynamicDone'],
+        handle: async () => ({ type: 'DynamicDone', data: {} }),
+      };
+      const pipeline = define('test').on('DynamicDone').emit('NextCmd', {}).build();
+      const server = new PipelineServer({ port: 0 });
+      server.registerCommandHandlers([handler]);
+      server.registerPipeline(pipeline);
+      await server.start();
+      const data = await fetchAs<GraphResponse>(`http://localhost:${server.port}/pipeline`);
+      const cmdNode = data.nodes.find((n) => n.id === 'cmd:DynamicCmd');
+      expect(cmdNode?.label).toBe('Dynamic Command');
+      await server.stop();
+    });
+
     it('should filter out event nodes when excludeTypes=event', async () => {
       const pipeline = define('test').on('Start').emit('Cmd', {}).build();
       const server = new PipelineServer({ port: 0 });
@@ -2225,7 +2243,7 @@ describe('PipelineServer', () => {
       const server = new PipelineServer({ port: 0 });
       server.registerItemKeyExtractor('ManualKeyCmd', (d) => {
         const data = d as Record<string, string>;
-        return data['customId'];
+        return data.customId;
       });
       server.registerCommandHandlers([handler]);
       server.registerPipeline(pipeline);

package/src/server/pipeline-server.ts

@@ -320,12 +320,7 @@ export class PipelineServer {
 
     this.app.get('/pipeline', (req, res) => {
       void (async () => {
-        const commandToEvents = this.buildCommandToEvents();
-        const rawGraph = this.buildCombinedGraph();
-        const pipelineEvents = this.extractPipelineEvents(rawGraph, commandToEvents);
-        const graphWithEvents = this.addCommandEventEdgesToGraph(rawGraph, commandToEvents, pipelineEvents);
-        const graphWithEnrichedEvents = this.enrichEventLabels(graphWithEvents);
-        const completeGraph = this.markBackLinks(graphWithEnrichedEvents);
+        const completeGraph = this.buildFullGraph();
         const filterOptions = this.parseFilterOptions(req.query);
         const filteredGraph = filterGraph(completeGraph, filterOptions);
         const correlationId = (req.query.correlationId as string | undefined) ?? this.currentSessionId;
@@ -559,7 +554,17 @@ export class PipelineServer {
       combinedGraph.edges.push(...graph.edges);
     }
 
-    return this.enrichCommandLabels(combinedGraph);
+    return combinedGraph;
+  }
+
+  private buildFullGraph(): GraphIR {
+    const commandToEvents = this.buildCommandToEvents();
+    const rawGraph = this.buildCombinedGraph();
+    const pipelineEvents = this.extractPipelineEvents(rawGraph, commandToEvents);
+    const graphWithEvents = this.addCommandEventEdgesToGraph(rawGraph, commandToEvents, pipelineEvents);
+    const graphWithEnrichedCommands = this.enrichCommandLabels(graphWithEvents);
+    const graphWithEnrichedEvents = this.enrichEventLabels(graphWithEnrichedCommands);
+    return this.markBackLinks(graphWithEnrichedEvents);
   }
 
   private enrichCommandLabels(graph: GraphIR): GraphIR {
@@ -568,7 +573,7 @@ export class PipelineServer {
         if (node.type !== 'command') {
           return node;
         }
-        const handler = this.commandHandlers.get(node.label);
+        const handler = this.commandHandlers.get(node.id.replace('cmd:', ''));
         if (handler?.displayName === undefined) {
           return node;
         }
@@ -585,7 +590,7 @@ export class PipelineServer {
         if (node.type !== 'event') {
           return node;
         }
-        const displayName = eventDisplayNames.get(node.label);
+        const displayName = eventDisplayNames.get(node.id.replace('evt:', ''));
         if (displayName === undefined) {
           return node;
         }
@@ -921,12 +926,7 @@ export class PipelineServer {
   }
 
   private buildMermaidDiagram(filterOptions: FilterOptions): string {
-    const commandToEvents = this.buildCommandToEvents();
-    const rawGraph = this.buildCombinedGraph();
-    const pipelineEvents = this.extractPipelineEvents(rawGraph, commandToEvents);
-    const graphWithEvents = this.addCommandEventEdgesToGraph(rawGraph, commandToEvents, pipelineEvents);
-    const graphWithEnrichedEvents = this.enrichEventLabels(graphWithEvents);
-    const completeGraph = this.markBackLinks(graphWithEnrichedEvents);
+    const completeGraph = this.buildFullGraph();
     const graph = filterGraph(completeGraph, filterOptions);
     const lines: string[] = ['flowchart LR'];