@koda-sl/baker-bridge 0.168.0-dev.d78500250 → 0.172.0-dev.b570aa3f2

package/dist/hono/bridge-job.test.js

@@ -262,8 +262,9 @@ function createExecutor(options) {
         ...(options.catchUpWithMain ? { catchUpWithMain: options.catchUpWithMain } : {}),
         ...(options.keepCliInStep ? { keepCliInStep: options.keepCliInStep } : {}),
         ...(options.holdsNoWork ? { holdsNoWork: options.holdsNoWork } : {}),
-        // Never the network: a turn nobody judges runs at the default effort.
-        effortJudge: options.effortJudge ?? (async () => null),
+        // Never the network: a turn nobody judges runs on the default model and effort.
+        routeJudge: options.routeJudge ?? (async () => null),
+        ...(options.routeModels ? { routeModels: true } : {}),
     });
 }
 function memoryInFlightTurns(initial) {
@@ -1345,6 +1346,17 @@ describe("BridgeJobExecutor", () => {
         await executor.execute({ threadId: "thread_bedrock", turnId: "turn_bedrock", prompt: "Build a landing", history: history([]) }, activeControls());
         expect(queryInputs[0]?.options.env).toMatchObject({ CLAUDE_CODE_MODEL_CAPABILITIES: "per_turn_effort" });
     });
+    // The CLI names each session with a structured-output request OpenRouter's Bedrock
+    // cannot serve, so it went to Azure on OpenRouter credits; nothing reads the name.
+    it("never sends the CLI's session-title request", async () => {
+        const queryInputs = [];
+        const executor = createExecutor({
+            queryInputs,
+            queryResults: [scriptedMessages(resultMessage(0, "session_untitled"))],
+        });
+        await executor.execute({ threadId: "thread_untitled", turnId: "turn_untitled", prompt: "Build a landing", history: history([]) }, activeControls());
+        expect(queryInputs[0]?.options.env).toMatchObject({ CLAUDE_CODE_DISABLE_TERMINAL_TITLE: "1" });
+    });
     it("silences the CLI task reminders and ANSI colour in the agent subprocess", async () => {
         const queryInputs = [];
         const executor = createExecutor({
@@ -1483,6 +1495,78 @@ describe("BridgeJobExecutor", () => {
         expect(options.env?.CLAUDE_CODE_ADDITIONAL_DIRECTORIES_CLAUDE_MD).toBe("1");
         expect(await readFile(join(instructionsDirMock.value, "CLAUDE.md"), "utf8")).toBe(`@${join(workspace, "COMPANY.md")}\n`);
     });
+    // Excluding CLAUDE.md from the memory files took it from every subagent that
+    // loaded it there too; measured through the real CLI, `general-purpose` and
+    // `research` lost it while still reading COMPANY.md.
+    it("gives the shared instructions back to the subagents that loaded CLAUDE.md, and only to them", async () => {
+        const workspace = await mkdtemp(join(tmpdir(), "baker-job-subagent-instructions-"));
+        await writeFile(join(workspace, "CLAUDE.md"), "# Project Instructions\n\n## Client Context\n\nOne client per repo. Everything known about them:\n\n@COMPANY.md\n\n## Rules\n\nStage, don't ask.\n");
+        await mkdir(join(workspace, ".claude", "agents"), { recursive: true });
+        await writeFile(join(workspace, ".claude", "agents", "claim-checker.md"), "---\nname: claim-checker\nmodel: claude-sonnet-5-5\nomitClaudeMd: true\n---\nCheck claims.\n");
+        projectRootMock.value = workspace;
+        const queryInputs = [];
+        const executor = createExecutor({
+            queryInputs,
+            queryResults: [scriptedMessages(resultMessage(0, "session_subagent_instructions"))],
+        });
+        await executor.execute({ threadId: "thread_sub_instr", turnId: "turn_sub_instr", prompt: "Build a landing", history: history([]) }, activeControls());
+        const start = queryInputs[0]?.options.hooks?.SubagentStart?.[0]?.hooks[0];
+        const contextFor = async (agentType) => (await start?.({
+            hook_event_name: "SubagentStart",
+            agent_type: agentType,
+            agent_id: "a",
+            session_id: "s",
+            transcript_path: "",
+            cwd: "",
+        }, undefined, { signal: new AbortController().signal }))?.hookSpecificOutput?.additionalContext;
+        expect(await contextFor("general-purpose")).toContain("Stage, don't ask.");
+        expect(await contextFor("general-purpose")).not.toContain("@COMPANY.md");
+        expect(await contextFor("claim-checker")).toBeUndefined();
+        expect(await contextFor("fork")).toBeUndefined();
+    });
+    it("moves a built-in subagent that only reads to Sonnet, and records it with the turn", async () => {
+        const queryInputs = [];
+        let spawned;
+        const executor = createExecutor({
+            queryInputs,
+            model: "claude-opus-5-5",
+            routeModels: true,
+            // The turn: finished work, so Opus. The spawn: only reading.
+            routeJudge: (questions) => Promise.resolve(String(questions[0]?.instructions.message).includes("Check which step")
+                ? [0.2, 0.8, 0.9, 0.1]
+                : [0.1, 0.9, 0.1, 0.9]),
+            queryResults: [
+                {
+                    async *[Symbol.asyncIterator]() {
+                        const preToolUse = queryInputs[0]?.options.hooks?.PreToolUse?.find((m) => m.matcher === "Agent")?.hooks[0];
+                        const result = (await preToolUse?.({
+                            hook_event_name: "PreToolUse",
+                            tool_name: "Agent",
+                            tool_input: {
+                                subagent_type: "general-purpose",
+                                description: "Walk the flow",
+                                prompt: "Check which step fires the conversion.",
+                            },
+                            tool_use_id: "toolu_1",
+                            session_id: "s",
+                            transcript_path: "",
+                            cwd: "",
+                        }, "toolu_1", { signal: new AbortController().signal }));
+                        spawned = result.hookSpecificOutput?.updatedInput;
+                        yield resultMessage(0, "session_subagent_route");
+                    },
+                },
+            ],
+        });
+        const outcome = await executor.execute({ threadId: "thread_sub_route", turnId: "turn_sub_route", prompt: "Rebuild the landing", history: history([]) }, activeControls());
+        expect(spawned).toMatchObject({ subagent_type: "general-purpose", model: "sonnet", effort: "high" });
+        expect(outcome).toMatchObject({
+            turnRoute: {
+                model: "claude-opus-5-5",
+                subagents: [{ agentType: "general-purpose", model: "sonnet", effort: "high", reason: "read_only" }],
+            },
+        });
+    });
     it("lets the CLI load a CLAUDE.md it cannot split, exactly as before", async () => {
         const workspace = await mkdtemp(join(tmpdir(), "baker-job-instructions-old-"));
         await writeFile(join(workspace, "CLAUDE.md"), "# Project Instructions\n\nNo company import.\n");
@@ -1700,9 +1784,11 @@ describe("BridgeJobExecutor", () => {
         const asked = [];
         const executor = createExecutor({
             queryInputs,
-            effortJudge: async (questions) => {
+            model: "claude-opus-5-5",
+            // small, complex, read-only, deliverable
+            routeJudge: (questions) => {
                 asked.push(...questions);
-                return [0.92, 0.05];
+                return Promise.resolve([0.92, 0.05, 0.6, 0.1]);
             },
             queryResults: [scriptedMessages(resultMessage(0, "session_effort_low"))],
         });
@@ -1721,17 +1807,63 @@ describe("BridgeJobExecutor", () => {
         });
         expect(outcome).toMatchObject({
             kind: "completed",
-            effortDecision: { effort: "low", reason: "small", small: 0.92, complex: 0.05 },
+            turnRoute: { model: "claude-opus-5-5", modelReason: "fixed", effort: "low", reason: "small" },
+        });
+    });
+    it("starts a chat that only reads on Sonnet, thinking harder, when the Runtime routes models", async () => {
+        const queryInputs = [];
+        const executor = createExecutor({
+            queryInputs,
+            model: "claude-opus-5-5",
+            routeModels: true,
+            routeJudge: () => Promise.resolve([0.2, 0.9, 0.85, 0.1]),
+            queryResults: [scriptedMessages(resultMessage(0, "session_reading"))],
         });
+        const outcome = await executor.execute({ threadId: "thread_reading", turnId: "turn_reading", prompt: "¿Cómo van los anuncios?", history: history([]) }, activeControls());
+        expect(queryInputs[0]?.options).toMatchObject({ model: "claude-sonnet-5-5", effort: "high" });
+        expect(outcome).toMatchObject({
+            model: "claude-sonnet-5-5",
+            turnRoute: { model: "claude-sonnet-5-5", modelReason: "read_only", effort: "high" },
+        });
+    });
+    it("moves a chat that ran on Sonnet to Opus once a turn asks for finished work", async () => {
+        const configDir = await mkdtemp(join(tmpdir(), "bridge-turn-route-"));
+        try {
+            await mkdir(join(configDir, "projects", "-home-user-repo"), { recursive: true });
+            await writeFile(join(configDir, "projects", "-home-user-repo", "session_sonnet.jsonl"), `${JSON.stringify({ type: "assistant", requestedModel: "claude-sonnet-5-5", message: { id: "m", model: "claude-sonnet-5-5", content: [] } })}\n`);
+            vi.stubEnv("CLAUDE_CONFIG_DIR", configDir);
+            const queryInputs = [];
+            const executor = createExecutor({
+                queryInputs,
+                model: "claude-opus-5-5",
+                routeModels: true,
+                loadedBoundary: cleanBoundary("session_sonnet"),
+                routeJudge: () => Promise.resolve([0.1, 0.95, 0.2, 0.9]),
+                queryResults: [scriptedMessages(resultMessage(0, "session_sonnet"))],
+            });
+            await executor.execute({
+                threadId: "thread_upgrade",
+                turnId: "turn_upgrade",
+                prompt: "Perfecto, créame una landing con esto",
+                history: history([
+                    { role: "user", text: "¿Cómo van los anuncios?", createdAt: 1 },
+                    { role: "assistant", text: "Van bien: el CPL bajó un 12%.", createdAt: 2 },
+                ]),
+            }, activeControls());
+            expect(queryInputs[0]?.options).toMatchObject({ model: "claude-opus-5-5", effort: "medium" });
+        }
+        finally {
+            await rm(configDir, { recursive: true, force: true });
+        }
     });
     it("keeps a plan-mode turn at medium without asking Jev", async () => {
         const queryInputs = [];
         let asked = false;
         const executor = createExecutor({
             queryInputs,
-            effortJudge: async () => {
+            routeJudge: () => {
                 asked = true;
-                return [0.99, 0];
+                return Promise.resolve([0.99, 0, 1, 0]);
             },
             queryResults: [scriptedMessages(resultMessage(0, "session_effort_plan"))],
         });