@victor-software-house/pi-acp 0.10.0 → 0.11.0

package/dist/{daemon-xclwSgis.mjs → daemon-COFaIaTB.mjs}

@@ -8,7 +8,7 @@ import { isAbsolute, join, resolve } from "node:path";
 import { Hono } from "hono";
 import { AgentSideConnection, RequestError, ndJsonStream } from "@agentclientprotocol/sdk";
 import { randomUUID } from "node:crypto";
-import { DefaultResourceLoader, SessionManager, createAgentSession, getAgentDir } from "@earendil-works/pi-coding-agent";
+import { DefaultResourceLoader, SessionManager, createAgentSession, createReadToolDefinition, getAgentDir } from "@earendil-works/pi-coding-agent";
 import * as z from "zod";
 import { parse } from "yaml";
 import { $ } from "bun";
@@ -184,6 +184,30 @@ function resolveIdleMs() {
 	return n * 1e3;
 }
 //#endregion
+//#region src/acp/acp-read-operations.ts
+function createAcpReadOperations(deps) {
+	const { conn, getSessionId } = deps;
+	return {
+		async readFile(absolutePath) {
+			const sessionId = getSessionId();
+			if (sessionId === "") throw new Error("pi-acp acp-fs read: sessionId not yet bound");
+			const response = await conn.readTextFile({
+				sessionId,
+				path: absolutePath
+			});
+			return Buffer.from(response.content, "utf8");
+		},
+		async access(absolutePath) {
+			const sessionId = getSessionId();
+			if (sessionId === "") throw new Error("pi-acp acp-fs access: sessionId not yet bound");
+			await conn.readTextFile({
+				sessionId,
+				path: absolutePath
+			});
+		}
+	};
+}
+//#endregion
 //#region src/acp/auth.ts
 const AUTH_METHOD_ID = "pi_terminal_login";
 function buildAuthMethods(opts) {
@@ -251,7 +275,8 @@ function parseClientCapabilities(caps) {
 	if (caps === void 0 || caps === null) return {
 		terminalOutput: false,
 		terminalAuth: false,
-		gatewayAuth: false
+		gatewayAuth: false,
+		fsReadTextFile: false
 	};
 	const meta = caps._meta;
 	const terminalOutput = typeof meta === "object" && meta !== null && meta["terminal_output"] === true;
@@ -264,10 +289,12 @@ function parseClientCapabilities(caps) {
 			if (typeof authMeta === "object" && authMeta !== null && "gateway" in authMeta) gatewayAuth = authMeta["gateway"] === true;
 		}
 	}
+	const fsReadTextFile = caps.fs?.readTextFile === true;
 	return {
 		terminalOutput,
 		terminalAuth,
-		gatewayAuth
+		gatewayAuth,
+		fsReadTextFile
 	};
 }
 //#endregion
