@narumitw/pi-subagents 3.0.1 → 3.0.2

package/package.json

@@ -1,56 +1,55 @@
 {
-	"name": "@narumitw/pi-subagents",
-	"version": "3.0.1",
-	"description": "Subagent jobs with asynchronous main-agent messaging for Pi.",
-	"type": "module",
-	"license": "MIT",
-	"private": false,
-	"keywords": [
-		"pi-package",
-		"pi-extension",
-		"pi",
-		"subagents",
-		"agents",
-		"delegation"
-	],
-	"files": [
-		"src",
-		"dist",
-		"docs",
-		"README.md",
-		"LICENSE"
-	],
-	"pi": {
-		"extensions": [
-			"./dist/index.ts"
-		]
-	},
-	"scripts": {
-		"build": "node scripts/build-runtime.mjs",
-		"check": "npm run build && biome check . && npm run typecheck",
-		"format": "biome check --write .",
-		"typecheck": "tsc --noEmit",
-		"prepack": "npm run build"
-	},
-	"peerDependencies": {
-		"@earendil-works/pi-ai": "*",
-		"@earendil-works/pi-coding-agent": "*",
-		"@earendil-works/pi-tui": "*",
-		"typebox": "*"
-	},
-	"devDependencies": {
-		"@biomejs/biome": "2.5.11",
-		"@earendil-works/pi-ai": "0.84.4",
-		"@earendil-works/pi-coding-agent": "0.84.4",
-		"@earendil-works/pi-tui": "0.84.4",
-		"@types/node": "26.4.0",
-		"esbuild": "0.28.2",
-		"typebox": "1.3.20",
-		"typescript": "7.0.2"
-	},
-	"repository": {
-		"type": "git",
-		"url": "https://github.com/narumiruna/pi-extensions",
-		"directory": "packages/pi-subagents"
-	}
+  "name": "@narumitw/pi-subagents",
+  "version": "3.0.2",
+  "description": "Subagent jobs with asynchronous main-agent messaging for Pi.",
+  "type": "module",
+  "license": "MIT",
+  "private": false,
+  "keywords": [
+    "pi-package",
+    "pi-extension",
+    "pi",
+    "subagents",
+    "agents",
+    "delegation"
+  ],
+  "files": [
+    "src",
+    "dist",
+    "docs",
+    "README.md",
+    "LICENSE"
+  ],
+  "pi": {
+    "extensions": [
+      "./dist/index.ts"
+    ]
+  },
+  "scripts": {
+    "build": "node scripts/build-runtime.mjs",
+    "check": "npm run build && biome check . && npm run typecheck",
+    "format": "biome check --write .",
+    "typecheck": "tsc --noEmit",
+    "prepack": "npm run build"
+  },
+  "peerDependencies": {
+    "@earendil-works/pi-ai": "*",
+    "@earendil-works/pi-coding-agent": "*",
+    "@earendil-works/pi-tui": "*",
+    "typebox": "*"
+  },
+  "devDependencies": {
+    "@earendil-works/pi-ai": "0.86.0",
+    "@earendil-works/pi-coding-agent": "0.86.0",
+    "@earendil-works/pi-tui": "0.86.0",
+    "@types/node": "26.6.2",
+    "esbuild": "0.28.2",
+    "typebox": "1.3.34",
+    "typescript": "7.0.2"
+  },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/narumiruna/pi-extensions",
+    "directory": "packages/pi-subagents"
+  }
 }

package/src/broker-credentials.ts

