@productbrain/mcp 0.0.1-beta.6286 → 0.0.1-beta.6294

package/dist/{chunk-XJRVTFDC.js → chunk-73WYXKBT.js}

@@ -17,7 +17,6 @@ import {
   kernelMutation,
   kernelQuery,
   recordSessionActivity,
-  refreshWorkspaceGovernanceMode,
   requireActiveSession,
   requireVendorTriageAccess,
   requireWriteAccess,
@@ -42,7 +41,7 @@ import {
   trackSessionCaptureRate,
   trackWriteBackHintServed,
   trackZeroCaptureAuditFired
-} from "./chunk-OT4PU7IF.js";
+} from "./chunk-Q2BQSINV.js";
 
 // src/server.ts
 import { McpServer as McpServer2 } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -815,65 +814,6 @@ var ScopedCache = class {
 };
 var COLLECTION_CACHE_TTL_MS = 6e4;
 
-// src/tools/coherency-refusal.ts
-function asDescription(value) {
-  if (typeof value !== "string") return void 0;
-  const trimmed = value.trim();
-  return trimmed.length > 0 ? trimmed : void 0;
-}
-var GENERIC_RECOVERY_LINE = "Respond and retry: link the entry to the strategic spine, pass `coherencyAcknowledgement`, or supply a typed `steeringOverrideReason`.";
-function coherencyRefusedResult(toolName, entryId, refusal, options) {
-  const headingLevel = options?.headingLevel ?? 1;
-  const heading = "#".repeat(headingLevel);
-  const lines = [
-    `${heading} Write refused by the coherency gate`,
-    "",
-    refusal.reason ?? `The coherency gate refused this write on ${entryId}.`
-  ];
-  if (refusal.subject?.name) {
-    lines.push(`On: ${refusal.subject.name}`);
-  }
-  if (refusal.mode) {
-    lines.push("", `Mode: ${refusal.mode}`);
-  }
-  const routes = refusal.routes;
-  const routeKeys = routes ? Object.keys(routes) : [];
-  if (routeKeys.length > 0) {
-    lines.push("", `${"#".repeat(headingLevel + 1)} Recovery routes`);
-    const beforeRoutes = lines.length;
-    if (routes && typeof routes.link === "object" && routes.link !== null) {
-      const link = routes.link;
-      lines.push(`- **link:** ${asDescription(link.description) ?? "Link the entry to the strategic spine."}`);
-    }
-    if (routes && typeof routes.acknowledge === "object" && routes.acknowledge !== null) {
-      const ack = routes.acknowledge;
-      const responses = ack.responses?.join(" | ") ?? "linked | accepted-fix | diverged";
-      lines.push(
-        `- **acknowledge** (\`${ack.param ?? "coherencyAcknowledgement"}\`): ${responses}${ack.minReasonChars ? ` (\u2265${ack.minReasonChars} chars for diverged)` : ""}. ${asDescription(ack.description) ?? ""}`.trim()
-      );
-    }
-    if (routes && typeof routes.override === "object" && routes.override !== null) {
-      const ov = routes.override;
-      lines.push(
-        `- **override** (\`${ov.param ?? "steeringOverrideReason"}\`): \u2265${ov.minChars ?? 12} chars. ${asDescription(ov.description) ?? ""}`.trim()
-      );
-    }
-    if (lines.length === beforeRoutes) {
-      lines.push(GENERIC_RECOVERY_LINE);
-    }
-  } else {
-    lines.push("", GENERIC_RECOVERY_LINE);
-  }
-  return failureResult(
-    lines.join("\n"),
-    "COHERENCY_REFUSED",
-    `Coherency gate refused ${toolName} on ${entryId}.`,
-    "Use one of the refusal's routes (link / coherencyAcknowledgement / steeringOverrideReason), then retry.",
-    void 0,
-    { entryId, refusal }
-  );
-}
-
 // src/lib/resolveCollection.ts
 async function resolveCollection(params) {
   const { name, description, typeHint, allowReviewRouting } = params;
@@ -1116,10 +1056,10 @@ var batchCaptureRealOutputSchema = z4.object({
     entryId: z4.string(),
     collection: z4.string(),
     name: z4.string(),
-    // "draft_on_failure" is the status emitted when an auto-commit is REFUSED/failed (the entry
-    // stays a draft) — the same path that now also emits coherencyRefusal. The strict schema must
-    // accept it or a refused response fails to validate on `status` (codex review, completing the
-    // coherencyRefusal contract fix).
+    // WP-751 M3a-1b (TASK-189, DEC-1700): "proposed" and "draft_on_failure" are never emitted
+    // by `buildBatchEntryFailure` or the create loop anymore — both were statuses the deleted
+    // client-side auto-commit attempt could produce. Left in the enum (not narrowed) since this
+    // is a shared output-contract schema; narrowing it is a separate, independently-scoped change.
     status: z4.enum(["draft", "committed", "proposed", "draft_on_failure", "draft_spine_floor"]),
     spineFloor: z4.object({ relationType: z4.string(), parentKind: z4.string(), exempt: z4.boolean() }).optional(),
     classifiedBy: z4.enum(["llm", "heuristic", "explicit"]).optional(),
@@ -1131,14 +1071,11 @@ var batchCaptureRealOutputSchema = z4.object({
       rejected: z4.array(z4.string()).optional()
     }).optional(),
     // TEN-2365: capture-time authority-domain proposal slug (PENDING ratification), when filed.
-    domain: z4.string().optional(),
-    // WP-465 surface parity: structured coherency refusal for a gate-refused auto-commit (batch
-    // shape). Loose record — same single-source rationale as captureSuccessOutputSchema.
-    coherencyRefusal: z4.record(z4.unknown()).optional(),
-    // WP-485 Slice 2b round 4 (Codex P2, FEAT-1370): per-entry contradiction advisory, emitted
-    // into structuredContent but previously undeclared here. Not `.strict()` so this one was
-    // silently stripped rather than rejected — declared anyway so batch consumers actually see it.
-    contradictionAdvisory: z4.record(z4.unknown()).optional()
+    domain: z4.string().optional()
+    // WP-751 M3a-1b (TASK-189, DEC-1700): `coherencyRefusal` and `contradictionAdvisory` are
+    // deleted — both were only ever populated by the client-side auto-commit attempt
+    // (smart-capture.ts), deleted entirely. Role is the only governance mode a reader honors
+    // now, and role never attempted a client-side commit.
   })),
   total: z4.number(),
   failed: z4.number(),
