@oh-my-pi/pi-coding-agent 16.0.10 → 16.0.11

package/src/commit/agentic/agent.ts

@@ -1,7 +1,8 @@
-import { INTENT_FIELD, type ThinkingLevel } from "@oh-my-pi/pi-agent-core";
+import type { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import type { Api, Model } from "@oh-my-pi/pi-ai";
 import { Markdown } from "@oh-my-pi/pi-tui";
 import { prompt } from "@oh-my-pi/pi-utils";
+import { INTENT_FIELD } from "@oh-my-pi/pi-wire";
 import chalk from "chalk";
 import typesDescriptionPrompt from "../../commit/prompts/types-description.md" with { type: "text" };
 import type { ModelRegistry } from "../../config/model-registry";

package/src/commit/agentic/tools/git-file-diff.ts

@@ -87,7 +87,7 @@ function truncateDiffContent(diff: string): { content: string; truncated: boolea
 	const truncatedCount = lines.length - KEEP_HEAD_LINES - KEEP_TAIL_LINES;
 
 	return {
-		content: [...head, `\n... (truncated ${truncatedCount} lines) ...\n`, ...tail].join("\n"),
+		content: [...head, `\n[…${truncatedCount}ln elided…]\n`, ...tail].join("\n"),
 		truncated: true,
 	};
 }
@@ -117,7 +117,7 @@ function processDiffs(files: string[], diffs: Map<string, string>): { result: st
 			}
 			content = truncated;
 			if (content.length > remaining) {
-				content = `${content.slice(0, remaining)}\n... (diff truncated due to size) ...`;
+				content = `${content.slice(0, remaining)}\n[…${content.length - remaining}ch elided…]`;
 				if (!truncatedFiles.includes(file)) {
 					truncatedFiles.push(file);
 				}

package/src/commit/changelog/index.ts

@@ -138,7 +138,7 @@ export async function applyChangelogProposals({
 
 function truncateDiff(diff: string, maxChars: number): string {
 	if (diff.length <= maxChars) return diff;
-	return `${diff.slice(0, maxChars)}\n... (truncated)`;
+	return `${diff.slice(0, maxChars)}\n[…${diff.length - maxChars}ch elided…]`;
 }
 
 function formatExistingEntries(entries: Record<string, string[]>): string {

package/src/commit/map-reduce/map-phase.ts

@@ -126,7 +126,7 @@ function generateContextHeader(files: FileDiff[], currentFile: string): string {
 	}
 
 	if (toShow.length < sorted.length) {
-		lines.push(`... and ${sorted.length - toShow.length} more files`);
+		lines.push(`[…${sorted.length - toShow.length} files elided…]`);
 	}
 
 	return lines.join("\n");

package/src/commit/map-reduce/utils.ts

@@ -5,5 +5,5 @@ export function estimateTokens(text: string): number {
 export function truncateToTokenLimit(text: string, maxTokens: number): string {
 	const maxChars = maxTokens * 4;
 	if (text.length <= maxChars) return text;
-	return `${text.slice(0, maxChars)}\n... (truncated)`;
+	return `${text.slice(0, maxChars)}\n[…${text.length - maxChars}ch elided…]`;
 }

package/src/config/settings-schema.ts

@@ -809,11 +809,6 @@ export const SETTINGS_SCHEMA = {
 			description: "Remove the 1-character horizontal padding from the left and right of the terminal output",
 		},
 	},
-	// Display rendering
-	"display.tabWidth": {
-		type: "number",
-		default: 3,
-	},
 
 	"display.shimmer": {
 		type: "enum",

package/src/config/settings.ts

@@ -22,7 +22,6 @@ import {
 	isEnoent,
 	logger,
 	procmgr,
-	setDefaultTabWidth,
 } from "@oh-my-pi/pi-utils";
 import { JSONC, YAML } from "bun";
 import { type Settings as SettingsCapabilityItem, settingsCapability } from "../capability/settings";
@@ -1103,11 +1102,6 @@ const SETTING_HOOKS: Partial<Record<SettingPath, SettingHook<any>>> = {
 			});
 		}
 	},
-	"display.tabWidth": value => {
-		if (typeof value === "number") {
-			setDefaultTabWidth(value);
-		}
-	},
 	"provider.appendOnlyContext": value => {
 		if (typeof value === "string") {
 			appendOnlyModeSignal.fire(value);

package/src/edit/file-snapshot-store.ts

@@ -102,7 +102,7 @@ const HASHLINE_LINE_PREFIX = /^[ *]?(\d+)(?:-(\d+))?:/;
 /**
  * The 1-indexed file lines a hashline-formatted body actually displayed.
  * Single `NN:` rows contribute that line; a collapsed summary `NN-MM:` row
- * (a `{ .. }` brace pair) contributes only its boundary lines `NN` and `MM` —
+ * (a `{ … }` brace pair) contributes only its boundary lines `NN` and `MM` —
  * the elided interior was never shown, so editing inside it must be rejected.
  */
 export function parseSeenLinesFromHashlineBody(body: string): number[] {

package/src/edit/renderer.ts

@@ -230,7 +230,7 @@ function truncateEditTitlePath(displayPath: string, maxWidth: number | undefined
 }
 
 function formatEditTitlePath(pathValue: string, maxWidth?: number): string {
-	return truncateEditTitlePath(replaceTabs(shortenPath(pathValue), pathValue), maxWidth);
+	return truncateEditTitlePath(replaceTabs(shortenPath(pathValue)), maxWidth);
 }
 
 function formatEditPathDisplay(
@@ -326,11 +326,11 @@ function renderEditHeader(
 	return buildHeader(fitted.description);
 }
 
-function renderPlainTextPreview(text: string, uiTheme: Theme, filePath?: string): string {
+function renderPlainTextPreview(text: string, uiTheme: Theme, _filePath?: string): string {
 	const previewLines = sanitizeText(text).split("\n");
 	let preview = "\n\n";
 	for (const line of previewLines.slice(0, CALL_TEXT_PREVIEW_LINES)) {
-		preview += `${uiTheme.fg("toolOutput", truncateToWidth(replaceTabs(line, filePath), CALL_TEXT_PREVIEW_WIDTH))}\n`;
+		preview += `${uiTheme.fg("toolOutput", truncateToWidth(replaceTabs(line), CALL_TEXT_PREVIEW_WIDTH))}\n`;
 	}
 	if (previewLines.length > CALL_TEXT_PREVIEW_LINES) {
 		preview += uiTheme.fg("dim", `… ${previewLines.length - CALL_TEXT_PREVIEW_LINES} more lines`);
@@ -395,7 +395,7 @@ function formatMultiFileStreamingDiff(
 		if (!preview.diff && !preview.error) continue;
 		const header = uiTheme.fg("dim", `\n\n── ${shortenPath(preview.path)} ──`);
 		if (preview.error) {
-			parts.push(`${header}\n${uiTheme.fg("error", replaceTabs(preview.error, preview.path))}`);
+			parts.push(`${header}\n${uiTheme.fg("error", replaceTabs(preview.error))}`);
 			continue;
 		}
 		if (preview.diff) {
@@ -637,7 +637,7 @@ export const editToolRenderer = {
 				callPreviewCaches,
 			);
 			if (applyPatchSummary?.error) {
-				body += `\n${uiTheme.fg("error", truncateToWidth(replaceTabs(applyPatchSummary.error, rawPath), Math.max(1, width - 2)))}`;
+				body += `\n${uiTheme.fg("error", truncateToWidth(replaceTabs(applyPatchSummary.error), Math.max(1, width - 2)))}`;
 			}
 			const bodyLines = body ? body.split("\n") : [];
 			while (bodyLines.length > 0 && bodyLines[0].trim() === "") bodyLines.shift();
@@ -733,11 +733,11 @@ function renderSingleFileResult(
 
 		let body = "";
 		if (isError) {
-			if (errorText) body = uiTheme.fg("error", replaceTabs(errorText, rawPath));
+			if (errorText) body = uiTheme.fg("error", replaceTabs(errorText));
 		} else if (details?.diff) {
 			body = renderDiffSection(details.diff, rawPath, expanded, uiTheme, renderDiffFn, diffSectionCache);
 		} else if (editDiffPreview) {
-			if ("error" in editDiffPreview) body = uiTheme.fg("error", replaceTabs(editDiffPreview.error, rawPath));
+			if ("error" in editDiffPreview) body = uiTheme.fg("error", replaceTabs(editDiffPreview.error));
 			else if (editDiffPreview.diff)
 				body = renderDiffSection(editDiffPreview.diff, rawPath, expanded, uiTheme, renderDiffFn, diffSectionCache);
 		}

package/src/eval/js/tool-bridge.ts

@@ -1,4 +1,5 @@
 import type { AgentTool, AgentToolResult } from "@oh-my-pi/pi-agent-core";
+import { INTENT_FIELD } from "@oh-my-pi/pi-wire";
 import type { ToolSession } from "../../tools";
 import { ToolError } from "../../tools/tool-errors";
 import { EVAL_AGENT_BRIDGE_NAME, runEvalAgent } from "../agent-bridge";
@@ -48,8 +49,8 @@ function normalizeArgs(args: unknown): unknown {
 		return args;
 	}
 	const record = { ...(args as Record<string, unknown>) };
-	if (record._i === undefined) {
-		record._i = "js prelude";
+	if (record[INTENT_FIELD] === undefined) {
+		record[INTENT_FIELD] = "js prelude";
 	}
 	return record;
 }

package/src/eval/py/prelude.py

@@ -5,6 +5,7 @@ if "__omp_prelude_loaded__" not in globals():
     from pathlib import Path
     import os, json, math, re
     from urllib.parse import unquote
+    INTENT_FIELD = "_i"
 
     # __omp_display is injected by runner.py before the prelude executes; it
     # mirrors IPython's display() semantics with the same MIME bundle output.
@@ -479,8 +480,8 @@ if "__omp_prelude_loaded__" not in globals():
                     f"tool.{self._name}(...) expects a dict of arguments (got {type(args).__name__})"
                 )
             merged.update(kwargs)
-            if "_i" not in merged:
-                merged["_i"] = "py prelude"
+            if INTENT_FIELD not in merged:
+                merged[INTENT_FIELD] = "py prelude"
             return _bridge_call(self._name, merged)
 
     class _ToolProxy: