scai 0.1.178 → 1.0.0

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

@@ -0,0 +1,227 @@
+/**
+ * Covers understand-owned coarse scope validation for the new agent.
+ *
+ * Why this file exists:
+ * - the understand model should return coarse scope once
+ * - runtime should only validate that scope, not classify it from scratch
+ * - direct tests keep scope validation changes separate from later lane behavior
+ */
+import { describe, expect, it } from "vitest";
+import { normalizeUnderstandResult } from "../../agent/runtime/understand.js";
+describe("normalizeUnderstandResult queryScope validation", () => {
+    it("keeps a valid model-provided scope", () => {
+        const result = normalizeUnderstandResult("Explain cli/src/agent/search/index.ts", {
+            normalizedQuery: "Explain cli/src/agent/search/index.ts",
+            queryScope: "broad-repo",
+            explicitTargets: ["cli/src/agent/search/index.ts"],
+            constraints: [],
+            primaryIntent: "Explain one file.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("broad-repo");
+    });
+    it("falls back safely on an invalid scope", () => {
+        const result = normalizeUnderstandResult("what is recursion", {
+            normalizedQuery: "what is recursion",
+            queryScope: "wide",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Answer a general knowledge question.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("general");
+    });
+    it("downgrades a generic tool-usage miss from targeted to general", () => {
+        const result = normalizeUnderstandResult("how do I use git bash to save a diff?", {
+            normalizedQuery: "how do I use git bash to save a diff",
+            queryScope: "targeted",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Learn how to save a git diff output to a file using Git Bash.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("general");
+    });
+    it("keeps a pure concept question general even when the model over-targets it", () => {
+        const result = normalizeUnderstandResult("what is recursion?", {
+            normalizedQuery: "what is recursion",
+            queryScope: "targeted",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Answer a general programming concept question.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("general");
+    });
+    it("keeps repo wording out of the general fallback", () => {
+        const result = normalizeUnderstandResult("how does this repo run tests?", {
+            normalizedQuery: "how does this repo run tests",
+            queryScope: "broad-repo",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Explain how this repo runs tests.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).not.toBe("general");
+    });
+    it("keeps explicit file questions targeted", () => {
+        const result = normalizeUnderstandResult("explain cli/src/agent/index.ts", {
+            normalizedQuery: "explain cli/src/agent/index.ts",
+            queryScope: "general",
+            explicitTargets: ["cli/src/agent/index.ts"],
+            constraints: [],
+            primaryIntent: "Explain the file.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("targeted");
+    });
+    it("upgrades general plus one explicit target to targeted", () => {
+        const result = normalizeUnderstandResult("Explain cli/src/agent/search/index.ts", {
+            normalizedQuery: "Explain cli/src/agent/search/index.ts",
+            queryScope: "general",
+            explicitTargets: ["cli/src/agent/search/index.ts"],
+            constraints: [],
+            primaryIntent: "Explain one file.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("targeted");
+    });
+    it("upgrades general plus many explicit targets to multi-file", () => {
+        const result = normalizeUnderstandResult("Explain how agentMemory and agentLoop divide responsibilities", {
+            normalizedQuery: "Explain how agentMemory and agentLoop divide responsibilities",
+            queryScope: "general",
+            explicitTargets: ["agentMemory.ts", "agentLoop.ts"],
+            constraints: [],
+            primaryIntent: "Explain how agentMemory and agentLoop divide responsibilities.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("multi-file");
+    });
+    it("upgrades targeted plus many explicit targets to multi-file", () => {
+        const result = normalizeUnderstandResult("Compare agentMemory.ts and agentLoop.ts", {
+            normalizedQuery: "Compare agentMemory.ts and agentLoop.ts",
+            queryScope: "targeted",
+            explicitTargets: ["agentMemory.ts", "agentLoop.ts"],
+            constraints: [],
+            primaryIntent: "Compare the two files.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.queryScope).toBe("multi-file");
+    });
+    it("drops non-target pronouns from explicit targets", () => {
+        const result = normalizeUnderstandResult("summarize this repo architecture and identify weak coupling points", {
+            normalizedQuery: "summarize repo architecture and identify weak coupling points",
+            queryScope: "broad-repo",
+            explicitTargets: ["this"],
+            constraints: [],
+            primaryIntent: "understand repo architecture",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.explicitTargets).toEqual([]);
+        expect(result.queryScope).toBe("broad-repo");
+    });
+    it("drops discourse words like Also from explicit targets in resumed follow-ups", () => {
+        const result = normalizeUnderstandResult("did you check the memory.ts file? Also outline what you already checked", {
+            normalizedQuery: "did you check the memory.ts file? Also outline what you already checked",
+            queryScope: "general",
+            explicitTargets: ["memory.ts", "Also"],
+            constraints: [],
+            primaryIntent: "Verify if memory.ts was checked and summarize prior checks.",
+            alternativeIntents: [],
+            ambiguityFlags: [],
+        });
+        expect(result.explicitTargets).toEqual(["memory.ts"]);
+        expect(result.queryScope).toBe("targeted");
+    });
+    it("normalizes bounded search term groups", () => {
+        const result = normalizeUnderstandResult("where is max turns set", {
+            normalizedQuery: "where is max turns set",
+            queryScope: "general",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Find the owner for the turn limit.",
+            alternativeIntents: [],
+            searchTermGroups: [
+                { kind: "repo-anchor", terms: [" maxTurns ", "turnCount", "maxTurns", "count", "loop", "limit", "extra"] },
+                { kind: "command-path", terms: ["memory.ts", "iteration", "constraint", "iteration", "limit"] },
+                { kind: "generic", terms: ["configuration", "setting", "configured"] },
+                { kind: "generic", terms: ["owner", "control"] },
+                { kind: "generic", terms: ["ignored", "group"] },
+            ],
+            ambiguityFlags: [],
+        });
+        expect(result.searchTermGroups).toEqual([
+            { kind: "repo-anchor", terms: ["maxTurns", "turnCount", "count", "loop", "limit", "extra"] },
+            { kind: "command-path", terms: ["memory.ts", "iteration", "constraint", "limit"] },
+            { kind: "generic", terms: ["configuration", "setting", "configured"] },
+            { kind: "generic", terms: ["owner", "control"] },
+        ]);
+    });
+    it("keeps anchor-like search terms and drops repeated generic filler across groups", () => {
+        const result = normalizeUnderstandResult("how do tests run", {
+            normalizedQuery: "how do tests run",
+            queryScope: "broad-repo",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Understand test execution.",
+            alternativeIntents: [],
+            searchTermGroups: [
+                { kind: "repo-anchor", terms: ["registerDevCliCommands", "test", "command", "/test-run", "package.json"] },
+                { kind: "generic", terms: ["test", "command", "config", "run"] },
+                { kind: "repo-anchor", terms: ["testCommands", "config", "entrypoint"] },
+            ],
+            ambiguityFlags: [],
+        });
+        expect(result.searchTermGroups).toEqual([
+            { kind: "repo-anchor", terms: ["registerDevCliCommands", "test", "command", "/test-run", "package.json"] },
+            { kind: "repo-anchor", terms: ["testCommands", "config", "entrypoint"] },
+        ]);
+    });
+    it("drops all-generic typed groups when config-style queries need a repo-anchor group", () => {
+        const result = normalizeUnderstandResult("where are test entry points configured", {
+            normalizedQuery: "where are test entry points configured",
+            queryScope: "general",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Find test entrypoint configuration.",
+            alternativeIntents: [],
+            searchTermGroups: [
+                { kind: "generic", terms: ["test", "config", "entrypoint"] },
+                { kind: "command-path", terms: ["package.json", "test-run"] },
+            ],
+            ambiguityFlags: [],
+        });
+        expect(result.searchTermGroups).toEqual([]);
+    });
+    it("drops weak command-path groups that have no path, file, or command-like terms", () => {
+        const result = normalizeUnderstandResult("where are test entry points configured", {
+            normalizedQuery: "where are test entry points configured",
+            queryScope: "general",
+            explicitTargets: [],
+            constraints: [],
+            primaryIntent: "Find test entrypoint configuration.",
+            alternativeIntents: [],
+            searchTermGroups: [
+                { kind: "repo-anchor", terms: ["registerDevCliCommands", "testCommands"] },
+                { kind: "command-path", terms: ["config", "entrypoint", "route"] },
+                { kind: "command-path", terms: ["package.json", "/test-run", "vitest.config.ts"] },
+            ],
+            ambiguityFlags: [],
+        });
+        expect(result.searchTermGroups).toEqual([
+            { kind: "repo-anchor", terms: ["registerDevCliCommands", "testCommands"] },
+            { kind: "command-path", terms: ["package.json", "/test-run", "vitest.config.ts"] },
+        ]);
+    });
+});
+//# sourceMappingURL=understandScope.test.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"understandScope.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/understandScope.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,yBAAyB,EAAE,MAAM,mCAAmC,CAAC;AAE9E,QAAQ,CAAC,iDAAiD,EAAE,GAAG,EAAE;IAC/D,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,MAAM,GAAG,yBAAyB,CAAC,uCAAuC,EAAE;YAChF,eAAe,EAAE,uCAAuC;YACxD,UAAU,EAAE,YAAY;YACxB,eAAe,EAAE,CAAC,+BAA+B,CAAC;YAClD,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,mBAAmB;YAClC,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,MAAM,GAAG,yBAAyB,CAAC,mBAAmB,EAAE;YAC5D,eAAe,EAAE,mBAAmB;YACpC,UAAU,EAAE,MAAe;YAC3B,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,sCAAsC;YACrD,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,GAAG,EAAE;QACvE,MAAM,MAAM,GAAG,yBAAyB,CAAC,uCAAuC,EAAE;YAChF,eAAe,EAAE,sCAAsC;YACvD,UAAU,EAAE,UAAU;YACtB,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,+DAA+D;YAC9E,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2EAA2E,EAAE,GAAG,EAAE;QACnF,MAAM,MAAM,GAAG,yBAAyB,CAAC,oBAAoB,EAAE;YAC7D,eAAe,EAAE,mBAAmB;YACpC,UAAU,EAAE,UAAU;YACtB,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,gDAAgD;YAC/D,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,MAAM,GAAG,yBAAyB,CAAC,+BAA+B,EAAE;YACxE,eAAe,EAAE,8BAA8B;YAC/C,UAAU,EAAE,YAAY;YACxB,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,mCAAmC;YAClD,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,MAAM,GAAG,yBAAyB,CAAC,gCAAgC,EAAE;YACzE,eAAe,EAAE,gCAAgC;YACjD,UAAU,EAAE,SAAS;YACrB,eAAe,EAAE,CAAC,wBAAwB,CAAC;YAC3C,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,mBAAmB;YAClC,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;QAC/D,MAAM,MAAM,GAAG,yBAAyB,CAAC,uCAAuC,EAAE;YAChF,eAAe,EAAE,uCAAuC;YACxD,UAAU,EAAE,SAAS;YACrB,eAAe,EAAE,CAAC,+BAA+B,CAAC;YAClD,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,mBAAmB;YAClC,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,MAAM,GAAG,yBAAyB,CACtC,+DAA+D,EAC/D;YACE,eAAe,EAAE,+DAA+D;YAChF,UAAU,EAAE,SAAS;YACrB,eAAe,EAAE,CAAC,gBAAgB,EAAE,cAAc,CAAC;YACnD,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,gEAAgE;YAC/E,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,MAAM,MAAM,GAAG,yBAAyB,CACtC,yCAAyC,EACzC;YACE,eAAe,EAAE,yCAAyC;YAC1D,UAAU,EAAE,UAAU;YACtB,eAAe,EAAE,CAAC,gBAAgB,EAAE,cAAc,CAAC;YACnD,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,wBAAwB;YACvC,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,MAAM,GAAG,yBAAyB,CACtC,oEAAoE,EACpE;YACE,eAAe,EAAE,+DAA+D;YAChF,UAAU,EAAE,YAAY;YACxB,eAAe,EAAE,CAAC,MAAM,CAAC;YACzB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,8BAA8B;YAC7C,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,GAAG,EAAE;QACrF,MAAM,MAAM,GAAG,yBAAyB,CACtC,yEAAyE,EACzE;YACE,eAAe,EAAE,yEAAyE;YAC1F,UAAU,EAAE,SAAS;YACrB,eAAe,EAAE,CAAC,WAAW,EAAE,MAAM,CAAC;YACtC,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,6DAA6D;YAC5E,kBAAkB,EAAE,EAAE;YACtB,cAAc,EAAE,EAAE;SACnB,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,OAAO,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,MAAM,GAAG,yBAAyB,CAAC,wBAAwB,EAAE;YACjE,eAAe,EAAE,wBAAwB;YACzC,UAAU,EAAE,SAAS;YACrB,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,oCAAoC;YACnD,kBAAkB,EAAE,EAAE;YACtB,gBAAgB,EAAE;gBAChB,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,YAAY,EAAE,WAAW,EAAE,UAAU,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,EAAE;gBAC1G,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC,WAAW,EAAE,WAAW,EAAE,YAAY,EAAE,WAAW,EAAE,OAAO,CAAC,EAAE;gBAC/F,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,eAAe,EAAE,SAAS,EAAE,YAAY,CAAC,EAAE;gBACtE,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,OAAO,EAAE,SAAS,CAAC,EAAE;gBAChD,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,SAAS,EAAE,OAAO,CAAC,EAAE;aACjD;YACD,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC;YACtC,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,UAAU,EAAE,WAAW,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,EAAE;YAC5F,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC,WAAW,EAAE,WAAW,EAAE,YAAY,EAAE,OAAO,CAAC,EAAE;YAClF,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,eAAe,EAAE,SAAS,EAAE,YAAY,CAAC,EAAE;YACtE,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,OAAO,EAAE,SAAS,CAAC,EAAE;SACjD,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE;QACxF,MAAM,MAAM,GAAG,yBAAyB,CAAC,kBAAkB,EAAE;YAC3D,eAAe,EAAE,kBAAkB;YACnC,UAAU,EAAE,YAAY;YACxB,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,4BAA4B;YAC3C,kBAAkB,EAAE,EAAE;YACtB,gBAAgB,EAAE;gBAChB,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,wBAAwB,EAAE,MAAM,EAAE,SAAS,EAAE,WAAW,EAAE,cAAc,CAAC,EAAE;gBAC1G,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,KAAK,CAAC,EAAE;gBAChE,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,cAAc,EAAE,QAAQ,EAAE,YAAY,CAAC,EAAE;aACzE;YACD,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC;YACtC,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,wBAAwB,EAAE,MAAM,EAAE,SAAS,EAAE,WAAW,EAAE,cAAc,CAAC,EAAE;YAC1G,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,cAAc,EAAE,QAAQ,EAAE,YAAY,CAAC,EAAE;SACzE,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mFAAmF,EAAE,GAAG,EAAE;QAC3F,MAAM,MAAM,GAAG,yBAAyB,CAAC,wCAAwC,EAAE;YACjF,eAAe,EAAE,wCAAwC;YACzD,UAAU,EAAE,SAAS;YACrB,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,qCAAqC;YACpD,kBAAkB,EAAE,EAAE;YACtB,gBAAgB,EAAE;gBAChB,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,CAAC,MAAM,EAAE,QAAQ,EAAE,YAAY,CAAC,EAAE;gBAC5D,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC,cAAc,EAAE,UAAU,CAAC,EAAE;aAC9D;YACD,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+EAA+E,EAAE,GAAG,EAAE;QACvF,MAAM,MAAM,GAAG,yBAAyB,CAAC,wCAAwC,EAAE;YACjF,eAAe,EAAE,wCAAwC;YACzD,UAAU,EAAE,SAAS;YACrB,eAAe,EAAE,EAAE;YACnB,WAAW,EAAE,EAAE;YACf,aAAa,EAAE,qCAAqC;YACpD,kBAAkB,EAAE,EAAE;YACtB,gBAAgB,EAAE;gBAChB,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,wBAAwB,EAAE,cAAc,CAAC,EAAE;gBAC1E,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC,QAAQ,EAAE,YAAY,EAAE,OAAO,CAAC,EAAE;gBAClE,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC,cAAc,EAAE,WAAW,EAAE,kBAAkB,CAAC,EAAE;aACnF;YACD,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC;YACtC,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,wBAAwB,EAAE,cAAc,CAAC,EAAE;YAC1E,EAAE,IAAI,EAAE,cAAc,EAAE,KAAK,EAAE,CAAC,cAAc,EAAE,WAAW,EAAE,kBAAkB,CAAC,EAAE;SACnF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["/**\n * Covers understand-owned coarse scope validation for the new agent.\n *\n * Why this file exists:\n * - the understand model should return coarse scope once\n * - runtime should only validate that scope, not classify it from scratch\n * - direct tests keep scope validation changes separate from later lane behavior\n */\nimport { describe, expect, it } from \"vitest\";\nimport { normalizeUnderstandResult } from \"../../agent/runtime/understand.js\";\n\ndescribe(\"normalizeUnderstandResult queryScope validation\", () => {\n  it(\"keeps a valid model-provided scope\", () => {\n    const result = normalizeUnderstandResult(\"Explain cli/src/agent/search/index.ts\", {\n      normalizedQuery: \"Explain cli/src/agent/search/index.ts\",\n      queryScope: \"broad-repo\",\n      explicitTargets: [\"cli/src/agent/search/index.ts\"],\n      constraints: [],\n      primaryIntent: \"Explain one file.\",\n      alternativeIntents: [],\n      ambiguityFlags: [],\n    });\n\n    expect(result.queryScope).toBe(\"broad-repo\");\n  });\n\n  it(\"falls back safely on an invalid scope\", () => {\n    const result = normalizeUnderstandResult(\"what is recursion\", {\n      normalizedQuery: \"what is recursion\",\n      queryScope: \"wide\" as never,\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Answer a general knowledge question.\",\n      alternativeIntents: [],\n      ambiguityFlags: [],\n    });\n\n    expect(result.queryScope).toBe(\"general\");\n  });\n\n  it(\"downgrades a generic tool-usage miss from targeted to general\", () => {\n    const result = normalizeUnderstandResult(\"how do I use git bash to save a diff?\", {\n      normalizedQuery: \"how do I use git bash to save a diff\",\n      queryScope: \"targeted\",\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Learn how to save a git diff output to a file using Git Bash.\",\n      alternativeIntents: [],\n      ambiguityFlags: [],\n    });\n\n    expect(result.queryScope).toBe(\"general\");\n  });\n\n  it(\"keeps a pure concept question general even when the model over-targets it\", () => {\n    const result = normalizeUnderstandResult(\"what is recursion?\", {\n      normalizedQuery: \"what is recursion\",\n      queryScope: \"targeted\",\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Answer a general programming concept question.\",\n      alternativeIntents: [],\n      ambiguityFlags: [],\n    });\n\n    expect(result.queryScope).toBe(\"general\");\n  });\n\n  it(\"keeps repo wording out of the general fallback\", () => {\n    const result = normalizeUnderstandResult(\"how does this repo run tests?\", {\n      normalizedQuery: \"how does this repo run tests\",\n      queryScope: \"broad-repo\",\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Explain how this repo runs tests.\",\n      alternativeIntents: [],\n      ambiguityFlags: [],\n    });\n\n    expect(result.queryScope).not.toBe(\"general\");\n  });\n\n  it(\"keeps explicit file questions targeted\", () => {\n    const result = normalizeUnderstandResult(\"explain cli/src/agent/index.ts\", {\n      normalizedQuery: \"explain cli/src/agent/index.ts\",\n      queryScope: \"general\",\n      explicitTargets: [\"cli/src/agent/index.ts\"],\n      constraints: [],\n      primaryIntent: \"Explain the file.\",\n      alternativeIntents: [],\n      ambiguityFlags: [],\n    });\n\n    expect(result.queryScope).toBe(\"targeted\");\n  });\n\n  it(\"upgrades general plus one explicit target to targeted\", () => {\n    const result = normalizeUnderstandResult(\"Explain cli/src/agent/search/index.ts\", {\n      normalizedQuery: \"Explain cli/src/agent/search/index.ts\",\n      queryScope: \"general\",\n      explicitTargets: [\"cli/src/agent/search/index.ts\"],\n      constraints: [],\n      primaryIntent: \"Explain one file.\",\n      alternativeIntents: [],\n      ambiguityFlags: [],\n    });\n\n    expect(result.queryScope).toBe(\"targeted\");\n  });\n\n  it(\"upgrades general plus many explicit targets to multi-file\", () => {\n    const result = normalizeUnderstandResult(\n      \"Explain how agentMemory and agentLoop divide responsibilities\",\n      {\n        normalizedQuery: \"Explain how agentMemory and agentLoop divide responsibilities\",\n        queryScope: \"general\",\n        explicitTargets: [\"agentMemory.ts\", \"agentLoop.ts\"],\n        constraints: [],\n        primaryIntent: \"Explain how agentMemory and agentLoop divide responsibilities.\",\n        alternativeIntents: [],\n        ambiguityFlags: [],\n      }\n    );\n\n    expect(result.queryScope).toBe(\"multi-file\");\n  });\n\n  it(\"upgrades targeted plus many explicit targets to multi-file\", () => {\n    const result = normalizeUnderstandResult(\n      \"Compare agentMemory.ts and agentLoop.ts\",\n      {\n        normalizedQuery: \"Compare agentMemory.ts and agentLoop.ts\",\n        queryScope: \"targeted\",\n        explicitTargets: [\"agentMemory.ts\", \"agentLoop.ts\"],\n        constraints: [],\n        primaryIntent: \"Compare the two files.\",\n        alternativeIntents: [],\n        ambiguityFlags: [],\n      }\n    );\n\n    expect(result.queryScope).toBe(\"multi-file\");\n  });\n\n  it(\"drops non-target pronouns from explicit targets\", () => {\n    const result = normalizeUnderstandResult(\n      \"summarize this repo architecture and identify weak coupling points\",\n      {\n        normalizedQuery: \"summarize repo architecture and identify weak coupling points\",\n        queryScope: \"broad-repo\",\n        explicitTargets: [\"this\"],\n        constraints: [],\n        primaryIntent: \"understand repo architecture\",\n        alternativeIntents: [],\n        ambiguityFlags: [],\n      }\n    );\n\n    expect(result.explicitTargets).toEqual([]);\n    expect(result.queryScope).toBe(\"broad-repo\");\n  });\n\n  it(\"drops discourse words like Also from explicit targets in resumed follow-ups\", () => {\n    const result = normalizeUnderstandResult(\n      \"did you check the memory.ts file? Also outline what you already checked\",\n      {\n        normalizedQuery: \"did you check the memory.ts file? Also outline what you already checked\",\n        queryScope: \"general\",\n        explicitTargets: [\"memory.ts\", \"Also\"],\n        constraints: [],\n        primaryIntent: \"Verify if memory.ts was checked and summarize prior checks.\",\n        alternativeIntents: [],\n        ambiguityFlags: [],\n      }\n    );\n\n    expect(result.explicitTargets).toEqual([\"memory.ts\"]);\n    expect(result.queryScope).toBe(\"targeted\");\n  });\n\n  it(\"normalizes bounded search term groups\", () => {\n    const result = normalizeUnderstandResult(\"where is max turns set\", {\n      normalizedQuery: \"where is max turns set\",\n      queryScope: \"general\",\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Find the owner for the turn limit.\",\n      alternativeIntents: [],\n      searchTermGroups: [\n        { kind: \"repo-anchor\", terms: [\" maxTurns \", \"turnCount\", \"maxTurns\", \"count\", \"loop\", \"limit\", \"extra\"] },\n        { kind: \"command-path\", terms: [\"memory.ts\", \"iteration\", \"constraint\", \"iteration\", \"limit\"] },\n        { kind: \"generic\", terms: [\"configuration\", \"setting\", \"configured\"] },\n        { kind: \"generic\", terms: [\"owner\", \"control\"] },\n        { kind: \"generic\", terms: [\"ignored\", \"group\"] },\n      ],\n      ambiguityFlags: [],\n    });\n\n    expect(result.searchTermGroups).toEqual([\n      { kind: \"repo-anchor\", terms: [\"maxTurns\", \"turnCount\", \"count\", \"loop\", \"limit\", \"extra\"] },\n      { kind: \"command-path\", terms: [\"memory.ts\", \"iteration\", \"constraint\", \"limit\"] },\n      { kind: \"generic\", terms: [\"configuration\", \"setting\", \"configured\"] },\n      { kind: \"generic\", terms: [\"owner\", \"control\"] },\n    ]);\n  });\n\n  it(\"keeps anchor-like search terms and drops repeated generic filler across groups\", () => {\n    const result = normalizeUnderstandResult(\"how do tests run\", {\n      normalizedQuery: \"how do tests run\",\n      queryScope: \"broad-repo\",\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Understand test execution.\",\n      alternativeIntents: [],\n      searchTermGroups: [\n        { kind: \"repo-anchor\", terms: [\"registerDevCliCommands\", \"test\", \"command\", \"/test-run\", \"package.json\"] },\n        { kind: \"generic\", terms: [\"test\", \"command\", \"config\", \"run\"] },\n        { kind: \"repo-anchor\", terms: [\"testCommands\", \"config\", \"entrypoint\"] },\n      ],\n      ambiguityFlags: [],\n    });\n\n    expect(result.searchTermGroups).toEqual([\n      { kind: \"repo-anchor\", terms: [\"registerDevCliCommands\", \"test\", \"command\", \"/test-run\", \"package.json\"] },\n      { kind: \"repo-anchor\", terms: [\"testCommands\", \"config\", \"entrypoint\"] },\n    ]);\n  });\n\n  it(\"drops all-generic typed groups when config-style queries need a repo-anchor group\", () => {\n    const result = normalizeUnderstandResult(\"where are test entry points configured\", {\n      normalizedQuery: \"where are test entry points configured\",\n      queryScope: \"general\",\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Find test entrypoint configuration.\",\n      alternativeIntents: [],\n      searchTermGroups: [\n        { kind: \"generic\", terms: [\"test\", \"config\", \"entrypoint\"] },\n        { kind: \"command-path\", terms: [\"package.json\", \"test-run\"] },\n      ],\n      ambiguityFlags: [],\n    });\n\n    expect(result.searchTermGroups).toEqual([]);\n  });\n\n  it(\"drops weak command-path groups that have no path, file, or command-like terms\", () => {\n    const result = normalizeUnderstandResult(\"where are test entry points configured\", {\n      normalizedQuery: \"where are test entry points configured\",\n      queryScope: \"general\",\n      explicitTargets: [],\n      constraints: [],\n      primaryIntent: \"Find test entrypoint configuration.\",\n      alternativeIntents: [],\n      searchTermGroups: [\n        { kind: \"repo-anchor\", terms: [\"registerDevCliCommands\", \"testCommands\"] },\n        { kind: \"command-path\", terms: [\"config\", \"entrypoint\", \"route\"] },\n        { kind: \"command-path\", terms: [\"package.json\", \"/test-run\", \"vitest.config.ts\"] },\n      ],\n      ambiguityFlags: [],\n    });\n\n    expect(result.searchTermGroups).toEqual([\n      { kind: \"repo-anchor\", terms: [\"registerDevCliCommands\", \"testCommands\"] },\n      { kind: \"command-path\", terms: [\"package.json\", \"/test-run\", \"vitest.config.ts\"] },\n    ]);\n  });\n});\n"]}

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

@@ -0,0 +1,52 @@
+// Verifies per-file validation results accumulate instead of replacing earlier files.
+// It uses deterministic artifacts so the step can validate without depending on model output.
+import { describe, expect, it } from "vitest";
+import { validateChangesStep } from "../../agents/validateChangesStep.js";
+// Pass example: validating "/repo/b.ts" keeps the earlier result for "/repo/a.ts" and appends the new one.
+// Fail example: each new validation replacing the previous file history.
+describe("validateChangesStep", () => {
+    it("preserves earlier validations when adding a new file result", async () => {
+        const context = {
+            task: {
+                currentStep: {
+                    filePath: "/repo/b.ts",
+                },
+            },
+            analysis: {
+                fileAnalysis: {
+                    "/repo/b.ts": {},
+                },
+                executionOutcome: {
+                    executedStepIds: [],
+                    validations: [
+                        {
+                            filePath: "/repo/a.ts",
+                            expectedChange: "change a",
+                            actualChange: "changed",
+                            status: "valid",
+                            rationale: "ok",
+                            requiresRedo: false,
+                        },
+                    ],
+                },
+            },
+            execution: {
+                codeTransformArtifacts: {
+                    files: [
+                        {
+                            filePath: "/repo/b.ts",
+                            content: "updated",
+                        },
+                    ],
+                },
+            },
+        };
+        await validateChangesStep.run(context);
+        expect(context.analysis.executionOutcome.validations).toHaveLength(2);
+        expect(context.analysis.executionOutcome.validations.map((v) => v.filePath)).toEqual([
+            "/repo/a.ts",
+            "/repo/b.ts",
+        ]);
+    });
+});
+//# sourceMappingURL=validateChangesStep.test.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"validateChangesStep.test.js","sourceRoot":"","sources":["../../../src/__tests__/agents/validateChangesStep.test.ts"],"names":[],"mappings":"AAAA,sFAAsF;AACtF,8FAA8F;AAC9F,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,mBAAmB,EAAE,MAAM,qCAAqC,CAAC;AAE1E,2GAA2G;AAC3G,yEAAyE;AACzE,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QACzE,MAAM,OAAO,GAAG;YACZ,IAAI,EAAE;gBACF,WAAW,EAAE;oBACT,QAAQ,EAAE,YAAY;iBACzB;aACJ;YACD,QAAQ,EAAE;gBACN,YAAY,EAAE;oBACV,YAAY,EAAE,EAAE;iBACnB;gBACD,gBAAgB,EAAE;oBACd,eAAe,EAAE,EAAE;oBACnB,WAAW,EAAE;wBACT;4BACI,QAAQ,EAAE,YAAY;4BACtB,cAAc,EAAE,UAAU;4BAC1B,YAAY,EAAE,SAAS;4BACvB,MAAM,EAAE,OAAO;4BACf,SAAS,EAAE,IAAI;4BACf,YAAY,EAAE,KAAK;yBACtB;qBACJ;iBACJ;aACJ;YACD,SAAS,EAAE;gBACP,sBAAsB,EAAE;oBACpB,KAAK,EAAE;wBACH;4BACI,QAAQ,EAAE,YAAY;4BACtB,OAAO,EAAE,SAAS;yBACrB;qBACJ;iBACJ;aACJ;SACG,CAAC;QAET,MAAM,mBAAmB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAEvC,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAC,WAAW,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACtE,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC,CAAuB,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,OAAO,CAAC;YACvG,YAAY;YACZ,YAAY;SACf,CAAC,CAAC;IACP,CAAC,CAAC,CAAC;AACP,CAAC,CAAC,CAAC","sourcesContent":["// Verifies per-file validation results accumulate instead of replacing earlier files.\n// It uses deterministic artifacts so the step can validate without depending on model output.\nimport { describe, expect, it } from \"vitest\";\nimport { validateChangesStep } from \"../../agents/validateChangesStep.js\";\n\n// Pass example: validating \"/repo/b.ts\" keeps the earlier result for \"/repo/a.ts\" and appends the new one.\n// Fail example: each new validation replacing the previous file history.\ndescribe(\"validateChangesStep\", () => {\n    it(\"preserves earlier validations when adding a new file result\", async () => {\n        const context = {\n            task: {\n                currentStep: {\n                    filePath: \"/repo/b.ts\",\n                },\n            },\n            analysis: {\n                fileAnalysis: {\n                    \"/repo/b.ts\": {},\n                },\n                executionOutcome: {\n                    executedStepIds: [],\n                    validations: [\n                        {\n                            filePath: \"/repo/a.ts\",\n                            expectedChange: \"change a\",\n                            actualChange: \"changed\",\n                            status: \"valid\",\n                            rationale: \"ok\",\n                            requiresRedo: false,\n                        },\n                    ],\n                },\n            },\n            execution: {\n                codeTransformArtifacts: {\n                    files: [\n                        {\n                            filePath: \"/repo/b.ts\",\n                            content: \"updated\",\n                        },\n                    ],\n                },\n            },\n        } as any;\n\n        await validateChangesStep.run(context);\n\n        expect(context.analysis.executionOutcome.validations).toHaveLength(2);\n        expect(context.analysis.executionOutcome.validations.map((v: { filePath: string }) => v.filePath)).toEqual([\n            \"/repo/a.ts\",\n            \"/repo/b.ts\",\n        ]);\n    });\n});\n"]}

package/dist/__tests__/askCommandTaskBinding.test.js

@@ -0,0 +1,176 @@
+/**
+ * Covers ask-command task binding defaults for the modular agent runtime.
+ *
+ * Why this file exists:
+ * - the shared ask entry should stay new-agent only
+ * - task binding rules should stay explicit for one-shot and plan-only runs
+ */
+import { describe, expect, it, vi, beforeEach } from "vitest";
+import { AgentRuntimeError } from "../agent/types.js";
+function stripAnsi(text) {
+    return text.replace(/\x1B\[[0-9;]*m/g, "");
+}
+vi.mock("../utils/checkModel.js", () => ({}));
+vi.mock(import("../constants.js"), async (importOriginal) => {
+    const actual = await importOriginal();
+    return {
+        ...actual,
+        getIndexDir: () => "/repo",
+        RELATED_FILES_LIMIT: 50,
+    };
+});
+vi.mock("../utils/promptLogHelper.js", () => ({
+    clearRunLog: vi.fn(),
+}));
+const logRuntimeRecord = vi.fn();
+const logRunStart = vi.fn();
+vi.mock("../utils/runtimeLogger.js", () => ({
+    logRunStart,
+    logRuntimeRecord,
+}));
+vi.mock("../db/schema.js", () => ({
+    initSchema: vi.fn(),
+}));
+vi.mock("../db/client.js", () => ({
+    getDbForRepo: vi.fn(() => ({})),
+}));
+vi.mock("../agent/persistence/index.js", () => ({
+    resolveAgentResumeTaskId: vi.fn((resumeTaskId, continueCurrentTask) => {
+        if (typeof resumeTaskId === "number") {
+            return resumeTaskId;
+        }
+        return continueCurrentTask === false ? undefined : 99;
+    }),
+}));
+const agentRunSpy = vi.fn();
+const agentCtor = vi.fn().mockImplementation((query, _ui, _memory, options) => ({
+    query,
+    options,
+    run: agentRunSpy,
+}));
+vi.mock("../agent/index.js", () => ({
+    Agent: agentCtor,
+}));
+describe("runAskCommand task binding", () => {
+    beforeEach(() => {
+        agentRunSpy.mockReset();
+        agentCtor.mockClear();
+        logRuntimeRecord.mockReset();
+        logRunStart.mockReset();
+    });
+    it("defaults normal ask runs to the new agent runtime", async () => {
+        const { runAskCommand } = await import("../commands/AskCmd.js");
+        agentRunSpy.mockResolvedValueOnce({
+            task: { id: "task-default" },
+            artifacts: { finalAnswer: "Default answer." },
+        });
+        await runAskCommand("fresh default", { continueCurrentTask: false });
+        expect(agentCtor).toHaveBeenCalledWith("fresh default", expect.any(Object), undefined, {
+            planOnly: false,
+            resumeTaskId: undefined,
+            bindAsCurrentTask: false,
+        });
+        expect(agentCtor).toHaveBeenCalledTimes(1);
+    });
+    it("uses the new agent runtime for one-shot agent runs", async () => {
+        const { runAskCommand } = await import("../commands/AskCmd.js");
+        const logSpy = vi.spyOn(console, "log").mockImplementation(() => { });
+        agentRunSpy.mockResolvedValueOnce({
+            task: { id: "task-1" },
+            artifacts: { finalAnswer: "Simple answer." },
+        });
+        await runAskCommand("fresh agent", { runtime: "agent", continueCurrentTask: false });
+        expect(agentCtor).toHaveBeenCalledWith("fresh agent", expect.any(Object), undefined, {
+            planOnly: false,
+            resumeTaskId: undefined,
+            bindAsCurrentTask: false,
+        });
+        expect(agentRunSpy).toHaveBeenCalledTimes(1);
+        expect(logSpy).toHaveBeenCalledWith("Using index root: /repo");
+        expect(logSpy).toHaveBeenCalledWith("Launching simple agent...");
+        expect(logSpy.mock.calls.some(([value]) => typeof value === "string" && stripAnsi(value).includes("[FINAL ANSWER]\nSimple answer."))).toBe(true);
+        expect(logRuntimeRecord).toHaveBeenCalledWith(expect.objectContaining({
+            runtime: "agent",
+            kind: "task",
+            phase: "finalized",
+            status: "done",
+        }));
+        logSpy.mockRestore();
+    });
+    it("passes explicit plan-only runs through the new agent", async () => {
+        const { runAskCommand } = await import("../commands/AskCmd.js");
+        agentRunSpy.mockResolvedValueOnce({
+            task: { id: "task-plan" },
+            artifacts: { finalAnswer: "Plan goal: map routing\n\nSteps:\n1. Read index.ts (read-file)" },
+        });
+        await runAskCommand("map routing", {
+            planOnly: true,
+            continueCurrentTask: false,
+        });
+        expect(agentCtor).toHaveBeenCalledWith("map routing", expect.any(Object), undefined, {
+            planOnly: true,
+            resumeTaskId: undefined,
+            bindAsCurrentTask: false,
+        });
+    });
+    it("prints an honest blocked-agent message when the simple agent cannot answer", async () => {
+        const { runAskCommand } = await import("../commands/AskCmd.js");
+        const logSpy = vi.spyOn(console, "log").mockImplementation(() => { });
+        agentRunSpy.mockResolvedValueOnce({
+            task: { id: "task-2", status: "blocked", stopReason: "max-turns" },
+            artifacts: {
+                stopMessage: "I could not answer from grounded repo evidence.",
+            },
+        });
+        await runAskCommand("fresh agent", { continueCurrentTask: false });
+        expect(logSpy).toHaveBeenCalledWith("I could not answer from grounded repo evidence.");
+        logSpy.mockRestore();
+    });
+    it("prints a waiting-for-input message when the simple agent needs one concrete follow-up", async () => {
+        const { runAskCommand } = await import("../commands/AskCmd.js");
+        const logSpy = vi.spyOn(console, "log").mockImplementation(() => { });
+        agentRunSpy.mockResolvedValueOnce({
+            task: { id: "task-3", status: "waiting", stopReason: "need-user-input" },
+            artifacts: {
+                stopMessage: "I need one thing from you:\n- the exact file path to inspect next",
+            },
+        });
+        await runAskCommand("fresh agent", { continueCurrentTask: false });
+        expect(logSpy).toHaveBeenCalledWith("I need one thing from you:\n- the exact file path to inspect next");
+        logSpy.mockRestore();
+    });
+    it("prints structured simple-agent failures through the shared UI path", async () => {
+        const logSpy = vi.spyOn(console, "log").mockImplementation(() => { });
+        agentRunSpy.mockRejectedValueOnce(new AgentRuntimeError({
+            stage: "validation",
+            message: "Invalid args for read-file: target is required",
+            stepId: "step-001",
+            action: "read-file",
+        }));
+        const { runAskCommand } = await import("../commands/AskCmd.js");
+        await runAskCommand("fresh agent", { continueCurrentTask: false });
+        expect(logSpy).toHaveBeenCalledWith([
+            "❌ Simple agent failed at validation.",
+            "Step: step-001",
+            "Action: read-file",
+            "Reason: Invalid args for read-file: target is required",
+            "Use /context to inspect the task, or check runtime.log for details.",
+        ].join("\n"));
+        expect(logRuntimeRecord).toHaveBeenCalled();
+        logSpy.mockRestore();
+    });
+    it("prints top-level agent failures through the shared UI path", async () => {
+        const logSpy = vi.spyOn(console, "log").mockImplementation(() => { });
+        agentRunSpy.mockRejectedValueOnce(new Error("boom"));
+        const { runAskCommand } = await import("../commands/AskCmd.js");
+        await runAskCommand("fresh agent", { continueCurrentTask: false });
+        expect(logSpy).toHaveBeenCalledWith("simple agent run failed: boom");
+        expect(logRuntimeRecord).toHaveBeenCalledWith(expect.objectContaining({
+            runtime: "agent",
+            kind: "error",
+            phase: "failed",
+        }));
+        logSpy.mockRestore();
+    });
+});
+//# sourceMappingURL=askCommandTaskBinding.test.js.map

package/dist/__tests__/askCommandTaskBinding.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"askCommandTaskBinding.test.js","sourceRoot":"","sources":["../../src/__tests__/askCommandTaskBinding.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,iBAAiB,EAAE,MAAM,mBAAmB,CAAC;AAEtD,SAAS,SAAS,CAAC,IAAY;IAC3B,OAAO,IAAI,CAAC,OAAO,CAAC,iBAAiB,EAAE,EAAE,CAAC,CAAC;AAC/C,CAAC;AAED,EAAE,CAAC,IAAI,CAAC,wBAAwB,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC9C,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC,iBAAiB,CAAC,EAAE,KAAK,EAAE,cAAc,EAAE,EAAE;IACxD,MAAM,MAAM,GAAG,MAAM,cAAc,EAAE,CAAC;IACtC,OAAO;QACH,GAAG,MAAM;QACT,WAAW,EAAE,GAAG,EAAE,CAAC,OAAO;QAC1B,mBAAmB,EAAE,EAAW;KACnC,CAAC;AACN,CAAC,CAAC,CAAC;AACH,EAAE,CAAC,IAAI,CAAC,6BAA6B,EAAE,GAAG,EAAE,CAAC,CAAC;IAC1C,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;CACvB,CAAC,CAAC,CAAC;AACJ,MAAM,gBAAgB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AACjC,MAAM,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC5B,EAAE,CAAC,IAAI,CAAC,2BAA2B,EAAE,GAAG,EAAE,CAAC,CAAC;IACxC,WAAW;IACX,gBAAgB;CACnB,CAAC,CAAC,CAAC;AACJ,EAAE,CAAC,IAAI,CAAC,iBAAiB,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9B,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;CACtB,CAAC,CAAC,CAAC;AACJ,EAAE,CAAC,IAAI,CAAC,iBAAiB,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9B,YAAY,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC;CAClC,CAAC,CAAC,CAAC;AACJ,EAAE,CAAC,IAAI,CAAC,+BAA+B,EAAE,GAAG,EAAE,CAAC,CAAC;IAC5C,wBAAwB,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,YAAqB,EAAE,mBAA6B,EAAE,EAAE;QACrF,IAAI,OAAO,YAAY,KAAK,QAAQ,EAAE,CAAC;YACnC,OAAO,YAAY,CAAC;QACxB,CAAC;QACD,OAAO,mBAAmB,KAAK,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC;IAC1D,CAAC,CAAC;CACL,CAAC,CAAC,CAAC;AAEJ,MAAM,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC5B,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,kBAAkB,CAAC,CAAC,KAAK,EAAE,GAAG,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,CAAC,CAAC;IAC5E,KAAK;IACL,OAAO;IACP,GAAG,EAAE,WAAW;CACnB,CAAC,CAAC,CAAC;AACJ,EAAE,CAAC,IAAI,CAAC,mBAAmB,EAAE,GAAG,EAAE,CAAC,CAAC;IAChC,KAAK,EAAE,SAAS;CACnB,CAAC,CAAC,CAAC;AAEJ,QAAQ,CAAC,4BAA4B,EAAE,GAAG,EAAE;IACxC,UAAU,CAAC,GAAG,EAAE;QACZ,WAAW,CAAC,SAAS,EAAE,CAAC;QACxB,SAAS,CAAC,SAAS,EAAE,CAAC;QACtB,gBAAgB,CAAC,SAAS,EAAE,CAAC;QAC7B,WAAW,CAAC,SAAS,EAAE,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAAC,uBAAuB,CAAC,CAAC;QAChE,WAAW,CAAC,qBAAqB,CAAC;YAC9B,IAAI,EAAE,EAAE,EAAE,EAAE,cAAc,EAAE;YAC5B,SAAS,EAAE,EAAE,WAAW,EAAE,iBAAiB,EAAE;SAChD,CAAC,CAAC;QAEH,MAAM,aAAa,CAAC,eAAe,EAAE,EAAE,mBAAmB,EAAE,KAAK,EAAE,CAAC,CAAC;QAErE,MAAM,CAAC,SAAS,CAAC,CAAC,oBAAoB,CAAC,eAAe,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,SAAS,EAAE;YACnF,QAAQ,EAAE,KAAK;YACf,YAAY,EAAE,SAAS;YACvB,iBAAiB,EAAE,KAAK;SAC3B,CAAC,CAAC;QACH,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAChE,MAAM,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAAC,uBAAuB,CAAC,CAAC;QAChE,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACrE,WAAW,CAAC,qBAAqB,CAAC;YAC9B,IAAI,EAAE,EAAE,EAAE,EAAE,QAAQ,EAAE;YACtB,SAAS,EAAE,EAAE,WAAW,EAAE,gBAAgB,EAAE;SAC/C,CAAC,CAAC;QAEH,MAAM,aAAa,CAAC,aAAa,EAAE,EAAE,OAAO,EAAE,OAAO,EAAE,mBAAmB,EAAE,KAAK,EAAE,CAAC,CAAC;QAErF,MAAM,CAAC,SAAS,CAAC,CAAC,oBAAoB,CAAC,aAAa,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,SAAS,EAAE;YACjF,QAAQ,EAAE,KAAK;YACf,YAAY,EAAE,SAAS;YACvB,iBAAiB,EAAE,KAAK;SAC3B,CAAC,CAAC;QACH,MAAM,CAAC,WAAW,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAAC,yBAAyB,CAAC,CAAC;QAC/D,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAAC,2BAA2B,CAAC,CAAC;QACjE,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CACzB,CAAC,CAAC,KAAK,CAAC,EAAE,EAAE,CAAC,OAAO,KAAK,KAAK,QAAQ,IAAI,SAAS,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC,gCAAgC,CAAC,CACxG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACd,MAAM,CAAC,gBAAgB,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC;YAClE,OAAO,EAAE,OAAO;YAChB,IAAI,EAAE,MAAM;YACZ,KAAK,EAAE,WAAW;YAClB,MAAM,EAAE,MAAM;SACjB,CAAC,CAAC,CAAC;QACJ,MAAM,CAAC,WAAW,EAAE,CAAC;IACzB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAAC,uBAAuB,CAAC,CAAC;QAChE,WAAW,CAAC,qBAAqB,CAAC;YAC9B,IAAI,EAAE,EAAE,EAAE,EAAE,WAAW,EAAE;YACzB,SAAS,EAAE,EAAE,WAAW,EAAE,gEAAgE,EAAE;SAC/F,CAAC,CAAC;QAEH,MAAM,aAAa,CAAC,aAAa,EAAE;YAC/B,QAAQ,EAAE,IAAI;YACd,mBAAmB,EAAE,KAAK;SAC7B,CAAC,CAAC;QAEH,MAAM,CAAC,SAAS,CAAC,CAAC,oBAAoB,CAAC,aAAa,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,SAAS,EAAE;YACjF,QAAQ,EAAE,IAAI;YACd,YAAY,EAAE,SAAS;YACvB,iBAAiB,EAAE,KAAK;SAC3B,CAAC,CAAC;IACP,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QACxF,MAAM,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAAC,uBAAuB,CAAC,CAAC;QAChE,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACrE,WAAW,CAAC,qBAAqB,CAAC;YAC9B,IAAI,EAAE,EAAE,EAAE,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,WAAW,EAAE;YAClE,SAAS,EAAE;gBACP,WAAW,EAAE,iDAAiD;aACjE;SACJ,CAAC,CAAC;QAEH,MAAM,aAAa,CAAC,aAAa,EAAE,EAAE,mBAAmB,EAAE,KAAK,EAAE,CAAC,CAAC;QAEnE,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAAC,iDAAiD,CAAC,CAAC;QACvF,MAAM,CAAC,WAAW,EAAE,CAAC;IACzB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uFAAuF,EAAE,KAAK,IAAI,EAAE;QACnG,MAAM,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAAC,uBAAuB,CAAC,CAAC;QAChE,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACrE,WAAW,CAAC,qBAAqB,CAAC;YAC9B,IAAI,EAAE,EAAE,EAAE,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,iBAAiB,EAAE;YACxE,SAAS,EAAE;gBACP,WAAW,EAAE,mEAAmE;aACnF;SACJ,CAAC,CAAC;QAEH,MAAM,aAAa,CAAC,aAAa,EAAE,EAAE,mBAAmB,EAAE,KAAK,EAAE,CAAC,CAAC;QAEnE,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAAC,mEAAmE,CAAC,CAAC;QACzG,MAAM,CAAC,WAAW,EAAE,CAAC;IACzB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;QAChF,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACrE,WAAW,CAAC,qBAAqB,CAAC,IAAI,iBAAiB,CAAC;YACpD,KAAK,EAAE,YAAY;YACnB,OAAO,EAAE,gDAAgD;YACzD,MAAM,EAAE,UAAU;YAClB,MAAM,EAAE,WAAW;SACtB,CAAC,CAAC,CAAC;QACJ,MAAM,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAAC,uBAAuB,CAAC,CAAC;QAEhE,MAAM,aAAa,CAAC,aAAa,EAAE,EAAE,mBAAmB,EAAE,KAAK,EAAE,CAAC,CAAC;QAEnE,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAAC;YAChC,sCAAsC;YACtC,gBAAgB;YAChB,mBAAmB;YACnB,wDAAwD;YACxD,qEAAqE;SACxE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QACd,MAAM,CAAC,gBAAgB,CAAC,CAAC,gBAAgB,EAAE,CAAC;QAC5C,MAAM,CAAC,WAAW,EAAE,CAAC;IACzB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QACxE,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACrE,WAAW,CAAC,qBAAqB,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC;QACrD,MAAM,EAAE,aAAa,EAAE,GAAG,MAAM,MAAM,CAAC,uBAAuB,CAAC,CAAC;QAEhE,MAAM,aAAa,CAAC,aAAa,EAAE,EAAE,mBAAmB,EAAE,KAAK,EAAE,CAAC,CAAC;QAEnE,MAAM,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAAC,+BAA+B,CAAC,CAAC;QACrE,MAAM,CAAC,gBAAgB,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC;YAClE,OAAO,EAAE,OAAO;YAChB,IAAI,EAAE,OAAO;YACb,KAAK,EAAE,QAAQ;SAClB,CAAC,CAAC,CAAC;QACJ,MAAM,CAAC,WAAW,EAAE,CAAC;IACzB,CAAC,CAAC,CAAC;AACP,CAAC,CAAC,CAAC","sourcesContent":["/**\n * Covers ask-command task binding defaults for the modular agent runtime.\n *\n * Why this file exists:\n * - the shared ask entry should stay new-agent only\n * - task binding rules should stay explicit for one-shot and plan-only runs\n */\nimport { describe, expect, it, vi, beforeEach } from \"vitest\";\nimport { AgentRuntimeError } from \"../agent/types.js\";\n\nfunction stripAnsi(text: string): string {\n    return text.replace(/\\x1B\\[[0-9;]*m/g, \"\");\n}\n\nvi.mock(\"../utils/checkModel.js\", () => ({}));\nvi.mock(import(\"../constants.js\"), async (importOriginal) => {\n    const actual = await importOriginal();\n    return {\n        ...actual,\n        getIndexDir: () => \"/repo\",\n        RELATED_FILES_LIMIT: 50 as const,\n    };\n});\nvi.mock(\"../utils/promptLogHelper.js\", () => ({\n    clearRunLog: vi.fn(),\n}));\nconst logRuntimeRecord = vi.fn();\nconst logRunStart = vi.fn();\nvi.mock(\"../utils/runtimeLogger.js\", () => ({\n    logRunStart,\n    logRuntimeRecord,\n}));\nvi.mock(\"../db/schema.js\", () => ({\n    initSchema: vi.fn(),\n}));\nvi.mock(\"../db/client.js\", () => ({\n    getDbForRepo: vi.fn(() => ({})),\n}));\nvi.mock(\"../agent/persistence/index.js\", () => ({\n    resolveAgentResumeTaskId: vi.fn((resumeTaskId?: number, continueCurrentTask?: boolean) => {\n        if (typeof resumeTaskId === \"number\") {\n            return resumeTaskId;\n        }\n        return continueCurrentTask === false ? undefined : 99;\n    }),\n}));\n\nconst agentRunSpy = vi.fn();\nconst agentCtor = vi.fn().mockImplementation((query, _ui, _memory, options) => ({\n    query,\n    options,\n    run: agentRunSpy,\n}));\nvi.mock(\"../agent/index.js\", () => ({\n    Agent: agentCtor,\n}));\n\ndescribe(\"runAskCommand task binding\", () => {\n    beforeEach(() => {\n        agentRunSpy.mockReset();\n        agentCtor.mockClear();\n        logRuntimeRecord.mockReset();\n        logRunStart.mockReset();\n    });\n\n    it(\"defaults normal ask runs to the new agent runtime\", async () => {\n        const { runAskCommand } = await import(\"../commands/AskCmd.js\");\n        agentRunSpy.mockResolvedValueOnce({\n            task: { id: \"task-default\" },\n            artifacts: { finalAnswer: \"Default answer.\" },\n        });\n\n        await runAskCommand(\"fresh default\", { continueCurrentTask: false });\n\n        expect(agentCtor).toHaveBeenCalledWith(\"fresh default\", expect.any(Object), undefined, {\n            planOnly: false,\n            resumeTaskId: undefined,\n            bindAsCurrentTask: false,\n        });\n        expect(agentCtor).toHaveBeenCalledTimes(1);\n    });\n\n    it(\"uses the new agent runtime for one-shot agent runs\", async () => {\n        const { runAskCommand } = await import(\"../commands/AskCmd.js\");\n        const logSpy = vi.spyOn(console, \"log\").mockImplementation(() => {});\n        agentRunSpy.mockResolvedValueOnce({\n            task: { id: \"task-1\" },\n            artifacts: { finalAnswer: \"Simple answer.\" },\n        });\n\n        await runAskCommand(\"fresh agent\", { runtime: \"agent\", continueCurrentTask: false });\n\n        expect(agentCtor).toHaveBeenCalledWith(\"fresh agent\", expect.any(Object), undefined, {\n            planOnly: false,\n            resumeTaskId: undefined,\n            bindAsCurrentTask: false,\n        });\n        expect(agentRunSpy).toHaveBeenCalledTimes(1);\n        expect(logSpy).toHaveBeenCalledWith(\"Using index root: /repo\");\n        expect(logSpy).toHaveBeenCalledWith(\"Launching simple agent...\");\n        expect(logSpy.mock.calls.some(\n            ([value]) => typeof value === \"string\" && stripAnsi(value).includes(\"[FINAL ANSWER]\\nSimple answer.\")\n        )).toBe(true);\n        expect(logRuntimeRecord).toHaveBeenCalledWith(expect.objectContaining({\n            runtime: \"agent\",\n            kind: \"task\",\n            phase: \"finalized\",\n            status: \"done\",\n        }));\n        logSpy.mockRestore();\n    });\n\n    it(\"passes explicit plan-only runs through the new agent\", async () => {\n        const { runAskCommand } = await import(\"../commands/AskCmd.js\");\n        agentRunSpy.mockResolvedValueOnce({\n            task: { id: \"task-plan\" },\n            artifacts: { finalAnswer: \"Plan goal: map routing\\n\\nSteps:\\n1. Read index.ts (read-file)\" },\n        });\n\n        await runAskCommand(\"map routing\", {\n            planOnly: true,\n            continueCurrentTask: false,\n        });\n\n        expect(agentCtor).toHaveBeenCalledWith(\"map routing\", expect.any(Object), undefined, {\n            planOnly: true,\n            resumeTaskId: undefined,\n            bindAsCurrentTask: false,\n        });\n    });\n\n    it(\"prints an honest blocked-agent message when the simple agent cannot answer\", async () => {\n        const { runAskCommand } = await import(\"../commands/AskCmd.js\");\n        const logSpy = vi.spyOn(console, \"log\").mockImplementation(() => {});\n        agentRunSpy.mockResolvedValueOnce({\n            task: { id: \"task-2\", status: \"blocked\", stopReason: \"max-turns\" },\n            artifacts: {\n                stopMessage: \"I could not answer from grounded repo evidence.\",\n            },\n        });\n\n        await runAskCommand(\"fresh agent\", { continueCurrentTask: false });\n\n        expect(logSpy).toHaveBeenCalledWith(\"I could not answer from grounded repo evidence.\");\n        logSpy.mockRestore();\n    });\n\n    it(\"prints a waiting-for-input message when the simple agent needs one concrete follow-up\", async () => {\n        const { runAskCommand } = await import(\"../commands/AskCmd.js\");\n        const logSpy = vi.spyOn(console, \"log\").mockImplementation(() => {});\n        agentRunSpy.mockResolvedValueOnce({\n            task: { id: \"task-3\", status: \"waiting\", stopReason: \"need-user-input\" },\n            artifacts: {\n                stopMessage: \"I need one thing from you:\\n- the exact file path to inspect next\",\n            },\n        });\n\n        await runAskCommand(\"fresh agent\", { continueCurrentTask: false });\n\n        expect(logSpy).toHaveBeenCalledWith(\"I need one thing from you:\\n- the exact file path to inspect next\");\n        logSpy.mockRestore();\n    });\n\n    it(\"prints structured simple-agent failures through the shared UI path\", async () => {\n        const logSpy = vi.spyOn(console, \"log\").mockImplementation(() => {});\n        agentRunSpy.mockRejectedValueOnce(new AgentRuntimeError({\n            stage: \"validation\",\n            message: \"Invalid args for read-file: target is required\",\n            stepId: \"step-001\",\n            action: \"read-file\",\n        }));\n        const { runAskCommand } = await import(\"../commands/AskCmd.js\");\n\n        await runAskCommand(\"fresh agent\", { continueCurrentTask: false });\n\n        expect(logSpy).toHaveBeenCalledWith([\n            \"❌ Simple agent failed at validation.\",\n            \"Step: step-001\",\n            \"Action: read-file\",\n            \"Reason: Invalid args for read-file: target is required\",\n            \"Use /context to inspect the task, or check runtime.log for details.\",\n        ].join(\"\\n\"));\n        expect(logRuntimeRecord).toHaveBeenCalled();\n        logSpy.mockRestore();\n    });\n\n    it(\"prints top-level agent failures through the shared UI path\", async () => {\n        const logSpy = vi.spyOn(console, \"log\").mockImplementation(() => {});\n        agentRunSpy.mockRejectedValueOnce(new Error(\"boom\"));\n        const { runAskCommand } = await import(\"../commands/AskCmd.js\");\n\n        await runAskCommand(\"fresh agent\", { continueCurrentTask: false });\n\n        expect(logSpy).toHaveBeenCalledWith(\"simple agent run failed: boom\");\n        expect(logRuntimeRecord).toHaveBeenCalledWith(expect.objectContaining({\n            runtime: \"agent\",\n            kind: \"error\",\n            phase: \"failed\",\n        }));\n        logSpy.mockRestore();\n    });\n});\n"]}

package/dist/__tests__/commandVisibility.test.js

@@ -0,0 +1,25 @@
+/**
+ * Verifies that dev-only commands require the explicit env flag.
+ *
+ * Why this file exists:
+ * - the public command surface should stay small by default
+ * - maintainers still need an explicit opt-in for eval and test helpers
+ */
+import { afterEach, describe, expect, it } from "vitest";
+const ORIGINAL_ENV = { ...process.env };
+afterEach(() => {
+    process.env = { ...ORIGINAL_ENV };
+});
+describe("isDevCommandsEnabled", () => {
+    it("is disabled by default", async () => {
+        delete process.env.SCAI_ENABLE_DEV_COMMANDS;
+        const { isDevCommandsEnabled } = await import("../commands/commandVisibility.js");
+        expect(isDevCommandsEnabled()).toBe(false);
+    });
+    it("only enables when the explicit env flag is set", async () => {
+        process.env.SCAI_ENABLE_DEV_COMMANDS = "1";
+        const { isDevCommandsEnabled } = await import("../commands/commandVisibility.js");
+        expect(isDevCommandsEnabled()).toBe(true);
+    });
+});
+//# sourceMappingURL=commandVisibility.test.js.map

package/dist/__tests__/commandVisibility.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"commandVisibility.test.js","sourceRoot":"","sources":["../../src/__tests__/commandVisibility.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEzD,MAAM,YAAY,GAAG,EAAE,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;AAExC,SAAS,CAAC,GAAG,EAAE;IACb,OAAO,CAAC,GAAG,GAAG,EAAE,GAAG,YAAY,EAAE,CAAC;AACpC,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,EAAE,CAAC,wBAAwB,EAAE,KAAK,IAAI,EAAE;QACtC,OAAO,OAAO,CAAC,GAAG,CAAC,wBAAwB,CAAC;QAC5C,MAAM,EAAE,oBAAoB,EAAE,GAAG,MAAM,MAAM,CAAC,kCAAkC,CAAC,CAAC;QAClF,MAAM,CAAC,oBAAoB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,OAAO,CAAC,GAAG,CAAC,wBAAwB,GAAG,GAAG,CAAC;QAC3C,MAAM,EAAE,oBAAoB,EAAE,GAAG,MAAM,MAAM,CAAC,kCAAkC,CAAC,CAAC;QAClF,MAAM,CAAC,oBAAoB,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["/**\n * Verifies that dev-only commands require the explicit env flag.\n *\n * Why this file exists:\n * - the public command surface should stay small by default\n * - maintainers still need an explicit opt-in for eval and test helpers\n */\nimport { afterEach, describe, expect, it } from \"vitest\";\n\nconst ORIGINAL_ENV = { ...process.env };\n\nafterEach(() => {\n  process.env = { ...ORIGINAL_ENV };\n});\n\ndescribe(\"isDevCommandsEnabled\", () => {\n  it(\"is disabled by default\", async () => {\n    delete process.env.SCAI_ENABLE_DEV_COMMANDS;\n    const { isDevCommandsEnabled } = await import(\"../commands/commandVisibility.js\");\n    expect(isDevCommandsEnabled()).toBe(false);\n  });\n\n  it(\"only enables when the explicit env flag is set\", async () => {\n    process.env.SCAI_ENABLE_DEV_COMMANDS = \"1\";\n    const { isDevCommandsEnabled } = await import(\"../commands/commandVisibility.js\");\n    expect(isDevCommandsEnabled()).toBe(true);\n  });\n});\n"]}