@productbrain/mcp 0.0.1-beta.11174 → 0.0.1-beta.11180

package/dist/{chunk-66VWT677.js → chunk-SW67SRBA.js}

@@ -146,8 +146,6 @@ var metaSchema = z.object({
   timing: z.object({
     classifyMs: z.number().optional(),
     createMs: z.number().optional(),
-    autoLinkMs: z.number().optional(),
-    userLinksMs: z.number().optional(),
     qualityMs: z.number().optional(),
     totalMs: z.number()
   }).passthrough().optional(),
@@ -673,24 +671,11 @@ This client stopped waiting after ${error.elapsedMs}ms. ${recovery}`
     })
   };
 }
-async function resumeReplayedCapture(replay, name, links, sessionId) {
-  const requested = (links ?? []).slice(0, 25);
-  let replayedLinks = [];
-  if (requested.length > 0) {
-    const batch = await kernelMutation("chain.createEntryRelations", {
-      relations: requested.map((l) => ({ fromEntryId: replay.entryId, toEntryId: l.to, type: l.type, proposedBy: "user" })),
-      sessionId
-    }).catch((err) => ({ results: requested.map((l) => ({ toEntryId: l.to, type: l.type, ok: false, error: err instanceof Error ? err.message : "unknown error" })) }));
-    replayedLinks = requested.map((l) => {
-      const row = batch.results.find((r) => r.toEntryId === l.to && r.type === l.type);
-      if (row?.ok) return { to: l.to, type: l.type, outcome: "written" };
-      const error = row?.error ?? "no answer for this link";
-      return /^Duplicate relation/.test(error) ? { to: l.to, type: l.type, outcome: "existing" } : { to: l.to, type: l.type, outcome: "failed", error };
-    });
-  }
+function resumeReplayedCapture(replay, name) {
+  const links = replay.links ?? [];
   const status2 = isTrustAccepted(replay.status) ? "committed" : "draft";
   const summary = `Already captured as ${replay.entryId}; this re-send returned that entry and wrote no second one.`;
-  const linkLines = replayedLinks.map((l) => `- ${l.type} ${l.to}: ${l.outcome === "written" ? "written now" : l.outcome === "existing" ? "already there" : `not written (${l.error})`}`);
+  const linkLines = links.map((l) => `- ${l.line}`);
   return {
     content: [{
       type: "text",
@@ -709,7 +694,7 @@ Read it with \`context action=get\`.`
       name,
       status: status2,
       replayed: true,
-      ...replayedLinks.length ? { replayedLinks } : {}
+      ...links.length ? { links } : {}
     })
   };
 }
@@ -1156,6 +1141,7 @@ var LABEL_DESCRIPTION_MAX_CHARS = 2e3;
 var RELATION_BATCH_MAX = 25;
 
 // src/tools/smart-capture.ts
+var CAPTURE_LINK_OUTCOMES = ["written", "refused"];
 var GENERIC_PROFILE = {
   governedDraft: false,
   descriptionField: null,
@@ -1203,7 +1189,7 @@ var captureSchema = z4.object({
   links: z4.array(z4.object({
     to: z4.string().describe("Target entry ID (e.g. '<PREFIX>-<n>')"),
     type: z4.string().describe("Relation type (e.g. 'governs', 'related_to', 'informs')")
-  })).optional().describe("Relations to create after capture. Without links, a capture writes no relations."),
+  })).optional().describe("Links written with the entry in one transaction; each returns written or refused with a reason; a refused link fails nothing."),
   sourceRef: z4.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: z4.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.
@@ -1489,7 +1475,8 @@ var captureSuccessOutputSchema = z4.object({
   // client-side auto-commit attempt that alone populated them (its gate refusal, the commit's advisory).
   replayed: z4.literal(true).optional(),
   // TASK-898: a re-sent capture answered with the first capture's entry
-  replayedLinks: z4.array(z4.object({ to: z4.string(), type: z4.string(), outcome: z4.enum(["written", "existing", "failed"]), error: z4.string().optional() })).optional(),
+  // TASK-897: the kernel's outcome per link, on a create and on a replay.
+  links: z4.array(z4.object({ toEntryId: z4.string(), type: z4.string(), outcome: z4.enum(CAPTURE_LINK_OUTCOMES), reason: z4.string().optional(), line: z4.string() }).passthrough()).optional(),
   fallbackApplied: z4.boolean().optional()
   // round 2 (PRRT_kwDORS4ils6cTNbF): classifier field above still names the original (pre-substitution) prediction
 }).strict();
