@oh-my-pi/pi-coding-agent 3.13.1337 → 3.15.0

package/src/core/tools/exa/render.ts

@@ -11,17 +11,11 @@ import type { RenderResultOptions } from "../../custom-tools/types";
 import { logger } from "../../logger";
 import type { ExaRenderDetails } from "./types";
 
-// Tree formatting constants
-const TREE_MID = "├─";
-const TREE_END = "└─";
-const TREE_PIPE = "│";
-const TREE_SPACE = " ";
-const TREE_HOOK = "⎿";
-
 /** Truncate text to max length with ellipsis */
-function truncate(text: string, maxLen: number): string {
+function truncate(text: string, maxLen: number, ellipsis: string): string {
 	if (text.length <= maxLen) return text;
-	return `${text.slice(0, maxLen - 1)}…`;
+	const sliceLen = Math.max(0, maxLen - ellipsis.length);
+	return `${text.slice(0, sliceLen)}${ellipsis}`;
 }
 
 /** Extract domain from URL */
@@ -35,16 +29,16 @@ function getDomain(url: string): string {
 }
 
 /** Get first N lines of text as preview */
-function getPreviewLines(text: string, maxLines: number, maxLineLen: number): string[] {
+function getPreviewLines(text: string, maxLines: number, maxLineLen: number, ellipsis: string): string[] {
 	const lines = text.split("\n").filter((l) => l.trim());
-	return lines.slice(0, maxLines).map((l) => truncate(l.trim(), maxLineLen));
+	return lines.slice(0, maxLines).map((l) => truncate(l.trim(), maxLineLen, ellipsis));
 }
 
 /** Render Exa result with tree-based layout */
 export function renderExaResult(
 	result: { content: Array<{ type: string; text?: string }>; details?: ExaRenderDetails },
 	options: RenderResultOptions,
-	theme: Theme,
+	uiTheme: Theme,
 ): Component {
 	const { expanded } = options;
 	const details = result.details;
@@ -52,7 +46,7 @@ export function renderExaResult(
 	// Handle error case
 	if (details?.error) {
 		logger.error("Exa render error", { error: details.error, toolName: details.toolName });
-		return new Text(theme.fg("error", `Error: ${details.error}`), 0, 0);
+		return new Text(uiTheme.fg("error", `Error: ${details.error}`), 0, 0);
 	}
 
 	const response = details?.response;
@@ -60,15 +54,15 @@ export function renderExaResult(
 		// Non-search response: show raw result
 		if (details?.raw) {
 			const rawText = typeof details.raw === "string" ? details.raw : JSON.stringify(details.raw, null, 2);
-			const preview = expanded ? rawText : truncate(rawText, 200);
+			const preview = expanded ? rawText : truncate(rawText, 200, uiTheme.format.ellipsis);
 			const toolLabel = details?.toolName ?? "Exa";
 			return new Text(
-				`${theme.fg("success", "●")} ${theme.fg("toolTitle", toolLabel)}\n ${theme.fg("dim", TREE_PIPE)}  ${preview}`,
+				`${uiTheme.fg("success", uiTheme.format.bullet)} ${uiTheme.fg("toolTitle", toolLabel)}\n ${uiTheme.fg("dim", uiTheme.tree.vertical)}  ${preview}`,
 				0,
 				0,
 			);
 		}
-		return new Text(theme.fg("error", "No response data"), 0, 0);
+		return new Text(uiTheme.fg("error", "No response data"), 0, 0);
 	}
 
 	const results = response.results ?? [];
@@ -76,21 +70,22 @@ export function renderExaResult(
 	const cost = response.costDollars?.total;
 	const time = response.searchTime;
 
-	// Build header: ● Exa Search · N results · $X.XX · Xs
-	const icon = resultCount > 0 ? theme.fg("success", "●") : theme.fg("warning", "●");
-	const expandHint = expanded ? "" : theme.fg("dim", " (Ctrl+O to expand)");
+	// Build header: Exa Search · N results · $X.XX · Xs
+	const icon =
+		resultCount > 0 ? uiTheme.fg("success", uiTheme.format.bullet) : uiTheme.fg("warning", uiTheme.format.bullet);
+	const expandHint = expanded ? "" : uiTheme.fg("dim", " (Ctrl+O for full results)");
 	const toolLabel = details?.toolName ?? "Exa Search";
 
-	let headerParts = `${icon} ${theme.fg("toolTitle", toolLabel)} · ${theme.fg(
+	let headerParts = `${icon} ${uiTheme.fg("toolTitle", toolLabel)}${uiTheme.sep.dot}${uiTheme.fg(
 		"dim",
 		`${resultCount} result${resultCount !== 1 ? "s" : ""}`,
 	)}`;
 
 	if (cost !== undefined) {
-		headerParts += ` · ${theme.fg("muted", `$${cost.toFixed(4)}`)}`;
+		headerParts += `${uiTheme.sep.dot}${uiTheme.fg("muted", `$${cost.toFixed(4)}`)}`;
 	}
 	if (time !== undefined) {
-		headerParts += ` · ${theme.fg("muted", `${time.toFixed(2)}s`)}`;
+		headerParts += `${uiTheme.sep.dot}${uiTheme.fg("muted", `${time.toFixed(2)}s`)}`;
 	}
 
 	let text = headerParts + expandHint;
@@ -100,19 +95,22 @@ export function renderExaResult(
 		if (resultCount > 0) {
 			const first = results[0];
 			const previewText = first.text ?? first.title ?? "";
-			const previewLines = getPreviewLines(previewText, 3, 100);
+			const previewLines = getPreviewLines(previewText, 3, 100, uiTheme.format.ellipsis);
 
 			for (const line of previewLines) {
-				text += `\n ${theme.fg("dim", TREE_PIPE)}  ${theme.fg("dim", line)}`;
+				text += `\n ${uiTheme.fg("dim", uiTheme.tree.vertical)}  ${uiTheme.fg("dim", line)}`;
 			}
 
 			const totalLines = previewText.split("\n").filter((l) => l.trim()).length;
 			if (totalLines > 3) {
-				text += `\n ${theme.fg("dim", TREE_PIPE)}  ${theme.fg("muted", `… ${totalLines - 3} more lines`)}`;
+				text += `\n ${uiTheme.fg("dim", uiTheme.tree.vertical)}  ${uiTheme.fg(
+					"muted",
+					`${uiTheme.format.ellipsis} ${totalLines - 3} more lines`,
+				)}`;
 			}
 
 			if (resultCount > 1) {
-				text += `\n ${theme.fg("dim", TREE_END)} ${theme.fg(
+				text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg(
 					"muted",
 					`${resultCount - 1} more result${resultCount !== 2 ? "s" : ""}`,
 				)}`;
@@ -121,38 +119,35 @@ export function renderExaResult(
 	} else {
 		// Expanded view: full results tree
 		if (resultCount > 0) {
-			text += `\n ${theme.fg("dim", TREE_PIPE)}`;
-			text += `\n ${theme.fg("dim", TREE_END)} ${theme.fg("accent", "Results")}`;
+			text += `\n ${uiTheme.fg("dim", uiTheme.tree.vertical)}`;
+			text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg("accent", "Results")}`;
 
 			for (let i = 0; i < results.length; i++) {
 				const res = results[i];
 				const isLast = i === results.length - 1;
-				const branch = isLast ? TREE_END : TREE_MID;
-				const cont = isLast ? TREE_SPACE : TREE_PIPE;
+				const branch = isLast ? uiTheme.tree.last : uiTheme.tree.branch;
+				const cont = isLast ? " " : uiTheme.tree.vertical;
 
 				// Title + domain
-				const title = truncate(res.title ?? "Untitled", 60);
+				const title = truncate(res.title ?? "Untitled", 60, uiTheme.format.ellipsis);
 				const domain = res.url ? getDomain(res.url) : "";
-				const domainPart = domain ? theme.fg("dim", ` (${domain})`) : "";
+				const domainPart = domain ? uiTheme.fg("dim", ` (${domain})`) : "";
 
-				text += `\n ${theme.fg("dim", TREE_SPACE)} ${theme.fg("dim", branch)} ${theme.fg(
-					"accent",
-					title,
-				)}${domainPart}`;
+				text += `\n ${uiTheme.fg("dim", " ")} ${uiTheme.fg("dim", branch)} ${uiTheme.fg("accent", title)}${domainPart}`;
 
 				// URL
 				if (res.url) {
-					text += `\n ${theme.fg("dim", cont)}   ${theme.fg("dim", TREE_HOOK)} ${theme.fg("mdLinkUrl", res.url)}`;
+					text += `\n ${uiTheme.fg("dim", cont)}   ${uiTheme.fg("dim", uiTheme.tree.hook)} ${uiTheme.fg("mdLinkUrl", res.url)}`;
 				}
 
 				// Author
 				if (res.author) {
-					text += `\n ${theme.fg("dim", cont)}   ${theme.fg("muted", `Author: ${res.author}`)}`;
+					text += `\n ${uiTheme.fg("dim", cont)}   ${uiTheme.fg("muted", `Author: ${res.author}`)}`;
 				}
 
 				// Published date
 				if (res.publishedDate) {
-					text += `\n ${theme.fg("dim", cont)}   ${theme.fg("muted", `Published: ${res.publishedDate}`)}`;
+					text += `\n ${uiTheme.fg("dim", cont)}   ${uiTheme.fg("muted", `Published: ${res.publishedDate}`)}`;
 				}
 
 				// Text content
@@ -160,22 +155,31 @@ export function renderExaResult(
 					const textLines = res.text.split("\n").filter((l) => l.trim());
 					const displayLines = textLines.slice(0, 5); // Show first 5 lines
 					for (const line of displayLines) {
-						text += `\n ${theme.fg("dim", cont)}   ${truncate(line.trim(), 90)}`;
+						text += `\n ${uiTheme.fg("dim", cont)}   ${truncate(line.trim(), 90, uiTheme.format.ellipsis)}`;
 					}
 					if (textLines.length > 5) {
-						text += `\n ${theme.fg("dim", cont)}   ${theme.fg("muted", `… ${textLines.length - 5} more lines`)}`;
+						text += `\n ${uiTheme.fg("dim", cont)}   ${uiTheme.fg(
+							"muted",
+							`${uiTheme.format.ellipsis} ${textLines.length - 5} more lines`,
+						)}`;
 					}
 				}
 
 				// Highlights
 				if (res.highlights?.length) {
-					text += `\n ${theme.fg("dim", cont)}   ${theme.fg("accent", "Highlights:")}`;
+					text += `\n ${uiTheme.fg("dim", cont)}   ${uiTheme.fg("accent", "Highlights:")}`;
 					for (let j = 0; j < Math.min(res.highlights.length, 3); j++) {
 						const h = res.highlights[j];
-						text += `\n ${theme.fg("dim", cont)}   ${theme.fg("muted", `• ${truncate(h, 80)}`)}`;
+						text += `\n ${uiTheme.fg("dim", cont)}   ${uiTheme.fg(
+							"muted",
+							`${uiTheme.format.bullet} ${truncate(h, 80, uiTheme.format.ellipsis)}`,
+						)}`;
 					}
 					if (res.highlights.length > 3) {
-						text += `\n ${theme.fg("dim", cont)}   ${theme.fg("muted", `… ${res.highlights.length - 3} more`)}`;
+						text += `\n ${uiTheme.fg("dim", cont)}   ${uiTheme.fg(
+							"muted",
+							`${uiTheme.format.ellipsis} ${res.highlights.length - 3} more`,
+						)}`;
 					}
 				}
 			}
@@ -186,11 +190,11 @@ export function renderExaResult(
 }
 
 /** Render Exa call (query/args preview) */
-export function renderExaCall(args: Record<string, unknown>, toolName: string, theme: Theme): Component {
-	const query = typeof args.query === "string" ? truncate(args.query, 80) : "";
+export function renderExaCall(args: Record<string, unknown>, toolName: string, uiTheme: Theme): Component {
+	const query = typeof args.query === "string" ? truncate(args.query, 80, uiTheme.format.ellipsis) : "";
 	const numResults = typeof args.num_results === "number" ? args.num_results : undefined;
-	const detail = numResults ? theme.fg("dim", ` (${numResults} results)`) : "";
+	const detail = numResults ? uiTheme.fg("dim", ` (${numResults} results)`) : "";
 
-	const text = `${theme.fg("toolTitle", toolName)} ${theme.fg("muted", query)}${detail}`;
+	const text = `${uiTheme.fg("toolTitle", toolName)} ${uiTheme.fg("muted", query)}${detail}`;
 	return new Text(text, 0, 0);
 }

package/src/core/tools/find.ts

@@ -3,6 +3,7 @@ import path from "node:path";
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
 import { Type } from "@sinclair/typebox";
 import { globSync } from "glob";
+import findDescription from "../../prompts/tools/find.md" with { type: "text" };
 import { ensureTool } from "../../utils/tools-manager";
 import { untilAborted } from "../utils";
 import { resolveToCwd } from "./path-utils";
@@ -32,6 +33,7 @@ export interface FindToolDetails {
 	truncation?: TruncationResult;
 	resultLimitReached?: number;
 	// Fields for TUI rendering
+	scopePath?: string;
 	fileCount?: number;
 	files?: string[];
 	truncated?: boolean;
@@ -42,12 +44,7 @@ export function createFindTool(cwd: string): AgentTool<typeof findSchema> {
 	return {
 		name: "find",
 		label: "Find",
-		description: `- Fast file pattern matching tool that works with any codebase size
-- Supports glob patterns like "**/*.js" or "src/**/*.ts"
-- Returns matching file paths sorted by modification time
-- Use this tool when you need to find files by name patterns
-- When you are doing an open ended search that may require multiple rounds of globbing and grepping, use the Agent tool instead
-- You can call multiple tools in a single response. It is always better to speculatively perform multiple searches in parallel if they are potentially useful.`,
+		description: findDescription,
 		parameters: findSchema,
 		execute: async (
 			_toolCallId: string,
@@ -76,6 +73,10 @@ export function createFindTool(cwd: string): AgentTool<typeof findSchema> {
 				}
 
 				const searchPath = resolveToCwd(searchDir || ".", cwd);
+				const scopePath = (() => {
+					const relative = path.relative(cwd, searchPath).replace(/\\/g, "/");
+					return relative.length === 0 ? "." : relative;
+				})();
 				const effectiveLimit = limit ?? DEFAULT_LIMIT;
 				const effectiveType = type ?? "all";
 				const includeHidden = hidden ?? false;
@@ -148,7 +149,7 @@ export function createFindTool(cwd: string): AgentTool<typeof findSchema> {
 				if (!output) {
 					return {
 						content: [{ type: "text", text: "No files found matching pattern" }],
-						details: { fileCount: 0, files: [], truncated: false },
+						details: { scopePath, fileCount: 0, files: [], truncated: false },
 					};
 				}
 
@@ -205,8 +206,9 @@ export function createFindTool(cwd: string): AgentTool<typeof findSchema> {
 
 				let resultOutput = truncation.content;
 				const details: FindToolDetails = {
+					scopePath,
 					fileCount: relativized.length,
-					files: relativized.slice(0, 50),
+					files: relativized,
 					truncated: resultLimitReached || truncation.truncated,
 				};
 

package/src/core/tools/grep.ts

@@ -3,6 +3,7 @@ import nodePath from "node:path";
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
 import { Type } from "@sinclair/typebox";
 import type { Subprocess } from "bun";
+import grepDescription from "../../prompts/tools/grep.md" with { type: "text" };
 import { ensureTool } from "../../utils/tools-manager";
 import { resolveToCwd } from "./path-utils";
 import {
@@ -50,11 +51,14 @@ const DEFAULT_LIMIT = 100;
 export interface GrepToolDetails {
 	truncation?: TruncationResult;
 	matchLimitReached?: number;
+	headLimitReached?: number;
 	linesTruncated?: boolean;
 	// Fields for TUI rendering
+	scopePath?: string;
 	matchCount?: number;
 	fileCount?: number;
 	files?: string[];
+	fileMatches?: Array<{ path: string; count: number }>;
 	mode?: "content" | "files_with_matches" | "count";
 	truncated?: boolean;
 	error?: string;
@@ -64,17 +68,7 @@ export function createGrepTool(cwd: string): AgentTool<typeof grepSchema> {
 	return {
 		name: "grep",
 		label: "Grep",
-		description: `A powerful search tool built on ripgrep
-
-Usage:
-- ALWAYS use grep for search tasks. NEVER invoke \`grep\` or \`rg\` as a bash command. The grep tool has been optimized for correct permissions and access.
-- Searches recursively by default - no need for -r flag
-- Supports full regex syntax (e.g., "log.*Error", "function\\s+\\w+")
-- Filter files with glob parameter (e.g., "*.ts", "**/*.spec.ts") or type parameter (e.g., "ts", "py", "rust") - equivalent to grep's --include
-- Output modes: "content" shows matching lines, "files_with_matches" shows only file paths (default), "count" shows match counts
-- Pagination: Use headLimit to limit results (like \`| head -N\`), offset to skip first N results
-- Pattern syntax: Uses ripgrep (not grep) - literal braces need escaping (use \`interface\\{\\}\` to find \`interface{}\` in Go code)
-- Multiline matching: By default patterns match within single lines only. For cross-line patterns like \`struct \\{[\\s\\S]*?field\`, use \`multiline: true\``,
+		description: grepDescription,
 		parameters: grepSchema,
 		execute: async (
 			_toolCallId: string,
@@ -119,6 +113,10 @@ Usage:
 			}
 
 			const searchPath = resolveToCwd(searchDir || ".", cwd);
+			const scopePath = (() => {
+				const relative = nodePath.relative(cwd, searchPath).replace(/\\/g, "/");
+				return relative.length === 0 ? "." : relative;
+			})();
 			let searchStat: Stats;
 			try {
 				searchStat = statSync(searchPath);
@@ -210,6 +208,7 @@ Usage:
 			const outputLines: string[] = [];
 			const files = new Set<string>();
 			const fileList: string[] = [];
+			const fileMatchCounts = new Map<string, number>();
 
 			const recordFile = (filePath: string) => {
 				const relative = formatPath(filePath);
@@ -219,6 +218,11 @@ Usage:
 				}
 			};
 
+			const recordFileMatch = (filePath: string) => {
+				const relative = formatPath(filePath);
+				fileMatchCounts.set(relative, (fileMatchCounts.get(relative) ?? 0) + 1);
+			};
+
 			const stopChild = (dueToLimit: boolean = false) => {
 				killedDueToLimit = dueToLimit;
 				child.kill();
@@ -281,6 +285,7 @@ Usage:
 					return {
 						content: [{ type: "text", text: "No matches found" }],
 						details: {
+							scopePath,
 							matchCount: 0,
 							fileCount: 0,
 							files: [],
@@ -303,6 +308,7 @@ Usage:
 				let fileCount = 0;
 				const simpleFiles = new Set<string>();
 				const simpleFileList: string[] = [];
+				const simpleFileMatchCounts = new Map<string, number>();
 
 				const recordSimpleFile = (filePath: string) => {
 					const relative = formatPath(filePath);
@@ -312,6 +318,11 @@ Usage:
 					}
 				};
 
+				const recordSimpleFileMatch = (filePath: string, count: number) => {
+					const relative = formatPath(filePath);
+					simpleFileMatchCounts.set(relative, count);
+				};
+
 				if (effectiveOutputMode === "files_with_matches") {
 					for (const line of lines) {
 						recordSimpleFile(line);
@@ -327,12 +338,13 @@ Usage:
 						recordSimpleFile(filePart);
 						if (!Number.isNaN(count)) {
 							simpleMatchCount += count;
+							recordSimpleFileMatch(filePart, count);
 						}
 					}
 					fileCount = simpleFiles.size;
 				}
 
-				const truncated = hasHeadLimit && processedLines.length < lines.length;
+				const truncatedByHeadLimit = hasHeadLimit && processedLines.length < lines.length;
 
 				// For count mode, format as "path:count"
 				if (effectiveOutputMode === "count") {
@@ -346,11 +358,17 @@ Usage:
 					return {
 						content: [{ type: "text", text: output }],
 						details: {
+							scopePath,
 							matchCount: simpleMatchCount,
 							fileCount,
-							files: simpleFileList.slice(0, 50),
+							files: simpleFileList,
+							fileMatches: simpleFileList.map((path) => ({
+								path,
+								count: simpleFileMatchCounts.get(path) ?? 0,
+							})),
 							mode: effectiveOutputMode,
-							truncated,
+							truncated: truncatedByHeadLimit,
+							headLimitReached: truncatedByHeadLimit ? headLimit : undefined,
 						},
 					};
 				}
@@ -361,11 +379,13 @@ Usage:
 				return {
 					content: [{ type: "text", text: output }],
 					details: {
+						scopePath,
 						matchCount: simpleMatchCount,
 						fileCount,
-						files: simpleFileList.slice(0, 50),
+						files: simpleFileList,
 						mode: effectiveOutputMode,
-						truncated,
+						truncated: truncatedByHeadLimit,
+						headLimitReached: truncatedByHeadLimit ? headLimit : undefined,
 					},
 				};
 			}
@@ -421,6 +441,7 @@ Usage:
 
 					if (filePath && typeof lineNumber === "number") {
 						recordFile(filePath);
+						recordFileMatch(filePath);
 						outputLines.push(...formatBlock(filePath, lineNumber));
 					}
 
@@ -488,6 +509,7 @@ Usage:
 				return {
 					content: [{ type: "text", text: "No matches found" }],
 					details: {
+						scopePath,
 						matchCount: 0,
 						fileCount: 0,
 						files: [],
@@ -513,11 +535,17 @@ Usage:
 			let output = truncation.content;
 			const truncatedByHeadLimit = hasHeadLimit && processedLines.length < outputLines.length;
 			const details: GrepToolDetails = {
+				scopePath,
 				matchCount,
 				fileCount: files.size,
-				files: fileList.slice(0, 50),
+				files: fileList,
+				fileMatches: fileList.map((path) => ({
+					path,
+					count: fileMatchCounts.get(path) ?? 0,
+				})),
 				mode: effectiveOutputMode,
 				truncated: matchLimitReached || truncation.truncated || truncatedByHeadLimit,
+				headLimitReached: truncatedByHeadLimit ? headLimit : undefined,
 			};
 
 			// Build notices

package/src/core/tools/ls.ts

@@ -14,7 +14,11 @@ const lsSchema = Type.Object({
 const DEFAULT_LIMIT = 500;
 
 export interface LsToolDetails {
+	entries?: string[];
+	dirCount?: number;
+	fileCount?: number;
 	truncation?: TruncationResult;
+	truncationReasons?: Array<"entryLimit" | "byteLimit">;
 	entryLimitReached?: number;
 }
 
@@ -58,6 +62,8 @@ export function createLsTool(cwd: string): AgentTool<typeof lsSchema> {
 				// Format entries with directory indicators
 				const results: string[] = [];
 				let entryLimitReached = false;
+				let dirCount = 0;
+				let fileCount = 0;
 
 				for (const entry of entries) {
 					if (results.length >= effectiveLimit) {
@@ -72,6 +78,9 @@ export function createLsTool(cwd: string): AgentTool<typeof lsSchema> {
 						const entryStat = statSync(fullPath);
 						if (entryStat.isDirectory()) {
 							suffix = "/";
+							dirCount += 1;
+						} else {
+							fileCount += 1;
 						}
 					} catch {
 						// Skip entries we can't stat
@@ -90,7 +99,12 @@ export function createLsTool(cwd: string): AgentTool<typeof lsSchema> {
 				const truncation = truncateHead(rawOutput, { maxLines: Number.MAX_SAFE_INTEGER });
 
 				let output = truncation.content;
-				const details: LsToolDetails = {};
+				const details: LsToolDetails = {
+					entries: results,
+					dirCount,
+					fileCount,
+				};
+				const truncationReasons: Array<"entryLimit" | "byteLimit"> = [];
 
 				// Build notices
 				const notices: string[] = [];
@@ -98,11 +112,17 @@ export function createLsTool(cwd: string): AgentTool<typeof lsSchema> {
 				if (entryLimitReached) {
 					notices.push(`${effectiveLimit} entries limit reached. Use limit=${effectiveLimit * 2} for more`);
 					details.entryLimitReached = effectiveLimit;
+					truncationReasons.push("entryLimit");
 				}
 
 				if (truncation.truncated) {
 					notices.push(`${formatSize(DEFAULT_MAX_BYTES)} limit reached`);
 					details.truncation = truncation;
+					truncationReasons.push("byteLimit");
+				}
+
+				if (truncationReasons.length > 0) {
+					details.truncationReasons = truncationReasons;
 				}
 
 				if (notices.length > 0) {
@@ -111,7 +131,7 @@ export function createLsTool(cwd: string): AgentTool<typeof lsSchema> {
 
 				return {
 					content: [{ type: "text", text: output }],
-					details: Object.keys(details).length > 0 ? details : undefined,
+					details,
 				};
 			});
 		},