pi-tool-display 0.1.5 → 0.1.7

package/CHANGELOG.md

@@ -7,6 +7,29 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.1.7] - 2026-03-07
+
+### Added
+- Added line-width safety utilities for diff rendering so collapsed and expanded diff output can be clamped to the current pane width.
+- Added utility test coverage for write display helpers, native user message box helpers, and narrow-width diff hint behavior.
+
+### Changed
+- Updated README documentation to reflect the current command surface, config model, width-safe diff behavior, native user message box pipeline, and project structure.
+- Refactored native user message box rendering into focused markdown, patching, renderer, and ANSI/background utility modules.
+
+### Fixed
+- Prevented diff rendering and collapsed diff hints from overflowing narrow terminal widths by progressively shortening hint text and clamping rendered lines.
+- Preserved inline `write` call summaries with line-count and byte-size metadata when content is available.
+- Prevented thinking label presentation changes from leaking into future assistant context by sanitizing stored thinking blocks during the `context` extension event.
+- Hardened thinking label normalization to strip ANSI residue fragments such as `38;5;208m` before display formatting.
+- Restored final-message thinking label persistence on `message_end` so themed labels remain consistent after streaming and across session reloads.
+- Improved native user message box rendering so markdown content, ANSI-only blank lines, and background fill behave more consistently.
+
+## [0.1.6] - 2026-03-04
+
+### Fixed
+- Use absolute GitHub raw URL for README image to fix npm display
+
 ## [0.1.5] - 2026-03-04
 
 ### Added

package/README.md

