@mgiles/perk 1.1.0 → 2.1.0

package/extension/substrate/toolGating.ts

@@ -1,15 +1,134 @@
 // The tool-gating primitive (the keystone). Structural read-only enforcement, NOT
 // prompting. Mirrors pi's authoritative `examples/extensions/plan-mode/` recipe (the
-// `setActiveTools` allowlist + `tool_call` bash sub-allowlist + `before_agent_start` injection +
+// `setActiveTools` allowlist + `tool_call` bash sub-allowlist + `before_agent_start` injection
+// (once-only: branch-scan dedup'd on the marker, so a session carries ONE live copy) +
 // `context` strip-when-off) and `preset.ts`'s snapshot-then-restore. The gate attaches to the
 // existing `perk:workflow-state.mode` field (`read-only`/`read-write`) — no new registry stage.
+// Beside the gate lives STAGE_TOOLS: per-stage active-tool scoping for the scoped universe
+// (perk's OWN registered tools + the enumerated borrowed-package census), keyed off the
+// workflow-state `stage` field and applied at the same rebuild points (contracts.md §8.40) —
+// fail-open where the gate is fail-closed.
 //
 // Substrate only: perk-owned plan mode and the read-only CI executor are the consumers of the
 // `enter`/`exit` surface; the allowlist-restore is wired into the existing
 // `session_start`/`session_tree` rebuild points.
 
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
-import { WORKFLOW_STATE_TYPE } from "./workflowState.ts";
+import { render } from "./prompts.ts";
+import { branchCarries, branchOf, WORKFLOW_STATE_TYPE } from "./workflowState.ts";
+
+/**
+ * The `web` seam providers' research tools: the UNION of all known web-provider tool names,
+ * enumerated statically and inert when the package is absent (the plan_review precedent —
+ * setActiveTools simply has nothing to enable). None mutate the repo — fetch_content's
+ * GitHub-clone path writes only to its own cache outside the worktree, morally equivalent to the
+ * already-allowlisted curl. perk does NOT normalize names, so all three providers' divergent
+ * names are listed: pi-web-access (default: web_search/code_search/fetch_content/
+ * get_search_content — `code_search` is not registered by any current version; kept as an inert
+ * static name for version tolerance), @ollama/pi-web-search (ollama_web_search/ollama_web_fetch),
+ * and @juicesharp/rpiv-web-tools (web_search shared, web_fetch). All register at load time.
+ */
+export const WEB_RESEARCH_TOOLS: readonly string[] = [
+  "web_search",
+  "code_search",
+  "fetch_content",
+  "get_search_content",
+  "ollama_web_search",
+  "ollama_web_fetch",
+  "web_fetch",
+];
+
+/**
+ * pi-mono-linear's read-only tools (the [issues] backend = "linear" selection): none mutate
+ * Linear or the repo. Foreign names are inert when the package is absent (the pi-web-access
+ * precedent above). The mutating/sensitive tools live in LINEAR_MUTATING_TOOLS and are
+ * deliberately excluded from the read-only gate AND from every stage list. All 25 register at
+ * load time (verified against the upstream pi-mono-extensions source).
+ */
+export const LINEAR_READ_TOOLS: readonly string[] = [
+  "linear_whoami",
+  "linear_workspace_metadata",
+  "linear_list_teams",
+  "linear_get_team",
+  "linear_list_users",
+  "linear_get_user",
+  "linear_list_issues",
+  "linear_get_issue",
+  "linear_search_issues",
+  "linear_list_my_issues",
+  "linear_list_projects",
+  "linear_get_project",
+  "linear_list_issue_statuses",
+  "linear_get_issue_status",
+  "linear_list_labels",
+  "linear_list_cycles",
+  "linear_list_documents",
+  "linear_get_document",
+  "linear_list_comments",
+];
+
+/**
+ * pi-mono-linear's mutating/sensitive tools — in the borrowed census so every stage session
+ * sheds their schemas, and in NO stage list: Linear mutations are the Python plane's job
+ * (`linear_configure_auth` even writes ~/.pi/agent/auth.json). Bare/unscoped sessions keep
+ * full access.
+ */
+export const LINEAR_MUTATING_TOOLS: readonly string[] = [
+  "linear_create_issue",
+  "linear_update_issue",
+  "linear_create_comment",
+  "linear_upload_file",
+  "linear_upload_file_to_issue_comment",
+  "linear_configure_auth",
+];
+
+/**
+ * pi-subagents' delegation family. `subagent`/`wait` register at load time; the parent intercom
+ * pair (`subagent_supervisor`, `intercom`) registers during `session_start` — AFTER perk's sync
+ * (perk is the first `packages` entry), so it LEAKS past rebuild-point filtering at launch
+ * (≈830 schema chars — accepted, documented, test-pinned). A later `session_tree` re-apply
+ * filters over the original snapshot (which lacks the late names), so a tree navigation drops
+ * them — the pre-existing snapshot behavior, unchanged. Child-side tools (`contact_supervisor`,
+ * `structured_output`) are out of scope: spawned children are unscoped by design (§8.40
+ * adopt-never-impersonates).
+ */
+export const SUBAGENT_TOOLS: readonly string[] = [
+  "subagent",
+  "wait",
+  "subagent_supervisor",
+  "intercom",
+];
+
+/**
+ * The enumerated borrowed-package tool census (contracts.md §8.40): every foreign tool name perk
+ * wires — via `BORROWED_PACKAGES`, a provider package, or the linear issue backend — joins the
+ * scoped universe beside PERK_TOOLS. Same static-name posture as READ_ONLY_TOOLS: names are
+ * inert when the package is absent, and un-enumerated foreign names always pass through every
+ * stage filter (fail-open — enumeration here is diet-completeness, not correctness).
+ *
+ * Audit records (the per-package census):
+ *  - Registration timing: every census name registers at load time EXCEPT pi-subagents' parent
+ *    supervisor pair (see SUBAGENT_TOOLS — session_start, leaks past rebuild-point filtering).
+ *  - Foreign `setActiveTools` owners: plannotator's phase machinery and @tombell/pi-plan's plan
+ *    mode run their OWN toggles — perk re-applies only at rebuild points, so a foreign toggle
+ *    between rebuilds wins (fail-open direction), and a mid-session rebuild re-installs perk's
+ *    stage set over a foreign restriction. Pre-existing interplay, recorded, not re-engineered.
+ *  - Zero-tool packages: @tombell/pi-diff (commands only), the footer providers, and the hunk
+ *    review CLI (not a Pi package) register nothing — nothing to enumerate.
+ *  - Single-governance rule: `ask_user_question` must stay OUT of this census — the
+ *    @juicesharp/rpiv-ask-user-question provider registers the IDENTICAL name perk does, so the
+ *    name-keyed PERK_TOOLS entry already governs both registrations (hygiene-tested).
+ */
+export const BORROWED_TOOLS: readonly string[] = [
+  ...WEB_RESEARCH_TOOLS,
+  ...LINEAR_READ_TOOLS,
+  ...LINEAR_MUTATING_TOOLS,
+  ...SUBAGENT_TOOLS,
+  "todo", // @juicesharp/rpiv-todo (the juicesharp-todo provider) — load-time
+  // @plannotator/pi-extension: perk never drives its plan phases (the adapter bridges
+  // `plan_review` to its event API), so the submit tool is dead weight in stage sessions.
+  "plannotator_submit_plan",
+];
 
 /**
  * Tools available while read-only mode is active (mirrors plan-mode's PLAN_MODE_TOOLS).
@@ -41,65 +160,154 @@ export const READ_ONLY_TOOLS = [
   // on — can only be written by calling this tool inside the gated session. Excluding it
   // silently breaks the warm `/objective-plan` path: the plan saves unlinked.
   "objective_node",
-  // The `web` seam providers' research tools: the UNION of all known web-provider tool
-  // names, allowlisted statically and inert when the package is absent (the plan_review precedent
-  // — setActiveTools simply has nothing to enable). None mutate the repo — fetch_content's
-  // GitHub-clone path writes only to its own cache outside the worktree, morally equivalent to the
-  // already-allowlisted curl. perk does NOT normalize names, so all three providers' divergent
-  // names are listed: pi-web-access (default: web_search/code_search/fetch_content/
-  // get_search_content), @ollama/pi-web-search (ollama_web_search/ollama_web_fetch), and
-  // @juicesharp/rpiv-web-tools (web_search shared, web_fetch).
-  "web_search",
-  "code_search",
-  "fetch_content",
-  "get_search_content",
-  "ollama_web_search",
-  "ollama_web_fetch",
-  "web_fetch",
-  // pi-mono-linear's read-only tools (the [issues] backend = "linear" selection):
-  // none mutate Linear or the repo. Foreign names are inert when the package is absent (the
-  // pi-web-access precedent above). The mutating/sensitive tools are deliberately excluded:
-  // linear_create_issue, linear_update_issue, linear_create_comment, linear_upload_file,
-  // linear_upload_file_to_issue_comment, linear_configure_auth (writes ~/.pi/agent/auth.json).
-  "linear_whoami",
-  "linear_workspace_metadata",
-  "linear_list_teams",
-  "linear_get_team",
-  "linear_list_users",
-  "linear_get_user",
-  "linear_list_issues",
-  "linear_get_issue",
-  "linear_search_issues",
-  "linear_list_my_issues",
-  "linear_list_projects",
-  "linear_get_project",
-  "linear_list_issue_statuses",
-  "linear_get_issue_status",
-  "linear_list_labels",
-  "linear_list_cycles",
-  "linear_list_documents",
-  "linear_get_document",
-  "linear_list_comments",
+  // The borrowed research families (extracted to family constants; set + order byte-identical).
+  ...WEB_RESEARCH_TOOLS,
+  ...LINEAR_READ_TOOLS,
+  // The delegation carve-in: the gated objective-plan seed/guidance names the
+  // `perk.objective-explorer` spawn, so `subagent`/`wait` (+ the parent supervisor pair, which
+  // already leaks active into cold-door gated sessions via late registration — keeping
+  // warm-entered gates consistent, and letting the parent answer child `contact_supervisor`
+  // asks) must be reachable while gated. ACCEPTED LENIENCY, deliberately documented: spawned
+  // children are unscoped by design (§8.40 adopt-never-impersonates). The explorer's agent def
+  // is structurally write-blocked (`tools: read, grep, find, ls, bash` frontmatter in
+  // agents/objective-explorer.md), but the `subagent` tool itself can spawn ad-hoc read-write
+  // children — a posture choice with NO agent-allowlist backstop, consistent with the arg-blind
+  // `curl`/`agent-browser` precedents (contracts.md §8.3).
+  ...SUBAGENT_TOOLS,
 ];
 
+/**
+ * Every tool perk itself registers (contracts.md §8.40). Name-keyed: `setActiveTools` ignores
+ * unknown names, so a vacated registration (e.g. `ask_user_question` under a foreign
+ * `[providers] askuser` selection registers the IDENTICAL name) or an absent tool is inert.
+ * Stage scoping filters the scoped universe `PERK_TOOLS ∪ BORROWED_TOOLS` — builtins and
+ * un-enumerated foreign names pass through untouched (fail-open).
+ */
+export const PERK_TOOLS: readonly string[] = [
+  "plan_review",
+  "plan_save",
+  "plan_draft",
+  "objective_save",
+  "objective_node",
+  "reconcile_objective",
+  "add_objective_node",
+  "objective_draft",
+  "learn",
+  "ask_user_question",
+  "land",
+  "post_pr_review",
+  "ready",
+  "resolve_review_threads",
+  "submit_pr_review",
+  "run_ci",
+  "submit",
+];
+
+/**
+ * The research bundle EVERY stage list carries: web research + Linear reads are useful in every
+ * stage session (authoring and worktree alike) and mutate nothing.
+ */
+const RESEARCH_TOOLS: readonly string[] = [...WEB_RESEARCH_TOOLS, ...LINEAR_READ_TOOLS];
+
+/**
+ * The PR-loop family shared by ALL FIVE worktree stages (implement/submit/address/land/learn) —
+ * deliberately one shared list, not per-stage cuts: any PR-loop warm command must work in any
+ * worktree session (warm doors inject guidance naming their companion tool, and a per-stage cut
+ * would dead-end e.g. `/land` run inside the implement session; the concrete forcing example is
+ * the post-land reconcile drive — `/land` auto-drives `/objective-reconcile` in-session, whose
+ * guidance names the reconcile trio, so the trio must be active in every worktree stage). The
+ * headless worker also REQUIRES the model-invoked `submit` (implement) /
+ * `resolve_review_threads` (address) to reach its completion bar. Borrowed additions: delegation
+ * (SUBAGENT_TOOLS — the `/pr-review`/`/address`/`/submit`-conflict/`/learn` orchestration flows)
+ * and `todo` (the foreign checklist overlay the implement-progress discipline rides) are
+ * worktree-family only.
+ */
+const WORKTREE_STAGE_TOOLS: readonly string[] = [
+  "ask_user_question",
+  "submit",
+  "ready",
+  "run_ci",
+  "land",
+  "learn",
+  "resolve_review_threads",
+  "post_pr_review",
+  "submit_pr_review",
+  // The reconcile trio: `/land` auto-drives the objective-reconcile pass inside the CURRENT
+  // worktree session (driveReconcileAfterLand), and the manual `/objective-reconcile` gesture is
+  // registered globally — both inject guidance naming these three tools.
+  "reconcile_objective",
+  "add_objective_node",
+  "objective_node",
+  ...RESEARCH_TOOLS,
+  ...SUBAGENT_TOOLS,
+  "todo",
+];
+
+/**
+ * Per-stage active perk tools for gate-OFF sessions (contracts.md §8.40). Keys = the registry
+ * stage ids; an unknown/absent stage id is fail-open (no filtering — version-skew safety).
+ * Rationale pins:
+ *  - `ask_user_question` is universal (every stage list carries it).
+ *  - `plan`/`save` cover the plan-family stage borrowers (`plan from`/`plan replan`/
+ *    `learn docs`/`learn code` borrow `plan`; `skills create/refine` borrow `save`).
+ *  - `objective-author`/`objective-save` cover `objective replan` + `objective author --from`.
+ *  - `objective-plan` keeps `objective_node` (the factory's claim/transition door) and
+ *    `plan_save` (the gate-off manual failsafe for the approval-driven save).
+ *  - the reconcile trio (`reconcile_objective`/`add_objective_node`/`objective_node`) rides the
+ *    three objective stages (the post-save `/objective-reconcile` gesture) AND the worktree
+ *    family (the post-land `driveReconcileAfterLand` drive + the manual `/objective-reconcile`
+ *    gesture — its guidance names all three).
+ */
+export const STAGE_TOOLS: Readonly<Record<string, readonly string[]>> = {
+  "objective-author": [
+    "ask_user_question",
+    "objective_draft",
+    "objective_save",
+    "reconcile_objective",
+    "add_objective_node",
+    "objective_node",
+    ...RESEARCH_TOOLS,
+  ],
+  "objective-save": [
+    "ask_user_question",
+    "objective_draft",
+    "objective_save",
+    "reconcile_objective",
+    "add_objective_node",
+    "objective_node",
+    ...RESEARCH_TOOLS,
+  ],
+  "objective-plan": [
+    "ask_user_question",
+    "plan_draft",
+    "plan_review",
+    "plan_save",
+    "objective_node",
+    "reconcile_objective",
+    "add_objective_node",
+    ...RESEARCH_TOOLS,
+  ],
+  plan: ["ask_user_question", "plan_draft", "plan_review", "plan_save", ...RESEARCH_TOOLS],
+  save: ["ask_user_question", "plan_draft", "plan_review", "plan_save", ...RESEARCH_TOOLS],
+  implement: WORKTREE_STAGE_TOOLS,
+  submit: WORKTREE_STAGE_TOOLS,
+  address: WORKTREE_STAGE_TOOLS,
+  land: WORKTREE_STAGE_TOOLS,
+  learn: WORKTREE_STAGE_TOOLS,
+};
+
 /** The read-only marker / custom-message type injected into context while active. */
 const MODE_CONTEXT_TYPE = "perk:mode-context";
 const READ_ONLY_MARKER = "[READ-ONLY MODE]";
 