@@ -1828,7 +1765,6 @@ var captureSchema = z6.object({
     to: z6.string().describe("Target entry ID (e.g. '<PREFIX>-<n>')"),
     type: z6.string().describe("Relation type (e.g. 'governs', 'related_to', 'informs')")
   })).optional().describe("Relations to create after capture. Skips auto-link discovery when provided."),
-  autoCommit: z6.boolean().optional().describe("If true, commits the entry immediately after capture + linking. If omitted, Open mode workspaces auto-commit by default and consensus/role modes stay draft-first."),
   sourceRef: z6.string().optional().describe("URI or path of the source document backing this entry (e.g. 'meeting-2026-03-28.md', 'import://batch-5'). Stored as top-level entry field, not in data."),
   sourceExcerpt: z6.string().optional().describe("Verbatim excerpt from the source that backs this entry's claims. Stored as top-level entry field, not in data."),
   // WP-316 S3: Preview gate — dry-run mode. Returns what would happen, no DB writes.
@@ -1863,9 +1799,6 @@ var batchCaptureSchema = z6.object({
     // ownerTeamEntryId deleted — team derives from the role's own placement.
     ownerRoleEntryId: z6.string().max(200).optional().describe("Owning role entry ID/ref (rung 2); its home team is derived automatically.")
   })).min(1).max(50).describe("Array of entries to capture"),
-  autoCommit: z6.boolean().optional().describe(
-    "If true, commits created entries immediately after linking. If omitted, Open mode workspaces commit by default and consensus/role modes stay draft-first."
-  ),
   // WP-316 S3: Preview gate — dry-run mode. Returns what would happen, no DB writes.
   preview: z6.boolean().optional().describe("If true, validates all captures without writing. Returns what would happen. Default false.")
 });
@@ -2149,14 +2082,10 @@ var captureSuccessOutputSchema = z6.object({
   }).optional(),
   // TASK-160: the server's birth-floor verdict must survive the strict output contract.
   spineFloor: z6.object({ relationType: z6.string(), parentKind: z6.string(), exempt: z6.boolean() }).optional(),
-  // WP-465 surface parity: structured coherency refusal when an auto-commit is gate-refused.
-  // Loose record — the CoherencyRefusal contract is authored in
-  // convex/lib/gates/coherencyControls.ts; re-declaring it here would be a second SSOT.
-  coherencyRefusal: z6.record(z6.unknown()).optional(),
-  // WP-485 Slice 2b round 4 (Codex P2, FEAT-1370): the server's contradiction advisory is emitted
-  // into structuredContent but was missing here — since this schema is `.strict()`, a real capture
-  // response carrying it failed validation with `unrecognized_keys`. Loose record, same rationale as coherencyRefusal above.
-  contradictionAdvisory: z6.record(z6.unknown()).optional(),
+  // WP-751 M3a-1b (TASK-189, DEC-1700): `coherencyRefusal` and `contradictionAdvisory` are
+  // deleted — both were only ever populated by the client-side auto-commit attempt (surfacing
+  // its own gate refusal, or the commit response's contradiction advisory), and that attempt is
+  // deleted entirely. Neither field is set anywhere in this file anymore.
   fallbackApplied: z6.boolean().optional()
   // round 2 (PRRT_kwDORS4ils6cTNbF): classifier field above still names the original (pre-substitution) prediction
 }).strict();
@@ -2168,10 +2097,6 @@ var captureOutputSchema = z6.union([
   captureSinglePreviewOutputSchema,
   captureClassifierOnlyOutputSchema
 ]);