@@ -7,65 +7,65 @@ export const BROKER_CREDENTIAL_FD_ENV = "PI_SUBAGENT_BROKER_FD";
 const MAX_CREDENTIAL_BYTES = 1_024;
 
 export function brokerCredentialEnvironment(): NodeJS.ProcessEnv {
-	return { [BROKER_CREDENTIAL_FD_ENV]: String(BROKER_CREDENTIAL_FD) };
+  return { [BROKER_CREDENTIAL_FD_ENV]: String(BROKER_CREDENTIAL_FD) };
 }
 
 export function serializeBrokerCredentials(credentials: BrokerCredentials): string {
-	return JSON.stringify(credentials);
+  return JSON.stringify(credentials);
 }
 
 export function captureBrokerCredentials(
-	readCredentials: () => string = readCredentialPipe,
+  readCredentials: () => string = readCredentialPipe,
 ): BrokerCredentials | undefined {
-	const descriptor = process.env[BROKER_CREDENTIAL_FD_ENV];
-	delete process.env[BROKER_CREDENTIAL_FD_ENV];
-	if (descriptor === undefined) return undefined;
-	if (descriptor !== String(BROKER_CREDENTIAL_FD)) {
-		throw new Error("Invalid pi-subagents broker credential descriptor.");
-	}
-	let serialized: string;
-	try {
-		serialized = readCredentials();
-	} catch {
-		throw new Error("Unable to read pi-subagents broker credentials.");
-	}
-	if (Buffer.byteLength(serialized, "utf8") > MAX_CREDENTIAL_BYTES) {
-		throw new Error("Invalid pi-subagents broker credentials.");
-	}
-	let value: unknown;
-	try {
-		value = JSON.parse(serialized);
-	} catch {
-		throw new Error("Invalid pi-subagents broker credentials.");
-	}
-	if (!isBrokerCredentials(value)) {
-		throw new Error("Invalid pi-subagents broker credentials.");
-	}
-	return value;
+  const descriptor = process.env[BROKER_CREDENTIAL_FD_ENV];
+  delete process.env[BROKER_CREDENTIAL_FD_ENV];
+  if (descriptor === undefined) return undefined;
+  if (descriptor !== String(BROKER_CREDENTIAL_FD)) {
+    throw new Error("Invalid pi-subagents broker credential descriptor.");
+  }
+  let serialized: string;
+  try {
+    serialized = readCredentials();
+  } catch {
+    throw new Error("Unable to read pi-subagents broker credentials.");
+  }
+  if (Buffer.byteLength(serialized, "utf8") > MAX_CREDENTIAL_BYTES) {
+    throw new Error("Invalid pi-subagents broker credentials.");
+  }
+  let value: unknown;
+  try {
+    value = JSON.parse(serialized);
+  } catch {
+    throw new Error("Invalid pi-subagents broker credentials.");
+  }
+  if (!isBrokerCredentials(value)) {
+    throw new Error("Invalid pi-subagents broker credentials.");
+  }
+  return value;
 }
 
 function readCredentialPipe(): string {
-	try {
-		return fs.readFileSync(BROKER_CREDENTIAL_FD, "utf8");
-	} finally {
-		try {
-			fs.closeSync(BROKER_CREDENTIAL_FD);
-		} catch {
-			// The descriptor may already be closed after a failed read.
-		}
-	}
+  try {
+    return fs.readFileSync(BROKER_CREDENTIAL_FD, "utf8");
+  } finally {
+    try {
+      fs.closeSync(BROKER_CREDENTIAL_FD);
+    } catch {
+      // The descriptor may already be closed after a failed read.
+    }
+  }
 }
 
 function isBrokerCredentials(value: unknown): value is BrokerCredentials {
-	if (!value || typeof value !== "object" || Array.isArray(value)) return false;
-	const candidate = value as Record<string, unknown>;
-	return (
-		Object.keys(candidate).length === 3 &&
-		candidate.host === "127.0.0.1" &&
-		Number.isSafeInteger(candidate.port) &&
-		(candidate.port as number) >= 1 &&
-		(candidate.port as number) <= 65_535 &&
-		typeof candidate.token === "string" &&
-		/^[a-f0-9]{64}$/u.test(candidate.token)
-	);
+  if (!value || typeof value !== "object" || Array.isArray(value)) return false;
+  const candidate = value as Record<string, unknown>;
+  return (
+    Object.keys(candidate).length === 3 &&
+    candidate.host === "127.0.0.1" &&
+    Number.isSafeInteger(candidate.port) &&
+    (candidate.port as number) >= 1 &&
+    (candidate.port as number) <= 65_535 &&
+    typeof candidate.token === "string" &&
+    /^[a-f0-9]{64}$/u.test(candidate.token)
+  );
 }

package/src/child-communication-bridge.ts

@@ -1,10 +1,7 @@
 import net from "node:net";
 import type { ExtensionFactory } from "@earendil-works/pi-coding-agent";
 import { captureBrokerCredentials } from "./broker-credentials.js";
-import {
-	type ChildCommunicationClient,
-	createChildCommunicationExtension,
-} from "./child-communication-tools.js";
+import { type ChildCommunicationClient, createChildCommunicationExtension } from "./child-communication-tools.js";
 import { MAX_FRAME_BYTES, MAX_IDENTIFIER_LENGTH } from "./message-broker.js";
 import type { BrokerCredentials } from "./types.js";
 
@@ -14,140 +11,135 @@ const SEND_RESPONSE_TIMEOUT_MS = 5_000;
 const captured = captureBrokerCredentials();
 
 const childCommunicationBridge: ExtensionFactory = captured
