scai 0.1.178 → 1.0.0

package/dist/__tests__/agents/routingDecisionStep.test.js

@@ -0,0 +1,134 @@
+// Verifies routing stays aligned with scope and write capability instead of drifting into the wrong lane.
+// It uses deterministic context snapshots and asserts only the routing output.
+import { describe, expect, it } from "vitest";
+import { routingDecisionStep } from "../../agents/routingDecisionStep.js";
+// Pass example: explicit two-file comments work routes to bounded execution, and repo-wide explanation work routes to research.
+// Fail example: a bounded edit request drifting into repo research or a read-only repo question allowing transforms.
+describe("routingDecisionStep", () => {
+    it("routes explicit multi-file docs work to bounded execution", async () => {
+        const context = {
+            initContext: {
+                userQuery: "Add comments to semanticAnalysisModule.ts and finalAnswerModule.ts",
+            },
+            analysis: {
+                scopeType: "multi-file",
+                intent: {
+                    intentCategory: "docsAndComments",
+                    confidence: 0.92,
+                    targetFiles: ["semanticAnalysisModule.ts", "finalAnswerModule.ts"],
+                    resolvedTargetFiles: ["/repo/semanticAnalysisModule.ts", "/repo/finalAnswerModule.ts"],
+                },
+            },
+            executionControl: {
+                constraints: {
+                    allowFileWrites: true,
+                },
+            },
+        };
+        const decision = await routingDecisionStep.run(context);
+        expect(decision.decision).toBe("has-info");
+        expect(decision.allowResearch).toBe(false);
+        expect(decision.allowSearch).toBe(false);
+        expect(decision.allowTransform).toBe(true);
+        expect(decision.scopeLocked).toBe(true);
+        expect(decision.rationale).toContain("scope=multi-file");
+    });
+    it("enables research for repo-wide explanation requests without allowing transforms", async () => {
+        const context = {
+            initContext: {
+                userQuery: "Explain the architecture and compare the trade-offs across the repo?",
+            },
+            analysis: {
+                scopeType: "repo-wide",
+                intent: {
+                    intentCategory: "explanation",
+                    confidence: 0.8,
+                    targetFiles: [],
+                },
+            },
+            executionControl: {
+                constraints: {
+                    allowFileWrites: false,
+                },
+            },
+        };
+        const decision = await routingDecisionStep.run(context);
+        expect(decision.decision).toBe("needs-info");
+        expect(decision.allowResearch).toBe(true);
+        expect(decision.allowTransform).toBe(false);
+        expect(decision.rationale).toContain("route=repo-research");
+        expect(decision.rationale).not.toContain("; ;");
+    });
+    it("fast-tracks resumed continuity summary queries without search", async () => {
+        const context = {
+            initContext: {
+                userQuery: "where were we?",
+            },
+            analysis: {
+                scopeType: "multi-file",
+                intent: {
+                    intentCategory: "question",
+                    confidence: 0.8,
+                    targetFiles: [],
+                },
+                resumeIntent: {
+                    kind: "continuity-summary",
+                    fastTrackEligible: true,
+                    answerFromResumeOnly: true,
+                    relevantResumeFiles: ["/repo/cli/src/agents/MainAgent.ts"],
+                    rationale: "Resumed continuity query.",
+                },
+                continuityDecision: {
+                    kind: "direct-resume-answer",
+                    answerNow: true,
+                    fallbackRequired: false,
+                    rationale: "Resumed state is sufficient for a direct answer.",
+                },
+            },
+            executionControl: {
+                constraints: {
+                    allowFileWrites: true,
+                },
+            },
+        };
+        const decision = await routingDecisionStep.run(context);
+        expect(decision.decision).toBe("has-info");
+        expect(decision.allowSearch).toBe(false);
+        expect(decision.allowResearch).toBe(false);
+        expect(decision.allowTransform).toBe(false);
+        expect(decision.scopeLocked).toBe(true);
+        expect(decision.rationale).toContain("resumeIntent=continuity-summary");
+        expect(decision.rationale).toContain("continuityDecision=direct-resume-answer");
+    });
+    it("keeps explicit folder targets on the explicit-target lane while still allowing search", async () => {
+        const context = {
+            initContext: {
+                userQuery: "Trace cli/src/agents",
+            },
+            analysis: {
+                scopeType: "multi-file",
+                focus: {
+                    resolvedTargetFiles: [],
+                    resolvedTargetFolders: ["/repo/cli/src/agents"],
+                },
+                intent: {
+                    intentCategory: "explanation",
+                    confidence: 0.88,
+                    targetFiles: ["cli/src/agents"],
+                },
+            },
+            executionControl: {
+                constraints: {
+                    allowFileWrites: false,
+                },
+            },
+        };
+        const decision = await routingDecisionStep.run(context);
+        expect(decision.decision).toBe("needs-info");
+        expect(decision.allowSearch).toBe(true);
+        expect(decision.scopeLocked).toBe(true);
+        expect(decision.rationale).toContain("route=explicit-target");
+        expect(decision.rationale).toContain("resolvedFolders=1");
+    });
+});
+//# sourceMappingURL=routingDecisionStep.test.js.map

