@oh-my-pi/pi-coding-agent 11.8.2 → 11.9.0

package/src/tools/calculator.ts

@@ -216,14 +216,14 @@ function tokenizeExpression(expression: string): Token[] {
  * the call stack.
  */
 class ExpressionParser {
-	private index = 0;
+	#index = 0;
 
 	constructor(private readonly tokens: Token[]) {}
 
 	/** Parse the full expression and ensure all tokens are consumed. */
 	parse(): number {
-		const value = this.parseExpression();
-		if (this.index < this.tokens.length) {
+		const value = this.#parseExpression();
+		if (this.#index < this.tokens.length) {
 			throw new Error("Unexpected token in expression");
 		}
 		return value;
@@ -233,15 +233,15 @@ class ExpressionParser {
 	 * Parse addition and subtraction (lowest precedence).
 	 * Left-associative: 1 - 2 - 3 = (1 - 2) - 3
 	 */
-	private parseExpression(): number {
-		let value = this.parseTerm();
+	#parseExpression(): number {
+		let value = this.#parseTerm();
 		while (true) {
-			if (this.matchOperator("+")) {
-				value += this.parseTerm();
+			if (this.#matchOperator("+")) {
+				value += this.#parseTerm();
 				continue;
 			}
-			if (this.matchOperator("-")) {
-				value -= this.parseTerm();
+			if (this.#matchOperator("-")) {
+				value -= this.#parseTerm();
 				continue;
 			}
 			break;
@@ -253,19 +253,19 @@ class ExpressionParser {
 	 * Parse multiplication, division, and modulo.
 	 * Left-associative: 8 / 4 / 2 = (8 / 4) / 2
 	 */
-	private parseTerm(): number {
-		let value = this.parseUnary();
+	#parseTerm(): number {
+		let value = this.#parseUnary();
 		while (true) {
-			if (this.matchOperator("*")) {
-				value *= this.parseUnary();
+			if (this.#matchOperator("*")) {
+				value *= this.#parseUnary();
 				continue;
 			}
-			if (this.matchOperator("/")) {
-				value /= this.parseUnary();
+			if (this.#matchOperator("/")) {
+				value /= this.#parseUnary();
 				continue;
 			}
-			if (this.matchOperator("%")) {
-				value %= this.parseUnary();
+			if (this.#matchOperator("%")) {
+				value %= this.#parseUnary();
 				continue;
 			}
 			break;
@@ -277,14 +277,14 @@ class ExpressionParser {
 	 * Parse unary + and - operators.
 	 * Recursive to handle chained unary: --x, +-x
 	 */
-	private parseUnary(): number {
-		if (this.matchOperator("+")) {
-			return this.parseUnary();
+	#parseUnary(): number {
+		if (this.#matchOperator("+")) {
+			return this.#parseUnary();
 		}
-		if (this.matchOperator("-")) {
-			return -this.parseUnary();
+		if (this.#matchOperator("-")) {
+			return -this.#parseUnary();
 		}
-		return this.parsePower();
+		return this.#parsePower();
 	}
 
 	/**
@@ -292,10 +292,10 @@ class ExpressionParser {
 	 * Right-associative: 2 ** 3 ** 2 = 2 ** (3 ** 2) = 512
 	 * Achieved by recursive call to parsePower for the right operand.
 	 */
-	private parsePower(): number {
-		let value = this.parsePrimary();
-		if (this.matchOperator("**")) {
-			value = value ** this.parsePower(); // Right-associative via recursion
+	#parsePower(): number {
+		let value = this.#parsePrimary();
+		if (this.#matchOperator("**")) {
+			value = value ** this.#parsePower(); // Right-associative via recursion
 		}
 		return value;
 	}
@@ -304,21 +304,21 @@ class ExpressionParser {
 	 * Parse primary expressions: number literals and parenthesized subexpressions.
 	 * Parentheses restart parsing at lowest precedence (parseExpression).
 	 */
-	private parsePrimary(): number {
-		const token = this.peek();
+	#parsePrimary(): number {
+		const token = this.#peek();
 		if (!token) {
 			throw new Error("Unexpected end of expression");
 		}
 
 		if (token.type === "number") {
-			this.index += 1;
+			this.#index += 1;
 			return token.value;
 		}
 
 		if (token.type === "paren" && token.value === "(") {
-			this.index += 1;
-			const value = this.parseExpression(); // Reset to lowest precedence
-			if (!this.matchParen(")")) {
+			this.#index += 1;
+			const value = this.#parseExpression(); // Reset to lowest precedence
+			if (!this.#matchParen(")")) {
 				throw new Error("Missing closing parenthesis");
 			}
 			return value;
@@ -328,28 +328,28 @@ class ExpressionParser {
 	}
 
 	/** Consume operator if it matches, advancing the token index. */
-	private matchOperator(value: Operator): boolean {
-		const token = this.tokens[this.index];
+	#matchOperator(value: Operator): boolean {
+		const token = this.tokens[this.#index];
 		if (token && token.type === "operator" && token.value === value) {
-			this.index += 1;
+			this.#index += 1;
 			return true;
 		}
 		return false;
 	}
 
 	/** Consume parenthesis if it matches, advancing the token index. */
-	private matchParen(value: "(" | ")"): boolean {
-		const token = this.tokens[this.index];
+	#matchParen(value: "(" | ")"): boolean {
+		const token = this.tokens[this.#index];
 		if (token && token.type === "paren" && token.value === value) {
-			this.index += 1;
+			this.#index += 1;
 			return true;
 		}
 		return false;
 	}
 
 	/** Look at current token without consuming it. */
-	private peek(): Token | undefined {
-		return this.tokens[this.index];
+	#peek(): Token | undefined {
+		return this.tokens[this.#index];
 	}
 }
 
@@ -395,16 +395,16 @@ type CalculatorParams = { calculations: Array<{ expression: string; prefix: stri
  * standard arithmetic operators, and parentheses.
  */
 export class CalculatorTool implements AgentTool<typeof calculatorSchema, CalculatorToolDetails> {
-	public readonly name = "calc";
-	public readonly label = "Calc";
-	public readonly description: string;
-	public readonly parameters = calculatorSchema;
+	readonly name = "calc";
+	readonly label = "Calc";
+	readonly description: string;
+	readonly parameters = calculatorSchema;
 
 	constructor(_session: ToolSession) {
 		this.description = renderPromptTemplate(calculatorDescription);
 	}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		{ calculations }: CalculatorParams,
 		signal?: AbortSignal,

package/src/tools/context.ts

@@ -12,28 +12,28 @@ declare module "@oh-my-pi/pi-agent-core" {
 }
 
 export class ToolContextStore {
-	private uiContext: ExtensionUIContext | undefined;
-	private hasUI = false;
-	private toolNames: string[] = [];
+	#uiContext: ExtensionUIContext | undefined;
+	#hasUI = false;
+	#toolNames: string[] = [];
 
 	constructor(private readonly getBaseContext: () => CustomToolContext) {}
 
 	getContext(toolCall?: ToolCallContext): AgentToolContext {
 		return {
 			...this.getBaseContext(),
-			ui: this.uiContext,
-			hasUI: this.hasUI,
-			toolNames: this.toolNames,
+			ui: this.#uiContext,
+			hasUI: this.#hasUI,
+			toolNames: this.#toolNames,
 			toolCall,
 		};
 	}
 
 	setUIContext(uiContext: ExtensionUIContext, hasUI: boolean): void {
-		this.uiContext = uiContext;
-		this.hasUI = hasUI;
+		this.#uiContext = uiContext;
+		this.#hasUI = hasUI;
 	}
 
 	setToolNames(names: string[]): void {
-		this.toolNames = names;
+		this.#toolNames = names;
 	}
 }

package/src/tools/exit-plan-mode.ts

@@ -16,16 +16,16 @@ export interface ExitPlanModeDetails {
 }
 
 export class ExitPlanModeTool implements AgentTool<typeof exitPlanModeSchema, ExitPlanModeDetails> {
-	public readonly name = "exit_plan_mode";
-	public readonly label = "ExitPlanMode";
-	public readonly description: string;
-	public readonly parameters = exitPlanModeSchema;
+	readonly name = "exit_plan_mode";
+	readonly label = "ExitPlanMode";
+	readonly description: string;
+	readonly parameters = exitPlanModeSchema;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(exitPlanModeDescription);
 	}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		_params: Record<string, never>,
 		_signal?: AbortSignal,

package/src/tools/fetch.ts

@@ -852,16 +852,16 @@ export interface FetchToolDetails {
 }
 
 export class FetchTool implements AgentTool<typeof fetchSchema, FetchToolDetails> {
-	public readonly name = "fetch";
-	public readonly label = "Fetch";
-	public readonly description: string;
-	public readonly parameters = fetchSchema;
+	readonly name = "fetch";
+	readonly label = "Fetch";
+	readonly description: string;
+	readonly parameters = fetchSchema;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(fetchDescription);
 	}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		params: Static<typeof fetchSchema>,
 		signal?: AbortSignal,

package/src/tools/find.ts

@@ -1,4 +1,4 @@
-import * as fs from "node:fs/promises";
+import * as fs from "node:fs";
 import * as path from "node:path";
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import { FileType, type GlobMatch, glob } from "@oh-my-pi/pi-natives";
@@ -123,22 +123,22 @@ export interface FindToolOptions {
 }
 
 export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
-	public readonly name = "find";
-	public readonly label = "Find";
-	public readonly description: string;
-	public readonly parameters = findSchema;
+	readonly name = "find";
+	readonly label = "Find";
+	readonly description: string;
+	readonly parameters = findSchema;
 
-	private readonly customOps?: FindOperations;
+	readonly #customOps?: FindOperations;
 
 	constructor(
 		private readonly session: ToolSession,
 		options?: FindToolOptions,
 	) {
-		this.customOps = options?.operations;
+		this.#customOps = options?.operations;
 		this.description = renderPromptTemplate(findDescription);
 	}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		params: Static<typeof findSchema>,
 		signal?: AbortSignal,
@@ -177,13 +177,13 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 			const includeHidden = hidden ?? true;
 
 			// If custom operations provided with glob, use that instead of fd
-			if (this.customOps?.glob) {
-				if (!(await this.customOps.exists(searchPath))) {
+			if (this.#customOps?.glob) {
+				if (!(await this.#customOps.exists(searchPath))) {
 					throw new ToolError(`Path not found: ${searchPath}`);
 				}
 
-				if (!hasGlob && this.customOps.stat) {
-					const stat = await this.customOps.stat(searchPath);
+				if (!hasGlob && this.#customOps.stat) {
+					const stat = await this.#customOps.stat(searchPath);
 					if (stat.isFile()) {
 						const files = [scopePath];
 						const details: FindToolDetails = {
@@ -196,7 +196,7 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 					}
 				}
 
-				const results = await this.customOps.glob(globPattern, searchPath, {
+				const results = await this.#customOps.glob(globPattern, searchPath, {
 					ignore: ["**/node_modules/**", "**/.git/**"],
 					limit: effectiveLimit,
 				});
@@ -239,9 +239,9 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 				return resultBuilder.done();
 			}
 
-			let searchStat: Awaited<ReturnType<typeof fs.stat>>;
+			let searchStat: fs.Stats;
 			try {
-				searchStat = await fs.stat(searchPath);
+				searchStat = await fs.promises.stat(searchPath);
 			} catch (err) {
 				if (isEnoent(err)) {
 					throw new ToolError(`Path not found: ${searchPath}`);
@@ -263,7 +263,7 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 				throw new ToolError(`Path is not a directory: ${searchPath}`);
 			}
 
-			let matches: Awaited<ReturnType<typeof glob>>["matches"];
+			let matches: GlobMatch[];
 			const onUpdateMatches: string[] = [];
 			const updateIntervalMs = 200;
 			let lastUpdate = 0;

package/src/tools/grep.ts

@@ -1,7 +1,7 @@
-import * as nodePath from "node:path";
+import * as path from "node:path";
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 
-import { grep } from "@oh-my-pi/pi-natives";
+import { type GrepResult, grep } from "@oh-my-pi/pi-natives";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Text } from "@oh-my-pi/pi-tui";
 import { untilAborted } from "@oh-my-pi/pi-utils";
@@ -51,31 +51,19 @@ export interface GrepToolDetails {
 	error?: string;
 }
 
-export interface GrepOperations {
-	isDirectory: (absolutePath: string) => Promise<boolean> | boolean;
-	readFile: (absolutePath: string) => Promise<string> | string;
-}
-
-export interface GrepToolOptions {
-	operations?: GrepOperations;
-}
-
 type GrepParams = Static<typeof grepSchema>;
 
 export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
-	public readonly name = "grep";
-	public readonly label = "Grep";
-	public readonly description: string;
-	public readonly parameters = grepSchema;
-
-	constructor(
-		private readonly session: ToolSession,
-		_options?: GrepToolOptions,
-	) {
+	readonly name = "grep";
+	readonly label = "Grep";
+	readonly description: string;
+	readonly parameters = grepSchema;
+
+	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(grepDescription);
 	}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		params: GrepParams,
 		signal?: AbortSignal,
@@ -111,7 +99,7 @@ export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
 
 			const searchPath = resolveToCwd(searchDir || ".", this.session.cwd);
 			const scopePath = (() => {
-				const relative = nodePath.relative(this.session.cwd, searchPath).replace(/\\/g, "/");
+				const relative = path.relative(this.session.cwd, searchPath).replace(/\\/g, "/");
 				return relative.length === 0 ? "." : relative;
 			})();
 
@@ -127,7 +115,7 @@ export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
 			const effectiveLimit = normalizedLimit ?? DEFAULT_MATCH_LIMIT;
 
 			// Run grep
-			let result: Awaited<ReturnType<typeof grep>>;
+			let result: GrepResult;
 			try {
 				result = await grep({
 					pattern: normalizedPattern,
@@ -157,7 +145,7 @@ export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
 				if (isDirectory) {
 					return cleanPath.replace(/\\/g, "/");
 				}
-				return nodePath.basename(cleanPath);
+				return path.basename(cleanPath);
 			};
 
 			// Build output

package/src/tools/index.ts

@@ -74,7 +74,7 @@ export { type ExitPlanModeDetails, ExitPlanModeTool } from "./exit-plan-mode";
 export { FetchTool, type FetchToolDetails } from "./fetch";
 export { type FindOperations, FindTool, type FindToolDetails, type FindToolInput, type FindToolOptions } from "./find";
 export { setPreferredImageProvider } from "./gemini-image";
-export { type GrepOperations, GrepTool, type GrepToolDetails, type GrepToolInput, type GrepToolOptions } from "./grep";
+export { GrepTool, type GrepToolDetails, type GrepToolInput } from "./grep";
 export { NotebookTool, type NotebookToolDetails } from "./notebook";
 export { PythonTool, type PythonToolDetails, type PythonToolOptions } from "./python";
 export { ReadTool, type ReadToolDetails, type ReadToolInput } from "./read";

package/src/tools/notebook.ts

@@ -60,16 +60,16 @@ function splitIntoLines(content: string): string[] {
 type NotebookParams = Static<typeof notebookSchema>;
 
 export class NotebookTool implements AgentTool<typeof notebookSchema, NotebookToolDetails> {
-	public readonly name = "notebook";
-	public readonly label = "Notebook";
-	public readonly description =
+	readonly name = "notebook";
+	readonly label = "Notebook";
+	readonly description =
 		"Completely replaces the contents of a specific cell in a Jupyter notebook (.ipynb file) with new source. Jupyter notebooks are interactive documents that combine code, text, and visualizations, commonly used for data analysis and scientific computing. The notebook_path parameter must be an absolute path, not a relative path. The cell_number is 0-indexed. Use edit_mode=insert to add a new cell at the index specified by cell_number. Use edit_mode=delete to delete the cell at the index specified by cell_number.";
-	public readonly parameters = notebookSchema;
-	public readonly concurrency = "exclusive";
+	readonly parameters = notebookSchema;
+	readonly concurrency = "exclusive";
 
 	constructor(private readonly session: ToolSession) {}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		params: NotebookParams,
 		signal?: AbortSignal,

package/src/tools/output-meta.ts

@@ -428,8 +428,16 @@ export function wrapToolWithMetaNotice<T extends AgentTool<any, any, any>>(tool:
 		return result;
 	};
 
-	return Object.create(tool, {
-		execute: { value: wrappedExecute, enumerable: true },
+	// Use Proxy so property access (description, parameters, mode) stays on the original
+	// tool. Object.create(tool) would make getters run with this=wrapper, breaking
+	// private fields on tools like EditTool.
+	return new Proxy(tool, {
+		get(target, prop) {
+			if (prop === "execute") return wrappedExecute;
+			const value = (target as Record<string | symbol, unknown>)[prop];
+			if (typeof value === "function") return value.bind(target);
+			return value;
+		},
 	}) as T;
 }
 

package/src/tools/python.ts

@@ -1,3 +1,4 @@
+import type * as fs from "node:fs";
 import * as path from "node:path";
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import type { ImageContent } from "@oh-my-pi/pi-ai";
@@ -142,31 +143,31 @@ export interface PythonToolOptions {
 }
 
 export class PythonTool implements AgentTool<typeof pythonSchema> {
-	public readonly name = "python";
-	public readonly label = "Python";
-	public readonly description: string;
-	public readonly parameters = pythonSchema;
-	public readonly concurrency = "exclusive";
+	readonly name = "python";
+	readonly label = "Python";
+	readonly description: string;
+	readonly parameters = pythonSchema;
+	readonly concurrency = "exclusive";
 
-	private readonly proxyExecutor?: PythonProxyExecutor;
+	readonly #proxyExecutor?: PythonProxyExecutor;
 
 	constructor(
 		private readonly session: ToolSession | null,
 		options?: PythonToolOptions,
 	) {
-		this.proxyExecutor = options?.proxyExecutor;
+		this.#proxyExecutor = options?.proxyExecutor;
 		this.description = getPythonToolDescription();
 	}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		params: Static<typeof pythonSchema>,
 		signal?: AbortSignal,
 		onUpdate?: AgentToolUpdateCallback,
 		_ctx?: AgentToolContext,
 	): Promise<AgentToolResult<PythonToolDetails | undefined>> {
-		if (this.proxyExecutor) {
-			return this.proxyExecutor(params, signal);
+		if (this.#proxyExecutor) {
+			return this.#proxyExecutor(params, signal);
 		}
 
 		if (!this.session) {
@@ -195,7 +196,7 @@ export class PythonTool implements AgentTool<typeof pythonSchema> {
 			}
 
 			const commandCwd = cwd ? resolveToCwd(cwd, this.session.cwd) : this.session.cwd;
-			let cwdStat: Awaited<ReturnType<Bun.BunFile["stat"]>>;
+			let cwdStat: fs.Stats;
 			try {
 				cwdStat = await Bun.file(commandCwd).stat();
 			} catch {

package/src/tools/read.ts

@@ -535,24 +535,24 @@ type ReadParams = { path: string; offset?: number; limit?: number; lines?: boole
  * Directories return a formatted listing with modification times.
  */
 export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
-	public readonly name = "read";
-	public readonly label = "Read";
-	public readonly description: string;
-	public readonly parameters = readSchema;
-	public readonly nonAbortable = true;
+	readonly name = "read";
+	readonly label = "Read";
+	readonly description: string;
+	readonly parameters = readSchema;
+	readonly nonAbortable = true;
 
-	private readonly autoResizeImages: boolean;
-	private readonly defaultLineNumbers: boolean;
+	readonly #autoResizeImages: boolean;
+	readonly #defaultLineNumbers: boolean;
 
 	constructor(private readonly session: ToolSession) {
-		this.autoResizeImages = session.settings.get("images.autoResize");
-		this.defaultLineNumbers = session.settings.get("readLineNumbers");
+		this.#autoResizeImages = session.settings.get("images.autoResize");
+		this.#defaultLineNumbers = session.settings.get("readLineNumbers");
 		this.description = renderPromptTemplate(readDescription, {
 			DEFAULT_MAX_LINES: String(DEFAULT_MAX_LINES),
 		});
 	}
 
-	public async execute(
+	async execute(
 		_toolCallId: string,
 		params: ReadParams,
 		signal?: AbortSignal,
@@ -564,7 +564,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 		// Handle internal URLs (agent://, skill://)
 		const internalRouter = this.session.internalRouter;
 		if (internalRouter?.canHandle(readPath)) {
-			return this.handleInternalUrl(readPath, offset, limit, lines);
+			return this.#handleInternalUrl(readPath, offset, limit, lines);
 		}
 
 		const absolutePath = resolveReadPath(readPath, this.session.cwd);
@@ -604,7 +604,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 		}
 
 		if (isDirectory) {
-			return this.readDirectory(absolutePath, limit, signal);
+			return this.#readDirectory(absolutePath, limit, signal);
 		}
 
 		const mimeType = await detectSupportedImageMimeTypeFromFile(absolutePath);
@@ -636,7 +636,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 				} else {
 					const base64 = new Uint8Array(buffer).toBase64();
 
-					if (this.autoResizeImages) {
+					if (this.#autoResizeImages) {
 						// Resize image if needed - catch errors from Photon
 						try {
 							const resized = await resizeImage({ type: "image", data: base64, mimeType });
@@ -754,7 +754,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 			};
 
 			// Add line numbers if requested (uses setting default if not specified)
-			const shouldAddLineNumbers = lines ?? this.defaultLineNumbers;
+			const shouldAddLineNumbers = lines ?? this.#defaultLineNumbers;
 			const prependLineNumbers = (text: string, startNum: number): string => {
 				const textLines = text.split("\n");
 				const lastLineNum = startNum + textLines.length - 1;
@@ -831,7 +831,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 	 * Handle internal URLs (agent://, skill://).
 	 * Supports pagination via offset/limit but rejects them when query extraction is used.
 	 */
-	private async handleInternalUrl(
+	async #handleInternalUrl(
 		url: string,
 		offset?: number,
 		limit?: number,
@@ -904,7 +904,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 		const truncation = truncateHead(selectedContent);
 
 		// Add line numbers if requested
-		const shouldAddLineNumbers = lines ?? this.defaultLineNumbers;
+		const shouldAddLineNumbers = lines ?? this.#defaultLineNumbers;
 		const prependLineNumbers = (text: string, startNum: number): string => {
 			const textLines = text.split("\n");
 			const lastLineNum = startNum + textLines.length - 1;
@@ -976,7 +976,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 	}
 
 	/** Read directory contents as a formatted listing */
-	private async readDirectory(
+	async #readDirectory(
 		absolutePath: string,
 		limit: number | undefined,
 		signal?: AbortSignal,