@pi-kaush/pi-tool-call-markers 0.3.11 → 0.3.13

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## Unreleased
 
+- Paint collapsed rows with the theme's `dim` token instead of
+  `syntaxComment`. Pi's generated `system` theme solves `syntaxComment` to
+  the same contrast as `muted` and `toolOutput`, so collapsed tool calls
+  read almost like chat text there. `dim` stays a step below in every
+  theme; `collapsedToolCall` and `collapsedThinkingCall` overrides still win.
+
 - Give the codemode tool its own `¢` anchor and collapsed label. Codemode
   rows used to collapse to a script preview; they now show the nested
   calls the script ran (repeats deduped to `name ×N`) followed by the

package/README.md

@@ -33,19 +33,19 @@ Collapsed tool rows use semantic theme colors with no gear, background fill, box
 
 ## Subagent plans
 
-A recognized `subagent` call renders as an unboxed plan in the shared tool aesthetic, marked with `↪` instead of the ordinary `│` tool-call rail:
+A recognized `subagent` call renders as an unboxed plan in the shared tool aesthetic, using the same `│` tool-call rail:
 
 ```text
-  ↪ subagent chain (3 steps) [repo-review]
+  │ subagent chain (3 steps) [repo-review]
     1. 🐝 bee [workhorse] Challenge the compatibility conclusion…
     2. 🐝 bee …
 ```
 
-Single calls stay on one `↪ [<emoji>] [<profile>][<agent>] <task preview>` row — no tool label; the `↪` marker identifies it. Chain calls get a heading with the kind, count, and scope followed by numbered steps (parallel tasks list without numbers), with each step ordered emoji, profile badge, name badge. Agent display names (emoji + name) are scraped from the native plan component — including the single-call heading — with an args fallback, and render in `accent`; everything else stays muted, and failed subagents go full red. Subagents never join ordinary tool groups.
+Single calls stay on one `│ [<emoji>] [<profile>][<agent>] <task preview>` row — no tool label. Chain calls get a heading with the kind, count, and scope followed by numbered steps (parallel tasks list without numbers), with each step ordered emoji, profile badge, name badge. Agent display names (emoji + name) are scraped from the native plan component — including the single-call heading — with an args fallback, and render in `accent`; everything else stays muted, and failed subagents go full red. Subagents never join ordinary tool groups.
 
 While a subagent runs, the plan headline's tail shows live progress from the streamed result details — `→ 1 turn · provider/model` in the warning tone — and a settled call keeps the same `→ N turns · provider/model` summary in muted (turns aggregate across tasks; the model shows only when every task used the same one).
 
-Malformed, ambiguous, future, or too-narrow shapes fall back to the generic `↪ subagent …` collapsed row rather than dropping information, and `Ctrl+O` still exposes the native subagent renderer.
+Malformed, ambiguous, future, or too-narrow shapes fall back to the generic `│ subagent …` collapsed row rather than dropping information, and `Ctrl+O` still exposes the native subagent renderer.
 
 ## Asked questions
 
@@ -135,7 +135,7 @@ Names are comma-separated and matched exactly. Listed rows never join a group, a
 
 ### Collapsed-row colors
 
-Collapsed rows (tool calls, `+ Thought`, subagents) default to the theme's `syntaxComment` color — it ships with every Pi theme and reads as a muted tone — with the row's anchor and the call content sharing it. Failures stay error-colored and live spinners keep their existing tints.
+Collapsed rows (tool calls, `+ Thought`, subagents) default to the theme's `dim` color — it ships with every Pi theme and sits below `muted`, including in the generated `system` theme — with the row's anchor and the call content sharing it. Failures stay error-colored and live spinners keep their existing tints.
 
 A theme can override each collapsed kind independently with optional color tokens:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-kaush/pi-tool-call-markers",
-  "version": "0.3.11",
+  "version": "0.3.13",
   "description": "Compact and group Pi tool calls, with a bundled extension for adjacent thinking blocks.",
   "license": "MIT",
   "author": "Kaushik Gopal",

package/src/index.ts

@@ -24,7 +24,7 @@ import { isQuestionToolCall, renderQuestionBlock } from "./question-block.ts";
 import { sanitizeInline } from "./sanitize.ts";
 
 const OUTER_INSET = 2;
