@dougbots/avenor-pi 0.6.0 → 0.7.1

package/README.md

@@ -151,6 +151,11 @@ When `agent` is set, avenor passes `PI_AGENT=<name>` to the pi subprocess. The `
 
 If you don't need a named profile, `model` alone is sufficient — no agent config or extension setup required.
 
+When `pi-agents` has selected a session-scoped `/agent-profile`, this extension
+also forwards that optional session metadata to Pi automatically. Avenor does
+not depend on `pi-agents`; absent or malformed metadata is ignored. The setting
+is not an `avenor_spawn` parameter and applies only to Pi-backed child runs.
+
 ## Typical workflows
 
 **Blocking (default):**

package/dist/index.js

@@ -552,6 +552,41 @@ function formatRunLine(entry) {
 	const phase = entry.phaseLabel ? ` (${entry.phaseLabel})` : "";
 	return `${emoji} ${entry.label} (${entry.agent}) — ${entry.status}${phase}${perm}`;
 }
+/**
+* decideCompletion determines whether to skip, defer, or send the completion
+* steering message for a tracked run that has reached terminal status.
+*
+* The deferral prevents duplication when avenor_result consumes the result
+* in the same turn: if the agent calls avenor_result between the defer and
+* the next tick, the run is deleted from tracking and the send never fires.
+*/
+function decideCompletion(run) {
+	if (run.blocking) return "skip";
+	if (!run.completionPending) return "defer";
+	return "send";
+}
+//#endregion
+//#region src/agent-profile.ts
+const PROFILE_ENTRY_TYPE = "pi-agents:profile";
+/**
+* Read the optional pi-agents session convention without depending on its
+* package. A namespaced environment variable is an optional launch override.
+*/
+function resolveAgentProfile(entries, environment = process.env) {
+	const fromEnvironment = environment.PI_AGENT_PROFILE;
+	if (fromEnvironment) return fromEnvironment;
+	let selected;
+	for (const entry of entries ?? []) {
+		if (!entry || typeof entry !== "object") continue;
+		const custom = entry;
+		if (custom.type !== "custom" || custom.customType !== PROFILE_ENTRY_TYPE) continue;
+		if (!custom.data || typeof custom.data !== "object") continue;
+		const name = custom.data.name;
+		if (typeof name === "string") selected = name;
+		if (name === null) selected = void 0;
+	}
+	return selected;
+}
 //#endregion
 //#region src/index.ts
 const POLL_INTERVAL_MS = 3e3;
@@ -703,7 +738,8 @@ function createExtension(deps = defaultDeps) {
 				startTime: current?.startTime ?? Date.now(),
 				lastStatus: status ?? current?.lastStatus,
 				blocking: current?.blocking ?? blocking,
-				permissionNotified: current?.permissionNotified
+				permissionNotified: current?.permissionNotified,
+				completionPending: current?.completionPending
 			});
 		}
 		async function resolveFinalOutput(run, status) {
@@ -823,19 +859,22 @@ function createExtension(deps = defaultDeps) {
 					if (!run) continue;
 					if (isTerminalStatus(entry.status)) {
 						const prevStatus = prevStatuses.get(entry.runId);
-						if (!prevStatus || !isTerminalStatus(prevStatus)) {
-							sessionCtx?.ui.notify(`Sub-agent "${entry.label}" finished: ${entry.status}`, entry.status === "done" ? "info" : "warning");
-							if (!run.blocking && run.lastStatus) {
-								const finalOutput = await resolveFinalOutput(run, run.lastStatus);
-								try {
-									pi.sendUserMessage(buildCompletionText({
-										runId: run.runId,
-										label: run.label
-									}, run.lastStatus, finalOutput), { deliverAs: "followUp" });
-								} catch (err) {
-									console.error("avenor tick: sendUserMessage failed", err);
-								}
-							}
+						if (!(prevStatus && isTerminalStatus(prevStatus))) sessionCtx?.ui.notify(`Sub-agent "${entry.label}" finished: ${entry.status}`, entry.status === "done" ? "info" : "warning");
+						if (run.blocking) continue;
+						const action = decideCompletion(run);
+						if (action === "defer") {
+							run.completionPending = true;
+							continue;
+						}
+						if (action === "skip") continue;
+						const finalOutput = await resolveFinalOutput(run, run.lastStatus);
+						try {
+							pi.sendUserMessage(buildCompletionText({
+								runId: run.runId,
+								label: run.label
+							}, run.lastStatus, finalOutput), { deliverAs: "followUp" });
+						} catch (err) {
+							console.error("avenor tick: sendUserMessage failed", err);
 						}
 						trackedRuns.delete(entry.runId);
 					} else if (entry.pendingPermission && !run.blocking && !run.permissionNotified) {
@@ -1134,6 +1173,8 @@ function createExtension(deps = defaultDeps) {
 				const dir = params.dir ?? ctx.cwd;
 				const label = params.label ?? `${params.agent}-${Date.now()}`;
 				const backend = params.backend ?? "pi";
+				const entries = ctx.sessionManager?.getEntries?.();
+				const agentProfile = backend === "pi" ? resolveAgentProfile(entries) : void 0;
 				const result = await deps.spawnTool({
 					agent: params.agent,
 					prompt: params.prompt,
@@ -1143,6 +1184,7 @@ function createExtension(deps = defaultDeps) {
 					timeout: params.timeout,
 					model: params.model,
 					backend,
+					agentProfile,
 					serverUrl: params.server_url,
 					supervisorId: params.supervisor_id
 				});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dougbots/avenor-pi",
-  "version": "0.6.0",
+  "version": "0.7.1",
   "type": "module",
   "keywords": [
     "pi-package"
@@ -35,7 +35,7 @@
     "dev": "tsdown --watch"
   },
   "dependencies": {
-    "@dougbots/avenor-core": "^0.6.0"
+    "@dougbots/avenor-core": "^0.7.1"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",