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

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

@@ -2,13 +2,13 @@
 import { a as removeStaleSocketIfAny, i as releaseLock, n as controlSocketPath, o as socketPath, r as ensureSocketParentDir, t as acquireLock } from "./socket-wvV053VI.mjs";
 import { t as piChangelogPath } from "./pi-package-aHs6rWNo.mjs";
 import { createServer } from "node:net";
-import { existsSync, readFileSync } from "node:fs";
-import { homedir } from "node:os";
+import { existsSync, mkdtempSync, readFileSync, rmSync } from "node:fs";
+import { homedir, tmpdir } from "node:os";
 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
@@ -849,6 +876,7 @@ var PiAcpSession = class {
 	lastAssistantStopReason = null;
 	lastEmit = Promise.resolve();
 	unsubscribe;
+	cleanups;
 	constructor(opts) {
 		this.sessionId = opts.sessionId;
 		this.cwd = opts.cwd;
@@ -856,11 +884,15 @@ var PiAcpSession = class {
 		this.piSession = opts.piSession;
 		this.conn = opts.conn;
 		this.supportsTerminalOutput = opts.supportsTerminalOutput ?? false;
+		this.cleanups = opts.cleanups ?? [];
 		this.unsubscribe = this.piSession.subscribe((ev) => this.handlePiEvent(ev));
 	}
 	dispose() {
 		this.unsubscribe?.();
 		this.piSession.dispose();
+		for (const cleanup of this.cleanups) try {
+			cleanup();
+		} catch {}
 	}
 	/**
 	* Drop event subscription without disposing the underlying piSession.
@@ -1652,6 +1684,58 @@ function tryFromFile(path, source, diagnostics) {
 	return null;
 }
 //#endregion
+//#region src/resources/modes.ts
+/**
+* Cwd-independence modes (PRD-002 §FR-5).
+*
+* | Mode      | cwd handling                               | Tool target   |
+* |-----------|--------------------------------------------|---------------|
+* | `local`   | ACP params.cwd (must be absolute)          | params.cwd    |
+* | `overlay` | same as local — manifest aux roots compose | params.cwd    |
+* | `none`    | substitute ephemeral tmpdir                | tmpdir        |
+*
+* `local` and `overlay` are functionally identical in the current substrate
+* because `VirtualResourceLoader` always overlays manifest roots on top of
+* the implicit local root. The `overlay` keyword is retained for
+* forward-compatibility and operator clarity (declaring `mode: overlay`
+* documents the intent even if the runtime path is the same).
+*
+* `none` mints `mkdtemp(...)` under the OS tmpdir and returns a cleanup
+* thunk the caller must invoke at session close. The cleanup is
+* best-effort — never throws — so a session dispose path that runs it
+* after the directory has already been removed is safe.
+*/
+const TMPDIR_PREFIX = "pi-acp-session-";
+function resolveMode(input) {
+	const mode = input.manifest.mode;
+	if (mode === "none") {
+		const dir = mkdtempSync(join(tmpdir(), TMPDIR_PREFIX));
+		let removed = false;
+		const cleanup = () => {
+			if (removed) return;
+			removed = true;
+			try {
+				rmSync(dir, {
+					recursive: true,
+					force: true
+				});
+			} catch {}
+		};
+		return {
+			mode,
+			cwd: dir,
+			cleanup,
+			ephemeral: true
+		};
+	}
+	return {
+		mode,
+		cwd: input.requestedCwd,
+		cleanup: () => {},
+		ephemeral: false
+	};
+}
+//#endregion
 //#region src/resources/sources/http.ts
 const DEFAULT_CACHE_TTL_SECONDS = 300;
 const DEFAULT_TIMEOUT_MS$1 = 5e3;
@@ -1870,7 +1954,7 @@ var SshBackend = class {
 //#endregion
 //#region package.json
 var name = "@victor-software-house/pi-acp";
-var version = "0.10.0";
+var version = "0.12.0";
 //#endregion
 //#region src/acp/agent.ts
 /** Builtin ACP slash commands handled directly by the adapter. */
@@ -1959,7 +2043,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. */
@@ -2006,13 +2091,18 @@ var PiAcpAgent = class {
 			cwd,
 			sessionParams
 		});
+		const modeResult = resolveMode({
+			manifest: loaded.manifest,
+			requestedCwd: cwd
+		});
+		const effectiveCwd = modeResult.cwd;
 		const diagnostics = [...loaded.diagnostics];
 		const sources = [];
 		for (const root of loaded.manifest.roots) {
 			if (root.kind === "local") {
 				sources.push(new LocalBackend({
 					id: root.id,
-					cwd: root.paths.cwd ?? cwd,
+					cwd: root.paths.cwd ?? effectiveCwd,
 					agentDir: root.paths.agentDir ?? getAgentDir()
 				}));
 				continue;
@@ -2045,7 +2135,7 @@ var PiAcpAgent = class {
 			diagnostics.push(diag);
 		}
 		if (!sources.some((s) => s.kind === "local")) sources.unshift(new LocalBackend({
-			cwd,
+			cwd: effectiveCwd,
 			agentDir: getAgentDir()
 		}));
 		const loader = new VirtualResourceLoader({
@@ -2057,7 +2147,48 @@ var PiAcpAgent = class {
 			const where = d.path !== void 0 ? ` ${d.path}` : "";
 			process.stderr.write(`pi-acp manifest [${d.source}${where}]: ${d.message}\n`);
 		}
-		return loader;
+		return {
+			loader,
+			modeResult
+		};
+	}
+	/**
+	* 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;
@@ -2092,23 +2223,40 @@ var PiAcpAgent = class {
 		};
 	}
 	async newSession(params) {
-		if (!isAbsolute(params.cwd)) throw RequestError.invalidParams(`cwd must be an absolute path: ${params.cwd}`);
+		const { loader: resourceLoader, modeResult } = await this.buildResourceLoader(params.cwd, params).catch((e) => {
+			const authErr = detectAuthError(e);
+			if (authErr !== null) throw authErr;
+			const msg = e instanceof Error ? e.message : String(e);
+			throw RequestError.internalError({}, `Failed to load pi-acp manifest: ${msg}`);
+		});
+		const effectiveCwd = modeResult.cwd;
+		if (modeResult.mode !== "none" && !isAbsolute(params.cwd)) {
+			modeResult.cleanup();
+			throw RequestError.invalidParams(`cwd must be an absolute path: ${params.cwd}`);
+		}
+		const acpReadOverlay = this.buildAcpReadOverlay(effectiveCwd);
 		let result;
 		try {
-			const resourceLoader = await this.buildResourceLoader(params.cwd, params);
 			result = await createAgentSession({
-				cwd: params.cwd,
-				resourceLoader
+				cwd: effectiveCwd,
+				resourceLoader,
+				...acpReadOverlay ? {
+					tools: acpReadOverlay.tools,
+					customTools: acpReadOverlay.customTools
+				} : {}
 			});
 		} catch (e) {
+			modeResult.cleanup();
 			const authErr = detectAuthError(e);
 			if (authErr !== null) throw authErr;
 			const msg = e instanceof Error ? e.message : String(e);
 			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();
+			modeResult.cleanup();
 			throw RequestError.authRequired({ authMethods: buildAuthMethods() }, "Configure an API key or log in with an OAuth provider.");
 		}
 		const sessionId = piSession.sessionManager.getSessionId();
@@ -2116,11 +2264,12 @@ var PiAcpAgent = class {
 		if (sessionFile !== void 0) this.sessionPaths.set(sessionId, sessionFile);
 		const session = new PiAcpSession({
 			sessionId,
-			cwd: params.cwd,
+			cwd: effectiveCwd,
 			mcpServers: params.mcpServers,
 			piSession,
 			conn: this.conn,
-			supportsTerminalOutput: this.clientCapabilities.terminalOutput
+			supportsTerminalOutput: this.clientCapabilities.terminalOutput,
+			cleanups: modeResult.ephemeral ? [modeResult.cleanup] : []
 		});
 		this.sessions.register(session);
 		this.registerWithDaemon({
@@ -2366,14 +2515,19 @@ 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);
-			const resourceLoader = await this.buildResourceLoader(params.cwd, params);
+			const { loader: resourceLoader } = await this.buildResourceLoader(params.cwd, params);
 			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 +2536,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,14 +2621,19 @@ 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);
-			const resourceLoader = await this.buildResourceLoader(params.cwd, params);
+			const { loader: resourceLoader } = await this.buildResourceLoader(params.cwd, params);
 			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 +2642,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,14 +2686,19 @@ 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);
-			const resourceLoader = await this.buildResourceLoader(params.cwd, params);
+			const { loader: resourceLoader } = await this.buildResourceLoader(params.cwd, params);
 			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 +2708,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 +3327,4 @@ async function runDaemon() {
 //#endregion
 export { runDaemon };
 
-//# sourceMappingURL=daemon-xclwSgis.mjs.map
+//# sourceMappingURL=daemon-D6QKWz5C.mjs.map