package/dist/__tests__/agents/routingDecisionStep.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"routingDecisionStep.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/routingDecisionStep.test.ts"],"names":[],"mappings":"AAAA,0GAA0G;AAC1G,+EAA+E;AAC/E,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,mBAAmB,EAAE,MAAM,qCAAqC,CAAC;AAE1E,gIAAgI;AAChI,qHAAqH;AACrH,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,oEAAoE;aAClF;YACD,QAAQ,EAAE;gBACN,SAAS,EAAE,YAAY;gBACvB,MAAM,EAAE;oBACJ,cAAc,EAAE,iBAAiB;oBACjC,UAAU,EAAE,IAAI;oBAChB,WAAW,EAAE,CAAC,2BAA2B,EAAE,sBAAsB,CAAC;oBAClE,mBAAmB,EAAE,CAAC,iCAAiC,EAAE,4BAA4B,CAAC;iBACzF;aACJ;YACD,gBAAgB,EAAE;gBACd,WAAW,EAAE;oBACT,eAAe,EAAE,IAAI;iBACxB;aACJ;SACG,CAAC;QAET,MAAM,QAAQ,GAAG,MAAM,mBAAmB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAExD,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC3C,MAAM,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC3C,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACzC,MAAM,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC3C,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxC,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iFAAiF,EAAE,KAAK,IAAI,EAAE;QAC7F,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,sEAAsE;aACpF;YACD,QAAQ,EAAE;gBACN,SAAS,EAAE,WAAW;gBACtB,MAAM,EAAE;oBACJ,cAAc,EAAE,aAAa;oBAC7B,UAAU,EAAE,GAAG;oBACf,WAAW,EAAE,EAAE;iBAClB;aACJ;YACD,gBAAgB,EAAE;gBACd,WAAW,EAAE;oBACT,eAAe,EAAE,KAAK;iBACzB;aACJ;SACG,CAAC;QAET,MAAM,QAAQ,GAAG,MAAM,mBAAmB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAExD,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC7C,MAAM,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC1C,MAAM,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC5C,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;QAC5D,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC3E,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,gBAAgB;aAC9B;YACD,QAAQ,EAAE;gBACN,SAAS,EAAE,YAAY;gBACvB,MAAM,EAAE;oBACJ,cAAc,EAAE,UAAU;oBAC1B,UAAU,EAAE,GAAG;oBACf,WAAW,EAAE,EAAE;iBAClB;gBACD,YAAY,EAAE;oBACV,IAAI,EAAE,oBAAoB;oBAC1B,iBAAiB,EAAE,IAAI;oBACvB,oBAAoB,EAAE,IAAI;oBAC1B,mBAAmB,EAAE,CAAC,mCAAmC,CAAC;oBAC1D,SAAS,EAAE,2BAA2B;iBACzC;gBACD,kBAAkB,EAAE;oBAChB,IAAI,EAAE,sBAAsB;oBAC5B,SAAS,EAAE,IAAI;oBACf,gBAAgB,EAAE,KAAK;oBACvB,SAAS,EAAE,kDAAkD;iBAChE;aACJ;YACD,gBAAgB,EAAE;gBACd,WAAW,EAAE;oBACT,eAAe,EAAE,IAAI;iBACxB;aACJ;SACG,CAAC;QAET,MAAM,QAAQ,GAAG,MAAM,mBAAmB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAExD,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC3C,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACzC,MAAM,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC3C,MAAM,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC5C,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxC,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,iCAAiC,CAAC,CAAC;QACxE,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,yCAAyC,CAAC,CAAC;IACpF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uFAAuF,EAAE,KAAK,IAAI,EAAE;QACnG,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,sBAAsB;aACpC;YACD,QAAQ,EAAE;gBACN,SAAS,EAAE,YAAY;gBACvB,KAAK,EAAE;oBACH,mBAAmB,EAAE,EAAE;oBACvB,qBAAqB,EAAE,CAAC,sBAAsB,CAAC;iBAClD;gBACD,MAAM,EAAE;oBACJ,cAAc,EAAE,aAAa;oBAC7B,UAAU,EAAE,IAAI;oBAChB,WAAW,EAAE,CAAC,gBAAgB,CAAC;iBAClC;aACJ;YACD,gBAAgB,EAAE;gBACd,WAAW,EAAE;oBACT,eAAe,EAAE,KAAK;iBACzB;aACJ;SACG,CAAC;QAET,MAAM,QAAQ,GAAG,MAAM,mBAAmB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAExD,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC7C,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxC,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxC,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,uBAAuB,CAAC,CAAC;QAC9D,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;IAC9D,CAAC,CAAC,CAAC;AACP,CAAC,CAAC,CAAC","sourcesContent":["// Verifies routing stays aligned with scope and write capability instead of drifting into the wrong lane.\n// It uses deterministic context snapshots and asserts only the routing output.\nimport { describe, expect, it } from \"vitest\";\nimport { routingDecisionStep } from \"../../agents/routingDecisionStep.js\";\n\n// Pass example: explicit two-file comments work routes to bounded execution, and repo-wide explanation work routes to research.\n// Fail example: a bounded edit request drifting into repo research or a read-only repo question allowing transforms.\ndescribe(\"routingDecisionStep\", () => {\n    it(\"routes explicit multi-file docs work to bounded execution\", async () => {\n        const context = {\n            initContext: {\n                userQuery: \"Add comments to semanticAnalysisModule.ts and finalAnswerModule.ts\",\n            },\n            analysis: {\n                scopeType: \"multi-file\",\n                intent: {\n                    intentCategory: \"docsAndComments\",\n                    confidence: 0.92,\n                    targetFiles: [\"semanticAnalysisModule.ts\", \"finalAnswerModule.ts\"],\n                    resolvedTargetFiles: [\"/repo/semanticAnalysisModule.ts\", \"/repo/finalAnswerModule.ts\"],\n                },\n            },\n            executionControl: {\n                constraints: {\n                    allowFileWrites: true,\n                },\n            },\n        } as any;\n\n        const decision = await routingDecisionStep.run(context);\n\n        expect(decision.decision).toBe(\"has-info\");\n        expect(decision.allowResearch).toBe(false);\n        expect(decision.allowSearch).toBe(false);\n        expect(decision.allowTransform).toBe(true);\n        expect(decision.scopeLocked).toBe(true);\n        expect(decision.rationale).toContain(\"scope=multi-file\");\n    });\n\n    it(\"enables research for repo-wide explanation requests without allowing transforms\", async () => {\n        const context = {\n            initContext: {\n                userQuery: \"Explain the architecture and compare the trade-offs across the repo?\",\n            },\n            analysis: {\n                scopeType: \"repo-wide\",\n                intent: {\n                    intentCategory: \"explanation\",\n                    confidence: 0.8,\n                    targetFiles: [],\n                },\n            },\n            executionControl: {\n                constraints: {\n                    allowFileWrites: false,\n                },\n            },\n        } as any;\n\n        const decision = await routingDecisionStep.run(context);\n\n        expect(decision.decision).toBe(\"needs-info\");\n        expect(decision.allowResearch).toBe(true);\n        expect(decision.allowTransform).toBe(false);\n        expect(decision.rationale).toContain(\"route=repo-research\");\n        expect(decision.rationale).not.toContain(\"; ;\");\n    });\n\n    it(\"fast-tracks resumed continuity summary queries without search\", async () => {\n        const context = {\n            initContext: {\n                userQuery: \"where were we?\",\n            },\n            analysis: {\n                scopeType: \"multi-file\",\n                intent: {\n                    intentCategory: \"question\",\n                    confidence: 0.8,\n                    targetFiles: [],\n                },\n                resumeIntent: {\n                    kind: \"continuity-summary\",\n                    fastTrackEligible: true,\n                    answerFromResumeOnly: true,\n                    relevantResumeFiles: [\"/repo/cli/src/agents/MainAgent.ts\"],\n                    rationale: \"Resumed continuity query.\",\n                },\n                continuityDecision: {\n                    kind: \"direct-resume-answer\",\n                    answerNow: true,\n                    fallbackRequired: false,\n                    rationale: \"Resumed state is sufficient for a direct answer.\",\n                },\n            },\n            executionControl: {\n                constraints: {\n                    allowFileWrites: true,\n                },\n            },\n        } as any;\n\n        const decision = await routingDecisionStep.run(context);\n\n        expect(decision.decision).toBe(\"has-info\");\n        expect(decision.allowSearch).toBe(false);\n        expect(decision.allowResearch).toBe(false);\n        expect(decision.allowTransform).toBe(false);\n        expect(decision.scopeLocked).toBe(true);\n        expect(decision.rationale).toContain(\"resumeIntent=continuity-summary\");\n        expect(decision.rationale).toContain(\"continuityDecision=direct-resume-answer\");\n    });\n\n    it(\"keeps explicit folder targets on the explicit-target lane while still allowing search\", async () => {\n        const context = {\n            initContext: {\n                userQuery: \"Trace cli/src/agents\",\n            },\n            analysis: {\n                scopeType: \"multi-file\",\n                focus: {\n                    resolvedTargetFiles: [],\n                    resolvedTargetFolders: [\"/repo/cli/src/agents\"],\n                },\n                intent: {\n                    intentCategory: \"explanation\",\n                    confidence: 0.88,\n                    targetFiles: [\"cli/src/agents\"],\n                },\n            },\n            executionControl: {\n                constraints: {\n                    allowFileWrites: false,\n                },\n            },\n        } as any;\n\n        const decision = await routingDecisionStep.run(context);\n\n        expect(decision.decision).toBe(\"needs-info\");\n        expect(decision.allowSearch).toBe(true);\n        expect(decision.scopeLocked).toBe(true);\n        expect(decision.rationale).toContain(\"route=explicit-target\");\n        expect(decision.rationale).toContain(\"resolvedFolders=1\");\n    });\n});\n"]}

package/dist/__tests__/agents/scopeClassificationStep.test.js