-function shouldAutoCommitCapture(autoCommit, governanceMode) {
-  if (governanceMode !== "open") return false;
-  return autoCommit === true || autoCommit === void 0;
-}
 function buildDataFromFields(fields, descriptionField, descriptionValue) {
   const data = {};
   for (const field of fields) {
@@ -2199,7 +2124,6 @@ var captureCompoundSchema = z6.object({
     to: z6.string().max(200).describe("Target entry ID (e.g. '<PREFIX>-<n>')"),
     type: z6.string().max(200).describe("Relation type (e.g. 'governs', 'related_to', 'informs')")
   })).max(20).optional().describe("For 'capture': relations to create after capture. Skips auto-link discovery when provided."),
-  autoCommit: z6.boolean().optional().describe("For 'capture'/'batch': if true, commits immediately after capture + linking."),
   sourceRef: z6.string().max(2e3).optional().describe("For 'capture': URI or path of the source document backing this entry."),
   sourceExcerpt: z6.string().max(5e3).optional().describe("For 'capture': verbatim excerpt from the source backing this entry's claims."),
   preview: z6.boolean().optional().describe("For 'capture'/'batch': if true, validates without writing."),
@@ -2215,7 +2139,6 @@ var captureSingleVariant = captureSchema.extend({ action: z6.literal("capture")
 var captureBatchVariant = z6.object({
   action: z6.literal("batch"),
   items: z6.array(captureItemSchema).min(1).max(50),
-  autoCommit: z6.boolean().optional(),
   preview: z6.boolean().optional()
 });
 var captureActionUnion = z6.discriminatedUnion("action", [
@@ -2224,21 +2147,21 @@ var captureActionUnion = z6.discriminatedUnion("action", [
 ]);
 var CAPTURE_ACTION_SPECS = {
   capture: {
-    params: ["collection", "name", "description", "context", "entryId", "canonicalKey", "data", "links", "autoCommit", "sourceRef", "sourceExcerpt", "preview", "suggestOnly", "format"],
+    params: ["collection", "name", "description", "context", "entryId", "canonicalKey", "data", "links", "sourceRef", "sourceExcerpt", "preview", "suggestOnly", "format"],
     description: "name and description are required."
   },
-  batch: { params: ["items", "autoCommit", "preview"], description: "items (1-50, each may carry inline relations) is required." }
+  batch: { params: ["items", "preview"], description: "items (1-50, each may carry inline relations) is required." }
 };
 function registerSmartCaptureTools(server) {
   const captureHandlers = {
     capture: (data) => handleCapture(server, data),
-    batch: (data) => handleBatchCapture(server, { entries: data.items, autoCommit: data.autoCommit, preview: data.preview })
+    batch: (data) => handleBatchCapture(server, { entries: data.items, preview: data.preview })
   };
   server.registerTool(
     "capture",
     {
       title: "Capture",
-      description: "The tool for creating knowledge entries \u2014 single or batch. Two actions:\n\n- **capture**: Create a single entry. Creates an entry, auto-links related entries, and returns a quality scorecard \u2014 all in one call. Provide a name and description; `collection` is optional \u2014 the classifier auto-routes when omitted.\n- **batch**: Create multiple entries in one call via `items[]` (absorbs batch-capture). Each item may carry inline `relations` created right after that entry is captured.\n\n**action=capture details:**\nSupported collections with smart profiles: tensions, business-rules, glossary, decisions, features, audiences, strategy, standards, chains, insights, principles.\nAll other collections get an ENT-{random} ID and sensible defaults.\n\n**Explicit data:** When you know the schema, pass `data: { field: value }` to set fields directly. Top-level `name` and `description` always win for those fields. `data` wins over inference for all other fields.\n\n**Compound capture:** Pass `links` to create relations in the same call (skips auto-link discovery). In Open mode, entries commit by default when `autoCommit` is omitted. Pass `autoCommit: true` to promote the entry from draft to SSOT immediately after linking when you want to be explicit. Governed collections (glossary, business-rules, principles, standards, strategy, features, architecture) will warn but still commit \u2014 use only when you're certain. An objective, key result or work package is always born draft first \u2014 `autoCommit: true` has no effect on that first birth even with a complete spine; commit it explicitly afterward.\n\nEntries are created as `draft` first, then may publish immediately depending on governance and `autoCommit`. Use `entries action=update` for post-creation adjustments.",
+      description: "The tool for creating knowledge entries \u2014 single or batch. Two actions:\n\n- **capture**: Create a single entry. Creates an entry, auto-links related entries, and returns a quality scorecard \u2014 all in one call. Provide a name and description; `collection` is optional \u2014 the classifier auto-routes when omitted.\n- **batch**: Create multiple entries in one call via `items[]` (absorbs batch-capture). Each item may carry inline `relations` created right after that entry is captured.\n\n**action=capture details:**\nSupported collections with smart profiles: tensions, business-rules, glossary, decisions, features, audiences, strategy, standards, chains, insights, principles.\nAll other collections get an ENT-{random} ID and sensible defaults.\n\n**Explicit data:** When you know the schema, pass `data: { field: value }` to set fields directly. Top-level `name` and `description` always win for those fields. `data` wins over inference for all other fields.\n\n**Compound capture:** Pass `links` to create relations in the same call (skips auto-link discovery). Entries are always created as `draft` \u2014 use `entries action=commit` to promote when ready.",
       inputSchema: captureCompoundSchema,
       annotations: { readOnlyHint: false, destructiveHint: false, openWorldHint: false }
     },
@@ -2253,7 +2176,7 @@ function registerSmartCaptureTools(server) {
     })
   );
 }
-async function handleCapture(server, { collection, name, description, context, entryId, canonicalKey, data: userData, links, autoCommit, sourceRef, sourceExcerpt, preview, suggestOnly, format, ownerRoleEntryId }) {
+async function handleCapture(server, { collection, name, description, context, entryId, canonicalKey, data: userData, links, sourceRef, sourceExcerpt, preview, suggestOnly, format, ownerRoleEntryId }) {
   requireWriteAccess();
   const timingStart = Date.now();
   const wsCtx = await getWorkspaceContext();
@@ -2403,7 +2326,6 @@ ${groundingReport.governance.map((g) => `- **${g.entryId}** ${g.name} [${g.colle
             canonicalKey,
             data: userData,
             links,
-            autoCommit,
             sourceRef,
             sourceExcerpt,
             ownerRoleEntryId
@@ -2553,7 +2475,6 @@ ${groundingReport.governance.map((g) => `- **${g.entryId}** ${g.name} [${g.colle
           canonicalKey,
           data: userData,
           links,
-          autoCommit,
           sourceRef,
           sourceExcerpt,
           ownerRoleEntryId
@@ -2632,7 +2553,6 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
   const linksSuggested = [];
   const userLinkResults = [];
   const advisedLinks = [];
-  const conflictCandidates = groundingReport.duplicates.filter((candidate) => candidate.matchType === "possible_duplicate" && candidate.entryId !== finalEntryId).slice(0, 3).map((candidate) => ({ entryId: candidate.entryId, name: candidate.name, collection: candidate.collectionSlug }));
   const pendingRelations = [];
   const skipAutoDiscovery = links && links.length > 0;
   const searchQuery = extractSearchTerms(name, description);
@@ -2829,12 +2749,7 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
     } catch {
     }
   }
-  const freshGovernanceMode = await refreshWorkspaceGovernanceMode();
-  const shouldAutoCommit = !spineFloor && shouldAutoCommitCapture(autoCommit, freshGovernanceMode);
   let finalStatus = "draft";
-  let commitError = null;
-  let commitRefusal = null;
-  let contradictionAdvisory = null;
   if (wasAutoCommittedServerSide) {
     finalStatus = "committed";
     await recordSessionActivity({ entryModified: internalId });
@@ -2844,38 +2759,6 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
       commit_method: "auto",
       surface: "mcp_capture"
     });
-  } else if (shouldAutoCommit && finalEntryId) {
-    try {
-      const semanticConflicts = await discoverSemanticConflicts(name, description, resolvedCollection);
-      const commitResult = await kernelMutation("chain.commitEntry", {
-        entryId: finalEntryId,
-        author: agentId ? `agent:${agentId}` : void 0,
-        sessionId: agentId ?? void 0,
-        ...semanticConflicts.length > 0 ? { conflicts: semanticConflicts } : {}
-      });
-      const refusalReason = coherencyRefusalReason(commitResult);
-      if (refusalReason) {
-        commitError = `coherency gate refused this accept \u2014 ${refusalReason}`;
-        commitRefusal = commitResultField(commitResult, "refusal") ?? null;
-        finalStatus = "draft_on_failure";
-      } else {
-        finalStatus = commitResultField(commitResult, "status") === "proposal_created" ? "proposed" : "committed";
-        contradictionAdvisory = commitResultField(commitResult, "contradictionAdvisory") ?? null;
-        if (finalStatus === "committed") {
-          await recordSessionActivity({ entryModified: internalId });
-          trackChainEntryCommitted(wsCtx.workspaceId, {
-            entry_id: finalEntryId,
-            collection: resolvedCollection,
-            commit_method: "auto",
-            surface: "mcp_capture"
-          });
-        }
-      }
-    } catch (e) {
-      commitError = e instanceof Error ? e.message : "unknown error";
-      finalStatus = "draft_on_failure";
-      await recordCommitFailure({ entryId: finalEntryId, error: e, sessionId: agentId, server });
-    }
   }
   const lines = [
     `# Captured: ${finalEntryId || name}`,
@@ -2928,54 +2811,18 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
     for (const r of userLinkResults) lines.push(`- ${r.label}`);
   }
   if (finalStatus === "committed") {
-    const wasAutoCommitted = autoCommit === void 0 && freshGovernanceMode === "open";
     lines.push("");
     lines.push(`## Accepted: ${finalEntryId}`);
-    if (wasAutoCommitted) {
-      lines.push(`**${name}** added to your knowledge base.`);
-    } else {
-      lines.push(`**${name}** promoted to SSOT on the Chain.`);
-    }
+    lines.push(`**${name}** added to your knowledge base.`);
     if (await isGoverned(resolvedCollection)) {
       lines.push(`_Note: \`${resolvedCollection}\` is a governed collection \u2014 ensure this entry has been reviewed._`);
     }
-    if (contradictionAdvisory) {
-      lines.push("");
-      lines.push("## Contradiction advisory");
-      lines.push(contradictionAdvisory.reason);
-      for (const c of contradictionAdvisory.conflictDetails) {
-        lines.push(`- \`${c.chainEntryId}\` ${c.chainEntryName}: ${c.explanation}`);
-      }
-    }
-  } else if (finalStatus === "proposed") {
-    lines.push("");
-    lines.push(`## Proposal created: ${finalEntryId}`);
-    lines.push(`**${name}** requires consent before it can be accepted, so a proposal was created instead of accepting directly.`);
-  } else if (commitError) {
-    lines.push("");
-    lines.push("## Accept failed");
-    if (commitRefusal) {
-      const refusalText = coherencyRefusedResult("capture", finalEntryId, commitRefusal, { headingLevel: 3 }).content[0].text;
-      lines.push(...refusalText.split("\n"));
-      lines.push(`Entry saved as draft \u2014 use \`entries action=commit entryId="${finalEntryId}"\` to promote when ready.`);
-    } else {
-      lines.push(`Error: ${commitError}. Entry saved as draft \u2014 use \`entries action=commit entryId="${finalEntryId}"\` to promote when ready.`);
-    }
   } else if (spineFloor) {
     lines.push("");
     lines.push("## Saved as draft");
     lines.push(`**${name}** ${spineFloor.exempt ? SPINE_FLOOR_EXEMPT_SENTENCE : SPINE_FLOOR_SENTENCE}`);
     lines.push(`Next: \`entries action=commit entryId="${finalEntryId}"\`.`);
   }
-  if (shouldAutoCommit && conflictCandidates.length > 0) {
-    lines.push("");
-    lines.push("## Potential Overlaps");
-    lines.push(`Auto-accepted with advisory: ${conflictCandidates.length} similar ${conflictCandidates.length === 1 ? "entry" : "entries"} found.`);
-    for (const c of conflictCandidates) {
-      lines.push(`- \`${c.entryId}\`: ${c.name} [${c.collection}]`);
-    }
-    lines.push("Consider reviewing for duplicates.");
-  }
   if (linksSuggested.length > 0) {
     lines.push("");
     lines.push("## Suggested links (review and use `relations action=create`)");
@@ -3088,11 +2935,6 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
     if (hasQualityGaps) {
       lines.push(`2. **Improve quality:** \`entries action=update entryId="${eid}"\` \u2014 fill missing fields`);
     }
-  } else if (finalStatus === "proposed") {
-    lines.push(`1. **Connect it:** \`relations action=suggest entryId="${eid}"\` \u2014 discover additional links to support the proposal`);
-    if (hasQualityGaps) {
-      lines.push(`2. **Improve quality:** \`entries action=update entryId="${eid}"\` \u2014 strengthen the entry before approval`);
-    }
   } else {
     if (userLinkResults.length === 0) {
       lines.push(`1. **Connect it:** \`relations action=suggest entryId="${eid}"\` \u2014 discover what this should link to`);
@@ -3117,7 +2959,7 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
   } catch {
   }
   const next = [];
-  if (finalStatus === "committed" || finalStatus === "proposed") {
+  if (finalStatus === "committed") {
     next.push({ tool: "relations", description: "Discover connections", parameters: { action: "suggest", entryId: finalEntryId } });
   } else {
     if (userLinkResults.length === 0) {
@@ -3127,7 +2969,7 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
     next.push({ tool: "entries", description: "Accept onto Chain", parameters: { action: "commit", entryId: finalEntryId } });
   }
   const qualityPhrase = verdictQualityPhrase(verdictResult);
-  const summary = finalStatus === "committed" ? `Captured and accepted ${finalEntryId} (${name}) onto ${resolvedCollection}. ${qualityPhrase}` : finalStatus === "proposed" ? `Captured ${finalEntryId} (${name}) in ${resolvedCollection} and created a proposal for acceptance. ${qualityPhrase}` : `Captured ${finalEntryId} (${name}) as draft in ${resolvedCollection}. ${qualityPhrase}`;
+  const summary = finalStatus === "committed" ? `Captured and accepted ${finalEntryId} (${name}) onto ${resolvedCollection}. ${qualityPhrase}` : `Captured ${finalEntryId} (${name}) as draft in ${resolvedCollection}. ${qualityPhrase}`;
   const expectedFields = (col.fields ?? []).map((f) => ({
     key: f.key,
     type: f.type,
@@ -3161,14 +3003,6 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
           collection: resolvedCollection,
           name,
           status: finalStatus,
-          // WP-465 surface parity: thread full refusal (routes + mode) for coherency-refused auto-commits.
-          ...commitRefusal ? { coherencyRefusal: commitRefusal } : {},
-          // WP-485 Slice 2b round 3 (Codex P2, FEAT-1370): mirror the server's contradiction
-          // advisory into the structured payload — it was rendered in the human text (the
-          // "## Contradiction advisory" section above) but silently absent from
-          // structuredContent, so a structured-only MCP consumer saw a committed capture
-          // with no warning even though the text response carried one.
-          ...contradictionAdvisory ? { contradictionAdvisory } : {},
           // WP-480 S1: the server verdict rides `qualityVerdict` (named breaking
           // output-contract change, pre-launch).
           qualityVerdict: verdictResult?.verdict ? { ...verdictResult.verdict, source: verdictResult.source ?? "heuristic" } : void 0,
@@ -3203,14 +3037,12 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
   };
   return toolResult;
 }
-async function handleBatchCapture(server, { entries, autoCommit, preview }) {
+async function handleBatchCapture(server, { entries, preview }) {
   requireWriteAccess();
   const batchTimingStart = Date.now();
   const agentId = getAgentSessionId();
   const createdBy = agentId ? `agent:${agentId}` : "capture";
   const wsCtx = await getWorkspaceContext();
-  const freshGovernanceModeForBatch = await refreshWorkspaceGovernanceMode();
-  const shouldAutoCommitBatch = shouldAutoCommitCapture(autoCommit, freshGovernanceModeForBatch);
   let autoCommitApplied = false;
   const needsClassification = entries.map((e, i) => ({ ...e, index: i })).filter((e) => !e.collection);
   let classificationMap = /* @__PURE__ */ new Map();
@@ -3465,10 +3297,7 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
       if (!preview) resolveGapsForEntry(entry.name, result2.entryId);
       const batchWasAutoCommittedServerSide = result2.status === "active";
       const batchSpineFloor = result2.spineFloor;
-      let finalStatus = batchWasAutoCommittedServerSide ? "committed" : batchSpineFloor ? "draft_spine_floor" : "draft";
-      let commitError;
-      let commitRefusal;
-      let contradictionAdvisory;
+      const finalStatus = batchWasAutoCommittedServerSide ? "committed" : batchSpineFloor ? "draft_spine_floor" : "draft";
       if (batchWasAutoCommittedServerSide) {
         if (!preview) autoCommitApplied = true;
         await recordSessionActivity({ entryModified: internalId });
@@ -3565,40 +3394,6 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
           if (relationsProposedCount > 0) {
             batchEntryWarnings.push(`${relationsProposedCount} inline relation(s) converted to an agent proposal (misuse pattern) \u2014 review in Cortex UI.`);
           }
-          if (!preview && shouldAutoCommitBatch && !batchWasAutoCommittedServerSide && !batchSpineFloor) {
-            try {
-              const semanticConflicts = await discoverSemanticConflicts(entry.name, entry.description, resolvedSlug);
-              autoCommitApplied = true;
-              const commitResult = await kernelMutation("chain.commitEntry", {
-                entryId: finalEntryId,
-                author: agentId ? `agent:${agentId}` : void 0,
-                sessionId: agentId ?? void 0,
-                ...semanticConflicts.length > 0 ? { conflicts: semanticConflicts } : {}
-              });
-              const refusalReason = coherencyRefusalReason(commitResult);
-              if (refusalReason) {
-                commitError = `coherency gate refused this accept \u2014 ${refusalReason}`;
-                commitRefusal = commitResultField(commitResult, "refusal") ?? void 0;
-                finalStatus = "draft_on_failure";
-              } else {
-                finalStatus = commitResultField(commitResult, "status") === "proposal_created" ? "proposed" : "committed";
-                contradictionAdvisory = commitResultField(commitResult, "contradictionAdvisory") ?? void 0;
-                if (finalStatus === "committed") {
-                  await recordSessionActivity({ entryModified: internalId });
-                  trackChainEntryCommitted(wsCtx.workspaceId, {
-                    entry_id: finalEntryId,
-                    collection: resolvedSlug ?? void 0,
-                    commit_method: "auto",
-                    surface: "mcp_capture"
-                  });
-                }
-              }
-            } catch (error) {
-              commitError = error instanceof Error ? error.message : String(error);
-              finalStatus = "draft_on_failure";
-              await recordCommitFailure({ entryId: finalEntryId, error, sessionId: agentId, server });
-            }
-          }
           const entryNorm = result2.normalization;
           results.push({
             entryIdx,
@@ -3620,14 +3415,8 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
             // Fix 1 (TEN-2918 follow-up): not caller-supplied — the preview projection
             // must not report this predicted-but-duplicable ID as concrete.
             ...!entry.entryId ? { entryIdAssignedAtCapture: true } : {},
-            ...commitError ? { commitError } : {},
-            // WP-465 surface parity: thread full refusal payload for coherency-refused batch entries.
-            ...commitRefusal ? { commitRefusal } : {},
             ...batchSpineFloor ? { spineFloor: batchSpineFloor } : {},
             // WP-745 TASK-160: read by the render below.
-            // WP-485 Slice 2b (R2, FEAT-1370): the server's contradiction advisory (DEC-1321 —
-            // advisory only, never blocking) — visible, never a silent success.
-            ...contradictionAdvisory ? { contradictionAdvisory } : {},
             ...batchEntryWarnings.length > 0 ? { warnings: batchEntryWarnings } : {},
             ...entryNorm && (Object.keys(entryNorm.remapped).length > 0 || entryNorm.rejected.length > 0) && {
               normalization: { remapped: entryNorm.remapped, rejected: entryNorm.rejected }
@@ -3672,7 +3461,7 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
     logger: "product-brain"
   });
   if (preview) {
-    return buildBatchPreviewFromResults(created, failed, skippedLowConfidence, entries.length, entries, autoCommit);
+    return buildBatchPreviewFromResults(created, failed, skippedLowConfidence, entries.length, entries);
   }
   const totalAutoLinks = created.reduce((sum, r) => sum + r.autoLinks, 0);
   const totalAdvisedLinks = created.reduce((sum, r) => sum + r.advisedLinks, 0);
@@ -3738,41 +3527,20 @@ _Use \`entries action=move\` to correct any misclassified entries._`);
     lines.push("");
     lines.push("_Re-capture these with an explicit `collection` or use the suggested collection._");
   }
-  if (commitFailed.length > 0 || spineFloored.length > 0) {
+  if (spineFloored.length > 0) {
     lines.push("");
     lines.push("## Saved as draft");
-    for (const r of [...spineFloored, ...commitFailed]) {
-      if (r.spineFloor) {
-        lines.push(`- **${r.entryId}**: ${r.name} [${r.collection}] \u2014 ${r.spineFloor.exempt ? SPINE_FLOOR_EXEMPT_SENTENCE : SPINE_FLOOR_SENTENCE} Next: \`entries action=commit entryId="${r.entryId}"\`.`);
-      } else if (r.commitRefusal) {
-        lines.push(`- **${r.entryId}**: ${r.name} [${r.collection}]`);
-        const refusalText = coherencyRefusedResult("batch-capture", r.entryId ?? "", r.commitRefusal, { headingLevel: 3 }).content[0].text;
-        lines.push(...refusalText.split("\n"));
-      } else {
-        lines.push(`- **${r.entryId}**: ${r.name} [${r.collection}] \u2014 ${r.commitError}`);
-      }
+    for (const r of spineFloored) {
+      lines.push(`- **${r.entryId}**: ${r.name} [${r.collection}] \u2014 ${r.spineFloor.exempt ? SPINE_FLOOR_EXEMPT_SENTENCE : SPINE_FLOOR_SENTENCE} Next: \`entries action=commit entryId="${r.entryId}"\`.`);
     }
   }
-  const entriesWithContradictionAdvisory = created.filter((r) => r.contradictionAdvisory);
-  if (entriesWithContradictionAdvisory.length > 0) {
+  const entriesWithOverlaps = created.filter((r) => r.overlapCount && r.overlapCount > 0);
+  if (entriesWithOverlaps.length > 0) {
     lines.push("");
-    lines.push("## Contradiction Advisories");
-    for (const r of entriesWithContradictionAdvisory) {
-      lines.push(`- **${r.entryId}**: ${r.name} \u2014 ${r.contradictionAdvisory.reason}`);
-      for (const c of r.contradictionAdvisory.conflictDetails) {
-        lines.push(`  - \`${c.chainEntryId}\` ${c.chainEntryName}: ${c.explanation}`);
-      }
-    }
-  }
-  if (autoCommitApplied) {
-    const entriesWithOverlaps = created.filter((r) => r.overlapCount && r.overlapCount > 0);
-    if (entriesWithOverlaps.length > 0) {
-      lines.push("");
-      lines.push("## Related-entry Suggestions");
-      lines.push(`${entriesWithOverlaps.length} ${entriesWithOverlaps.length === 1 ? "entry has" : "entries have"} link candidates. Review their relevance before linking:`);
-      for (const r of entriesWithOverlaps) {
-        lines.push(`- **${r.entryId}**: ${r.name} \u2014 ${r.overlapCount} link ${r.overlapCount === 1 ? "candidate" : "candidates"}`);
-      }
+    lines.push("## Related-entry Suggestions");
+    lines.push(`${entriesWithOverlaps.length} ${entriesWithOverlaps.length === 1 ? "entry has" : "entries have"} link candidates. Review their relevance before linking:`);
+    for (const r of entriesWithOverlaps) {
+      lines.push(`- **${r.entryId}**: ${r.name} \u2014 ${r.overlapCount} link ${r.overlapCount === 1 ? "candidate" : "candidates"}`);
     }
   }
   const entriesWithWarnings = created.filter((r) => r.warnings && r.warnings.length > 0);
@@ -3848,20 +3616,16 @@ _Use \`entries action=move\` to correct any misclassified entries._`);
             ...r.confidenceTier ? { confidenceTier: r.confidenceTier } : {},
             ...r.warnings?.length ? { warnings: r.warnings } : {},
             ...r.normalization ? { normalization: r.normalization } : {},
-            ...r.commitError ? { commitError: r.commitError } : {},
-            // WP-465 surface parity: thread full refusal payload into batch structured output.
-            ...r.commitRefusal ? { coherencyRefusal: r.commitRefusal } : {},
+            // WP-751 M3a-1b (TASK-189, DEC-1700): `commitError`, `coherencyRefusal` (from
+            // `r.commitRefusal`), and `contradictionAdvisory` are deleted — all three were
+            // only ever populated by the client-side auto-commit attempt, deleted entirely.
             ...r.spineFloor ? { spineFloor: r.spineFloor } : {},
             // WP-745 TASK-160: the projection a machine consumer receives; was human-render only (Copilot review, PR #901).
             ...r.authorityDomain ? { domain: r.authorityDomain.slug } : {},
             // TEN-957 (WP-484 S2): inline relations created for this entry.
             ...r.relationsCreated ? { relationsCreated: r.relationsCreated } : {},
             // Finding #14: proposal outcomes are distinct from writes — never folded in.
-            ...r.relationsProposed ? { relationsProposed: r.relationsProposed } : {},
-            // WP-485 Slice 2b round 2 (Codex P1): the server's per-entry contradiction
-            // advisory — was reaching `results` but dropped here, the final projection the
-            // caller actually receives. Visible, never a silent success.
-            ...r.contradictionAdvisory ? { contradictionAdvisory: r.contradictionAdvisory } : {}
+            ...r.relationsProposed ? { relationsProposed: r.relationsProposed } : {}
           })),
           total: created.length,
           failed: failed.length,
@@ -3903,46 +3667,6 @@ _Use \`entries action=move\` to correct any misclassified entries._`);
     }
   };
 }
-function coherencyRefusalReason(commitResult) {
-  if (!commitResult || typeof commitResult !== "object" || !("status" in commitResult) || commitResult.status !== "coherency_refused") {
-    return null;
-  }
-  const refusal = "refusal" in commitResult ? commitResult.refusal : void 0;
-  return refusal?.reason ?? "refused by the coherency gate";
-}
-function commitResultField(commitResult, field) {
-  if (!commitResult || typeof commitResult !== "object" || !(field in commitResult)) return void 0;
-  return commitResult[field];
-}
-async function recordCommitFailure({
-  entryId,
-  error,
-  sessionId,
-  server
-}) {
-  const errorMessage = error instanceof Error ? error.message : String(error);
-  const logLine = `commitEntry failed for ${entryId}: ${errorMessage}${sessionId ? ` (session: ${sessionId})` : ""}`;
-  try {
-    await server.sendLoggingMessage({ level: "error", data: logLine, logger: "product-brain" });
-  } catch {
-  }
-  if (sessionId) {
-    try {
-      const tenName = `commitEntry failed for ${entryId} \u2014 ${errorMessage}`.slice(0, 250);
-      await kernelMutation("chain.createEntry", {
-        collectionSlug: "tensions",
-        name: tenName,
-        // Failure audit TENs intentionally stay draft — they need explicit human review,
-        // not auto-commit, even in Open mode.
-        status: "draft",
-        data: {},
-        createdBy: `agent:${sessionId}`,
-        sessionId
-      });
-    } catch {
-    }
-  }
-}
 async function runContradictionCheck(name, description, excludeEntryId, excludeDocId) {
   const warnings = [];
   try {
@@ -3983,25 +3707,6 @@ async function runContradictionCheck(name, description, excludeEntryId, excludeD
   }
   return warnings;
 }
-async function discoverSemanticConflicts(name, description, collectionHint) {
-  try {
-    const conflicts = await kernelQuery("chain.detectSemanticConflicts", {
-      name,
-      description,
-      ...collectionHint ? { collectionHint } : {}
-    });
-    return (conflicts ?? []).map((c) => ({
-      chainEntryId: c.chainEntryId,
-      chainEntryName: c.chainEntryName,
-      ...c.chainEntryDescription ? { chainEntryDescription: c.chainEntryDescription } : {},
-      type: c.type,
-      confidence: c.confidence,
-      explanation: c.explanation
-    }));
-  } catch {
-    return [];
-  }
-}
 
 // src/tools/conflict-preflight.ts
 async function runConflictPreflight(name, description, collectionHint) {
@@ -4027,6 +3732,65 @@ async function runConflictPreflight(name, description, collectionHint) {
   }
 }
 
+// src/tools/coherency-refusal.ts
+function asDescription(value) {
+  if (typeof value !== "string") return void 0;
+  const trimmed = value.trim();
+  return trimmed.length > 0 ? trimmed : void 0;
+}
+var GENERIC_RECOVERY_LINE = "Respond and retry: link the entry to the strategic spine, pass `coherencyAcknowledgement`, or supply a typed `steeringOverrideReason`.";
+function coherencyRefusedResult(toolName, entryId, refusal, options) {
+  const headingLevel = options?.headingLevel ?? 1;
+  const heading = "#".repeat(headingLevel);
+  const lines = [
+    `${heading} Write refused by the coherency gate`,
+    "",
+    refusal.reason ?? `The coherency gate refused this write on ${entryId}.`
+  ];
+  if (refusal.subject?.name) {
+    lines.push(`On: ${refusal.subject.name}`);
+  }
+  if (refusal.mode) {
+    lines.push("", `Mode: ${refusal.mode}`);
+  }
+  const routes = refusal.routes;
+  const routeKeys = routes ? Object.keys(routes) : [];
+  if (routeKeys.length > 0) {
+    lines.push("", `${"#".repeat(headingLevel + 1)} Recovery routes`);
+    const beforeRoutes = lines.length;
+    if (routes && typeof routes.link === "object" && routes.link !== null) {
+      const link = routes.link;
+      lines.push(`- **link:** ${asDescription(link.description) ?? "Link the entry to the strategic spine."}`);
+    }
+    if (routes && typeof routes.acknowledge === "object" && routes.acknowledge !== null) {
+      const ack = routes.acknowledge;
+      const responses = ack.responses?.join(" | ") ?? "linked | accepted-fix | diverged";
+      lines.push(
+        `- **acknowledge** (\`${ack.param ?? "coherencyAcknowledgement"}\`): ${responses}${ack.minReasonChars ? ` (\u2265${ack.minReasonChars} chars for diverged)` : ""}. ${asDescription(ack.description) ?? ""}`.trim()
+      );
+    }
+    if (routes && typeof routes.override === "object" && routes.override !== null) {
+      const ov = routes.override;
+      lines.push(
+        `- **override** (\`${ov.param ?? "steeringOverrideReason"}\`): \u2265${ov.minChars ?? 12} chars. ${asDescription(ov.description) ?? ""}`.trim()
+      );
+    }
+    if (lines.length === beforeRoutes) {
+      lines.push(GENERIC_RECOVERY_LINE);
+    }
+  } else {
+    lines.push("", GENERIC_RECOVERY_LINE);
+  }
+  return failureResult(
+    lines.join("\n"),
+    "COHERENCY_REFUSED",
+    `Coherency gate refused ${toolName} on ${entryId}.`,
+    "Use one of the refusal's routes (link / coherencyAcknowledgement / steeringOverrideReason), then retry.",
+    void 0,
+    { entryId, refusal }
+  );
+}
+
 // src/tools/knowledge/getHistory.ts
 import { z as z7 } from "zod/v3";
 var getHistorySchema = z7.object({
@@ -15635,7 +15399,7 @@ async function handleQuestionCreate(text, domain, blocks) {
   requireWriteAccess();
   try {
     const resolvedBlocks = blocks ? await Promise.all(blocks.map((b) => resolveEntryRef(b, "Blocked entry"))) : void 0;
-    const result2 = await kernelMutation("question.create", { text, domainSlug: domain, blocks: resolvedBlocks });
+    const result2 = await kernelMutation("question.create", { text, domainSlug: domain, blocks: resolvedBlocks, sessionId: getAgentSessionId() ?? void 0 });
     const message = await renderCreateAdoptConfirmation(result2, false);
     const next = result2.status === "unrouted" ? [{ tool: "question", description: "Confirm or reassign the suggestion", parameters: { action: "assign", entryId: result2.entryId } }] : void 0;
     return successResult(message, message, result2, next);
@@ -15648,7 +15412,7 @@ async function handleQuestionAdopt(entryId, domain, blocks) {
   try {
     const resolvedEntryId = await resolveEntryRef(entryId, "Entry");
     const resolvedBlocks = blocks ? await Promise.all(blocks.map((b) => resolveEntryRef(b, "Blocked entry"))) : void 0;
-    const result2 = await kernelMutation("question.adopt", { entryId: resolvedEntryId, domainSlug: domain, blocks: resolvedBlocks });
+    const result2 = await kernelMutation("question.adopt", { entryId: resolvedEntryId, domainSlug: domain, blocks: resolvedBlocks, sessionId: getAgentSessionId() ?? void 0 });
     const message = await renderCreateAdoptConfirmation(result2, true);
     return successResult(message, message, result2);
   } catch (err) {
@@ -16956,4 +16720,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-XJRVTFDC.js.map
+//# sourceMappingURL=chunk-73WYXKBT.js.map