-	? createChildCommunicationExtension(createBrokerClient(captured))
-	: () => undefined;
+  ? createChildCommunicationExtension(createBrokerClient(captured))
+  : () => undefined;
 
 export default childCommunicationBridge;
 
 export function createBrokerClient(credentials: BrokerCredentials): ChildCommunicationClient {
-	return {
-		async send(params, signal) {
-			const response = await requestBroker(
-				credentials,
-				{ type: "send", token: credentials.token, ...params },
-				signal,
-				SEND_RESPONSE_TIMEOUT_MS,
-			);
-			if (response.ok !== true) throw brokerError(response);
-			if (
-				typeof response.requestId !== "string" ||
-				!response.requestId ||
-				response.requestId.length > MAX_IDENTIFIER_LENGTH ||
-				typeof response.accepted !== "boolean" ||
-				typeof response.duplicate !== "boolean"
-			) {
-				throw new Error("Subagent broker returned an invalid send acknowledgement.");
-			}
-			return {
-				requestId: response.requestId,
-				accepted: response.accepted,
-				duplicate: response.duplicate,
-			};
-		},
-		async wait(requestId, timeoutMs, signal) {
-			const response = await requestBroker(
-				credentials,
-				{
-					type: "wait",
-					token: credentials.token,
-					requestId,
-					...(timeoutMs !== undefined ? { timeoutMs } : {}),
-				},
-				signal,
-			);
-			if (response.ok !== true) throw brokerError(response);
-			if (typeof response.response !== "string") {
-				throw new Error("Subagent broker returned an invalid plain-text response.");
-			}
-			return response.response;
-		},
-	};
+  return {
+    async send(params, signal) {
+      const response = await requestBroker(
+        credentials,
+        { type: "send", token: credentials.token, ...params },
+        signal,
+        SEND_RESPONSE_TIMEOUT_MS,
+      );
+      if (response.ok !== true) throw brokerError(response);
+      if (
+        typeof response.requestId !== "string" ||
+        !response.requestId ||
+        response.requestId.length > MAX_IDENTIFIER_LENGTH ||
+        typeof response.accepted !== "boolean" ||
+        typeof response.duplicate !== "boolean"
+      ) {
+        throw new Error("Subagent broker returned an invalid send acknowledgement.");
+      }
+      return {
+        requestId: response.requestId,
+        accepted: response.accepted,
+        duplicate: response.duplicate,
+      };
+    },
+    async wait(requestId, timeoutMs, signal) {
+      const response = await requestBroker(
+        credentials,
+        {
+          type: "wait",
+          token: credentials.token,
+          requestId,
+          ...(timeoutMs !== undefined ? { timeoutMs } : {}),
+        },
+        signal,
+      );
+      if (response.ok !== true) throw brokerError(response);
+      if (typeof response.response !== "string") {
+        throw new Error("Subagent broker returned an invalid plain-text response.");
+      }
+      return response.response;
+    },
+  };
 }
 
 function requestBroker(
-	credentials: BrokerCredentials,
-	request: Record<string, unknown>,
-	signal?: AbortSignal,
-	responseTimeoutMs?: number,
+  credentials: BrokerCredentials,
+  request: Record<string, unknown>,
+  signal?: AbortSignal,
+  responseTimeoutMs?: number,
 ): Promise<Record<string, unknown>> {
-	if (signal?.aborted) return Promise.reject(abortError("Subagent broker request was cancelled."));
-	return new Promise((resolve, reject) => {
-		const socket = net.createConnection({ host: credentials.host, port: credentials.port });
-		let response = Buffer.alloc(0);
-		let settled = false;
-		let responseTimer: NodeJS.Timeout | undefined;
-		const connectTimer = setTimeout(
-			() => finish(new Error("Subagent broker connection timed out.")),
-			CONNECT_TIMEOUT_MS,
-		);
-		connectTimer.unref();
-		const onAbort = () => finish(abortError("Subagent broker request was cancelled."));
-		const finish = (error?: Error, value?: Record<string, unknown>) => {
-			if (settled) return;
-			settled = true;
-			clearTimeout(connectTimer);
-			if (responseTimer) clearTimeout(responseTimer);
-			signal?.removeEventListener("abort", onAbort);
-			socket.destroy();
-			if (error) reject(error);
-			else resolve(value ?? {});
-		};
-		signal?.addEventListener("abort", onAbort, { once: true });
-		socket.once("connect", () => {
-			clearTimeout(connectTimer);
-			if (responseTimeoutMs !== undefined) {
-				responseTimer = setTimeout(
-					() => finish(new Error("Subagent broker response timed out.")),
-					responseTimeoutMs,
-				);
-				responseTimer.unref();
-			}
-			socket.write(`${JSON.stringify(request)}\n`);
-		});
-		socket.on("data", (chunk: Buffer) => {
-			if (settled) return;
-			response = Buffer.concat([response, chunk]);
-			if (response.byteLength > MAX_FRAME_BYTES) {
-				finish(new Error("Subagent broker response exceeded its size limit."));
-				return;
-			}
-			const newline = response.indexOf(0x0a);
-			if (newline < 0) return;
-			const trailing = response
-				.subarray(newline + 1)
-				.toString("utf8")
-				.trim();
-			if (trailing) {
-				finish(new Error("Subagent broker returned more than one response frame."));
-				return;
-			}
-			try {
-				const parsed = JSON.parse(response.subarray(0, newline).toString("utf8")) as unknown;
-				if (!isRecord(parsed)) throw new Error();
-				finish(undefined, parsed);
-			} catch {
-				finish(new Error("Subagent broker returned malformed JSON."));
-			}
-		});
-		socket.once("error", (error) => finish(error));
-		socket.once("close", () => {
-			if (!settled) finish(new Error("Subagent broker closed without a response."));
-		});
-	});
+  if (signal?.aborted) return Promise.reject(abortError("Subagent broker request was cancelled."));
+  return new Promise((resolve, reject) => {
+    const socket = net.createConnection({ host: credentials.host, port: credentials.port });
+    let response = Buffer.alloc(0);
+    let settled = false;
+    let responseTimer: NodeJS.Timeout | undefined;
+    const connectTimer = setTimeout(
+      () => finish(new Error("Subagent broker connection timed out.")),
+      CONNECT_TIMEOUT_MS,
+    );
+    connectTimer.unref();
+    const onAbort = () => finish(abortError("Subagent broker request was cancelled."));
+    const finish = (error?: Error, value?: Record<string, unknown>) => {
+      if (settled) return;
+      settled = true;
+      clearTimeout(connectTimer);
+      if (responseTimer) clearTimeout(responseTimer);
+      signal?.removeEventListener("abort", onAbort);
+      socket.destroy();
+      if (error) reject(error);
+      else resolve(value ?? {});
+    };
+    signal?.addEventListener("abort", onAbort, { once: true });
+    socket.once("connect", () => {
+      clearTimeout(connectTimer);
+      if (responseTimeoutMs !== undefined) {
+        responseTimer = setTimeout(() => finish(new Error("Subagent broker response timed out.")), responseTimeoutMs);
+        responseTimer.unref();
+      }
+      socket.write(`${JSON.stringify(request)}\n`);
+    });
+    socket.on("data", (chunk: Buffer) => {
+      if (settled) return;
+      response = Buffer.concat([response, chunk]);
+      if (response.byteLength > MAX_FRAME_BYTES) {
+        finish(new Error("Subagent broker response exceeded its size limit."));
+        return;
+      }
+      const newline = response.indexOf(0x0a);
+      if (newline < 0) return;
+      const trailing = response
+        .subarray(newline + 1)
+        .toString("utf8")
+        .trim();
+      if (trailing) {
+        finish(new Error("Subagent broker returned more than one response frame."));
+        return;
+      }
+      try {
+        const parsed = JSON.parse(response.subarray(0, newline).toString("utf8")) as unknown;
+        if (!isRecord(parsed)) throw new Error();
+        finish(undefined, parsed);
+      } catch {
+        finish(new Error("Subagent broker returned malformed JSON."));
+      }
+    });
+    socket.once("error", (error) => finish(error));
+    socket.once("close", () => {
+      if (!settled) finish(new Error("Subagent broker closed without a response."));
+    });
+  });
 }
 
 function brokerError(response: Record<string, unknown>): Error {
-	return new Error(
-		typeof response.error === "string" ? response.error : "Subagent broker rejected the request.",
-	);
+  return new Error(typeof response.error === "string" ? response.error : "Subagent broker rejected the request.");
 }
 
 function abortError(message: string): Error {
-	const error = new Error(message);
-	error.name = "AbortError";
-	return error;
+  const error = new Error(message);
+  error.name = "AbortError";
+  return error;
 }
 
 function isRecord(value: unknown): value is Record<string, unknown> {
-	return typeof value === "object" && value !== null && !Array.isArray(value);
+  return typeof value === "object" && value !== null && !Array.isArray(value);
 }