@@ -0,0 +1,118 @@
+// Verifies explicit two-file requests stay multi-file even if the model says single-file.
+// It mocks the scope classifier response and checks that deterministic normalization corrects the result.
+import { beforeEach, describe, expect, it, vi } from "vitest";
+vi.mock("../../lib/generate.js", () => ({
+    generate: vi.fn(),
+}));
+import { generate } from "../../lib/generate.js";
+import { scopeClassificationStep } from "../../agents/scopeClassificationStep.js";
+// Pass example: "edit a.ts and b.ts" resolves to multi-file, while "edit a.ts" stays single-file.
+// Fail example: two explicit files collapsing to single-file or repo-wide.
+describe("scopeClassificationStep", () => {
+    beforeEach(() => {
+        vi.mocked(generate).mockReset();
+    });
+    // Guards the reported bug where explicit two-file edits were collapsed into single-file scope.
+    it("normalizes obvious multi-file requests when the LLM returns single-file", async () => {
+        vi.mocked(generate).mockResolvedValue({
+            query: "",
+            data: '{"scopeType":"single-file"}',
+        });
+        const context = {
+            initContext: {
+                userQuery: "Add concise comments to semanticAnalysisModule.ts and finalAnswerModule.ts describing phase boundaries",
+            },
+            analysis: {
+                intent: {
+                    targetFiles: ["semanticAnalysisModule.ts", "finalAnswerModule.ts"],
+                },
+            },
+        };
+        const result = await scopeClassificationStep.run(context);
+        expect(result.scopeType).toBe("multi-file");
+        expect(context.analysis.scopeType).toBe("multi-file");
+    });
+    it("preserves a clear single-file request", async () => {
+        vi.mocked(generate).mockResolvedValue({
+            query: "",
+            data: '{"scopeType":"single-file"}',
+        });
+        const context = {
+            initContext: {
+                userQuery: "Add a short comment to finalAnswerModule.ts",
+            },
+            analysis: {
+                intent: {
+                    targetFiles: ["finalAnswerModule.ts"],
+                },
+            },
+        };
+        const result = await scopeClassificationStep.run(context);
+        expect(result.scopeType).toBe("single-file");
+    });
+    it("preserves a repo-wide architecture query", async () => {
+        vi.mocked(generate).mockResolvedValue({
+            query: "",
+            data: '{"scopeType":"repo-wide"}',
+        });
+        const context = {
+            initContext: {
+                userQuery: "Summarize the architecture of the whole repo and identify weak coupling points",
+            },
+            analysis: {
+                intent: {
+                    targetFiles: [],
+                },
+            },
+        };
+        const result = await scopeClassificationStep.run(context);
+        expect(result.scopeType).toBe("repo-wide");
+    });
+    it("normalizes multi-question requests away from single-file scope", async () => {
+        vi.mocked(generate).mockResolvedValue({
+            query: "",
+            data: '{"scopeType":"single-file"}',
+        });
+        const context = {
+            initContext: {
+                userQuery: "Where is search routed? How is verify triggered?",
+            },
+            analysis: {
+                intent: {
+                    questions: [
+                        { id: "q1", order: 0, text: "Where is search routed?" },
+                        { id: "q2", order: 1, text: "How is verify triggered?" },
+                    ],
+                    targetFiles: [],
+                },
+            },
+        };
+        const result = await scopeClassificationStep.run(context);
+        const options = vi.mocked(generate).mock.calls[0]?.[1];
+        expect(result.scopeType).toBe("multi-file");
+        expect(options?.inputContext?.orderedQuestions).toHaveLength(2);
+    });
+    it("keeps multi-question repo-wide asks at repo-wide scope", async () => {
+        vi.mocked(generate).mockResolvedValue({
+            query: "",
+            data: '{"scopeType":"single-file"}',
+        });
+        const context = {
+            initContext: {
+                userQuery: "Is error handling consistent across the codebase and how do I run the test suite?",
+            },
+            analysis: {
+                intent: {
+                    questions: [
+                        { id: "q1", order: 0, text: "Is error handling consistent across the codebase?" },
+                        { id: "q2", order: 1, text: "How do I run the test suite?" },
+                    ],
+                    targetFiles: [],
+                },
+            },
+        };
+        const result = await scopeClassificationStep.run(context);
+        expect(result.scopeType).toBe("repo-wide");
+    });
+});
+//# sourceMappingURL=scopeClassificationStep.test.js.map

package/dist/__tests__/agents/scopeClassificationStep.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"scopeClassificationStep.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/scopeClassificationStep.test.ts"],"names":[],"mappings":"AAAA,0FAA0F;AAC1F,0GAA0G;AAC1G,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9D,EAAE,CAAC,IAAI,CAAC,uBAAuB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;CACpB,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACjD,OAAO,EAAE,uBAAuB,EAAE,MAAM,yCAAyC,CAAC;AAElF,kGAAkG;AAClG,2EAA2E;AAC3E,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACrC,UAAU,CAAC,GAAG,EAAE;QACZ,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,EAAE,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,+FAA+F;IAC/F,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACrF,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,iBAAiB,CAAC;YAClC,KAAK,EAAE,EAAE;YACT,IAAI,EAAE,6BAA6B;SACtC,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,wGAAwG;aACtH;YACD,QAAQ,EAAE;gBACN,MAAM,EAAE;oBACJ,WAAW,EAAE,CAAC,2BAA2B,EAAE,sBAAsB,CAAC;iBACrE;aACJ;SACG,CAAC;QAET,MAAM,MAAM,GAAG,MAAM,uBAAuB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAE1D,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC5C,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,KAAK,IAAI,EAAE;QACnD,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,iBAAiB,CAAC;YAClC,KAAK,EAAE,EAAE;YACT,IAAI,EAAE,6BAA6B;SACtC,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,6CAA6C;aAC3D;YACD,QAAQ,EAAE;gBACN,MAAM,EAAE;oBACJ,WAAW,EAAE,CAAC,sBAAsB,CAAC;iBACxC;aACJ;SACG,CAAC;QAET,MAAM,MAAM,GAAG,MAAM,uBAAuB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAE1D,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,KAAK,IAAI,EAAE;QACtD,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,iBAAiB,CAAC;YAClC,KAAK,EAAE,EAAE;YACT,IAAI,EAAE,2BAA2B;SACpC,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,gFAAgF;aAC9F;YACD,QAAQ,EAAE;gBACN,MAAM,EAAE;oBACJ,WAAW,EAAE,EAAE;iBAClB;aACJ;SACG,CAAC;QAET,MAAM,MAAM,GAAG,MAAM,uBAAuB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAE1D,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC5E,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,iBAAiB,CAAC;YAClC,KAAK,EAAE,EAAE;YACT,IAAI,EAAE,6BAA6B;SAC/B,CAAC,CAAC;QAEV,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,kDAAkD;aAChE;YACD,QAAQ,EAAE;gBACN,MAAM,EAAE;oBACJ,SAAS,EAAE;wBACP,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,yBAAyB,EAAE;wBACvD,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,0BAA0B,EAAE;qBAC3D;oBACD,WAAW,EAAE,EAAE;iBAClB;aACJ;SACG,CAAC;QAET,MAAM,MAAM,GAAG,MAAM,uBAAuB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAC1D,MAAM,OAAO,GAAG,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAA2D,CAAC;QAEjH,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC5C,MAAM,CAAC,OAAO,EAAE,YAAY,EAAE,gBAAgB,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACpE,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,iBAAiB,CAAC;YAClC,KAAK,EAAE,EAAE;YACT,IAAI,EAAE,6BAA6B;SAC/B,CAAC,CAAC;QAEV,MAAM,OAAO,GAAG;YACZ,WAAW,EAAE;gBACT,SAAS,EAAE,mFAAmF;aACjG;YACD,QAAQ,EAAE;gBACN,MAAM,EAAE;oBACJ,SAAS,EAAE;wBACP,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,mDAAmD,EAAE;wBACjF,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,8BAA8B,EAAE;qBAC/D;oBACD,WAAW,EAAE,EAAE;iBAClB;aACJ;SACG,CAAC;QAET,MAAM,MAAM,GAAG,MAAM,uBAAuB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAE1D,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;AACP,CAAC,CAAC,CAAC","sourcesContent":["// Verifies explicit two-file requests stay multi-file even if the model says single-file.\n// It mocks the scope classifier response and checks that deterministic normalization corrects the result.\nimport { beforeEach, describe, expect, it, vi } from \"vitest\";\n\nvi.mock(\"../../lib/generate.js\", () => ({\n    generate: vi.fn(),\n}));\n\nimport { generate } from \"../../lib/generate.js\";\nimport { scopeClassificationStep } from \"../../agents/scopeClassificationStep.js\";\n\n// Pass example: \"edit a.ts and b.ts\" resolves to multi-file, while \"edit a.ts\" stays single-file.\n// Fail example: two explicit files collapsing to single-file or repo-wide.\ndescribe(\"scopeClassificationStep\", () => {\n    beforeEach(() => {\n        vi.mocked(generate).mockReset();\n    });\n\n    // Guards the reported bug where explicit two-file edits were collapsed into single-file scope.\n    it(\"normalizes obvious multi-file requests when the LLM returns single-file\", async () => {\n        vi.mocked(generate).mockResolvedValue({\n            query: \"\",\n            data: '{\"scopeType\":\"single-file\"}',\n        });\n\n        const context = {\n            initContext: {\n                userQuery: \"Add concise comments to semanticAnalysisModule.ts and finalAnswerModule.ts describing phase boundaries\",\n            },\n            analysis: {\n                intent: {\n                    targetFiles: [\"semanticAnalysisModule.ts\", \"finalAnswerModule.ts\"],\n                },\n            },\n        } as any;\n\n        const result = await scopeClassificationStep.run(context);\n\n        expect(result.scopeType).toBe(\"multi-file\");\n        expect(context.analysis.scopeType).toBe(\"multi-file\");\n    });\n\n    it(\"preserves a clear single-file request\", async () => {\n        vi.mocked(generate).mockResolvedValue({\n            query: \"\",\n            data: '{\"scopeType\":\"single-file\"}',\n        });\n\n        const context = {\n            initContext: {\n                userQuery: \"Add a short comment to finalAnswerModule.ts\",\n            },\n            analysis: {\n                intent: {\n                    targetFiles: [\"finalAnswerModule.ts\"],\n                },\n            },\n        } as any;\n\n        const result = await scopeClassificationStep.run(context);\n\n        expect(result.scopeType).toBe(\"single-file\");\n    });\n\n    it(\"preserves a repo-wide architecture query\", async () => {\n        vi.mocked(generate).mockResolvedValue({\n            query: \"\",\n            data: '{\"scopeType\":\"repo-wide\"}',\n        });\n\n        const context = {\n            initContext: {\n                userQuery: \"Summarize the architecture of the whole repo and identify weak coupling points\",\n            },\n            analysis: {\n                intent: {\n                    targetFiles: [],\n                },\n            },\n        } as any;\n\n        const result = await scopeClassificationStep.run(context);\n\n        expect(result.scopeType).toBe(\"repo-wide\");\n    });\n\n    it(\"normalizes multi-question requests away from single-file scope\", async () => {\n        vi.mocked(generate).mockResolvedValue({\n            query: \"\",\n            data: '{\"scopeType\":\"single-file\"}',\n        } as any);\n\n        const context = {\n            initContext: {\n                userQuery: \"Where is search routed? How is verify triggered?\",\n            },\n            analysis: {\n                intent: {\n                    questions: [\n                        { id: \"q1\", order: 0, text: \"Where is search routed?\" },\n                        { id: \"q2\", order: 1, text: \"How is verify triggered?\" },\n                    ],\n                    targetFiles: [],\n                },\n            },\n        } as any;\n\n        const result = await scopeClassificationStep.run(context);\n        const options = vi.mocked(generate).mock.calls[0]?.[1] as { inputContext?: Record<string, unknown> } | undefined;\n\n        expect(result.scopeType).toBe(\"multi-file\");\n        expect(options?.inputContext?.orderedQuestions).toHaveLength(2);\n    });\n\n    it(\"keeps multi-question repo-wide asks at repo-wide scope\", async () => {\n        vi.mocked(generate).mockResolvedValue({\n            query: \"\",\n            data: '{\"scopeType\":\"single-file\"}',\n        } as any);\n\n        const context = {\n            initContext: {\n                userQuery: \"Is error handling consistent across the codebase and how do I run the test suite?\",\n            },\n            analysis: {\n                intent: {\n                    questions: [\n                        { id: \"q1\", order: 0, text: \"Is error handling consistent across the codebase?\" },\n                        { id: \"q2\", order: 1, text: \"How do I run the test suite?\" },\n                    ],\n                    targetFiles: [],\n                },\n            },\n        } as any;\n\n        const result = await scopeClassificationStep.run(context);\n\n        expect(result.scopeType).toBe(\"repo-wide\");\n    });\n});\n"]}

