@staix/agent-hub 0.12.30 → 0.12.31

package/src/hub/daemon.ts

@@ -1,3 +1,8 @@
+import { ContainerRunner, type RunnerOptions, type RunReceipt } from "./check-runner.ts";
+import { fetchSpec, specAvailable, specRefs } from "./spec-read.ts";
+import { isDenied } from "../local/deny.ts";
+import { supersededDeliveryReason, deliveryTaskContext } from "./delivery-superseded.ts";
+import { renderHeldHint } from "./delivery-hints.ts";
 import { correctionInput, MemoryTelemetry } from "./memory-candidates.ts";
 import { MemoryObservations } from "./memory-observations.ts";
 import { createStalenessProbe } from "./memory-staleness.ts";
@@ -16,7 +21,7 @@ import { JcodePeer } from "../adapters/jcode.ts";
 import { jcodeMcpReady } from "../cli/jcode-mcp.ts";
 import { loadSdlc } from "./sdlc.ts";
 import { peerRosterConfig, rosterAllows, type PeerRoster } from "./roster.ts";
-import { KNOWN_PEERS, OPTIONAL_PEERS, SUPPORTED_PEERS, isSupportedPeer } from "./peer-catalog.ts";
+import { KNOWN_PEERS, OPTIONAL_PEERS, PLANNER_PEERS, SUPPORTED_PEERS, isSupportedPeer } from "./peer-catalog.ts";
 import { databaseBusySite, isDatabaseBusy, RecoveryDatabaseBusy } from "./recovery-error.ts";
 import { buildKimiLaunch, buildQwenLaunch } from "../cli/launch.ts";
 import { ProgressObserver, normalizeCodexObservation, normalizeClaudeObservation } from "./progress.ts";
@@ -51,16 +56,16 @@ import { Capture, skipTools } from "../memory/capture.ts";
 import { DEFAULT_OMNIROUTE, OmniRoute, type OmniRouteConfig } from "../omniroute/client.ts";
 import { Sidecar } from "../switchyard/sidecar.ts";
 import { Briefs } from "../memory/brief.ts";
-import { Board, taskProposalAt, taskHasStarted, CLASSES, type Task, type TaskClass } from "./board.ts";
+import { Board, isClosedTaskState, taskProposalAt, taskHasStarted, CLASSES, type Task, type TaskClass } from "./board.ts";
 import { Budget, claudeWindows, codexWindows, DEFAULT_BUDGET, type BudgetConfig } from "./budget.ts";
 import { HUB } from "./envelope.ts";
 import { recallMemoryLines, trimToTokens } from "../memory/recall.ts";
 import { closeSync, constants as fsConstants, openSync, readSync, statSync } from "node:fs";
 import { basename, isAbsolute } from "node:path";
 import { fileURLToPath } from "node:url";
-import { realPath } from "./project.ts";
+import { realPath, hubHome } from "./project.ts";
 import type { BusEvent } from "./bus.ts";
-import { CONDUCTOR_TOOLS, CONDUCTOR_TOOL_NAMES, DEFAULT_ROLES, roleContract, TASK_TOOLS, profileTaskTools } from "./hub-tools.ts";
+import { CONDUCTOR_TOOLS, CONDUCTOR_TOOL_NAMES, DEFAULT_ROLES, roleContract, TASK_TOOLS, BOUNDED_TASK_TOOLS, profileTaskTools } from "./hub-tools.ts";
 import { Conductor, ConductorHolds, conductorPeer, conductorProgressSink, publicConductorTask, publicPeerBudget, type ConductEvent } from "./conductor.ts";
 import { SupervisionFeed } from "./supervision.ts";
 import { drainCliAudits } from "../cli/identity-audit.ts";
@@ -74,7 +79,7 @@ import { Bus } from "./bus.ts";
 import { attribute, deliveryTask, type TaskAttribution } from "./attribution.ts";
 import { DeliveryJournal } from "./delivery-journal.ts";
 import { startDashboard, type DashboardSession } from "./ui.ts";
-import { PROTOCOL, stateDirFor } from "./control-client.ts";
+import { DAEMON_FEATURES, PROTOCOL, stateDirFor } from "./control-client.ts";
 import { newEnvelope, parseMarker, replyParent, sanitize, USER, type Envelope, type PeerId, type Priority } from "./envelope.ts";
 import { BasePeer, DEFAULT_WATCHDOG_MS, type PeerAdapter } from "./peers.ts";
 import { MemoryClient, workerUrl } from "../memory/client.ts";
@@ -104,7 +109,7 @@ import { syncMemory } from "@staix/agent-mem";
 import { MEMORY_READ_TOOLS, memoryRead, validateMemoryRead } from "./read-tools.ts";
 import { memoryProjectKey } from "@staix/agent-mem";
 import { nativeRecall } from "../memory/recall.ts";
