pi-subagents 0.83.0 → 0.83.1

package/bundle/intercom-bridge.js

@@ -1,11 +1,11 @@
 import {
   resolveIntercomSessionTarget
-} from "./chunk-MHX22USZ.js";
+} from "./chunk-C5P5CHYK.js";
 import "./chunk-7F6RKT6W.js";
-import "./chunk-7F5SK2U3.js";
+import "./chunk-LNRJCP2Y.js";
 import "./chunk-JEZOSEWG.js";
 import "./chunk-YFCK7H2S.js";
-import "./chunk-Z7KVHQHL.js";
+import "./chunk-QXASAHIA.js";
 export {
   resolveIntercomSessionTarget
 };

package/bundle/preflight.js

@@ -2,22 +2,22 @@ import {
   buildEffectiveSystemPrompt,
   canPreferForkFromSnapshot,
   resolveSubagentLaunchContext
-} from "./chunk-HLE22FSH.js";
-import "./chunk-Y3I6H6DV.js";
+} from "./chunk-SSG7IEP5.js";
+import "./chunk-LFDBD4PW.js";
 import {
   loadConfig
-} from "./chunk-RZJN63GU.js";
+} from "./chunk-QP5ZWNIS.js";
 import {
   nestedResultsPath,
   processTerminalCandidatePath,
   processTerminalPath,
   resultFilePath
-} from "./chunk-5ESXK62X.js";
+} from "./chunk-RNIZLSEA.js";
 import {
   applyIntercomBridgeToAgent,
   resolveIntercomBridge,
   validateIntercomBridgeConfig
-} from "./chunk-MHX22USZ.js";
+} from "./chunk-C5P5CHYK.js";
 import {
   AGENT_DEFINITION_PROJECTION_VERSION,
   resolveLaunchBinding,
@@ -26,7 +26,7 @@ import {
 import {
   applyThinkingSuffix,
   resolvePiLaunchToolPlan
-} from "./chunk-CSSZRTKY.js";
+} from "./chunk-V2BZYLEP.js";
 import {
   assertRequiredChildExtensionsAdmitted,
   resolveRequiredChildExtensions
@@ -34,7 +34,7 @@ import {
 import {
   getArtifactPaths,
   getArtifactsDir
-} from "./chunk-VTMNBPCQ.js";
+} from "./chunk-D3JAFGMO.js";
 import {
   assertClaudeCodeModelScope,
   assertThinkingWithinCeiling,
@@ -61,7 +61,7 @@ import {
   resolveSkillsWithFallback,
   resolveStepBehavior,
   unknownAgentDiagnosticContext
-} from "./chunk-7F5SK2U3.js";
+} from "./chunk-LNRJCP2Y.js";
 import {
   DIRS,
   SUBAGENT_LIFECYCLE_ARTIFACT_VERSION,
@@ -71,7 +71,7 @@ import "./chunk-YFCK7H2S.js";
 import {
   capabilityCeilingAgentRestrictionMessage,
   intersectSubagentCapabilityCeilings
-} from "./chunk-Z7KVHQHL.js";
+} from "./chunk-QXASAHIA.js";
 
 // src/api/preflight.ts
 import * as fs from "node:fs";

package/bundle/project-panes.js

@@ -8,13 +8,13 @@ import {
   openProjectPane,
   projectPaneBindingPath,
   readProjectPaneBinding
-} from "./chunk-WG2OTTAK.js";
+} from "./chunk-2DHZTCOL.js";
 import "./chunk-7J4ZXMY7.js";
-import "./chunk-VTMNBPCQ.js";
-import "./chunk-7F5SK2U3.js";
+import "./chunk-D3JAFGMO.js";
+import "./chunk-LNRJCP2Y.js";
 import "./chunk-JEZOSEWG.js";
 import "./chunk-YFCK7H2S.js";
-import "./chunk-Z7KVHQHL.js";
+import "./chunk-QXASAHIA.js";
 export {
   PROJECT_PANES_API_VERSION,
   PROJECT_PANE_TRUST_STATUS,

package/docs/configuration.md

@@ -202,7 +202,18 @@ With `"summary"`, a tool result looks like this:
 
 Set to `false` to register none of the pi-subagents slash commands (`/run`, `/subagents`, `/subagents-fleet`, `/subagents-watchdog`, `/prompt-workflow`, and the rest) and no [`foregroundDetachShortcut`](#foregrounddetachshortcut). The `subagent` tool, RPC, the async widget, and the extension's other features are unchanged. Prompt templates whose frontmatter has `chain:` launch through `/prompt-workflow`, so they stop working too. Defaults to `true`. Restart Pi or run `/reload` after changing it.
 
-The package's prompt shortcuts, such as `/parallel-review` and `/review-loop`, are Pi prompt templates, not commands, so this setting leaves them alone. Pi's package filter removes them: `{ "packages": [{ "source": "npm:pi-subagents", "prompts": [] }] }` in Pi's `settings.json`, or `pi config`.
+The package's prompt shortcuts, such as `/parallel-review` and `/review-loop`, are Pi prompt templates, not commands, so this setting leaves them alone. The same goes for its skills (`pi-subagents` and `council-mode`) and built-in agents. To turn them off, use Pi's package filter for prompts and skills and `disableBuiltins` for the agents, in Pi's `settings.json`. This turns off every prompt, the `council-mode` skill and every built-in agent:
+
+```json
+{
+  "packages": [
+    { "source": "npm:pi-subagents", "prompts": [], "skills": ["!council-mode"] }
+  ],
+  "subagents": { "disableBuiltins": true }
+}
+```
+
+A `!` entry such as `"!review-loop.md"` drops one file, and `[]` drops all of that type. `subagents.agentOverrides.<name>.disabled: true` turns off one built-in agent instead of all of them ([agents.md](agents.md)). `pi config` toggles the package's prompts and skills interactively.
 
 ## `foregroundDetachShortcut`
 

package/docs/extension-api.md

@@ -430,6 +430,8 @@ const restriction = registerSubagentCapabilityCeiling({
 // restriction.dispose() removes only this provider's registration.
 ```
 
+Register with **`ctx.sessionManager.getSessionId()`**; a ceiling registered by session file path does not apply.
+
 Semantics:
 
 - Active registrations intersect their `allowedTools` and `allowedAgents` sets and OR `denyExtensions`.

package/docs/missions.md

@@ -13,6 +13,8 @@ Missions are durable wrappers around runs. The noun map:
 
 Ordinary workflow launches create one enclosing mission by default, with detailed JSON records under `~/.pi/agent/missions/projects/<project-hash>/` linking objectives, run ids, lifecycle status, decisions, artifact paths, and delivery receipts. Workflow children do not create separate missions. Each workflow child attempt is stored in the enclosing mission with its stable workflow key, run id when known, agent, task metadata, timestamps, session and artifact paths, and latest status heartbeat.
 
+A finished run completes a mission you asked for with `mission` or `missionId` only when its result was accepted. If the run, or any workflow child in the mission, ends with acceptance `review-required`, `rejected`, `claimed`, or `pending`, the mission stays `waiting` and `mission.show` lists each child's acceptance status. Check the result and close the mission with `mission.close`. A `gate` or `acceptance.verify` command lets the runtime accept the result, except for writer runs: they require a review by default and stay `review-required` without one. When the command is enough proof, use `acceptance: { level: "verified", verify: [{ id: "check", command: "..." }], review: { required: false } }` instead of `gate`. Missions made by default for ordinary launches still complete when their run finishes.
+
 Records created under the old default `<project>/.pi/subagents/missions` stay on disk. Continue them by setting `missions.directory` to that path for the project or by copying the record into the new agent-dir project store. There is no automatic migration.
 
 Behavior:

package/docs/tool-reference.md

@@ -344,7 +344,7 @@ subagent({ action: "doctor" })
 - `view: "transcript"` tails the selected run's live `output-<index>.log` or persisted session transcript, with `lines` capped at 500.
 - Nested status shows the root/parent path, nested children, session/artifact paths when known, and nested control commands.
 - Inside child-safe fanout mode, bare `status` requires an id when no local foreground run is active, so children cannot enumerate unrelated top-level async runs.
-- Bare `interrupt` still targets only the visible top-level run; interrupting a nested run requires its explicit nested id.
+- `interrupt` requires an id, like `stop`, `steer`, and `resume`; for a nested run, pass its nested id.
 
 ### Command controls
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "pi-subagents",
-  "version": "0.83.0",
+  "version": "0.83.1",
   "description": "Pi extension for single-agent delegation and scripted multi-agent workflows",
   "author": "Nico Bailon",
   "license": "MIT",

package/src/agents/agent-management.d.ts

@@ -6,7 +6,7 @@ import { type RuntimeAgentOwner } from "./runtime-agent-registry.ts";
 type ManagementContext = Pick<ExtensionContext, "cwd" | "modelRegistry"> & {
     model?: ExtensionContext["model"];
     config?: ExtensionConfig;
-    currentSessionId?: string;
+    piSessionId?: string;
     runtimeAgentOwner?: RuntimeAgentOwner;
     onAgentsChanged?: () => void;
     discoverAgentsAll?: typeof discoverAgentsAll;

package/src/agents/agent-management.js

@@ -1023,7 +1023,7 @@ export function handleList(params, ctx) {
     let scopedAgents = effectiveAgentsForScope(scope, d, ctx.runtimeAgentOwner, ctx.model?.provider);
     scopedAgents = scopedAgents
         .sort((a, b) => a.name.localeCompare(b.name));
-    const capabilityCeiling = resolveCurrentSubagentCapabilityCeiling(ctx.currentSessionId);
+    const capabilityCeiling = resolveCurrentSubagentCapabilityCeiling(ctx.piSessionId);
     const visibleAgents = scopedAgents.filter((a) => !a.disabled);
     const agents = visibleAgents.filter((a) => isAgentAllowedByCapabilityCeiling(a.name, capabilityCeiling));
     const restrictedAgents = visibleAgents.filter((a) => !isAgentAllowedByCapabilityCeiling(a.name, capabilityCeiling));
@@ -1082,7 +1082,7 @@ function handleModels(params, ctx) {
     const availableModels = ctx.modelRegistry.getAvailable().map(toModelInfo);
     const currentModel = ctx.model ? { provider: ctx.model.provider, id: ctx.model.id } : undefined;
     const preferredProvider = ctx.model?.provider;
-    const capabilityCeiling = resolveCurrentSubagentCapabilityCeiling(ctx.currentSessionId);
+    const capabilityCeiling = resolveCurrentSubagentCapabilityCeiling(ctx.piSessionId);
     let selectedAgents = effectiveAgents;
     if (requestedAgent) {
         const matches = findAgentsInDiscovery(requestedAgent, discovered, scope, ctx.runtimeAgentOwner, ctx.model?.provider);

package/src/agents/agents.js

@@ -2432,8 +2432,7 @@ function buildAgentDiscoverySources(cwd, preferredModelProvider, globalNpmRoot,
     const watchPaths = new Set([
         userSettingsPath,
         ...(projectSettingsPath ? [projectSettingsPath] : []),
-        ...projectDiscoveryWatchPaths(effectiveCwd),
-        ...findProjectRootCandidates(effectiveCwd).map((root) => path.join(getProjectConfigDir(root), "settings.json")),
+        ...(projectTrusted ? [...projectDiscoveryWatchPaths(effectiveCwd), ...findProjectRootCandidates(effectiveCwd).map((root) => path.join(getProjectConfigDir(root), "settings.json"))] : []),
         ...userScanDirs.watchPaths,
         ...projectScanDirs.watchPaths,
         ...packageSubagentPaths.watchPaths,

package/src/agents/frontmatter.js

@@ -97,22 +97,15 @@ export function parseFrontmatter(content) {
         // neither end a pending list block nor become part of its value.
         if (trimmed.startsWith("#") && !currentFolded && !currentLiteral)
             continue;
-        if (currentKey !== null && currentBlockLines !== null && (indent > (currentIndent ?? 0) || ((currentFolded || currentLiteral) && trimmed === ""))) {
+        // Exactly-empty lines belong to any pending block, not only to | and > scalars.
+        if (currentKey !== null && currentBlockLines !== null && (indent > (currentIndent ?? 0) || trimmed === "")) {
             // This line is part of the current block value
             currentBlockLines.push(line);
             continue;
         }
         // Flush any pending block value
         if (currentKey !== null && currentBlockLines !== null) {
-            // Strip the common leading whitespace from the block so the
-            // serializer can add its own indentation level.
-            const rawBlock = currentBlockLines.join("\n");
-            const leadingSpaces = rawBlock.match(/^[ \t]+(?=\S)/m);
-            const prefix = leadingSpaces?.[0] ?? "";
-            const stripped = prefix
-                ? rawBlock.replace(new RegExp(`^${escapeRegex(prefix)}`, "gm"), "").replace(/^\n/, "")
-                : rawBlock;
-            frontmatter[currentKey] = currentFolded ? foldBlock(stripped) : stripped;
+            frontmatter[currentKey] = flushBlockLines(currentBlockLines, currentFolded, currentLiteral);
             currentKey = null;
             currentBlockLines = null;
             currentIndent = null;
@@ -146,17 +139,28 @@ export function parseFrontmatter(content) {
                 frontmatter[key] = value;
             }
         }
-        // Lines that don't match a key pattern (e.g., comments, empty lines) are ignored
     }
     // Flush final block value
     if (currentKey !== null && currentBlockLines !== null) {
-        const rawBlock = currentBlockLines.join("\n");
-        const leadingSpaces = rawBlock.match(/^[ \t]+(?=\S)/m);
-        const prefix = leadingSpaces?.[0] ?? "";
-        const stripped = prefix
-            ? rawBlock.replace(new RegExp(`^${escapeRegex(prefix)}`, "gm"), "").replace(/^\n/, "")
-            : rawBlock;
-        frontmatter[currentKey] = currentFolded ? foldBlock(stripped) : stripped;
+        frontmatter[currentKey] = flushBlockLines(currentBlockLines, currentFolded, currentLiteral);
     }
     return { frontmatter, body };
 }
+function flushBlockLines(blockLines, folded, literal) {
+    let lines = blockLines;
+    if (!folded && !literal) {
+        // Drop exactly-empty edge lines, and every line from the first exactly-empty one after
+        // the last content line, so nested values without inner blank lines parse as before.
+        const end = lines.indexOf("", lines.findLastIndex((line) => line.trim() !== "") + 1);
+        lines = lines.slice(lines.findIndex((line) => line !== ""), end < 0 ? lines.length : end);
+    }
+    // Strip the common leading whitespace from the block so the
+    // serializer can add its own indentation level.
+    const rawBlock = lines.join("\n");
+    const leadingSpaces = rawBlock.match(/^[ \t]+(?=\S)/m);
+    const prefix = leadingSpaces?.[0] ?? "";
+    const stripped = prefix
+        ? rawBlock.replace(new RegExp(`^${escapeRegex(prefix)}`, "gm"), "").replace(/^\n/, "")
+        : rawBlock;
+    return folded ? foldBlock(stripped) : stripped;
+}

package/src/extension/index.js

@@ -735,7 +735,7 @@ export default function registerSubagentExtension(pi) {
             event.systemPromptOptions.sections.active_subagents = activeRuns;
         if (!event.systemPromptOptions.selectedTools.includes("subagent"))
             return;
-        const catalog = buildAdvertisedAgentCatalog(advertisedAgents, resolveCurrentSubagentCapabilityCeiling(sessionId), recordedPromptSection(pi, "advertised_subagents"));
+        const catalog = buildAdvertisedAgentCatalog(advertisedAgents, resolveCurrentSubagentCapabilityCeiling(ctx.sessionManager.getSessionId() ?? undefined), recordedPromptSection(pi, "advertised_subagents"));
         if (catalog)
             event.systemPromptOptions.sections.advertised_subagents = catalog;
     });
@@ -1207,7 +1207,7 @@ export default function registerSubagentExtension(pi) {
         hasPendingSupervisorRequest: () => [...supervisorChannel.pending.values()].some((request) => request.expectsReply),
         advertisedPrompt: async () => {
             await waitForAdvertisement();
-            return buildAdvertisedAgentPrompt(advertisedAgents, resolveCurrentSubagentCapabilityCeiling(state.currentSessionId ?? undefined));
+            return buildAdvertisedAgentPrompt(advertisedAgents, resolveCurrentSubagentCapabilityCeiling(state.supervisorOwnerSessionId ?? undefined));
         },
     });
 }

package/src/missions/actions.js

@@ -2,6 +2,7 @@ import * as fs from "node:fs";
 import * as path from "node:path";
 import { MISSION_STATUSES, } from "./types.js";
 import { missionStatePath } from "./workflow-state.js";
+import { finishedMissionStatus, missionAcceptanceStatus, readMissionBinding } from "./lifecycle.js";
 import { createMission, listGlobalMissions, listMissions, missionRecordPath, readMission, resolveMissionStoreLocation, updateMission, validateMissionId, } from "./store.js";
 export const MISSION_ACTIONS = [
     "mission.create",
@@ -184,6 +185,7 @@ function validateMissionUpdate(value) {
 }
 function refreshLinkedRunStatus(location, record) {
     const warnings = [];
+    const runAcceptance = [];
     const updates = record.runs.flatMap((run) => {
         if (!run.asyncDir)
             return [];
@@ -207,6 +209,7 @@ function refreshLinkedRunStatus(location, record) {
             warnings.push(`Linked run status '${statusPath}' is missing state`);
             return [];
         }
+        runAcceptance.push({ runId: run.runId, status: missionAcceptanceStatus(status.steps) });
         if (state === run.status)
             return [];
         return [{
@@ -232,7 +235,8 @@ function refreshLinkedRunStatus(location, record) {
                         : states.some((state) => state === "stopped" || state === "rejected" || state === "cancelled")
                             ? "cancelled"
                             : states.length > 0 && states.every((state) => state === "complete" || state === "completed")
-                                ? "completed"
+                                // A run without a binding counts as auto-created.
+                                ? finishedMissionStatus(record, !merged.some((run) => run.asyncDir && readMissionBinding(run.asyncDir)?.autoCreated === false), runAcceptance)
                                 : record.status;
     return { record: updateMission(location, record.id, { status, addRuns: updates }), warnings };
 }
@@ -261,7 +265,8 @@ function formatMission(record) {
             const identity = child.runId ? `${child.key} (${child.runId})` : child.key;
             const heartbeat = child.heartbeat ? `; heartbeat ${child.heartbeat.status ?? child.status}${child.heartbeat.phase ? `/${child.heartbeat.phase}` : ""} at ${child.heartbeat.updatedAt}` : "";
             const recovery = child.sessionPath ? `; session ${child.sessionPath}` : child.artifactPaths.length ? `; artifacts ${child.artifactPaths.length}` : "";
-            lines.push(`  ${identity}: ${child.status}${child.agent ? ` — ${child.agent}` : ""}${child.phase ? ` [${child.phase}]` : ""}; updated ${child.updatedAt}${heartbeat}${recovery}`);
+            const acceptance = child.acceptanceStatus ? `; acceptance ${child.acceptanceStatus}` : "";
+            lines.push(`  ${identity}: ${child.status}${child.agent ? ` — ${child.agent}` : ""}${child.phase ? ` [${child.phase}]` : ""}${acceptance}; updated ${child.updatedAt}${heartbeat}${recovery}`);
         }
     }
     if (record.decisions.length) {

package/src/missions/lifecycle.d.ts

@@ -21,6 +21,7 @@ export interface MissionLaunchParams {
 export interface MissionLaunchBinding {
     missionId: string;
     location: MissionStoreLocation;
+    /** True when the launch made the mission without a `mission` or `missionId` request. */
     autoCreated: boolean;
     announceInContent?: boolean;
 }
@@ -30,6 +31,16 @@ export declare function prepareMissionLaunch(input: {
     config?: MissionStoreConfig;
     ownerSessionId?: string;
 }): MissionLaunchBinding | undefined;
+/** The first unaccepted acceptance status among child results, else the first status found. */
+export declare function missionAcceptanceStatus(results: unknown): string | undefined;
+/**
+ * The status of a mission whose runs all finished successfully. Every path that
+ * completes a mission from run state goes through here.
+ */
+export declare function finishedMissionStatus(record: MissionRecord, autoCreated: boolean, runAcceptance: ReadonlyArray<{
+    runId: string;
+    status: string | undefined;
+}>): "completed" | "waiting";
 export declare function writeMissionAsyncBinding(asyncDir: string, binding: MissionLaunchBinding): void;
 export declare function attachMissionToLaunchResult(input: {
     binding: MissionLaunchBinding;

package/src/missions/lifecycle.js

@@ -50,7 +50,7 @@ export function prepareMissionLaunch(input) {
         ...(mission?.labels ? { labels: mission.labels } : {}),
         ...(input.ownerSessionId ? { ownerSessionId: input.ownerSessionId } : {}),
     }, new Date(), input.config?.retainTerminal);
-    return { missionId: record.id, location, autoCreated: true, announceInContent: true };
+    return { missionId: record.id, location, autoCreated: input.params.mission === undefined, announceInContent: true };
 }
 function toolResultIsError(result) {
     return "isError" in result && result.isError === true;
@@ -67,7 +67,30 @@ function runStatusForResult(result) {
         return "failed";
     return "completed";
 }
-function missionStatusForRun(record, runId, runStatus) {
+/** Acceptance statuses that mean nobody accepted the child's result. */
+const UNACCEPTED_ACCEPTANCE_STATUSES = new Set(["pending", "claimed", "review-required", "rejected"]);
+/** The first unaccepted acceptance status among child results, else the first status found. */
+export function missionAcceptanceStatus(results) {
+    const statuses = (Array.isArray(results) ? results : [])
+        .map((result) => result?.acceptance?.status)
+        .filter((status) => typeof status === "string" && status.trim() !== "");
+    return statuses.find((status) => UNACCEPTED_ACCEPTANCE_STATUSES.has(status)) ?? statuses[0];
+}
+/**
+ * The status of a mission whose runs all finished successfully. Every path that
+ * completes a mission from run state goes through here.
+ */
+export function finishedMissionStatus(record, autoCreated, runAcceptance) {
+    // Nobody is expected to close a mission made by default for an ordinary launch.
+    if (autoCreated)
+        return "completed";
+    if (runAcceptance.some((run) => UNACCEPTED_ACCEPTANCE_STATUSES.has(run.status)))
+        return "waiting";
+    // A run reported here supersedes its older workflow child record.
+    const reported = new Set(runAcceptance.map((run) => run.runId));
+    return record.workflowChildren.some((child) => !reported.has(child.runId) && UNACCEPTED_ACCEPTANCE_STATUSES.has(child.acceptanceStatus)) ? "waiting" : "completed";
+}
+function missionStatusForRun(record, runId, runStatus, acceptanceStatus, autoCreated) {
     if (record.status === "completed" || record.status === "failed" || record.status === "cancelled")
         return record.status;
     if (runStatus === "active" || runStatus === "queued" || runStatus === "running")
@@ -80,7 +103,7 @@ function missionStatusForRun(record, runId, runStatus) {
     if (otherActive)
         return "active";
     if (runStatus === "completed" || runStatus === "complete")
-        return "completed";
+        return finishedMissionStatus(record, autoCreated, [{ runId, status: acceptanceStatus }]);
     if (runStatus === "stopped" || runStatus === "rejected" || runStatus === "cancelled")
         return "cancelled";
     return "failed";
@@ -129,6 +152,7 @@ function persistedBinding(binding) {
         globalIndexDir: binding.location.globalIndexDir,
         writeGlobalIndex: binding.location.writeGlobalIndex,
         ...(binding.location.retainTerminal !== undefined ? { retainTerminal: binding.location.retainTerminal } : {}),
+        autoCreated: binding.autoCreated,
     };
 }
 export function writeMissionAsyncBinding(asyncDir, binding) {
@@ -170,7 +194,7 @@ export function attachMissionToLaunchResult(input) {
         ...(usage ? { usage } : {}),
     };
     let mission = updateMission(input.binding.location, input.binding.missionId, {
-        status: missionStatusForRun(current, runId, runStatus),
+        status: missionStatusForRun(current, runId, runStatus, missionAcceptanceStatus(input.result.details.results), input.binding.autoCreated),
         addRuns: [run],
         addArtifacts: artifactsForResult(input.result),
         ...(firstText(input.result) && runStatus !== "active" ? { summary: firstText(input.result) } : {}),
@@ -188,6 +212,8 @@ export function attachMissionToLaunchResult(input) {
                         asyncDir: input.result.details.asyncDir,
                         mode: input.result.details.mode,
                         state: status.state,
+                        // A run that finished before this launch result was attached keeps its acceptance in the status steps.
+                        ...(Array.isArray(status.steps) ? { results: status.steps.map((step) => ({ acceptance: step?.acceptance })) } : {}),
                         ...(firstText(input.result) ? { summary: firstText(input.result) } : {}),
                     }) ?? mission;
                 }
@@ -237,9 +263,12 @@ function parsePersistedBinding(value, source) {
         throw new Error(`${source}.writeGlobalIndex must be boolean`);
     if (input.retainTerminal !== undefined && (!Number.isInteger(input.retainTerminal) || input.retainTerminal < 1))
         throw new Error(`${source}.retainTerminal must be a positive integer`);
+    if (input.autoCreated !== undefined && typeof input.autoCreated !== "boolean")
+        throw new Error(`${source}.autoCreated must be boolean`);
     return {
         missionId: validateMissionId(input.missionId, `${source}.missionId`),
-        autoCreated: false,
+        // Bindings saved before this field existed belong to runs that completed their mission without an acceptance check.
+        autoCreated: input.autoCreated !== false,
         location: {
             projectRoot: input.projectRoot,
             missionDir: input.missionDir,
@@ -339,8 +368,9 @@ export function syncMissionFromAsyncCompletion(value) {
                 ? "stopped"
                 : "failed";
     const workflowChildTerminal = !["running", "queued", "active", "paused"].includes(workflowChildStatus);
+    const acceptanceStatus = missionAcceptanceStatus(event.results);
     return updateMission(binding.location, binding.missionId, {
-        status: missionStatusForRun(current, runId, runStatus),
+        status: missionStatusForRun(current, runId, runStatus, acceptanceStatus, binding.autoCreated),
         addRuns: [{ runId, mode: typeof event.mode === "string" && ["single", "parallel", "chain", "workflow"].includes(event.mode) ? event.mode : "external", asyncDir: event.asyncDir, status: runStatus, completedAt, ...(usage && usage.tokens > 0 ? { usage } : {}) }],
         addArtifacts: artifacts,
         ...(workflowRunId && workflowKey ? { upsertWorkflowChildren: [{
@@ -350,6 +380,7 @@ export function syncMissionFromAsyncCompletion(value) {
                     status: workflowChildStatus,
                     artifactPaths: artifacts.map((artifact) => artifact.path),
                     ...(workflowChildTerminal ? { completedAt } : {}),
+                    ...(acceptanceStatus ? { acceptanceStatus } : {}),
                     heartbeat: { status: workflowChildStatus, ...(summary ? { message: summary } : {}) },
                 }] } : {}),
         ...(summary ? { summary } : {}),

package/src/missions/store.js

@@ -133,6 +133,7 @@ function parseWorkflowChild(value, label) {
         ...(optionalString(input.phase, `${label}.phase`) ? { phase: input.phase } : {}),
         ...(input.completedAt !== undefined ? { completedAt: timestamp(input.completedAt, `${label}.completedAt`) } : {}),
         ...(optionalString(input.sessionPath, `${label}.sessionPath`) ? { sessionPath: input.sessionPath } : {}),
+        ...(optionalString(input.acceptanceStatus, `${label}.acceptanceStatus`) ? { acceptanceStatus: input.acceptanceStatus } : {}),
         ...(heartbeat ? { heartbeat: {
                 updatedAt: timestamp(heartbeat.updatedAt, `${label}.heartbeat.updatedAt`),
                 ...(optionalString(heartbeat.status, `${label}.heartbeat.status`) ? { status: heartbeat.status } : {}),

package/src/missions/types.d.ts

@@ -57,6 +57,8 @@ export interface MissionWorkflowChild {
     sessionPath?: string;
     artifactPaths: string[];
     heartbeat?: MissionChildHeartbeat;
+    /** Latest acceptance ledger status reported for the child. */
+    acceptanceStatus?: string;
 }
 export type MissionWorkflowChildUpdate = Pick<MissionWorkflowChild, "workflowRunId" | "key" | "status"> & Partial<Omit<MissionWorkflowChild, "workflowRunId" | "key" | "status" | "startedAt" | "updatedAt" | "artifactPaths" | "heartbeat">> & {
     startedAt?: string;

package/src/runs/background/async-execution.d.ts

@@ -25,7 +25,7 @@ interface AsyncExecutionContext {
     cwd: string;
     currentSessionId: string;
     completionOwnerId?: string;
-    /** Parent session id used by permission-system ask forwarding. */
+    /** Parent Pi session id (`getSessionId()`): keys permission-system ask forwarding and capability ceilings. */
     parentSessionId?: string;
     /** Parent session file, recorded as each new child session's `parentSession` header. */
     parentSessionFile?: string;

package/src/runs/background/async-execution.js

@@ -1233,7 +1233,8 @@ export function executeAsyncChain(id, params) {
         .find((name) => name !== undefined && !lookupRunnerLauncher(ctx.runnerLaunchers, name));
     if (undefinedLauncher !== undefined)
         return formatAsyncStartError(resultMode, `Launcher '${undefinedLauncher}' used by this ${resultMode} is not defined in runnerLaunchers in the user subagent config.`);
-    const capabilityCeiling = params.capabilityCeiling ?? resolveCurrentSubagentCapabilityCeiling(ctx.currentSessionId);
+    const launchParentSessionId = ctx.parentSessionId ?? ctx.currentSessionId;
+    const capabilityCeiling = params.capabilityCeiling ?? resolveCurrentSubagentCapabilityCeiling(launchParentSessionId);
     const inheritedNestedRoute = inheritedNestedRouteOf(ctx.childRuntime);
     const nestedAddress = inheritedNestedRoute ? inheritedNestedParentAddressOf(ctx.childRuntime) : undefined;
     const asyncDir = inheritedNestedRoute
@@ -1336,7 +1337,6 @@ export function executeAsyncChain(id, params) {
     const initialStatusAt = Date.now();
     const initialCompletionOwnerId = ctx.completionOwnerId ?? currentCompletionOwnerId();
     const completionOwnerProcess = currentRecordedPid();
-    const launchParentSessionId = ctx.parentSessionId ?? ctx.currentSessionId;
     try {
         writeRetainedRequiredChildExtensions(asyncDir, params.requiredExtensions ?? ctx.childRuntime?.requiredExtensions ?? resolveRequiredChildExtensions(launchParentSessionId ?? undefined));
     }
@@ -1600,7 +1600,7 @@ export function executeAsyncSingle(id, params) {
         if (unsupported.length > 0)
             return formatAsyncStartError("single", `Agent '${agentConfig.name}' uses runner.type='${externalRunnerType}' and does not support: ${unsupported.join(", ")}.`);
     }
-    const capabilityCeiling = intersectSubagentCapabilityCeilings(params.capabilityCeiling ?? resolveCurrentSubagentCapabilityCeiling(ctx.currentSessionId), ctx.childRuntime?.capabilityCeiling);
+    const capabilityCeiling = intersectSubagentCapabilityCeilings(params.capabilityCeiling ?? resolveCurrentSubagentCapabilityCeiling(ctx.parentSessionId ?? ctx.currentSessionId), ctx.childRuntime?.capabilityCeiling);
     try {
         assertAgentAllowedByCapabilityCeiling(agentConfig.name, capabilityCeiling);
     }

package/src/runs/background/async-resume.js

@@ -484,7 +484,8 @@ export function resolveAsyncResumeTarget(params, deps = {}, options = {}) {
     const requireSessionFile = options.requireSessionFile ?? true;
     const location = resolveAsyncRunLocation(params, asyncDirRoot, resultsDir);
     if (!location.asyncDir && !location.resultPath) {
-        throw new Error("Async run not found. Provide id or dir.");
+        const requested = params.id ?? params.runId;
+        throw new Error(requested ? `Async run not found for '${requested}'.` : "Async run not found. Provide id or dir.");
     }
     const reconciliation = location.asyncDir
         ? reconcileAsyncRun(location.asyncDir, { resultsDir, kill: deps.kill, now: deps.now })
@@ -606,7 +607,7 @@ export function resolveAsyncResumeTarget(params, deps = {}, options = {}) {
         ? intersectSubagentCapabilityCeilings(status?.admissionCapabilityCeiling, result?.admissionCapabilityCeiling)
         : intersectSubagentCapabilityCeilings(status?.capabilityCeiling, statusSteps[index]?.capabilityCeiling, result?.capabilityCeiling, resultSteps[index]?.capabilityCeiling);
     const managedWorktreeCwd = location.asyncDir
-        ? resolveRetainedWorktreeCwd(parallelHandoffPath(location.asyncDir), runId, index)
+        ? resolveRetainedWorktreeCwd(parallelHandoffPath(location.asyncDir), runId, index, statusSteps[index]?.worktreePath)
         : undefined;
     const resumeCwd = validateResumeCwd(runId, managedWorktreeCwd ?? status?.cwd ?? result?.cwd ?? recoveryDescriptor?.cwd);
     return {

package/src/runs/background/retained-children.js

@@ -52,7 +52,7 @@ function childResumability(run, step) {
     }
     let requiredCwd;
     try {
-        requiredCwd = resolveRetainedWorktreeCwd(parallelHandoffPath(run.asyncDir), run.id, step.index) ?? recoveryDescriptor.cwd ?? run.cwd;
+        requiredCwd = resolveRetainedWorktreeCwd(parallelHandoffPath(run.asyncDir), run.id, step.index, step.worktreePath) ?? recoveryDescriptor.cwd ?? run.cwd;
         if (!requiredCwd || !fs.statSync(requiredCwd).isDirectory())
             return { state: "not-resumable", reason: `required cwd is missing: ${requiredCwd ?? "unknown"}` };
     }