@substrate-ai/factory 0.19.54

package/dist/__tests__/integration/stylesheet-application.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"stylesheet-application.test.d.ts","sourceRoot":"","sources":["../../../src/__tests__/integration/stylesheet-application.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG"}

package/dist/__tests__/integration/stylesheet-application.test.js

@@ -0,0 +1,119 @@
+/**
+ * Integration test: AC6 — model stylesheet properties applied before handler dispatch
+ *
+ * Verifies that the stylesheet resolver correctly applies per-node LLM routing
+ * properties with specificity-based precedence. The test applies the stylesheet
+ * resolver to graph nodes before execution so that spy handlers receive the
+ * resolved node attributes.
+ *
+ * Story 42-15.
+ */
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { parseGraph } from '../../graph/parser.js';
+import { createValidator } from '../../graph/validator.js';
+import { createGraphExecutor } from '../../graph/executor.js';
+import { parseStylesheet } from '../../stylesheet/parser.js';
+import { resolveNodeStyles } from '../../stylesheet/resolver.js';
+import { makeTmpDir, cleanDir, makeMockRegistry, makeEventSpy } from './helpers.js';
+import { STYLESHEET_DOT } from './graphs.js';
+describe('AC6: model stylesheet properties applied before handler dispatch', () => {
+    let logsRoot;
+    beforeEach(async () => {
+        logsRoot = await makeTmpDir();
+    });
+    afterEach(async () => {
+        await cleanDir(logsRoot);
+    });
+    it('parses graph with non-empty modelStylesheet', () => {
+        const graph = parseGraph(STYLESHEET_DOT);
+        expect(graph.modelStylesheet).not.toBe('');
+        expect(graph.modelStylesheet).toContain('claude-3-haiku-20240307');
+        expect(graph.modelStylesheet).toContain('claude-opus-4-5');
+    });
+    it('stylesheet parser resolves analyze to opus and summarize to haiku (unit check)', () => {
+        const graph = parseGraph(STYLESHEET_DOT);
+        const stylesheet = parseStylesheet(graph.modelStylesheet);
+        const analyzeNode = graph.nodes.get('analyze');
+        const summarizeNode = graph.nodes.get('summarize');
+        const analyzeResolved = resolveNodeStyles(analyzeNode, stylesheet);
+        const summarizeResolved = resolveNodeStyles(summarizeNode, stylesheet);
+        expect(analyzeResolved.llmModel).toBe('claude-opus-4-5');
+        expect(summarizeResolved.llmModel).toBe('claude-3-haiku-20240307');
+    });
+    it('id-level specificity wins over universal selector for analyze node', () => {
+        const graph = parseGraph(STYLESHEET_DOT);
+        const stylesheet = parseStylesheet(graph.modelStylesheet);
+        const analyzeNode = graph.nodes.get('analyze');
+        const resolved = resolveNodeStyles(analyzeNode, stylesheet);
+        // #analyze rule (specificity=3) overrides * rule (specificity=0)
+        expect(resolved.llmModel).toBe('claude-opus-4-5');
+    });
+    it('universal selector applies to summarize (no id override)', () => {
+        const graph = parseGraph(STYLESHEET_DOT);
+        const stylesheet = parseStylesheet(graph.modelStylesheet);
+        const summarizeNode = graph.nodes.get('summarize');
+        const resolved = resolveNodeStyles(summarizeNode, stylesheet);
+        // Only * rule applies (no #summarize rule)
+        expect(resolved.llmModel).toBe('claude-3-haiku-20240307');
+    });
+    it('spy handler for analyze receives node with llmModel=claude-opus-4-5 after stylesheet application', async () => {
+        const graph = parseGraph(STYLESHEET_DOT);
+        // Apply stylesheet to graph nodes before running the executor
+        // (integration between the stylesheet resolver and the execution layer)
+        const stylesheet = parseStylesheet(graph.modelStylesheet);
+        for (const node of graph.nodes.values()) {
+            const resolved = resolveNodeStyles(node, stylesheet);
+            // Explicit node attribute takes priority (merge: explicit wins over resolved).
+            // Cast through unknown to mutate the plain object (buildGraphNode returns a mutable struct).
+            const mutableNode = node;
+            if (!node.llmModel && resolved.llmModel) {
+                mutableNode['llmModel'] = resolved.llmModel;
+            }
+            if (!node.llmProvider && resolved.llmProvider) {
+                mutableNode['llmProvider'] = resolved.llmProvider;
+            }
+            if (!node.reasoningEffort && resolved.reasoningEffort) {
+                mutableNode['reasoningEffort'] = resolved.reasoningEffort;
+            }
+        }
+        // Verify the mutation was applied
+        expect(graph.nodes.get('analyze').llmModel).toBe('claude-opus-4-5');
+        expect(graph.nodes.get('summarize').llmModel).toBe('claude-3-haiku-20240307');
+        // --- Build spy registry ---
+        const { registry, spies } = makeMockRegistry();
+        const { bus } = makeEventSpy();
+        // --- Execute ---
+        const outcome = await createGraphExecutor().run(graph, {
+            runId: 'test-run-ac6',
+            logsRoot,
+            handlerRegistry: registry,
+            eventBus: bus,
+        });
+        expect(outcome.status).toBe('SUCCESS');
+        // --- Verify spy handler received node with stylesheet-applied llmModel ---
+        const analyzeSpy = spies.get('analyze');
+        const summarizeSpy = spies.get('summarize');
+        expect(analyzeSpy).toBeDefined();
+        expect(summarizeSpy).toBeDefined();
+        // First argument to the handler is the GraphNode
+        const analyzeNode = analyzeSpy.mock.calls[0][0];
+        const summarizeNode = summarizeSpy.mock.calls[0][0];
+        expect(analyzeNode.llmModel).toBe('claude-opus-4-5');
+        expect(summarizeNode.llmModel).toBe('claude-3-haiku-20240307');
+    });
+    it('executor runs the stylesheet graph to SUCCESS with all nodes executing', async () => {
+        const graph = parseGraph(STYLESHEET_DOT);
+        // Validate: no errors
+        const validator = createValidator();
+        const errors = validator.validate(graph).filter((d) => d.severity === 'error');
+        expect(errors).toHaveLength(0);
+        const { registry } = makeMockRegistry();
+        const outcome = await createGraphExecutor().run(graph, {
+            runId: 'test-run-ac6-simple',
+            logsRoot,
+            handlerRegistry: registry,
+        });
+        expect(outcome.status).toBe('SUCCESS');
+    });
+});
+//# sourceMappingURL=stylesheet-application.test.js.map

