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

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

@@ -2,8 +2,8 @@
 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";
@@ -876,6 +876,8 @@ var PiAcpSession = class {
 	lastAssistantStopReason = null;
 	lastEmit = Promise.resolve();
 	unsubscribe;
+	cleanups;
+	pendingDiagnosticsReport;
 	constructor(opts) {
 		this.sessionId = opts.sessionId;
 		this.cwd = opts.cwd;
@@ -883,11 +885,16 @@ var PiAcpSession = class {
 		this.piSession = opts.piSession;
 		this.conn = opts.conn;
 		this.supportsTerminalOutput = opts.supportsTerminalOutput ?? false;
+		this.cleanups = opts.cleanups ?? [];
+		this.pendingDiagnosticsReport = opts.diagnosticsReport !== void 0 && opts.diagnosticsReport !== "" ? opts.diagnosticsReport : null;
 		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.
@@ -925,6 +932,20 @@ var PiAcpSession = class {
 			};
 		});
 		const imageContents = Array.isArray(images) ? images.filter((img) => typeof img === "object" && img !== null && "type" in img && img.type === "image") : [];
+		if (this.pendingDiagnosticsReport !== null) {
+			const report = this.pendingDiagnosticsReport;
+			this.pendingDiagnosticsReport = null;
+			this.conn.sessionUpdate({
+				sessionId: this.sessionId,
+				update: {
+					sessionUpdate: "agent_message_chunk",
+					content: {
+						type: "text",
+						text: `${report}\n`
+					}
+				}
+			}).catch(() => {});
+		}
 		this.piSession.prompt(message, { images: imageContents }).catch(() => {
 			this.flushEmits().finally(() => {
 				const reason = this.cancelRequested ? "cancelled" : "error";
@@ -1340,6 +1361,43 @@ function acpPromptToPiMessage(blocks) {
 	};
 }
 //#endregion
+//#region src/resources/diagnostics.ts
+function buildDiagnosticsReport(input) {
+	const sourceStats = input.sources.map((source) => {
+		const skills = source.getSkills();
+		const prompts = source.getPrompts();
+		const failures = skills.diagnostics.filter((d) => d.type === "warning" || d.type === "error").map((d) => d.message);
+		return {
+			id: source.id,
+			kind: source.kind,
+			agentsFiles: source.getAgentsFiles().length,
+			skills: skills.skills.length,
+			prompts: prompts.prompts.length,
+			failures
+		};
+	});
+	const lines = [];
+	if (sourceStats.length > 0) {
+		lines.push("[pi-acp] resources active:");
+		for (const s of sourceStats) lines.push(`  ${s.id.padEnd(20)} kind=${s.kind} (${s.agentsFiles} AGENTS files, ${s.skills} skills, ${s.prompts} prompts)`);
+	}
+	const allFailures = [];
+	for (const s of sourceStats) for (const f of s.failures) allFailures.push(`  ${s.id.padEnd(20)} ${f}`);
+	for (const d of input.manifestDiagnostics) {
+		const where = d.path !== void 0 ? ` ${d.path}` : "";
+		allFailures.push(`  manifest[${d.source}${where}] ${d.message}`);
+	}
+	if (allFailures.length > 0) {
+		if (lines.length > 0) lines.push("");
+		lines.push("[pi-acp] resource failures:");
+		lines.push(...allFailures);
+	}
+	return {
+		text: lines.join("\n"),
+		sourceStats
+	};
+}
+//#endregion
 //#region src/resources/sources/local.ts
 /**
 * LocalBackend: wraps pi's DefaultResourceLoader for one (cwd, agentDir) root.
@@ -1679,6 +1737,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;
@@ -1897,7 +2007,7 @@ var SshBackend = class {
 //#endregion
 //#region package.json
 var name = "@victor-software-house/pi-acp";
-var version = "0.11.0";
+var version = "0.13.0";
 //#endregion
 //#region src/acp/agent.ts
 /** Builtin ACP slash commands handled directly by the adapter. */
@@ -2034,13 +2144,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;
@@ -2073,7 +2188,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({
@@ -2085,7 +2200,12 @@ 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,
+			diagnosticsEnabled: loaded.manifest.diagnostics === true,
+			manifestDiagnostics: diagnostics
+		};
 	}
 	/**
 	* PRD-002 §FR-6 — `read` tool ACP-FS delegation overlay.
@@ -2158,13 +2278,26 @@ 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);
+		const { loader: resourceLoader, modeResult, diagnosticsEnabled, manifestDiagnostics } = 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 diagnosticsReport = diagnosticsEnabled ? buildDiagnosticsReport({
+			sources: resourceLoader.listSources(),
+			manifestDiagnostics
+		}).text : "";
+		const acpReadOverlay = this.buildAcpReadOverlay(effectiveCwd);
 		let result;
 		try {
-			const resourceLoader = await this.buildResourceLoader(params.cwd, params);
 			result = await createAgentSession({
-				cwd: params.cwd,
+				cwd: effectiveCwd,
 				resourceLoader,
 				...acpReadOverlay ? {
 					tools: acpReadOverlay.tools,
@@ -2172,6 +2305,7 @@ var PiAcpAgent = class {
 				} : {}
 			});
 		} catch (e) {
+			modeResult.cleanup();
 			const authErr = detectAuthError(e);
 			if (authErr !== null) throw authErr;
 			const msg = e instanceof Error ? e.message : String(e);
@@ -2181,6 +2315,7 @@ var PiAcpAgent = class {
 		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();
@@ -2188,11 +2323,13 @@ 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] : [],
+			...diagnosticsReport !== "" ? { diagnosticsReport } : {}
 		});
 		this.sessions.register(session);
 		this.registerWithDaemon({
@@ -2442,7 +2579,7 @@ var PiAcpAgent = class {
 		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,
@@ -2548,7 +2685,7 @@ var PiAcpAgent = class {
 		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,
@@ -2613,7 +2750,7 @@ var PiAcpAgent = class {
 		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,
@@ -3250,4 +3387,4 @@ async function runDaemon() {
 //#endregion
 export { runDaemon };
 
-//# sourceMappingURL=daemon-COFaIaTB.mjs.map
+//# sourceMappingURL=daemon-BdW4nUzA.mjs.map