@@ -2,24 +2,25 @@
 
 OpenCode-style tool rendering for the [Pi coding agent](https://github.com/mariozechner/pi).
 
-`pi-tool-display` provides compact, expandable tool call/result rendering that keeps your terminal clean while preserving access to full details when needed.
+`pi-tool-display` keeps tool calls compact by default, adds richer diff rendering for file edits, and improves a few core chat UI details such as thinking labels and the native user prompt box.
 
-![Screenshot](assets/pi-tool-display.png)
+![Screenshot](https://raw.githubusercontent.com/MasuRii/pi-tool-display/main/assets/pi-tool-display.png)
 
 ## Features
 
-- **Compact Tool Rendering** — Overrides built-in `read`, `grep`, `find`, `ls`, `bash`, `edit`, and `write` tools with collapsed-by-default output
-- **MCP Tool Support** — Custom rendering for MCP gateway tool calls with configurable verbosity
-- **Rich Diff Display** — Adaptive split/unified diff views with syntax highlighting and inline change emphasis for `edit` and `write` operations
-- **Three Presets** — Quick verbosity profiles: `opencode` (minimal), `balanced` (summaries), `verbose` (previews)
-- **Thinking Labels** — Prefixes AI thinking blocks with themed labels for better readability
-- **Native User Message Box** — Bordered user prompt styling (optional)
-- **Extension Compatibility** — Per-tool ownership toggles to avoid conflicts with other extensions
-- **Capability Detection** — Auto-hides MCP/RTK settings when those features aren't available
+- **Compact built-in tool rendering** for `read`, `grep`, `find`, `ls`, `bash`, `edit`, and `write`
+- **MCP-aware rendering** with hidden, summary, and preview modes
+- **Adaptive edit/write diffs** with split or unified layouts, syntax highlighting, inline emphasis, and narrow-pane width clamping
+- **Progressive collapsed diff hints** that shorten automatically on small terminal widths instead of overflowing
+- **Three presets**: `opencode`, `balanced`, and `verbose`
+- **Thinking labels** during streaming and final message rendering, with context sanitization to avoid leaking presentation labels back into future model turns
+- **Optional native user message box** with markdown-aware rendering and safer ANSI/background handling
+- **Per-tool ownership toggles** so this extension can coexist with other renderer extensions
+- **Capability-aware settings** that automatically hide MCP and RTK-specific controls when those features are unavailable
 
 ## Installation
 
-### Local Extension Folder
+### Local extension folder
 
 Place this folder in one of Pi's auto-discovery locations:
 
@@ -31,13 +32,13 @@ Place this folder in one of Pi's auto-discovery locations:
 .pi/extensions/pi-tool-display
 ```
 
-### npm Package
+### npm package
 
 ```bash
 pi install npm:pi-tool-display
 ```
 
-### Git Repository
+### Git repository
 
 ```bash
 pi install git:github.com/MasuRii/pi-tool-display
@@ -45,7 +46,7 @@ pi install git:github.com/MasuRii/pi-tool-display
 
 ## Usage
 
-### Interactive Settings
+### Interactive settings
 
 Open the settings modal:
 
@@ -53,11 +54,24 @@ Open the settings modal:
 /tool-display
 ```
 
-### Direct Commands
+The modal exposes the day-to-day controls most people change regularly:
+
+- preset profile
+- read output mode
+- grep/find/ls output mode
+- MCP output mode (when MCP is available)
+- preview line count
+- bash collapsed line count
+- diff layout mode
+- native user message box toggle
+
+Advanced options remain in `config.json`.
+
+### Direct commands
 
 ```text
-/tool-display show                    # Display current configuration
-/tool-display reset                   # Reset to default settings
+/tool-display show                    # Show the effective config summary
+/tool-display reset                   # Reset to the default opencode preset
 /tool-display preset opencode         # Apply opencode preset
 /tool-display preset balanced         # Apply balanced preset
 /tool-display preset verbose          # Apply verbose preset
@@ -71,9 +85,9 @@ Open the settings modal:
 | `balanced` | summary | count | summary | 8 | 10 |
 | `verbose` | preview | preview | preview | 12 | 20 |
 
-**opencode** (default) — Minimal inline-only display; tool results stay collapsed  
-**balanced** — Compact summaries showing line counts and match totals  
-**verbose** — Expanded previews with more visible content by default
+- **`opencode`** (default): minimal inline-only display; tool results stay collapsed
+- **`balanced`**: compact summaries with line counts and match totals
+- **`verbose`**: larger previews for read/search/MCP output and more visible bash output
 
 ## Configuration
 
@@ -83,30 +97,30 @@ Runtime configuration is stored at:
 ~/.pi/agent/extensions/pi-tool-display/config.json
 ```
 
-A starter template is available at `config/config.example.json`.
+A starter template is included at `config/config.example.json`.
 
-### Configuration Options
+### Configuration options
 
 | Option | Type | Default | Description |
 |--------|------|---------|-------------|
-| `registerToolOverrides` | object | all `true` | Per-tool ownership flags (see below) |
-| `enableNativeUserMessageBox` | boolean | `true` | Enable bordered user message styling |
+| `registerToolOverrides` | object | all `true` | Per-tool ownership flags |
+| `enableNativeUserMessageBox` | boolean | `true` | Enable bordered user prompt rendering |
 | `readOutputMode` | string | `"hidden"` | `hidden`, `summary`, or `preview` |
 | `searchOutputMode` | string | `"hidden"` | `hidden`, `count`, or `preview` |
 | `mcpOutputMode` | string | `"hidden"` | `hidden`, `summary`, or `preview` |
-| `previewLines` | number | `8` | Lines shown in collapsed preview |
-| `expandedPreviewMaxLines` | number | `4000` | Max lines when expanded |
-| `bashCollapsedLines` | number | `10` | Lines shown for bash output |
+| `previewLines` | number | `8` | Lines shown in collapsed preview mode |
+| `expandedPreviewMaxLines` | number | `4000` | Max preview lines when fully expanded |
+| `bashCollapsedLines` | number | `10` | Lines shown for collapsed bash output |
 | `diffViewMode` | string | `"auto"` | `auto`, `split`, or `unified` |
-| `diffSplitMinWidth` | number | `120` | Minimum terminal width for split view |
-| `diffCollapsedLines` | number | `24` | Lines shown in collapsed diff |
-| `diffWordWrap` | boolean | `true` | Wrap long lines in diff view |
-| `showTruncationHints` | boolean | `true` | Show truncation indicators |
-| `showRtkCompactionHints` | boolean | `true` | Show RTK compaction hints |
+| `diffSplitMinWidth` | number | `120` | Minimum width before auto mode prefers split diffs |
+| `diffCollapsedLines` | number | `24` | Diff lines shown before collapsing |
+| `diffWordWrap` | boolean | `true` | Wrap long diff lines when needed |
+| `showTruncationHints` | boolean | `true` | Show truncation indicators for compacted output |
+| `showRtkCompactionHints` | boolean | `true` | Show RTK compaction hints when RTK metadata exists |
 
-### Tool Override Ownership
+### Tool ownership
 
-Control which tools this extension overrides:
+Use `registerToolOverrides` to control which built-in tools this extension owns:
 
 ```json
 {
@@ -122,11 +136,11 @@ Control which tools this extension overrides:
 }
 ```
 
-Set any tool to `false` to leave rendering ownership to another extension.
+Set any entry to `false` if another extension should handle that tool instead.
 
-> **Note:** Changes to tool ownership require `/reload` to take effect.
+> Changes to tool ownership take effect after `/reload`.
 
-### Example Configuration
+### Example config
 
 ```json
 {
@@ -155,55 +169,85 @@ Set any tool to `false` to leave rendering ownership to another extension.
 }
 ```
 
-## Capability Detection
+## Rendering notes
+
+### Edit and write diffs
+
+`edit` and `write` results use the same diff renderer. In `auto` mode the extension chooses split or unified layout based on available width. On narrow panes it clamps rendered lines and shortens collapsed hint text so the diff stays readable instead of spilling past the terminal width.
+
+### Write summaries
+
+When content is available, `write` call summaries include line count and byte size information inline so you can quickly see the size of the pending write before expanding the result.
+
+### Thinking labels
 
-The extension automatically detects available capabilities:
+Thinking blocks are labeled during streaming and on final messages. Before the next model turn, the extension sanitizes those presentation labels out of the stored assistant context so they do not accumulate or pollute future prompts.
 
-- **MCP Tooling** — When no MCP tools are available, MCP-related settings are hidden and MCP output mode is forced to `hidden`
-- **RTK Optimizer** — When [pi-rtk-optimizer](https://github.com/MasuRii/pi-rtk-optimizer) isn't installed, RTK compaction hint settings are hidden and hints are disabled
+### Native user message box
 
-This prevents confusion from settings that have no effect in your environment.
+When enabled, user prompts render inside a bordered box using Pi's native user message component. The renderer preserves markdown content more safely and normalizes ANSI/background handling to avoid odd nested background artifacts.
+
+## Capability detection
+
+The extension checks the current Pi environment and adjusts behavior automatically:
+
+- **MCP tooling unavailable**: MCP settings are hidden and MCP output is forced off
+- **RTK optimizer unavailable**: RTK hint settings are hidden and RTK compaction hints are disabled
+
+This keeps the UI aligned with what the current environment can actually support.
 
 ## Troubleshooting
 
-### Tool Ownership Conflicts
+### Tool ownership conflicts
 
-If another extension owns a tool and you see rendering conflicts:
+If another extension is already rendering one of the built-in tools:
 
-1. Set the corresponding `registerToolOverrides.<tool>` to `false` in your config
-2. Run `/reload` in Pi
-3. Verify with `/tool-display show` that the ownership reflects expected `off` values
+1. Set `registerToolOverrides.<tool>` to `false`
+2. Run `/reload`
+3. Use `/tool-display show` to confirm the effective ownership state
 
-### Configuration Not Loading
+### Config not loading
 
-If settings aren't applying:
+If your settings are not being applied:
 
-1. Check the config file exists at `~/.pi/agent/extensions/pi-tool-display/config.json`
-2. Verify JSON syntax is valid
-3. Run `/tool-display show` to see current effective configuration
+1. Check that `~/.pi/agent/extensions/pi-tool-display/config.json` exists
+2. Make sure the JSON is valid
+3. Run `/tool-display show` to inspect the effective config summary
 
-## Project Structure
+### MCP or RTK settings missing
+
+Those controls only appear when the corresponding capability is available in the current Pi environment.
+
+## Project structure
 
 ```text
 pi-tool-display/
-├── index.ts                    # Extension entrypoint (Pi auto-discovery)
+├── index.ts                         # Extension entrypoint for Pi auto-discovery
 ├── src/
-│   ├── index.ts                # Extension bootstrap and registration
-│   ├── tool-overrides.ts       # Built-in and MCP renderer overrides
-│   ├── diff-renderer.ts        # Edit/write diff rendering engine
-│   ├── config-modal.ts         # /tool-display settings UI
-│   ├── capabilities.ts         # MCP/RTK capability detection
-│   ├── config-store.ts         # Config load/save and normalization
-│   ├── presets.ts              # Preset definitions and matching
-│   ├── render-utils.ts         # Shared rendering helpers
-│   ├── thinking-label.ts       # Thinking block label formatting
-│   ├── user-message-box-native.ts  # User message border styling
-│   ├── types.ts                # TypeScript type definitions
-│   └── zellij-modal.ts         # Modal UI primitives
+│   ├── index.ts                     # Bootstrap and extension registration
+│   ├── capabilities.ts              # MCP/RTK capability detection
+│   ├── config-modal.ts              # /tool-display settings UI and command handling
+│   ├── config-store.ts              # Config load/save and normalization
+│   ├── diff-renderer.ts             # Edit/write diff rendering engine
+│   ├── line-width-safety.ts         # Width clamping helpers for narrow panes
+│   ├── presets.ts                   # Preset definitions and matching
+│   ├── render-utils.ts              # Shared rendering helpers
+│   ├── thinking-label.ts            # Thinking label formatting and context sanitization
+│   ├── tool-overrides.ts            # Built-in and MCP renderer overrides
+│   ├── types.ts                     # Shared config and type definitions
+│   ├── user-message-box-markdown.ts # Markdown extraction for user message rendering
+│   ├── user-message-box-native.ts   # Native user message box registration
+│   ├── user-message-box-patch.ts    # Safe native render patching helpers
+│   ├── user-message-box-renderer.ts # User message border renderer
+│   ├── user-message-box-utils.ts    # ANSI/background normalization helpers
+│   ├── write-display-utils.ts       # Write summary helpers
+│   └── zellij-modal.ts              # Modal UI primitives
 ├── config/
-│   └── config.example.json     # Starter config template
+│   └── config.example.json          # Starter config template
+├── tests/
+│   └── tool-ui-utils.test.ts        # Utility tests for user message and diff helpers
 └── assets/
-    └── pi-tool-display.png     # README screenshot
+    └── pi-tool-display.png          # README screenshot
 ```
 
 ## Development
@@ -212,17 +256,14 @@ pi-tool-display/
 # Type check
 npm run build
 
-# Lint (same as build)
-npm run lint
-
 # Run tests
 npm run test
 
-# Full check
+# Full verification
 npm run check
 ```
 
-## Related Extensions
+## Related extensions
 
 - [pi-rtk-optimizer](https://github.com/MasuRii/pi-rtk-optimizer) — RTK optimizer for token-efficient source output
 

package/config/config.example.json

@@ -1,24 +1,24 @@
-{
-  "registerToolOverrides": {
-    "read": true,
-    "grep": true,
-    "find": true,
-    "ls": true,
-    "bash": true,
-    "edit": true,
-    "write": true
-  },
-  "enableNativeUserMessageBox": true,
-  "readOutputMode": "hidden",
-  "searchOutputMode": "hidden",
-  "mcpOutputMode": "hidden",
-  "previewLines": 8,
-  "expandedPreviewMaxLines": 4000,
-  "bashCollapsedLines": 10,
-  "diffViewMode": "auto",
-  "diffSplitMinWidth": 120,
-  "diffCollapsedLines": 24,
-  "diffWordWrap": true,
-  "showTruncationHints": true,
-  "showRtkCompactionHints": true
-}
+{
+  "registerToolOverrides": {
+    "read": true,
+    "grep": true,
+    "find": true,
+    "ls": true,
+    "bash": true,
+    "edit": true,
+    "write": true
+  },
+  "enableNativeUserMessageBox": true,
+  "readOutputMode": "hidden",
+  "searchOutputMode": "hidden",
+  "mcpOutputMode": "hidden",
+  "previewLines": 8,
+  "expandedPreviewMaxLines": 4000,
+  "bashCollapsedLines": 10,
+  "diffViewMode": "auto",
+  "diffSplitMinWidth": 120,
+  "diffCollapsedLines": 24,
+  "diffWordWrap": true,
+  "showTruncationHints": true,
+  "showRtkCompactionHints": true
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-tool-display",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "OpenCode-style tool call/result rendering extension for Pi coding agent.",
   "type": "module",
   "main": "./index.ts",

package/src/capabilities.ts

@@ -1,94 +1,94 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
-import { existsSync } from "node:fs";
-import { homedir } from "node:os";
-import { join } from "node:path";
-import type { ToolDisplayConfig } from "./types.js";
-
-export interface ToolDisplayCapabilities {
-	hasMcpTooling: boolean;
-	hasRtkOptimizer: boolean;
-}
-
-function toRecord(value: unknown): Record<string, unknown> {
-	if (!value || typeof value !== "object" || Array.isArray(value)) {
-		return {};
-	}
-	return value as Record<string, unknown>;
-}
-
-function getTextField(value: unknown, field: string): string | undefined {
-	const record = toRecord(value);
-	const raw = record[field];
-	return typeof raw === "string" && raw.trim().length > 0 ? raw.trim() : undefined;
-}
-
-function isMcpToolCandidate(tool: unknown): boolean {
-	const name = getTextField(tool, "name");
-	if (name === "mcp") {
-		return true;
-	}
-
-	const label = getTextField(tool, "label");
-	if (label?.startsWith("MCP ")) {
-		return true;
-	}
-
-	const description = getTextField(tool, "description");
-	return typeof description === "string" && /\bmcp\b/i.test(description);
-}
-
-function hasMcpTooling(pi: ExtensionAPI): boolean {
-	try {
-		const allTools = pi.getAllTools();
-		return allTools.some((tool) => isMcpToolCandidate(tool));
-	} catch {
-		return false;
-	}
-}
-
-function hasRtkCommand(pi: ExtensionAPI): boolean {
-	try {
-		const commands = pi.getCommands();
-		return commands.some((command) => command.name === "rtk" || command.name.startsWith("rtk-"));
-	} catch {
-		return false;
-	}
-}
-
-function hasRtkExtensionPath(cwd: string): boolean {
-	const candidates = [
-		join(homedir(), ".pi", "agent", "extensions", "pi-rtk-optimizer"),
-		join(cwd, ".pi", "extensions", "pi-rtk-optimizer"),
-	];
-
-	for (const candidate of candidates) {
-		try {
-			if (existsSync(candidate)) {
-				return true;
-			}
-		} catch {
-			// Ignore filesystem errors and continue probing other candidates.
-		}
-	}
-
-	return false;
-}
-
-export function detectToolDisplayCapabilities(pi: ExtensionAPI, cwd: string): ToolDisplayCapabilities {
-	return {
-		hasMcpTooling: hasMcpTooling(pi),
-		hasRtkOptimizer: hasRtkCommand(pi) || hasRtkExtensionPath(cwd),
-	};
-}
-
-export function applyCapabilityConfigGuards(
-	config: ToolDisplayConfig,
-	capabilities: ToolDisplayCapabilities,
-): ToolDisplayConfig {
-	return {
-		...config,
-		registerToolOverrides: { ...config.registerToolOverrides },
-		mcpOutputMode: capabilities.hasMcpTooling ? config.mcpOutputMode : "hidden",
-		showRtkCompactionHints: capabilities.hasRtkOptimizer ? config.showRtkCompactionHints : false,
-	};
-}
+import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import { existsSync } from "node:fs";
+import { homedir } from "node:os";
+import { join } from "node:path";
+import type { ToolDisplayConfig } from "./types.js";
+
+export interface ToolDisplayCapabilities {
+	hasMcpTooling: boolean;
+	hasRtkOptimizer: boolean;
+}
+
+function toRecord(value: unknown): Record<string, unknown> {
+	if (!value || typeof value !== "object" || Array.isArray(value)) {
+		return {};
+	}
+	return value as Record<string, unknown>;
+}
+
+function getTextField(value: unknown, field: string): string | undefined {
+	const record = toRecord(value);
+	const raw = record[field];
+	return typeof raw === "string" && raw.trim().length > 0 ? raw.trim() : undefined;
+}
+
+function isMcpToolCandidate(tool: unknown): boolean {
+	const name = getTextField(tool, "name");
+	if (name === "mcp") {
+		return true;
+	}
+
+	const label = getTextField(tool, "label");
+	if (label?.startsWith("MCP ")) {
+		return true;
+	}
+
+	const description = getTextField(tool, "description");
+	return typeof description === "string" && /\bmcp\b/i.test(description);
+}
+
+function hasMcpTooling(pi: ExtensionAPI): boolean {
+	try {
+		const allTools = pi.getAllTools();
+		return allTools.some((tool) => isMcpToolCandidate(tool));
+	} catch {
+		return false;
+	}
+}
+
+function hasRtkCommand(pi: ExtensionAPI): boolean {
+	try {
+		const commands = pi.getCommands();
+		return commands.some((command) => command.name === "rtk" || command.name.startsWith("rtk-"));
+	} catch {
+		return false;
+	}
+}
+
+function hasRtkExtensionPath(cwd: string): boolean {
+	const candidates = [
+		join(homedir(), ".pi", "agent", "extensions", "pi-rtk-optimizer"),
+		join(cwd, ".pi", "extensions", "pi-rtk-optimizer"),
+	];
+
+	for (const candidate of candidates) {
+		try {
+			if (existsSync(candidate)) {
+				return true;
+			}
+		} catch {
+			// Ignore filesystem errors and continue probing other candidates.
+		}
+	}
+
+	return false;
+}
+
+export function detectToolDisplayCapabilities(pi: ExtensionAPI, cwd: string): ToolDisplayCapabilities {
+	return {
+		hasMcpTooling: hasMcpTooling(pi),
+		hasRtkOptimizer: hasRtkCommand(pi) || hasRtkExtensionPath(cwd),
+	};
+}
+
+export function applyCapabilityConfigGuards(
+	config: ToolDisplayConfig,
+	capabilities: ToolDisplayCapabilities,
+): ToolDisplayConfig {
+	return {
+		...config,
+		registerToolOverrides: { ...config.registerToolOverrides },
+		mcpOutputMode: capabilities.hasMcpTooling ? config.mcpOutputMode : "hidden",
+		showRtkCompactionHints: capabilities.hasRtkOptimizer ? config.showRtkCompactionHints : false,
+	};
+}