-import { promotionScreen, withoutPii } from "./memory-policy.ts";
+import { promotionScreen, withoutPii, mentionsPrivatePath } from "./memory-policy.ts";
 
 export interface HubConfig {
   watchdog_ms: number;
@@ -125,6 +130,8 @@ export interface HubConfig {
   memory: { enabled: boolean; worker_url?: string; inject_tokens: number; brief_items: number; native?: boolean; short_ttl_days?: number; sync_interval_min?: number; suggest_edges?: boolean };
   roster: PeerRoster;
   roles: Record<string, string[]>;
+  /** #455: the one peer that holds the planner role; unset, the `roles` planners stand, or the hub assigns one. */
+  planner?: string;
   profile?: ProfileName;
   profile_overrides?: Record<string, unknown>;
   pii?: { paths: string[] };
@@ -147,7 +154,7 @@ export interface HubConfig {
   /** Between-turn task escalation, disabled unless explicitly enabled (#186). */
   task_sweep: TaskSweepConfig;
   /** A command per task class run when the owner marks the task done, and its timeout (issue #7). */
-  checks: { timeout_s: number; [cls: string]: string | number };
+  checks: { timeout_s: number; runner?: "container"; [cls: string]: string | number | undefined };
   /** A git tree at each turn boundary for `ahub turns` and `ahub undo`, the last `keep` per peer (issue #33). */
   snapshots: { enabled: boolean; keep: number };
   /** #247: opt-in research records of each approved task in ~/.agenthub/research (ids, counts and tokens only). */
@@ -231,14 +238,19 @@ const CONFIG_FILES = ["config.json", "config.local.json"] as const;
 const CONFIG_BLOCKS = ["memory", "roster", "roles", "catalog_roles", "pii", "data", "conductor", "budget", "context", "inference", "omniroute", "local", "pi", "approvals", "tasks", "task_sweep", "checks", "snapshots", "research", "bench", "limits", "review", "recovery", "capabilities", "mlx", "permission_modes", "peers", "terminal"];
 
 /** Connection-time policy refresh reads role/feed fields only, never launches or machine-local configuration. */
-export function loadConductorPolicy(cwd: string): { roles: Record<string, string[]>; conductor: HubConfig["conductor"] } {
+export function loadConductorPolicy(cwd: string): { roles: Record<string, string[]>; conductor: HubConfig["conductor"]; planner?: string } {
   const roles: Record<string, string[]> = { ...DEFAULT_ROLES };
   const conductor = { ...DEFAULT_CONFIG.conductor };
+  let planner: string | undefined;
   for (const name of CONFIG_FILES) {
     let file: Record<string, unknown>;
     try { file = JSON.parse(readFileSync(join(cwd, ".agenthub", name), "utf8")); }
     catch (error) { if ((error as NodeJS.ErrnoException).code === "ENOENT") continue; throw error; }
     if (file.roles !== undefined) { conductorPeer(file.roles); Object.assign(roles, file.roles); }
+    if (file.planner !== undefined) {
+      if (!(PLANNER_PEERS as readonly unknown[]).includes(file.planner)) throw new Error(`planner must be one of ${PLANNER_PEERS.join(", ")}`);
+      planner = file.planner as string;
+    }
     if (file.conductor !== undefined) {
       if (!file.conductor || typeof file.conductor !== "object" || Array.isArray(file.conductor)) throw new Error("conductor must be an object");
       const block = file.conductor as Record<string, unknown>;
@@ -253,7 +265,7 @@ export function loadConductorPolicy(cwd: string): { roles: Record<string, string
     }
   }
   conductorPeer(roles);
-  return { roles, conductor };
+  return { roles, conductor, ...(planner ? { planner } : {}) };
 }
 
 /** `memory.short_ttl_days`: a whole number of days from 1 to 3650; anything else would make every short-term deadline invalid. */
@@ -345,6 +357,7 @@ export function loadConfig(cwd: string): HubConfig {
       ...(file.memory?.sync_interval_min !== undefined ? { sync_interval_min: syncMinutes(file.memory.sync_interval_min) } : {}) }, // the native store is on with any project config (#258)
     roster: peerRosterConfig(file.roster),
     roles: conductorPolicy.roles,
+    planner: conductorPolicy.planner,
     conductor: conductorPolicy.conductor,
     budget: { ...DEFAULT_CONFIG.budget, wait_max_min: 30, ...file.budget }, // on with any project config (issue #36)
     context: { ...DEFAULT_CONTEXT, ...file.context },
@@ -358,7 +371,7 @@ export function loadConfig(cwd: string): HubConfig {
     },
     tasks: { ...DEFAULT_CONFIG.tasks, ...file.tasks, ...(file.tasks?.wip_limits !== undefined ? { wip_limits: planningWipLimits(file.tasks.wip_limits) } : {}) },
     task_sweep: taskSweepConfig(file.task_sweep),
-    checks: { ...DEFAULT_CONFIG.checks, ...file.checks },
+    checks: { ...DEFAULT_CONFIG.checks, ...file.checks, runner: file.checks?.runner === "container" ? "container" : undefined },
     snapshots: { ...DEFAULT_CONFIG.snapshots, enabled: true, ...file.snapshots },
     research: { enabled: file.research?.enabled === true }, // opt-in: anything but true is off
     bench: { enabled: file.bench?.enabled === true }, // opt-in: a run resets this project's work tree
@@ -423,6 +436,12 @@ export interface DaemonOptions {
   researchGraceMs?: number;
   /** Tests observe timer-driven sync without opening a remote; production uses the package sync. */
   memorySync?: typeof syncMemory;
+  /** Host-only test/embedding transports. Never read from config or tool input. */
+  boundedTools?: {
+    specAvailable?: typeof specAvailable;
+    fetchSpec?: typeof fetchSpec;
+    runner?: (options: RunnerOptions) => Pick<ContainerRunner, "available" | "start" | "result" | "consumeOutput" | "cancel" | "sweep" | "stopAll" | "markStale"> & Partial<Pick<ContainerRunner, "unknownReceipts" | "maintenanceUnavailable">>;
+  };
 }
 
 interface Client {
@@ -560,6 +579,39 @@ export async function startDaemon(opts: DaemonOptions) {
   const logFile = join(opts.stateDir, "hub.log");
   // The state dir can vanish under a running hub (issue #56): a log line must never take a handler down with it.
   const log = (line: string) => { try { appendFileSync(logFile, `${new Date().toISOString()} ${line}\n`); } catch { /* the state dir is gone; the watchdog is stopping the hub */ } };
+  // #455: the planner in force. `policyRoles` is what the config files say; settlePlanner derives config.roles from it in
+  // place (Tasks holds the same object): an explicit `planner` key is exclusive, and with none and no rostered planner
+  // the first rostered PLANNER_PEERS entry plans. Never written to disk: a person pins it with the `planner` setting.
+  let policyRoles = structuredClone(config.roles);
+  let plannerSource: "setting" | "roles" | "automatic" | "none" = "none";
+  let plannerLogged = "";
+  const settlePlanner = () => {
+    const roles = structuredClone(policyRoles);
+    const pinned = (PLANNER_PEERS as readonly string[]).includes(config.planner ?? "") ? config.planner! : undefined;
+    let source: typeof plannerSource;
+    if (pinned) {
+      for (const peer of Object.keys(roles)) roles[peer] = roles[peer]!.filter(role => role !== "planner");
+      roles[pinned] = ["planner", ...(roles[pinned] ?? [])];
+      source = "setting";
+    } else if (Object.entries(roles).some(([peer, list]) => list.includes("planner") && onRoster(peer))) source = "roles";
+    else {
+      const auto = PLANNER_PEERS.find(onRoster);
+      if (auto) { // the off-roster holders give the role up while the automatic planner holds it
+        for (const peer of Object.keys(roles)) roles[peer] = roles[peer]!.filter(role => role !== "planner");
+        roles[auto] = [...(roles[auto] ?? []), "planner"];
+      }
+      source = auto ? "automatic" : "none";
+    }
+    conductorPeer(roles);
+    for (const key of Object.keys(config.roles)) delete config.roles[key];
+    Object.assign(config.roles, roles);
+    plannerSource = source;
+    const line = `${plannerPeers().join(", ") || "nobody"} (${source})`;
+    if (line !== plannerLogged) { log(`planner: ${line}${source === "automatic" ? "; pin it with ahub settings set planner <peer>" : ""}`); plannerLogged = line; }
+  };
+  const plannerPeers = () => Object.keys(config.roles).filter(peer => config.roles[peer]!.includes("planner"));
+  if (config.planner !== undefined && !(PLANNER_PEERS as readonly string[]).includes(config.planner)) log(`planner: ${JSON.stringify(config.planner)} is not one of ${PLANNER_PEERS.join(", ")}; ignored`);
+  settlePlanner();
   const catalog = loadRoleCatalog(opts.cwd);
   for (const error of catalog.errors) log(`catalog role ${error.id}: ${error.reason}`);
   const resolvedProfile = resolveProfile(config.profile, config.profile_overrides);
@@ -718,12 +770,12 @@ export async function startDaemon(opts: DaemonOptions) {
   if (crashed) for (const d of journal.list()) if (d.state === "needs_review" && d.reason === "daemon stopped during delivery" && d.updatedAt >= startedAt) lost.set(d.peer, [...(lost.get(d.peer) ?? []), d]);
   // Capabilities (issue #39): enforced here and in taskOp, never by role text alone. Unlisted peers keep everything.
   // A listed peer whose value is not a list gets nothing: whoever listed it meant to narrow it.
-  const may = (peer: PeerId, cap: "propose" | "assign" | "remember" | "memory_correct" | "memory_read" | "important"): boolean => {
+  const may = (peer: PeerId, cap: "propose" | "assign" | "remember" | "memory_correct" | "memory_read" | "important" | "spec_read" | "check_run"): boolean => {
     if (peer === USER || peer === HUB || !Object.hasOwn(config.capabilities, peer)) return true;
     const list = config.capabilities[peer];
     return Array.isArray(list) && list.includes(cap);
   };
-  const CAPABILITIES = ["propose", "assign", "remember", "memory_correct", "memory_read", "important"];
+  const CAPABILITIES = ["propose", "assign", "remember", "memory_correct", "memory_read", "important", "spec_read", "check_run"];
   for (const [peer, list] of Object.entries(config.capabilities)) {
     if (!PEER_ID.test(peer)) log(`capabilities.${peer} is not a peer id; ignored (capabilities is an object of lists, one per peer)`);
     else if (!Array.isArray(list)) log(`capabilities.${peer} is not a list: ${peer} gets no capabilities`);
@@ -738,7 +790,8 @@ export async function startDaemon(opts: DaemonOptions) {
   // silence without facts would leave overlapping owners blind. Facts and silence ask the same question, so both switch
   // together. Cached until the board changes: it is asked per recipient of every message.
   let turnFreeNow: boolean | undefined;
-  const turnFree = (): boolean => (turnFreeNow ??= coordination === "turn-free" && !board.list().some((t) => t.state !== "approved" && tasks.isPii(t)));
+  const turns = new Map<PeerId, { id: string; start: number; tokens: number; tree?: string; snapshotMs?: number; private?: boolean; deliveryTask?: number; attribution: TaskAttribution; preparedRoles: RoleIdentity[] | null }>();
+  const turnFree = (): boolean => ![...turns.values()].some(turn => turn.private) && (turnFreeNow ??= coordination === "turn-free" && !board.list().some((t) => !isClosedTaskState(t.state) && tasks.isPii(t)));
   /**
    * Why `to` does not get an agent's message (issue #107): sender and recipient are in one silent cohort, and the sender
    * has not stopped since its cohort task closed. Workflow events and anything to the console are never held back.
@@ -786,7 +839,7 @@ export async function startDaemon(opts: DaemonOptions) {
   const staleOff = config.experiments?.stale_notices === "deliver";
   if (staleOff) log("experiment: stale notices are delivered as before 0.12.4 (the issue #106 ablation)");
   const requestedPeerStops = new Set<string>();
-  const bus = new Bus({ rostered: membershipAllowed, deliveryHeld: peer => requestedPeerStops.has(peer) || rosterChanging === peer, journal, batchMax: config.batch_max, batchMs: config.batch_ms, queueCap: config.queue_cap, condense: (envs) => inference?.condense(envs) ?? Promise.resolve(envs), admit, relevant: (peer, env) => env.refs?.supervision || env.refs?.backlog ? relevantNotice(peer, env) : staleOff || relevantNotice(peer, env), silence,
+  const bus = new Bus({ rostered: membershipAllowed, deliveryHeld: peer => requestedPeerStops.has(peer) || rosterChanging === peer, journal, batchMax: config.batch_max, batchMs: config.batch_ms, queueCap: config.queue_cap, condense: (envs) => inference?.condense(envs) ?? Promise.resolve(envs), admit, relevant: (peer, env) => env.refs?.boundedCheck || env.refs?.supervision || env.refs?.backlog ? relevantNotice(peer, env) : staleOff || relevantNotice(peer, env), silence,
     onImportantQueued: (peer, counts) => { const p = bus.peers.get(peer); if (p instanceof WsPeer) p.notifyQueued(counts); } });
   startupCleanup.push(() => bus.closeJournal());
   const manualPaused = new Set<PeerId>(bus.manualPausedPeers()); // recovery never lifts an operator's pause
@@ -1012,9 +1065,10 @@ export async function startDaemon(opts: DaemonOptions) {
     if (opts.config) return; // explicit embedding/test configuration remains its authority
     const before = conductorPeer(config.roles);
     const policy = loadConductorPolicy(opts.cwd);
-    for (const key of Object.keys(config.roles)) delete config.roles[key];
-    Object.assign(config.roles, policy.roles);
+    policyRoles = structuredClone(policy.roles);
+    config.planner = policy.planner;
     config.conductor = policy.conductor;
+    settlePlanner();
     const after = conductorPeer(config.roles);
     if (before && (before !== after || config.conductor.feed === "off")) { bus.revokeSupervision(before); supervision.resetPending(before); budgetFeedSeen.clear(); }
     if (config.conductor.feed === "off" && after) bus.revokeSupervision(after);
@@ -1023,8 +1077,8 @@ export async function startDaemon(opts: DaemonOptions) {
   startupCleanup.push(() => executionBudget.close());
   // Completion checks (issue #7): only from a config file git does not track, one at a time, killed on stop.
   const allowedChecks = new Set([...CLASSES, ...Object.keys(resolvedProfile.classes)]);
-  const checkCommands = Object.fromEntries(Object.entries(config.checks).filter(([k, v]) => allowedChecks.has(k) && typeof v === "string" && v.trim())) as Record<string, string>;
-  const strayChecks = Object.keys(config.checks).filter((k) => k !== "timeout_s" && !(k in checkCommands));
+  const checkCommands = Object.fromEntries(Object.entries(config.checks).filter(([k, v]) => k !== "runner" && allowedChecks.has(k) && typeof v === "string" && v.trim())) as Record<string, string>;
+  const strayChecks = Object.keys(config.checks).filter((k) => k !== "timeout_s" && k !== "runner" && !(k in checkCommands));
   if (strayChecks.length) log(`checks ignored for ${strayChecks.join(", ")}: not a task class with a command (classes: ${CLASSES.join(", ")})`);
   // Commands and the other machine-local fields came only from a file git vouched for (loadConfig, issue #17).
   for (const line of config.ignored ?? []) log(`${line}; only a config file nobody committed may set it`);
@@ -1034,7 +1088,96 @@ export async function startDaemon(opts: DaemonOptions) {
   const runningChecks = new Set<() => void>();
   let checksClosed = false; // set on stop: a check still queued then must not start and outlive the hub
   const interrupted = { code: null, timedOut: false, interrupted: true, tail: "" };
+  // Bounded capabilities (#418): inactive sandboxed peers need no host account probe.
+  // A config-file daemon probes configured Pi/runner at startup; embeddings opt in or probe lazily on use.
+  let specState: Awaited<ReturnType<typeof specAvailable>> = { available: false, reason: "gh unavailable: not probed" };
+  let specProbe: Promise<void> | undefined, specProbeSettled = false;
+  const ensureSpec = (refresh = false): Promise<void> => {
+    if (specProbe && (!refresh || !specProbeSettled)) return specProbe;
+    specProbeSettled = false;
+    specProbe = Promise.resolve().then(() => (opts.boundedTools?.specAvailable ?? specAvailable)({ projectDir: opts.cwd }))
+      .then(value => { specState = value; }, () => { specState = { available: false, reason: "gh unavailable" }; })
+      .finally(() => { specProbeSettled = true; });
+    return specProbe;
+  };
+  const checkPending = new Set<string>();
+  const checkConnections = new WeakMap<object, string>();
+  const checkBindings = new Map<string, Pick<RunReceipt, "task" | "owner" | "generation" | "updated" | "revision">>();
+  const connectionGeneration = (id: string): string | undefined => {
+    const peer = bus.peers.get(id);
+    if (!peer || peer.state === "offline") return undefined;
+    let token = checkConnections.get(peer);
+    if (!token) { token = randomUUID(); checkConnections.set(peer, token); }
+    const native = peer instanceof WsPeer || peer instanceof CodexPeer || peer instanceof PiPeer ? peer.sessionGeneration : peer.recoveryMetadata?.().sessionId;
+    if (peer instanceof PiPeer && native === undefined) return undefined;
+    return `${instanceId}:${token}:${String(native ?? "")}`;
+  };
+  const currentCheck = (receipt: { task: number; owner: string; updated: number | string; generation: number | string; revision?: number }) => {
+    const task = board.get(receipt.task);
+    return !!task && task.owner === receipt.owner && task.state === "in_progress" && task.updated === receipt.updated && task.history.length === receipt.revision
+      && connectionGeneration(receipt.owner) === receipt.generation && may(receipt.owner, "check_run") && membershipAllowed(receipt.owner) && !tasks.isPii(task)
+      && !turns.get(receipt.owner)?.private && !contextPii(receipt.owner);
+  };
+  const boundedSafe = (text: string) => !looksSecret(text) && screenMemory([{ title: null, text }]).length === 1
+    && !mentionsPrivatePath(text, [], [...(config.pii?.paths ?? []), ...config.local.deny], opts.cwd);
+  const publicCheck = (receipt: RunReceipt) => {
+    const safeTail = boundedSafe(receipt.tail), safeError = !receipt.error || boundedSafe(receipt.error);
+    return { run: receipt.run, task: receipt.task, status: receipt.status, sha: receipt.sha, tree: receipt.tree, lockfile: receipt.lockfile,
+      excluded: receipt.excluded, runner: receipt.runner, image: receipt.image, platform: receipt.platform,
+      check: receipt.check, ...(receipt.files ? { files: receipt.files } : {}), exit: receipt.exit, timed_out: receipt.timed_out,
+      cancelled: receipt.cancelled, interrupted: receipt.interrupted, duration_ms: receipt.duration_ms,
+      tail: safeTail ? receipt.tail : "", output_bytes: receipt.output_bytes, truncated: receipt.truncated,
+      output_withheld: receipt.output_withheld || !safeTail || !safeError, ...(safeError && receipt.error ? { error: receipt.error } : {}) };
+  };
+  const checkOptions: RunnerOptions = {
+    projectDir: opts.cwd, stateDir: opts.stateDir, storageDir: join(hubHome(), "checks"),
+    projectKey: createHash("sha256").update(realPath(opts.cwd)).digest("hex").slice(0, 24),
+    enabled: () => config.checks.runner === "container",
+    isDenied: path => isDenied(path, config.local.deny) || matchesPolicy(opts.cwd, path, config.pii?.paths ?? [], true),
+    isCurrent: currentCheck,
+    onResult: async receipt => {
+      if (!currentCheck(receipt)) { await checkRunner.markStale(receipt.run); return; }
+      // Only a screened, current-owner result is delivered; no tail enters capture or diagnostic logs.
+      const view = publicCheck(receipt);
+      checkBindings.set(receipt.run, { task: receipt.task, owner: receipt.owner, generation: receipt.generation, updated: receipt.updated, revision: receipt.revision });
+      while (checkBindings.size > 200) checkBindings.delete(checkBindings.keys().next().value!);
+      if (stopping || !currentCheck(receipt)) { await checkRunner.markStale(receipt.run); return; }
+      bus.publish(newEnvelope(HUB, sanitize(JSON.stringify(view)), { to: [receipt.owner], kind: "task", refs: { task: String(receipt.task), boundedCheck: receipt.run } }));
+      log(`check ${receipt.run}: exit ${receipt.exit ?? "unknown"}, bytes ${receipt.output_bytes}`);
+    },
+  };
+  const checkRunner = opts.boundedTools?.runner?.(checkOptions) ?? new ContainerRunner(checkOptions);
+  const runnerRefreshAbort = new AbortController();
+  startupCleanup.push(() => { runnerRefreshAbort.abort(); void checkRunner.stopAll(); });
+  let runnerState = await checkRunner.available(runnerRefreshAbort.signal);
+  const orphanChecks = (await checkRunner.sweep()).map(line => boundedSafe(line) ? sanitize(line) : "orphan check: details withheld by privacy policy");
+  if (opts.boundedTools?.specAvailable || !opts.config && (config.pi.enabled || config.checks.runner === "container")) await ensureSpec();
+  let runnerStateAt = runnerState.checkedAt ?? Date.now(), refreshingRunner: Promise<void> | undefined;
+  const refreshRunner = (): Promise<void> => {
+    if (stopping || runnerRefreshAbort.signal.aborted) return Promise.resolve();
+    if (Date.now() - runnerStateAt < 60_000) return Promise.resolve();
+    if (refreshingRunner) return refreshingRunner;
+    refreshingRunner = checkRunner.available(runnerRefreshAbort.signal).then(value => {
+      if (stopping || runnerRefreshAbort.signal.aborted) return;
+      runnerState = value; runnerStateAt = value.checkedAt ?? Date.now();
+    }, () => {
+      if (stopping || runnerRefreshAbort.signal.aborted) return;
+      runnerState = { available: false, reason: "container unavailable" }; runnerStateAt = Date.now();
+    }).finally(() => { refreshingRunner = undefined; });
+    return refreshingRunner;
+  };
+  const peerCapabilities = () => {
+    // Keep the last confirmed capability during a bounded, single-flight refresh; replace it only on a result.
+    if (Date.now() - runnerStateAt >= 60_000) void refreshRunner();
+    return Object.fromEntries([...new Set([...SUPPORTED_PEERS, ...bus.peers.keys()])].map(peer => {
+    if (peer !== "pi" && peer !== "local") return [peer, { tests: true, spec: true }];
+    return [peer, { tests: runnerState.available && may(peer, "check_run"), spec: specState.available && may(peer, "spec_read"),
+      testsReason: !may(peer, "check_run") ? `capabilities.${peer}` : `runner unavailable: ${runnerState.reason ?? "unknown"}`,
+      specReason: !may(peer, "spec_read") ? `capabilities.${peer}` : specState.reason ?? "gh unauthenticated" }];
+    }));
+  };
   const tasks: Tasks = new Tasks({
+    capabilities: peerCapabilities, repository: memoryRepo,
     recordPlanning: change => event({ type: "planning", ...change }),
     wipLimit: peer => Object.hasOwn(config.tasks.wip_limits ?? {}, peer) ? config.tasks.wip_limits![peer]! : 1,
     profile: profileName(config.profile),
@@ -1094,6 +1237,10 @@ export async function startDaemon(opts: DaemonOptions) {
     review: config.review,
     sweep: taskSweepConfig(config.task_sweep),
     sweepHeld: () => stopping || recoveryActive(),
+    cancellationHeld: () => stopping || recoveryActive(),
+    offerRead: (peer, envelopeId) => journal.offerRead(peer, envelopeId),
+    offerPullOnly: (id) => { const peer = bus.peers.get(id); return peer instanceof WsPeer && peer.pullOnly; },
+    stallNotice: stall => { supervision.taskStalled(stall); },
     roles: config.roles,
     turnFree,
     capable: (peer) => capable.has(peer),
@@ -1116,6 +1263,21 @@ export async function startDaemon(opts: DaemonOptions) {
     held: () => Object.fromEntries(bus.knownPeers().flatMap((peer) => { const hold = queueHold(peer); return hold ? [[peer, hold]] : []; })),
   });
   relevantNotice = (peer, env) => {
+    if (env.from === HUB && env.refs?.boundedCheck) {
+      const run = env.refs.boundedCheck, binding = checkBindings.get(run);
+      let outputSafe = false;
+      try {
+        const value = JSON.parse(env.body) as { tail?: unknown; error?: unknown };
+        outputSafe = typeof value.tail === "string" && boundedSafe(value.tail)
+          && (value.error === undefined || typeof value.error === "string" && boundedSafe(value.error));
+      } catch { /* An unknown projection never reaches a model. */ }
+      if (!binding || binding.owner !== peer || !currentCheck(binding) || !outputSafe) {
+        checkBindings.delete(run);
+        void checkRunner.markStale(run).catch(() => {});
+        return false;
+      }
+      return true;
+    }
     if (env.from !== HUB || !env.refs?.supervision) return tasks.relevant(peer, env);
     if (conductorPeer(config.roles) !== peer || config.conductor.feed === "off") return false;
     const key = env.refs.supervisionKey ?? "";
@@ -1139,7 +1301,7 @@ export async function startDaemon(opts: DaemonOptions) {
   const facts: Facts = new Facts({ root: opts.cwd, repo: factsRepo, tmp: join(opts.stateDir, "facts"), instance: instanceId.slice(0, 8), scope: (peer) => repositoryFacts ? (scopes.has(peer) ? scopes.get(peer) : scopes.set(peer, tasks.factScope(peer)).get(peer)) : undefined, peers: () => [...bus.peers.keys()], nameable: tasks.nameable, deny: config.local.deny, privatePath: path => matchesPolicy(opts.cwd, path, config.pii?.paths ?? []), projection: file => projections.project(file) });
   const progress = new ProgressObserver({
     tasks: () => board.list(),
-    isPrivate: () => board.list().some(task => task.state !== "approved" && tasks.isPii(task)),
+    isPrivate: () => [...turns.values()].some(turn => turn.private) || board.list().some(task => !isClosedTaskState(task.state) && tasks.isPii(task)),
     inference: { escalate: (conversation, turn) => inference?.escalate(conversation, turn) ?? Promise.resolve(undefined) },
     emit: conductorProgressSink(event, supervision),
     notify,
@@ -1322,7 +1484,7 @@ export async function startDaemon(opts: DaemonOptions) {
     return undefined;
   };
   const activityRings = { kimi: new ActivityRing(), qwen: new ActivityRing() };
-  const contextPii = (peer: PeerId) => board.list().some(t => t.state !== "approved" && (t.owner === peer || t.reviewer === peer) && tasks.isPii(t));
+  const contextPii = (peer: PeerId) => !!turns.get(peer)?.private || board.list().some(t => !isClosedTaskState(t.state) && (t.owner === peer || t.reviewer === peer) && tasks.isPii(t));
   const requestCheckpoint = (peer: PeerId, reason: "quota" | "context"): Promise<string | undefined> => {
     // One outstanding request per peer. A quota transition may proceed without a summary rather than steal a context waiter.
     if (checkpointWaits.has(peer) || recoveryActive() || stopping) return Promise.resolve(undefined);
@@ -1416,7 +1578,6 @@ export async function startDaemon(opts: DaemonOptions) {
   // Turns and their tokens for telemetry (issue #40). Turn ids carry the hub run, so they stay unique across restarts.
   const runId = Date.now().toString(36);
   let turnSeq = 0;
-  const turns = new Map<PeerId, { id: string; start: number; tokens: number; tree?: string; snapshotMs?: number; private?: boolean; deliveryTask?: number; attribution: TaskAttribution; preparedRoles: RoleIdentity[] | null }>();
   const pendingDeliveryTask = new Map<PeerId, number>();
   const taskAttribution = (peer: PeerId, delivery = turns.get(peer)?.deliveryTask): TaskAttribution => {
     const result = attribute(delivery, board.list("in_progress").filter(t => t.owner === peer).map(t => t.id));
@@ -1448,7 +1609,7 @@ export async function startDaemon(opts: DaemonOptions) {
   const turnLog = repo ? new Turns(join(opts.stateDir, "hub.db")) : undefined;
   if (turnLog) startupCleanup.push(() => turnLog.close());
   /** Whether a peer has a PII task open in any state: assigned and not yet accepted counts. */
-  const holdsPii = (peer: PeerId) => board.list().some((t) => t.owner === peer && t.state !== "approved" && t.state !== "in_review" && tasks.isPii(t));
+  const holdsPii = (peer: PeerId) => !!turns.get(peer)?.private || board.list().some((t) => t.owner === peer && !isClosedTaskState(t.state) && t.state !== "in_review" && tasks.isPii(t));
   const memoryObservations = memoryTelemetry ? new MemoryObservations(memoryTelemetry, (peer, envs) => {
     if (turns.get(peer)?.private || contextPii(peer) || envs.some(env => env.private)) return;
     for (const env of envs) {
@@ -1460,7 +1621,7 @@ export async function startDaemon(opts: DaemonOptions) {
     if (row && tasks.isPii(row)) return;
     return { ...(row ? { task: taskKey(memoryProject, row) } : {}), turn: turns.get(peer)?.id ?? null, session: contextSession(peer) ?? null };
   }) : undefined;
-  bus.onMemoryDelivery = (peer, id, out, field) => memoryObservations?.delivery(peer, id, out, field);
+  bus.onMemoryDelivery = (peer, id, out, field) => { if (!out.some(env => env.refs?.boundedCheck)) memoryObservations?.delivery(peer, id, out, field); };
   const citeMemory = (peer: PeerId, text: string, id: string, envs: readonly Envelope[] = []) => {
     if (!tasks.nameable(text) || looksSecret(text) || !screenMemory([{ title: null, text }]).length) return;
     memoryObservations?.cite(peer, text, id, envs);
@@ -1482,7 +1643,7 @@ export async function startDaemon(opts: DaemonOptions) {
     return { tree, ms: Math.round(performance.now() - t0) };
   };
   function privateSnapshotWindow(): boolean {
-    return [...turns.values()].some(turn => turn.private) || board.list().some(task => task.state !== "approved" && tasks.isPii(task));
+    return [...turns.values()].some(turn => turn.private) || board.list().some(task => !isClosedTaskState(task.state) && tasks.isPii(task));
   }
   function reviewProjection(task: Task): string | undefined {
     if (!projections.enabled || tasks.isPii(task)) return undefined;
@@ -1525,6 +1686,7 @@ export async function startDaemon(opts: DaemonOptions) {
   const contextTailSeen = new Map<PeerId, string>();
   const intervals = [
     setInterval(() => {
+      void refreshRunner().catch(() => {});
       try { tasks.flushPlannerNotices(); } catch { /* the board remains authoritative if notice delivery fails */ }
       for (const pending of checkpointWaits.values()) if (!pending.valid()) pending.done(undefined);
       for (const [id, request] of permissions) if (request.expiresAt > Date.now()) supervision.approvalWaiting({ id, peer: request.peer, tool: request.tool ?? "unknown", createdAt: request.createdAt });
@@ -1607,7 +1769,7 @@ export async function startDaemon(opts: DaemonOptions) {
   // names are answered by the hub; every call still passes daemon role/capability checks before its effects.
   const HUB_TOOLS = ["hub_send", ...TASK_TOOLS.map((t) => t.name), ...CONDUCTOR_TOOLS.map((t) => t.name)];
   const HUB_TOOL_TITLES = new Set(HUB_TOOLS.map((name) => `mcp__agent-hub__${name}`));
-  const toolEnv = (peer: PeerId) => ({ AGENTHUB_MODE: "tools", AGENTHUB_PEER_ID: peer, AGENTHUB_STATE_DIR: opts.stateDir, AGENTHUB_PROJECT_DIR: opts.cwd });
+  const toolEnv = (peer: PeerId, deliveryOnly = false) => ({ ...(deliveryOnly ? { AGENTHUB_DELIVERY_ONLY: "true" } : {}), AGENTHUB_MODE: "tools", AGENTHUB_PEER_ID: peer, AGENTHUB_STATE_DIR: opts.stateDir, AGENTHUB_PROJECT_DIR: opts.cwd });
 
   /** One entry point for the task tools, whoever calls them: MCP clients, the local worker, the console. */
   // A task op can write the board across awaits (triage, briefs, the dependents an approval releases): a recovery
@@ -1627,11 +1789,21 @@ export async function startDaemon(opts: DaemonOptions) {
         const id = Number.isSafeInteger(proposed) && proposed! > 0 ? proposed : admittedId;
         auditConduct({ kind: "conduct", actor: by, action: op.slice(4), ...(id === undefined ? {} : { task: id }) });
       }
-      return text;
+      const hint = args[3] ? ownHeldHint(by) : "";
+      return hint ? `${text}\n\n${hint}` : text;
+    } catch (error) {
+      // Keep proven pre-effect refusal identity/code for native receipts; only add body-free metadata.
+      const hint = args[3] ? ownHeldHint(by) : "";
+      if (hint && error instanceof Error) error.message += `\n\n${hint}`;
+      throw error;
     } finally {
       taskOpsInFlight--;
     }
   };
+  function ownHeldHint(peer: PeerId): string {
+    try { return renderHeldHint(bus.heldDeliveryHints(peer), Date.now(), !(bus.peers.get(peer) instanceof LocalPeer || bus.peers.get(peer) instanceof AcpPeer)); }
+    catch { return ""; } // unavailable telemetry is not a zero count or a reason to fail the tool itself
+  }
   /** A model-written peer argument: a peer id, or nothing for absent, null or "". Anything else is refused. */
   const peerArg = (v: unknown, name: string): PeerId | undefined => {
     if (v == null || v === "") return undefined;
@@ -1640,6 +1812,12 @@ export async function startDaemon(opts: DaemonOptions) {
     return id;
   };
   async function taskOpBody(by: PeerId, op: string, a: Record<string, any>, inProcess = false, piiTurn = false): Promise<string> {
+    if (BOUNDED_TASK_TOOLS.has(op)) {
+      const keys = op === "hub_task_spec" ? ["id"] : op === "hub_check_start" ? ["id", "check", "files"] : ["run"];
+      if (!a || typeof a !== "object" || Array.isArray(a) || Object.keys(a).some(key => !keys.includes(key))
+        || (keys.includes("id") ? !Number.isSafeInteger(a.id) || a.id < 1 : typeof a.run !== "string" || !/^[a-f0-9]{8}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{12}$/.test(a.run))) throw new PreEffectToolRefusal("invalid bounded task request");
+      if (by !== USER && (piiTurn || turns.get(by)?.private || contextPii(by))) throw new PreEffectToolRefusal("bounded task tools unavailable during a PII turn");
+    }
     if (MEMORY_READ_TOOLS.has(op)) validateMemoryRead(op, a);
     if (op === "hub_memory_correct") correctionInput(a);
     if (CONDUCTOR_TOOL_NAMES.has(op)) {
@@ -1660,7 +1838,7 @@ export async function startDaemon(opts: DaemonOptions) {
     // console reads a PII task's text deliberately, with `ahub task show <id>`.
     const onPrem = inProcess && by === "local" && piiTurn;
     const line = (t: { id: number; state: string; owner: PeerId | null; reviewer: PeerId | null }) => { const task = board.get(t.id); return task && by === USER ? tasks.resultLine(task) : `task #${t.id}: ${t.state}, owner ${t.owner ?? "none"}, reviewer ${t.reviewer ?? "none"}`; };
-    const need = (cap: "propose" | "assign" | "remember" | "memory_read" | "memory_correct", what: string) => {
+    const need = (cap: "propose" | "assign" | "remember" | "memory_read" | "memory_correct" | "spec_read" | "check_run", what: string) => {
       if (may(by, cap)) return;
       log(`capabilities: ${by} may not ${what} (${op})`);
       throw new PreEffectToolRefusal(`${by} may not ${what} (no "${cap}" in capabilities.${by} in .agenthub/config.json)`);
@@ -1678,7 +1856,88 @@ export async function startDaemon(opts: DaemonOptions) {
     }
     if (op === "hub_memory_correct") { need("memory_correct", "propose memory corrections"); return tasks.correct(by, a); }
     if (op === "hub_remember") need("remember", "save notes to shared memory");
+    if (op === "hub_task_spec") {
+      need("spec_read", "read assigned specifications");
+      const authorized = () => {
+        const task = board.get(a.id);
+        if (!task || task.state === "backlog" || task.owner !== by && task.reviewer !== by || tasks.isPii(task) || turns.get(by)?.private || contextPii(by)) throw new PreEffectToolRefusal("spec unavailable");
+        return task;
+      };
+      const task = authorized(), generation = connectionGeneration(by), updated = task.updated, revision = task.history.length;
+      const deadline = Date.now() + 25_000; // retain a response margin inside the control client's 30-second cap
+      const issues = specRefs(task, memoryRepo);
+      if (!issues.length) throw new PreEffectToolRefusal("no assigned spec references");
+      await ensureSpec();
+      authorized();
+      if (!specState.available) throw new PreEffectToolRefusal(specState.reason ?? "gh unauthenticated");
+      const specs = [];
+      for (const issue of issues) {
+        const remaining = deadline - Date.now();
+        if (remaining <= 0) throw new PreEffectToolRefusal("timeout");
+        specs.push(await (opts.boundedTools?.fetchSpec ?? fetchSpec)(memoryRepo, issue, { screen: boundedSafe, projectDir: opts.cwd, timeoutMs: Math.min(15_000, remaining) }));
+        const current = authorized();
+        if (current.updated !== updated || current.history.length !== revision || connectionGeneration(by) !== generation || !may(by, "spec_read")) throw new PreEffectToolRefusal("spec unavailable");
+      }
+      const result = JSON.stringify({ specs });
+      if (Buffer.byteLength(result) > 80 * 1024) throw new PreEffectToolRefusal("too large");
+      if (!specs.every(spec => boundedSafe(`${spec.title}\n${spec.body}`))) throw new PreEffectToolRefusal("spec withheld by privacy policy");
+      return result;
+    }
+    if (op === "hub_check_start") {
+      need("check_run", "run isolated checks");
+      if (stopping || recoveryActive()) throw new PreEffectToolRefusal("runner unavailable: recovery or shutdown");
+      const authorized = () => {
+        const task = board.get(a.id);
+        if (!task || task.owner !== by || task.state !== "in_progress" || tasks.isPii(task) || turns.get(by)?.private || contextPii(by)) throw new PreEffectToolRefusal("check unavailable");
+        return task;
+      };
+      const task = authorized(), updated = task.updated, revision = task.history.length, generation = connectionGeneration(by);
+      if (!generation) throw new PreEffectToolRefusal("check requires an attached owner");
+      if (a.check !== "test" && a.check !== "typecheck"
+        || a.check === "test" && (!Array.isArray(a.files) || a.files.length < 1 || a.files.length > 32 || a.files.some((file: unknown) => typeof file !== "string" || !/^test\/[A-Za-z0-9._/-]+\.test\.ts$/.test(file) || file.split("/").some(part => !part || part === "." || part === "..")))
+        || a.check === "typecheck" && a.files !== undefined) throw new PreEffectToolRefusal("invalid check request");
+      await refreshRunner();
+      const current = authorized();
+      if (current.updated !== updated || current.history.length !== revision || connectionGeneration(by) !== generation || !may(by, "check_run") || stopping || recoveryActive()) throw new PreEffectToolRefusal("check unavailable");
+      if (!runnerState.available) throw new PreEffectToolRefusal(`runner unavailable: ${runnerState.reason ?? "unknown"}`);
+      const ref = current.refs.commit ?? current.refs.branch;
+      if (!ref) throw new PreEffectToolRefusal("check requires task branch or commit");
+      const handle = checkRunner.start({ task: current.id, owner: by, generation, updated, revision: current.history.length, ref, check: a.check, ...(a.files === undefined ? {} : { files: a.files }) });
+      checkPending.add(handle.run);
+      void handle.done.finally(() => checkPending.delete(handle.run)).catch(() => {});
+      return JSON.stringify({ run: handle.run });
+    }
+    if (op === "hub_check_result" || op === "hub_check_cancel") {
+      need("check_run", "read or cancel isolated checks");
+      let receipt = await checkRunner.result(a.run);
+      const task = receipt ? board.get(receipt.task) : undefined;
+      if (!receipt || by !== USER && (receipt.owner !== by || !task || tasks.isPii(task))) throw new PreEffectToolRefusal("check unavailable");
+      if (receipt.status === "stale" && by !== USER) throw new PreEffectToolRefusal("check unavailable: stale binding");
+      if (!currentCheck(receipt)) {
+        await checkRunner.markStale(receipt.run); receipt = { ...receipt, status: "stale" };
+        // markStale persists and can abort a run: a refusal after it is never proof of absent effects.
+        if (by !== USER) throw new Error("check unavailable: stale binding");
+      }
+      if (op === "hub_check_cancel") {
+        await checkRunner.cancel(receipt.run);
+        const actual = await checkRunner.result(receipt.run);
+        if (!actual) throw new Error("check receipt unavailable after cancellation");
+        return JSON.stringify({ run: actual.run, status: actual.status, cancelled: actual.cancelled });
+      }
+      if (by !== USER && !currentCheck(receipt)) throw new PreEffectToolRefusal("check unavailable: stale binding");
+      if (by === receipt.owner) await checkRunner.consumeOutput(receipt.run);
+      if (by !== USER && (!currentCheck(receipt) || turns.get(by)?.private || contextPii(by))) throw new Error("check unavailable: stale binding");
+      // Policy can change during output cleanup, so form the final projection again.
+      return JSON.stringify(publicCheck(receipt));
+    }
     switch (op) {
+      case "hub_delivery_retry": {
+        if (by === USER || by === HUB || !a || typeof a !== "object" || Array.isArray(a) || Object.keys(a).some(key => key !== "delivery_id") || typeof a.delivery_id !== "string" || a.delivery_id.length < 1 || a.delivery_id.length > 256) throw new PreEffectToolRefusal("hub_delivery_retry accepts only your delivery_id; completed and discard require the person");
+        if (bus.peers.get(by) instanceof LocalPeer || bus.peers.get(by) instanceof AcpPeer) throw new PreEffectToolRefusal("delivery-only peers require the person to resolve held deliveries");
+        if (stopping || recoveryActive()) throw new PreEffectToolRefusal("recovery or shutdown is holding delivery mutations");
+        bus.retryDelivery(by, a.delivery_id);
+        return `retry queued for delivery ${a.delivery_id}; check for earlier effects before acting`;
+      }
       case "hub_backlog_list":
         return JSON.stringify(tasks.backlogList(by));
       case "hub_release_list":
@@ -1710,6 +1969,14 @@ export async function startDaemon(opts: DaemonOptions) {
         if (silent && factsOn()) facts.sawPlans(by, tasks.overlapTasks(t)); // the plans in this answer need no fact later
         return [line(t), overlap, tasks.planGuardText(t)].filter(Boolean).join("\n");
       }
+      case "hub_task_cancel": {
+        if (!a || typeof a !== "object" || Array.isArray(a) || Object.keys(a).some(key => !["id", "reason", "superseded_by", "revision"].includes(key))
+          || !Number.isSafeInteger(a.id) || a.id < 1 || typeof a.reason !== "string" || !a.reason.trim() || a.reason.length > 2000
+          || a.superseded_by !== undefined && (typeof a.superseded_by !== "string" || !a.superseded_by.trim() || a.superseded_by.length > 2000)
+          || a.revision !== undefined && (!Number.isSafeInteger(a.revision) || a.revision < 0)) throw new PreEffectToolRefusal("invalid cancellation request");
+        if (by !== USER && (piiTurn || turns.get(by)?.private || contextPii(by))) throw new PreEffectToolRefusal("cancellation unavailable during a PII turn");
+        return JSON.stringify(tasks.publicView(await tasks.cancel(a.id, by, { reason: a.reason, supersededBy: a.superseded_by, expectedFlowRevision: a.revision })));
+      }
       case "hub_task_decline":
         return line(await tasks.decline(by, a.id, a.reason));
       case "hub_task_done": {
@@ -1814,8 +2081,13 @@ export async function startDaemon(opts: DaemonOptions) {
       }
       case "task_escalate":
         return line(await tasks.escalate(USER, a.id, undefined, undefined, true));
-      case "route_explain":
-        return tasks.explain(a.id !== undefined ? Number(a.id) : { title: String(a.title ?? ""), class: a.class as TaskClass }).join("\n");
+      case "route_explain": {
+        const trace = tasks.explain(a.id !== undefined ? Number(a.id) : { title: String(a.title ?? ""), class: a.class as TaskClass });
+        return [...trace, ...bus.knownPeers().flatMap(peer => {
+          if (trace.some(line => line.startsWith(`  hold: ${peer}'s queue is held:`))) return [];
+          const hold = queueHold(peer); return hold ? [`${peer} queue: ${hold}`] : [];
+        })].join("\n");
+      }
       case "turn_revert": {
         // `ahub undo --context` (issue #33): the conversation half of an undo; the CLI restores the files itself.
         const turn = turnLog?.get(String(a.turn));
@@ -1840,7 +2112,7 @@ export async function startDaemon(opts: DaemonOptions) {
   }
   function recoveryTaskPreface(peer: PeerId): void {
     if (recoveryPhase !== "restored") return;
-    const open = board.list().filter((task) => (task.owner === peer || task.reviewer === peer) && !["approved"].includes(task.state));
+    const open = board.list().filter((task) => (task.owner === peer || task.reviewer === peer) && !isClosedTaskState(task.state));
     if (!open.length) return;
     const lines = open.map((task) => {
       const view = tasks.publicView(task) as { title?: unknown; detail?: unknown; state?: unknown; owner?: unknown; reviewer?: unknown };
@@ -1875,6 +2147,7 @@ export async function startDaemon(opts: DaemonOptions) {
   let releasing = false; // gone-owner release (#6) and the ready sweep (#34): one run at a time, and a recovery commit waits for it
   const recoveryReady = () => {
     if (!recoveryActive() || releasing || taskOpsInFlight !== 0 || tasks.checksPending() !== 0 || (piReceipts?.inFlight ?? 0) !== 0 || permissions.size !== 0 || starting.size !== 0 || requestedPeerStops.size !== 0 || rosterChanging !== undefined || !budget.recoverySettled || [...bus.peers.values()].some((peer) => peer.state === "busy" || (peer instanceof PiPeer && !peer.recoveryReady))) return false;
+    if (checkPending.size !== 0) return false;
     if (!recoveryPeerSnapshot) return true;
     const current = recoveryPeers();
     return recoveryPeerSnapshot.every((saved) => {
@@ -2029,7 +2302,7 @@ export async function startDaemon(opts: DaemonOptions) {
     }
     return now;
   };
-  const queueHoldStatus = (peer: PeerId, heldBy?: string) => heldBy ? { heldBy, holdNote: queueHold(peer) } : {};
+  const queueHoldStatus = (peer: PeerId, heldBy?: string) => heldBy ? { heldBy, holdNote: queueHold(peer), heldRows: heldRows(peer) } : {};
   const logPermissionStart = (peer: string) => {
     const requested = defaultSources[peer];
     const selected = permissionMode(peer), pending = pendingPermissionDefaults.has(peer);
@@ -2062,6 +2335,8 @@ export async function startDaemon(opts: DaemonOptions) {
   };
   const status = (includeAttentionItems = false, includeAttentionInput = false) => {
     const taskNeeds = board.statusTasks();
+    const taskStalls = tasks.stalled();
+    const pendingTaskOffers = tasks.pendingOffers().filter(offer => offer.pullOnly === true);
     const peerIds = statusPeerIds(taskNeeds);
     const view = {
     ...(crashReport.length ? { crash: crashReport } : {}),
@@ -2071,6 +2346,7 @@ export async function startDaemon(opts: DaemonOptions) {
     memoryProfile: memoryProfileStatus(includeAttentionItems),
     lifecycleStages: [...lifecycleConfig.stages],
     protocol: PROTOCOL,
+    features: DAEMON_FEATURES,
     stopping,
     pid: process.pid,
     cwd: opts.cwd,
@@ -2082,11 +2358,15 @@ export async function startDaemon(opts: DaemonOptions) {
     ),
     taskWip: Object.fromEntries(peerIds.map(id => [id, { wip: tasks.wip(id), wipLimit: tasks.wipLimit(id) }])),
     boardReadCapabilities: { releaseFilter: true, columnMetadata: true },
-    roleInfo: Object.fromEntries(peerIds.map(id => [id, { flowRoles: config.roles[id] ?? [], catalogRoles: roleState(id) }])),
+    roleInfo: Object.fromEntries(peerIds.map(id => [id, { flowRoles: config.roles[id] ?? [], catalogRoles: roleState(id), ...(config.roles[id]?.includes("planner") ? { planner: plannerSource } : {}) }])),
+    planner: { peers: plannerPeers(), source: plannerSource },
     ...(pendingPermissionDefaults.size ? { permissionDefaults: [...pendingPermissionDefaults].map(([peer, source]) => ({ peer, mode: "never-ask", source })) } : {}),
     ...(sidecar ? { switchyard: sidecar.status } : {}),
     ...(modelRelay ? { models: modelRelay.status() } : {}),
+    isolatedChecks: { pending: checkPending.size, orphans: orphanChecks, unknownReceipts: checkRunner.unknownReceipts ?? [], maintenanceUnavailable: checkRunner.maintenanceUnavailable === true },
+    taskCapabilities: peerCapabilities(),
     tasks: board.counts(),
+    taskStalls, pendingTaskOffers,
     ...(bus.storageError ? { deliveryError: bus.storageError } : {}),
     ...(recoveryOperationId ? { recovery: { operationId: recoveryOperationId, phase: recoveryPhase, ready: recoveryReady() } } : {}),
     };
@@ -2103,6 +2383,7 @@ export async function startDaemon(opts: DaemonOptions) {
       storageBlocked: !!bus.storageError,
       recovery: recoveryOperationId ? { id: recoveryOperationId, running: true } : undefined,
       tasks: taskNeeds,
+      facts: [...taskStalls.map(stall => ({ kind: "stall" as const, id: stall.task })), ...pendingTaskOffers.map(offer => ({ kind: "offer" as const, id: offer.task, peer: offer.peer }))],
       budget: Object.fromEntries(Object.entries(budget.status()).map(([peer, row]) => [peer, row.paused ? { paused: { since: row.paused.since, resetsAt: row.paused.resetsAt } } : {}])),
     };
     const attention = needs(attentionInput, Date.now());
@@ -2122,37 +2403,58 @@ export async function startDaemon(opts: DaemonOptions) {
   const taskChanged = board.onChange;
   board.onChange = (task, entry) => {
     try { taskChanged?.(task, entry); }
-    finally { writeStatus(); } // committed task transitions refresh token-free attention even without bus activity
+    finally {
+      try { settleSupersededDeliveries(); }
+      finally { writeStatus(); } // committed task transitions refresh token-free attention even without bus activity
+    }
   };
 
   // A queue change is not a bus event: without this, status.json reports the queue as it was before the last
   // message and keeps a phantom `queued N` after the queue drains (measured on a live 0.6.3 hub).
-  function movedDeliveryTasks(row: JournalDelivery): string[] {
-    return [...new Set(row.originals.flatMap((env) => {
-      const task = env.refs?.task ? board.get(Number(env.refs.task)) : undefined;
-      if (!task || (task.owner === row.peer && task.state !== "approved")) return [];
-      return [`task ${tasks.publicTitle(task)} ${task.state === "approved" ? "has been approved" : `has moved to ${task.owner ?? "nobody"}`}`];
-    }))];
+  function heldRows(peer: PeerId) {
+    return bus.queueList(peer).filter(row => row.state === "needs_review").map(row => ({
+      id: row.id, senders: [...new Set(row.originals.map(env => sanitize(env.from)))],
+      kinds: [...new Set(row.originals.map(env => env.kind))], createdAt: row.createdAt,
+      context: deliveryTaskContext(row.originals, board, peer),
+    }));
   }
   function queueHold(peer: PeerId): string | undefined {
-    const id = bus.queueSummary(peer).heldBy;
-    const row = id ? bus.queueShow(id) : undefined;
-    if (!row) return undefined;
-    const moved = movedDeliveryTasks(row);
-    return `held by needs_review ${id}${moved.length ? ` (${moved.join("; ")})` : ""}; ahub queue resolve ${id} --action completed|retry|discard --reason <text>`;
+    const rows = heldRows(peer);
+    if (!rows.length) return undefined;
+    return rows.map(row => `held by needs_review ${row.id} (from ${row.senders.join(",")}; kind ${row.kinds.join(",")}; age ${Math.max(0, Math.floor((Date.now() - row.createdAt) / 1000))}s${row.context ? `; ${row.context}` : ""}); ahub queue resolve ${row.id} --action completed|retry|discard --reason <text>`).join("; ");
+  }
+  let settlingSuperseded = false;
+  function settleSupersededDeliveries() {
+    if (settlingSuperseded || stopping || recoveryActive() || bus.isRecoveryHeld) return;
+    settlingSuperseded = true;
+    try {
+      for (const row of bus.queueList()) {
+        if (row.state !== "needs_review" || row.reason?.startsWith("requested stop:")) continue;
+        const reason = supersededDeliveryReason(row.originals, board, row.peer);
+        if (!reason) continue;
+        try { bus.resolveDelivery(row.id, row.revision, "completed", reason, "hub"); }
+        catch { /* Busy peers, storage failures and changed revisions remain held for the next queue/board change. */ }
+      }
+    } finally { settlingSuperseded = false; }
   }
   const noticedHolds = new Set<string>();
+  let reportingQueues = false;
   bus.onQueues = () => {
-    if (stopping) return;
-    for (const row of bus.queueList()) {
-      if (row.state !== "needs_review" || noticedHolds.has(row.id)) continue;
-      noticedHolds.add(row.id);
-      supervision.needsReview(row.peer, row.id);
-      const moved = movedDeliveryTasks(row);
-      notify(`${row.peer} queue held by needs_review ${row.id}${moved.length ? ` (${[...new Set(moved)].join("; ")})` : ""}; inspect: ahub queue show ${row.id}; resolve: ahub queue resolve ${row.id} --action completed|retry|discard --reason <text>`);
-    }
-    writeStatus();
+    if (stopping || settlingSuperseded || reportingQueues) return;
+    reportingQueues = true;
+    try {
+      settleSupersededDeliveries();
+      for (const peer of bus.knownPeers()) {
+        const rows = heldRows(peer);
+        if (!rows.some(row => !noticedHolds.has(row.id))) continue;
+        for (const row of rows) { noticedHolds.add(row.id); supervision.needsReview(peer, row.id); }
+        notify(`${peer} queue ${queueHold(peer)}`);
+      }
+      writeStatus();
+    } finally { reportingQueues = false; }
   };
+  // Opening the journal/restoring the snapshot preceded callbacks; classify those holds too.
+  settleSupersededDeliveries();
 
   // When each peer went offline; a peer never seen attached counts from hub start (issue #6).
   const offlineSince = new Map<PeerId, number>();
@@ -2580,6 +2882,7 @@ export async function startDaemon(opts: DaemonOptions) {
     if (!startable(peer)) return { ok: false, error: "unknown peer; the hub starts claude, codex, kimi, pi (when enabled) and local" };
     if (recoveryActive()) return { ok: false, error: "recovery is holding mutations" };
     const mode = startMode(peer);
+    if (mode === "tui" && (peer === "pi" || peer === "local")) { await ensureSpec(true); await refreshRunner(); }
     const owner = bus.peers.get(peer);
     if (mode === "headless") {
       // A Pi a person runs in a terminal is theirs: a start made for them never takes it over as a headless one.
@@ -2662,6 +2965,7 @@ export async function startDaemon(opts: DaemonOptions) {
   async function startPeerOnce(peer: string, args: { model?: string; route?: string; mode?: "headless" | "tui"; backend?: "auto" | "dgx" | "mlx"; sessionId?: string; sessionFile?: string; replaceSession?: boolean; fresh?: boolean; unattended?: boolean; providerProfile?: string }): Promise<Record<string, unknown>> {
     let unmute: (() => void) | undefined;
     try {
+      if (peer === "pi" || peer === "local") { await ensureSpec(true); await refreshRunner(); }
       return await startPeerBody(peer, args, (p) => { unmute = muteState(p); });
     } finally {
       unmute?.();
@@ -2776,7 +3080,7 @@ export async function startDaemon(opts: DaemonOptions) {
             : { source: "acp_usage_update", sessionId, measuredAt: Date.now(), tokens: null, window: null, used: null }, current);
         },
         onTurnFailure: () => { supervisionTurns.delete("qwen"); },
-        mcpServers: [{ name: "agent-hub", command: "bun", args: ["run", SERVER_JS], env: Object.entries(toolEnv("qwen")).map(([name, value]) => ({ name, value })) }],
+        mcpServers: [{ name: "agent-hub", command: "bun", args: ["run", SERVER_JS], env: Object.entries(toolEnv("qwen", true)).map(([name, value]) => ({ name, value })) }],
         preamble: sessionRoleContract("qwen"),
       });
       recoveryTaskPreface("qwen"); await ensurePreface("qwen");
@@ -2828,7 +3132,7 @@ export async function startDaemon(opts: DaemonOptions) {
             current);
         },
         onTurnFailure: () => { supervisionTurns.delete("kimi"); },
-        mcpServers: [{ name: "agent-hub", command: "bun", args: ["run", SERVER_JS], env: Object.entries(toolEnv("kimi")).map(([name, value]) => ({ name, value })) }],
+        mcpServers: [{ name: "agent-hub", command: "bun", args: ["run", SERVER_JS], env: Object.entries(toolEnv("kimi", true)).map(([name, value]) => ({ name, value })) }],
         preamble: sessionRoleContract("kimi"),
       });
       recoveryTaskPreface("kimi");
@@ -3364,7 +3668,7 @@ export async function startDaemon(opts: DaemonOptions) {
       tasks: boardTasks.map((task) => {
         const view = tasks.publicView(task);
         // Refs and history can themselves quote private text. The dashboard needs neither.
-        return { id: task.id, title: view.title, detail: view.detail, class: task.class, ...(task.profile ? { profile: task.profile, profileClass: task.profileClass } : {}), ...(task.role ? { role: task.role } : {}), state: task.state, owner: task.owner, reviewer: task.reviewer, deps: view.deps ?? [], ...(task.reserved ? { reserved: task.reserved } : {}), planningRevision: task.planningRevision ?? 0, rank: task.rank ?? null, priority: task.priority ?? null, release: task.release ?? null, plannedStart: task.plannedStart ?? null, due: task.due ?? null, pinned: task.pinned === true, actualDates: view.actualDates, ...(view.external ? { external: view.external } : {}), started: taskHasStarted(task), urgent: task.signals.includes("urgent"), updated: task.updated, reworkRound: task.history.filter(entry => entry.event === "changes_requested").length, checking: tasks.isChecking(task.id), ...(view.lifecycle ? { lifecycle: view.lifecycle } : {}) };
+        return { id: task.id, title: view.title, detail: view.detail, class: task.class, ...(task.profile ? { profile: task.profile, profileClass: task.profileClass } : {}), ...(task.role ? { role: task.role } : {}), state: task.state, owner: task.owner, reviewer: task.reviewer, deps: view.deps ?? [], ...(task.reserved ? { reserved: task.reserved } : {}), planningRevision: task.planningRevision ?? 0, rank: task.rank ?? null, priority: task.priority ?? null, release: task.release ?? null, plannedStart: task.plannedStart ?? null, due: task.due ?? null, pinned: task.pinned === true, actualDates: view.actualDates, ...(view.cancelled ? { cancelled: view.cancelled } : {}), ...(view.external ? { external: view.external } : {}), started: taskHasStarted(task), urgent: task.signals.includes("urgent"), updated: task.updated, reworkRound: task.history.filter(entry => entry.event === "changes_requested").length, checking: tasks.isChecking(task.id), ...(view.lifecycle ? { lifecycle: view.lifecycle } : {}) };
       }),
       links: links.filter(link => link.tasks.every(id => { const task = board.get(id); return task && !tasks.isPii(task); })),
       budget: Object.fromEntries(peers.map((id) => [id, { ...(quota[id] ?? { windows: [] }), context: contexts.view(id, contextSession(id), bus.peers.has(id) && ["idle", "busy", "paused"].includes(bus.stateOf(id))) }])),
@@ -3399,6 +3703,8 @@ export async function startDaemon(opts: DaemonOptions) {
     return storedConfigCache.config;
   };
   /** Stored start modes and the scoped instruction policy follow a settings write or undo. */
+  /** #455: the stored planner key, applied at once; planning authority and backlog notices follow config.roles. */
+  const applyPlanner = () => { const stored = storedConfig(); if (stored) { config.planner = stored.planner; settlePlanner(); writeStatus(); } };
   const applyStartModes = () => { const stored = storedConfig(); if (stored) { config.peers = stored.peers; config.protect_instruction_files = stored.protect_instruction_files; } };
   /** Apply only the named peer; unrelated hand edits retain their prior publication scope. */
   const applyRoleSelection = (peer: string) => {
@@ -3435,7 +3741,8 @@ export async function startDaemon(opts: DaemonOptions) {
       },
     });
     const undo = pendingUndo(opts.stateDir);
-    return { rows, ...(undo ? { undo } : {}) };
+    const planner = `In force: ${plannerPeers().join(", ") || "nobody"} (${plannerSource}). Setting it makes that peer the only planner; agents cannot change it.`;
+    return { rows: rows.map(row => row.key === "planner" ? { ...row, hint: planner } : row), ...(undo ? { undo } : {}) };
   }
   /** The same announcement whatever surface made the change: events.jsonl, hub.log and every open console. */
   const recordSetting = (key: string, from: SettingValue, to: SettingValue, source: "dashboard" | "terminal", undo = false) => {
@@ -3444,7 +3751,7 @@ export async function startDaemon(opts: DaemonOptions) {
   };
   const settingSource = (authority: SettingAuthority) => authority === "terminal" ? "terminal" as const : "dashboard" as const;
   function rosterView() {
-    const open = board.list().filter(task => task.state !== "approved");
+    const open = board.list().filter(task => !isClosedTaskState(task.state));
     return [...KNOWN_PEERS, ...OPTIONAL_PEERS.filter(peer => config.roster?.[peer] !== undefined)].map(peer => ({ peer, on: onRoster(peer), startMode: startMode(peer), state: bus.stateOf(peer), queued: bus.queued(peer), tasks: open.filter(task => task.owner === peer || task.reviewer === peer).length }));
   }
   function rosterCandidate(def: SettingDef, value: SettingValue, undo = false): PeerRoster {
@@ -3494,6 +3801,7 @@ export async function startDaemon(opts: DaemonOptions) {
       else writeConfigSetting(opts.cwd, opts.stateDir, def, value, (scratch) => loadConfig(scratch));
       // Apply exactly the authorized key. Other file edits stay pending until the next hub start.
       config.roster = next;
+      settlePlanner(); // #455: an automatic planner follows the roster
       saved = true;
       recordSetting(def.key, before, after, source, undo);
       bus.rosterChanged(peer); // fence the queue before stopping or detaching the adapter
@@ -3518,7 +3826,7 @@ export async function startDaemon(opts: DaemonOptions) {
         if (owner instanceof PiPeer && owner.mode === "tui") await owner.resumeFromRoster();
       }
       writeStatus();
-      const stayed = !after ? board.list().filter(task => task.state !== "approved" && (task.owner === peer || task.reviewer === peer)) : [];
+      const stayed = !after ? board.list().filter(task => !isClosedTaskState(task.state) && (task.owner === peer || task.reviewer === peer)) : [];
       return { ok: true, text: [`${peer}: ${after ? "on" : "off"} roster (in force now)`, ...(!after && terminalNotice ? [terminalNotice] : []), ...(after && peer === "codex" ? [`Codex 0.146.0 needs a manual return: resume the previous native session with ${codexResumeCommand()}.`] : []), ...(!after ? handoff.map(move => sanitize(`task ${move.title}: ${move.role} assignment moved to ${move.to}`)) : []), ...stayed.map(task => sanitize(`task ${tasks.publicTitle(task)} stays with ${peer} as ${[task.owner === peer ? "owner" : "", task.reviewer === peer ? "reviewer" : ""].filter(Boolean).join(" and ")} (${task.state})`)), ...(!after && !handoff.length && !stayed.length ? ["No open tasks to hand over."] : [])].join("\n"), ...(!after ? { handoff } : {}) };
     } catch (error) {
       if (saved) writeStatus();
@@ -3568,7 +3876,7 @@ export async function startDaemon(opts: DaemonOptions) {
       if (!writeConfigSetting(opts.cwd, opts.stateDir, def, value, (scratch) => def.type === "roles" ? validateRoleSetting(scratch, def.path[1]!) : loadConfig(scratch))) return { ok: true, text: `${def.key}: config.local.json already ${value === null ? "leaves it unset" : `holds ${settingText(value)}`}; nothing was written` };
       storedConfigCache = undefined;
       const to = settingScalar(valueAt(storedConfig(), def.path));
-      if (def.type === "roles") applyRoleSelection(def.path[1]!); else applyStartModes();
+      if (def.type === "roles") applyRoleSelection(def.path[1]!); else if (def.key === "planner") applyPlanner(); else applyStartModes();
       recordSetting(def.key, from, to, source);
       if (def.type === "roles") event({ type: "role", peer: def.path[1]!, actor: "user", stage: "assigned", roles: selectedRoles(def.path[1]!, config.roles, roleAssignments, catalog).map(roleIdentity) });
       // Config trust (#17) can drop a machine-local field at load: the write happened, and the person is told it will not count.
@@ -3595,7 +3903,7 @@ export async function startDaemon(opts: DaemonOptions) {
       const from = read();
       undoSetting(opts.cwd, opts.stateDir, (scratch) => def.type === "roles" ? validateRoleSetting(scratch, def.path[1]!) : loadConfig(scratch), routingCheck);
       storedConfigCache = undefined;
-      if (def.type === "roles") applyRoleSelection(def.path[1]!); else applyStartModes();
+      if (def.type === "roles") applyRoleSelection(def.path[1]!); else if (def.key === "planner") applyPlanner(); else applyStartModes();
       recordSetting(def.key, from, read(), settingSource(authority), true);
       if (def.type === "roles") event({ type: "role", peer: def.path[1]!, actor: "user", stage: "assigned", roles: selectedRoles(def.path[1]!, config.roles, roleAssignments, catalog).map(roleIdentity) });
       return { ok: true, text: `${def.key}: put back to ${settingText(read())}` };
@@ -3619,10 +3927,10 @@ export async function startDaemon(opts: DaemonOptions) {
       const peer = def.path.at(-1)!;
       return { ok: true, lines: [permissionBoundary(peer), ...(isPermissionMode(value) ? [`${value}: ${permissionGrant(peer, value)}`] : ["inherit: the shared file's default, or ask"])] };
     }
-    if (def.store !== "routing") return { ok: true, lines: [`${def.key}: ${settingText(value)}; ${def.applies === "peer start" ? "applies at the peer's next start" : "read at the next hub start"}`] };
+    if (def.store !== "routing") return { ok: true, lines: [`${def.key}: ${settingText(value)}; ${def.applies === "peer start" ? "applies at the peer's next start" : def.applies === "live" ? "in force as soon as it is saved" : "read at the next hub start"}`] };
     try {
       const after = parseRouting(routingText(opts.cwd), routingCandidate(readOverlay(opts.cwd), def, value));
-      const open = board.list().filter((task) => task.state !== "approved");
+      const open = board.list().filter((task) => !isClosedTaskState(task.state));
       // A PII task's routing is fixed by its constraint and its text is private: it is counted, not shown.
       const shown = open.filter((task) => !tasks.isPii(task)).slice(0, 20);
       return { ok: true, lines: [`${def.key}: ${settingText(routingValue(currentRouting(opts.cwd, log), def))} -> ${settingText(routingValue(after, def))}`, ...(open.length ? [] : ["no open task: nothing is routed differently now"])],
@@ -3735,6 +4043,7 @@ export async function startDaemon(opts: DaemonOptions) {
       // #272: the rest of what recoveryReady() waits for, so the upgrade screen can name the wait. Display only.
       ...(taskOpsInFlight !== 0 ? ["a task command in flight"] : []),
       ...(tasks.checksPending() !== 0 ? ["completion checks queued or running"] : []),
+      ...(checkPending.size !== 0 ? ["isolated checks queued or running"] : []),
       ...((piReceipts?.inFlight ?? 0) !== 0 ? ["Pi tool calls in flight"] : []),
       ...(releasing ? ["ready tasks being released"] : []),
       ...([...bus.peers].filter(([, peer]) => peer instanceof PiPeer && peer.state !== "busy" && !peer.recoveryReady).map(([id, peer]) => !membershipAllowed(id) && peer instanceof PiPeer && peer.mode === "tui" ? `${id} is not settled; end the retained Pi TUI, then prepare again` : `${id} is not settled`)),
@@ -3750,10 +4059,19 @@ export async function startDaemon(opts: DaemonOptions) {
   async function recoveryOp(msg: any): Promise<Record<string, unknown>> {
     if (typeof msg.op !== "string" || !["inspect", "prepare", "commit", "abort", "release"].includes(msg.op)) return recoveryError("unknown recovery operation");
     if (typeof msg.expectedInstanceId !== "string" || msg.expectedInstanceId !== instanceId) return recoveryError("expected daemon instance does not match");
+    const incompatibleCancellationTarget = () => {
+      const ids = board.list().filter(task => task.state === "cancelled").map(task => task.id);
+      return ids.length && (!Array.isArray(msg.targetFeatures) || !msg.targetFeatures.includes("task_cancel"))
+        ? `target lacks task_cancel; cancelled tasks: ${ids.map(id => `#${id}`).join(", ")}` : undefined;
+    };
+    const checkTarget = msg.op === "prepare" || msg.op === "commit" || msg.op === "inspect" && msg.targetFeatures !== undefined;
+    if (checkTarget) { const error = incompatibleCancellationTarget(); if (error) return recoveryError(error); }
+
     if (msg.op === "inspect" || msg.op === "prepare" || msg.op === "commit") {
       const pi = bus.peers.get("pi");
       if (pi instanceof PiPeer && pi.state === "idle") {
         try { await pi.captureResume(); } catch (error) { return recoveryError((error as Error).message); }
+        if (checkTarget) { const error = incompatibleCancellationTarget(); if (error) return recoveryError(error); }
       }
     }
     if (msg.op === "inspect" && msg.operationId === undefined) return { t: "recovery", ok: true, recovery: recoveryView() };
@@ -3769,6 +4087,7 @@ export async function startDaemon(opts: DaemonOptions) {
         recoveryPeerSnapshot = undefined;
       }
       if (recoveryOperationId && recoveryOperationId !== op) return recoveryError("another recovery operation is active");
+      { const error = incompatibleCancellationTarget(); if (error) return recoveryError(error); }
       recoveryOperationId = op;
       if (!recoveryPhase) recoveryPhase = "preparing";
       armRecoveryLease();
@@ -3776,6 +4095,7 @@ export async function startDaemon(opts: DaemonOptions) {
       budget.setRecoveryHold(true);
       bus.setRecoveryHold(true);
       await bus.fenceRecovery();
+      { const error = incompatibleCancellationTarget(); if (error) return recoveryError(error); }
       const blocked = [...bus.peers].filter(([, peer]) => peer.state === "busy").map(([id]) => id);
       if (!blocked.length && permissions.size === 0 && recoveryReady()) {
         recoveryPeerSnapshot ??= Object.values(recoveryPeers());
@@ -3823,6 +4143,7 @@ export async function startDaemon(opts: DaemonOptions) {
         peers,
         integrity: integrity(),
       };
+      { const error = incompatibleCancellationTarget(); if (error) return recoveryError(error); }
       try { writeRestartSnapshot(opts.stateDir, snapshot); } catch { return recoveryError("could not persist restart state"); }
       recoveryCommitted = true;
       clearTimeout(recoveryLeaseTimer);
@@ -4009,7 +4330,7 @@ export async function startDaemon(opts: DaemonOptions) {
           if (important) ws.notifyQueued({ queued: bus.queued(c.peer!), important });
         });
       }
-      return void reply({ t: "welcome", projectId, instanceId, cwd: opts.cwd, protocol: PROTOCOL });
+      return void reply({ t: "welcome", projectId, instanceId, cwd: opts.cwd, protocol: PROTOCOL, features: DAEMON_FEATURES });
     }
     if (stopping && msg.t !== "status" && msg.t !== "kill") return void reply({ ok: false, error: "hub is stopping" });
     if (c.role !== "console" && c.peer && !membershipAllowed(c.peer)) return void reply({ ok: false, error: `${c.peer} is off roster` });
@@ -4048,11 +4369,19 @@ export async function startDaemon(opts: DaemonOptions) {
           return void reply({ ok: true, delivery: queueView(bus.queueShow(msg.id)) });
         } catch { return void reply({ ok: false, error: "resolution refused: refresh the delivery revision and inspect active or already resolved work" }); }
       }
+      case "delivery_holds": {
+        if (!c.peer || (c.role !== "peer" && c.role !== "tools")) return void reply({ ok: false, error: "held metadata requires a peer identity" });
+        try {
+          const held = bus.heldDeliveryHints(c.peer);
+          return void reply(held ? { ok: true, held } : { ok: false, error: "held metadata unavailable" });
+        } catch { return void reply({ ok: false, error: "held metadata unavailable" }); }
+      }
       case "delivery_receipt": {
         const peer = c.peer ? bus.peers.get(c.peer) : undefined;
-        if (c.role !== "peer" || !(peer instanceof WsPeer) || typeof msg.deliveryId !== "string" || !peer.ownsDelivery(sock, msg.generation, msg.deliveryId) || !["accepted", "needs_review"].includes(msg.state)) return void reply({ ok: false, error: "invalid delivery receipt" });
+        if (c.role !== "peer" || !(peer instanceof WsPeer) || typeof msg.deliveryId !== "string" || !peer.ownsDelivery(sock, msg.generation, msg.deliveryId) || !["accepted", "needs_review", "failed_safe"].includes(msg.state)) return void reply({ ok: false, error: "invalid delivery receipt" });
+        if (msg.state === "failed_safe" && (!peer.pullOnly || bus.queueShow(msg.deliveryId)?.state !== "dispatching")) return void reply({ ok: false, error: "safe failure requires an unseen push to a pull-only session" });
         const reason = typeof msg.reason === "string" ? sanitize(msg.reason).replace(/\s+/g, " ").trim().slice(0, 300) : "";
-        bus.deliveryReceipt(peer.id, { id: msg.deliveryId, state: msg.state, ...(msg.state === "needs_review" ? { reason: reason || "Claude bridge could not confirm notification delivery" } : {}) });
+        bus.deliveryReceipt(peer.id, { id: msg.deliveryId, state: msg.state, ...(msg.state !== "accepted" ? { reason: reason || "Claude bridge could not confirm notification delivery" } : {}) });
         return void reply({ ok: true });
       }
       case "inbox": {
@@ -4501,6 +4830,7 @@ export async function startDaemon(opts: DaemonOptions) {
   }
   async function stopOnce(): Promise<void> {
     stopping = true;
+    runnerRefreshAbort.abort();
     // A stop someone asked for is not a crash, even if it then runs past the shutdown deadline: forget the sessions now.
     try { removeSessions(opts.stateDir, instanceId); } catch { /* the state dir is gone */ }
     checksClosed = true;
@@ -4514,6 +4844,8 @@ export async function startDaemon(opts: DaemonOptions) {
     try {
       // A peer can still be inside memory recall or its native handshake when kill arrives.
       // Drain those starts before taking the final owned-process snapshot.
+      // Fence/abort the runner now, but slow container teardown must not consume the peer-stop deadline.
+      const runnerStopped = Promise.allSettled([checkRunner.stopAll(), refreshingRunner ?? Promise.resolve()]);
       await Promise.allSettled([...starting.values()]);
       // A fingerprint still being detected belongs in the research records written below; its peer must still be
       // attached when it settles, so this comes before the peers stop (each detection is bounded by its probe deadline).
@@ -4522,6 +4854,7 @@ export async function startDaemon(opts: DaemonOptions) {
       // A peer whose stop rejects must not strand the daemon (issue #56): its failure is
       // logged and shutdown finishes anyway, or the process ignores SIGTERM forever.
       for (const exit of exits) if (exit.status === "rejected") log(`peer stop failed during shutdown: ${(exit.reason as Error)?.message ?? exit.reason}`);
+      await runnerStopped;
       await modelRelay?.close();
       await egress?.close();
       await piReceipts?.close();
@@ -4566,7 +4899,7 @@ export async function startDaemon(opts: DaemonOptions) {
   writeFileSync(tokenFile, token, { mode: 0o600 });
   chmodSync(tokenFile, 0o600);
   budget.restore(); // pauses recorded by an earlier hub run stay in force; unfinished handoffs wait for peers to attach
-  writeStatus(); // #226: no hub.pid any more; status.json carries the pid with its signature
+  bus.onQueues(); // initial restored holds are reported only after the listener is bound; writes status.json too
   log(`${RUN_START}${process.pid} control=127.0.0.1:${server.port} cwd=${opts.cwd}`);
   ready = true;
   // #258: approvals whose settle entry a crash cut off are settled now (decisions, contracts and fails reach long-term).