package/dist/__tests__/integration/stylesheet-application.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"stylesheet-application.test.js","sourceRoot":"","sources":["../../../src/__tests__/integration/stylesheet-application.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AACpE,OAAO,EAAE,UAAU,EAAE,MAAM,uBAAuB,CAAA;AAClD,OAAO,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAA;AAC1D,OAAO,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAA;AAC7D,OAAO,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAA;AAC5D,OAAO,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAA;AAChE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,gBAAgB,EAAE,YAAY,EAAE,MAAM,cAAc,CAAA;AACnF,OAAO,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AAG5C,QAAQ,CAAC,kEAAkE,EAAE,GAAG,EAAE;IAChF,IAAI,QAAgB,CAAA;IAEpB,UAAU,CAAC,KAAK,IAAI,EAAE;QACpB,QAAQ,GAAG,MAAM,UAAU,EAAE,CAAA;IAC/B,CAAC,CAAC,CAAA;IAEF,SAAS,CAAC,KAAK,IAAI,EAAE;QACnB,MAAM,QAAQ,CAAC,QAAQ,CAAC,CAAA;IAC1B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,KAAK,GAAG,UAAU,CAAC,cAAc,CAAC,CAAA;QACxC,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QAC1C,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,yBAAyB,CAAC,CAAA;QAClE,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAA;IAC5D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE;QACxF,MAAM,KAAK,GAAG,UAAU,CAAC,cAAc,CAAC,CAAA;QACxC,MAAM,UAAU,GAAG,eAAe,CAAC,KAAK,CAAC,eAAe,CAAC,CAAA;QAEzD,MAAM,WAAW,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAE,CAAA;QAC/C,MAAM,aAAa,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,WAAW,CAAE,CAAA;QAEnD,MAAM,eAAe,GAAG,iBAAiB,CAAC,WAAW,EAAE,UAAU,CAAC,CAAA;QAClE,MAAM,iBAAiB,GAAG,iBAAiB,CAAC,aAAa,EAAE,UAAU,CAAC,CAAA;QAEtE,MAAM,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;QACxD,MAAM,CAAC,iBAAiB,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAA;IACpE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oEAAoE,EAAE,GAAG,EAAE;QAC5E,MAAM,KAAK,GAAG,UAAU,CAAC,cAAc,CAAC,CAAA;QACxC,MAAM,UAAU,GAAG,eAAe,CAAC,KAAK,CAAC,eAAe,CAAC,CAAA;QAEzD,MAAM,WAAW,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAE,CAAA;QAC/C,MAAM,QAAQ,GAAG,iBAAiB,CAAC,WAAW,EAAE,UAAU,CAAC,CAAA;QAE3D,iEAAiE;QACjE,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;IACnD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,MAAM,KAAK,GAAG,UAAU,CAAC,cAAc,CAAC,CAAA;QACxC,MAAM,UAAU,GAAG,eAAe,CAAC,KAAK,CAAC,eAAe,CAAC,CAAA;QAEzD,MAAM,aAAa,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,WAAW,CAAE,CAAA;QACnD,MAAM,QAAQ,GAAG,iBAAiB,CAAC,aAAa,EAAE,UAAU,CAAC,CAAA;QAE7D,2CAA2C;QAC3C,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAA;IAC3D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kGAAkG,EAAE,KAAK,IAAI,EAAE;QAChH,MAAM,KAAK,GAAG,UAAU,CAAC,cAAc,CAAC,CAAA;QAExC,8DAA8D;QAC9D,wEAAwE;QACxE,MAAM,UAAU,GAAG,eAAe,CAAC,KAAK,CAAC,eAAe,CAAC,CAAA;QACzD,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC;YACxC,MAAM,QAAQ,GAAG,iBAAiB,CAAC,IAAI,EAAE,UAAU,CAAC,CAAA;YACpD,+EAA+E;YAC/E,6FAA6F;YAC7F,MAAM,WAAW,GAAG,IAA0C,CAAA;YAC9D,IAAI,CAAC,IAAI,CAAC,QAAQ,IAAI,QAAQ,CAAC,QAAQ,EAAE,CAAC;gBACxC,WAAW,CAAC,UAAU,CAAC,GAAG,QAAQ,CAAC,QAAQ,CAAA;YAC7C,CAAC;YACD,IAAI,CAAC,IAAI,CAAC,WAAW,IAAI,QAAQ,CAAC,WAAW,EAAE,CAAC;gBAC9C,WAAW,CAAC,aAAa,CAAC,GAAG,QAAQ,CAAC,WAAW,CAAA;YACnD,CAAC;YACD,IAAI,CAAC,IAAI,CAAC,eAAe,IAAI,QAAQ,CAAC,eAAe,EAAE,CAAC;gBACtD,WAAW,CAAC,iBAAiB,CAAC,GAAG,QAAQ,CAAC,eAAe,CAAA;YAC3D,CAAC;QACH,CAAC;QAED,kCAAkC;QAClC,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAE,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;QACpE,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,WAAW,CAAE,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAA;QAE9E,6BAA6B;QAC7B,MAAM,EAAE,QAAQ,EAAE,KAAK,EAAE,GAAG,gBAAgB,EAAE,CAAA;QAC9C,MAAM,EAAE,GAAG,EAAE,GAAG,YAAY,EAAE,CAAA;QAE9B,kBAAkB;QAClB,MAAM,OAAO,GAAG,MAAM,mBAAmB,EAAE,CAAC,GAAG,CAAC,KAAK,EAAE;YACrD,KAAK,EAAE,cAAc;YACrB,QAAQ;YACR,eAAe,EAAE,QAAQ;YACzB,QAAQ,EAAE,GAAG;SACd,CAAC,CAAA;QAEF,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QAEtC,4EAA4E;QAC5E,MAAM,UAAU,GAAG,KAAK,CAAC,GAAG,CAAC,SAAS,CAAC,CAAA;QACvC,MAAM,YAAY,GAAG,KAAK,CAAC,GAAG,CAAC,WAAW,CAAC,CAAA;QAE3C,MAAM,CAAC,UAAU,CAAC,CAAC,WAAW,EAAE,CAAA;QAChC,MAAM,CAAC,YAAY,CAAC,CAAC,WAAW,EAAE,CAAA;QAElC,iDAAiD;QACjD,MAAM,WAAW,GAAG,UAAW,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,CAAC,CAAc,CAAA;QAC9D,MAAM,aAAa,GAAG,YAAa,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,CAAC,CAAc,CAAA;QAElE,MAAM,CAAC,WAAW,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;QACpD,MAAM,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAA;IAChE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wEAAwE,EAAE,KAAK,IAAI,EAAE;QACtF,MAAM,KAAK,GAAG,UAAU,CAAC,cAAc,CAAC,CAAA;QAExC,sBAAsB;QACtB,MAAM,SAAS,GAAG,eAAe,EAAE,CAAA;QACnC,MAAM,MAAM,GAAG,SAAS,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAA;QAC9E,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAE9B,MAAM,EAAE,QAAQ,EAAE,GAAG,gBAAgB,EAAE,CAAA;QACvC,MAAM,OAAO,GAAG,MAAM,mBAAmB,EAAE,CAAC,GAAG,CAAC,KAAK,EAAE;YACrD,KAAK,EAAE,qBAAqB;YAC5B,QAAQ;YACR,eAAe,EAAE,QAAQ;SAC1B,CAAC,CAAA;QAEF,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/__tests__/integration/subgraph-execution.test.d.ts

@@ -0,0 +1,24 @@
+/**
+ * Integration tests for subgraph execution handler.
+ * Story 50-11 AC3.
+ *
+ * Tests cover:
+ *   - Basic subgraph execution (happy path → SUCCESS)
+ *   - Missing graph_file attribute → FAILURE
+ *   - graphFileLoader throws → FAILURE with load error
+ *   - Invalid child DOT → FAILURE with parse error
+ *   - Max-depth guard → FAILURE when depth >= maxDepth
+ *   - graph:subgraph-started emitted with correct payload
+ *   - graph:subgraph-completed emitted on success
+ *   - Event ordering (started before completed)
+ *   - Absent eventBus handling
+ *   - runId resolution from context.__runId
+ *   - Custom maxDepth option honoured
+ *
+ * Test approach: invoke `createSubgraphHandler(options)` directly with an
+ * injectable `graphFileLoader` — no real disk access.
+ *
+ * ≥12 `it(...)` cases required (AC7).
+ */
+export {};
+//# sourceMappingURL=subgraph-execution.test.d.ts.map

package/dist/__tests__/integration/subgraph-execution.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"subgraph-execution.test.d.ts","sourceRoot":"","sources":["../../../src/__tests__/integration/subgraph-execution.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG"}

package/dist/__tests__/integration/subgraph-execution.test.js

@@ -0,0 +1,291 @@
+/**
+ * Integration tests for subgraph execution handler.
+ * Story 50-11 AC3.
+ *
+ * Tests cover:
+ *   - Basic subgraph execution (happy path → SUCCESS)
+ *   - Missing graph_file attribute → FAILURE
+ *   - graphFileLoader throws → FAILURE with load error
+ *   - Invalid child DOT → FAILURE with parse error
+ *   - Max-depth guard → FAILURE when depth >= maxDepth
+ *   - graph:subgraph-started emitted with correct payload
+ *   - graph:subgraph-completed emitted on success
+ *   - Event ordering (started before completed)
+ *   - Absent eventBus handling
+ *   - runId resolution from context.__runId
+ *   - Custom maxDepth option honoured
+ *
+ * Test approach: invoke `createSubgraphHandler(options)` directly with an
+ * injectable `graphFileLoader` — no real disk access.
+ *
+ * ≥12 `it(...)` cases required (AC7).
+ */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import os from 'node:os';
+import { parseGraph } from '../../graph/parser.js';
+import { GraphContext } from '../../graph/context.js';
+import { HandlerRegistry } from '../../handlers/registry.js';
+import { startHandler } from '../../handlers/start.js';
+import { exitHandler } from '../../handlers/exit.js';
+import { createSubgraphHandler } from '../../handlers/subgraph.js';
+import { SUBGRAPH_PARENT_DOT, CHILD_GRAPH_DOT, } from '../fixtures/subgraph-parent.dot.js';
+function makeEventSpy() {
+    const events = [];
+    const bus = {
+        emit(event, payload) {
+            events.push({ event: event, payload });
+        },
+        on() { },
+        off() { },
+    };
+    return { bus, events };
+}
+/**
+ * Build a HandlerRegistry suitable for executing the child DOT graphs in tests.
+ * Registers start/exit plus a default SUCCESS handler for any other node type.
+ */
+function makeChildRegistry() {
+    const registry = new HandlerRegistry();
+    registry.register('start', startHandler);
+    registry.register('exit', exitHandler);
+    registry.setDefault(vi.fn().mockResolvedValue({ status: 'SUCCESS' }));
+    return registry;
+}
+/** Parsed parent graph with sg_node.attrs post-processed for a given graph_file path. */
+function makeParentGraph(graphFile = '/test/child.dot') {
+    const graph = parseGraph(SUBGRAPH_PARENT_DOT);
+    const sgNode = graph.nodes.get('sg_node');
+    sgNode.attrs = { graph_file: graphFile };
+    return { graph, sgNodeId: 'sg_node' };
+}
+// ---------------------------------------------------------------------------
+// AC3: Subgraph basic execution
+// ---------------------------------------------------------------------------
+describe('subgraph execution — basic success path', () => {
+    it('handler returns SUCCESS when child graph executes successfully', async () => {
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const registry = makeChildRegistry();
+        const handler = createSubgraphHandler({
+            handlerRegistry: registry,
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+        });
+        const context = new GraphContext({ __runId: 'test-sg-basic' });
+        const outcome = await handler(sgNode, context, graph);
+        expect(outcome.status).toBe('SUCCESS');
+    });
+    it('handler returns FAILURE when node.attrs is missing graph_file', async () => {
+        const graph = parseGraph(SUBGRAPH_PARENT_DOT);
+        const sgNode = graph.nodes.get('sg_node');
+        // Intentionally do NOT set sgNode.attrs
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+        });
+        const context = new GraphContext({ __runId: 'test-no-attr' });
+        const outcome = await handler(sgNode, context, graph);
+        expect(outcome.status).toBe('FAILURE');
+        expect(outcome.failureReason).toContain('missing required attribute graph_file');
+    });
+    it('handler returns FAILURE when graphFileLoader throws', async () => {
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const failingLoader = vi.fn().mockRejectedValue(new Error('file not found'));
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: failingLoader,
+        });
+        const context = new GraphContext({ __runId: 'test-loader-fail' });
+        const outcome = await handler(sgNode, context, graph);
+        expect(outcome.status).toBe('FAILURE');
+        expect(outcome.failureReason).toContain('failed to load');
+        expect(outcome.failureReason).toContain('file not found');
+    });
+    it('handler returns FAILURE when child DOT string is invalid (parse error)', async () => {
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue('THIS IS NOT VALID DOT SYNTAX @@@###'),
+        });
+        const context = new GraphContext({ __runId: 'test-invalid-dot' });
+        const outcome = await handler(sgNode, context, graph);
+        expect(outcome.status).toBe('FAILURE');
+        expect(outcome.failureReason).toContain('failed to parse');
+    });
+});
+// ---------------------------------------------------------------------------
+// Depth guard tests
+// ---------------------------------------------------------------------------
+describe('subgraph execution — depth guard', () => {
+    it('returns FAILURE when subgraph._depth >= maxDepth (default 5)', async () => {
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            // maxDepth defaults to 5
+        });
+        // Set depth to 5 — equals maxDepth (5) → guard fires
+        const context = new GraphContext({ __runId: 'test-depth-max', 'subgraph._depth': 5 });
+        const outcome = await handler(sgNode, context, graph);
+        expect(outcome.status).toBe('FAILURE');
+        expect(outcome.failureReason).toContain('depth limit exceeded');
+    });
+    it('custom maxDepth=2: depth=1 succeeds, depth=2 fails', async () => {
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            maxDepth: 2,
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+        });
+        // depth=1 < maxDepth=2 → should succeed
+        const contextOk = new GraphContext({ __runId: 'test-custom-depth-ok', 'subgraph._depth': 1 });
+        const successOutcome = await handler(sgNode, contextOk, graph);
+        expect(successOutcome.status).toBe('SUCCESS');
+        // depth=2 >= maxDepth=2 → should fail
+        const contextFail = new GraphContext({ __runId: 'test-custom-depth-fail', 'subgraph._depth': 2 });
+        const failOutcome = await handler(sgNode, contextFail, graph);
+        expect(failOutcome.status).toBe('FAILURE');
+        expect(failOutcome.failureReason).toContain('depth limit exceeded');
+    });
+});
+// ---------------------------------------------------------------------------
+// Event emission tests
+// ---------------------------------------------------------------------------
+describe('subgraph execution — event emission', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+    });
+    it('graph:subgraph-started emitted once with correct nodeId', async () => {
+        const { bus, events } = makeEventSpy();
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+            eventBus: bus,
+            runId: 'evt-test',
+        });
+        const context = new GraphContext({ __runId: 'evt-test' });
+        await handler(sgNode, context, graph);
+        const started = events.filter((e) => e.event === 'graph:subgraph-started');
+        expect(started).toHaveLength(1);
+        expect(started[0].payload.nodeId).toBe('sg_node');
+    });
+    it('graph:subgraph-completed emitted once on success', async () => {
+        const { bus, events } = makeEventSpy();
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+            eventBus: bus,
+            runId: 'evt-test',
+        });
+        const context = new GraphContext({ __runId: 'evt-test' });
+        await handler(sgNode, context, graph);
+        const completed = events.filter((e) => e.event === 'graph:subgraph-completed');
+        expect(completed).toHaveLength(1);
+        expect(completed[0].payload.status).toBe('SUCCESS');
+    });
+    it('graph:subgraph-completed has durationMs >= 0', async () => {
+        const { bus, events } = makeEventSpy();
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+            eventBus: bus,
+        });
+        const context = new GraphContext({ __runId: 'evt-duration' });
+        await handler(sgNode, context, graph);
+        const completed = events.filter((e) => e.event === 'graph:subgraph-completed');
+        const durationMs = completed[0].payload.durationMs;
+        expect(typeof durationMs).toBe('number');
+        expect(durationMs).toBeGreaterThanOrEqual(0);
+    });
+    it('graph:subgraph-started emitted before graph:subgraph-completed', async () => {
+        const { bus, events } = makeEventSpy();
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+            eventBus: bus,
+        });
+        const context = new GraphContext({ __runId: 'evt-order' });
+        await handler(sgNode, context, graph);
+        const types = events.map((e) => e.event);
+        const startIdx = types.indexOf('graph:subgraph-started');
+        const endIdx = types.indexOf('graph:subgraph-completed');
+        expect(startIdx).toBeGreaterThanOrEqual(0);
+        expect(endIdx).toBeGreaterThanOrEqual(0);
+        expect(startIdx).toBeLessThan(endIdx);
+    });
+    it('graph:subgraph-started carries depth=0 for un-nested (root) subgraph', async () => {
+        const { bus, events } = makeEventSpy();
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+            eventBus: bus,
+        });
+        // No subgraph._depth in context → defaults to 0
+        const context = new GraphContext({ __runId: 'evt-depth-root' });
+        await handler(sgNode, context, graph);
+        const started = events.find((e) => e.event === 'graph:subgraph-started');
+        expect(started.payload.depth).toBe(0);
+    });
+    it('runId from context.__runId flows into subgraph event payloads', async () => {
+        const { bus, events } = makeEventSpy();
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+            eventBus: bus,
+            // No runId in options — should fall back to context.__runId
+        });
+        const context = new GraphContext({ __runId: 'my-run-42' });
+        await handler(sgNode, context, graph);
+        const started = events.find((e) => e.event === 'graph:subgraph-started');
+        expect(started.payload.runId).toBe('my-run-42');
+    });
+    it('absent eventBus: handler completes without crash', async () => {
+        const { graph, sgNodeId } = makeParentGraph();
+        const sgNode = graph.nodes.get(sgNodeId);
+        const handler = createSubgraphHandler({
+            handlerRegistry: makeChildRegistry(),
+            baseDir: '/test',
+            graphFileLoader: vi.fn().mockResolvedValue(CHILD_GRAPH_DOT),
+            logsRoot: os.tmpdir(),
+            // No eventBus
+        });
+        const context = new GraphContext({ __runId: 'no-bus' });
+        await expect(handler(sgNode, context, graph)).resolves.toMatchObject({ status: 'SUCCESS' });
+    });
+});
+//# sourceMappingURL=subgraph-execution.test.js.map