package/dist/__tests__/agents/searchContext.test.js

@@ -0,0 +1,97 @@
+/**
+ * Covers the shared pre-search context packet.
+ *
+ * Why this file exists:
+ * - Stage 2 moves search facts behind one derived packet
+ * - later lane work should consume this packet instead of raw task state
+ * - direct tests keep this structural refactor from quietly changing routing inputs
+ */
+import fs from "fs";
+import os from "os";
+import path from "path";
+import { afterEach, describe, expect, it } from "vitest";
+import { createAgentMemory } from "../../agents/agentMemory.js";
+import { buildSearchContext } from "../../agent/search/searchContext.js";
+const tempDirs = [];
+afterEach(() => {
+    while (tempDirs.length > 0) {
+        const dir = tempDirs.pop();
+        if (dir) {
+            fs.rmSync(dir, { recursive: true, force: true });
+        }
+    }
+});
+describe("buildSearchContext", () => {
+    it("marks explicit file paths as targeted", () => {
+        const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), "scai-search-context-file-"));
+        tempDirs.push(repoRoot);
+        const targetFile = path.join(repoRoot, "cli", "src", "sample.ts");
+        fs.mkdirSync(path.dirname(targetFile), { recursive: true });
+        fs.writeFileSync(targetFile, "export const sample = true;\n", "utf8");
+        const memory = createAgentMemory(targetFile, repoRoot, repoRoot);
+        memory.task.queryScope = "targeted";
+        memory.task.explicitTargets = [targetFile];
+        const context = buildSearchContext(memory, targetFile);
+        expect(context.scopeKind).toBe("targeted");
+        expect(context.resolvedTargets).toEqual(expect.arrayContaining([
+            expect.objectContaining({
+                kind: "resolved-file",
+                filePath: targetFile,
+            }),
+        ]));
+    });
+    it("marks explicit folder paths as targeted with resolved folder targets", () => {
+        const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), "scai-search-context-folder-"));
+        tempDirs.push(repoRoot);
+        const guardsDir = path.join(repoRoot, "cli", "src", "agents", "guards");
+        fs.mkdirSync(guardsDir, { recursive: true });
+        fs.writeFileSync(path.join(guardsDir, "guardState.ts"), "export const guard = true;\n", "utf8");
+        const memory = createAgentMemory(guardsDir, repoRoot, repoRoot);
+        memory.task.queryScope = "targeted";
+        memory.task.explicitTargets = [guardsDir];
+        const context = buildSearchContext(memory, guardsDir);
+        expect(context.scopeKind).toBe("targeted");
+        expect(context.resolvedTargets).toEqual(expect.arrayContaining([
+            expect.objectContaining({
+                kind: "resolved-folder",
+                folderPath: guardsDir,
+            }),
+        ]));
+    });
+    it("marks basename folder asks as folder-shaped", () => {
+        const memory = createAgentMemory("check guards", "/repo", "/repo");
+        memory.task.queryScope = "general";
+        memory.task.explicitTargets = ["guards"];
+        const context = buildSearchContext(memory, "check guards");
+        expect(context.folderShaped).toBe(true);
+    });
+    it("consumes the stored scope directly without folder upgrades", () => {
+        const memory = createAgentMemory("check guards", "/repo", "/repo");
+        memory.task.queryScope = "general";
+        memory.task.explicitTargets = ["guards"];
+        const context = buildSearchContext(memory, "check guards");
+        expect(context.scopeKind).toBe("general");
+        expect(context.folderShaped).toBe(true);
+    });
+    it("extracts quoted fragments from the query", () => {
+        const memory = createAgentMemory('Find "currentQuestionId = task.currentQuestionId"', "/repo", "/repo");
+        memory.task.queryScope = "targeted";
+        const context = buildSearchContext(memory, 'Find "currentQuestionId = task.currentQuestionId"');
+        expect(context.quotedFragments).toEqual(["currentQuestionId = task.currentQuestionId"]);
+    });
+    it("marks regex-like queries", () => {
+        const memory = createAgentMemory("runSearch|semanticSearchFiles", "/repo", "/repo");
+        memory.task.queryScope = "general";
+        const context = buildSearchContext(memory, "runSearch|semanticSearchFiles", {
+            allowRegexLane: true,
+        });
+        expect(context.regexLike).toBe(true);
+    });
+    it("marks broad repo asks as broad-repo scope", () => {
+        const memory = createAgentMemory("Explain the repo structure", "/repo", "/repo");
+        memory.task.queryScope = "broad-repo";
+        const context = buildSearchContext(memory, "Explain the repo structure");
+        expect(context.scopeKind).toBe("broad-repo");
+    });
+});
+//# sourceMappingURL=searchContext.test.js.map

