@productbrain/mcp 0.0.1-beta.10769 → 0.0.1-beta.10778

package/dist/{chunk-UNDEFOTU.js → chunk-W3NKFJSJ.js}

@@ -625,98 +625,80 @@ async function foldWhyIntoData(slug, why, userData) {
 // src/tools/smart-capture.ts
 import { z as z4 } from "zod/v3";
 
+// src/lib/statusCategories.ts
+var STATUS_CATEGORIES = ["draft", "current", "past", "discarded"];
+var STATUS_CATEGORY_SET = new Set(STATUS_CATEGORIES);
+var TRUST_ACCEPTED_WORDS = ["accepted", "active", "current"];
+function isTrustAccepted(status2) {
+  return TRUST_ACCEPTED_WORDS.includes(status2);
+}
+var TRUST_ARCHIVED_WORDS = ["archived", "deprecated", "past", "discarded"];
+var ENTRY_STATUS_READ_WORDS = ["draft", ...TRUST_ACCEPTED_WORDS, ...TRUST_ARCHIVED_WORDS];
+var ENTRY_STATUS_WRITE_WORDS = ["draft", "archived"];
+
 // src/lib/captureTimeoutOutcome.ts
 function isGatewayTimeout(error) {
   return error?.name === "GatewayTimeoutError";
 }
-function buildCaptureOutcomeUnknownResult(error, name, msg) {
-  if (!error.mayHaveLanded) return buildDryRunTimeoutResult(error, msg);
-  const recovery = `Search for "${name}" before retrying \u2014 a retry will duplicate it if the write landed. If found, read back its commit status and coaching output before assuming either is missing.`;
+function buildCaptureOutcomeUnknownResult(error, msg) {
+  const recovery = error.mayHaveLanded ? "Re-send the same capture; the kernel replays it if the first one landed, so it never writes twice." : "Re-run the preview; nothing was written.";
   return {
     content: [{
       type: "text",
-      text: `# Capture Outcome Unknown \u2014 Not Failed
+      text: `# Capture Timed Out
 
 ${msg}
 
-**What is known:** the request reached the server and this client gave up waiting after ${error.elapsedMs}ms. The entry **may or may not** have been written.
-
-**What definitely did NOT run:** relation linking \u2014 a separate mutation issued after the create returns, so it is never reached when create itself times out.
-
-**What may also have run \u2014 outcome unknown, same reason as the entry itself:** auto-commit and quality coaching. Both execute INSIDE the create call, before it returns, so if the server finished after this client stopped waiting, they ran too. Do not assume either is absent \u2014 read the entry back to find out.
-
-**Do this next:** ${recovery}`
+This client stopped waiting after ${error.elapsedMs}ms. ${recovery}`
     }],
-    structuredContent: failure(
-      "CAPTURE_OUTCOME_UNKNOWN",
-      msg,
-      recovery,
-      [{ tool: "context", description: "Search for the entry before retrying", parameters: { action: "search", query: name } }],
-      {
-        // Structured so a caller can branch on this without parsing prose — the
-        // reconciliation signal TEN-1126 says the capture path never had.
-        partialSuccess: true,
-        mayHaveLanded: error.mayHaveLanded,
-        budgetMs: error.budgetMs,
-        elapsedMs: error.elapsedMs,
-        completedBeforeTimeout: [],
-        // WP-575 review (P1): genuinely definite — relation linking is a separate mutation
-        // never reached when create throws. See the doc comment above for why `commit` and
-        // `coaching` were removed from this list.
-        didNotRun: ["relations"],
-        // WP-575 review (P1): distinct from `didNotRun` on purpose — these ran INSIDE the
-        // same server call as the create, so their outcome is exactly as unknown as the
-        // entry's. A caller must read the entry back, never assume either ran or didn't.
-        mayHaveRun: ["commit", "coaching"]
-      }
-    )
+    structuredContent: failure("CAPTURE_OUTCOME_UNKNOWN", msg, recovery, [], {
+      mayHaveLanded: error.mayHaveLanded,
+      budgetMs: error.budgetMs,
+      elapsedMs: error.elapsedMs
+    })
   };
 }
-function buildDryRunTimeoutResult(error, msg) {
-  const recovery = "Re-run the preview \u2014 nothing was written, so there is nothing to search for and nothing to duplicate.";
+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 };
+    });
+  }
+  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})`}`);
   return {
     content: [{
       type: "text",
-      text: `# Preview Outcome Unknown \u2014 Nothing Was Written
+      text: `# Already Captured: ${replay.entryId}
 
-${msg}
-
-**What is known:** the request reached the server and this client gave up waiting after ${error.elapsedMs}ms. This was a **preview**, and the server returns its validation result before performing any write \u2014 so no entry, auto-commit, quality coaching, or relation linking happened.
+${summary}${linkLines.length ? `
 
-**What is unknown:** the validation verdict this preview was asked for.
+**Links:**
+${linkLines.join("\n")}` : ""}
 
-**Do this next:** ${recovery}`
+Read it with \`context action=get\`.`
     }],
-    structuredContent: failure(
-      "CAPTURE_OUTCOME_UNKNOWN",
-      msg,
-      recovery,
-      [],
-      {
-        // No partial success to reconcile: a dry run has no server-side state to converge on.
-        partialSuccess: false,
-        mayHaveLanded: false,
-        budgetMs: error.budgetMs,
-        elapsedMs: error.elapsedMs,
-        completedBeforeTimeout: [],
-        didNotRun: ["entry", "relations", "commit", "coaching"],
-        mayHaveRun: []
-      }
-    )
+    structuredContent: success(summary, {
+      entryId: replay.entryId,
+      collection: replay.collection,
+      name,
+      status: status2,
+      replayed: true,
+      ...replayedLinks.length ? { replayedLinks } : {}
+    })
   };
 }
 