package/dist/__tests__/integration/subgraph-execution.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"subgraph-execution.test.js","sourceRoot":"","sources":["../../../src/__tests__/integration/subgraph-execution.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAA;AAC7D,OAAO,EAAE,MAAM,SAAS,CAAA;AAExB,OAAO,EAAE,UAAU,EAAE,MAAM,uBAAuB,CAAA;AAClD,OAAO,EAAE,YAAY,EAAE,MAAM,wBAAwB,CAAA;AACrD,OAAO,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAA;AAC5D,OAAO,EAAE,YAAY,EAAE,MAAM,yBAAyB,CAAA;AACtD,OAAO,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAA;AACpD,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAA;AAKlE,OAAO,EACL,mBAAmB,EACnB,eAAe,GAChB,MAAM,oCAAoC,CAAA;AAY3C,SAAS,YAAY;IACnB,MAAM,MAAM,GAAe,EAAE,CAAA;IAC7B,MAAM,GAAG,GAAiC;QACxC,IAAI,CAAgC,KAAQ,EAAE,OAAyB;YACrE,MAAM,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,KAAe,EAAE,OAAO,EAAE,CAAC,CAAA;QAClD,CAAC;QACD,EAAE,KAAU,CAAC;QACb,GAAG,KAAU,CAAC;KACf,CAAA;IACD,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,CAAA;AACxB,CAAC;AAED;;;GAGG;AACH,SAAS,iBAAiB;IACxB,MAAM,QAAQ,GAAG,IAAI,eAAe,EAAE,CAAA;IACtC,QAAQ,CAAC,QAAQ,CAAC,OAAO,EAAE,YAAY,CAAC,CAAA;IACxC,QAAQ,CAAC,QAAQ,CAAC,MAAM,EAAE,WAAW,CAAC,CAAA;IACtC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC,CAAA;IACrE,OAAO,QAAQ,CAAA;AACjB,CAAC;AAED,yFAAyF;AACzF,SAAS,eAAe,CAAC,SAAS,GAAG,iBAAiB;IACpD,MAAM,KAAK,GAAG,UAAU,CAAC,mBAAmB,CAAC,CAAA;IAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAE,CAAA;IAC1C,MAAM,CAAC,KAAK,GAAG,EAAE,UAAU,EAAE,SAAS,EAAE,CAAA;IACxC,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,SAAS,EAAE,CAAA;AACvC,CAAC;AAED,8EAA8E;AAC9E,gCAAgC;AAChC,8EAA8E;AAE9E,QAAQ,CAAC,yCAAyC,EAAE,GAAG,EAAE;IACvD,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QACzC,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QAEpC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,QAAQ;YACzB,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;SACtB,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,eAAe,EAAE,CAAC,CAAA;QAC9D,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErD,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,KAAK,GAAG,UAAU,CAAC,mBAAmB,CAAC,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAE,CAAA;QAC1C,wCAAwC;QAExC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;SAC5D,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,cAAc,EAAE,CAAC,CAAA;QAC7D,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErD,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QACtC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,SAAS,CAAC,uCAAuC,CAAC,CAAA;IAClF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,aAAa,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,gBAAgB,CAAC,CAAC,CAAA;QAC5E,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,aAAa;SAC/B,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,kBAAkB,EAAE,CAAC,CAAA;QACjE,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErD,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QACtC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAA;QACzD,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAA;IAC3D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wEAAwE,EAAE,KAAK,IAAI,EAAE;QACtF,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,qCAAqC,CAAC;SAClF,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,kBAAkB,EAAE,CAAC,CAAA;QACjE,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErD,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QACtC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAA;IAC5D,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,8EAA8E;AAC9E,oBAAoB;AACpB,8EAA8E;AAE9E,QAAQ,CAAC,kCAAkC,EAAE,GAAG,EAAE;IAChD,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,yBAAyB;SAC1B,CAAC,CAAA;QAEF,qDAAqD;QACrD,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,CAAC,EAAE,CAAC,CAAA;QACrF,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErD,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QACtC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAA;IACjE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,QAAQ,EAAE,CAAC;YACX,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;SACtB,CAAC,CAAA;QAEF,wCAAwC;QACxC,MAAM,SAAS,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,sBAAsB,EAAE,iBAAiB,EAAE,CAAC,EAAE,CAAC,CAAA;QAC7F,MAAM,cAAc,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,SAAS,EAAE,KAAK,CAAC,CAAA;QAC9D,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QAE7C,sCAAsC;QACtC,MAAM,WAAW,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,wBAAwB,EAAE,iBAAiB,EAAE,CAAC,EAAE,CAAC,CAAA;QACjG,MAAM,WAAW,GAAG,MAAM,OAAO,CAAC,MAAM,EAAE,WAAW,EAAE,KAAK,CAAC,CAAA;QAC7D,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QAC1C,MAAM,CAAC,WAAW,CAAC,aAAa,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAA;IACrE,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,8EAA8E;AAC9E,uBAAuB;AACvB,8EAA8E;AAE9E,QAAQ,CAAC,qCAAqC,EAAE,GAAG,EAAE;IACnD,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAA;IACpB,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,YAAY,EAAE,CAAA;QACtC,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;YACrB,QAAQ,EAAE,GAAG;YACb,KAAK,EAAE,UAAU;SAClB,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAA;QACzD,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErC,MAAM,OAAO,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,wBAAwB,CAAC,CAAA;QAC1E,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAC/B,MAAM,CAAE,OAAO,CAAC,CAAC,CAAE,CAAC,OAA8B,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IAC5E,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;QAChE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,YAAY,EAAE,CAAA;QACtC,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;YACrB,QAAQ,EAAE,GAAG;YACb,KAAK,EAAE,UAAU;SAClB,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAA;QACzD,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErC,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,0BAA0B,CAAC,CAAA;QAC9E,MAAM,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACjC,MAAM,CAAE,SAAS,CAAC,CAAC,CAAE,CAAC,OAA8B,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;IAC9E,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,YAAY,EAAE,CAAA;QACtC,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;YACrB,QAAQ,EAAE,GAAG;SACd,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,cAAc,EAAE,CAAC,CAAA;QAC7D,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErC,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,0BAA0B,CAAC,CAAA;QAC9E,MAAM,UAAU,GAAI,SAAS,CAAC,CAAC,CAAE,CAAC,OAAkC,CAAC,UAAU,CAAA;QAC/E,MAAM,CAAC,OAAO,UAAU,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACxC,MAAM,CAAC,UAAU,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAA;IAC9C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,YAAY,EAAE,CAAA;QACtC,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;YACrB,QAAQ,EAAE,GAAG;SACd,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC,CAAA;QAC1D,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErC,MAAM,KAAK,GAAG,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAA;QACxC,MAAM,QAAQ,GAAG,KAAK,CAAC,OAAO,CAAC,wBAAwB,CAAC,CAAA;QACxD,MAAM,MAAM,GAAG,KAAK,CAAC,OAAO,CAAC,0BAA0B,CAAC,CAAA;QACxD,MAAM,CAAC,QAAQ,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAA;QAC1C,MAAM,CAAC,MAAM,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAA;QACxC,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,CAAA;IACvC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACpF,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,YAAY,EAAE,CAAA;QACtC,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;YACrB,QAAQ,EAAE,GAAG;SACd,CAAC,CAAA;QAEF,gDAAgD;QAChD,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,CAAA;QAC/D,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErC,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,wBAAwB,CAAC,CAAA;QACxE,MAAM,CAAE,OAAQ,CAAC,OAA6B,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAC/D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,YAAY,EAAE,CAAA;QACtC,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;YACrB,QAAQ,EAAE,GAAG;YACb,4DAA4D;SAC7D,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC,CAAA;QAC1D,MAAM,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAA;QAErC,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,KAAK,wBAAwB,CAAC,CAAA;QACxE,MAAM,CAAE,OAAQ,CAAC,OAA6B,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;IACzE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;QAChE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,EAAE,CAAA;QAC7C,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAE,CAAA;QAEzC,MAAM,OAAO,GAAG,qBAAqB,CAAC;YACpC,eAAe,EAAE,iBAAiB,EAAE;YACpC,OAAO,EAAE,OAAO;YAChB,eAAe,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC3D,QAAQ,EAAE,EAAE,CAAC,MAAM,EAAE;YACrB,cAAc;SACf,CAAC,CAAA;QAEF,MAAM,OAAO,GAAG,IAAI,YAAY,CAAC,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,CAAA;QACvD,MAAM,MAAM,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,KAAK,CAAC,CAAC,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAA;IAC7F,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/__tests__/integration/validation-errors.test.d.ts

@@ -0,0 +1,8 @@
+/**
+ * Integration tests: AC3 — error rules block execution
+ *                    AC4 — warning rules allow execution
+ *
+ * Story 42-15.
+ */
+export {};
+//# sourceMappingURL=validation-errors.test.d.ts.map

package/dist/__tests__/integration/validation-errors.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation-errors.test.d.ts","sourceRoot":"","sources":["../../../src/__tests__/integration/validation-errors.test.ts"],"names":[],"mappings":"AAAA;;;;;GAKG"}

package/dist/__tests__/integration/validation-errors.test.js

@@ -0,0 +1,150 @@
+/**
+ * Integration tests: AC3 — error rules block execution
+ *                    AC4 — warning rules allow execution
+ *
+ * Story 42-15.
+ */
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { parseGraph } from '../../graph/parser.js';
+import { createValidator } from '../../graph/validator.js';
+import { createGraphExecutor } from '../../graph/executor.js';
+import { makeTmpDir, cleanDir, makeMockRegistry } from './helpers.js';
+import { ERROR_RULE_VIOLATION_DOT, WARNING_RULE_VIOLATION_DOT } from './graphs.js';
+// ---------------------------------------------------------------------------
+// runWithValidation helper
+// ---------------------------------------------------------------------------
+/**
+ * Parse `dotString`, validate the graph, throw if any error-level diagnostics
+ * are found, otherwise run the executor with `config`.
+ *
+ * @throws Error with "validation errors" in the message when the graph has errors.
+ */
+async function runWithValidation(dotString, config) {
+    const graph = parseGraph(dotString);
+    const validator = createValidator();
+    const errors = validator.validate(graph).filter((d) => d.severity === 'error');
+    if (errors.length > 0) {
+        const ruleIds = errors.map((e) => e.ruleId).join(', ');
+        throw new Error(`Graph has validation errors: ${ruleIds}`);
+    }
+    // Cast needed: executor returns events.ts:Outcome (StageStatus) but test
+    // types are compatible for the SUCCESS/FAIL cases we assert.
+    return createGraphExecutor().run(graph, config);
+}
+// ---------------------------------------------------------------------------
+// AC3: Error-rule violations block execution
+// ---------------------------------------------------------------------------
+describe('AC3: error-rule violations block execution', () => {
+    let logsRoot;
+    beforeEach(async () => {
+        logsRoot = await makeTmpDir();
+    });
+    afterEach(async () => {
+        await cleanDir(logsRoot);
+    });
+    it('validate() returns at least 2 error diagnostics naming reachability and start_no_incoming', () => {
+        const graph = parseGraph(ERROR_RULE_VIOLATION_DOT);
+        const validator = createValidator();
+        const diagnostics = validator.validate(graph);
+        const errors = diagnostics.filter((d) => d.severity === 'error');
+        expect(errors.length).toBeGreaterThanOrEqual(2);
+        const ruleIds = errors.map((d) => d.ruleId);
+        expect(ruleIds).toContain('reachability');
+        expect(ruleIds).toContain('start_no_incoming');
+    });
+    it('reachability error correctly identifies the orphan node', () => {
+        const graph = parseGraph(ERROR_RULE_VIOLATION_DOT);
+        const validator = createValidator();
+        const diagnostics = validator.validate(graph);
+        const reachabilityErrors = diagnostics.filter((d) => d.ruleId === 'reachability');
+        expect(reachabilityErrors.length).toBeGreaterThanOrEqual(1);
+        const orphanError = reachabilityErrors.find((d) => d.nodeId === 'orphan');
+        expect(orphanError).toBeDefined();
+        expect(orphanError?.severity).toBe('error');
+    });
+    it('start_no_incoming error correctly identifies the violating edge', () => {
+        const graph = parseGraph(ERROR_RULE_VIOLATION_DOT);
+        const validator = createValidator();
+        const diagnostics = validator.validate(graph);
+        const incomingErrors = diagnostics.filter((d) => d.ruleId === 'start_no_incoming');
+        expect(incomingErrors.length).toBeGreaterThanOrEqual(1);
+        expect(incomingErrors[0]?.severity).toBe('error');
+    });
+    it('runWithValidation throws with /validation errors/ for error graph', async () => {
+        const { registry } = makeMockRegistry();
+        await expect(runWithValidation(ERROR_RULE_VIOLATION_DOT, {
+            runId: 'test-run-ac3',
+            logsRoot,
+            handlerRegistry: registry,
+        })).rejects.toThrow(/validation errors/);
+    });
+    it('runWithValidation thrown message includes rule IDs', async () => {
+        const { registry } = makeMockRegistry();
+        let thrownMessage = '';
+        try {
+            await runWithValidation(ERROR_RULE_VIOLATION_DOT, {
+                runId: 'test-run-ac3-msg',
+                logsRoot,
+                handlerRegistry: registry,
+            });
+        }
+        catch (err) {
+            thrownMessage = err.message;
+        }
+        expect(thrownMessage).toMatch(/validation errors/);
+        expect(thrownMessage).toMatch(/reachability|start_no_incoming/);
+    });
+});
+// ---------------------------------------------------------------------------
+// AC4: Warning-rule violations allow execution
+// ---------------------------------------------------------------------------
+describe('AC4: warning-rule violations allow execution', () => {
+    let logsRoot;
+    beforeEach(async () => {
+        logsRoot = await makeTmpDir();
+    });
+    afterEach(async () => {
+        await cleanDir(logsRoot);
+    });
+    it('validate() returns at least 2 warning diagnostics and zero error diagnostics', () => {
+        const graph = parseGraph(WARNING_RULE_VIOLATION_DOT);
+        const validator = createValidator();
+        const diagnostics = validator.validate(graph);
+        const errors = diagnostics.filter((d) => d.severity === 'error');
+        const warnings = diagnostics.filter((d) => d.severity === 'warning');
+        expect(errors).toHaveLength(0);
+        expect(warnings.length).toBeGreaterThanOrEqual(2);
+    });
+    it('prompt_on_llm_nodes warning is present', () => {
+        const graph = parseGraph(WARNING_RULE_VIOLATION_DOT);
+        const validator = createValidator();
+        const diagnostics = validator.validate(graph);
+        const promptWarning = diagnostics.find((d) => d.ruleId === 'prompt_on_llm_nodes');
+        expect(promptWarning).toBeDefined();
+        expect(promptWarning?.severity).toBe('warning');
+    });
+    it('fidelity_valid warning is present', () => {
+        const graph = parseGraph(WARNING_RULE_VIOLATION_DOT);
+        const validator = createValidator();
+        const diagnostics = validator.validate(graph);
+        const fidelityWarning = diagnostics.find((d) => d.ruleId === 'fidelity_valid');
+        expect(fidelityWarning).toBeDefined();
+        expect(fidelityWarning?.severity).toBe('warning');
+    });
+    it('executor runs successfully despite warnings (warnings do not block execution)', async () => {
+        const graph = parseGraph(WARNING_RULE_VIOLATION_DOT);
+        // Validate: confirm warnings present but no errors
+        const validator = createValidator();
+        const errors = validator.validate(graph).filter((d) => d.severity === 'error');
+        expect(errors).toHaveLength(0);
+        // Execute directly (no pre-flight error check — warnings are non-blocking)
+        const { registry } = makeMockRegistry();
+        const outcome = await createGraphExecutor().run(graph, {
+            runId: 'test-run-ac4',
+            logsRoot,
+            handlerRegistry: registry,
+        });
+        expect(outcome.status).toBe('SUCCESS');
+    });
+});
+//# sourceMappingURL=validation-errors.test.js.map