package/dist/__tests__/agents/searchContext.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"searchContext.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/searchContext.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,EAAE,MAAM,IAAI,CAAC;AACpB,OAAO,EAAE,MAAM,IAAI,CAAC;AACpB,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzD,OAAO,EAAE,iBAAiB,EAAE,MAAM,6BAA6B,CAAC;AAChE,OAAO,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAEzE,MAAM,QAAQ,GAAa,EAAE,CAAC;AAE9B,SAAS,CAAC,GAAG,EAAE;IACb,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3B,MAAM,GAAG,GAAG,QAAQ,CAAC,GAAG,EAAE,CAAC;QAC3B,IAAI,GAAG,EAAE,CAAC;YACR,EAAE,CAAC,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACnD,CAAC;IACH,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,QAAQ,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,2BAA2B,CAAC,CAAC,CAAC;QACrF,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACxB,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,KAAK,EAAE,WAAW,CAAC,CAAC;QAClE,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5D,EAAE,CAAC,aAAa,CAAC,UAAU,EAAE,+BAA+B,EAAE,MAAM,CAAC,CAAC;QACtE,MAAM,MAAM,GAAG,iBAAiB,CAAC,UAAU,EAAE,QAAQ,EAAE,QAAQ,CAAC,CAAC;QACjE,MAAM,CAAC,IAAI,CAAC,UAAU,GAAG,UAAU,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,eAAe,GAAG,CAAC,UAAU,CAAC,CAAC;QAE3C,MAAM,OAAO,GAAG,kBAAkB,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;QAEvD,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC3C,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,eAAe,CAAC;YAC7D,MAAM,CAAC,gBAAgB,CAAC;gBACtB,IAAI,EAAE,eAAe;gBACrB,QAAQ,EAAE,UAAU;aACrB,CAAC;SACH,CAAC,CAAC,CAAC;IACN,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;QAC9E,MAAM,QAAQ,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,6BAA6B,CAAC,CAAC,CAAC;QACvF,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACxB,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,QAAQ,CAAC,CAAC;QACxE,EAAE,CAAC,SAAS,CAAC,SAAS,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC7C,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,eAAe,CAAC,EAAE,8BAA8B,EAAE,MAAM,CAAC,CAAC;QAChG,MAAM,MAAM,GAAG,iBAAiB,CAAC,SAAS,EAAE,QAAQ,EAAE,QAAQ,CAAC,CAAC;QAChE,MAAM,CAAC,IAAI,CAAC,UAAU,GAAG,UAAU,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,eAAe,GAAG,CAAC,SAAS,CAAC,CAAC;QAE1C,MAAM,OAAO,GAAG,kBAAkB,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;QAEtD,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC3C,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,eAAe,CAAC;YAC7D,MAAM,CAAC,gBAAgB,CAAC;gBACtB,IAAI,EAAE,iBAAiB;gBACvB,UAAU,EAAE,SAAS;aACtB,CAAC;SACH,CAAC,CAAC,CAAC;IACN,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,MAAM,GAAG,iBAAiB,CAAC,cAAc,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QACnE,MAAM,CAAC,IAAI,CAAC,UAAU,GAAG,SAAS,CAAC;QACnC,MAAM,CAAC,IAAI,CAAC,eAAe,GAAG,CAAC,QAAQ,CAAC,CAAC;QAEzC,MAAM,OAAO,GAAG,kBAAkB,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC;QAE3D,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,MAAM,MAAM,GAAG,iBAAiB,CAAC,cAAc,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QACnE,MAAM,CAAC,IAAI,CAAC,UAAU,GAAG,SAAS,CAAC;QACnC,MAAM,CAAC,IAAI,CAAC,eAAe,GAAG,CAAC,QAAQ,CAAC,CAAC;QAEzC,MAAM,OAAO,GAAG,kBAAkB,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC;QAE3D,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QAC1C,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,MAAM,GAAG,iBAAiB,CAAC,mDAAmD,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QACxG,MAAM,CAAC,IAAI,CAAC,UAAU,GAAG,UAAU,CAAC;QAEpC,MAAM,OAAO,GAAG,kBAAkB,CAAC,MAAM,EAAE,mDAAmD,CAAC,CAAC;QAEhG,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,OAAO,CAAC,CAAC,4CAA4C,CAAC,CAAC,CAAC;IAC1F,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0BAA0B,EAAE,GAAG,EAAE;QAClC,MAAM,MAAM,GAAG,iBAAiB,CAAC,+BAA+B,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QACpF,MAAM,CAAC,IAAI,CAAC,UAAU,GAAG,SAAS,CAAC;QAEnC,MAAM,OAAO,GAAG,kBAAkB,CAAC,MAAM,EAAE,+BAA+B,EAAE;YAC1E,cAAc,EAAE,IAAI;SACrB,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,MAAM,MAAM,GAAG,iBAAiB,CAAC,4BAA4B,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QACjF,MAAM,CAAC,IAAI,CAAC,UAAU,GAAG,YAAY,CAAC;QAEtC,MAAM,OAAO,GAAG,kBAAkB,CAAC,MAAM,EAAE,4BAA4B,CAAC,CAAC;QAEzE,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["/**\n * Covers the shared pre-search context packet.\n *\n * Why this file exists:\n * - Stage 2 moves search facts behind one derived packet\n * - later lane work should consume this packet instead of raw task state\n * - direct tests keep this structural refactor from quietly changing routing inputs\n */\nimport fs from \"fs\";\nimport os from \"os\";\nimport path from \"path\";\nimport { afterEach, describe, expect, it } from \"vitest\";\nimport { createAgentMemory } from \"../../agents/agentMemory.js\";\nimport { buildSearchContext } from \"../../agent/search/searchContext.js\";\n\nconst tempDirs: string[] = [];\n\nafterEach(() => {\n  while (tempDirs.length > 0) {\n    const dir = tempDirs.pop();\n    if (dir) {\n      fs.rmSync(dir, { recursive: true, force: true });\n    }\n  }\n});\n\ndescribe(\"buildSearchContext\", () => {\n  it(\"marks explicit file paths as targeted\", () => {\n    const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), \"scai-search-context-file-\"));\n    tempDirs.push(repoRoot);\n    const targetFile = path.join(repoRoot, \"cli\", \"src\", \"sample.ts\");\n    fs.mkdirSync(path.dirname(targetFile), { recursive: true });\n    fs.writeFileSync(targetFile, \"export const sample = true;\\n\", \"utf8\");\n    const memory = createAgentMemory(targetFile, repoRoot, repoRoot);\n    memory.task.queryScope = \"targeted\";\n    memory.task.explicitTargets = [targetFile];\n\n    const context = buildSearchContext(memory, targetFile);\n\n    expect(context.scopeKind).toBe(\"targeted\");\n    expect(context.resolvedTargets).toEqual(expect.arrayContaining([\n      expect.objectContaining({\n        kind: \"resolved-file\",\n        filePath: targetFile,\n      }),\n    ]));\n  });\n\n  it(\"marks explicit folder paths as targeted with resolved folder targets\", () => {\n    const repoRoot = fs.mkdtempSync(path.join(os.tmpdir(), \"scai-search-context-folder-\"));\n    tempDirs.push(repoRoot);\n    const guardsDir = path.join(repoRoot, \"cli\", \"src\", \"agents\", \"guards\");\n    fs.mkdirSync(guardsDir, { recursive: true });\n    fs.writeFileSync(path.join(guardsDir, \"guardState.ts\"), \"export const guard = true;\\n\", \"utf8\");\n    const memory = createAgentMemory(guardsDir, repoRoot, repoRoot);\n    memory.task.queryScope = \"targeted\";\n    memory.task.explicitTargets = [guardsDir];\n\n    const context = buildSearchContext(memory, guardsDir);\n\n    expect(context.scopeKind).toBe(\"targeted\");\n    expect(context.resolvedTargets).toEqual(expect.arrayContaining([\n      expect.objectContaining({\n        kind: \"resolved-folder\",\n        folderPath: guardsDir,\n      }),\n    ]));\n  });\n\n  it(\"marks basename folder asks as folder-shaped\", () => {\n    const memory = createAgentMemory(\"check guards\", \"/repo\", \"/repo\");\n    memory.task.queryScope = \"general\";\n    memory.task.explicitTargets = [\"guards\"];\n\n    const context = buildSearchContext(memory, \"check guards\");\n\n    expect(context.folderShaped).toBe(true);\n  });\n\n  it(\"consumes the stored scope directly without folder upgrades\", () => {\n    const memory = createAgentMemory(\"check guards\", \"/repo\", \"/repo\");\n    memory.task.queryScope = \"general\";\n    memory.task.explicitTargets = [\"guards\"];\n\n    const context = buildSearchContext(memory, \"check guards\");\n\n    expect(context.scopeKind).toBe(\"general\");\n    expect(context.folderShaped).toBe(true);\n  });\n\n  it(\"extracts quoted fragments from the query\", () => {\n    const memory = createAgentMemory('Find \"currentQuestionId = task.currentQuestionId\"', \"/repo\", \"/repo\");\n    memory.task.queryScope = \"targeted\";\n\n    const context = buildSearchContext(memory, 'Find \"currentQuestionId = task.currentQuestionId\"');\n\n    expect(context.quotedFragments).toEqual([\"currentQuestionId = task.currentQuestionId\"]);\n  });\n\n  it(\"marks regex-like queries\", () => {\n    const memory = createAgentMemory(\"runSearch|semanticSearchFiles\", \"/repo\", \"/repo\");\n    memory.task.queryScope = \"general\";\n\n    const context = buildSearchContext(memory, \"runSearch|semanticSearchFiles\", {\n      allowRegexLane: true,\n    });\n\n    expect(context.regexLike).toBe(true);\n  });\n\n  it(\"marks broad repo asks as broad-repo scope\", () => {\n    const memory = createAgentMemory(\"Explain the repo structure\", \"/repo\", \"/repo\");\n    memory.task.queryScope = \"broad-repo\";\n\n    const context = buildSearchContext(memory, \"Explain the repo structure\");\n\n    expect(context.scopeKind).toBe(\"broad-repo\");\n  });\n});\n"]}

