lsd-pi 1.3.9 → 1.3.11

package/dist/resources/extensions/slash-commands/plan.js

@@ -277,7 +277,7 @@ async function setModelIfNeeded(pi, ctx, modelRef) {
     return true;
 }
 function buildExecutionKickoffMessage(options) {
-    const { permissionMode, executeWithSubagent = false } = options;
+    const { permissionMode, executeWithSubagent = false, executionNote } = options;
     const task = state.task.trim();
     if (!executeWithSubagent) {
         const details = [
@@ -287,6 +287,9 @@ function buildExecutionKickoffMessage(options) {
             details.push(`Original task: ${task}`);
         if (state.latestPlanPath)
             details.push(`Use the approved plan artifact at ${state.latestPlanPath} as the execution plan.`);
+        if (executionNote)
+            details.push(`User execution note: ${executionNote}`);
+        details.push("After implementation: guide the user through verification by presenting a concise checklist based on the plan's Acceptance Criteria and Verification Plan. Run applicable checks (build, lint, tests) and report results.");
         return details.join(" ");
     }
     const codingModel = readPlanModeCodingModel();
@@ -303,13 +306,24 @@ function buildExecutionKickoffMessage(options) {
         details.push(`Original task: ${task}`);
     if (state.latestPlanPath)
         details.push(`Primary plan artifact: ${state.latestPlanPath}`);
+    if (executionNote) {
+        details.push(`User execution note: ${executionNote}`);
+        // If the note contains a model request, surface it explicitly so the agent
+        // doesn't silently drop it. The model override will be resolved by
+        // normalizeSubagentModel when the subagent tool is invoked.
+        const modelMatch = executionNote.match(/\b(?:model|use\s+model)\s+["']?([\w.-]+(?:\/[\w.-]+)?)["']?\b/i);
+        if (modelMatch?.[1]) {
+            details.push(`The user explicitly requested model "${modelMatch[1]}". You MUST pass model="${modelMatch[1]}" in the subagent tool call. If normalizeSubagentModel cannot resolve this model, report the error to the user instead of silently falling back.`);
+        }
+    }
     details.push("Important: if the plan is large and you estimate it would exceed a single subagent's context window (~200k tokens), " +
         "split execution across multiple sequential subagents instead of one. " +
         "Use the subagent tool's chain mode: pass a \"chain\" array where each entry covers one self-contained phase or group of steps from the plan. " +
         "Each chain entry should include the agent name, a focused task description for that phase, and may reference {previous} to receive the prior phase's output as handoff context. " +
         "Only split when genuinely needed — prefer a single subagent for plans that fit comfortably.");
     details.push("After all subagents complete: (1) do a quick review of the implementation — check that the plan steps were actually carried out, spot obvious issues or missed pieces, and verify the code compiles/passes lint if applicable. " +
-        "(2) Then summarize what was done, what (if anything) needs follow-up, and flag any concerns found during review.");
+        "(2) Then guide the user through verification: present a concise checklist based on the plan's Acceptance Criteria and Verification Plan sections. Run applicable checks (build, lint, tests) and report results. " +
+        "(3) Summarize what was done, what (if anything) needs follow-up, and flag any concerns found during review.");
     return details.join(" ");
 }
 let pendingNewSession = null;
@@ -331,7 +345,7 @@ function scheduleNewSession(pi, ctx) {
     // Must use the /prefix so tryExecuteExtensionCommand parses the name correctly.
     pi.executeSlashCommand("/plan-execute-new-session");
 }
-async function approvePlan(pi, ctx, permissionMode, executeWithSubagent = false) {
+async function approvePlan(pi, ctx, permissionMode, executeWithSubagent = false, executionNote) {
     // Do NOT switch to reasoning model during execution.
     // The reasoning model is only for plan-mode investigation, not execution.
     // If a coding model is configured and we're using a subagent, the explicit
@@ -347,7 +361,7 @@ async function approvePlan(pi, ctx, permissionMode, executeWithSubagent = false)
     // subagent tool with the default session model BEFORE it ever sees the
     // explicit model="<planModeCodingModel>" instruction. Steering ensures the
     // configured plan-mode coding model reaches the subagent invocation.
-    await pi.sendUserMessage(buildExecutionKickoffMessage({ permissionMode, executeWithSubagent }), { deliverAs: "steer" });
+    await pi.sendUserMessage(buildExecutionKickoffMessage({ permissionMode, executeWithSubagent, executionNote }), { deliverAs: "steer" });
 }
 async function cancelPlan(pi, ctx, clearTask = true) {
     const restoreMode = state.previousMode ?? "accept-on-edit";
@@ -359,10 +373,12 @@ async function cancelPlan(pi, ctx, clearTask = true) {
 function buildPlanModeSystemPrompt() {
     const details = [
         "You are currently in plan mode.",
+        "Terse output. All technical substance stays. Only fluff dies. Fragments OK.",
         "Investigate, clarify scope, and produce a persisted execution plan before making source changes.",
         "If requirements are ambiguous or constraints are missing, ask concise clarifying questions before drafting or saving a plan.",
         `Before writing or updating a plan artifact, make sure your confidence is at least ${MIN_PLAN_CONFIDENCE}/10. If confidence is lower, investigate more or ask clarifying questions first.`,
         "Include an explicit confidence line in every saved plan, for example: \"Confidence: 8/10\" or higher.",
+        "Every saved plan MUST include explicit \"Acceptance Criteria\" and \"Verification Plan\" sections. Plans missing these sections will be rejected for approval.",
         "When adjusting an existing saved plan, prefer the edit tool for targeted changes. Rewrite the whole file only when the structure changes substantially or an exact edit is impractical.",
         "Do not modify source files or run side-effect commands while plan mode is active.",
         "Persist plan artifacts under .lsd/plan/.",
@@ -411,6 +427,29 @@ function buildApprovalActionInstructions() {
 function buildApprovalDialogInstructions() {
     return buildApprovalActionInstructions();
 }
+/** Required heading patterns for plan artifacts. Matches common variants. */
+const REQUIRED_PLAN_SECTIONS = [
+    { pattern: /\b(acceptance\s*criteria|success\s*criteria|done\s*criteria)\b/i, label: "Acceptance Criteria" },
+    { pattern: /\b(verification\s*(plan|steps|strategy)|how\s+to\s+verify|testing\s*plan)\b/i, label: "Verification Plan" },
+];
+function validatePlanArtifact(markdown) {
+    const missing = [];
+    for (const section of REQUIRED_PLAN_SECTIONS) {
+        if (!section.pattern.test(markdown)) {
+            missing.push(section.label);
+        }
+    }
+    return { valid: missing.length === 0, missing };
+}
+function buildPlanValidationSteeringMessage(planPath, missing) {
+    const missingList = missing.map((m) => `- **${m}**`).join("\n");
+    return [
+        `Plan artifact saved at ${planPath}.`,
+        `The plan is missing required sections before it can be approved for implementation:`,
+        missingList,
+        `Please revise the plan to include these sections, then re-save. Do not ask for approval until all required sections are present.`,
+    ].join("\n\n");
+}
 function buildApprovalSteeringMessage(planPath) {
     return [
         `Plan artifact saved at ${planPath}.`,
@@ -621,13 +660,26 @@ export default function planCommand(pi) {
                     return;
                 }
                 const planMarkdown = readPlanArtifact(path);
-                pi.sendMessage({
-                    customType: "plan-mode-preview",
-                    content: buildPlanPreviewMessage(path, planMarkdown),
-                    display: true,
-                });
-                ctx.ui?.notify?.("/plan to show plan", "info");
-                pi.sendUserMessage(buildApprovalSteeringMessage(path), { deliverAs: "steer" });
+                if (!planMarkdown) {
+                    ctx.ui?.notify?.("Plan artifact could not be read for validation", "warning");
+                    pi.sendUserMessage(buildApprovalSteeringMessage(path), { deliverAs: "steer" });
+                }
+                else {
+                    const validation = validatePlanArtifact(planMarkdown);
+                    if (!validation.valid) {
+                        ctx.ui?.notify?.("Plan missing required sections — see guidance below", "warning");
+                        pi.sendUserMessage(buildPlanValidationSteeringMessage(path, validation.missing), { deliverAs: "steer" });
+                    }
+                    else {
+                        pi.sendMessage({
+                            customType: "plan-mode-preview",
+                            content: buildPlanPreviewMessage(path, planMarkdown),
+                            display: true,
+                        });
+                        ctx.ui?.notify?.("/plan to show plan", "info");
+                        pi.sendUserMessage(buildApprovalSteeringMessage(path), { deliverAs: "steer" });
+                    }
+                }
             }
             return;
         }
@@ -676,12 +728,13 @@ export default function planCommand(pi) {
                 permissionMode: DEFAULT_APPROVAL_PERMISSION_MODE,
                 executeWithSubagent: false,
             };
+            const permissionNote = getAnswerNote(permissionAnswer);
             state = { ...state, targetPermissionMode: executionMode.permissionMode };
             if (executionMode.executeWithSubagent) {
                 const modeLabel = executionMode.permissionMode === "danger-full-access" ? "bypass" : "auto";
                 ctx.ui?.notify?.(`Plan approved: subagent(${modeLabel})`, "info");
             }
-            await approvePlan(pi, ctx, executionMode.permissionMode, executionMode.executeWithSubagent);
+            await approvePlan(pi, ctx, executionMode.permissionMode, executionMode.executeWithSubagent, permissionNote);
             return;
         }
         // ── First question answered (action) ──────────────────────────────────
@@ -699,12 +752,13 @@ export default function planCommand(pi) {
                 permissionMode: DEFAULT_APPROVAL_PERMISSION_MODE,
                 executeWithSubagent: false,
             };
+            const actionNote = getAnswerNote(actionAnswer);
             state = { ...state, targetPermissionMode: executionMode.permissionMode };
             if (executionMode.executeWithSubagent) {
                 const modeLabel = executionMode.permissionMode === "danger-full-access" ? "bypass" : "auto";
                 ctx.ui?.notify?.(`Plan approved: subagent(${modeLabel})`, "info");
             }
-            await approvePlan(pi, ctx, executionMode.permissionMode, executionMode.executeWithSubagent);
+            await approvePlan(pi, ctx, executionMode.permissionMode, executionMode.executeWithSubagent, actionNote);
             return;
         }
         if (actionSelection.includes(REVIEW_LABEL)) {

package/dist/resources/extensions/subagent/agents.js

@@ -6,6 +6,8 @@ import * as path from "node:path";
 import { fileURLToPath } from "node:url";
 import { getAgentDir, parseFrontmatter } from "@gsd/pi-coding-agent";
 const PROJECT_AGENT_DIR_CANDIDATES = [".lsd", ".gsd", ".pi"];
+/** Fixed read-only tool set for the reserved `scout` agent. */
+const SCOUT_ALLOWED_TOOLS = ["read", "lsp", "grep", "find", "ls"];
 function normalizeAgentModel(model) {
     const trimmed = model?.trim();
     if (!trimmed)
@@ -117,6 +119,11 @@ export function discoverAgents(cwd, scope) {
     else {
         addAgents(projectAgents);
     }
+    // Enforce reserved agent tool policies — scout is always read-only
+    const scout = agentMap.get("scout");
+    if (scout) {
+        scout.tools = [...SCOUT_ALLOWED_TOOLS];
+    }
     return { agents: Array.from(agentMap.values()), projectAgentsDir };
 }
 export function formatAgentList(agents, maxItems) {

package/dist/resources/extensions/subagent/index.js

@@ -997,11 +997,13 @@ export default function (pi) {
             const hasTasks = (params.tasks?.length ?? 0) > 0;
             const hasSingle = Boolean(params.agent && params.task);
             const modeCount = Number(hasChain) + Number(hasTasks) + Number(hasSingle);
-            const makeDetails = (mode) => (results) => ({
+            const makeDetails = (mode) => (results, opts) => ({
                 mode,
                 agentScope,
                 projectAgentsDir: discovery.projectAgentsDir,
                 results,
+                totalChainSteps: opts?.totalSteps,
+                currentChainStep: opts?.currentStep,
             });
             const trackInProcessDepth = (started, depth, ancestry) => {
                 const sessionId = started.handle.sessionId;
@@ -1090,7 +1092,8 @@ export default function (pi) {
                 for (let i = 0; i < params.chain.length; i++) {
                     const step = params.chain[i];
                     const taskWithContext = step.task.replace(/\{previous\}/g, previousOutput);
-                    // Create update callback that includes all previous results
+                    // Create update callback that includes all previous results + chain progress
+                    const chainTotalSteps = params.chain.length;
                     const chainUpdate = onUpdate
                         ? (partial) => {
                             // Combine completed results with current streaming result
@@ -1099,7 +1102,10 @@ export default function (pi) {
                                 const allResults = [...results, currentResult];
                                 onUpdate({
                                     content: partial.content,
-                                    details: makeDetails("chain")(allResults),
+                                    details: makeDetails("chain")(allResults, {
+                                        totalSteps: chainTotalSteps,
+                                        currentStep: i + 1,
+                                    }),
                                 });
                             }
                         }
@@ -1117,7 +1123,7 @@ export default function (pi) {
                         const errorMsg = result.errorMessage || result.stderr || getFinalOutput(result.messages) || "(no output)";
                         return {
                             content: [{ type: "text", text: `Chain stopped at step ${i + 1} (${step.agent}): ${errorMsg}` }],
-                            details: makeDetails("chain")(results),
+                            details: makeDetails("chain")(results, { totalSteps: params.chain.length }),
                             isError: true,
                         };
                     }
@@ -1125,7 +1131,7 @@ export default function (pi) {
                 }
                 return {
                     content: [{ type: "text", text: getFinalOutput(results[results.length - 1].messages) || "(no output)" }],
-                    details: makeDetails("chain")(results),
+                    details: makeDetails("chain")(results, { totalSteps: params.chain.length }),
                 };
             }
             if (params.tasks && params.tasks.length > 0) {
@@ -1653,13 +1659,24 @@ export default function (pi) {
             };
             if (details.mode === "chain") {
                 const successCount = details.results.filter((r) => r.exitCode === 0).length;
-                const icon = successCount === details.results.length ? theme.fg("success", "✓") : theme.fg("error", "✗");
+                const isRunning = details.currentChainStep != null;
+                const icon = !isRunning && successCount === details.results.length
+                    ? theme.fg("success", "✓")
+                    : isRunning
+                        ? theme.fg("accent", "◉")
+                        : theme.fg("error", "✗");
+                const totalLabel = details.totalChainSteps != null
+                    ? `${details.results.length}/${details.totalChainSteps} steps`
+                    : `${details.results.length} steps`;
+                const progressLabel = isRunning
+                    ? ` (step ${details.currentChainStep} running)`
+                    : "";
                 if (expanded) {
                     const container = new Container();
                     container.addChild(new Text(icon +
                         " " +
                         theme.fg("toolTitle", theme.bold("chain ")) +
-                        theme.fg("accent", `${successCount}/${details.results.length} steps`), 0, 0));
+                        theme.fg("accent", `${totalLabel}${progressLabel}`), 0, 0));
                     for (const r of details.results) {
                         const rIcon = r.exitCode === 0 ? theme.fg("success", "✓") : theme.fg("error", "✗");
                         const displayItems = getDisplayItems(r.messages);
@@ -1693,7 +1710,7 @@ export default function (pi) {
                 let text = icon +
                     " " +
                     theme.fg("toolTitle", theme.bold("chain ")) +
-                    theme.fg("accent", `${successCount}/${details.results.length} steps`);
+                    theme.fg("accent", `${totalLabel}${progressLabel}`);
                 for (const r of details.results) {
                     const rIcon = r.exitCode === 0 ? theme.fg("success", "✓") : theme.fg("error", "✗");
                     const displayItems = getDisplayItems(r.messages);

package/dist/resources/extensions/subagent/model-resolution.js

@@ -11,6 +11,7 @@ const BARE_MODEL_PROVIDER_RULES = [
         matches: (modelId) => /^(mistral-|ministral-|codestral-)/.test(modelId),
     },
     { provider: "groq", matches: (modelId) => modelId.startsWith("llama-") || modelId.startsWith("mixtral-") },
+    { provider: "zhipu", matches: (modelId) => modelId.startsWith("glm-") },
 ];
 export function inferProviderForBareModel(modelId) {
     const normalizedModelId = modelId.trim().toLowerCase();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lsd-pi",
-  "version": "1.3.9",
+  "version": "1.3.11",
   "description": "LSD — Looks Sort of Done coding agent",
   "license": "MIT",
   "repository": {

package/packages/pi-coding-agent/dist/modes/interactive/components/__tests__/tool-summary-line.test.js

@@ -1,7 +1,7 @@
 import { describe, it } from "node:test";
 import assert from "node:assert/strict";
 import stripAnsi from "strip-ansi";
-import { ToolSummaryLine } from "../tool-summary-line.js";
+import { ToolSummaryLine, extractToolLabel } from "../tool-summary-line.js";
 import { initTheme } from "../../theme/theme.js";
 initTheme("dark");
 describe("ToolSummaryLine", () => {
@@ -13,7 +13,8 @@ describe("ToolSummaryLine", () => {
         assert.match(rendered, /^ ● /);
         assert.ok(rendered.includes("reading 2 files · 0.8s"));
         assert.equal(summary.canGroupWith("read"), true);
-        assert.equal(summary.canGroupWith("find"), false);
+        assert.equal(summary.canGroupWith("find"), true);
+        assert.equal(summary.canGroupWith("bash"), false);
         assert.equal(rendered.includes("collapsed tools"), false);
         assert.equal(rendered.includes("⎯"), false);
     });
@@ -30,5 +31,106 @@ describe("ToolSummaryLine", () => {
         summary.setHidden(true);
         assert.deepEqual(summary.render(80), []);
     });
+    it("renders spinner and label line when pending tools exist", () => {
+        const summary = new ToolSummaryLine();
+        summary.addPendingTool("t1", "read", { path: "src/foo/bar.ts" });
+        const rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(!rendered.startsWith(" ●"));
+        assert.ok(rendered.includes("  └ bar.ts"));
+        assert.ok(rendered.includes("bar.ts"));
+    });
+    it("keeps last tool label after pending tool completes", () => {
+        const summary = new ToolSummaryLine();
+        summary.addPendingTool("t1", "read", { path: "file.ts" });
+        assert.equal(summary.hasPendingTools(), true);
+        summary.removePendingTool("t1");
+        summary.addTool("read", 500);
+        assert.equal(summary.hasPendingTools(), false);
+        const rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(rendered.includes("●"));
+        assert.ok(rendered.includes("0.5s"));
+        assert.ok(rendered.includes("└ file.ts"));
+    });
+    it("aggregates completed and pending tools in summary text", () => {
+        const summary = new ToolSummaryLine();
+        summary.addTool("read", 300);
+        summary.addPendingTool("t1", "grep", { pattern: "TODO" });
+        const rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(rendered.includes("1 file"));
+        assert.ok(rendered.includes("1 pattern"));
+        assert.ok(rendered.includes("…"));
+        assert.ok(rendered.includes("TODO"));
+    });
+    it("shows expand hint when set and tools are pending", () => {
+        const summary = new ToolSummaryLine();
+        summary.setExpandHint("(ctrl+o to expand)");
+        summary.addPendingTool("t1", "read", { path: "test.ts" });
+        const rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(rendered.includes("ctrl+o to expand"));
+    });
+    it("does not show expand hint when no pending tools", () => {
+        const summary = new ToolSummaryLine();
+        summary.setExpandHint("(ctrl+o to expand)");
+        summary.addTool("read", 300);
+        const rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(!rendered.includes("ctrl+o to expand"));
+    });
+    it("updates label when pending tool args change", () => {
+        const summary = new ToolSummaryLine();
+        summary.addPendingTool("t1", "read", { path: "old.ts" });
+        let rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(rendered.includes("old.ts"));
+        summary.updatePendingToolArgs("t1", { path: "new.ts" });
+        rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(rendered.includes("new.ts"));
+    });
+    it("clears pending spinner without removing last label", () => {
+        const summary = new ToolSummaryLine();
+        summary.addPendingTool("t1", "read", { path: "file.ts" });
+        summary.clearPendingTools();
+        const rendered = stripAnsi(summary.render(160).join("\n"));
+        assert.ok(!rendered.includes("◯"));
+        assert.ok(!rendered.includes("◔"));
+        assert.ok(!rendered.includes("◑"));
+        assert.ok(!rendered.includes("◕"));
+        assert.ok(!rendered.includes("● Listing"));
+        assert.ok(!rendered.includes("…"));
+        assert.ok(rendered === "" || rendered.includes("file.ts"));
+    });
+    it("canGroupWith considers pending tools", () => {
+        const summary = new ToolSummaryLine();
+        summary.addPendingTool("t1", "read", { path: "a.ts" });
+        assert.equal(summary.canGroupWith("read"), true);
+        assert.equal(summary.canGroupWith("grep"), true);
+        assert.equal(summary.canGroupWith("bash"), false);
+    });
+});
+describe("extractToolLabel", () => {
+    it("extracts basename for read tool", () => {
+        assert.equal(extractToolLabel("read", { path: "src/foo/bar.ts" }), "bar.ts");
+        assert.equal(extractToolLabel("read", { file_path: "/abs/path/file.json" }), "file.json");
+        assert.equal(extractToolLabel("read", {}), "read");
+    });
+    it("extracts pattern for grep tool", () => {
+        assert.equal(extractToolLabel("grep", { pattern: "TODO" }), "TODO");
+        assert.equal(extractToolLabel("grep", {}), "grep");
+    });
+    it("extracts pattern for find tool", () => {
+        assert.equal(extractToolLabel("find", { pattern: "*.ts" }), "*.ts");
+        assert.equal(extractToolLabel("find", {}), "find");
+    });
+    it("extracts path for ls tool", () => {
+        assert.equal(extractToolLabel("ls", { path: "src/components" }), "components");
+        assert.equal(extractToolLabel("ls", {}), ".");
+    });
+    it("extracts symbol or file for lsp tool", () => {
+        assert.equal(extractToolLabel("lsp", { symbol: "MyClass" }), "MyClass");
+        assert.equal(extractToolLabel("lsp", { file: "src/index.ts" }), "index.ts");
+        assert.equal(extractToolLabel("lsp", { symbol: "foo", file: "bar.ts" }), "foo");
+        assert.equal(extractToolLabel("lsp", {}), "lsp");
+    });
+    it("returns tool name for unknown tools", () => {
+        assert.equal(extractToolLabel("custom_tool", { whatever: "value" }), "custom_tool");
+    });
 });
 //# sourceMappingURL=tool-summary-line.test.js.map

package/packages/pi-coding-agent/dist/modes/interactive/components/__tests__/tool-summary-line.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"tool-summary-line.test.js","sourceRoot":"","sources":["../../../../../src/modes/interactive/components/__tests__/tool-summary-line.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,SAAS,MAAM,YAAY,CAAC;AAEnC,OAAO,EAAE,eAAe,EAAE,MAAM,yBAAyB,CAAC;AAC1D,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAC;AAEjD,SAAS,CAAC,MAAM,CAAC,CAAC;AAElB,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACrE,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAC7B,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAE7B,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,KAAK,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAC/B,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,wBAAwB,CAAC,CAAC,CAAC;QACvD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC;QAClD,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,iBAAiB,CAAC,EAAE,KAAK,CAAC,CAAC;QAC1D,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,KAAK,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QAClD,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,OAAO,CAAC,aAAa,EAAE,GAAG,CAAC,CAAC;QAEpC,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,oBAAoB,CAAC,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC/C,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAEzC,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAC7B,OAAO,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACxB,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["import { describe, it } from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport stripAnsi from \"strip-ansi\";\n\nimport { ToolSummaryLine } from \"../tool-summary-line.js\";\nimport { initTheme } from \"../../theme/theme.js\";\n\ninitTheme(\"dark\");\n\ndescribe(\"ToolSummaryLine\", () => {\n\tit(\"renders action-based summaries for grouped identical tools\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addTool(\"read\", 600);\n\t\tsummary.addTool(\"read\", 150);\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.match(rendered, /^ ● /);\n\t\tassert.ok(rendered.includes(\"reading 2 files · 0.8s\"));\n\t\tassert.equal(summary.canGroupWith(\"read\"), true);\n\t\tassert.equal(summary.canGroupWith(\"find\"), false);\n\t\tassert.equal(rendered.includes(\"collapsed tools\"), false);\n\t\tassert.equal(rendered.includes(\"⎯\"), false);\n\t});\n\n\tit(\"keeps fallback format for unknown tools\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addTool(\"custom_tool\", 100);\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(rendered.includes(\"custom_tool · 0.1s\"));\n\t});\n\n\tit(\"renders nothing when empty or hidden\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tassert.deepEqual(summary.render(80), []);\n\n\t\tsummary.addTool(\"grep\", 100);\n\t\tsummary.setHidden(true);\n\t\tassert.deepEqual(summary.render(80), []);\n\t});\n});\n"]}
+{"version":3,"file":"tool-summary-line.test.js","sourceRoot":"","sources":["../../../../../src/modes/interactive/components/__tests__/tool-summary-line.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,SAAS,MAAM,YAAY,CAAC;AAEnC,OAAO,EAAE,eAAe,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAC5E,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAC;AAEjD,SAAS,CAAC,MAAM,CAAC,CAAC;AAElB,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACrE,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAC7B,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAE7B,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,KAAK,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAC/B,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,wBAAwB,CAAC,CAAC,CAAC;QACvD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC;QAClD,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,iBAAiB,CAAC,EAAE,KAAK,CAAC,CAAC;QAC1D,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,KAAK,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QAClD,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,OAAO,CAAC,aAAa,EAAE,GAAG,CAAC,CAAC;QAEpC,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,oBAAoB,CAAC,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC/C,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAEzC,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAC7B,OAAO,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACxB,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;QAClE,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,gBAAgB,EAAE,CAAC,CAAC;QAEjE,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC;QACtC,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC7D,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;QAC1D,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,eAAe,EAAE,EAAE,IAAI,CAAC,CAAC;QAE9C,OAAO,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;QAChC,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAC7B,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,eAAe,EAAE,EAAE,KAAK,CAAC,CAAC;QAE/C,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;QAClC,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC;QACrC,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;QACjE,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAC7B,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,CAAC,CAAC;QAE1D,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC;QACvC,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC;QAC1C,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;QAClC,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC3D,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,aAAa,CAAC,oBAAoB,CAAC,CAAC;QAC5C,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;QAE1D,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,kBAAkB,CAAC,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QAC1D,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,aAAa,CAAC,oBAAoB,CAAC,CAAC;QAC5C,OAAO,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAE7B,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,kBAAkB,CAAC,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACtD,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC;QAEzD,IAAI,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QACzD,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC;QAEvC,OAAO,CAAC,qBAAqB,CAAC,IAAI,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC;QACxD,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QACrD,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC7D,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;QAE1D,OAAO,CAAC,iBAAiB,EAAE,CAAC;QAE5B,MAAM,QAAQ,GAAG,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC;QAC3C,MAAM,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,EAAE,CAAC,QAAQ,KAAK,EAAE,IAAI,QAAQ,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC/C,MAAM,OAAO,GAAG,IAAI,eAAe,EAAE,CAAC;QACtC,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,CAAC;QAEvD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,IAAI,CAAC,CAAC;QACjD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QAC1C,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,gBAAgB,EAAE,CAAC,EAAE,QAAQ,CAAC,CAAC;QAC7E,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,MAAM,EAAE,EAAE,SAAS,EAAE,qBAAqB,EAAE,CAAC,EAAE,WAAW,CAAC,CAAC;QAC1F,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,MAAM,EAAE,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;QACzC,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,MAAM,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;QACpE,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,MAAM,EAAE,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;QACzC,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,MAAM,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;QACpE,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,MAAM,EAAE,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACpC,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,IAAI,EAAE,EAAE,IAAI,EAAE,gBAAgB,EAAE,CAAC,EAAE,YAAY,CAAC,CAAC;QAC/E,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,IAAI,EAAE,EAAE,CAAC,EAAE,GAAG,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC/C,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,KAAK,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,EAAE,SAAS,CAAC,CAAC;QACxE,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,KAAK,EAAE,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,EAAE,UAAU,CAAC,CAAC;QAC5E,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,KAAK,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,CAAC,CAAC;QAChF,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,KAAK,EAAE,EAAE,CAAC,EAAE,KAAK,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,aAAa,EAAE,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC,EAAE,aAAa,CAAC,CAAC;IACrF,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["import { describe, it } from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport stripAnsi from \"strip-ansi\";\n\nimport { ToolSummaryLine, extractToolLabel } from \"../tool-summary-line.js\";\nimport { initTheme } from \"../../theme/theme.js\";\n\ninitTheme(\"dark\");\n\ndescribe(\"ToolSummaryLine\", () => {\n\tit(\"renders action-based summaries for grouped identical tools\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addTool(\"read\", 600);\n\t\tsummary.addTool(\"read\", 150);\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.match(rendered, /^ ● /);\n\t\tassert.ok(rendered.includes(\"reading 2 files · 0.8s\"));\n\t\tassert.equal(summary.canGroupWith(\"read\"), true);\n\t\tassert.equal(summary.canGroupWith(\"find\"), true);\n\t\tassert.equal(summary.canGroupWith(\"bash\"), false);\n\t\tassert.equal(rendered.includes(\"collapsed tools\"), false);\n\t\tassert.equal(rendered.includes(\"⎯\"), false);\n\t});\n\n\tit(\"keeps fallback format for unknown tools\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addTool(\"custom_tool\", 100);\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(rendered.includes(\"custom_tool · 0.1s\"));\n\t});\n\n\tit(\"renders nothing when empty or hidden\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tassert.deepEqual(summary.render(80), []);\n\n\t\tsummary.addTool(\"grep\", 100);\n\t\tsummary.setHidden(true);\n\t\tassert.deepEqual(summary.render(80), []);\n\t});\n\n\tit(\"renders spinner and label line when pending tools exist\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addPendingTool(\"t1\", \"read\", { path: \"src/foo/bar.ts\" });\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(!rendered.startsWith(\" ●\"));\n\t\tassert.ok(rendered.includes(\"  └ bar.ts\"));\n\t\tassert.ok(rendered.includes(\"bar.ts\"));\n\t});\n\n\tit(\"keeps last tool label after pending tool completes\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addPendingTool(\"t1\", \"read\", { path: \"file.ts\" });\n\t\tassert.equal(summary.hasPendingTools(), true);\n\n\t\tsummary.removePendingTool(\"t1\");\n\t\tsummary.addTool(\"read\", 500);\n\t\tassert.equal(summary.hasPendingTools(), false);\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(rendered.includes(\"●\"));\n\t\tassert.ok(rendered.includes(\"0.5s\"));\n\t\tassert.ok(rendered.includes(\"└ file.ts\"));\n\t});\n\n\tit(\"aggregates completed and pending tools in summary text\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addTool(\"read\", 300);\n\t\tsummary.addPendingTool(\"t1\", \"grep\", { pattern: \"TODO\" });\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(rendered.includes(\"1 file\"));\n\t\tassert.ok(rendered.includes(\"1 pattern\"));\n\t\tassert.ok(rendered.includes(\"…\"));\n\t\tassert.ok(rendered.includes(\"TODO\"));\n\t});\n\n\tit(\"shows expand hint when set and tools are pending\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.setExpandHint(\"(ctrl+o to expand)\");\n\t\tsummary.addPendingTool(\"t1\", \"read\", { path: \"test.ts\" });\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(rendered.includes(\"ctrl+o to expand\"));\n\t});\n\n\tit(\"does not show expand hint when no pending tools\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.setExpandHint(\"(ctrl+o to expand)\");\n\t\tsummary.addTool(\"read\", 300);\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(!rendered.includes(\"ctrl+o to expand\"));\n\t});\n\n\tit(\"updates label when pending tool args change\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addPendingTool(\"t1\", \"read\", { path: \"old.ts\" });\n\n\t\tlet rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(rendered.includes(\"old.ts\"));\n\n\t\tsummary.updatePendingToolArgs(\"t1\", { path: \"new.ts\" });\n\t\trendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(rendered.includes(\"new.ts\"));\n\t});\n\n\tit(\"clears pending spinner without removing last label\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addPendingTool(\"t1\", \"read\", { path: \"file.ts\" });\n\n\t\tsummary.clearPendingTools();\n\n\t\tconst rendered = stripAnsi(summary.render(160).join(\"\\n\"));\n\t\tassert.ok(!rendered.includes(\"◯\"));\n\t\tassert.ok(!rendered.includes(\"◔\"));\n\t\tassert.ok(!rendered.includes(\"◑\"));\n\t\tassert.ok(!rendered.includes(\"◕\"));\n\t\tassert.ok(!rendered.includes(\"● Listing\"));\n\t\tassert.ok(!rendered.includes(\"…\"));\n\t\tassert.ok(rendered === \"\" || rendered.includes(\"file.ts\"));\n\t});\n\n\tit(\"canGroupWith considers pending tools\", () => {\n\t\tconst summary = new ToolSummaryLine();\n\t\tsummary.addPendingTool(\"t1\", \"read\", { path: \"a.ts\" });\n\n\t\tassert.equal(summary.canGroupWith(\"read\"), true);\n\t\tassert.equal(summary.canGroupWith(\"grep\"), true);\n\t\tassert.equal(summary.canGroupWith(\"bash\"), false);\n\t});\n});\n\ndescribe(\"extractToolLabel\", () => {\n\tit(\"extracts basename for read tool\", () => {\n\t\tassert.equal(extractToolLabel(\"read\", { path: \"src/foo/bar.ts\" }), \"bar.ts\");\n\t\tassert.equal(extractToolLabel(\"read\", { file_path: \"/abs/path/file.json\" }), \"file.json\");\n\t\tassert.equal(extractToolLabel(\"read\", {}), \"read\");\n\t});\n\n\tit(\"extracts pattern for grep tool\", () => {\n\t\tassert.equal(extractToolLabel(\"grep\", { pattern: \"TODO\" }), \"TODO\");\n\t\tassert.equal(extractToolLabel(\"grep\", {}), \"grep\");\n\t});\n\n\tit(\"extracts pattern for find tool\", () => {\n\t\tassert.equal(extractToolLabel(\"find\", { pattern: \"*.ts\" }), \"*.ts\");\n\t\tassert.equal(extractToolLabel(\"find\", {}), \"find\");\n\t});\n\n\tit(\"extracts path for ls tool\", () => {\n\t\tassert.equal(extractToolLabel(\"ls\", { path: \"src/components\" }), \"components\");\n\t\tassert.equal(extractToolLabel(\"ls\", {}), \".\");\n\t});\n\n\tit(\"extracts symbol or file for lsp tool\", () => {\n\t\tassert.equal(extractToolLabel(\"lsp\", { symbol: \"MyClass\" }), \"MyClass\");\n\t\tassert.equal(extractToolLabel(\"lsp\", { file: \"src/index.ts\" }), \"index.ts\");\n\t\tassert.equal(extractToolLabel(\"lsp\", { symbol: \"foo\", file: \"bar.ts\" }), \"foo\");\n\t\tassert.equal(extractToolLabel(\"lsp\", {}), \"lsp\");\n\t});\n\n\tit(\"returns tool name for unknown tools\", () => {\n\t\tassert.equal(extractToolLabel(\"custom_tool\", { whatever: \"value\" }), \"custom_tool\");\n\t});\n});\n"]}

package/packages/pi-coding-agent/dist/modes/interactive/components/assistant-message.d.ts

@@ -1,8 +1,30 @@
 import type { AssistantMessage } from "@gsd/pi-ai";
-import { Container, type MarkdownTheme } from "@gsd/pi-tui";
+import type { Component } from "@gsd/pi-tui";
+import { Container, Markdown, type MarkdownTheme, Text } from "@gsd/pi-tui";
 import { type TimestampFormat } from "./timestamp.js";
 /**
- * Component that renders a complete assistant message
+ * Create a Markdown component for an assistant text block.
+ * @param text - Text content (should be trimmed by caller)
+ * @param withMarker - Whether to prefix with the response marker
+ * @param markdownTheme - Markdown theme
+ */
+export declare function createTextMarkdown(text: string, withMarker: boolean, markdownTheme: MarkdownTheme): Markdown;
+/**
+ * Create a Markdown component for a thinking block.
+ */
+export declare function createThinkingMarkdown(thinking: string, markdownTheme: MarkdownTheme): Markdown;
+/**
+ * Create an error/abort Text component.
+ */
+export declare function createErrorText(message: string): Text;
+/**
+ * Component that renders a complete assistant message.
+ *
+ * Supports two rendering modes:
+ * 1. Legacy: `updateContent(message)` renders all text/thinking into a contentContainer.
+ *    Tool rows are expected to be added as siblings in the parent container.
+ * 2. Interleaved: `updateContentOrdered(message, toolComponents)` renders text/thinking
+ *    AND tool components in content order. Tool components become children of this container.
  */
 export declare class AssistantMessageComponent extends Container {
     private contentContainer;
@@ -15,6 +37,21 @@ export declare class AssistantMessageComponent extends Container {
     invalidate(): void;
     setHideThinkingBlock(hide: boolean): void;
     setThinkingLevel(level: string): void;
+    /**
+     * Legacy rendering: renders text/thinking blocks into contentContainer.
+     * Stops rendering at the first tool-type block (toolCall/serverToolUse).
+     * Post-tool text blocks are handled by the chat-controller to preserve
+     * content ordering relative to tool rows.
+     */
     updateContent(message: AssistantMessage): void;
+    /**
+     * Interleaved rendering: renders text/thinking AND tool components in content order.
+     * Tool components become children of this container, preserving visual ordering.
+     *
+     * @param message - The assistant message
+     * @param toolComponents - Map of content block ID → pre-created Component (ToolExecutionComponent etc.)
+     * @returns Map of content block ID → the tool Component that was placed (for pending tool tracking)
+     */
+    updateContentOrdered(message: AssistantMessage, toolComponents?: Map<string, Component>): Map<string, Component>;
 }
 //# sourceMappingURL=assistant-message.d.ts.map

package/packages/pi-coding-agent/dist/modes/interactive/components/assistant-message.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"assistant-message.d.ts","sourceRoot":"","sources":["../../../../src/modes/interactive/components/assistant-message.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,YAAY,CAAC;AACnD,OAAO,EAAE,SAAS,EAAY,KAAK,aAAa,EAAgB,MAAM,aAAa,CAAC;AAEpF,OAAO,EAAmB,KAAK,eAAe,EAAE,MAAM,gBAAgB,CAAC;AAEvE;;GAEG;AACH,qBAAa,yBAA0B,SAAQ,SAAS;IACvD,OAAO,CAAC,gBAAgB,CAAY;IACpC,OAAO,CAAC,iBAAiB,CAAU;IACnC,OAAO,CAAC,aAAa,CAAS;IAC9B,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,WAAW,CAAC,CAAmB;IACvC,OAAO,CAAC,eAAe,CAAkB;gBAGxC,OAAO,CAAC,EAAE,gBAAgB,EAC1B,iBAAiB,UAAQ,EACzB,aAAa,GAAE,aAAkC,EACjD,eAAe,GAAE,eAAiC,EAClD,aAAa,SAAQ;IAkBb,UAAU,IAAI,IAAI;IAO3B,oBAAoB,CAAC,IAAI,EAAE,OAAO,GAAG,IAAI;IAIzC,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI;IAIrC,aAAa,CAAC,OAAO,EAAE,gBAAgB,GAAG,IAAI;CAoF9C"}
+{"version":3,"file":"assistant-message.d.ts","sourceRoot":"","sources":["../../../../src/modes/interactive/components/assistant-message.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,YAAY,CAAC;AACnD,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAC7C,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,KAAK,aAAa,EAAU,IAAI,EAAE,MAAM,aAAa,CAAC;AAEpF,OAAO,EAAmB,KAAK,eAAe,EAAE,MAAM,gBAAgB,CAAC;AAUvE;;;;;GAKG;AACH,wBAAgB,kBAAkB,CACjC,IAAI,EAAE,MAAM,EACZ,UAAU,EAAE,OAAO,EACnB,aAAa,EAAE,aAAa,GAC1B,QAAQ,CAGV;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CACrC,QAAQ,EAAE,MAAM,EAChB,aAAa,EAAE,aAAa,GAC1B,QAAQ,CAKV;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAErD;AAED;;;;;;;;GAQG;AACH,qBAAa,yBAA0B,SAAQ,SAAS;IACvD,OAAO,CAAC,gBAAgB,CAAY;IACpC,OAAO,CAAC,iBAAiB,CAAU;IACnC,OAAO,CAAC,aAAa,CAAS;IAC9B,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,WAAW,CAAC,CAAmB;IACvC,OAAO,CAAC,eAAe,CAAkB;gBAGxC,OAAO,CAAC,EAAE,gBAAgB,EAC1B,iBAAiB,UAAQ,EACzB,aAAa,GAAE,aAAkC,EACjD,eAAe,GAAE,eAAiC,EAClD,aAAa,SAAQ;IAkBb,UAAU,IAAI,IAAI;IAO3B,oBAAoB,CAAC,IAAI,EAAE,OAAO,GAAG,IAAI;IAIzC,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI;IAIrC;;;;;OAKG;IACH,aAAa,CAAC,OAAO,EAAE,gBAAgB,GAAG,IAAI;IA8E9C;;;;;;;OAOG;IACH,oBAAoB,CACnB,OAAO,EAAE,gBAAgB,EACzB,cAAc,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,SAAS,CAAC,GACrC,GAAG,CAAC,MAAM,EAAE,SAAS,CAAC;CA+EzB"}