@runfusion/fusion 0.75.1-beta.0 → 0.75.1-beta.2

package/dist/extension.js

@@ -1103,6 +1103,9 @@ function isFusionSiblingBranch(branch) {
 function resolveEffectiveAutoMerge(task, settings) {
   return task.autoMerge ?? settings.autoMerge;
 }
+function hasUserAutoMergeHold(task) {
+  return task.autoMerge === false && task.autoMergeProvenance === "user";
+}
 function allowsAutoMergeProcessing(task, settings) {
   return (settings.autoMerge !== false || task.autoMerge === true) && !taskHasManualOpenPullRequest(task);
 }
@@ -9968,6 +9971,7 @@ function validateForeach(node, topLevelNodeIds, columnIds) {
     );
   }
   for (const inner of templateNodes) {
+    validateReviewKind(inner, "nested");
     if (inner.kind === "foreach" || inner.kind === "loop") {
       throw new WorkflowIrError(
         `foreach node '${node.id}' template may not contain nested loop/foreach ('${inner.id}')`
@@ -10086,6 +10090,7 @@ function validateLoop(node, topLevelNodeIds, columnIds) {
     );
   }
   for (const inner of templateNodes) {
+    validateReviewKind(inner, "nested");
     if (inner.kind === "loop" || inner.kind === "foreach") {
       throw new WorkflowIrError(
         `loop node '${node.id}' template may not contain nested loop/foreach ('${inner.id}')`
@@ -10179,6 +10184,7 @@ function validateOptionalGroup(node, topLevelNodeIds, columnIds) {
     throw new WorkflowIrError(`optional-group node '${node.id}' template has duplicate node ids`);
   }
   for (const inner of templateNodes) {
+    validateReviewKind(inner, "nested");
     if (inner.kind === "loop" || inner.kind === "foreach" || inner.kind === "optional-group") {
       throw new WorkflowIrError(
         `optional-group node '${node.id}' template may not contain nested loop/foreach/optional-group ('${inner.id}')`
@@ -10850,6 +10856,19 @@ function validateColumnRecovery(column) {
     );
   }
 }
+function validateReviewKind(node, placement) {
+  const value = node.config?.reviewKind;
+  if (value === void 0) return;
+  if (value !== "plan" && value !== "code") {
+    throw new WorkflowIrError(`Workflow node '${node.id}' has invalid reviewKind '${String(value)}'; expected 'plan' or 'code'`);
+  }
+  if (placement === "nested") {
+    throw new WorkflowIrError(`Workflow node '${node.id}' has reviewKind in an unsupported nested template placement`);
+  }
+  if (node.kind !== "prompt" && node.kind !== "gate" && node.kind !== "script" && node.kind !== "optional-group") {
+    throw new WorkflowIrError(`Workflow node '${node.id}' has reviewKind on unsupported node kind '${node.kind}'`);
+  }
+}
 function validateV2(ir2) {
   validateColumns(ir2);
   for (const [index2, column] of ir2.columns.entries()) {
@@ -10872,6 +10891,7 @@ function validateV2(ir2) {
   }
   const nodesById = new Map(ir2.nodes.map((n) => [n.id, n]));
   for (const node of ir2.nodes) {
+    validateReviewKind(node, "top-level");
     validateExtensionMetadata(`Workflow node '${node.id}'`, node.extensions);
     if (node.column !== void 0 && !columnIds.has(node.column)) {
       throw new WorkflowIrError(
@@ -13335,6 +13355,7 @@ function codeReviewOptionalGroupNode(column, options = {}) {
     column,
     config: {
       name: CODE_REVIEW_NAME,
+      reviewKind: "code",
       // Default-ON: runs for every coding task by default, but operators can toggle it
       // off per task (remove `code-review` from enabledWorkflowSteps).
       defaultOn: options.defaultOn ?? true,
@@ -13480,6 +13501,7 @@ function planReviewOptionalGroupNode(column, options = {}) {
     ...column ? { column } : {},
     config: {
       name: PLAN_REVIEW_NAME,
+      reviewKind: "plan",
       defaultOn: options.defaultOn ?? true,
       /*
        * FNXC:WorkflowRemediation 2026-06-29-12:14:
@@ -63324,7 +63346,7 @@ function rowToTask(row) {
     executionStartBranch: row.executionStartBranch || void 0,
     branch: row.branch || void 0,
     autoMerge: row.autoMerge === null ? void 0 : row.autoMerge === 1,
-    autoMergeProvenance: row.autoMergeProvenance === "user" || row.autoMergeProvenance === "legacy-stamp" ? row.autoMergeProvenance : void 0,
+    autoMergeProvenance: row.autoMergeProvenance === "user" || row.autoMergeProvenance === "mission" || row.autoMergeProvenance === "legacy-stamp" ? row.autoMergeProvenance : void 0,
     baseCommitSha: row.baseCommitSha || void 0,
     scopeOverride: row.scopeOverride ? true : void 0,
     scopeOverrideReason: row.scopeOverrideReason || void 0,
@@ -72840,10 +72862,10 @@ ${feature.acceptanceCriteria}`);
               Apply the selected Missions header workflow atomically during TaskStore.createTask so newly triaged features land in the intended workflow lane. Duplicate-guard reuses skip this create path, preserving existing duplicate tasks without workflow mutation.
               */
               /*
-              FNXC:MissionAutoMerge 2026-07-18-12:00:
-              An autoMerge:false mission stamps each newly triaged task so its shared branch produces one PR instead of per-task auto-merges. Duplicate reuse intentionally bypasses this create-only override.
+              FNXC:MissionAutoMerge 2026-08-05-22:50:
+              An autoMerge:false mission stamps each newly triaged task as mission policy so its shared branch produces one PR instead of per-task auto-merges. Duplicate reuse intentionally bypasses this create-only override; policy must not impersonate an operator manual-hold choice.
               */
-              ...mission?.autoMerge === false ? { autoMerge: false } : {},
+              ...mission?.autoMerge === false ? { autoMerge: false, autoMergeProvenance: "mission" } : {},
               // FNXC:MissionTaskPrefix 2026-07-26-12:00: thread the mission's optional taskPrefix into TaskCreateInput for distributed id minting.
               ...mission?.taskPrefix ? { taskPrefix: mission.taskPrefix } : {},
               ...branchOptions?.workflowId !== void 0 ? { workflowId: branchOptions.workflowId } : {}
@@ -76021,10 +76043,10 @@ ${feature.acceptanceCriteria}`);
                 }
               } : {},
               /*
-              FNXC:MissionAutoMerge 2026-07-18-12:00:
-              An autoMerge:false mission stamps each newly triaged task so its shared branch produces one PR instead of per-task auto-merges. Duplicate reuse intentionally bypasses this create-only override.
+              FNXC:MissionAutoMerge 2026-08-05-22:50:
+              An autoMerge:false mission stamps each newly triaged task as mission policy so its shared branch produces one PR instead of per-task auto-merges. Duplicate reuse intentionally bypasses this create-only override; policy must not impersonate an operator manual-hold choice.
               */
-              ...mission?.autoMerge === false ? { autoMerge: false } : {},
+              ...mission?.autoMerge === false ? { autoMerge: false, autoMergeProvenance: "mission" } : {},
               // FNXC:MissionTaskPrefix 2026-07-26-12:00: thread the mission's optional taskPrefix into TaskCreateInput so the distributed allocator mints ERR-N (etc.) instead of the project prefix.
               ...mission?.taskPrefix ? { taskPrefix: mission.taskPrefix } : {},
               ...branchOptions?.workflowId !== void 0 ? { workflowId: branchOptions.workflowId } : {}
@@ -86375,7 +86397,9 @@ async function _createTaskInternalBackendImpl(store, input, title, resolvedWorkf
     sourceMetadata: withTaskBranchContextInSourceMetadata(input.source?.sourceMetadata, input.branchContext),
     branchContext: input.branchContext,
     autoMerge: input.autoMerge,
-    autoMergeProvenance: input.autoMerge === void 0 ? void 0 : "user",
+    // FNXC:SharedBranchMemberHold 2026-08-05-22:50: trusted mission creation
+    // preserves policy provenance; operator/API create requests retain user provenance.
+    autoMergeProvenance: input.autoMerge === void 0 ? void 0 : input.autoMergeProvenance ?? "user",
     // FNXC:CodingIdeasWorkflow 2026-07-05-19:45: land the task in its
     // workflow's manual intake column (e.g. Coding (Ideas) → "ideas") when
     // no explicit column is given (main FN-7591 parity).
@@ -127080,6 +127104,36 @@ var init_async_knowledge = __esm({
 });
 
 // ../core/src/workflows/workflow-step-results.ts
+function normalizeWorkflowReviewFindings(raw) {
+  if (!Array.isArray(raw)) return void 0;
+  const normalized = [];
+  const usedIds = /* @__PURE__ */ new Set();
+  for (const candidate of raw) {
+    if (normalized.length >= MAX_WORKFLOW_REVIEW_FINDINGS || !candidate || typeof candidate !== "object") continue;
+    const value = candidate;
+    const title = boundedTrimmedString(value.title, MAX_FINDING_TITLE_LENGTH);
+    const body = boundedTrimmedString(value.body, MAX_FINDING_BODY_LENGTH);
+    if (!title || !body) continue;
+    const baseId = boundedTrimmedString(value.id, MAX_FINDING_ID_LENGTH) || `finding-${normalized.length + 1}`;
+    let id = baseId;
+    let suffix = 2;
+    while (usedIds.has(id)) id = `${baseId.slice(0, Math.max(1, MAX_FINDING_ID_LENGTH - String(suffix).length - 1))}-${suffix++}`;
+    usedIds.add(id);
+    const filePath = boundedTrimmedString(value.filePath, MAX_FINDING_PATH_LENGTH);
+    const line2 = typeof value.line === "number" && Number.isFinite(value.line) && value.line > 0 ? Math.floor(value.line) : void 0;
+    const severity = isWorkflowReviewFindingSeverity(value.severity) ? value.severity : void 0;
+    normalized.push({ id, title, body, ...filePath ? { filePath } : {}, ...line2 ? { line: line2 } : {}, ...severity ? { severity } : {} });
+  }
+  return normalized.length > 0 ? normalized : void 0;
+}
+function boundedTrimmedString(value, maxLength) {
+  if (typeof value !== "string") return void 0;
+  const trimmed = value.trim();
+  return trimmed && trimmed.length <= maxLength ? trimmed : void 0;
+}
+function isWorkflowReviewFindingSeverity(value) {
+  return typeof value === "string" && WORKFLOW_REVIEW_FINDING_SEVERITIES.includes(value);
+}
 function isTerminalFailure(result2) {
   return TERMINAL_FAILURE_STATUSES.has(result2.status);
 }
@@ -127145,10 +127199,16 @@ function makeReviewLeaseRecord(args) {
     leaseOwner: args.owner
   };
 }
-var MAX_WORKFLOW_STEP_PRIOR_ATTEMPTS, TERMINAL_FAILURE_STATUSES, PLAN_REVIEW_LEASE_STALENESS_MS, TERMINAL_STEP_STATUSES;
+var WORKFLOW_REVIEW_FINDING_SEVERITIES, MAX_WORKFLOW_REVIEW_FINDINGS, MAX_FINDING_ID_LENGTH, MAX_FINDING_TITLE_LENGTH, MAX_FINDING_BODY_LENGTH, MAX_FINDING_PATH_LENGTH, MAX_WORKFLOW_STEP_PRIOR_ATTEMPTS, TERMINAL_FAILURE_STATUSES, PLAN_REVIEW_LEASE_STALENESS_MS, TERMINAL_STEP_STATUSES;
 var init_workflow_step_results = __esm({
   "../core/src/workflows/workflow-step-results.ts"() {
     "use strict";
+    WORKFLOW_REVIEW_FINDING_SEVERITIES = ["low", "medium", "high", "critical"];
+    MAX_WORKFLOW_REVIEW_FINDINGS = 20;
+    MAX_FINDING_ID_LENGTH = 128;
+    MAX_FINDING_TITLE_LENGTH = 240;
+    MAX_FINDING_BODY_LENGTH = 4e3;
+    MAX_FINDING_PATH_LENGTH = 1e3;
     MAX_WORKFLOW_STEP_PRIOR_ATTEMPTS = 5;
     TERMINAL_FAILURE_STATUSES = /* @__PURE__ */ new Set([
       "failed",
@@ -127217,6 +127277,7 @@ var init_task_timing = __esm({
 });
 
 // ../core/src/session-identity-registry.ts
+import { AsyncLocalStorage } from "node:async_hooks";
 import { resolve as resolve21 } from "node:path";
 import { realpathSync as realpathSync5 } from "node:fs";
 function getRegistry() {
@@ -127228,6 +127289,16 @@ function getRegistry() {
   }
   return registry3;
 }
+function getActiveInvocationStorage() {
+  const holder = globalThis;
+  const existing = holder[ACTIVE_INVOCATION_KEY];
+  if (existing instanceof AsyncLocalStorage) {
+    return existing;
+  }
+  const storage = new AsyncLocalStorage();
+  holder[ACTIVE_INVOCATION_KEY] = storage;
+  return storage;
+}
 function canonicalizeCwd(cwd) {
   const resolved = resolve21(cwd);
   try {
@@ -127255,8 +127326,13 @@ function registerFusionSessionIdentity(cwd, identity) {
   };
 }
 function resolveFusionSessionPrincipal(cwd) {
+  const canonicalCwd = canonicalizeCwd(cwd);
+  const activeInvocation = getActiveInvocationStorage().getStore();
+  if (activeInvocation?.cwdKeys.has(canonicalCwd)) {
+    return { kind: "agent", identity: activeInvocation.identity };
+  }
   const registry3 = getRegistry();
-  const list = registry3.get(canonicalizeCwd(cwd));
+  const list = registry3.get(canonicalCwd);
   if (!list || list.length === 0) {
     return { kind: "operator" };
   }
@@ -127265,14 +127341,24 @@ function resolveFusionSessionPrincipal(cwd) {
   }
   return { kind: "ambiguous", identities: [...list] };
 }
+function runWithFusionSessionIdentity(cwdKeys, identity, callback) {
+  const invocation = {
+    identity: { ...identity, registeredAt: Date.now() },
+    cwdKeys: new Set(cwdKeys.map(canonicalizeCwd))
+  };
+  return getActiveInvocationStorage().run(invocation, callback);
+}
 function __clearFusionSessionIdentityRegistryForTests() {
-  globalThis[REGISTRY_KEY] = /* @__PURE__ */ new Map();
+  const holder = globalThis;
+  holder[REGISTRY_KEY] = /* @__PURE__ */ new Map();
+  holder[ACTIVE_INVOCATION_KEY] = new AsyncLocalStorage();
 }
-var REGISTRY_KEY;
+var REGISTRY_KEY, ACTIVE_INVOCATION_KEY;
 var init_session_identity_registry = __esm({
   "../core/src/session-identity-registry.ts"() {
     "use strict";
     REGISTRY_KEY = "__FUSION_SESSION_IDENTITY_REGISTRY_V1__";
+    ACTIVE_INVOCATION_KEY = "__FUSION_SESSION_IDENTITY_ACTIVE_INVOCATION_V1__";
   }
 });
 
@@ -127635,6 +127721,7 @@ __export(src_exports, {
   MAX_TITLE_LENGTH: () => MAX_TITLE_LENGTH,
   MAX_TITLE_SUMMARIZE_INPUT_LENGTH: () => MAX_TITLE_SUMMARIZE_INPUT_LENGTH,
   MAX_WORKFLOW_ICON_LENGTH: () => MAX_WORKFLOW_ICON_LENGTH,
+  MAX_WORKFLOW_REVIEW_FINDINGS: () => MAX_WORKFLOW_REVIEW_FINDINGS,
   MAX_WORKFLOW_STEP_PRIOR_ATTEMPTS: () => MAX_WORKFLOW_STEP_PRIOR_ATTEMPTS,
   MEMORY_AUDIT_PATH: () => MEMORY_AUDIT_PATH,
   MEMORY_BACKEND_SETTINGS_KEYS: () => MEMORY_BACKEND_SETTINGS_KEYS,
@@ -127836,6 +127923,7 @@ __export(src_exports, {
   WAKE_DELTA_ASSIGNED_TASKS_CAP: () => WAKE_DELTA_ASSIGNED_TASKS_CAP,
   WEDGE_RENOTIFY_COOLDOWN_MS: () => WEDGE_RENOTIFY_COOLDOWN_MS,
   WORKFLOW_EXTENSION_SCHEMA_VERSION: () => WORKFLOW_EXTENSION_SCHEMA_VERSION,
+  WORKFLOW_REVIEW_FINDING_SEVERITIES: () => WORKFLOW_REVIEW_FINDING_SEVERITIES,
   WORKFLOW_SETTING_REJECTION_CODES: () => WORKFLOW_SETTING_REJECTION_CODES,
   WORKFLOW_SETTING_TYPES: () => WORKFLOW_SETTING_TYPES,
   WORKFLOW_WORK_ITEM_KINDS: () => WORKFLOW_WORK_ITEM_KINDS,
@@ -128241,6 +128329,7 @@ __export(src_exports, {
   hasRoleOverrides: () => hasRoleOverrides,
   hasSyncPassphraseConfigured: () => hasSyncPassphraseConfigured,
   hasTitleIdDrift: () => hasTitleIdDrift,
+  hasUserAutoMergeHold: () => hasUserAutoMergeHold,
   hashWorkflowIr: () => hashWorkflowIr,
   healSchemaDrift: () => healSchemaDrift,
   importMcpServersJson: () => importMcpServersJson,
@@ -128363,6 +128452,7 @@ __export(src_exports, {
   isWipColumn: () => isWipColumn,
   isWipColumnRole: () => isWipColumnRole,
   isWorkflowOptionalGroupEnabled: () => isWorkflowOptionalGroupEnabled,
+  isWorkflowReviewFindingSeverity: () => isWorkflowReviewFindingSeverity,
   isWorkspaceTask: () => isWorkspaceTask,
   layoutForIr: () => layoutForIr,
   listActiveAiSessions: () => listActiveAiSessions,
@@ -128436,6 +128526,7 @@ __export(src_exports, {
   normalizeValidationDiagnostics: () => normalizeValidationDiagnostics,
   normalizeWorkflowIcon: () => normalizeWorkflowIcon,
   normalizeWorkflowPromptOverrides: () => normalizeWorkflowPromptOverrides,
+  normalizeWorkflowReviewFindings: () => normalizeWorkflowReviewFindings,
   notifyOperatorOfNonOperatorDelete: () => notifyOperatorOfNonOperatorDelete,
   overseerAdviceSeverityRank: () => overseerAdviceSeverityRank,
   parseAgentManifest: () => parseAgentManifest,
@@ -128699,6 +128790,7 @@ __export(src_exports, {
   runMemoryBackupCommand: () => runMemoryBackupCommand,
   runReconciliationAbort: () => runReconciliationAbort,
   runScheduledEvalBatch: () => runScheduledEvalBatch,
+  runWithFusionSessionIdentity: () => runWithFusionSessionIdentity,
   sanitizeBranchSegment: () => sanitizeBranchSegment,
   sanitizeCliAgentSettings: () => sanitizeCliAgentSettings,
   sanitizeCliAgentsSettings: () => sanitizeCliAgentsSettings,
@@ -157348,6 +157440,7 @@ import { existsSync as existsSync43, readFileSync as readFileSync15, realpathSyn
 import { exec as exec2, execFile as execFile7 } from "node:child_process";
 import { promisify as promisify6 } from "node:util";
 import { createRequire as createRequire5 } from "node:module";
+import { randomUUID as randomUUID42 } from "node:crypto";
 import { basename as basename13, dirname as dirname21, join as join56, relative as relative6, isAbsolute as isAbsolute8, resolve as resolve24 } from "node:path";
 import {
   createAgentSession,
@@ -159058,7 +159151,8 @@ async function createFnAgent2(options) {
     });
   };
   const sessionIdentity = (() => {
-    const principalAgentId = options.actionGateContext?.agentId ?? options.permanentAgentGating?.requester?.actorId ?? "engine-session";
+    const namedAgentId = options.actionGateContext?.agentId ?? options.permanentAgentGating?.requester?.actorId;
+    const principalAgentId = namedAgentId ?? `engine-session-${randomUUID42()}`;
     const principalAgentName = options.actionGateContext?.agentName ?? options.permanentAgentGating?.requester?.actorName;
     return {
       agentId: principalAgentId,
@@ -159070,6 +159164,14 @@ async function createFnAgent2(options) {
   const sessionIdentityKeys = [...new Set([options.cwd, resolvedProjectRoot].filter((key) => Boolean(key)))];
   const attachSessionIdentity = (session) => {
     const identityDisposers = sessionIdentityKeys.map((key) => registerFusionSessionIdentity(key, sessionIdentity));
+    const sessionInvocations = session;
+    const wrapInvocation = (methodName) => {
+      const original = sessionInvocations[methodName];
+      if (typeof original !== "function") return;
+      sessionInvocations[methodName] = (...args) => runWithFusionSessionIdentity(sessionIdentityKeys, sessionIdentity, () => original.apply(session, args));
+    };
+    wrapInvocation("prompt");
+    wrapInvocation("promptWithFallback");
     const disposeBeforeIdentity = typeof session.dispose === "function" ? session.dispose.bind(session) : () => void 0;
     session.dispose = async () => {
       for (const disposeIdentity of identityDisposers) {
@@ -167269,10 +167371,11 @@ function createMergeAttemptHandler(deps) {
     );
   };
 }
-function createMergeGateHandler() {
+function createMergeGateHandler(deps = {}) {
   return async (_node, ctx) => {
     const settingsAutoMerge = ctx.settings?.autoMerge;
-    const autoMerge = ctx.task.autoMerge !== false && settingsAutoMerge !== false;
+    const settings = { autoMerge: settingsAutoMerge ?? true };
+    const autoMerge = !hasUserAutoMergeHold(ctx.task) && (await deps.isLiveSharedBranchMember?.(ctx.task, settings) === true || ctx.task.autoMerge !== false && settings.autoMerge !== false);
     return {
       outcome: "success",
       value: autoMerge ? "auto-on" : "auto-off"
@@ -167282,6 +167385,7 @@ function createMergeGateHandler() {
 var init_merge_runner = __esm({
   "../engine/src/workflow-node-runners/merge-runner.ts"() {
     "use strict";
+    init_src2();
     init_workflow_merge_nodes();
   }
 });
@@ -167625,7 +167729,7 @@ function createDefaultNodeHandlers(seams, runCustomNode, deps) {
     "parse-steps": parseSteps,
     code: createCodeNodeHandler(deps?.runCode),
     notify: createNotifyHandler(deps?.notifyDispatch),
-    "merge-gate": createMergeGateHandler(),
+    "merge-gate": createMergeGateHandler({ isLiveSharedBranchMember: deps?.isLiveSharedBranchMember }),
     "merge-attempt": createMergeAttemptHandler({
       primitives: deps?.primitives,
       seams,
@@ -168023,7 +168127,7 @@ var init_code_node_runner = __esm({
 // ../engine/src/agent-tools.ts
 import { appendFile as appendFile3, mkdir as mkdir26, readFile as readFile32, readdir as readdir9, realpath, stat as stat9, writeFile as writeFile24 } from "node:fs/promises";
 import { existsSync as existsSync50 } from "node:fs";
-import { createHash as createHash19, randomUUID as randomUUID42 } from "node:crypto";
+import { createHash as createHash19, randomUUID as randomUUID43 } from "node:crypto";
 import { tmpdir as tmpdir5 } from "node:os";
 import { extname as extname3, isAbsolute as isAbsolute12, join as join62, relative as relative11, resolve as resolve33, sep as sep9 } from "node:path";
 import { Type as Type2 } from "@earendil-works/pi-ai";
@@ -168669,7 +168773,7 @@ function createTaskCreateTool(store, provenance, options) {
               toType: "user",
               type: "agent-to-user",
               content: `Task proposal awaiting validation: ${title}`,
-              metadata: { kind: "task-proposal", proposalStatus: "pending", proposalIdempotencyKey: randomUUID42(), taskId: options.sourceTaskId, proposedTask: { title, description: params.description, priority: params.priority, workflowId: params.workflow_id, dependencies: params.dependencies } }
+              metadata: { kind: "task-proposal", proposalStatus: "pending", proposalIdempotencyKey: randomUUID43(), taskId: options.sourceTaskId, proposedTask: { title, description: params.description, priority: params.priority, workflowId: params.workflow_id, dependencies: params.dependencies } }
             });
             return { content: [{ type: "text", text: "Task proposal submitted to the operator for validation; no task was created." }], details: { proposed: true } };
           }
@@ -173094,37 +173198,6 @@ var init_effective_settings = __esm({
   }
 });
 
-// ../engine/src/errors/transient-merge-error-classifier.ts
-function classifyTransientMergeError(error3) {
-  if (!error3) return null;
-  if (/lease-handoff-failed[^a-z]+target-not-queued/i.test(error3)) {
-    return "lease-handoff-target-not-queued";
-  }
-  if (/\bACP turn failed\b/i.test(error3) || /\bacp rpc code -32(?:603|00[0-3])\b/i.test(error3)) {
-    return "ai-provider-turn-failure";
-  }
-  if (/\bspawn(?:\s+\S+)?\s+ENO(?:TDIR|ENT)\b/i.test(error3)) {
-    return "process-spawn-failure";
-  }
-  if (/\bis not a working tree\b/i.test(error3)) {
-    return "process-spawn-failure";
-  }
-  const sameSha = error3.match(/advanced concurrently \(expected ([0-9a-f]{7,40}),\s+observed ([0-9a-f]{7,40})\)/i);
-  if (sameSha && sameSha[1].toLowerCase() === sameSha[2].toLowerCase()) {
-    return "spurious-concurrent-advance-same-sha";
-  }
-  if (isTransientError(error3)) {
-    return "network-transport-failure";
-  }
-  return null;
-}
-var init_transient_merge_error_classifier = __esm({
-  "../engine/src/errors/transient-merge-error-classifier.ts"() {
-    "use strict";
-    init_transient_error_patterns();
-  }
-});
-
 // ../engine/src/notification/ntfy-provider.ts
 function resolveParticipantLabel(metadata, kind) {
   const nameKey = kind === "from" ? "fromName" : "toName";
@@ -173598,7 +173671,50 @@ var init_webhook_provider = __esm({
   }
 });
 
+// ../engine/src/errors/transient-merge-error-classifier.ts
+function classifyTransientMergeError(error3) {
+  if (!error3) return null;
+  if (/lease-handoff-failed[^a-z]+target-not-queued/i.test(error3)) {
+    return "lease-handoff-target-not-queued";
+  }
+  if (/\bACP turn failed\b/i.test(error3) || /\bacp rpc code -32(?:603|00[0-3])\b/i.test(error3)) {
+    return "ai-provider-turn-failure";
+  }
+  if (/\bspawn(?:\s+\S+)?\s+ENO(?:TDIR|ENT)\b/i.test(error3)) {
+    return "process-spawn-failure";
+  }
+  if (/\bis not a working tree\b/i.test(error3)) {
+    return "process-spawn-failure";
+  }
+  const sameSha = error3.match(/advanced concurrently \(expected ([0-9a-f]{7,40}),\s+observed ([0-9a-f]{7,40})\)/i);
+  if (sameSha && sameSha[1].toLowerCase() === sameSha[2].toLowerCase()) {
+    return "spurious-concurrent-advance-same-sha";
+  }
+  if (isTransientError(error3)) {
+    return "network-transport-failure";
+  }
+  return null;
+}
+function hasTransientMergeRecoveryOwner(task) {
+  return classifyTransientMergeError(task.error) !== null && typeof task.mergeTransientRetryCount === "number" && Number.isInteger(task.mergeTransientRetryCount) && task.mergeTransientRetryCount >= 0 && task.mergeTransientRetryCount < MAX_AUTO_MERGE_TRANSIENT_RETRIES;
+}
+var MAX_AUTO_MERGE_TRANSIENT_RETRIES;
+var init_transient_merge_error_classifier = __esm({
+  "../engine/src/errors/transient-merge-error-classifier.ts"() {
+    "use strict";
+    init_transient_error_patterns();
+    MAX_AUTO_MERGE_TRANSIENT_RETRIES = 5;
+  }
+});
+
 // ../engine/src/notification/task-wedge-notification.ts
+function describeTaskRecoveryOwner(task) {
+  if (hasTransientMergeRecoveryOwner(task)) return { kind: "transient-merge-retry" };
+  if (typeof task.recoveryRetryCount === "number" && Number.isInteger(task.recoveryRetryCount) && task.recoveryRetryCount >= 0 && typeof task.nextRecoveryAt === "string" && Number.isFinite(Date.parse(task.nextRecoveryAt))) {
+    return { kind: "scheduled-recovery" };
+  }
+  return null;
+}
 function describeSelfHealingNoActionWedge(task, stage, metadata) {
   const description = SELF_HEALING_NO_ACTIONS[stage];
   if (!description || task.userPaused || task.paused || task.autoMerge === false) return null;
@@ -173656,6 +173772,7 @@ function describeTaskWedge(task) {
   if (/merge.*(?:blocked|verification|gate)|(?:verification|gate).*(?:failed|exhaust)/i.test(error3)) {
     return { reasonKey: "merge-blocked", reason: "Merge verification cannot progress without operator action.", action: "Fix the failing verification, then retry the task." };
   }
+  if (describeTaskRecoveryOwner(task)) return null;
   if (!error3 && (task.mergeRetries ?? 0) < 3) return null;
   return {
     reasonKey: "terminal-failed",
@@ -173667,6 +173784,7 @@ var SELF_HEALING_NO_ACTIONS;
 var init_task_wedge_notification = __esm({
   "../engine/src/notification/task-wedge-notification.ts"() {
     "use strict";
+    init_transient_merge_error_classifier();
     SELF_HEALING_NO_ACTIONS = {
       "reclaim-pr-conflict": {
         reason: "Self-healing could not reclaim a stalled pull-request conflict.",
@@ -173732,7 +173850,6 @@ var init_notification_service = __esm({
     init_src2();
     init_notifier();
     init_logger3();
-    init_transient_merge_error_classifier();
     init_ntfy_provider();
     init_webhook_provider();
     init_task_wedge_notification();
@@ -173958,10 +174075,10 @@ var init_notification_service = __esm({
         }
       }
       handleTaskUpdated = (task, meta2) => {
-        const transientFailure = task.status === "failed" ? classifyTransientMergeError(task.error) : null;
-        const wedge = transientFailure ? null : describeTaskWedge(task);
+        const recoveryOwner = task.status === "failed" ? describeTaskRecoveryOwner(task) : null;
+        const wedge = describeTaskWedge(task);
         if (wedge) this.cancelPendingFailureNotification(task.id, "classified-terminal-wedge");
-        if (!transientFailure) void this.enqueueWedgeHandling(task.id, () => this.maybeNotifyTaskWedge(task, wedge));
+        void this.enqueueWedgeHandling(task.id, () => this.maybeNotifyTaskWedge(task, wedge));
         void this.maybeSuppressTransientFailedNotification(task, `status=${task.status ?? "undefined"}`);
         if (task.status === "awaiting-approval") {
           void this.writeAwaitingApprovalMailboxMessage(task);
@@ -173972,17 +174089,12 @@ var init_notification_service = __esm({
         if (!this.notificationsEnabled) {
           return;
         }
-        if (task.status === "failed" && !wedge) {
-          const transientClass = classifyTransientMergeError(task.error);
-          if (transientClass) {
-            this.failureNotificationSuppressedCount += 1;
-            schedulerLog.debug(
-              `[notify] ${task.id} transient merge failure (${transientClass}) \u2014 suppressed notification (self-heal in flight)`
-            );
-            return;
-          }
+        if (task.status === "failed" && recoveryOwner) {
+          this.failureNotificationSuppressedCount += 1;
+          schedulerLog.debug(`[notify] ${task.id} recovery-owned failure \u2014 suppressed notification`);
+        } else if (task.status === "failed" && !wedge) {
           if (this.failureNotificationMode === "all") {
-            this.maybeNotify(task.id, "failed", this.createTaskPayload(task, "failed"));
+            void this.maybeNotifyImmediateFailure(task);
           } else {
             this.scheduleFailureNotification(task);
           }
@@ -174069,7 +174181,17 @@ var init_notification_service = __esm({
         await this.enqueueWedgeHandling(task.id, () => this.maybeNotifyTaskWedge(task, descriptor));
       }
       async maybeNotifyTaskWedge(task, suppliedDescriptor) {
-        const descriptor = suppliedDescriptor ?? describeTaskWedge(task);
+        const liveTask = this.store.getTask ? await this.store.getTask(task.id) ?? task : task;
+        const recoveryOwner = describeTaskRecoveryOwner(liveTask);
+        if (recoveryOwner) {
+          if (this.activeWedgeReasons.has(task.id) || liveTask.wedgeNotification?.status === "active") {
+            this.activeWedgeReasons.delete(task.id);
+            await this.store.claimTaskWedgeNotificationEpisode?.(task.id, null);
+          }
+          return;
+        }
+        const descriptor = suppliedDescriptor ?? describeTaskWedge(liveTask);
+        task = liveTask;
         let episode;
         if (!descriptor) {
           const isActiveSelfHealingNoAction = task.wedgeNotification?.status === "active" && task.wedgeNotification.reasonKey.startsWith("self-healing-no-action:");
@@ -174436,6 +174558,14 @@ var init_notification_service = __esm({
         this.failureNotificationDelayMs = typeof settings.failureNotificationDelayMs === "number" && settings.failureNotificationDelayMs >= 0 ? settings.failureNotificationDelayMs : this.failedNotificationGraceMs;
         this.failureNotificationMode = settings.failureNotificationMode ?? "sticky-only";
       }
+      async maybeNotifyImmediateFailure(task) {
+        const liveTask = this.store.getTask ? await this.store.getTask(task.id) ?? task : task;
+        if (liveTask.status !== "failed" || describeTaskRecoveryOwner(liveTask) || describeTaskWedge(liveTask)) {
+          this.failureNotificationSuppressedCount += 1;
+          return;
+        }
+        this.maybeNotify(liveTask.id, "failed", this.createTaskPayload(liveTask, "failed"));
+      }
       scheduleFailureNotification(task) {
         if (this.pendingFailureNotifications.has(task.id)) {
           return;
@@ -174492,12 +174622,9 @@ var init_notification_service = __esm({
           schedulerLog.debug(`[notify] ${taskId} no longer failed at dispatch time \u2014 suppressed notification`);
           return;
         }
-        const transientClassAtDispatch = classifyTransientMergeError(task.error);
-        if (transientClassAtDispatch) {
+        if (describeTaskRecoveryOwner(task)) {
           this.failureNotificationSuppressedCount += 1;
-          schedulerLog.debug(
-            `[notify] ${taskId} transient merge failure (${transientClassAtDispatch}) at dispatch time \u2014 suppressed notification (self-heal in flight)`
-          );
+          schedulerLog.debug(`[notify] ${taskId} recovery-owned failure at dispatch time \u2014 suppressed notification`);
           return;
         }
         if (describeTaskWedge(task)) {
@@ -186152,10 +186279,21 @@ var init_scheduler = __esm({
               const unmetDeps = getUnmetSchedulingDependencies(task, tasks2, schedulingDependencyOptions);
               if (unmetDeps.length > 0) {
                 const normalizedUnmetDeps = [...new Set(unmetDeps)].sort();
+                let activeOverlapBlockedBy = null;
+                if (activeScopes.size > 0) {
+                  try {
+                    const taskScope = await getFilteredFileScope(task.id);
+                    if (taskScope.length > 0 && !isCoordinationOnlyTask(task, taskScope)) {
+                      activeOverlapBlockedBy = Array.from(activeScopes.entries()).sort(([aId], [bId]) => aId.localeCompare(bId)).find(([, activeScope]) => this.pathsOverlap(taskScope, activeScope))?.[0] ?? null;
+                    }
+                  } catch (error3) {
+                    schedulerLog.warn(`Failed to refresh file-scope overlap blocker for dependency-blocked task ${task.id}`, error3);
+                  }
+                }
                 await this.transitionQueuedEpisode(task, {
                   signature: `dependency:${normalizedUnmetDeps.join(",")}`,
                   blockedBy: unmetDeps[0] ?? null,
-                  overlapBlockedBy: task.overlapBlockedBy ?? null,
+                  overlapBlockedBy: activeOverlapBlockedBy,
                   action: `queued \u2014 unmet dependencies: ${unmetDeps.join(", ")}`
                 });
                 this.options.onBlocked?.(task, unmetDeps);
@@ -202212,16 +202350,29 @@ var init_self_healing = __esm({
           activeWork: await this.resolveActiveWorkColumnsFor(taskId, cache)
         };
       }
-      classifyPausedAbortWorkflowRecovery(task, settings, isExecuting, columns) {
+      /*
+      FNXC:SharedBranchMemberHold 2026-08-05-23:35:
+      Pause-abort recovery must resolve the same live intermediate-group predicate as
+      merge admission. Shared metadata alone is stale after group closure or a
+      default-branch collision, so it cannot suppress that task's standalone hold.
+      */
+      async isLiveSharedMemberIntegration(task, settings) {
+        const groupId = task.branchContext?.groupId?.trim();
+        const group = groupId ? await this.store.getBranchGroup?.(groupId) : null;
+        const defaultBranch = await resolveIntegrationBranch(this.options.rootDir, settings);
+        return isLiveSharedBranchGroupMemberIntegration(task, group, defaultBranch);
+      }
+      async classifyPausedAbortWorkflowRecovery(task, settings, isExecuting, columns) {
         const isPausedAbortPark = task.status === "failed" && typeof task.error === "string" && task.error.includes(PAUSE_ABORT_PARK_OPERATOR_MARKER) && task.error.includes(PAUSE_ABORT_PARK_ERROR_MARKER);
         if (!isPausedAbortPark) return { kind: "no-action", reason: "not-pause-abort" };
         if (task.paused || task.userPaused || isExecuting) return { kind: "no-action", reason: "unsafe-or-not-routable" };
         const errorText = typeof task.error === "string" ? task.error.toLowerCase() : "";
         const isTerminalMergePark = errorText.includes("conflict") || errorText.includes("contamination") || errorText.includes("foreign") || errorText.includes("retry-exhausted") || errorText.includes("retries exhausted") || errorText.includes("max retries");
         const completedSteps = task.steps.length > 0 && task.steps.every((step) => step.status === "done" || step.status === "skipped");
-        const sharedBranchMember = isSharedBranchGroupMemberIntegration(task);
-        const hasReviewProgress = columns.review.has(task.column) && allowsAutoMergeProcessing(task, settings) && task.mergeDetails?.mergeConfirmed !== true && !isTerminalMergePark && completedSteps;
-        const hasManualMergeHoldProgress = columns.review.has(task.column) && (!allowsAutoMergeProcessing(task, settings) || resolveEffectiveAutoMerge(task, settings) === false) && !sharedBranchMember && task.mergeDetails?.mergeConfirmed !== true && !isTerminalMergePark && completedSteps;
+        const liveSharedBranchMember = await this.isLiveSharedMemberIntegration(task, settings);
+        const autoMergeProcessing = allowsAutoMergeProcessing(task, settings) || liveSharedBranchMember;
+        const hasReviewProgress = columns.review.has(task.column) && autoMergeProcessing && task.mergeDetails?.mergeConfirmed !== true && !isTerminalMergePark && completedSteps;
+        const hasManualMergeHoldProgress = columns.review.has(task.column) && (hasUserAutoMergeHold(task) || !allowsAutoMergeProcessing(task, settings) || resolveEffectiveAutoMerge(task, settings) === false) && (!liveSharedBranchMember || hasUserAutoMergeHold(task)) && task.mergeDetails?.mergeConfirmed !== true && !isTerminalMergePark && completedSteps;
         if (hasManualMergeHoldProgress) {
           return { kind: "work-item-resume", reason: "pause-abort-manual-merge-hold" };
         }
@@ -207889,8 +208040,23 @@ var init_self_healing = __esm({
             return own.length > 0 ? new Set(own) : /* @__PURE__ */ new Set(["in-review"]);
           };
           const executingIds = this.options.getExecutingTaskIds?.() ?? /* @__PURE__ */ new Set();
+          const canRecoverMergeableReviewTask = async (task) => {
+            if (hasUserAutoMergeHold(task)) return false;
+            const groupId = task.branchContext?.groupId?.trim();
+            const branchGroup = groupId ? await this.store.getBranchGroup(groupId) : null;
+            const projectDefaultBranch = await resolveIntegrationBranch(this.options.rootDir, settings);
+            if (isLiveSharedBranchGroupMemberIntegration(task, branchGroup, projectDefaultBranch)) {
+              return true;
+            }
+            return allowsAutoMergeProcessing(task, settings) && resolveEffectiveAutoMerge(task, settings) !== false;
+          };
+          const mergeAdmission = await Promise.all(tasks2.map(async (task) => [
+            task.id,
+            await canRecoverMergeableReviewTask(task)
+          ]));
+          const mergeAdmissionByTaskId = new Map(mergeAdmission);
           const mergeable = tasks2.filter(
-            (t) => allowsAutoMergeProcessing(t, settings) && !t.paused && !executingIds.has(t.id) && t.status !== "failed" && // Exclude transient merge statuses. Active merges should be left alone;
+            (t) => mergeAdmissionByTaskId.get(t.id) === true && !t.paused && !executingIds.has(t.id) && t.status !== "failed" && // Exclude transient merge statuses. Active merges should be left alone;
             // stale ones are handled by recoverStaleMergingStatus().
             t.status !== "merging" && t.status !== "merging-pr" && // FNXC:Workspace 2026-06-22-09:30 (Phase D U1, KTD1 — admit workspace tasks):
             // A workspace task has task.worktree===null (its worktrees live per-repo in
@@ -210937,7 +211103,7 @@ var init_self_healing = __esm({
           for (const t of tasks2) {
             if (!this.isPauseAbortParkCandidate(t)) continue;
             const columns = await this.resolvePauseAbortColumnsFor(t.id, columnCache);
-            if (this.classifyPausedAbortWorkflowRecovery(t, settings, executingIds.has(t.id), columns).kind !== "no-action") {
+            if ((await this.classifyPausedAbortWorkflowRecovery(t, settings, executingIds.has(t.id), columns)).kind !== "no-action") {
               parked.push(t);
             }
           }
@@ -210954,7 +211120,7 @@ var init_self_healing = __esm({
                 continue;
               }
               const freshColumns = await this.resolvePauseAbortColumnsFor(fresh.id, columnCache);
-              const route = this.classifyPausedAbortWorkflowRecovery(fresh, settings, latestExecutingIds.has(fresh.id), freshColumns);
+              const route = await this.classifyPausedAbortWorkflowRecovery(fresh, settings, latestExecutingIds.has(fresh.id), freshColumns);
               if (route.kind === "no-action") {
                 continue;
               }
@@ -217957,7 +218123,7 @@ function recoverPassedPlanReviewFromLatestLog(task) {
     completedAt: latest.timestamp
   };
 }
-var PLAN_REVIEW_PROVIDER_FAILURE_HOLD_VALUE, PLAN_REVIEW_LEASE_HELD_VALUE, SESSION_CONTENTION_HOLD_VALUE, WORKFLOW_INTERRUPTED_NODE_ID_CONTEXT_KEY, WORKFLOW_INTERRUPTED_NODE_ABORT_KIND_CONTEXT_KEY, WORKFLOW_OPTIONAL_GROUP_CONTEXT_KEY, WORKFLOW_NODE_ENGINE_PAUSE_ABORT_KIND, WORKFLOW_DRIFT_PARK_CONTEXT_KEY, WorkflowGraphSuspended, MERGE_REGION_KINDS, WorkflowGraphExecutor;
+var PLAN_REVIEW_PROVIDER_FAILURE_HOLD_VALUE, PLAN_REVIEW_LEASE_HELD_VALUE, SESSION_CONTENTION_HOLD_VALUE, WORKFLOW_INTERRUPTED_NODE_ID_CONTEXT_KEY, WORKFLOW_INTERRUPTED_NODE_ABORT_KIND_CONTEXT_KEY, WORKFLOW_OPTIONAL_GROUP_CONTEXT_KEY, WORKFLOW_REVIEW_KIND_CONTEXT_KEY, WORKFLOW_NODE_ENGINE_PAUSE_ABORT_KIND, WORKFLOW_DRIFT_PARK_CONTEXT_KEY, WorkflowGraphSuspended, MERGE_REGION_KINDS, WorkflowGraphExecutor;
 var init_workflow_graph_executor = __esm({
   "../engine/src/workflows/workflow-graph-executor.ts"() {
     "use strict";
@@ -217977,6 +218143,7 @@ var init_workflow_graph_executor = __esm({
     WORKFLOW_INTERRUPTED_NODE_ID_CONTEXT_KEY = "workflow:interruptedNodeId";
     WORKFLOW_INTERRUPTED_NODE_ABORT_KIND_CONTEXT_KEY = "workflow:interruptedNodeAbortKind";
     WORKFLOW_OPTIONAL_GROUP_CONTEXT_KEY = "workflow:optionalGroupActive";
+    WORKFLOW_REVIEW_KIND_CONTEXT_KEY = "workflow:reviewKind";
     WORKFLOW_NODE_ENGINE_PAUSE_ABORT_KIND = "engine-pause";
     WORKFLOW_DRIFT_PARK_CONTEXT_KEY = "workflow:driftPark";
     WorkflowGraphSuspended = class extends Error {
@@ -218004,7 +218171,8 @@ var init_workflow_graph_executor = __esm({
             parseSteps: deps.parseStepsDeps,
             runCode: deps.runCode,
             notifyDispatch: deps.notifyDispatch,
-            prNodes: deps.prNodes
+            prNodes: deps.prNodes,
+            isLiveSharedBranchMember: deps.isLiveSharedBranchMember
           }),
           ...deps.runnerRegistry?.toHandlers() ?? {},
           ...deps.handlers ?? {}
@@ -218225,6 +218393,7 @@ var init_workflow_graph_executor = __esm({
               phase: stepPhase,
               status: "pending",
               source: "optional-group",
+              ...this.workflowReviewKind(node) ? { reviewKind: this.workflowReviewKind(node) } : {},
               startedAt: stepStartedAt,
               // U3/KTD-4: stamp the lease owner so a concurrent/crashed re-entry
               // adopts this pending gate instead of dispatching a second reviewer.
@@ -218244,7 +218413,8 @@ var init_workflow_graph_executor = __esm({
               runTemplateNode: (tNode, sig, contextOverride) => {
                 const optionalGroupContext = {
                   ...contextOverride ?? context,
-                  [WORKFLOW_OPTIONAL_GROUP_CONTEXT_KEY]: node.id
+                  [WORKFLOW_OPTIONAL_GROUP_CONTEXT_KEY]: node.id,
+                  ...this.workflowReviewKind(node) ? { [WORKFLOW_REVIEW_KIND_CONTEXT_KEY]: this.workflowReviewKind(node) } : {}
                 };
                 return this.executeNodeWithRetries(tNode, task, settings, optionalGroupContext, ir2, sig, false);
               },
@@ -218262,6 +218432,7 @@ var init_workflow_graph_executor = __esm({
             const exitContextPatch = exitResult?.contextPatch;
             let stepOutput = typeof exitContextPatch?.output === "string" ? exitContextPatch.output : void 0;
             const stepNotes = typeof exitContextPatch?.notes === "string" ? exitContextPatch.notes : void 0;
+            const stepFindings = this.workflowReviewKind(node) && Array.isArray(exitContextPatch?.findings) ? exitContextPatch.findings : void 0;
             if (stepStatus === "failed" && !verdict && stepOutput === void 0 && stepNotes === void 0) {
               const lastVisited = groupResult.visitedNodeIds[groupResult.visitedNodeIds.length - 1];
               const templateNodeId = lastVisited?.includes("::") ? lastVisited.slice(lastVisited.indexOf("::") + 2) : void 0;
@@ -218279,9 +218450,11 @@ var init_workflow_graph_executor = __esm({
               phase: stepPhase,
               source: "optional-group",
               status: stepStatus,
+              ...this.workflowReviewKind(node) ? { reviewKind: this.workflowReviewKind(node) } : {},
               ...verdict ? { verdict } : {},
               ...stepOutput !== void 0 ? { output: stepOutput } : {},
               ...stepNotes !== void 0 ? { notes: stepNotes } : {},
+              ...stepFindings?.length ? { findings: stepFindings } : {},
               startedAt: stepStartedAt,
               completedAt: (/* @__PURE__ */ new Date()).toISOString()
             });
@@ -218741,9 +218914,15 @@ var init_workflow_graph_executor = __esm({
         }
         return failureResult;
       }
+      /** FNXC:WorkflowReviewKind 2026-08-05-02:31: Persist only the declared closed
+       * marker; execution never derives review semantics from a label, verdict, or gate mode. */
+      workflowReviewKind(node) {
+        return node.config?.reviewKind === "plan" || node.config?.reviewKind === "code" ? node.config.reviewKind : void 0;
+      }
       shouldRecordNodeProgress(node) {
         const skillName = typeof node.config?.skillName === "string" ? node.config.skillName.trim() : "";
-        return skillName.length > 0 && (node.kind === "prompt" || node.kind === "gate");
+        const markedReview = this.workflowReviewKind(node) !== void 0;
+        return markedReview && (node.kind === "prompt" || node.kind === "gate" || node.kind === "script") || skillName.length > 0 && (node.kind === "prompt" || node.kind === "gate");
       }
       workflowNodeProgressName(node) {
         const configuredName = typeof node.config?.name === "string" ? node.config.name.trim() : "";
@@ -218757,6 +218936,7 @@ var init_workflow_graph_executor = __esm({
           phase: node.config?.phase === "post-merge" ? "post-merge" : "pre-merge",
           source: "node",
           status: "pending",
+          ...this.workflowReviewKind(node) ? { reviewKind: this.workflowReviewKind(node) } : {},
           startedAt
         };
         await this.recordOptionalGroupStepResult(taskId, result2);
@@ -218767,6 +218947,7 @@ var init_workflow_graph_executor = __esm({
         const contextPatch = nodeResult.contextPatch ?? {};
         let output = typeof contextPatch.output === "string" ? contextPatch.output : void 0;
         const notes = typeof contextPatch.notes === "string" ? contextPatch.notes : void 0;
+        const findings = this.workflowReviewKind(node) && Array.isArray(contextPatch.findings) ? contextPatch.findings : void 0;
         if (status === "failed" && output === void 0 && notes === void 0) {
           output = this.synthesizeNonVerdictFailureOutput({
             stepLabel: this.workflowNodeProgressName(node),
@@ -218781,8 +218962,10 @@ var init_workflow_graph_executor = __esm({
           phase: started?.phase ?? (node.config?.phase === "post-merge" ? "post-merge" : "pre-merge"),
           source: "node",
           status,
+          ...this.workflowReviewKind(node) ? { reviewKind: this.workflowReviewKind(node) } : {},
           ...output !== void 0 ? { output } : {},
           ...notes !== void 0 ? { notes } : {},
+          ...findings?.length ? { findings } : {},
           startedAt: started?.startedAt ?? (/* @__PURE__ */ new Date()).toISOString(),
           completedAt: (/* @__PURE__ */ new Date()).toISOString()
         });
@@ -219225,6 +219408,10 @@ var init_workflow_graph_task_runner = __esm({
             runCode: this.deps.runCode,
             notifyDispatch: this.deps.notifyDispatch,
             prNodes: this.deps.prNodes,
+            // FNXC:SharedBranchMemberHold 2026-08-06-00:24: carry the executor's
+            // live group resolver into the graph merge gate; omitting it turns
+            // mission-policy members into manual holds before integration.
+            isLiveSharedBranchMember: this.deps.isLiveSharedBranchMember,
             /*
             FNXC:WorkflowResume 2026-06-29-08:49:
             Production graph runs must fetch live task steps during foreach replay. The runner is the workflow boundary that has store access, so it supplies the fresh projection seam instead of making executor self-healing guess after a stale step node fails.
@@ -224063,9 +224250,11 @@ function parseWorkflowStepVerdict(rawOutput) {
         verdict = "REVISE";
       }
       if (!verdict) continue;
+      const findings = normalizeWorkflowReviewFindings(parsed.findings);
       return {
         verdict,
-        notes: typeof parsed.notes === "string" ? parsed.notes : ""
+        notes: typeof parsed.notes === "string" ? parsed.notes : "",
+        ...findings ? { findings } : {}
       };
     } catch {
     }
@@ -224097,7 +224286,8 @@ function parseWorkflowStepOutput(rawOutput, options = {}) {
     return {
       output: parsed.notes || "",
       verdict: parsed.verdict,
-      notes: parsed.notes
+      notes: parsed.notes,
+      ...parsed.findings ? { findings: parsed.findings } : {}
     };
   }
   const inferred = inferWorkflowStepVerdictFromProse(trimmed);
@@ -227334,6 +227524,7 @@ ${feedback}`,
         if (info.phase !== "pre-merge") return false;
         if (info.status !== "advisory_failure" && info.status !== "failed") return false;
         const liveTask = await this.store.getTask(taskId).catch(() => fallbackTask);
+        if (hasUserAutoMergeHold(liveTask)) return false;
         const missingArtifactKeys = parseRequiredArtifactMissingValue(info.failureValue);
         if (missingArtifactKeys) {
           await this.recoverMissingRequiredArtifacts(liveTask, missingArtifactKeys, {
@@ -227590,6 +227781,7 @@ ${info.feedback}`, revisionKey),
        */
       async recoverFailedPreMergeWorkflowStep(task) {
         try {
+          if (hasUserAutoMergeHold(task)) return false;
           const failed = (task.workflowStepResults ?? []).filter((r) => (r.phase || "pre-merge") === "pre-merge" && r.status === "failed").sort((a, b3) => {
             const aTs = Date.parse(a.completedAt || a.startedAt || "");
             const bTs = Date.parse(b3.completedAt || b3.startedAt || "");
@@ -228116,6 +228308,7 @@ ${info.feedback}`, revisionKey),
               getTask: (taskId) => this.store.getTask(taskId)
             },
             runId: resolvedRunId,
+            isLiveSharedBranchMember: (nodeTask) => this.isLiveSharedBranchGroupMember(nodeTask),
             primitives: this.createAuthoritativeWorkflowPrimitives(settings),
             seams: this.createAuthoritativeWorkflowSeams(settings),
             prepareNodeExecution: (node, nodeTask, requirement) => this.prepareGraphNodeExecution(node, nodeTask, settings, requirement),
@@ -230358,6 +230551,7 @@ ${info.feedback}`, revisionKey),
           return this.runCliAgentNode(node, await this.store.getTask(live.id), cfg);
         }
         const optionalGroupId = typeof graphContext?.[WORKFLOW_OPTIONAL_GROUP_CONTEXT_KEY] === "string" ? graphContext[WORKFLOW_OPTIONAL_GROUP_CONTEXT_KEY] : void 0;
+        const declaredReviewKind = cfg.reviewKind === "plan" || cfg.reviewKind === "code" ? cfg.reviewKind : graphContext?.[WORKFLOW_REVIEW_KIND_CONTEXT_KEY] === "plan" || graphContext?.[WORKFLOW_REVIEW_KIND_CONTEXT_KEY] === "code" ? graphContext[WORKFLOW_REVIEW_KIND_CONTEXT_KEY] : void 0;
         const isCompletionSummaryNode2 = cfg.summaryTarget === "task" || node.id === "completion-summary";
         if (live.executionMode === "fast" && !isCompletionSummaryNode2 && !optionalGroupId && !cfg.seam && (node.kind === "prompt" || node.kind === "script" || node.kind === "gate")) {
           executorLog.debug(`${live.id}: fast mode \u2014 skipping custom graph node '${node.id}'`);
@@ -230506,6 +230700,9 @@ ${prompt}`;
         if (optionalGroupId) {
           step.optionalGroupId = optionalGroupId;
         }
+        if (declaredReviewKind) {
+          step.reviewKind = declaredReviewKind;
+        }
         if (cfg.reviewCanFixInline === true) {
           step.reviewCanFixInline = true;
         }
@@ -230518,7 +230715,11 @@ ${prompt}`;
           executorLog.debug(`${live.id}: graph node '${node.id}' runtime env injected (${injected.pathEntryCount} PATH entries, ${injected.injectedKeyCount} env keys)`);
         }
         const unattended = this.graphUnattendedRuns.has(live.id);
-        const outcome = mode === "script" ? await this.executeScriptWorkflowStep(live, step, worktreePath, settings, nodeEnv) : await this.executeWorkflowStep(live, step, worktreePath, settings, nodeEnv, { unattended });
+        let outcome = mode === "script" ? await this.executeScriptWorkflowStep(live, step, worktreePath, settings, nodeEnv) : await this.executeWorkflowStep(live, step, worktreePath, settings, nodeEnv, { unattended });
+        if (declaredReviewKind && typeof outcome.output === "string") {
+          const parsedReviewOutput = parseWorkflowStepOutput(outcome.output, { requireVerdict: false });
+          if (parsedReviewOutput.findings?.length) outcome = { ...outcome, findings: parsedReviewOutput.findings };
+        }
         const awaitQuestion = parseAwaitInputSentinel(outcome.output);
         if (awaitQuestion) {
           await this.store.logEntry(
@@ -230541,6 +230742,8 @@ ${prompt}`;
         const contextPatch = {};
         if (typeof stepOutput === "string") contextPatch.output = stepOutput;
         if (typeof stepNotes === "string" && stepNotes) contextPatch.notes = stepNotes;
+        const stepFindings = outcome.findings;
+        if (stepFindings?.length) contextPatch.findings = stepFindings;
         if (cfg.summaryTarget === "task" && typeof stepOutput === "string" && stepOutput.trim()) {
           contextPatch.summary = stepOutput.trim();
         }
@@ -231047,6 +231250,7 @@ Graph value: ${failureValue}` : ""}`, budget.key),
         } catch {
           return false;
         }
+        if (hasUserAutoMergeHold(live)) return false;
         const sharedBranchMember = await this.isLiveSharedBranchGroupMember(live);
         if (!sharedBranchMember && !allowsAutoMergeProcessing(live, settings)) return false;
         if (!sharedBranchMember && resolveEffectiveAutoMerge(live, settings) === false) return false;
@@ -231108,8 +231312,8 @@ Graph value: ${failureValue}` : ""}`, budget.key),
         } catch {
           return false;
         }
-        if (await this.isLiveSharedBranchGroupMember(live)) return false;
-        return !allowsAutoMergeProcessing(live, settings) || resolveEffectiveAutoMerge(live, settings) === false;
+        if (await this.isLiveSharedBranchGroupMember(live) && !hasUserAutoMergeHold(live)) return false;
+        return hasUserAutoMergeHold(live) || !allowsAutoMergeProcessing(live, settings) || resolveEffectiveAutoMerge(live, settings) === false;
       }
       async handleStaleInReviewPlanPauseAbortReplay(live, result2, abortProvenance, pausedAborted, userCanceled, resumeLanesMemo) {
         if (!pausedAborted) return false;
@@ -231273,9 +231477,9 @@ Graph value: ${failureValue}` : ""}`, budget.key),
           if (settings.globalPause === true) return false;
         }
         if (live.column === resumeLanes.review) {
-          if (live.autoMerge === false) return false;
           if (!settings) return false;
           const sharedBranchMember = await this.isLiveSharedBranchGroupMember(live);
+          if (hasUserAutoMergeHold(live) || live.autoMerge === false && !sharedBranchMember) return false;
           if (!sharedBranchMember && !allowsAutoMergeProcessing(live, settings)) return false;
           if (live.mergeDetails?.mergeConfirmed === true) return false;
         }
@@ -237043,13 +237247,14 @@ CRITICAL SCOPING RULES \u2014 read before doing anything else:
         const isSkillStep = typeof workflowStep.skillName === "string" && workflowStep.skillName.trim().length > 0;
         const isSummaryProjectionStep = workflowStep.summaryTarget === "task";
         const requireVerdict = !isSummaryProjectionStep && (workflowStep.gateMode === "gate" || !isSkillStep);
+        const reviewFindingsContract = workflowStepMetadata.reviewKind === "plan" || workflowStepMetadata.reviewKind === "code";
         const verdictBlock = requireVerdict ? `
 
 ## Feedback Format
 
 When your review is complete, your final line MUST be a single JSON object (no markdown fences):
 
-{"verdict":"APPROVE|APPROVE_WITH_NOTES|REVISE","notes":"..."}
+${reviewFindingsContract ? '{"verdict":"APPROVE|APPROVE_WITH_NOTES|REVISE","notes":"...","findings":[{"id":"stable-id","title":"concise issue","body":"actionable detail","filePath":"optional/path","line":1,"severity":"low|medium|high|critical"}]}' : '{"verdict":"APPROVE|APPROVE_WITH_NOTES|REVISE","notes":"..."}'}
 
 Rules:
 - Output exactly one trailing JSON object and stop.
@@ -237359,7 +237564,8 @@ ${detectedQuestion}
                 revisionRequested,
                 output: parsed.output,
                 verdict: parsed.verdict,
-                notes: parsed.notes
+                notes: parsed.notes,
+                ...parsed.findings ? { findings: parsed.findings } : {}
               };
             }
             if (parsed.malformed) {
@@ -248348,7 +248554,7 @@ var init_metric_parser = __esm({
 });
 
 // ../engine/src/eval/experiment-executor.ts
-import { randomUUID as randomUUID43 } from "node:crypto";
+import { randomUUID as randomUUID44 } from "node:crypto";
 var ExperimentMaxIterationsError, ExperimentGitNotConfiguredError, ExperimentExecutor;
 var init_experiment_executor = __esm({
   "../engine/src/eval/experiment-executor.ts"() {
@@ -248416,7 +248622,7 @@ var init_experiment_executor = __esm({
         }
         await this.semaphore.acquire();
         const controller = new AbortController();
-        const runHandle = randomUUID43();
+        const runHandle = randomUUID44();
         if (opts?.abortSignal) {
           opts.abortSignal.addEventListener("abort", () => controller.abort(), { once: true });
         }
@@ -257423,6 +257629,7 @@ var init_in_process_runtime = __esm({
             }
           }
           const prNodeGithubOps = this.config.prNodeGithubOps;
+          const secretsStore = await this.taskStore.getSecretsStore();
           const executorOptions = {
             /*
             FNXC:PlanReviewLease 2026-07-26-21:12:
@@ -257438,6 +257645,7 @@ var init_in_process_runtime = __esm({
             cliAgentRuntime: this.cliAgentRuntime?.bundle,
             pluginRunner: this.pluginRunner,
             messageStore: this.messageStore,
+            secretsStore,
             missionStore,
             reflectionService,
             // PR-entity nodes (U3): assemble the handler deps from the CLI-injected
@@ -257539,6 +257747,7 @@ var init_in_process_runtime = __esm({
               reflectionService,
               selfImproveService,
               credentialRotator: this.credentialRotator,
+              secretsStore,
               snapshotManager: autoClaimSnapshotManager,
               onMissed: (agentId, reason) => {
                 runtimeLog.warn(`Agent ${agentId} missed heartbeat: ${reason}`);
@@ -261400,7 +261609,21 @@ var init_project_engine = __esm({
       Cleared on the first non-busy outcome (success path resets it).
       */
       workspaceBusyReenqueues = /* @__PURE__ */ new Map();
+      workspaceBusyReenqueueTimers = /* @__PURE__ */ new Set();
       static WORKSPACE_BUSY_MAX_REENQUEUES = 10;
+      /*
+      FNXC:WorkspaceMergeDispatch 2026-08-05-23:56:
+      Workspace lease-contention retries are engine-owned lifecycle work, not detached callbacks.
+      Track only these busy re-enqueue timers so stop() can cancel them and tests can measure the
+      capped workspace ladder without confusing it with merge-body or maintenance timers.
+      */
+      scheduleWorkspaceBusyReenqueue(taskId, delayMs) {
+        const timer2 = setTimeout(() => {
+          this.workspaceBusyReenqueueTimers.delete(timer2);
+          if (!this.shuttingDown) this.internalEnqueueMerge(taskId);
+        }, delayMs);
+        this.workspaceBusyReenqueueTimers.add(timer2);
+      }
       /**
        * Pending manual merge resolvers — keyed by taskId.
        * When `onMerge` is called, the task is enqueued like auto-merge but a
@@ -261461,7 +261684,7 @@ var init_project_engine = __esm({
        *
        *  Readable (not private) so tests derive the cap from this single source of truth rather
        *  than hardcoding it — the FN-8004 bump broke two suites that had baked in the old `3`. */
-      static MAX_AUTO_MERGE_TRANSIENT_RETRIES = 5;
+      static MAX_AUTO_MERGE_TRANSIENT_RETRIES = MAX_AUTO_MERGE_TRANSIENT_RETRIES;
       static MERGE_REQUEST_RETRY_EXHAUSTED_AGE_MS = 30 * 60 * 1e3;
       /** Cap on outer in-review→in-progress bounces caused by deterministic
        *  verification failures during auto-merge. After this many failed merges
@@ -261935,6 +262158,10 @@ ${detail}`);
           clearInterval(this.mergeActiveReconcileTimer);
           this.mergeActiveReconcileTimer = null;
         }
+        for (const timer2 of this.workspaceBusyReenqueueTimers) {
+          clearTimeout(timer2);
+        }
+        this.workspaceBusyReenqueueTimers.clear();
         for (const [taskId, deferred] of this.capacityDeferredMerges) {
           clearTimeout(deferred.timer);
           for (const resolver of deferred.resolvers) {
@@ -263234,16 +263461,15 @@ ${detail}`);
        * older low-priority task would start before a later urgent one.
        */
       async allowInReviewMergeProcessing(task, settings, store = this.runtime.getTaskStore()) {
-        if (allowsAutoMergeProcessing(task, settings)) {
-          return true;
-        }
+        if (hasUserAutoMergeHold(task)) return false;
         const groupId = task.branchContext?.groupId?.trim();
         const branchGroup = groupId ? await store.getBranchGroup?.(groupId) : null;
-        if (!branchGroup || branchGroup.status !== "open" || !branchGroup.branchName.trim()) {
-          return false;
-        }
         const projectDefaultBranch = await resolveIntegrationBranch(this.config.workingDirectory, settings);
-        return isLiveSharedBranchGroupMemberIntegration(task, branchGroup, projectDefaultBranch);
+        if (isLiveSharedBranchGroupMemberIntegration(task, branchGroup, projectDefaultBranch)) {
+          return true;
+        }
+        if (task.autoMerge === false && groupId) return false;
+        return allowsAutoMergeProcessing(task, settings);
       }
       async emitLegacyAutoMergeStampAdvisory(store) {
         if (this.legacyAutoMergeStampAdvisoryEmitted) {
@@ -264196,9 +264422,7 @@ ${detail}`);
                   runtimeLog.log(
                     `Workspace land busy re-enqueue ${busyCount + 1}/${_ProjectEngine.WORKSPACE_BUSY_MAX_REENQUEUES} for ${taskId} in ${delayMs / 1e3}s (no mergeRetry consumed \u2014 pure lease contention)`
                   );
-                  setTimeout(() => {
-                    if (!this.shuttingDown) this.internalEnqueueMerge(taskId);
-                  }, delayMs);
+                  this.scheduleWorkspaceBusyReenqueue(taskId, delayMs);
                 } else {
                   this.workspaceBusyReenqueues.delete(taskId);
                   await store.updateTask(taskId, { status: "failed", error: errorMsg }).catch(() => void 0);
@@ -267690,7 +267914,7 @@ var init_task_lifecycle_lanes = __esm({
 });
 
 // ../dashboard/src/ai-session-diagnostics.ts
-import { randomUUID as randomUUID44 } from "node:crypto";
+import { randomUUID as randomUUID45 } from "node:crypto";
 function defaultSink(level, scope, message, context) {
   const log41 = createLogger(scope);
   try {
@@ -267733,7 +267957,7 @@ function emit(level, scope, message, context) {
   const fullContext = {
     ...context,
     _emittedAt: (/* @__PURE__ */ new Date()).toISOString(),
-    _diagnosticsId: randomUUID44()
+    _diagnosticsId: randomUUID45()
   };
   try {
     _sink(level, scope, message, fullContext);
@@ -268449,7 +268673,7 @@ var init_planning_board_tools = __esm({
 });
 
 // ../dashboard/src/planning.ts
-import { randomUUID as randomUUID45 } from "node:crypto";
+import { randomUUID as randomUUID46 } from "node:crypto";
 import { EventEmitter as EventEmitter34 } from "node:events";
 async function createPlanningRuntimeSession(options) {
   const { pluginRunner, runtimeHint, settings, ...runtimeOptions } = options ?? {};
@@ -268832,7 +269056,7 @@ async function createSession(ip, initialPlan, store, rootDir, promptOverrides, p
   if (!store) {
     throw new Error("store is required for AI-powered planning sessions");
   }
-  const sessionId = randomUUID45();
+  const sessionId = randomUUID46();
   const session = {
     id: sessionId,
     ip,
@@ -269402,7 +269626,7 @@ function normalizePlanningQuestion(input, userInput = "") {
   const type = source.type === "multi_select" || source.type === "single_select" ? source.type : "single_select";
   const question = typeof source.question === "string" && source.question.trim() ? source.question.trim() : fallback.question;
   const normalized = {
-    id: typeof source.id === "string" && source.id.trim() ? source.id : randomUUID45(),
+    id: typeof source.id === "string" && source.id.trim() ? source.id : randomUUID46(),
     type,
     question,
     ...typeof source.description === "string" && source.description.trim() ? { description: source.description.trim() } : {}
@@ -269428,7 +269652,7 @@ function normalizePlanningQuestion(input, userInput = "") {
 }
 function coerceQuestionResponse(response, session) {
   if (response.type === "question") return normalizePlanningQuestion(response.data, session.initialPlan);
-  return normalizePlanningQuestion({ id: randomUUID45(), type: "single_select", question: planningFallbackCopy(session.initialPlan).question }, session.initialPlan);
+  return normalizePlanningQuestion({ id: randomUUID46(), type: "single_select", question: planningFallbackCopy(session.initialPlan).question }, session.initialPlan);
 }
 async function validateSession(sessionId) {
   const session = await getSession(sessionId);
@@ -270176,7 +270400,7 @@ async function createTaskFromPlanSession(sessionId, store, options) {
       throw new InvalidSessionStateError("PLANNING_CREATED_TASK_MISSING");
     }
   }
-  const claimOwnerToken = randomUUID45();
+  const claimOwnerToken = randomUUID46();
   let claimed = await claimPlanningTaskCreation(sessionId, claimOwnerToken, (/* @__PURE__ */ new Date()).toISOString(), claimEpoch);
   if (!claimed) {
     session = await getDurablePlanningSession(sessionId).catch(() => void 0) ?? session;
@@ -282764,7 +282988,7 @@ var init_register_gitlab = __esm({
 
 // ../dashboard/src/terminal.ts
 import { spawn as spawn11 } from "node:child_process";
-import { randomUUID as randomUUID46 } from "node:crypto";
+import { randomUUID as randomUUID47 } from "node:crypto";
 import { EventEmitter as EventEmitter41 } from "node:events";
 function extractBaseCommand(command) {
   let trimmed = command.trim();
@@ -282924,7 +283148,7 @@ var init_terminal = __esm({
         if (!validation.valid) {
           return { sessionId: "", error: validation.error };
         }
-        const sessionId = randomUUID46();
+        const sessionId = randomUUID47();
         const childProcess5 = spawn11(command, [], {
           cwd,
           shell: true,
@@ -284396,7 +284620,7 @@ var init_src8 = __esm({
 
 // ../../plugins/fusion-plugin-openclaw-runtime/src/pi-module.ts
 import { spawn as spawn13 } from "node:child_process";
-import { randomUUID as randomUUID47 } from "node:crypto";
+import { randomUUID as randomUUID48 } from "node:crypto";
 import { readFile as readFile44 } from "node:fs/promises";
 function asString2(v2) {
   return typeof v2 === "string" && v2.trim() !== "" ? v2.trim() : void 0;
@@ -284456,7 +284680,7 @@ function extractStderrError(stderr, stdout) {
 }
 function createCliSession(opts) {
   return {
-    sessionId: randomUUID47(),
+    sessionId: randomUUID48(),
     agentId: opts.agentId ?? DEFAULT_AGENT_ID,
     systemPrompt: opts.systemPrompt,
     messages: [{ role: "developer", content: opts.systemPrompt }],
@@ -284651,7 +284875,7 @@ var init_mcp_config2 = __esm({
 });
 
 // ../../plugins/fusion-plugin-openclaw-runtime/src/runtime-adapter.ts
-import { randomUUID as randomUUID48 } from "node:crypto";
+import { randomUUID as randomUUID49 } from "node:crypto";
 var OpenClawRuntimeAdapter;
 var init_runtime_adapter2 = __esm({
   "../../plugins/fusion-plugin-openclaw-runtime/src/runtime-adapter.ts"() {
@@ -284674,8 +284898,8 @@ var init_runtime_adapter2 = __esm({
         let mcpProfile;
         let mcpConfigPath;
         if (toolDefs.length > 0) {
-          const bridge = writeOpenClawMcpBridgeFiles(toolDefs, randomUUID48());
-          mcpProfile = `fusion-${randomUUID48()}`;
+          const bridge = writeOpenClawMcpBridgeFiles(toolDefs, randomUUID49());
+          mcpProfile = `fusion-${randomUUID49()}`;
           mcpConfigPath = bridge.serverConfigPath;
           await configureOpenClawMcpServer({
             binaryPath: this.config.binaryPath,
@@ -287149,7 +287373,7 @@ import { existsSync as existsSync73, writeFileSync as writeFileSync15 } from "no
 import { tmpdir as tmpdir16 } from "node:os";
 import { dirname as dirname41, join as join93 } from "node:path";
 import { fileURLToPath as fileURLToPath9 } from "node:url";
-import { randomUUID as randomUUID49 } from "node:crypto";
+import { randomUUID as randomUUID50 } from "node:crypto";
 function toolsToMcpToolDefs2(tools) {
   if (!Array.isArray(tools)) return [];
   return tools.filter((tool) => tool && typeof tool.name === "string" && tool.name.trim().length > 0 && !BUILT_IN_TOOL_NAMES2.has(tool.name)).map((tool) => ({
@@ -287200,7 +287424,7 @@ async function startFusionToolBridge(tools) {
       byName.set(tool.name, tool);
     }
   }
-  const schemaPath = join93(tmpdir16(), `fusion-grok-mcp-schemas-${process.pid}-${randomUUID49()}.json`);
+  const schemaPath = join93(tmpdir16(), `fusion-grok-mcp-schemas-${process.pid}-${randomUUID50()}.json`);
   writeFileSync15(schemaPath, JSON.stringify(defs));
   const server = createServer2(async (req, res) => {
     if (req.method !== "POST" || req.url !== "/tool-call") {
@@ -287226,7 +287450,7 @@ async function startFusionToolBridge(tools) {
       return;
     }
     try {
-      const result2 = await tool.execute(`grok-mcp-${randomUUID49()}`, parsed.arguments ?? {}, void 0, void 0, void 0);
+      const result2 = await tool.execute(`grok-mcp-${randomUUID50()}`, parsed.arguments ?? {}, void 0, void 0, void 0);
       res.statusCode = 200;
       res.setHeader("content-type", "application/json");
       res.end(
@@ -289270,7 +289494,7 @@ import { existsSync as existsSync77, rmSync as rmSync13, writeFileSync as writeF
 import { tmpdir as tmpdir18 } from "node:os";
 import { dirname as dirname46, join as join98 } from "node:path";
 import { fileURLToPath as fileURLToPath13 } from "node:url";
-import { randomBytes as randomBytes7, randomUUID as randomUUID50 } from "node:crypto";
+import { randomBytes as randomBytes7, randomUUID as randomUUID51 } from "node:crypto";
 function toolsToMcpToolDefs3(tools) {
   if (!Array.isArray(tools)) return [];
   return tools.filter((tool) => tool && typeof tool.name === "string" && tool.name.trim().length > 0 && !BUILT_IN_TOOL_NAMES3.has(tool.name)).map((tool) => ({
@@ -289321,7 +289545,7 @@ async function startFusionToolBridge2(tools, options = {}) {
       byName.set(tool.name, tool);
     }
   }
-  const schemaPath = join98(tmpdir18(), `fusion-claude-mcp-schemas-${process.pid}-${randomUUID50()}.json`);
+  const schemaPath = join98(tmpdir18(), `fusion-claude-mcp-schemas-${process.pid}-${randomUUID51()}.json`);
   writeFileSync17(schemaPath, JSON.stringify(defs));
   const capabilityToken = randomBytes7(32).toString("base64url");
   const server = createServer3(async (req, res) => {
@@ -289386,7 +289610,7 @@ async function startFusionToolBridge2(tools, options = {}) {
       return;
     }
     try {
-      const result2 = await tool.execute(`claude-mcp-${randomUUID50()}`, parsed.arguments ?? {}, void 0, void 0, void 0);
+      const result2 = await tool.execute(`claude-mcp-${randomUUID51()}`, parsed.arguments ?? {}, void 0, void 0, void 0);
       res.statusCode = 200;
       res.setHeader("content-type", "application/json");
       res.end(
@@ -291463,7 +291687,7 @@ import { existsSync as existsSync79, unlinkSync as unlinkSync5, writeFileSync as
 import { tmpdir as tmpdir19 } from "node:os";
 import { dirname as dirname49, join as join101 } from "node:path";
 import { fileURLToPath as fileURLToPath15 } from "node:url";
-import { randomUUID as randomUUID51 } from "node:crypto";
+import { randomUUID as randomUUID52 } from "node:crypto";
 function toolsToMcpToolDefs4(tools) {
   if (!Array.isArray(tools)) return [];
   return tools.filter(
@@ -291520,7 +291744,7 @@ async function startFusionToolBridge3(tools) {
       byName.set(tool.name, tool);
     }
   }
-  const schemaPath = join101(tmpdir19(), `fusion-omp-mcp-schemas-${process.pid}-${randomUUID51()}.json`);
+  const schemaPath = join101(tmpdir19(), `fusion-omp-mcp-schemas-${process.pid}-${randomUUID52()}.json`);
   writeFileSync18(schemaPath, JSON.stringify(defs));
   const server = createServer4(async (req, res) => {
     if (req.method !== "POST" || req.url !== "/tool-call") {
@@ -291547,7 +291771,7 @@ async function startFusionToolBridge3(tools) {
     }
     try {
       const result2 = await tool.execute(
-        `omp-mcp-${randomUUID51()}`,
+        `omp-mcp-${randomUUID52()}`,
         parsed.arguments ?? {},
         void 0,
         void 0,
@@ -292440,7 +292664,7 @@ var init_paperclip_client = __esm({
 });
 
 // ../../plugins/fusion-plugin-paperclip-runtime/src/runtime-adapter.ts
-import { randomUUID as randomUUID52 } from "node:crypto";
+import { randomUUID as randomUUID53 } from "node:crypto";
 function sleep7(ms2) {
   return new Promise((resolve54) => setTimeout(resolve54, ms2));
 }
@@ -292569,7 +292793,7 @@ var init_runtime_adapter10 = __esm({
           apiKey: effectiveApiKey,
           agentId,
           companyId,
-          sessionId: randomUUID52(),
+          sessionId: randomUUID53(),
           systemPrompt: options.systemPrompt,
           cwd: options.cwd,
           mode: normalizeMode3(this.config.mode),
@@ -293247,7 +293471,7 @@ var init_types5 = __esm({
 });
 
 // ../dashboard/src/stt/model-manager.ts
-import { createHash as createHash28, randomUUID as randomUUID53 } from "node:crypto";
+import { createHash as createHash28, randomUUID as randomUUID54 } from "node:crypto";
 import { mkdir as mkdir31, open as open4, readFile as readFile46, readdir as readdir11, rename as rename10, rm as rm17, stat as stat12, writeFile as writeFile33 } from "node:fs/promises";
 import { join as join102, relative as relative19, resolve as resolve51, sep as sep15 } from "node:path";
 function parseSafeTarListing(listing) {
@@ -293328,8 +293552,8 @@ function createVoiceModelManager(options = {}) {
       if (!current(g2, signal)) return state;
       await mkdir31(cacheDir(), { recursive: true });
       if (!current(g2, signal)) return state;
-      partial2 = join102(cacheDir(), `.partial-${g2}-${randomUUID53()}`);
-      staging = join102(cacheDir(), `.staging-${g2}-${randomUUID53()}`);
+      partial2 = join102(cacheDir(), `.partial-${g2}-${randomUUID54()}`);
+      staging = join102(cacheDir(), `.staging-${g2}-${randomUUID54()}`);
       const file = await open4(partial2, "wx");
       try {
         const response = await (options.fetch ?? globalThis.fetch)(asset.url, { signal });
@@ -293377,7 +293601,7 @@ function createVoiceModelManager(options = {}) {
       await writeFile33(join102(staging, "manifest.json"), JSON.stringify({ filename: asset.filename, sha256: asset.sha256, installedAt: (/* @__PURE__ */ new Date()).toISOString(), expectedFiles: asset.expectedFiles }));
       await options.beforePromote?.(signal);
       if (!current(g2, signal)) return state;
-      const backup = join102(cacheDir(), `.backup-${g2}-${randomUUID53()}`);
+      const backup = join102(cacheDir(), `.backup-${g2}-${randomUUID54()}`);
       let movedPrevious = false;
       try {
         try {
@@ -293598,7 +293822,7 @@ var init_parakeet_service = __esm({
 });
 
 // ../dashboard/src/routes/register-voice-routes.ts
-import { randomUUID as randomUUID54 } from "node:crypto";
+import { randomUUID as randomUUID55 } from "node:crypto";
 import express from "express";
 async function settingsFor(ctx, req) {
   const scoped = await ctx.getScopedStore(req);
@@ -293677,7 +293901,7 @@ function createRegisterVoiceRoutes(deps = {}) {
           recognizer.close();
           return error2(res, 409, { error: "unavailable", reason: "model-removed" });
         }
-        const id = randomUUID54();
+        const id = randomUUID55();
         const now = Date.now();
         sessions6.set(id, { projectId, recognizer, next: 0, bytes: 0, last: now, expires: now + 3e5 });
         res.status(201).json({ sessionId: id, expiresAt: new Date(now + 3e5).toISOString(), modelId: model.id, language: language.language });
@@ -297003,14 +297227,14 @@ init_src2();
 init_gh_cli();
 init_src5();
 init_src15();
-import { randomUUID as randomUUID55 } from "node:crypto";
+import { randomUUID as randomUUID56 } from "node:crypto";
 import { Type as Type9 } from "typebox";
 import { StringEnum } from "@earendil-works/pi-ai";
 import { resolve as resolve53, relative as relative20, isAbsolute as isAbsolute26, sep as sep16, basename as basename29, extname as extname4, join as join105 } from "node:path";
 import { readFile as readFile47 } from "node:fs/promises";
 import { existsSync as existsSync82 } from "node:fs";
 import { spawn as spawn23 } from "node:child_process";
-import { AsyncLocalStorage } from "node:async_hooks";
+import { AsyncLocalStorage as AsyncLocalStorage2 } from "node:async_hooks";
 function truncateAgentDiagnosticText(value, maxChars) {
   return value.length > maxChars ? `${value.slice(0, maxChars)}\u2026` : value;
 }
@@ -297115,7 +297339,7 @@ var storeCache2 = extensionStoreState.cache;
 var storeBootInflight = extensionStoreState.bootInflight;
 var storeBootFailureCooldown = extensionStoreState.bootFailureCooldown;
 var BOOT_FAILURE_COOLDOWN_MS = 5e3;
-var extensionToolSignal = new AsyncLocalStorage();
+var extensionToolSignal = new AsyncLocalStorage2();
 var EXTENSION_STORE_BOOT_TIMEOUT_MS = 3e4;
 var EXTENSION_TOOL_TIMEOUT_MS = 6e4;
 var SKILLS_INSTALL_TIMEOUT_MS = 3e5;
@@ -297417,6 +297641,28 @@ async function isEphemeralCallerAgent(cwd, callerAgentId) {
   }
 }
 var AMBIGUOUS_AGENT_PRINCIPAL_ID = "unknown-agent";
+function resolveSecretAccessPrincipal(ctx) {
+  const principal = resolveExtensionCallerPrincipal(ctx);
+  if (principal.kind === "ambiguous") return { kind: "ambiguous" };
+  if (principal.kind === "operator") {
+    return {
+      kind: "resolved",
+      actor: { actorId: "user", actorType: "user", actorName: "CLI User" },
+      agentId: null
+    };
+  }
+  return {
+    kind: "resolved",
+    actor: {
+      actorId: principal.identity.agentId,
+      actorType: "agent",
+      actorName: principal.identity.agentName ?? principal.identity.agentId
+    },
+    agentId: principal.identity.agentId,
+    ...principal.identity.agentName ? { agentName: principal.identity.agentName } : {},
+    ...principal.identity.taskId ? { taskId: principal.identity.taskId } : {}
+  };
+}
 function resolveExtensionCallerPrincipal(ctx) {
   const explicitAgentId = typeof ctx.agentId === "string" && ctx.agentId.trim().length > 0 ? ctx.agentId.trim() : void 0;
   if (explicitAgentId) {
@@ -297976,7 +298222,7 @@ function kbExtension(pi2) {
           const layer = store.getAsyncLayer();
           const messageStore = layer ? new MessageStore(null, { asyncLayer: layer }) : new MessageStore(store.getDatabase());
           const title = params.description.split(/\r?\n/, 1)[0]?.trim().slice(0, 80) || "Follow-up task";
-          await messageStore.sendMessage({ fromId: fnCtx.agentId ?? "ephemeral-worker", fromType: "agent", toId: DASHBOARD_USER_ID, toType: "user", type: "agent-to-user", content: `Task proposal awaiting validation: ${title}`, metadata: { kind: "task-proposal", proposalStatus: "pending", proposalIdempotencyKey: randomUUID55(), proposedTask: { title, description: params.description, priority: params.priority, workflowId: params.workflow_id, dependencies: params.depends } } });
+          await messageStore.sendMessage({ fromId: fnCtx.agentId ?? "ephemeral-worker", fromType: "agent", toId: DASHBOARD_USER_ID, toType: "user", type: "agent-to-user", content: `Task proposal awaiting validation: ${title}`, metadata: { kind: "task-proposal", proposalStatus: "pending", proposalIdempotencyKey: randomUUID56(), proposedTask: { title, description: params.description, priority: params.priority, workflowId: params.workflow_id, dependencies: params.depends } } });
           return { content: [{ type: "text", text: "Task proposal submitted to the operator for validation; no task was created." }], details: { proposed: true } };
         }
       }
@@ -299226,6 +299472,20 @@ Planning session completed. Task ${taskId} is now in planning and will be auto-p
     }),
     async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
       const fnCtx = ctx;
+      const secretPrincipal = resolveSecretAccessPrincipal(fnCtx);
+      if (secretPrincipal.kind === "ambiguous") {
+        return {
+          content: [{ type: "text", text: "Secret access was not requested because the calling agent identity is ambiguous. End one concurrent session and retry." }],
+          isError: true,
+          details: { error: "ambiguous-caller-identity", key: params.key, scope: params.scope ?? null }
+        };
+      }
+      const effectiveCtx = {
+        ...secretPrincipal.agentId ? { agentId: secretPrincipal.agentId } : {},
+        ...secretPrincipal.agentName ? { agentName: secretPrincipal.agentName } : {},
+        ...typeof fnCtx.taskId === "string" ? { taskId: fnCtx.taskId } : secretPrincipal.taskId ? { taskId: secretPrincipal.taskId } : {},
+        ...typeof fnCtx.runId === "string" ? { runId: fnCtx.runId } : {}
+      };
       const store = await getStore(ctx.cwd);
       const secretsStore = await store.getSecretsStore();
       const scopes = params.scope ? [params.scope] : ["project", "global"];
@@ -299248,36 +299508,32 @@ Planning session completed. Task ${taskId} is now in planning and will be auto-p
         settings: { secretsAccessPolicy: globalSettings.secretsAccessPolicy }
       });
       if (decision.policy === "deny") {
-        emitSecretAudit(store, fnCtx, "secret:approval-denied", `${resolvedScope}:${params.key}`);
+        emitSecretAudit(store, effectiveCtx, "secret:approval-denied", `${resolvedScope}:${params.key}`);
         return { content: [{ type: "text", text: "Secret access denied by policy." }], details: { error: "denied", key: params.key, scope: resolvedScope, policySource: decision.source } };
       }
       if (decision.policy === "prompt") {
         const cliLayer = requireProjectLayer(store, "CLI secret approval store");
         const approvalStore = new ApprovalRequestStore(null, { asyncLayer: cliLayer });
-        const dedupeKey = `secret-read:${resolvedScope}:${params.key}:${fnCtx.agentId ?? "unknown"}`;
-        const requesterActorId = fnCtx.agentId ?? "user";
-        const requesterSnapshot = fnCtx.agentId ? {
-          actorId: requesterActorId,
-          actorType: "agent",
-          actorName: fnCtx.agentName ?? fnCtx.agentId
-        } : { actorId: "user", actorType: "user", actorName: "CLI User" };
-        const existing = await findLatestApprovalRequestByDedupeKey(approvalStore, { requesterActorId, ...fnCtx.taskId ? { taskId: fnCtx.taskId } : {}, dedupeKey });
+        const requesterSnapshot = secretPrincipal.actor;
+        const requesterActorId = requesterSnapshot.actorId;
+        const dedupeKey = `secret-read:${resolvedScope}:${params.key}:${requesterActorId}`;
+        const existing = await findLatestApprovalRequestByDedupeKey(approvalStore, { requesterActorId, ...effectiveCtx.taskId ? { taskId: effectiveCtx.taskId } : {}, dedupeKey });
         if (existing?.status === "pending") {
-          emitSecretAudit(store, fnCtx, "secret:approval-requested", `${resolvedScope}:${params.key}`);
+          emitSecretAudit(store, effectiveCtx, "secret:approval-requested", `${resolvedScope}:${params.key}`);
           return {
             content: [{ type: "text", text: `Secret access approval request ${existing.id} is still pending. Approve via POST /api/approvals/:id/decision.` }],
             details: { outcome: "pending_approval", approvalRequestId: existing.id, key: params.key, scope: resolvedScope }
           };
         }
         if (existing?.status === "denied") {
-          emitSecretAudit(store, fnCtx, "secret:approval-denied", `${resolvedScope}:${params.key}`);
+          emitSecretAudit(store, effectiveCtx, "secret:approval-denied", `${resolvedScope}:${params.key}`);
           return {
             content: [{ type: "text", text: `Secret access request ${existing.id} was denied by the operator. Do not retry without operator direction.` }],
             details: { outcome: "denied", approvalRequestId: existing.id, key: params.key, scope: resolvedScope }
           };
         }
         if (existing?.status === "approved") {
-          const revealedAfterApproval = await secretsStore.revealSecret(record2.id, resolvedScope, { agentId: fnCtx.agentId ?? null });
+          const revealedAfterApproval = await secretsStore.revealSecret(record2.id, resolvedScope, { agentId: secretPrincipal.agentId });
           await approvalStore.markCompleted(existing.id, {
             actor: requesterSnapshot,
             note: "Secret revealed after approval",
@@ -299285,7 +299541,7 @@ Planning session completed. Task ${taskId} is now in planning and will be auto-p
             // get the same expectedRequesterActorId enforcement as the gate path.
             expectedRequesterActorId: requesterActorId
           });
-          emitSecretAudit(store, fnCtx, "secret:read", `${resolvedScope}:${params.key}`, { key: params.key, scope: resolvedScope, approvalRequestId: existing.id });
+          emitSecretAudit(store, effectiveCtx, "secret:read", `${resolvedScope}:${params.key}`, { key: params.key, scope: resolvedScope, approvalRequestId: existing.id });
           return {
             content: [{ type: "text", text: `Loaded secret '${params.key}' from ${resolvedScope} scope (approval ${existing.id} consumed).` }],
             details: { key: params.key, value: revealedAfterApproval.plaintextValue, scope: resolvedScope, approvalRequestId: existing.id }
@@ -299301,17 +299557,17 @@ Planning session completed. Task ${taskId} is now in planning and will be auto-p
             resourceId: record2.id,
             context: { approvalDedupeKey: dedupeKey, key: params.key, scope: resolvedScope }
           },
-          ...fnCtx.runId ? { runId: fnCtx.runId } : {},
-          ...fnCtx.taskId ? { taskId: fnCtx.taskId } : {}
+          ...effectiveCtx.runId ? { runId: effectiveCtx.runId } : {},
+          ...effectiveCtx.taskId ? { taskId: effectiveCtx.taskId } : {}
         });
-        emitSecretAudit(store, fnCtx, "secret:approval-requested", `${resolvedScope}:${params.key}`);
+        emitSecretAudit(store, effectiveCtx, "secret:approval-requested", `${resolvedScope}:${params.key}`);
         return {
           content: [{ type: "text", text: `Secret access requires approval. Request ${request2.id} is pending. Approve via POST /api/approvals/:id/decision.` }],
           details: { outcome: "pending_approval", approvalRequestId: request2.id, key: params.key, scope: resolvedScope }
         };
       }
-      const revealed = await secretsStore.revealSecret(record2.id, resolvedScope, { agentId: fnCtx.agentId ?? null });
-      emitSecretAudit(store, fnCtx, "secret:read", `${resolvedScope}:${params.key}`, { key: params.key, scope: resolvedScope });
+      const revealed = await secretsStore.revealSecret(record2.id, resolvedScope, { agentId: secretPrincipal.agentId });
+      emitSecretAudit(store, effectiveCtx, "secret:read", `${resolvedScope}:${params.key}`, { key: params.key, scope: resolvedScope });
       return {
         content: [{ type: "text", text: `Loaded secret '${params.key}' from ${resolvedScope} scope.` }],
         details: { key: params.key, value: revealed.plaintextValue, scope: resolvedScope }