package/dist/__tests__/agents/selectRelevantSourcesStep.test.js

@@ -0,0 +1,73 @@
+// Verifies selected execution files become working files while candidates remain untouched.
+// It creates real temp files so the module reads source content through the normal file path flow.
+import fs from "fs";
+import os from "os";
+import path from "path";
+import { afterEach, describe, expect, it } from "vitest";
+import { selectRelevantSourcesStep } from "../../agents/selectRelevantSourcesStep.js";
+const tempDirs = [];
+afterEach(() => {
+    for (const dir of tempDirs.splice(0)) {
+        fs.rmSync(dir, { recursive: true, force: true });
+    }
+});
+// Pass example: execution-required files become working files and candidate-only files stay candidates.
+// Fail example: candidate files being promoted into working files without entering the execution lane.
+describe("selectRelevantSourcesStep", () => {
+    it("mirrors execution files into working files without promoting candidates", async () => {
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "scai-select-"));
+        tempDirs.push(dir);
+        const selectedFile = path.join(dir, "selected.ts");
+        const candidateFile = path.join(dir, "candidate.ts");
+        fs.writeFileSync(selectedFile, "export const selected = true;\n", "utf-8");
+        fs.writeFileSync(candidateFile, "export const candidate = true;\n", "utf-8");
+        const context = {
+            analysis: {
+                focus: {
+                    selectedFiles: [selectedFile],
+                    candidateFiles: [candidateFile],
+                    rationale: "preselected",
+                },
+                executionPolicy: {
+                    phaseAllowed: { transform: true },
+                    routeAllowed: { transform: true },
+                    executionRequired: [selectedFile],
+                    executionLock: null,
+                },
+            },
+        };
+        const out = await selectRelevantSourcesStep.run({ query: "q", context });
+        const data = out.data;
+        expect(context.workingFiles.map((file) => file.path)).toEqual([selectedFile]);
+        expect(data.selectedFiles).toEqual([selectedFile]);
+        expect(data.candidateFiles).toEqual([candidateFile]);
+    });
+    it("keeps verify-focused selected files when transform is not allowed", async () => {
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "scai-select-"));
+        tempDirs.push(dir);
+        const verifyFile = path.join(dir, "mainAgentVerify.ts");
+        const executionFile = path.join(dir, "analysisPlanGenStep.ts");
+        fs.writeFileSync(verifyFile, "export function getGroundingWaveBudget() { return 4; }\n", "utf-8");
+        fs.writeFileSync(executionFile, "export const analysisPlanGenStep = {};\n", "utf-8");
+        const context = {
+            analysis: {
+                focus: {
+                    selectedFiles: [verifyFile],
+                    candidateFiles: [],
+                    rationale: "verify-focused",
+                },
+                executionPolicy: {
+                    phaseAllowed: { transform: false },
+                    routeAllowed: { transform: false },
+                    executionRequired: [executionFile],
+                    executionLock: null,
+                },
+            },
+        };
+        const out = await selectRelevantSourcesStep.run({ query: "q", context });
+        const data = out.data;
+        expect(context.workingFiles.map((file) => file.path)).toEqual([verifyFile]);
+        expect(data.selectedFiles).toEqual([verifyFile]);
+    });
+});
+//# sourceMappingURL=selectRelevantSourcesStep.test.js.map

