@dougbots/avenor-pi 0.17.0 → 0.18.0

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import { InspectResult, MessageProvider, StatusResult, Supervisor, answerPermissionTool, attachOwner, brokerReceiveTool, dial, eventsTool, getSupervisorClient, inspectTool, killTool, messageTool, observeRun, resultTool, shutdownTool, spawnTool, statusTool, workflowCancelTool, workflowCompleteTool, workflowControllerStatusTool, workflowEventsTool, workflowGateTool, workflowInspectTool, workflowReadyTool, workflowStatusTool, workflowWaitTool } from "@dougbots/avenor-core";
+import { InspectResult, MessageProvider, RunObserver, StatusResult, Supervisor, answerPermissionTool, attachOwner, brokerReceiveTool, dial, eventsTool, getSupervisorClient, inspectTool, killTool, messageTool, observeRun, resultTool, shutdownTool, spawnTool, statusTool, workflowCancelTool, workflowCompleteTool, workflowControllerStatusTool, workflowEventsTool, workflowGateTool, workflowInspectTool, workflowReadyTool, workflowStatusTool, workflowWaitTool } from "@dougbots/avenor-core";
 import { EventBus, ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 //#region src/subagent-broker.d.ts
@@ -173,7 +173,9 @@ interface ExtensionDeps {
   hostProviderFor?: (supervisorId: string | undefined) => Promise<MessageProvider>;
   /** Dial a supervisor for the default host provider; tests inject a fake client. */
   getSupervisorClient?: typeof getSupervisorClient;
-  observeRun: typeof observeRun;
+  /** A watch transport may decline to observe (no runtime id yet), so a test
+   * double that only asserts the call can answer null. */
+  observeRun: (client: Parameters<typeof observeRun>[0], options?: Parameters<typeof observeRun>[1]) => RunObserver | null;
   dial: typeof dial;
   attachOwner: typeof attachOwner;
   Supervisor: typeof Supervisor;

package/dist/index.js

@@ -1987,6 +1987,18 @@ function spawnIdentityMetadata(params, status) {
 function sleep(ms) {
 	return new Promise((resolve) => setTimeout(resolve, ms));
 }
+/** The text of a tool result's first text part; image parts carry none. */
+function firstTextPart(content) {
+	return content?.find((part) => part.type === "text")?.text ?? "";
+}
+/** The status a tool result records; a stored transcript record may carry no details. */
+function resultStatus(details) {
+	return details && typeof details === "object" && "status" in details ? String(details.status ?? "") : "";
+}
+/** Close an observer a concurrent watch assigned, ignoring a close failure. */
+async function closeObserverQuietly(observer) {
+	await observer?.close().catch(() => {});
+}
 function normalizeMultilinePreview(text, limit = COMPLETION_OUTPUT_CHARS) {
 	if (!text) return void 0;
 	const clean = sanitizeText(text).trim();
@@ -2270,7 +2282,7 @@ function createExtension(deps = defaultDeps, options = {}) {
 					refusedSessionName = null;
 				} catch (error) {
 					if (error instanceof BrokerHttpError && error.status === 400 && error.code === "invalid-name") {
-						refusedSessionName = name;
+						refusedSessionName = name ?? null;
 						console.error("[avenor] session_name_set refused:", error.message);
 						return;
 					}
@@ -2911,7 +2923,7 @@ function createExtension(deps = defaultDeps, options = {}) {
 			const outcome = abortPromise ? await Promise.race([waitPromise, abortPromise]) : await waitPromise;
 			if (signal && abortListener) signal.removeEventListener("abort", abortListener);
 			unsubscribe?.();
-			await observer?.close().catch(() => {});
+			await closeObserverQuietly(observer);
 			if (outcome === "aborted") return { aborted: true };
 			try {
 				return {
@@ -3221,10 +3233,13 @@ function createExtension(deps = defaultDeps, options = {}) {
 				wait: Type.Optional(Type.Boolean({ description: "Block until complete (default true)" }))
 			}),
 			async execute(_toolCallId, params, signal, onUpdate, ctx) {
-				if (signal?.aborted) return { content: [{
-					type: "text",
-					text: "Cancelled"
-				}] };
+				if (signal?.aborted) return {
+					content: [{
+						type: "text",
+						text: "Cancelled"
+					}],
+					details: {}
+				};
 				validateSpawnSelection({
 					agent: params.agent,
 					model: params.model,
@@ -3303,10 +3318,11 @@ function createExtension(deps = defaultDeps, options = {}) {
 				if (waitResult.aborted) {
 					const stop = blockingWait.stopped;
 					if (stop && !stop.synthetic) {
-						const probed = asStatusResult(await deps.statusTool({
+						const raw = await deps.statusTool({
 							runId: result.run_id,
 							supervisorId
-						}).catch(() => void 0));
+						}).catch(() => void 0);
+						const probed = raw ? asStatusResult(raw) : void 0;
 						if (probed && (isTerminalStatus(probed.status) || probed.status === "waiting")) waitStatus = probed;
 						else {
 							const tracked = getTrackedRunById(result.run_id, supervisorId);
@@ -3390,7 +3406,7 @@ function createExtension(deps = defaultDeps, options = {}) {
 						run: tracked,
 						runtimeId: status.runtime_id ?? runRef.runtimeId,
 						label,
-						agent: params.agent,
+						agent: tracked?.agent ?? params.agent ?? "unknown",
 						status: status.status,
 						nestedCount: tracked?.nestedCount,
 						backend: status.backend
@@ -3436,17 +3452,11 @@ function createExtension(deps = defaultDeps, options = {}) {
 				return new Text(agent + agentName + label + wait, 0, 0);
 			},
 			renderResult(result, { isPartial }, theme) {
-				if (isPartial) {
-					const status = result.details?.status ?? "running";
-					const emoji = statusEmoji(String(status));
-					return new Text(theme.fg("accent", `${emoji} ${result.content?.[0]?.text ?? "Running…"}`), 0, 0);
-				}
-				const status = String(result.details?.status ?? "");
-				if (status && isTerminalStatus(status)) {
-					const emoji = statusEmoji(status);
-					return new Text(`${theme.fg(status === "done" ? "success" : "error", `${emoji} ${status}`)}\n${(result.content?.[0]?.text ?? "").split("\n").map((line) => theme.fg("muted", `  ${line}`)).join("\n")}`, 0, 0);
-				}
-				return new Text((result.content?.[0]?.text ?? "").split("\n").map((line) => theme.fg("muted", line)).join("\n"), 0, 0);
+				const status = resultStatus(result.details);
+				const text = firstTextPart(result.content);
+				if (isPartial) return new Text(theme.fg("accent", `${statusEmoji(status || "running")} ${text || "Running…"}`), 0, 0);
+				if (status && isTerminalStatus(status)) return new Text(`${theme.fg(status === "done" ? "success" : "error", `${statusEmoji(status)} ${status}`)}\n${text.split("\n").map((line) => theme.fg("muted", `  ${line}`)).join("\n")}`, 0, 0);
+				return new Text(text.split("\n").map((line) => theme.fg("muted", line)).join("\n"), 0, 0);
 			}
 		});
 		pi.registerTool({
@@ -3639,7 +3649,7 @@ function createExtension(deps = defaultDeps, options = {}) {
 		pi.registerTool({
 			name: "avenor_message",
 			label: "Avenor Message",
-			description: "Send a message to another agent run, answer an inbound ask, withdraw a pending ask, or resume a completed run. Every action is explicit and the result states whether a failed ask can still be withdrawn. `control_reply` is available only on the claude-channel surface.",
+			description: "Send a message to another agent run, answer an inbound ask, withdraw a pending ask, or resume a completed run. Every action is explicit and the result states whether a failed ask can still be withdrawn. `control_reply` is retained in the contract but no surface can deliver one.",
 			parameters: Type.Object({
 				action: Type.Union([
 					Type.Literal("tell"),
@@ -3719,7 +3729,7 @@ function createExtension(deps = defaultDeps, options = {}) {
 					workflow_id: params.workflow_id,
 					actor: params.actor,
 					reason: params.reason,
-					supervisor_id: params.supervisor_id
+					supervisorId: params.supervisor_id
 				});
 				return {
 					content: [{

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dougbots/avenor-pi",
-  "version": "0.17.0",
+  "version": "0.18.0",
   "type": "module",
   "keywords": [
     "pi-package"
@@ -31,11 +31,12 @@
   },
   "scripts": {
     "build": "tsdown",
+    "typecheck": "tsc --noEmit",
     "test": "bun test",
     "dev": "tsdown --watch"
   },
   "dependencies": {
-    "@dougbots/avenor-core": "^0.17.0"
+    "@dougbots/avenor-core": "^0.18.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": ">=1.4.0",