@oh-my-pi/pi-coding-agent 8.12.1 → 8.12.2

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## [Unreleased]
 
+## [8.12.2] - 2026-01-28
+
+### Changed
+- Replaced ripgrep-based file listing with fs.glob for project scans and find/read tooling
+
 ## [8.11.14] - 2026-01-28
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "8.12.1",
+	"version": "8.12.2",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -83,11 +83,11 @@
 		"test": "bun test"
 	},
 	"dependencies": {
-		"@oh-my-pi/omp-stats": "8.12.1",
-		"@oh-my-pi/pi-agent-core": "8.12.1",
-		"@oh-my-pi/pi-ai": "8.12.1",
-		"@oh-my-pi/pi-tui": "8.12.1",
-		"@oh-my-pi/pi-utils": "8.12.1",
+		"@oh-my-pi/omp-stats": "8.12.2",
+		"@oh-my-pi/pi-agent-core": "8.12.2",
+		"@oh-my-pi/pi-ai": "8.12.2",
+		"@oh-my-pi/pi-tui": "8.12.2",
+		"@oh-my-pi/pi-utils": "8.12.2",
 		"@openai/agents": "^0.4.4",
 		"@sinclair/typebox": "^0.34.48",
 		"ajv": "^8.17.1",

package/src/extensibility/extensions/types.ts

@@ -47,6 +47,8 @@ export interface ExtensionUIDialogOptions {
 	timeout?: number;
 	/** Initial cursor position for select dialogs (0-indexed) */
 	initialIndex?: number;
+	/** Render an outlined list for select dialogs */
+	outline?: boolean;
 }
 
 /**

package/src/modes/components/hook-selector.ts

@@ -2,7 +2,7 @@
  * Generic selector component for hooks.
  * Displays a list of string options with keyboard navigation.
  */
-import { Container, matchesKey, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
+import { Container, matchesKey, Spacer, Text, type TUI, visibleWidth } from "@oh-my-pi/pi-tui";
 import { theme } from "../../modes/theme/theme";
 import { CountdownTimer } from "./countdown-timer";
 import { DynamicBorder } from "./dynamic-border";
@@ -11,12 +11,36 @@ export interface HookSelectorOptions {
 	tui?: TUI;
 	timeout?: number;
 	initialIndex?: number;
+	outline?: boolean;
+	maxVisible?: number;
+}
+
+class OutlinedList extends Container {
+	private lines: string[] = [];
+
+	setLines(lines: string[]): void {
+		this.lines = lines;
+		this.invalidate();
+	}
+
+	render(width: number): string[] {
+		const borderColor = (text: string) => theme.fg("border", text);
+		const horizontal = borderColor(theme.boxSharp.horizontal.repeat(Math.max(1, width)));
+		const innerWidth = Math.max(1, width - 2);
+		const content = this.lines.map(line => {
+			const pad = Math.max(0, innerWidth - visibleWidth(line));
+			return `${borderColor(theme.boxSharp.vertical)}${line}${" ".repeat(pad)}${borderColor(theme.boxSharp.vertical)}`;
+		});
+		return [horizontal, ...content, horizontal];
+	}
 }
 
 export class HookSelectorComponent extends Container {
 	private options: string[];
 	private selectedIndex: number;
-	private listContainer: Container;
+	private maxVisible: number;
+	private listContainer: Container | undefined;
+	private outlinedList: OutlinedList | undefined;
 	private onSelectCallback: (option: string) => void;
 	private onCancelCallback: () => void;
 	private titleText: Text;
@@ -34,6 +58,7 @@ export class HookSelectorComponent extends Container {
 
 		this.options = options;
 		this.selectedIndex = Math.min(opts?.initialIndex ?? 0, options.length - 1);
+		this.maxVisible = Math.max(3, opts?.maxVisible ?? 12);
 		this.onSelectCallback = onSelect;
 		this.onCancelCallback = onCancel;
 		this.baseTitle = title;
@@ -62,8 +87,13 @@ export class HookSelectorComponent extends Container {
 			);
 		}
 
-		this.listContainer = new Container();
-		this.addChild(this.listContainer);
+		if (opts?.outline) {
+			this.outlinedList = new OutlinedList();
+			this.addChild(this.outlinedList);
+		} else {
+			this.listContainer = new Container();
+			this.addChild(this.listContainer);
+		}
 		this.addChild(new Spacer(1));
 		this.addChild(new Text(theme.fg("dim", "up/down navigate  enter select  esc cancel"), 1, 0));
 		this.addChild(new Spacer(1));
@@ -73,13 +103,31 @@ export class HookSelectorComponent extends Container {
 	}
 
 	private updateList(): void {
-		this.listContainer.clear();
-		for (let i = 0; i < this.options.length; i++) {
+		const lines: string[] = [];
+		const startIndex = Math.max(
+			0,
+			Math.min(this.selectedIndex - Math.floor(this.maxVisible / 2), this.options.length - this.maxVisible),
+		);
+		const endIndex = Math.min(startIndex + this.maxVisible, this.options.length);
+
+		for (let i = startIndex; i < endIndex; i++) {
 			const isSelected = i === this.selectedIndex;
 			const text = isSelected
 				? theme.fg("accent", `${theme.nav.cursor} `) + theme.fg("accent", this.options[i])
 				: `  ${theme.fg("text", this.options[i])}`;
-			this.listContainer.addChild(new Text(text, 1, 0));
+			lines.push(text);
+		}
+
+		if (startIndex > 0 || endIndex < this.options.length) {
+			lines.push(theme.fg("dim", `  (${this.selectedIndex + 1}/${this.options.length})`));
+		}
+		if (this.outlinedList) {
+			this.outlinedList.setLines(lines);
+			return;
+		}
+		this.listContainer?.clear();
+		for (const line of lines) {
+			this.listContainer?.addChild(new Text(line, 1, 0));
 		}
 	}
 

package/src/modes/controllers/extension-ui-controller.ts

@@ -1,4 +1,4 @@
-import type { Component, TUI } from "@oh-my-pi/pi-tui";
+import type { Component, OverlayHandle, TUI } from "@oh-my-pi/pi-tui";
 import { Spacer, Text } from "@oh-my-pi/pi-tui";
 import { logger } from "@oh-my-pi/pi-utils";
 import { KeybindingsManager } from "../../config/keybindings";
@@ -18,6 +18,17 @@ import type { InteractiveModeContext } from "../../modes/types";
 import { setTerminalTitle } from "../../utils/title-generator";
 
 export class ExtensionUiController {
+	private hookSelectorOverlay: OverlayHandle | undefined;
+	private hookInputOverlay: OverlayHandle | undefined;
+
+	private readonly dialogOverlayOptions = {
+		anchor: "bottom-center",
+		width: "80%",
+		minWidth: 40,
+		maxHeight: "70%",
+		margin: 1,
+	} as const;
+
 	constructor(private ctx: InteractiveModeContext) {}
 
 	/**
@@ -491,6 +502,9 @@ export class ExtensionUiController {
 		dialogOptions?: ExtensionUIDialogOptions,
 	): Promise<string | undefined> {
 		const { promise, resolve } = Promise.withResolvers<string | undefined>();
+		this.hookSelectorOverlay?.hide();
+		this.hookSelectorOverlay = undefined;
+		const maxVisible = Math.max(4, Math.min(15, this.ctx.ui.terminal.rows - 12));
 		this.ctx.hookSelector = new HookSelectorComponent(
 			title,
 			options,
@@ -502,13 +516,15 @@ export class ExtensionUiController {
 				this.hideHookSelector();
 				resolve(undefined);
 			},
-			{ initialIndex: dialogOptions?.initialIndex, timeout: dialogOptions?.timeout, tui: this.ctx.ui },
+			{
+				initialIndex: dialogOptions?.initialIndex,
+				timeout: dialogOptions?.timeout,
+				tui: this.ctx.ui,
+				outline: dialogOptions?.outline,
+				maxVisible,
+			},
 		);
-
-		this.ctx.editorContainer.clear();
-		this.ctx.editorContainer.addChild(this.ctx.hookSelector);
-		this.ctx.ui.setFocus(this.ctx.hookSelector);
-		this.ctx.ui.requestRender();
+		this.hookSelectorOverlay = this.ctx.ui.showOverlay(this.ctx.hookSelector, this.dialogOverlayOptions);
 		return promise;
 	}
 
@@ -516,8 +532,9 @@ export class ExtensionUiController {
 	 * Hide the hook selector.
 	 */
 	hideHookSelector(): void {
-		this.ctx.editorContainer.clear();
-		this.ctx.editorContainer.addChild(this.ctx.editor);
+		this.ctx.hookSelector?.dispose();
+		this.hookSelectorOverlay?.hide();
+		this.hookSelectorOverlay = undefined;
 		this.ctx.hookSelector = undefined;
 		this.ctx.ui.setFocus(this.ctx.editor);
 		this.ctx.ui.requestRender();
@@ -536,6 +553,8 @@ export class ExtensionUiController {
 	 */
 	showHookInput(title: string, placeholder?: string): Promise<string | undefined> {
 		const { promise, resolve } = Promise.withResolvers<string | undefined>();
+		this.hookInputOverlay?.hide();
+		this.hookInputOverlay = undefined;
 		this.ctx.hookInput = new HookInputComponent(
 			title,
 			placeholder,
@@ -548,11 +567,7 @@ export class ExtensionUiController {
 				resolve(undefined);
 			},
 		);
-
-		this.ctx.editorContainer.clear();
-		this.ctx.editorContainer.addChild(this.ctx.hookInput);
-		this.ctx.ui.setFocus(this.ctx.hookInput);
-		this.ctx.ui.requestRender();
+		this.hookInputOverlay = this.ctx.ui.showOverlay(this.ctx.hookInput, this.dialogOverlayOptions);
 		return promise;
 	}
 
@@ -560,8 +575,9 @@ export class ExtensionUiController {
 	 * Hide the hook input.
 	 */
 	hideHookInput(): void {
-		this.ctx.editorContainer.clear();
-		this.ctx.editorContainer.addChild(this.ctx.editor);
+		this.ctx.hookInput?.dispose();
+		this.hookInputOverlay?.hide();
+		this.hookInputOverlay = undefined;
 		this.ctx.hookInput = undefined;
 		this.ctx.ui.setFocus(this.ctx.editor);
 		this.ctx.ui.requestRender();

package/src/prompts/system/custom-system-prompt.md

@@ -85,4 +85,4 @@ Read `rule://<name>` when working in their domain.
 {{/if}}
 
 Current date and time: {{dateTime}}
-Current working directory: {{cwd}}
+Current working directory: {{cwd}}

package/src/system-prompt.ts

@@ -1,10 +1,10 @@
 /**
  * System prompt construction and project context loading
  */
-import type * as fsTypes from "node:fs";
 import * as fs from "node:fs/promises";
 import * as os from "node:os";
 import * as path from "node:path";
+import { globPaths } from "@oh-my-pi/pi-utils";
 import { $ } from "bun";
 import chalk from "chalk";
 import { contextFileCapability } from "./capability/context-file";
@@ -16,8 +16,6 @@ import { loadSkills, type Skill } from "./extensibility/skills";
 import customSystemPromptTemplate from "./prompts/system/custom-system-prompt.md" with { type: "text" };
 import systemPromptTemplate from "./prompts/system/system-prompt.md" with { type: "text" };
 import type { ToolName } from "./tools";
-import { runRg } from "./tools/grep";
-import { ensureTool } from "./utils/tools-manager";
 
 interface GitContext {
 	isRepo: boolean;
@@ -200,24 +198,20 @@ type ProjectTreeScan = {
 	truncatedDirs: Set<string>;
 };
 
-const RG_TIMEOUT_MS = 5000;
+const GLOB_TIMEOUT_MS = 5000;
 
 /**
- * Scan project tree using ripgrep to respect gitignore.
- * Returns null if ripgrep is unavailable.
+ * Scan project tree using fs.promises.glob with exclusion filters.
+ * Returns null if glob fails.
  */
-async function scanProjectTreeWithRg(root: string): Promise<ProjectTreeScan | null> {
-	const rgPath = await ensureTool("rg", { silent: true });
-	if (!rgPath) return null;
-
-	const args = ["--files", "--no-require-git", "--color=never", root];
-
-	let stdout: string;
+async function scanProjectTreeWithGlob(root: string): Promise<ProjectTreeScan | null> {
+	let entries: string[];
 	try {
-		const signal = AbortSignal.timeout(RG_TIMEOUT_MS);
-		const result = await runRg(rgPath, args, signal);
-		if (result.exitCode !== 0 && result.exitCode !== 1) return null;
-		stdout = result.stdout;
+		entries = await globPaths("**/*", {
+			cwd: root,
+			gitignore: true,
+			timeoutMs: GLOB_TIMEOUT_MS,
+		});
 	} catch {
 		return null;
 	}
@@ -227,19 +221,19 @@ async function scanProjectTreeWithRg(root: string): Promise<ProjectTreeScan | nu
 	const dirContents = new Map<string, Map<string, boolean>>();
 	dirContents.set(root, new Map());
 
-	for (const line of stdout.split("\n")) {
-		const filePath = line.trim();
+	for (const entry of entries) {
+		const filePath = entry.trim();
 		if (!filePath) continue;
-
+		const absolutePath = path.join(root, filePath);
 		// Check static ignores on path components
-		const relative = path.relative(root, filePath);
+		const relative = path.relative(root, absolutePath);
 		const parts = relative.split(path.sep);
 		if (parts.some(p => PROJECT_TREE_IGNORED.has(p))) continue;
 
 		// Add file to its parent directory
-		const parent = path.dirname(filePath);
+		const parent = path.dirname(absolutePath);
 		if (!dirContents.has(parent)) dirContents.set(parent, new Map());
-		dirContents.get(parent)!.set(filePath, false);
+		dirContents.get(parent)!.set(absolutePath, false);
 
 		// Add all intermediate directories
 		let dir = parent;
@@ -317,82 +311,10 @@ async function scanProjectTreeWithRg(root: string): Promise<ProjectTreeScan | nu
 	return { children, truncated, truncatedDirs };
 }
 
-/**
- * Fallback scan using readdir when ripgrep is unavailable.
- */
-async function scanProjectTreeFallback(root: string): Promise<ProjectTreeScan> {
-	const children = new Map<string, ProjectTreeEntry[]>();
-	let entryCount = 0;
-	let truncated = false;
-	const truncatedDirs = new Set<string>();
-
-	const queue: Array<{ dirPath: string; depth: number }> = [{ dirPath: root, depth: 0 }];
-	let cursor = 0;
-
-	while (cursor < queue.length && !truncated) {
-		const { dirPath, depth } = queue[cursor];
-		cursor += 1;
-		let entries: fsTypes.Dirent[];
-		try {
-			entries = await fs.readdir(dirPath, { withFileTypes: true });
-		} catch {
-			continue;
-		}
-
-		const filtered = entries.filter(entry => !PROJECT_TREE_IGNORED.has(entry.name));
-		const withStats = await Promise.all(
-			filtered.map(async entry => {
-				const entryPath = path.join(dirPath, entry.name);
-				try {
-					const stats = await fs.stat(entryPath);
-					return { entry, entryPath, mtimeMs: stats.mtimeMs };
-				} catch {
-					return { entry, entryPath, mtimeMs: 0 };
-				}
-			}),
-		);
-
-		withStats.sort((a, b) => {
-			if (a.mtimeMs !== b.mtimeMs) return b.mtimeMs - a.mtimeMs;
-			return a.entry.name.localeCompare(b.entry.name);
-		});
-
-		const perDirLimit = depth >= PROJECT_TREE_PER_DIR_DEPTH ? PROJECT_TREE_PER_DIR_LIMIT : null;
-		const limited = perDirLimit === null ? withStats : withStats.slice(0, perDirLimit);
-		const hasMoreEntries = perDirLimit !== null && withStats.length > perDirLimit;
-
-		const mapped: ProjectTreeEntry[] = [];
-		for (const entryWithStat of limited) {
-			if (entryCount >= PROJECT_TREE_LIMIT) {
-				truncated = true;
-				break;
-			}
-
-			mapped.push({
-				name: entryWithStat.entry.name,
-				isDirectory: entryWithStat.entry.isDirectory(),
-				path: entryWithStat.entryPath,
-			});
-			entryCount += 1;
-
-			if (entryWithStat.entry.isDirectory()) {
-				queue.push({ dirPath: entryWithStat.entryPath, depth: depth + 1 });
-			}
-		}
-
-		if (!truncated && hasMoreEntries) {
-			truncatedDirs.add(dirPath);
-		}
-		children.set(dirPath, mapped);
-	}
-
-	return { children, truncated, truncatedDirs };
-}
-
 async function scanProjectTree(root: string): Promise<ProjectTreeScan> {
-	const rgResult = await scanProjectTreeWithRg(root);
-	if (rgResult) return rgResult;
-	return scanProjectTreeFallback(root);
+	const globResult = await scanProjectTreeWithGlob(root);
+	if (globResult) return globResult;
+	return { children: new Map(), truncated: false, truncatedDirs: new Set() };
 }
 
 function renderProjectTree(scan: ProjectTreeScan, root: string): string {

package/src/tools/ask.ts

@@ -77,6 +77,7 @@ export interface AskToolDetails {
 
 const OTHER_OPTION = "Other (type your own)";
 const RECOMMENDED_SUFFIX = " (Recommended)";
+const ASK_TIMEOUT_MS = 30000;
 
 function getDoneOptionLabel(): string {
 	return `${theme.status.success} Done selecting`;
@@ -113,7 +114,7 @@ interface UIContext {
 	select(
 		prompt: string,
 		options: string[],
-		options_?: { initialIndex?: number; timeout?: number },
+		options_?: { initialIndex?: number; timeout?: number; outline?: boolean },
 	): Promise<string | undefined>;
 	input(prompt: string): Promise<string | undefined>;
 }
@@ -131,7 +132,7 @@ async function askSingleQuestion(
 
 	if (multi) {
 		const selected = new Set<string>();
-		let cursorIndex = 0;
+		let cursorIndex = Math.min(Math.max(recommended ?? 0, 0), optionLabels.length - 1);
 
 		while (true) {
 			const opts: string[] = [];
@@ -148,13 +149,22 @@ async function askSingleQuestion(
 			opts.push(OTHER_OPTION);
 
 			const prefix = selected.size > 0 ? `(${selected.size} selected) ` : "";
-			const choice = await ui.select(`${prefix}${question}`, opts, { initialIndex: cursorIndex });
+			const selectionStart = Date.now();
+			const choice = await ui.select(`${prefix}${question}`, opts, {
+				initialIndex: cursorIndex,
+				timeout: ASK_TIMEOUT_MS,
+				outline: true,
+			});
+			const elapsed = Date.now() - selectionStart;
+			const timedOut = elapsed >= ASK_TIMEOUT_MS;
 
 			if (choice === undefined || choice === doneLabel) break;
 
 			if (choice === OTHER_OPTION) {
-				const input = await ui.input("Enter your response:");
-				if (input) customInput = input;
+				if (!timedOut) {
+					const input = await ui.input("Enter your response:");
+					if (input) customInput = input;
+				}
 				break;
 			}
 
@@ -179,13 +189,18 @@ async function askSingleQuestion(
 					selected.add(opt);
 				}
 			}
+
+			if (timedOut) {
+				break;
+			}
 		}
 		selectedOptions = Array.from(selected);
 	} else {
 		const displayLabels = addRecommendedSuffix(optionLabels, recommended);
 		const choice = await ui.select(question, [...displayLabels, OTHER_OPTION], {
-			timeout: 30000,
+			timeout: ASK_TIMEOUT_MS,
 			initialIndex: recommended,
+			outline: true,
 		});
 		if (choice === OTHER_OPTION) {
 			const input = await ui.input("Enter your response:");

package/src/tools/find.ts

@@ -3,7 +3,7 @@ import * as path from "node:path";
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Text } from "@oh-my-pi/pi-tui";
-import { isEnoent, untilAborted } from "@oh-my-pi/pi-utils";
+import { globPaths, isEnoent, untilAborted } from "@oh-my-pi/pi-utils";
 import type { Static } from "@sinclair/typebox";
 import { Type } from "@sinclair/typebox";
 import { renderPromptTemplate } from "../config/prompt-templates";
@@ -11,14 +11,12 @@ import type { RenderResultOptions } from "../extensibility/custom-tools/types";
 import type { Theme } from "../modes/theme/theme";
 import findDescription from "../prompts/tools/find.md" with { type: "text" };
 import { renderFileList, renderStatusLine, renderTreeList } from "../tui";
-import { ensureTool } from "../utils/tools-manager";
 import type { ToolSession } from ".";
-import { runRg } from "./grep";
 import { applyListLimit } from "./list-limit";
 import type { OutputMeta } from "./output-meta";
 import { resolveToCwd } from "./path-utils";
 import { formatCount, formatEmptyMessage, formatErrorMessage, PREVIEW_LIMITS } from "./render-utils";
-import { ToolError, throwIfAborted } from "./tool-errors";
+import { ToolAbortError, ToolError, throwIfAborted } from "./tool-errors";
 import { toolResult } from "./tool-result";
 import { type TruncationResult, truncateHead } from "./truncate";
 
@@ -30,7 +28,7 @@ const findSchema = Type.Object({
 });
 
 const DEFAULT_LIMIT = 1000;
-const RG_TIMEOUT_MS = 5000;
+const GLOB_TIMEOUT_MS = 5000;
 
 export interface FindToolDetails {
 	truncation?: TruncationResult;
@@ -80,7 +78,7 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 		params: Static<typeof findSchema>,
 		signal?: AbortSignal,
 		_onUpdate?: AgentToolUpdateCallback<FindToolDetails>,
-		context?: AgentToolContext,
+		_context?: AgentToolContext,
 	): Promise<AgentToolResult<FindToolDetails>> {
 		const { pattern, path: searchDir, limit, hidden } = params;
 
@@ -107,7 +105,6 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 			}
 			const includeHidden = hidden ?? true;
 			const globPattern = normalizedPattern.replace(/\\/g, "/");
-			const globMatcher = new Bun.Glob(globPattern);
 
 			// If custom operations provided with glob, use that instead of fd
 			if (this.customOps?.glob) {
@@ -171,44 +168,30 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 				throw new ToolError(`Path is not a directory: ${searchPath}`);
 			}
 
-			// Default: use rg
-			const rgPath = await ensureTool("rg", {
-				silent: true,
-				notify: message => context?.ui?.notify(message, "info"),
-			});
-			if (!rgPath) {
-				throw new ToolError("rg is not available and could not be downloaded");
+			let lines: string[];
+			try {
+				lines = await globPaths(globPattern, {
+					cwd: searchPath,
+					gitignore: true,
+					dot: includeHidden,
+					signal,
+					timeoutMs: GLOB_TIMEOUT_MS,
+				});
+			} catch (error) {
+				if (error instanceof Error && error.name === "AbortError") {
+					throw new ToolAbortError();
+				}
+				if (error instanceof Error && error.name === "TimeoutError") {
+					const timeoutSeconds = Math.max(1, Math.round(GLOB_TIMEOUT_MS / 1000));
+					throw new ToolError(`glob timed out after ${timeoutSeconds}s`);
+				}
+				throw error;
 			}
 
-			const args = [
-				"--files",
-				...(includeHidden ? ["--hidden"] : []),
-				"--no-require-git",
-				"--color=never",
-				"--glob",
-				"!**/.git/**",
-				"--glob",
-				"!**/node_modules/**",
-				searchPath,
-			];
-
-			// Run rg with timeout
-			const mainTimeoutSignal = AbortSignal.timeout(RG_TIMEOUT_MS);
-			const mainCombinedSignal = signal ? AbortSignal.any([signal, mainTimeoutSignal]) : mainTimeoutSignal;
-			const { stdout, stderr, exitCode } = await runRg(rgPath, args, mainCombinedSignal);
-			const output = stdout.trim();
-
-			// rg exit codes: 0 = found files, 1 = no matches, other = error
-			// Treat exit code 1 with no output as "no files found"
-			if (!output) {
-				if (exitCode !== 0 && exitCode !== 1) {
-					throw new ToolError(stderr.trim() || `rg failed (exit ${exitCode})`);
-				}
+			if (lines.length === 0) {
 				const details: FindToolDetails = { scopePath, fileCount: 0, files: [], truncated: false };
 				return toolResult(details).text("No files found matching pattern").done();
 			}
-
-			const lines = output.split("\n");
 			const relativized: string[] = [];
 			const mtimes: number[] = [];
 
@@ -220,16 +203,7 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 				}
 
 				const hadTrailingSlash = line.endsWith("/") || line.endsWith("\\");
-				let relativePath = line;
-				if (line.startsWith(searchPath)) {
-					relativePath = line.slice(searchPath.length + 1); // +1 for the /
-				} else {
-					relativePath = path.relative(searchPath, line);
-				}
-				const matchPath = relativePath.replace(/\\/g, "/");
-				if (!globMatcher.match(matchPath)) {
-					continue;
-				}
+				let relativePath = line.replace(/\\/g, "/");
 
 				let mtimeMs = 0;
 				let isDirectory = false;

package/src/tools/grep.ts

@@ -73,13 +73,22 @@ export interface RgResult {
  *
  * @throws ToolAbortError if signal is aborted
  */
-export async function runRg(rgPath: string, args: string[], signal?: AbortSignal): Promise<RgResult> {
-	const child = ptree.cspawn([rgPath, ...args], { signal });
+export async function runRg(
+	rgPath: string,
+	args: string[],
+	options?: { signal?: AbortSignal; timeoutMs?: number },
+): Promise<RgResult> {
+	const child = ptree.cspawn([rgPath, ...args], { signal: options?.signal, timeout: options?.timeoutMs });
+	const timeoutSeconds = options?.timeoutMs ? Math.max(1, Math.round(options.timeoutMs / 1000)) : undefined;
+	const timeoutMessage = timeoutSeconds ? `rg timed out after ${timeoutSeconds}s` : "rg timed out";
 
 	let stdout: string;
 	try {
 		stdout = await child.nothrow().text();
 	} catch (err) {
+		if (err instanceof ptree.TimeoutError) {
+			throw new ToolError(timeoutMessage);
+		}
 		if (err instanceof ptree.Exception && err.aborted) {
 			throw new ToolAbortError();
 		}
@@ -91,6 +100,9 @@ export async function runRg(rgPath: string, args: string[], signal?: AbortSignal
 		await child.exited;
 	} catch (err) {
 		exitError = err;
+		if (err instanceof ptree.TimeoutError) {
+			throw new ToolError(timeoutMessage);
+		}
 		if (err instanceof ptree.Exception && err.aborted) {
 			throw new ToolAbortError();
 		}

package/src/tools/read.ts

@@ -4,7 +4,7 @@ import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallb
 import type { ImageContent, TextContent } from "@oh-my-pi/pi-ai";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Text } from "@oh-my-pi/pi-tui";
-import { ptree } from "@oh-my-pi/pi-utils";
+import { globPaths, ptree } from "@oh-my-pi/pi-utils";
 import { Type } from "@sinclair/typebox";
 import { CONFIG_DIR_NAME } from "../config";
 import { renderPromptTemplate } from "../config/prompt-templates";
@@ -16,7 +16,6 @@ import { renderCodeCell, renderOutputBlock, renderStatusLine } from "../tui";
 import { formatDimensionNote, resizeImage } from "../utils/image-resize";
 import { detectSupportedImageMimeTypeFromFile } from "../utils/mime";
 import { ensureTool } from "../utils/tools-manager";
-import { runRg } from "./grep";
 import { applyListLimit } from "./list-limit";
 import { LsTool } from "./ls";
 import type { OutputMeta } from "./output-meta";
@@ -49,6 +48,7 @@ const MAX_FUZZY_RESULTS = 5;
 const MAX_FUZZY_CANDIDATES = 20000;
 const MIN_BASE_SIMILARITY = 0.5;
 const MIN_FULL_SIMILARITY = 0.6;
+const GLOB_TIMEOUT_MS = 5000;
 
 function normalizePathForMatch(value: string): string {
 	return value
@@ -162,95 +162,32 @@ function similarityScore(a: string, b: string): number {
 async function listCandidateFiles(
 	searchRoot: string,
 	signal?: AbortSignal,
-	notify?: (message: string) => void,
+	_notify?: (message: string) => void,
 ): Promise<{ files: string[]; truncated: boolean; error?: string }> {
-	let rgPath: string | undefined;
+	let files: string[];
 	try {
-		rgPath = await ensureTool("rg", { silent: true, notify });
-	} catch {
-		return { files: [], truncated: false, error: "rg not available" };
-	}
-
-	if (!rgPath) {
-		return { files: [], truncated: false, error: "rg not available" };
-	}
-
-	const args: string[] = [
-		"--files",
-		"--color=never",
-		"--hidden",
-		"--glob",
-		"!**/.git/**",
-		"--glob",
-		"!**/node_modules/**",
-	];
-
-	const gitignoreFiles = new Set<string>();
-	const rootGitignore = path.join(searchRoot, ".gitignore");
-	if (await Bun.file(rootGitignore).exists()) {
-		gitignoreFiles.add(rootGitignore);
-	}
-
-	try {
-		const gitignoreArgs = [
-			"--files",
-			"--color=never",
-			"--hidden",
-			"--no-ignore",
-			"--glob",
-			"!**/.git/**",
-			"--glob",
-			"!**/node_modules/**",
-			"--glob",
-			".gitignore",
-			searchRoot,
-		];
-		const { stdout } = await runRg(rgPath, gitignoreArgs, signal);
-		const output = stdout.trim();
-		if (output) {
-			const nestedGitignores = output
-				.split("\n")
-				.map(line => line.replace(/\r$/, "").trim())
-				.filter(line => line.length > 0);
-			for (const file of nestedGitignores) {
-				const normalized = file.replace(/\\/g, "/");
-				if (normalized.includes("/node_modules/") || normalized.includes("/.git/")) {
-					continue;
-				}
-				gitignoreFiles.add(file);
-			}
-		}
+		files = await globPaths("**/*", {
+			cwd: searchRoot,
+			gitignore: true,
+			dot: true,
+			signal,
+			timeoutMs: GLOB_TIMEOUT_MS,
+		});
 	} catch (error) {
-		if (error instanceof ToolAbortError) {
-			throw error;
+		if (error instanceof Error && error.name === "AbortError") {
+			throw new ToolAbortError();
 		}
-		// Ignore gitignore scan errors.
-	}
-
-	for (const gitignorePath of gitignoreFiles) {
-		args.push("--ignore-file", gitignorePath);
-	}
-
-	args.push(searchRoot);
-
-	const { stdout, stderr, exitCode } = await runRg(rgPath, args, signal);
-	const output = stdout.trim();
-
-	if (!output) {
-		// rg exit codes: 0 = ok, 1 = no matches, other = error
-		if (exitCode !== 0 && exitCode !== 1) {
-			return { files: [], truncated: false, error: stderr.trim() || `rg failed (exit ${exitCode})` };
+		if (error instanceof Error && error.name === "TimeoutError") {
+			const timeoutSeconds = Math.max(1, Math.round(GLOB_TIMEOUT_MS / 1000));
+			return { files: [], truncated: false, error: `glob timed out after ${timeoutSeconds}s` };
 		}
-		return { files: [], truncated: false };
+		const message = error instanceof Error ? error.message : String(error);
+		return { files: [], truncated: false, error: message };
 	}
 
-	const files = output
-		.split("\n")
-		.map(line => line.replace(/\r$/, "").trim())
-		.filter(line => line.length > 0);
-
-	const truncated = files.length > MAX_FUZZY_CANDIDATES;
-	const limited = truncated ? files.slice(0, MAX_FUZZY_CANDIDATES) : files;
+	const normalizedFiles = files.map(line => line.replace(/\r$/, "").trim()).filter(line => line.length > 0);
+	const truncated = normalizedFiles.length > MAX_FUZZY_CANDIDATES;
+	const limited = truncated ? normalizedFiles.slice(0, MAX_FUZZY_CANDIDATES) : normalizedFiles;
 
 	return { files: limited, truncated };
 }
@@ -304,11 +241,7 @@ async function findReadPathSuggestions(
 		const cleaned = file.replace(/\r$/, "").trim();
 		if (!cleaned) continue;
 
-		const relativePath = path.isAbsolute(cleaned)
-			? cleaned.startsWith(searchRoot)
-				? cleaned.slice(searchRoot.length + 1)
-				: path.relative(searchRoot, cleaned)
-			: cleaned;
+		const relativePath = cleaned;
 
 		if (!relativePath || relativePath.startsWith("..")) {
 			continue;