package/dist/__tests__/agents/selectRelevantSourcesStep.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"selectRelevantSourcesStep.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/selectRelevantSourcesStep.test.ts"],"names":[],"mappings":"AAAA,4FAA4F;AAC5F,mGAAmG;AACnG,OAAO,EAAE,MAAM,IAAI,CAAC;AACpB,OAAO,EAAE,MAAM,IAAI,CAAC;AACpB,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzD,OAAO,EAAE,yBAAyB,EAAE,MAAM,2CAA2C,CAAC;AAEtF,MAAM,QAAQ,GAAa,EAAE,CAAC;AAE9B,SAAS,CAAC,GAAG,EAAE;IACX,KAAK,MAAM,GAAG,IAAI,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC;QACnC,EAAE,CAAC,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACrD,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,wGAAwG;AACxG,uGAAuG;AACvG,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;IACvC,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACrF,MAAM,GAAG,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,cAAc,CAAC,CAAC,CAAC;QACnE,QAAQ,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAEnB,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC;QACnD,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;QACrD,EAAE,CAAC,aAAa,CAAC,YAAY,EAAE,iCAAiC,EAAE,OAAO,CAAC,CAAC;QAC3E,EAAE,CAAC,aAAa,CAAC,aAAa,EAAE,kCAAkC,EAAE,OAAO,CAAC,CAAC;QAE7E,MAAM,OAAO,GAAG;YACZ,QAAQ,EAAE;gBACN,KAAK,EAAE;oBACH,aAAa,EAAE,CAAC,YAAY,CAAC;oBAC7B,cAAc,EAAE,CAAC,aAAa,CAAC;oBAC/B,SAAS,EAAE,aAAa;iBAC3B;gBACD,eAAe,EAAE;oBACb,YAAY,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE;oBACjC,YAAY,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE;oBACjC,iBAAiB,EAAE,CAAC,YAAY,CAAC;oBACjC,aAAa,EAAE,IAAI;iBACtB;aACJ;SACG,CAAC;QAET,MAAM,GAAG,GAAG,MAAM,yBAAyB,CAAC,GAAG,CAAC,EAAE,KAAK,EAAE,GAAG,EAAE,OAAO,EAAE,CAAC,CAAC;QACzE,MAAM,IAAI,GAAG,GAAG,CAAC,IAA6D,CAAC;QAE/E,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,IAAsB,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC;QAChG,MAAM,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC;QACnD,MAAM,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC;IACzD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,GAAG,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,cAAc,CAAC,CAAC,CAAC;QACnE,QAAQ,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAEnB,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,oBAAoB,CAAC,CAAC;QACxD,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,wBAAwB,CAAC,CAAC;QAC/D,EAAE,CAAC,aAAa,CAAC,UAAU,EAAE,0DAA0D,EAAE,OAAO,CAAC,CAAC;QAClG,EAAE,CAAC,aAAa,CAAC,aAAa,EAAE,0CAA0C,EAAE,OAAO,CAAC,CAAC;QAErF,MAAM,OAAO,GAAG;YACZ,QAAQ,EAAE;gBACN,KAAK,EAAE;oBACH,aAAa,EAAE,CAAC,UAAU,CAAC;oBAC3B,cAAc,EAAE,EAAE;oBAClB,SAAS,EAAE,gBAAgB;iBAC9B;gBACD,eAAe,EAAE;oBACb,YAAY,EAAE,EAAE,SAAS,EAAE,KAAK,EAAE;oBAClC,YAAY,EAAE,EAAE,SAAS,EAAE,KAAK,EAAE;oBAClC,iBAAiB,EAAE,CAAC,aAAa,CAAC;oBAClC,aAAa,EAAE,IAAI;iBACtB;aACJ;SACG,CAAC;QAET,MAAM,GAAG,GAAG,MAAM,yBAAyB,CAAC,GAAG,CAAC,EAAE,KAAK,EAAE,GAAG,EAAE,OAAO,EAAE,CAAC,CAAC;QACzE,MAAM,IAAI,GAAG,GAAG,CAAC,IAAmC,CAAC;QAErD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,IAAsB,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC;QAC9F,MAAM,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,OAAO,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;AACP,CAAC,CAAC,CAAC","sourcesContent":["// Verifies selected execution files become working files while candidates remain untouched.\n// It creates real temp files so the module reads source content through the normal file path flow.\nimport fs from \"fs\";\nimport os from \"os\";\nimport path from \"path\";\nimport { afterEach, describe, expect, it } from \"vitest\";\nimport { selectRelevantSourcesStep } from \"../../agents/selectRelevantSourcesStep.js\";\n\nconst tempDirs: string[] = [];\n\nafterEach(() => {\n    for (const dir of tempDirs.splice(0)) {\n        fs.rmSync(dir, { recursive: true, force: true });\n    }\n});\n\n// Pass example: execution-required files become working files and candidate-only files stay candidates.\n// Fail example: candidate files being promoted into working files without entering the execution lane.\ndescribe(\"selectRelevantSourcesStep\", () => {\n    it(\"mirrors execution files into working files without promoting candidates\", async () => {\n        const dir = fs.mkdtempSync(path.join(os.tmpdir(), \"scai-select-\"));\n        tempDirs.push(dir);\n\n        const selectedFile = path.join(dir, \"selected.ts\");\n        const candidateFile = path.join(dir, \"candidate.ts\");\n        fs.writeFileSync(selectedFile, \"export const selected = true;\\n\", \"utf-8\");\n        fs.writeFileSync(candidateFile, \"export const candidate = true;\\n\", \"utf-8\");\n\n        const context = {\n            analysis: {\n                focus: {\n                    selectedFiles: [selectedFile],\n                    candidateFiles: [candidateFile],\n                    rationale: \"preselected\",\n                },\n                executionPolicy: {\n                    phaseAllowed: { transform: true },\n                    routeAllowed: { transform: true },\n                    executionRequired: [selectedFile],\n                    executionLock: null,\n                },\n            },\n        } as any;\n\n        const out = await selectRelevantSourcesStep.run({ query: \"q\", context });\n        const data = out.data as { selectedFiles: string[]; candidateFiles: string[] };\n\n        expect(context.workingFiles.map((file: { path: string }) => file.path)).toEqual([selectedFile]);\n        expect(data.selectedFiles).toEqual([selectedFile]);\n        expect(data.candidateFiles).toEqual([candidateFile]);\n    });\n\n    it(\"keeps verify-focused selected files when transform is not allowed\", async () => {\n        const dir = fs.mkdtempSync(path.join(os.tmpdir(), \"scai-select-\"));\n        tempDirs.push(dir);\n\n        const verifyFile = path.join(dir, \"mainAgentVerify.ts\");\n        const executionFile = path.join(dir, \"analysisPlanGenStep.ts\");\n        fs.writeFileSync(verifyFile, \"export function getGroundingWaveBudget() { return 4; }\\n\", \"utf-8\");\n        fs.writeFileSync(executionFile, \"export const analysisPlanGenStep = {};\\n\", \"utf-8\");\n\n        const context = {\n            analysis: {\n                focus: {\n                    selectedFiles: [verifyFile],\n                    candidateFiles: [],\n                    rationale: \"verify-focused\",\n                },\n                executionPolicy: {\n                    phaseAllowed: { transform: false },\n                    routeAllowed: { transform: false },\n                    executionRequired: [executionFile],\n                    executionLock: null,\n                },\n            },\n        } as any;\n\n        const out = await selectRelevantSourcesStep.run({ query: \"q\", context });\n        const data = out.data as { selectedFiles: string[] };\n\n        expect(context.workingFiles.map((file: { path: string }) => file.path)).toEqual([verifyFile]);\n        expect(data.selectedFiles).toEqual([verifyFile]);\n    });\n});\n"]}

package/dist/__tests__/agents/structuredOutput.test.js

@@ -0,0 +1,45 @@
+/**
+ * Covers shared structured output parsing for new-agent model calls.
+ *
+ * Why this file exists:
+ * - stage 1 moves JSON parsing under one owned module
+ * - fenced JSON and fallback parsing are easy to regress in small refactors
+ * - direct tests make it safe to delete the old prompt parser entrypoint
+ */
+import { describe, expect, it } from "vitest";
+import { parseStructuredJson, parseStructuredJsonResult } from "../../agent/structuredOutput/index.js";
+describe("structured output parsing", () => {
+    it("parses plain JSON", () => {
+        expect(parseStructuredJson('{"ok":true,"count":2}', { fallback: { ok: false, count: 0 } })).toEqual({
+            ok: true,
+            count: 2,
+        });
+    });
+    it("parses fenced JSON", () => {
+        expect(parseStructuredJson('```json\n{"status":"valid"}\n```', { fallback: { status: "invalid" } })).toEqual({
+            status: "valid",
+        });
+    });
+    it("returns the fallback on bad JSON", () => {
+        expect(parseStructuredJson("not json", { fallback: { status: "fallback" } })).toEqual({
+            status: "fallback",
+        });
+    });
+    it("returns parse metadata for fenced JSON", () => {
+        expect(parseStructuredJsonResult('```json\n{"status":"valid"}\n```', { fallback: { status: "invalid" } })).toEqual({
+            value: { status: "valid" },
+            meta: {
+                usedFallback: false,
+                candidateSource: "fenced",
+            },
+        });
+    });
+    it("returns parse metadata when fallback is used", () => {
+        const result = parseStructuredJsonResult("not json", { fallback: { status: "fallback" } });
+        expect(result.value).toEqual({ status: "fallback" });
+        expect(result.meta.usedFallback).toBe(true);
+        expect(result.meta.candidateSource).toBe("plain");
+        expect(result.meta.parseError).toContain("Unexpected token");
+    });
+});
+//# sourceMappingURL=structuredOutput.test.js.map

package/dist/__tests__/agents/structuredOutput.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"structuredOutput.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/structuredOutput.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,mBAAmB,EAAE,yBAAyB,EAAE,MAAM,uCAAuC,CAAC;AAEvG,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;IACzC,EAAE,CAAC,mBAAmB,EAAE,GAAG,EAAE;QAC3B,MAAM,CAAC,mBAAmB,CAAC,uBAAuB,EAAE,EAAE,QAAQ,EAAE,EAAE,EAAE,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC;YAClG,EAAE,EAAE,IAAI;YACR,KAAK,EAAE,CAAC;SACT,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAC5B,MAAM,CAAC,mBAAmB,CAAC,kCAAkC,EAAE,EAAE,QAAQ,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC;YAC3G,MAAM,EAAE,OAAO;SAChB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC1C,MAAM,CAAC,mBAAmB,CAAC,UAAU,EAAE,EAAE,QAAQ,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC;YACpF,MAAM,EAAE,UAAU;SACnB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,CAAC,yBAAyB,CAAC,kCAAkC,EAAE,EAAE,QAAQ,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC;YACjH,KAAK,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE;YAC1B,IAAI,EAAE;gBACJ,YAAY,EAAE,KAAK;gBACnB,eAAe,EAAE,QAAQ;aAC1B;SACF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,MAAM,GAAG,yBAAyB,CAAC,UAAU,EAAE,EAAE,QAAQ,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,EAAE,CAAC,CAAC;QAC3F,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAC;QACrD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC5C,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["/**\n * Covers shared structured output parsing for new-agent model calls.\n *\n * Why this file exists:\n * - stage 1 moves JSON parsing under one owned module\n * - fenced JSON and fallback parsing are easy to regress in small refactors\n * - direct tests make it safe to delete the old prompt parser entrypoint\n */\nimport { describe, expect, it } from \"vitest\";\nimport { parseStructuredJson, parseStructuredJsonResult } from \"../../agent/structuredOutput/index.js\";\n\ndescribe(\"structured output parsing\", () => {\n  it(\"parses plain JSON\", () => {\n    expect(parseStructuredJson('{\"ok\":true,\"count\":2}', { fallback: { ok: false, count: 0 } })).toEqual({\n      ok: true,\n      count: 2,\n    });\n  });\n\n  it(\"parses fenced JSON\", () => {\n    expect(parseStructuredJson('```json\\n{\"status\":\"valid\"}\\n```', { fallback: { status: \"invalid\" } })).toEqual({\n      status: \"valid\",\n    });\n  });\n\n  it(\"returns the fallback on bad JSON\", () => {\n    expect(parseStructuredJson(\"not json\", { fallback: { status: \"fallback\" } })).toEqual({\n      status: \"fallback\",\n    });\n  });\n\n  it(\"returns parse metadata for fenced JSON\", () => {\n    expect(parseStructuredJsonResult('```json\\n{\"status\":\"valid\"}\\n```', { fallback: { status: \"invalid\" } })).toEqual({\n      value: { status: \"valid\" },\n      meta: {\n        usedFallback: false,\n        candidateSource: \"fenced\",\n      },\n    });\n  });\n\n  it(\"returns parse metadata when fallback is used\", () => {\n    const result = parseStructuredJsonResult(\"not json\", { fallback: { status: \"fallback\" } });\n    expect(result.value).toEqual({ status: \"fallback\" });\n    expect(result.meta.usedFallback).toBe(true);\n    expect(result.meta.candidateSource).toBe(\"plain\");\n    expect(result.meta.parseError).toContain(\"Unexpected token\");\n  });\n});\n"]}