-const SUBAGENT_MARKER = "↪";
+const SUBAGENT_MARKER = "│";
 const GROUP_CALL_MARKER = "│";
 // Collapsed rows anchor with a glyph instead of repeating the tool name;
 // tools that read as one family share a glyph, and unmapped tools fall back
@@ -901,7 +901,7 @@ function codemodeCallLabel(row: ToolExecutionRow): string | undefined {
 // The collapsed-row seam lives here and nowhere else: which anchor a row
 // takes, how that anchor is styled, and the call text that follows it.
 // Anchors always render bold and take no joining colon — `│ ● src/a.ts`,
-// `│ $ npm test` — and subagent fallbacks keep their `↪ subagent …`
+// `│ $ npm test` — and subagent fallbacks keep their `│ subagent …`
 // reading instead of a glyph. Callers append the outcome tail.
 function collapsedSeam(
   row: ToolExecutionRow,
@@ -1173,10 +1173,10 @@ function subagentProgressText(row: ToolExecutionRow): string | undefined {
   return parts.length > 0 ? parts.join(" · ") : undefined;
 }
 
-// Subagents render as an ordinary unboxed tool block: a `↪` heading with the
+// Subagents render as an ordinary unboxed tool block: a `│` heading with the
 // plan kind/count/scope, then the numbered chain steps or parallel tasks with
 // agent identities (emoji, profile badge, name badge) in accent. Single calls
-// collapse to one row: `↪ <emoji> [<profile>][<name>] <task preview>`. Everything
+// collapse to one row: `│ <emoji> [<profile>][<name>] <task preview>`. Everything
 // else stays muted (or error on failure), matching the shared tool-row aesthetic.
 function renderSubagentPlan(
   row: ToolExecutionRow,

package/src/muted.ts

@@ -1,17 +1,18 @@
 // Collapsed-row color policy.
 //
 // Collapsed blocks (tool calls, settled "+ Thought" labels, subagent rows)
-// default to the theme's `syntaxComment` token — it ships with every Pi
-// theme and reads as a muted tone — instead of the louder
-// muted/toolTitle/toolOutput split. A theme can override each collapsed
-// kind independently with a `collapsedToolCall` or `collapsedThinkingCall`
-// color token.
+// default to the theme's `dim` token — it ships with every Pi theme and sits
+// below `muted` — instead of the louder muted/toolTitle/toolOutput split.
+// Pi's generated `system` theme solves `syntaxComment` to the same contrast
+// as `muted`, so only `dim` keeps collapsed rows clearly behind chat text.
+// A theme can override each collapsed kind independently with a
+// `collapsedToolCall` or `collapsedThinkingCall` color token.
 //
 // Failures always fall back to the historical tokens.
 
 const TOOL_OVERRIDE_TOKEN = "collapsedToolCall";
 const THINKING_OVERRIDE_TOKEN = "collapsedThinkingCall";
-const DEFAULT_TOKEN = "syntaxComment";
+const DEFAULT_TOKEN = "dim";
 
 type CollapsedTheme = {
   fg(color: string, text: string): string;
@@ -57,7 +58,7 @@ export function paletteSample(theme: CollapsedTheme): string {
   return PALETTE_TOKENS.map((token) => tokenAnsi(theme, token) ?? "").join(",");
 }
 
-// Override token when defined, else the theme's syntaxComment color, else
+// Override token when defined, else the theme's dim color, else
 // null (caller falls back to the historical tokens).
 function collapsedAnsi(
   theme: CollapsedTheme,
@@ -95,8 +96,7 @@ export function fgCollapsed(
 }
 
 // Decoration rails recede further than collapsed text. No palette token is
-// guaranteed lighter than the collapsed color (e.g. catppuccin-latte's `dim`
-// resolves to the same overlay0 as syntaxComment), so apply the terminal's
+// guaranteed lighter than the collapsed `dim` color, so apply the terminal's
 // faint attribute: it lightens toward the background on light themes and
 // darkens on dark ones. The rail keeps this muted weight even on failed
 // rows, where only the row's text turns error-colored.