@@ -1870,7 +1897,7 @@ var SshBackend = class {
 //#endregion
 //#region package.json
 var name = "@victor-software-house/pi-acp";
-var version = "0.10.0";
+var version = "0.11.0";
 //#endregion
 //#region src/acp/agent.ts
 /** Builtin ACP slash commands handled directly by the adapter. */
@@ -1959,7 +1986,8 @@ var PiAcpAgent = class {
 	clientCapabilities = {
 		terminalOutput: false,
 		terminalAuth: false,
-		gatewayAuth: false
+		gatewayAuth: false,
+		fsReadTextFile: false
 	};
 	daemonContext;
 	/** Unique ID for this ACP connection. Used as the ownership key in the daemon SessionRegistry. */
@@ -2059,6 +2087,44 @@ var PiAcpAgent = class {
 		}
 		return loader;
 	}
+	/**
+	* PRD-002 §FR-6 — `read` tool ACP-FS delegation overlay.
+	*
+	* When the client advertises `fs.readTextFile`, we override pi's
+	* built-in `read` with a custom `read` tool that proxies to
+	* `connection.fs.readTextFile`. The allowlist MUST include "read" so
+	* pi's customTool registration loop (which filters by name) can
+	* register the override; the override then shadows the builtin via
+	* the tool-definition `Map.set` path inside AgentSession.
+	*
+	* The sessionId ref is mutated by the caller right after
+	* `createAgentSession` returns, before any model turn — the tool
+	* isn't invoked until prompt-time, so the late binding is safe.
+	*
+	* Returns `null` when the client doesn't advertise the capability;
+	* callers then skip the overlay and pi's built-in `read` handles
+	* everything locally.
+	*/
+	buildAcpReadOverlay(cwd) {
+		if (!this.clientCapabilities.fsReadTextFile) return null;
+		const sessionIdRef = { current: "" };
+		return {
+			sessionIdRef,
+			tools: [
+				"read",
+				"bash",
+				"edit",
+				"write",
+				"grep",
+				"find",
+				"ls"
+			],
+			customTools: [createReadToolDefinition(cwd, { operations: createAcpReadOperations({
+				conn: this.conn,
+				getSessionId: () => sessionIdRef.current
+			}) })]
+		};
+	}
 	async initialize(params) {
 		const supportedVersion = 1;
 		const requested = params.protocolVersion;
@@ -2093,12 +2159,17 @@ var PiAcpAgent = class {
 	}
 	async newSession(params) {
 		if (!isAbsolute(params.cwd)) throw RequestError.invalidParams(`cwd must be an absolute path: ${params.cwd}`);
+		const acpReadOverlay = this.buildAcpReadOverlay(params.cwd);
 		let result;
 		try {
 			const resourceLoader = await this.buildResourceLoader(params.cwd, params);
 			result = await createAgentSession({
 				cwd: params.cwd,
-				resourceLoader
+				resourceLoader,
+				...acpReadOverlay ? {
+					tools: acpReadOverlay.tools,
+					customTools: acpReadOverlay.customTools
+				} : {}
 			});
 		} catch (e) {
 			const authErr = detectAuthError(e);
@@ -2107,6 +2178,7 @@ var PiAcpAgent = class {
 			throw RequestError.internalError({}, `Failed to create pi session: ${msg}`);
 		}
 		const piSession = result.session;
+		if (acpReadOverlay !== null) acpReadOverlay.sessionIdRef.current = piSession.sessionManager.getSessionId();
 		if (piSession.modelRegistry.getAvailable().length === 0) {
 			piSession.dispose();
 			throw RequestError.authRequired({ authMethods: buildAuthMethods() }, "Configure an API key or log in with an OAuth provider.");
@@ -2366,6 +2438,7 @@ var PiAcpAgent = class {
 		this.sessions.close(params.sessionId);
 		const sessionFile = await this.resolveSessionFile(params.sessionId);
 		if (sessionFile === null) throw RequestError.invalidParams(`Unknown sessionId: ${params.sessionId}`);
+		const acpReadOverlay = this.buildAcpReadOverlay(params.cwd);
 		let result;
 		try {
 			const sm = SessionManager.open(sessionFile);
@@ -2373,7 +2446,11 @@ var PiAcpAgent = class {
 			result = await createAgentSession({
 				cwd: params.cwd,
 				sessionManager: sm,
-				resourceLoader
+				resourceLoader,
+				...acpReadOverlay ? {
+					tools: acpReadOverlay.tools,
+					customTools: acpReadOverlay.customTools
+				} : {}
 			});
 		} catch (e) {
 			const authErr = detectAuthError(e);
@@ -2382,6 +2459,7 @@ var PiAcpAgent = class {
 			throw RequestError.internalError({}, `Failed to load pi session: ${msg}`);
 		}
 		const piSession = result.session;
+		if (acpReadOverlay !== null) acpReadOverlay.sessionIdRef.current = piSession.sessionManager.getSessionId();
 		const session = new PiAcpSession({
 			sessionId: params.sessionId,
 			cwd: params.cwd,
@@ -2466,6 +2544,7 @@ var PiAcpAgent = class {
 		}
 		const sessionFile = await this.resolveSessionFile(params.sessionId);
 		if (sessionFile === null) throw RequestError.invalidParams(`Unknown sessionId: ${params.sessionId}`);
+		const acpReadOverlay = this.buildAcpReadOverlay(params.cwd);
 		let result;
 		try {
 			const sm = SessionManager.open(sessionFile);
@@ -2473,7 +2552,11 @@ var PiAcpAgent = class {
 			result = await createAgentSession({
 				cwd: params.cwd,
 				sessionManager: sm,
-				resourceLoader
+				resourceLoader,
+				...acpReadOverlay ? {
+					tools: acpReadOverlay.tools,
+					customTools: acpReadOverlay.customTools
+				} : {}
 			});
 		} catch (e) {
 			const authErr = detectAuthError(e);
@@ -2482,6 +2565,7 @@ var PiAcpAgent = class {
 			throw RequestError.internalError({}, `Failed to resume pi session: ${msg}`);
 		}
 		const piSession = result.session;
+		if (acpReadOverlay !== null) acpReadOverlay.sessionIdRef.current = piSession.sessionManager.getSessionId();
 		const session = new PiAcpSession({
 			sessionId: params.sessionId,
 			cwd: params.cwd,
@@ -2525,6 +2609,7 @@ var PiAcpAgent = class {
 		if (!isAbsolute(params.cwd)) throw RequestError.invalidParams(`cwd must be an absolute path: ${params.cwd}`);
 		const sourceFile = await this.resolveSessionFile(params.sessionId);
 		if (sourceFile === null) throw RequestError.invalidParams(`Unknown sessionId: ${params.sessionId}`);
+		const acpReadOverlay = this.buildAcpReadOverlay(params.cwd);
 		let result;
 		try {
 			const sm = SessionManager.forkFrom(sourceFile, params.cwd);
@@ -2532,7 +2617,11 @@ var PiAcpAgent = class {
 			result = await createAgentSession({
 				cwd: params.cwd,
 				sessionManager: sm,
-				resourceLoader
+				resourceLoader,
+				...acpReadOverlay ? {
+					tools: acpReadOverlay.tools,
+					customTools: acpReadOverlay.customTools
+				} : {}
 			});
 		} catch (e) {
 			const authErr = detectAuthError(e);
@@ -2542,6 +2631,7 @@ var PiAcpAgent = class {
 		}
 		const piSession = result.session;
 		const newSessionId = piSession.sessionManager.getSessionId();
+		if (acpReadOverlay !== null) acpReadOverlay.sessionIdRef.current = newSessionId;
 		const newSessionFile = piSession.sessionManager.getSessionFile();
 		if (newSessionFile !== void 0) this.sessionPaths.set(newSessionId, newSessionFile);
 		const session = new PiAcpSession({
@@ -3160,4 +3250,4 @@ async function runDaemon() {
 //#endregion
 export { runDaemon };
 
-//# sourceMappingURL=daemon-xclwSgis.mjs.map
+//# sourceMappingURL=daemon-COFaIaTB.mjs.map