package/dist/__tests__/agents/transformPlanGenStep.fallback.test.js

@@ -0,0 +1,59 @@
+// Verifies transform planning falls back to semantic analysis when the planner reply is not valid JSON.
+// It covers the loop where shouldModify was true but transformPlanGen returned no steps and nothing was written.
+import { beforeEach, describe, expect, it, vi } from "vitest";
+vi.mock("../../lib/generate.js", () => ({
+    generate: vi.fn(),
+}));
+import { generate } from "../../lib/generate.js";
+import { transformPlanGenStep } from "../../agents/transformPlanGenStep.js";
+describe("transformPlanGenStep fallback", () => {
+    beforeEach(() => {
+        vi.mocked(generate).mockReset();
+    });
+    it("builds a codeTransform fallback step when the planner reply is not parseable", async () => {
+        vi.mocked(generate).mockResolvedValue({
+            query: "",
+            data: "I would add a small hello world comment near the top of the file.",
+        });
+        const context = {
+            task: {
+                currentStep: {
+                    filePath: "/repo/actionRegistry.ts",
+                },
+            },
+            analysis: {
+                intent: {
+                    normalizedQuery: "add a hello world statement to the actionRegistry file",
+                    intentCategory: "code_change",
+                },
+                fileAnalysis: {
+                    "/repo/actionRegistry.ts": {
+                        action: {
+                            shouldModify: true,
+                        },
+                        proposedChanges: {
+                            summary: "Add a hello world comment near the imports.",
+                            rationale: "Keep the change small and non-functional.",
+                        },
+                        excerpts: [],
+                    },
+                },
+                routingDecision: {
+                    confidence: 0.91,
+                },
+            },
+            execution: {},
+        };
+        await transformPlanGenStep.run(context);
+        const steps = context.analysis.planSuggestion?.plan?.steps ?? [];
+        expect(steps).toHaveLength(1);
+        expect(steps[0]).toMatchObject({
+            action: "codeTransform",
+            actionId: "codeTransform",
+            targetFile: "/repo/actionRegistry.ts",
+            groups: ["transform"],
+        });
+        expect(steps[0].description).toContain("hello world");
+    });
+});
+//# sourceMappingURL=transformPlanGenStep.fallback.test.js.map

package/dist/__tests__/agents/transformPlanGenStep.fallback.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"transformPlanGenStep.fallback.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/transformPlanGenStep.fallback.test.ts"],"names":[],"mappings":"AAAA,wGAAwG;AACxG,iHAAiH;AACjH,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9D,EAAE,CAAC,IAAI,CAAC,uBAAuB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;CACpB,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACjD,OAAO,EAAE,oBAAoB,EAAE,MAAM,sCAAsC,CAAC;AAE5E,QAAQ,CAAC,+BAA+B,EAAE,GAAG,EAAE;IAC3C,UAAU,CAAC,GAAG,EAAE;QACZ,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,EAAE,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8EAA8E,EAAE,KAAK,IAAI,EAAE;QAC1F,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,iBAAiB,CAAC;YAClC,KAAK,EAAE,EAAE;YACT,IAAI,EAAE,mEAAmE;SAC5E,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG;YACZ,IAAI,EAAE;gBACF,WAAW,EAAE;oBACT,QAAQ,EAAE,yBAAyB;iBACtC;aACJ;YACD,QAAQ,EAAE;gBACN,MAAM,EAAE;oBACJ,eAAe,EAAE,wDAAwD;oBACzE,cAAc,EAAE,aAAa;iBAChC;gBACD,YAAY,EAAE;oBACV,yBAAyB,EAAE;wBACvB,MAAM,EAAE;4BACJ,YAAY,EAAE,IAAI;yBACrB;wBACD,eAAe,EAAE;4BACb,OAAO,EAAE,6CAA6C;4BACtD,SAAS,EAAE,2CAA2C;yBACzD;wBACD,QAAQ,EAAE,EAAE;qBACf;iBACJ;gBACD,eAAe,EAAE;oBACb,UAAU,EAAE,IAAI;iBACnB;aACJ;YACD,SAAS,EAAE,EAAE;SACT,CAAC;QAET,MAAM,oBAAoB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAExC,MAAM,KAAK,GAAG,OAAO,CAAC,QAAQ,CAAC,cAAc,EAAE,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC;QACjE,MAAM,CAAC,KAAK,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC9B,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC;YAC3B,MAAM,EAAE,eAAe;YACvB,QAAQ,EAAE,eAAe;YACzB,UAAU,EAAE,yBAAyB;YACrC,MAAM,EAAE,CAAC,WAAW,CAAC;SACxB,CAAC,CAAC;QACH,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;AACP,CAAC,CAAC,CAAC","sourcesContent":["// Verifies transform planning falls back to semantic analysis when the planner reply is not valid JSON.\n// It covers the loop where shouldModify was true but transformPlanGen returned no steps and nothing was written.\nimport { beforeEach, describe, expect, it, vi } from \"vitest\";\n\nvi.mock(\"../../lib/generate.js\", () => ({\n    generate: vi.fn(),\n}));\n\nimport { generate } from \"../../lib/generate.js\";\nimport { transformPlanGenStep } from \"../../agents/transformPlanGenStep.js\";\n\ndescribe(\"transformPlanGenStep fallback\", () => {\n    beforeEach(() => {\n        vi.mocked(generate).mockReset();\n    });\n\n    it(\"builds a codeTransform fallback step when the planner reply is not parseable\", async () => {\n        vi.mocked(generate).mockResolvedValue({\n            query: \"\",\n            data: \"I would add a small hello world comment near the top of the file.\",\n        });\n\n        const context = {\n            task: {\n                currentStep: {\n                    filePath: \"/repo/actionRegistry.ts\",\n                },\n            },\n            analysis: {\n                intent: {\n                    normalizedQuery: \"add a hello world statement to the actionRegistry file\",\n                    intentCategory: \"code_change\",\n                },\n                fileAnalysis: {\n                    \"/repo/actionRegistry.ts\": {\n                        action: {\n                            shouldModify: true,\n                        },\n                        proposedChanges: {\n                            summary: \"Add a hello world comment near the imports.\",\n                            rationale: \"Keep the change small and non-functional.\",\n                        },\n                        excerpts: [],\n                    },\n                },\n                routingDecision: {\n                    confidence: 0.91,\n                },\n            },\n            execution: {},\n        } as any;\n\n        await transformPlanGenStep.run(context);\n\n        const steps = context.analysis.planSuggestion?.plan?.steps ?? [];\n        expect(steps).toHaveLength(1);\n        expect(steps[0]).toMatchObject({\n            action: \"codeTransform\",\n            actionId: \"codeTransform\",\n            targetFile: \"/repo/actionRegistry.ts\",\n            groups: [\"transform\"],\n        });\n        expect(steps[0].description).toContain(\"hello world\");\n    });\n});\n"]}