@@ -1515,7 +1502,7 @@ var captureCompoundSchema = z4.object({
   links: z4.array(z4.object({
     to: z4.string().max(200).describe("Target entry ID (e.g. '<PREFIX>-<n>')"),
     type: z4.string().max(200).describe("Relation type (e.g. 'governs', 'related_to', 'informs')")
-  })).max(RELATION_BATCH_MAX).optional().describe("Relations to create after capture. Without links, a capture writes no relations."),
+  })).max(RELATION_BATCH_MAX).optional().describe("Links written with the entry in one transaction; each returns written or refused with a reason; a refused link fails nothing."),
   sourceRef: z4.string().max(2e3).optional().describe("URI or path of the source document backing this entry."),
   sourceExcerpt: z4.string().max(SOURCE_EXCERPT_MAX_CHARS).optional().describe("Verbatim excerpt from the source backing this entry's claims."),
   preview: z4.boolean().optional().describe("If true, validates without writing."),
@@ -1640,7 +1627,6 @@ async function handleCapture(server, { collection, name, description, context, c
       context,
       data,
       entryId: "",
-      linksCreated: [],
       collectionFields: col.fields ?? []
     });
     for (const [key, val] of Object.entries(inferred)) {
@@ -1695,6 +1681,7 @@ async function handleCapture(server, { collection, name, description, context, c
   let coachingFromCreate;
   let wasAutoCommittedServerSide = false;
   let spineFloor;
+  let linkOutcomes = [];
   try {
     const result2 = await kernelMutation("chain.createEntry", {
       collectionSlug: resolvedCollection,
@@ -1712,16 +1699,11 @@ async function handleCapture(server, { collection, name, description, context, c
       // WP-513: explicit rung-2 owner override (WP-753 O1b: its team derives server-side). TASK-681: `team` places a role or team.
       ...ownerRoleEntryId ? { ownerRoleEntryId } : {},
       ...team ? { teamRef: team } : {},
-      // WP-475 E1 (Codex P2 — CLI parity): user links are known UPFRONT, so the
-      // post-relations re-evaluation below owns the single LLM run — defer the
-      // create-time LLM schedule (deterministic, mirrors `pb capture --link`).
-      // A capture without links writes no relations (TASK-599): no defer, it keeps its LLM run.
-      // Fail-safe: if the relations batch then creates nothing, the deferred row
-      // keeps its delivered heuristic verdict with llmStatus 'skipped' (same
-      // accepted shape as the CLI dying between create and re-eval).
-      ...links && links.length > 0 ? { deferLLMSchedule: true } : {}
+      // TASK-897 (DEC-2113): the caller's links ride in this one request; the kernel resolves each target, judges the
+      // verb and writes the row before its one coaching verdict, and answers per link (a refused link fails nothing).
+      ...links && links.length > 0 ? { links: links.map((l) => ({ toEntryId: l.to, type: l.type })) } : {}
     });
-    if (result2 && "replayed" in result2) return resumeReplayedCapture(result2, name, links, agentId ?? void 0);
+    if (result2 && "replayed" in result2) return resumeReplayedCapture(result2, name);
     if (result2 && isPreviewCreateEntryResult(result2)) {
       const preflight = await resolvePreviewPreflight(result2, links);
       const dupCount = groundingReport.duplicates.length;
@@ -1791,6 +1773,7 @@ No DB writes \u2014 call without \`preview:true\` to capture for real.${result2.
     normalizationMeta = created.normalization;
     formativeHints = created.qualityHints ?? [];
     coachingFromCreate = created.coaching;
+    linkOutcomes = created.links ?? [];
     resolveGapsForEntry(name, created.entryId);
     const createdStatus = created.status;
     if (createdStatus !== void 0 && isTrustAccepted(createdStatus)) wasAutoCommittedServerSide = true;
@@ -1826,78 +1809,8 @@ Use \`context action=get\` to inspect the existing entry, or \`entries action=up
     throw error;
   }
   const tAfterCreate = Date.now();
-  const linksCreated = [];
-  const userLinkResults = [];
-  const pendingRelations = [];
-  if (links && links.length > 0 && finalEntryId) {
-    const slugByPrefix = new Map((await getCollections()).filter((c) => c.idPrefix).map((c) => [c.idPrefix, c.slug]));
-    for (const link of links) {
-      pendingRelations.push({
-        fromEntryId: finalEntryId,
-        toEntryId: link.to,
-        type: link.type,
-        // WP-474 E4-REWIRE: user-chosen type — mark provenance, NO confidence (human-chosen).
-        proposedBy: "user",
-        meta: { targetName: link.to, targetCollection: slugByPrefix.get(link.to.split("-")[0]) ?? "unknown" }
-      });
-    }
-  }
-  const resolvedCK = canonicalKey;
-  const batchRelationsPromise = pendingRelations.length > 0 ? kernelMutation(
-    "chain.createEntryRelations",
-    {
-      // WP-474 E4-REWIRE: carry provenance through to the server seam.
-      relations: pendingRelations.slice(0, 25).map((r) => ({
-        fromEntryId: r.fromEntryId,
-        toEntryId: r.toEntryId,
-        type: r.type,
-        ...r.proposedBy !== void 0 ? { proposedBy: r.proposedBy } : {}
-      })),
-      sessionId: agentId ?? void 0
-    }
-  ).catch((err) => {
-    entryWarnings.push(`Linking partially failed: ${err instanceof Error ? err.message : "unknown error"}`);
-    return null;
-  }) : Promise.resolve(null);
-  const cardinalityPromise = resolvedCK ? kernelQuery("chain.checkCardinalityWarning", { canonicalKey: resolvedCK, excludeEntryId: internalId }).catch(() => null) : Promise.resolve(null);
-  const coachingPromise = coachingFromCreate !== void 0 ? Promise.resolve(coachingFromCreate) : kernelMutation("quality.evaluateHeuristicAndSchedule", {
-    entryId: finalEntryId,
-    context: "capture"
-  }).catch(() => null);
-  const [batchResult, cardinalityCheck, coachingAtCreate] = await Promise.all([
-    batchRelationsPromise,
-    cardinalityPromise,
-    coachingPromise
-  ]);
-  let verdictResult = coachingAtCreate;
-  if ((batchResult?.created ?? 0) > 0) {
-    const reEval = await kernelMutation("quality.evaluateHeuristicAndSchedule", {
-      entryId: finalEntryId,
-      context: "capture"
-    }).catch(() => null);
-    if (reEval?.verdict) verdictResult = reEval;
-  }
-  const tAfterRelations = Date.now();
-  if (batchResult) {
-    for (let i = 0; i < pendingRelations.length; i++) {
-      const pending = pendingRelations[i];
-      const result2 = batchResult.results[i];
-      if (!result2) continue;
-      if (result2.ok) {
-        userLinkResults.push({ label: `\u2713 ${pending.type} \u2192 ${pending.toEntryId}`, ok: true });
-        linksCreated.push({
-          targetEntryId: pending.toEntryId,
-          targetName: pending.meta.targetName,
-          targetCollection: pending.meta.targetCollection,
-          relationType: pending.type
-        });
-      } else {
-        userLinkResults.push({ label: `\u2717 ${pending.type} \u2192 ${pending.toEntryId}: ${result2.error ?? "failed"}`, ok: false });
-      }
-    }
-  }
+  const verdictResult = coachingFromCreate;
   const tAfterQuality = Date.now();
-  const cardinalityWarning = cardinalityCheck?.warning ?? null;
   let coachingSection = "";
   if (verdictResult?.verdict && verdictResult.verdict.tier !== "passive" && verdictResult.verdict.criteria.length > 0) {
     coachingSection = formatRubricCoaching(verdictResult);
@@ -1969,10 +1882,10 @@ Use \`context action=get\` to inspect the existing entry, or \`entries action=up
     lines.push("");
     lines.push(`**View in Cortex:** ${studioUrl}`);
   }
-  if (userLinkResults.length > 0) {
+  if (linkOutcomes.length > 0) {
     lines.push("");
     lines.push("## Requested links");
-    for (const r of userLinkResults) lines.push(`- ${r.label}`);
+    for (const l of linkOutcomes) lines.push(`- ${l.line}`);
   }
   if (finalStatus === "committed") {
     lines.push("");
@@ -2005,9 +1918,16 @@ Use \`context action=get\` to inspect the existing entry, or \`entries action=up
       hint_fields: formativeHints.map((h) => h.field)
     });
   }
-  const isWorkPackageOrBetCapture = isWorkPackageCapture || resolvedCK === "bet";
-  const isBetOrGoal = isWorkPackageOrBetCapture || resolvedCK === "goal";
-  const hasStrategyLink = linksCreated.some((l) => l.targetCollection === "strategy");
+  const isWorkPackageOrBetCapture = isWorkPackageCapture || canonicalKey === "bet";
+  const isBetOrGoal = isWorkPackageOrBetCapture || canonicalKey === "goal";
+  let hasStrategyLink = false;
+  if (isBetOrGoal) {
+    const written = linkOutcomes.filter((l) => l.outcome === "written");
+    if (written.length > 0) {
+      const slugByPrefix = new Map((await getCollections()).filter((c) => c.idPrefix).map((c) => [c.idPrefix, c.slug]));
+      hasStrategyLink = written.some((l) => slugByPrefix.get(l.toEntryId.split("-")[0]) === "strategy");
+    }
+  }
   if (isBetOrGoal && !hasStrategyLink) {
     lines.push("");
     lines.push(
@@ -2015,10 +1935,6 @@ Use \`context action=get\` to inspect the existing entry, or \`entries action=up
     );
     await recordSessionActivity({ strategyLinkWarnedForEntryId: internalId });
   }
-  if (cardinalityWarning) {
-    lines.push("");
-    lines.push(`**Cardinality warning:** ${cardinalityWarning}`);
-  }
   if (entryWarnings.length > 0) {
     lines.push("");
     lines.push("## Validation Warnings");
@@ -2040,12 +1956,12 @@ Use \`context action=get\` to inspect the existing entry, or \`entries action=up
       lines.push(`2. **Improve quality:** \`entries action=update entryId="${eid}"\` \u2014 fill missing fields`);
     }
   } else {
-    if (userLinkResults.length === 0) {
+    if (linkOutcomes.length === 0) {
       lines.push(`1. **Connect it:** find related entries with \`context action=search\`, then \`relations action=create from="${eid}" to="<ID>" type="<verb>"\``);
     }
-    lines.push(`${userLinkResults.length === 0 ? "2" : "1"}. **Review the possibility:** Keep this visible draft while it is an alternative. Accept only when commitment is intended and your role is authorized. Types reserved for human acceptance require human ratification. An audit advisory is not a reason to accept it.`);
+    lines.push(`${linkOutcomes.length === 0 ? "2" : "1"}. **Review the possibility:** Keep this visible draft while it is an alternative. Accept only when commitment is intended and your role is authorized. Types reserved for human acceptance require human ratification. An audit advisory is not a reason to accept it.`);
     if (hasQualityGaps) {
-      lines.push(`${userLinkResults.length === 0 ? "3" : "2"}. **Improve quality:** \`entries action=update entryId="${eid}"\` \u2014 fill missing fields`);
+      lines.push(`${linkOutcomes.length === 0 ? "3" : "2"}. **Improve quality:** \`entries action=update entryId="${eid}"\` \u2014 fill missing fields`);
     }
   }
   try {
@@ -2088,8 +2004,7 @@ Use \`context action=get\` to inspect the existing entry, or \`entries action=up
   const timing = {
     classifyMs: tAfterClassify - timingStart,
     createMs: tAfterCreate - tAfterClassify,
-    autoLinkMs: tAfterRelations - tAfterCreate,
-    qualityMs: tAfterQuality - tAfterRelations,
+    qualityMs: tAfterQuality - tAfterCreate,
     totalMs
   };
   const toolResult = {
@@ -2120,6 +2035,8 @@ Use \`context action=get\` to inspect the existing entry, or \`entries action=up
           qualityHints: formativeHints,
           // WP-745 TASK-160 (PR #901 review): the machine-readable projection — was reaching the human text only.
           ...spineFloor && { spineFloor },
+          ...linkOutcomes.length > 0 && { links: linkOutcomes },
+          // TASK-897
           // WP-318 S2: the possible-duplicate report — always included (empty when no ask match)
           groundingReport
         },
@@ -12910,4 +12827,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-66VWT677.js.map
+//# sourceMappingURL=chunk-SW67SRBA.js.map