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

package/dist/{daemon-tHPrf3qs.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
@@ -1652,6 +1679,121 @@ function tryFromFile(path, source, diagnostics) {
 	return null;
 }
 //#endregion
+//#region src/resources/sources/http.ts
+const DEFAULT_CACHE_TTL_SECONDS = 300;
+const DEFAULT_TIMEOUT_MS$1 = 5e3;
+var HttpBackend = class {
+	id;
+	kind = "http";
+	baseUrl;
+	paths;
+	cacheTtlMs;
+	timeoutMs;
+	fetchImpl;
+	urlCache = /* @__PURE__ */ new Map();
+	cache = null;
+	constructor(opts) {
+		if (!opts.baseUrl.startsWith("https://")) throw new Error(`pi-acp http source '${opts.id}': baseUrl must use https:// (got "${opts.baseUrl}")`);
+		this.id = opts.id;
+		this.baseUrl = opts.baseUrl.replace(/\/+$/, "");
+		this.paths = opts.paths ?? {};
+		this.cacheTtlMs = (opts.cacheTtlSeconds ?? DEFAULT_CACHE_TTL_SECONDS) * 1e3;
+		this.timeoutMs = opts.timeoutMs ?? DEFAULT_TIMEOUT_MS$1;
+		this.fetchImpl = opts.fetchImpl ?? globalThis.fetch.bind(globalThis);
+	}
+	async reload() {
+		const diagnostics = [];
+		for (const kind of [
+			"skills",
+			"prompts",
+			"extensions"
+		]) if (this.paths[kind] !== void 0) diagnostics.push(this.unsupportedDiagnostic(kind));
+		const list = this.paths.agentsFiles ?? [];
+		const files = [];
+		if (list.length > 0) {
+			const results = await Promise.all(list.map((path) => this.fetchPath(path).then((content) => ({
+				path,
+				content,
+				error: null
+			}), (err) => ({
+				path,
+				content: null,
+				error: err instanceof Error ? err.message : String(err)
+			}))));
+			for (const r of results) {
+				if (r.content !== null) {
+					files.push({
+						path: this.qualifyPath(r.path),
+						content: r.content
+					});
+					continue;
+				}
+				diagnostics.push({
+					type: "warning",
+					message: `pi-acp http source '${this.id}' (${this.baseUrl}): agentsFile '${r.path}' unreadable — ${r.error ?? "(unknown)"}`,
+					path: r.path
+				});
+			}
+		}
+		this.cache = {
+			files,
+			diagnostics
+		};
+	}
+	getAgentsFiles() {
+		return this.cache?.files ?? [];
+	}
+	getSkills() {
+		return {
+			skills: [],
+			diagnostics: this.cache?.diagnostics ?? []
+		};
+	}
+	getPrompts() {
+		return {
+			prompts: [],
+			diagnostics: []
+		};
+	}
+	getSystemPrompt() {}
+	getAppendSystemPrompt() {
+		return [];
+	}
+	qualifyPath(path) {
+		return `${this.baseUrl}/${path.replace(/^\/+/, "")}`;
+	}
+	unsupportedDiagnostic(kind) {
+		return {
+			type: "warning",
+			message: `pi-acp http source '${this.id}' (${this.baseUrl}): ${kind} discovery over HTTP not yet implemented — declare individual files via paths.agentsFiles for now, or omit paths.${kind}.`
+		};
+	}
+	async fetchPath(path) {
+		const url = this.qualifyPath(path);
+		const now = Date.now();
+		const cached = this.urlCache.get(url);
+		if (cached !== void 0 && cached.expiresAt > now) return cached.content;
+		const controller = new AbortController();
+		const timer = setTimeout(() => controller.abort(), this.timeoutMs);
+		let response;
+		try {
+			response = await this.fetchImpl(url, { signal: controller.signal });
+		} catch (err) {
+			if (err instanceof Error && err.name === "AbortError") throw new Error(`fetch timed out after ${this.timeoutMs}ms`);
+			throw err;
+		} finally {
+			clearTimeout(timer);
+		}
+		if (!response.ok) throw new Error(`HTTP ${response.status} ${response.statusText || ""}`.trim());
+		const content = await response.text();
+		this.urlCache.set(url, {
+			content,
+			expiresAt: now + this.cacheTtlMs
+		});
+		return content;
+	}
+};
+//#endregion
 //#region src/resources/sources/ssh.ts
 const DEFAULT_TIMEOUT_MS = 5e3;
 var SshBackend = class {
@@ -1755,7 +1897,7 @@ var SshBackend = class {
 //#endregion
 //#region package.json
 var name = "@victor-software-house/pi-acp";
-var version = "0.9.0";
+var version = "0.11.0";
 //#endregion
 //#region src/acp/agent.ts
 /** Builtin ACP slash commands handled directly by the adapter. */
@@ -1844,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. */
@@ -1883,8 +2026,8 @@ var PiAcpAgent = class {
 	* least one LocalBackend is present for the extension / theme passthrough.
 	*
 	* Phase 5 materializes `kind: "local"`. Phase 6 adds `kind: "ssh"`.
-	* `http` and `acp-fs` still parse fine but surface as diagnostics until
-	* their backends land in subsequent phases.
+	* Phase 7 adds `kind: "http"`. `acp-fs` still parses fine but surfaces as
+	* a diagnostic until its backend lands in a subsequent phase.
 	*/
 	async buildResourceLoader(cwd, sessionParams) {
 		const loaded = await loadManifest({
@@ -1912,6 +2055,16 @@ var PiAcpAgent = class {
 				sources.push(new SshBackend(sshOpts));
 				continue;
 			}
+			if (root.kind === "http") {
+				const httpOpts = {
+					id: root.id,
+					baseUrl: root.baseUrl,
+					paths: root.paths
+				};
+				if (root.cache !== void 0) httpOpts.cacheTtlSeconds = root.cache.ttl;
+				sources.push(new HttpBackend(httpOpts));
+				continue;
+			}
 			const diag = {
 				source: loaded.source,
 				message: `root "${root.id}" kind="${root.kind}" not yet supported in this build (skipped)`
@@ -1934,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;
@@ -1968,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);
@@ -1982,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.");
@@ -2241,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);
@@ -2248,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);
@@ -2257,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,
@@ -2341,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);
@@ -2348,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);
@@ -2357,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,
@@ -2400,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);
@@ -2407,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);
@@ -2417,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({
@@ -3035,4 +3250,4 @@ async function runDaemon() {
 //#endregion
 export { runDaemon };
 
-//# sourceMappingURL=daemon-tHPrf3qs.mjs.map
+//# sourceMappingURL=daemon-COFaIaTB.mjs.map