-/** Exported for tests: the injected read-only mode context (interpolates the allowlist). */
-export const READ_ONLY_CONTEXT = `${READ_ONLY_MARKER}
-You are in perk read-only mode — a structurally enforced exploration mode.
-
-- You can only use: ${READ_ONLY_TOOLS.join(", ")}.
-- You CANNOT use edit or write (file modifications are blocked).
-- plan_draft is the sole sanctioned write: it writes only the working-plan artifact in the session data dir.
-- bash is restricted to an allowlist of read-only commands.
-- For GitHub data use read-only \`gh\` subcommands (view/list/diff/status/checks/search) — never raw curl/fetch against github.com (private repos reject unauthenticated requests).
-
-These restrictions are enforced by perk, not advisory. Do not attempt to make changes.`;
+/** Exported for tests: the injected read-only mode context. (No tool enumeration — gate-ON
+ * already applies READ_ONLY_TOOLS via setActiveTools, so the active tool set IS the list.) */
+export const READ_ONLY_CONTEXT = render("contexts/read-only.md", {
+  marker: READ_ONLY_MARKER,
+});
 
 // --- pure policy (copied from plan-mode/utils.ts so this primitive is self-contained; perk-owned
-// so T2's eventual retirement of the borrowed pi-plan extension leaves no dangling import) -------
+// so retiring the borrowed pi-plan extension leaves no dangling import) -------
 
 const DESTRUCTIVE_PATTERNS = [
   /\brm\b/i,
@@ -134,7 +342,11 @@ const DESTRUCTIVE_PATTERNS = [
   /\bshutdown\b/i,
   /\bsystemctl\s+(start|stop|restart|enable|disable)/i,
   /\bservice\s+\S+\s+(start|stop|restart)/i,
-  /\b(vim?|nano|emacs|code|subl)\b/i,
+  /\b(vim?|nano|emacs|subl)\b/i,
+  // `code` (the editor) is vetoed in command position only — the old bare \bcode\b veto blocked
+  // every command CONTAINING the word (destructive-wins), so the allowlisted `gh search code`
+  // could never run. `code f.ts`, `ls; code .`, `x && code .`, `$(code y)` stay blocked.
+  /(^|[;&|(]|\$\()\s*code\b/i,
 ];
 
 const SAFE_PATTERNS = [
@@ -203,9 +415,11 @@ const SAFE_PATTERNS = [
   /^\s*npx\s+agent-browser\b/,
   /^\s*bat\b/,
   /^\s*eza\b/,
-  // perk's own read-only objective queries (show/next + their s/n aliases). The trailing \b keeps
-  // the `n` alias from matching the mutating `node` subcommand; create/node/reconcile stay blocked.
-  /^\s*perk\s+(objective|obj)\s+(show|s|next|n)\b/i,
+  // perk's own read-only objective queries (show/next + their s/n aliases, plus the non-mutating
+  // node-engagement read the objective-plan factory needs). The trailing \b keeps the `n` alias
+  // from matching the mutating `node` subcommand; node-engagement allowed; create/node/reconcile
+  // stay blocked.
+  /^\s*perk\s+(objective|obj)\s+(show|s|next|n|node-engagement)\b/i,
   // Read-only `gh` queries — the guidance in the managed AGENTS block ("GitHub access goes
   // through gh") must be followable in read-only sessions. Query-shaped subcommands only;
   // `gh api` stays blocked (it can POST/PATCH), as do all mutating subcommands (create/edit/
@@ -294,13 +508,16 @@ export function isReadOnlyBashCommand(command: string): boolean {
 
 // --- the controller -----------------------------------------------------------------------------
 
-/** The API T2/T5 consume + the lifecycle hooks index.ts wires. */
+/** The API the plan-mode and read-only-stage consumers use + the lifecycle hooks index.ts wires. */
 export interface ToolGating {
-  /** Reapply the allowlist from a rebuilt `mode` (called on session_start AND session_tree). */
-  syncFromState(mode: string | undefined): void;
-  /** Enter read-only mode: persist `mode=read-only` + snapshot/restrict tools. (T2/T5 call site.) */
+  /**
+   * Reapply the gate + stage scoping from a rebuilt `mode` + `stage` (called on session_start
+   * AND session_tree). `stage` is the branch-LWW workflow-state stage id (undefined = unscoped).
+   */
+  syncFromState(mode: string | undefined, stage: string | undefined): void;
+  /** Enter read-only mode: persist `mode=read-only` + snapshot/restrict tools. (Called by the plan-mode toggle and the objective-plan factory.) */
   enter(ctx?: ExtensionContext): void;
-  /** Exit read-only mode: persist `mode=read-write` + restore tools. (T2/T5 call site.) */
+  /** Exit read-only mode: persist `mode=read-write` + restore tools. (Called by the plan-mode toggle and the save/exit doors.) */
   exit(ctx?: ExtensionContext): void;
   /** Whether the gate is currently active (in-memory source of truth for `tool_call`). */
   isActive(): boolean;
@@ -314,23 +531,54 @@ export function registerToolGating(pi: ExtensionAPI): ToolGating {
   // In-memory gate (mirrors plan-mode's `planModeEnabled`): the authority `tool_call` consults.
   // Fail-closed — a failed sync never opens this; tool_call blocks on any internal error.
   let active = false;
-  // Pre-gate tool snapshot, taken once on the off→on transition (preset.ts discipline).
+  // The branch-LWW stage id this session is scoped to (null = unscoped). Fail-open by contrast
+  // with `active`: no stage / unknown stage / any lookup miss → no filtering.
+  let stageId: string | null = null;
+  // Pre-engagement tool snapshot, taken ONCE on the first engagement of either concern (the
+  // preset.ts discipline, shared by the gate and stage scoping).
   let snapshot: string[] | null = null;
 
-  function applyActive(next: boolean): void {
-    if (next && !active) {
-      // off → on: snapshot the current tool set, then restrict.
+  /**
+   * Recompute + install the active tool set from both concerns (contracts.md §8.40):
+   *  - gate ON → exactly READ_ONLY_TOOLS, NO stage filter (the decided composition: the gated
+   *    set is already the diet, and a strict intersection would break the documented warm
+   *    `/objective-plan` carve-out and recreate the seed/gate contradiction class). "The gate
+   *    never widens a stage's set and vice versa" still holds: engaging the gate only ever
+   *    narrows, and stage scoping never adds a tool.
+   *  - gate OFF + stage scoped → a SUBTRACTIVE filter over the snapshot: names outside the
+   *    scoped universe (builtins, un-enumerated foreign tools) pass through untouched; scoped
+   *    names (PERK_TOOLS ∪ BORROWED_TOOLS) survive only when the stage's list carries them.
+   *  - neither engaged → restore the snapshot if one exists (a session that never engages gets
+   *    ZERO setActiveTools calls — bare warm sessions stay byte-identical).
+   * While engaged the set is re-installed on every sync (tree navigation across mode entries
+   * must recompute correctly).
+   */
+  // The scoped universe: perk's own tools + the enumerated borrowed census. Builtins and
+  // un-enumerated foreign names pass through every stage filter untouched (fail-open).
+  const SCOPED_TOOL_NAMES: ReadonlySet<string> = new Set([...PERK_TOOLS, ...BORROWED_TOOLS]);
+
+  function apply(nextActive: boolean, nextStage: string | null): void {
+    const stageList = nextStage === null ? undefined : STAGE_TOOLS[nextStage];
+    // First engagement of either concern: take the one snapshot.
+    if ((nextActive || stageList !== undefined) && snapshot === null) {
       snapshot = pi.getActiveTools();
-      pi.setActiveTools(READ_ONLY_TOOLS);
-    } else if (!next && active) {
-      // on → off: restore the pre-gate snapshot. If none exists (near-unreachable — the off→on
-      // branch always snapshots first), fall back to the FULL configured tool set
-      // (pi.getAllTools()) like plan-mode, never a hardcoded list that would silently drop
-      // grep/find/ls and perk's custom tools (plan_save/submit/land/learn).
-      pi.setActiveTools(snapshot ?? pi.getAllTools().map((t) => t.name));
+    }
+    if (nextActive) {
+      pi.setActiveTools([...READ_ONLY_TOOLS]);
+    } else if (stageList !== undefined) {
+      // The snapshot-missing fallback mirrors the restore path below: the FULL configured tool
+      // set (pi.getAllTools()), never a hardcoded list that would silently drop grep/find/ls
+      // and perk's custom tools (plan_save/submit/land/learn).
+      const base = snapshot ?? pi.getAllTools().map((t) => t.name);
+      pi.setActiveTools(
+        base.filter((name) => !SCOPED_TOOL_NAMES.has(name) || stageList.includes(name)),
+      );
+    } else if (snapshot !== null) {
+      pi.setActiveTools(snapshot);
       snapshot = null;
     }
-    active = next;
+    active = nextActive;
+    stageId = nextStage;
   }
 
   // Structural backstop: block writes + non-allowlisted bash while active. Fail-closed on error.
@@ -360,8 +608,11 @@ export function registerToolGating(pi: ExtensionAPI): ToolGating {
   });
 
   // Inject the hidden read-only mode context while active (display:false → not shown in transcript).
-  pi.on("before_agent_start", async () => {
+  pi.on("before_agent_start", async (_event, ctx) => {
     if (!active) return;
+    // Once-only: injected customs persist to the branch, so a live copy suppresses re-injection;
+    // compaction dropping it makes the scan come up clean and the next turn re-injects.
+    if (branchCarries(branchOf(ctx), READ_ONLY_MARKER)) return;
     return {
       message: { customType: MODE_CONTEXT_TYPE, content: READ_ONLY_CONTEXT, display: false },
     };
@@ -390,16 +641,16 @@ export function registerToolGating(pi: ExtensionAPI): ToolGating {
   });
 
   return {
-    syncFromState(mode: string | undefined): void {
-      applyActive(isReadOnlyMode(mode));
+    syncFromState(mode: string | undefined, stage: string | undefined): void {
+      apply(isReadOnlyMode(mode), stage ?? null);
     },
     enter(_ctx?: ExtensionContext): void {
       pi.appendEntry(WORKFLOW_STATE_TYPE, { mode: "read-only" });
-      applyActive(true);
+      apply(true, stageId);
     },
     exit(_ctx?: ExtensionContext): void {
       pi.appendEntry(WORKFLOW_STATE_TYPE, { mode: "read-write" });
-      applyActive(false);
+      apply(false, stageId);
     },
     isActive(): boolean {
       return active;

package/extension/substrate/toolParams.ts

@@ -27,6 +27,13 @@ export function stringParam(p: ToolParams, key: string): string | undefined | nu
   return typeof value === "string" ? value : null;
 }
 
+/** Tri-state boolean field: undefined = absent; null = present-but-mistyped. */
+export function booleanParam(p: ToolParams, key: string): boolean | undefined | null {
+  const value = p[key];
+  if (value === undefined) return undefined;
+  return typeof value === "boolean" ? value : null;
+}
+
 /** Tri-state number field: undefined = absent; null = present-but-mistyped. */
 export function numberParam(p: ToolParams, key: string): number | undefined | null {
   const value = p[key];

package/extension/substrate/workflowState.ts

@@ -43,6 +43,12 @@ export interface WorkflowState {
    * `rebuildWorkflowState`, no rebuild change). The PR comment stays canonical.
    */
   last_pr_review?: unknown;
+  /**
+   * The last review-door outcome posted via the `submit_pr_review` warm tool (§8.3):
+   * `{pr, event, comment_count, mode, at}`. Best-effort tier (per-field LWW in
+   * `rebuildWorkflowState`, no rebuild change). The submitted PR review stays canonical.
+   */
+  last_review?: unknown;
   /** Session-artifact provenance pointers, keyed by artifact name (§8.3). */
   session_artifacts?: Record<string, SessionArtifactPointer> | null;
   /**
@@ -84,6 +90,17 @@ export function branchOf(source: BranchSource): BranchEntry[] {
   return source.sessionManager.getBranch() as BranchEntry[];
 }
 
+/**
+ * Whether any entry on the branch already carries `needle` — the once-only injection dedup guard
+ * (the bindingDelivery `branchHasHeader` form). Serializing each entry is the robust,
+ * shape-agnostic scan; safe while the needle is a distinctive literal that other entries' data
+ * can't casually contain (known accepted false positive: a tool result quoting perk's own source;
+ * the typed customType scan is the escalation if that bites — docs/learned/pi/context-injection.md).
+ */
+export function branchCarries(branch: readonly BranchEntry[], needle: string): boolean {
+  return branch.some((entry) => JSON.stringify(entry).includes(needle));
+}
+
 /**
  * Per-field last-write-wins over the `perk:workflow-state` custom entries on a branch.
  * Non-perk entries are ignored; `undefined` fields never clobber (but explicit `null` does).
@@ -185,6 +202,14 @@ export type ClaimDecision =
       state: WorkflowState;
     }
   | { action: "claim"; source: "env"; runId: string }
+  | {
+      action: "adopt";
+      source: "env-child";
+      childRunId: string;
+      parentRunId: string;
+      /** Inherited from the parent's handoff so read-only gating survives into the child. */
+      mode?: string;
+    }
   | { action: "none"; source: "none"; state: WorkflowState };
 
 /**
@@ -193,12 +218,15 @@ export type ClaimDecision =
  * (NOT `event.reason`, which is "startup" for a headless `pi --fork`): if the branch already
  * carries a `run_id` whose recorded `pi_session_id` differs from the current session, the id
  * was inherited across a fork → derive a child; if it matches (or is absent), it's a reload.
+ * An env-inherited run id whose handoff was already CONSUMED by a different session is a
+ * spawned child, not the launched session → `adopt` (derive a sibling id, inherit `mode`).
  */
 /**
  * The registry stage id the launched run is acting on, read from its handoff blob, or null.
  * Only `claim` (cold) and `keep` (reload) sessions have a settled run whose handoff records a
- * `stage`; `fork` and `none` carry no launched stage (LWW restores their state instead). The
- * stage gates whether `session_start` reconciles `cache.plan-ref` into `active_plan_ref`.
+ * `stage`; `fork`, `adopt`, and `none` carry no launched stage (an adopted env-child must never
+ * impersonate the launched stage; LWW restores fork/none state instead). The stage gates whether
+ * `session_start` reconciles `cache.plan-ref` into `active_plan_ref`.
  */
 export function resolveRunStage(decision: ClaimDecision, cwd: string): string | null {
   const runId =
@@ -227,6 +255,30 @@ export function decideClaim(args: {
     return { action: "fork", source: "fork", childRunId, parentRunId: state.run_id, state };
   }
   if (envRunId !== null && envRunId !== "") {
+    // Env-child detection (contracts §8.2): subagent children are spawned as separate `pi`
+    // processes with the parent's env, so they arrive here carrying the parent's PERK_RUN_ID.
+    // A handoff already consumed by a DIFFERENT (or unrecorded) session belongs to someone else:
+    // adopt a derived `<run_id>.<n>` child identity instead of re-claiming — never re-consume the
+    // handoff, never capture pointers, never impersonate the launched stage. The parent's `mode`
+    // is inherited so read-only gating survives into exploration children. Everything else —
+    // absent/corrupt/mismatched handoff (the loud unclaimed error), unconsumed (the normal cold
+    // claim), or consumed by THIS session (idempotent re-claim after lost branch state) — stays
+    // the claim arm.
+    const handoff = readHandoff(cwd, envRunId);
+    if (
+      handoff !== null &&
+      handoff.run_id === envRunId &&
+      handoff.consumed === true &&
+      handoff.pi_session_id !== currentSessionId
+    ) {
+      return {
+        action: "adopt",
+        source: "env-child",
+        childRunId: deriveForkRunId(envRunId, cwd),
+        parentRunId: envRunId,
+        mode: handoff.mode,
+      };
+    }
     return { action: "claim", source: "env", runId: envRunId };
   }
   return { action: "none", source: "none", state };

package/extension/surfaces/footerProvider.ts

@@ -21,14 +21,18 @@ import {
 
 /**
  * The resolved `[providers] footer` selection id for `cwd`. Fail-safe to the perk-footer
- * reference: any load/resolution failure (corrupt bundled set, etc.) returns the reference id so
- * perk keeps installing its own footer — the default path is the hard guarantee. Mirror of
- * `resolvedAskUserProviderId`.
+ * reference: any load/resolution failure returns the reference id so perk keeps installing its
+ * own footer — the default path is the hard guarantee. Mirror of `resolvedAskUserProviderId`: the
+ * catch narrows to genuine file-read/parse failures (the resolver is per-seam fail-open) and is
+ * logged, never silent.
  */
 export function resolvedFooterProviderId(cwd: string): string {
   try {
     return resolveProviders(loadPerkConfig(cwd).providers, loadProviders()).footer.id;
-  } catch {
+  } catch (error) {
+    console.error(
+      `perk: footer provider resolution failed — falling back to ${PERK_FOOTER_PROVIDER_ID}: ${error}`,
+    );
     return PERK_FOOTER_PROVIDER_ID;
   }
 }