-// src/lib/statusCategories.ts
-var STATUS_CATEGORIES = ["draft", "current", "past", "discarded"];
-var STATUS_CATEGORY_SET = new Set(STATUS_CATEGORIES);
-var TRUST_ACCEPTED_WORDS = ["accepted", "active", "current"];
-function isTrustAccepted(status2) {
-  return TRUST_ACCEPTED_WORDS.includes(status2);
-}
-var TRUST_ARCHIVED_WORDS = ["archived", "deprecated", "past", "discarded"];
-var ENTRY_STATUS_READ_WORDS = ["draft", ...TRUST_ACCEPTED_WORDS, ...TRUST_ARCHIVED_WORDS];
-var ENTRY_STATUS_WRITE_WORDS = ["draft", "archived"];
-
 // src/lib/scopedCache.ts
 var ScopedCache = class {
   constructor(ttlMs, maxScopes = 100) {
@@ -1466,8 +1448,7 @@ var captureSuccessOutputSchema = z4.object({
   collection: z4.string(),
   name: z4.string(),
   status: z4.enum(["draft", "committed", "proposed", "draft_on_failure"]),
-  // WP-480 S1: `qualityScore` (required, client N/10) deleted — a named, accepted
-  // breaking output-contract change. The server verdict stays as `qualityVerdict`.
+  // WP-480 S1: `qualityScore` deleted (accepted breaking output change); the server verdict stays as `qualityVerdict`.
   qualityVerdict: z4.record(z4.unknown()).optional(),
   qualityHints: z4.array(z4.record(z4.unknown())).optional(),
   groundingReport: captureSinglePreviewOutputSchema.shape.groundingReport.optional(),
@@ -1485,10 +1466,11 @@ var captureSuccessOutputSchema = z4.object({
   })).optional(),
   // TASK-160: the server's birth-floor verdict must survive the strict output contract.
   spineFloor: z4.object({ relationType: z4.string(), parentKind: z4.string(), exempt: z4.boolean() }).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.
+  // WP-751 M3a-1b (TASK-189, DEC-1700): `coherencyRefusal` and `contradictionAdvisory` are deleted with the
+  // 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(),
   fallbackApplied: z4.boolean().optional()
   // round 2 (PRRT_kwDORS4ils6cTNbF): classifier field above still names the original (pre-substitution) prediction
 }).strict();
@@ -1726,6 +1708,7 @@ async function handleCapture(server, { collection, name, description, context, c
       // accepted shape as the CLI dying between create and re-eval).
       ...links && links.length > 0 ? { deferLLMSchedule: true } : {}
     });
+    if (result2 && "replayed" in result2) return resumeReplayedCapture(result2, name, links, agentId ?? void 0);
     if (result2 && isPreviewCreateEntryResult(result2)) {
       const preflight = await resolvePreviewPreflight(result2, links);
       const dupCount = groundingReport.duplicates.length;
@@ -1800,7 +1783,7 @@ No DB writes \u2014 call without \`preview:true\` to capture for real.${result2.
     spineFloor = created.spineFloor;
   } catch (error) {
     const msg = error instanceof Error ? error.message : String(error);
-    if (isGatewayTimeout(error)) return buildCaptureOutcomeUnknownResult(error, name, msg);
+    if (isGatewayTimeout(error)) return buildCaptureOutcomeUnknownResult(error, msg);
     if (msg.includes("Duplicate") || msg.includes("already exists")) {
       return {
         content: [{
@@ -3740,9 +3723,9 @@ async function handleSearch(server, query, collection, status2) {
     data: `Searching${scope} for "${query}"...`,
     logger: "product-os"
   });
-  const answer = await kernelQuery("chain.search", { query, collectionSlug: collection, status: status2 });
-  const results = answer.rows;
-  const platform = platformSearchLines(answer.platform);
+  const answer2 = await kernelQuery("chain.search", { query, collectionSlug: collection, status: status2 });
+  const results = answer2.rows;
+  const platform = platformSearchLines(answer2.platform);
   const matches = results.length + platform.count;
   if (matches === 0) {
     const recorded = recordGap({ query, tool: "context", action: "search", gapType: "search_zero", collectionScope: collection });
@@ -3780,7 +3763,7 @@ async function handleSearch(server, query, collection, status2) {
   }).join("\n");
   const header = `## Search Result
 
-# Search Results for "${query}"${scope} (${matches}${answer.truncated ? "+" : ""} match${matches === 1 ? "" : "es"})
+# Search Results for "${query}"${scope} (${matches}${answer2.truncated ? "+" : ""} match${matches === 1 ? "" : "es"})
 
 **By collection:** ${collSummary}`;
   const footer = `_Tip: Use \`collection\` param to scope search. Use context action=get with an entry ID for full details._`;
@@ -3802,7 +3785,7 @@ async function handleSearch(server, query, collection, status2) {
   return {
     content: [{ type: "text", text: `${header}
 
-${[...platform.lines, formatted, answer.cutLine].filter(Boolean).join("\n")}
+${[...platform.lines, formatted, answer2.cutLine].filter(Boolean).join("\n")}
 
 ${footer}` }],
     structuredContent: success(`Found ${total} entries${platform.count ? ` and ${platform.count} platform glossary terms` : ""} matching '${query}'.`, {
@@ -3810,8 +3793,8 @@ ${footer}` }],
       results: structuredResults,
       entries: entriesForView,
       total,
-      truncated: answer.truncated,
-      cutLine: answer.cutLine,
+      truncated: answer2.truncated,
+      cutLine: answer2.cutLine,
       query
     })
   };
@@ -11669,16 +11652,20 @@ var PRODUCT_FEEDBACK_STATUSES = ["new", "triaged", "planned", "resolved", "dismi
 var VENDOR_SETTABLE_STATUSES = PRODUCT_FEEDBACK_STATUSES.filter(
   (status2) => status2 !== "screening"
 );
+var FEEDBACK_TENSION_ID_PATTERN = /^TEN-[1-9][0-9]*$/;
+var MAX_FEEDBACK_ANSWER_LENGTH = 240;
 
 // src/tools/feedback.ts
 var actions = ["submit", "list", "queue", "note", "group", "status"];
 var category = z30.enum(PRODUCT_FEEDBACK_CATEGORIES);
 var status = z30.enum(PRODUCT_FEEDBACK_STATUSES);
 var vendorStatus = z30.enum(VENDOR_SETTABLE_STATUSES);
+var tensionId = z30.string().regex(FEEDBACK_TENSION_ID_PATTERN);
+var answer = z30.string().trim().min(1).max(MAX_FEEDBACK_ANSWER_LENGTH);
 var MESSAGE_DISPLAY_LIMIT = 1e3;
 var FULL_MESSAGE_MAX_LIMIT = 5;
 var feedbackSchema = z30.object({
-  action: z30.enum(actions).describe("submit requires message; list returns your own workspace's feedback (all statuses); queue is the vendor triage queue (system admins only) and accepts filters; note requires feedbackId+note; group requires feedbackIds+groupId (null clears); status requires feedbackId+status."),
+  action: z30.enum(actions).describe("submit requires message; list returns your own workspace's feedback (all statuses); queue is the vendor triage queue (system admins only) and accepts filters; note requires feedbackId+note; group requires feedbackIds+groupId (null clears); status requires feedbackId+status and may carry tensionId and answer."),
   message: z30.string().max(1e4).optional().describe("Required for submit: the product feedback text (max 10,000 characters)."),
   category: category.optional().describe("Submit/queue category; submit defaults to other."),
   command: z30.string().max(1e3).optional().describe("Optional submit command context; argument values are scrubbed server-side."),
@@ -11688,6 +11675,8 @@ var feedbackSchema = z30.object({
   feedbackIds: z30.array(z30.string().max(200)).max(100).optional().describe("Required for group: 1\u2013100 target feedback IDs."),
   note: z30.string().max(4e3).optional().describe("Required for note: replacement triage note (max 4,000 characters)."),
   status: status.optional().describe("list/queue filter (any status, including 'screening'); the status action's required target value excludes 'screening' (system-only)."),
+  tensionId: tensionId.optional().describe("status only: the tension the report joined, its display id TEN- and a number (vendor-only, never shown to the reporter)."),
+  answer: answer.optional().describe(`status only: the one sentence the reporter reads under their report, dated (max ${MAX_FEEDBACK_ANSWER_LENGTH} characters).`),
   groupId: z30.string().min(1).max(200).nullable().optional().describe("Group filter, or required group destination; null explicitly clears grouping."),
   workspaceId: z30.string().max(200).optional().describe("Queue-only filter (vendor cross-workspace triage); omitted means all workspaces. Has no effect on list \u2014 your own workspace is always injected server-side."),
   since: z30.number().optional().describe("Queue-only lower createdAt window bound (inclusive) in epoch milliseconds."),
@@ -11702,7 +11691,7 @@ var union = z30.discriminatedUnion("action", [
   z30.object({ action: z30.literal("queue"), status: status.optional(), category: category.optional(), groupId: z30.string().min(1).max(200).optional(), workspaceId: z30.string().max(200).optional(), since: z30.number().optional(), before: z30.number().optional(), limit: z30.number().int().min(1).max(100).optional(), cursor: z30.string().max(2e3).optional(), full: z30.boolean().optional() }),
   z30.object({ action: z30.literal("note"), feedbackId: z30.string().min(1).max(200), note: z30.string().max(4e3) }),
   z30.object({ action: z30.literal("group"), feedbackIds: z30.array(z30.string().min(1).max(200)).min(1).max(100), groupId: z30.string().min(1).max(200).nullable() }),
-  z30.object({ action: z30.literal("status"), feedbackId: z30.string().min(1).max(200), status: vendorStatus })
+  z30.object({ action: z30.literal("status"), feedbackId: z30.string().min(1).max(200), status: vendorStatus, tensionId: tensionId.optional(), answer: answer.optional() })
 ]);
 var specs = {
   submit: { params: ["message", "category", "command", "route", "client"], description: "message is required; category defaults to other." },
@@ -11711,7 +11700,10 @@ var specs = {
   queue: { params: ["status", "category", "groupId", "workspaceId", "since", "before", "limit", "cursor", "full"], description: "Vendor cross-workspace triage queue (system admins only); filters are optional, status defaults to new; cursor pages, full lifts message truncation for limit <= 5." },
   note: { params: ["feedbackId", "note"], description: "feedbackId and note are required." },
   group: { params: ["feedbackIds", "groupId"], description: "feedbackIds and groupId are required; null clears a group." },
-  status: { params: ["feedbackId", "status"], description: "feedbackId and status are required; status cannot be set to 'screening' (system-only)." }
+  status: {
+    params: ["feedbackId", "status", "tensionId", "answer"],
+    description: "feedbackId and status are required; status cannot be set to 'screening' (system-only); optional tensionId and answer are written in the same call."
+  }
 };
 function result(action, data) {
   const summary = action === "submit" ? "Feedback submitted." : `Feedback ${action} complete.`;
@@ -12840,4 +12832,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-UNDEFOTU.js.map
+//# sourceMappingURL=chunk-W3NKFJSJ.js.map