@quandev104/pi-style 0.1.7 → 0.2.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [0.2.0] - 2026-08-08
+
+### Bug Fixes
+
+- *(turn-summary,box)* Broken frames on collapse/expand, faileds plural, stray placeholder rows
+
+### Features
+
+- *(turn-summary)* Keep mutating tool blocks visible after the turn
+
 ## [0.1.7] - 2026-08-07
 
 ### Features

package/dist/extensions/pi-style.js

@@ -949,8 +949,16 @@ function renderBoxedOutputLines(theme, outputLines, width, rawLineBudget = DEFAU
   let nextInputIndex = 0;
   let truncated = false;
   for (; nextInputIndex < outputLines.length; nextInputIndex++) {
-    const line = boxedTruncatedLine(theme, outputLines[nextInputIndex] ?? "", width);
-    if (!pushBoundedLines(head, [line], headLimit)) {
+    const fragments = (outputLines[nextInputIndex] ?? "").split("\n");
+    let headExceeded = false;
+    for (const fragment of fragments) {
+      const line = boxedTruncatedLine(theme, fragment, width);
+      if (!pushBoundedLines(head, [line], headLimit)) {
+        headExceeded = true;
+        break;
+      }
+    }
+    if (headExceeded) {
       truncated = true;
       nextInputIndex++;
       break;
@@ -960,9 +968,12 @@ function renderBoxedOutputLines(theme, outputLines, width, rawLineBudget = DEFAU
   const tail = [];
   const tailStart = Math.max(nextInputIndex, outputLines.length - tailLimit);
   for (let i = tailStart; i < outputLines.length; i++) {
-    const line = boxedTruncatedLine(theme, outputLines[i] ?? "", width);
-    tail.push(line);
-    if (tail.length > tailLimit) tail.splice(0, tail.length - tailLimit);
+    const fragments = (outputLines[i] ?? "").split("\n");
+    for (const fragment of fragments) {
+      const line = boxedTruncatedLine(theme, fragment, width);
+      tail.push(line);
+      if (tail.length > tailLimit) tail.splice(0, tail.length - tailLimit);
+    }
   }
   const skippedInputLines = Math.max(0, tailStart - nextInputIndex);
   const skippedText = skippedInputLines > 0 ? `\u2026 rendered output truncated; ${skippedInputLines} input lines skipped before tail` : "\u2026 rendered output truncated";
@@ -1083,6 +1094,7 @@ function renderBoxedToolResult(theme, body, options = {}) {
       const bodyLines = typeof body === "function" ? body(maxContentWidth) : body.render(maxContentWidth);
       const errorPrefix = options.isError ? [theme.fg("error", options.errorLabel ?? "\u2717 Error")] : [];
       const outputLines = bodyLines.length > 0 ? [...errorPrefix, ...bodyLines] : [theme.fg("muted", `\u2205 ${options.emptyText ?? "(no output)"}`)];
+      const outputFragments = outputLines.flatMap((line) => line.split("\n"));
       const footerText = (options.footerLines ?? []).join(" \xB7 ");
       const dividerText = typeof options.dividerLabel === "function" ? options.dividerLabel(renderedWidth) : options.dividerLabel ?? "Response";
       const rendered = [
@@ -1097,7 +1109,12 @@ function renderBoxedToolResult(theme, body, options = {}) {
           )
         ],
         boxBlankLine(theme, renderedWidth),
-        ...renderBoxedOutputLines(theme, outputLines, renderedWidth, options.renderLineBudget ?? outputLines.length),
+        ...renderBoxedOutputLines(
+          theme,
+          outputFragments,
+          renderedWidth,
+          options.renderLineBudget ?? outputFragments.length
+        ),
         boxBlankLine(theme, renderedWidth),
         boxLabeledBorder(
           theme,
@@ -1419,7 +1436,8 @@ var sessionToolsConfig = {
   showElapsed: true,
   batchOpenGlyph: "\u25CF",
   nerdFonts: false,
-  collapseAfterTurn: true
+  collapseAfterTurn: true,
+  collapseMutatingTools: false
 };
 function setToolsRenderConfig(config) {
   sessionToolsConfig = { ...sessionToolsConfig, ...config };
@@ -1775,6 +1793,13 @@ function emptyBatchResult() {
 }
 
 // extension-src/pi-style/features/tools/boxed/turn-summary.ts
+var MUTATING_TOOLS = /* @__PURE__ */ new Set(["edit", "write", "quick_edit", "substitute_edit", "target_edit"]);
+function isMutatingTool(toolName) {
+  return MUTATING_TOOLS.has(toolName);
+}
+function mutatingCollapses() {
+  return getToolsRenderConfig().collapseMutatingTools;
+}
 var memberByCallId = /* @__PURE__ */ new Map();
 var invalidateByCallId = /* @__PURE__ */ new Map();
 function resetTurnRegistry() {
@@ -1805,7 +1830,7 @@ function registerTurn(calls, isErrorById, ended) {
       isError: isErrorById.get(toolCallId) === true
     };
   });
-  const leader = members.find((member) => !member.isError);
+  const leader = members.find((member) => !member.isError && (!isMutatingTool(member.toolName) || mutatingCollapses()));
   const turn = {
     leaderId: leader?.toolCallId ?? "",
     ended: ended && complete,
@@ -1835,7 +1860,9 @@ function registerTurnFromMessage(message, toolResults) {
       isError: isErrorById.get(toolCallId) === true
     };
   });
-  const leader = newMembers.find((member) => !member.isError);
+  const leader = newMembers.find(
+    (member) => !member.isError && (!isMutatingTool(member.toolName) || mutatingCollapses())
+  );
   if (!currentRun) {
     currentRun = {
       leaderId: leader?.toolCallId ?? "",
@@ -1935,11 +1962,13 @@ function turnSummaryParts(turn) {
   const order = [];
   let failedCount = 0;
   let elapsedMs;
+  const collapseMutating = mutatingCollapses();
   for (const member of turn.members) {
     if (member.isError) {
       failedCount++;
       continue;
     }
+    if (!collapseMutating && isMutatingTool(member.toolName)) continue;
     if (member.elapsedMs !== void 0) elapsedMs = (elapsedMs ?? 0) + member.elapsedMs;
     const existing = counts.get(member.toolName);
     if (existing === void 0) {
@@ -1949,8 +1978,8 @@ function turnSummaryParts(turn) {
   }
   const parts = order.map((toolName) => {
     const count = counts.get(toolName) ?? 0;
-    const style = TURN_SUMMARY_STYLE[toolName] ?? { verb: "ran", unit: `${toolName} call` };
-    return `${style.verb} ${count} ${pluralForm(style.unit, count)}`;
+    const style = TURN_SUMMARY_STYLE[toolName];
+    return style ? `${style.verb} ${count} ${pluralForm(style.unit, count)}` : `used ${count} ${toolName}`;
   });
   return { parts, failedCount, elapsedMs };
 }
@@ -1959,7 +1988,7 @@ function formatTurnSummaryLine(theme, turn) {
   const parts = summary.parts.join(", ");
   let line = `${theme.fg("dim", `\u2794 ${parts}`)}`;
   if (summary.failedCount > 0)
-    line += theme.fg("error", ` \xB7 ${summary.failedCount} ${pluralForm("failed", summary.failedCount)}`);
+    line += theme.fg("error", ` \xB7 ${summary.failedCount} ${pluralForm("failure", summary.failedCount)}`);
   if (summary.elapsedMs !== void 0) line += theme.fg("dim", ` \xB7 ${(summary.elapsedMs / 1e3).toFixed(2)}s`);
   return line;
 }
@@ -6314,9 +6343,11 @@ var REGISTRY = {
   target_edit: quickEditToolFor("target_edit")
 };
 function collapsedTurnFor(toolCallId, expanded) {
-  if (expanded || !getToolsRenderConfig().collapseAfterTurn) return void 0;
+  const config = getToolsRenderConfig();
+  if (expanded || !config.collapseAfterTurn) return void 0;
   const entry = getTurnEntry(toolCallId);
   if (!entry?.turn.ended || entry.member.isError) return void 0;
+  if (isMutatingTool(entry.member.toolName) && !config.collapseMutatingTools) return void 0;
   return entry.turn;
 }
 function renderBoxedToolCall2(toolName, args, theme, context) {
@@ -6596,14 +6627,27 @@ function createToolDecorationOwner(snapshot = {}) {
         if (typeof renderer !== "function") {
           neutralizeToolContainerBackground(instance);
           if (subtype === "tool-call-renderer")
-            return (callArgs, theme, context) => renderBoxedToolCall2(toolName, callArgs, theme, context);
-          return (result, options, theme, context) => renderBoxedToolResult2(
-            toolName,
-            result,
-            options,
-            theme,
-            context
-          );
+            return (callArgs, theme, context) => {
+              const component = renderBoxedToolCall2(
+                toolName,
+                callArgs,
+                theme,
+                context
+              );
+              if (component === EMPTY_BATCH_COMPONENT) hideBatchMember(instance);
+              return component;
+            };
+          return (result, options, theme, context) => {
+            const component = renderBoxedToolResult2(
+              toolName,
+              result,
+              options,
+              theme,
+              context
+            );
+            if (component === EMPTY_BATCH_COMPONENT) hideBatchMember(instance);
+            return component;
+          };
         }
         return function(...rendererArgs) {
           const valid = subtype === "tool-call-renderer" ? validCallArgs(rendererArgs) : validResultArgs(rendererArgs);
@@ -6824,7 +6868,8 @@ var DEFAULT_CONFIG = Object.freeze({
     maxExpandedLines: 50,
     dimOutput: false,
     showElapsed: true,
-    collapseAfterTurn: true
+    collapseAfterTurn: true,
+    collapseMutatingTools: false
   }),
   theme: Object.freeze({
     nerdFonts: "auto",
@@ -6940,7 +6985,8 @@ function normalizeConfig(input, defaults = DEFAULT_CONFIG) {
       maxExpandedLines: maxExpanded,
       dimOutput: bool(tools.dimOutput, defaults.tools.dimOutput),
       showElapsed: bool(tools.showElapsed, defaults.tools.showElapsed),
-      collapseAfterTurn: bool(tools.collapseAfterTurn, defaults.tools.collapseAfterTurn)
+      collapseAfterTurn: bool(tools.collapseAfterTurn, defaults.tools.collapseAfterTurn),
+      collapseMutatingTools: bool(tools.collapseMutatingTools, defaults.tools.collapseMutatingTools)
     }),
     theme: Object.freeze({
       nerdFonts: stringEnum(theme.nerdFonts, ["auto", "on", "off"], defaults.theme.nerdFonts),
@@ -6988,6 +7034,7 @@ var BOOL_PATHS = /* @__PURE__ */ new Set([
   "tools.showElapsed",
   "tools.dimOutput",
   "tools.collapseAfterTurn",
+  "tools.collapseMutatingTools",
   "compatibility.allowSafePatches",
   "compatibility.allowCorePatches",
   "compatibility.preferExistingEditor",
@@ -7458,6 +7505,7 @@ var allowedPaths = /* @__PURE__ */ new Set([
   "tools.dimOutput",
   "tools.showElapsed",
   "tools.collapseAfterTurn",
+  "tools.collapseMutatingTools",
   "theme.nerdFonts",
   "theme.terminalBackgroundSync",
   "theme.autoApply",
@@ -7486,6 +7534,7 @@ function validatePathValue(path, value) {
     "tools.showElapsed",
     "tools.dimOutput",
     "tools.collapseAfterTurn",
+    "tools.collapseMutatingTools",
     "compatibility.allowSafePatches",
     "compatibility.allowCorePatches",
     "compatibility.preferExistingEditor",