@productbrain/mcp 0.0.1-beta.11155 → 0.0.1-beta.11161

package/dist/{chunk-A56KNDI2.js → chunk-HOGSWOSY.js}

@@ -2363,6 +2363,8 @@ var updateEntrySchema = z6.object({
   sourceExcerpt: z6.string().optional().describe("Verbatim excerpt from the source that backs this entry's claims. Write-once: can only be set if currently empty."),
   // WP-465 slice ⑤ — relay-only (TEN-2233): validation/min-length/recording live in Convex.
   steeringOverrideReason: z6.string().optional().describe("Typed override (\u226512 chars) that clears a steering coherency block on a misaligned governance write \u2014 always recorded with author attribution."),
+  qualityOverrideReason: z6.string().optional().describe("Why (\u226512 chars) to keep an accepted spine text its quality checklist failed \u2014 recorded as an override with who wrote it."),
+  // TASK-954
   coherencyAcknowledgement: z6.object({
     response: z6.enum(["linked", "accepted-fix", "diverged"]).describe("Explicit response to an acknowledge-required coherency challenge."),
     entryId: z6.string().optional().describe("Authorizing entry being linked/accepted (required for 'linked' and 'accepted-fix')."),
@@ -2375,24 +2377,22 @@ var commitEntrySchema = z6.object({
   preview: z6.boolean().optional().describe("If true, validates the accept without writing. Returns what would happen. Default false."),
   // WP-465 slice ⑤ — relay-only (TEN-2233): validation/recording live in Convex.
   steeringOverrideReason: z6.string().optional().describe("Typed override (\u226512 chars) that clears a steering coherency block at the publish chokepoint \u2014 always recorded with author attribution."),
+  qualityOverrideReason: z6.string().optional().describe("Why (\u226512 chars) to accept over a failing quality checklist \u2014 recorded as an override with who wrote it."),
+  // TASK-954
   coherencyAcknowledgement: z6.object({
     response: z6.enum(["linked", "accepted-fix", "diverged"]).describe("Explicit response to an acknowledge-required coherency challenge."),
     entryId: z6.string().optional().describe("Authorizing entry being linked/accepted (required for 'linked' and 'accepted-fix')."),
     reason: z6.string().optional().describe("Divergence rationale (required for 'diverged', \u226512 chars).")
   }).optional().describe("Explicit response to a coherency challenge (standard/strict workspace modes).")
 });
-async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowStatus: rawWorkflowStatus, data, order, canonicalKey, autoPublish, changeNote, changeReason, sourceRef, sourceExcerpt, steeringOverrideReason, coherencyAcknowledgement }, toolName = "entries") {
+async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowStatus: rawWorkflowStatus, data, order, canonicalKey, autoPublish, changeNote, changeReason, sourceRef, sourceExcerpt, steeringOverrideReason, qualityOverrideReason, coherencyAcknowledgement }, toolName = "entries") {
   requireWriteAccess();
   const fieldsProvided = [];
-  if (name !== void 0) fieldsProvided.push("name");
-  if (rawStatus !== void 0) fieldsProvided.push("status");
-  if (rawWorkflowStatus !== void 0) fieldsProvided.push("workflowStatus");
-  if (data !== void 0) fieldsProvided.push("data");
-  if (order !== void 0) fieldsProvided.push("order");
+  for (const [k, v] of [["name", name], ["status", rawStatus], ["workflowStatus", rawWorkflowStatus], ["data", data], ["order", order]]) if (v !== void 0) fieldsProvided.push(k);
   for (const [k, v] of [["canonicalKey", canonicalKey], ["sourceRef", sourceRef], ["sourceExcerpt", sourceExcerpt]]) if (v !== void 0) fieldsProvided.push(k);
   const status2 = rawStatus;
   const workflowStatus = rawWorkflowStatus;
-  const updateResult = await kernelMutation("chain.updateEntry", {
+  const send = () => kernelMutation("chain.updateEntry", {
     entryId,
     name,
     status: status2,
@@ -2407,16 +2407,31 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
     ...sourceRef !== void 0 ? { sourceRef } : {},
     ...sourceExcerpt !== void 0 ? { sourceExcerpt } : {},
     ...steeringOverrideReason ? { steeringOverrideReason } : {},
+    ...qualityOverrideReason ? { qualityOverrideReason } : {},
     ...coherencyAcknowledgement ? { coherencyAcknowledgement } : {}
   });
+  let updateResult = await send();
+  const held = updateResult.refusal;
+  if (held?.code === "QUALITY_HELD" && held.state === "judging") {
+    await settleVerdictBeforeAccept(async () => ({ judgeInFlight: true }), async () => {
+      const read = await kernelQuery("quality.getLatestVerdictForEntry", { entryId, textHash: held.textHash });
+      return read?.verdict && !read.stale ? read : { verdict: { llmStatus: "pending" } };
+    });
+    updateResult = await send();
+  }
   const maybeRefused = updateResult;
   if (maybeRefused.outcome === "refused" && maybeRefused.refusal) {
+    const mcp = maybeRefused.refusal.mcp;
+    if (mcp) return failureResult(`# ${entryId} not saved
+
+${mcp}`, "QUALITY_HELD", mcp.split("\n")[0] ?? mcp, mcp.split("\n")[1] ?? mcp, void 0, { entryId, refusal: maybeRefused.refusal });
     return updateRefusedResult(toolName, entryId, maybeRefused.refusal);
   }
-  const id = updateResult.id;
+  const applied = updateResult;
+  const id = applied.id;
   const fieldArgs = { name, status: rawStatus, workflowStatus: rawWorkflowStatus, order, canonicalKey, sourceRef, sourceExcerpt };
-  const revertedByServer = updateResult.revertedFields ?? [];
-  const { appliedFields, rejectedDataFields, revertedFields } = computeUpdateEntryAppliedFields(fieldArgs, updateResult.normalization, updateResult.provenanceApplied, revertedByServer);
+  const revertedByServer = applied.revertedFields ?? [];
+  const { appliedFields, rejectedDataFields, revertedFields } = computeUpdateEntryAppliedFields(fieldArgs, applied.normalization, applied.provenanceApplied, revertedByServer);
   const totalFailure = buildFieldsRejectedFailure(entryId, fieldsProvided, appliedFields, rejectedDataFields, revertedFields);
   if (totalFailure) return totalFailure;
   await recordSessionActivity({ entryModified: id });
@@ -2427,29 +2442,28 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
   const responseLines = [
     `# Entry Updated`,
     ``,
-    updateResult.receipt ?? `**${entryId}** \u2014 ${versionMode}.`,
+    applied.receipt ?? `**${entryId}** \u2014 ${versionMode}.`,
     // TASK-773: a draft team's discard names the drafts it took
-    ...updateResult.restsOnCount !== void 0 ? [restsOnCountLine(updateResult.restsOnCount, updateResult.restsOnBounded === true)] : [],
+    ...applied.qualityOverride ? [applied.qualityOverride] : [],
+    // TASK-954: the recorded line, never a silent clear
+    ...applied.restsOnCount !== void 0 ? [restsOnCountLine(applied.restsOnCount, applied.restsOnBounded === true)] : [],
     ``,
     // TASK-935
     `Internal ID: ${id}`,
     `**Workspace:** ${wsCtx.workspaceSlug} (${wsCtx.workspaceId})`
   ];
   pushAppliedFieldsLines(responseLines, fieldsProvided, appliedFields, rejectedDataFields, revertedFields);
-  const warnings = updateResult.normalizationWarnings ?? [];
+  const warnings = applied.normalizationWarnings ?? [];
   if (warnings.length > 0) {
     responseLines.push("");
     responseLines.push("\u26A0\uFE0F **Normalization warnings:**");
     for (const warning of warnings) responseLines.push(`  - ${warning}`);
   }
-  if ((updateResult.validationWarnings ?? []).length > 0) {
-    responseLines.push("");
-    responseLines.push("\u26A0\uFE0F **Validation warnings:**");
-    for (const warning of updateResult.validationWarnings ?? []) {
-      responseLines.push(`  - ${warning}`);
-    }
+  if ((applied.validationWarnings ?? []).length > 0) {
+    responseLines.push("", "\u26A0\uFE0F **Validation warnings:**");
+    for (const warning of applied.validationWarnings ?? []) responseLines.push(`  - ${warning}`);
   }
-  const summary = updateResult.receipt ?? `Updated ${entryId} \u2014 ${versionMode}. Fields: ${appliedFields.join(", ") || "none"}.`;
+  const summary = [applied.receipt ?? `Updated ${entryId} \u2014 ${versionMode}. Fields: ${appliedFields.join(", ") || "none"}.`, applied.qualityOverride].filter(Boolean).join(" ");
   const next = [
     { tool: "context", description: "View updated entry", parameters: { action: "get", entryId } },
     // DEC-1894: reads live on context
@@ -2463,26 +2477,26 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
       fieldsApplied: appliedFields,
       // PR B: renamed from fieldsProvided (caller intent) — this is the server's verdict, matching the text above
       // BET-192: normalization breakdown in structuredContent — symmetric with capture tool
-      ...updateResult.normalization && (Object.keys(updateResult.normalization.remapped).length > 0 || updateResult.normalization.rejected.length > 0) && {
+      ...applied.normalization && (Object.keys(applied.normalization.remapped).length > 0 || applied.normalization.rejected.length > 0) && {
         normalization: {
-          remapped: updateResult.normalization.remapped,
-          rejected: updateResult.normalization.rejected
+          remapped: applied.normalization.remapped,
+          rejected: applied.normalization.rejected
         }
       },
-      ...(updateResult.normalizationWarnings ?? []).length > 0 && {
-        normalizationWarnings: updateResult.normalizationWarnings
+      ...(applied.normalizationWarnings ?? []).length > 0 && {
+        normalizationWarnings: applied.normalizationWarnings
       },
-      ...(updateResult.validationWarnings ?? []).length > 0 && {
-        validationWarnings: updateResult.validationWarnings
+      ...(applied.validationWarnings ?? []).length > 0 && {
+        validationWarnings: applied.validationWarnings
       },
       ...revertedFields.length > 0 && { revertedFields },
       // #1026 R2: symmetric with the text above
-      ...updateResult.restsOnCount !== void 0 ? { restsOnCount: updateResult.restsOnCount, restsOnBounded: updateResult.restsOnBounded } : {}
+      ...applied.restsOnCount !== void 0 ? { restsOnCount: applied.restsOnCount, restsOnBounded: applied.restsOnBounded } : {}
       // TASK-935
     }, next)
   };
 }
-async function handleCommitEntry({ entryId, preview, steeringOverrideReason, coherencyAcknowledgement }, toolName = "entries") {
+async function handleCommitEntry({ entryId, preview, steeringOverrideReason, qualityOverrideReason, coherencyAcknowledgement }, toolName = "entries") {
   requireWriteAccess();
   const entry = await kernelQuery("chain.getEntry", { entryId });
   if (!entry) {
@@ -2524,6 +2538,8 @@ async function handleCommitEntry({ entryId, preview, steeringOverrideReason, coh
       sessionId: getAgentSessionId() ?? void 0,
       ...preview ? { preview: true } : {},
       ...steeringOverrideReason ? { steeringOverrideReason } : {},
+      ...qualityOverrideReason ? { qualityOverrideReason } : {},
+      // TASK-954
       ...coherencyAcknowledgement ? { coherencyAcknowledgement } : {},
       ...conflictEvidence.length > 0 ? { conflicts: conflictEvidence } : {}
     });
@@ -3268,6 +3284,9 @@ var entriesSchema = z10.object({
   steeringOverrideReason: z10.string().max(2e3).optional().describe(
     "For 'update'/'commit': typed override (\u226512 chars) that clears a steering coherency block."
   ),
+  qualityOverrideReason: z10.string().max(2e3).optional().describe(
+    "For 'update'/'commit': why (\u226512 chars) to keep a spine text its quality checklist failed; recorded as an override with who wrote it."
+  ),
   coherencyAcknowledgement: coherencyAcknowledgementFlatSchema.optional().describe(
     "For 'update'/'commit': explicit response to a coherency challenge (standard/strict workspace modes)."
   ),
@@ -3297,8 +3316,8 @@ var entriesActionUnion = z10.discriminatedUnion("action", [
 ]);
 var ENTRIES_ACTION_SPECS = {
   preflight: { params: ["collection", "canonicalKey", "links"], description: "Inspect structure before composition; no writes or permission grant." },
-  update: { params: ["entryId", "name", "status", "workflowStatus", "data", "order", "canonicalKey", "autoPublish", "changeNote", "changeReason", "sourceRef", "sourceExcerpt", "steeringOverrideReason", "coherencyAcknowledgement"], description: "entryId is required; every other field is optional." },
-  commit: { params: ["entryId", "preview", "steeringOverrideReason", "coherencyAcknowledgement"], description: "entryId is required." },
+  update: { params: ["entryId", "name", "status", "workflowStatus", "data", "order", "canonicalKey", "autoPublish", "changeNote", "changeReason", "sourceRef", "sourceExcerpt", "steeringOverrideReason", "qualityOverrideReason", "coherencyAcknowledgement"], description: "entryId is required; every other field is optional." },
+  commit: { params: ["entryId", "preview", "steeringOverrideReason", "qualityOverrideReason", "coherencyAcknowledgement"], description: "entryId is required." },
   move: { params: ["entryId", "toCollection", "canonicalKey"], description: "entryId and toCollection are required; canonicalKey is optional (atomic move+retype)." },
   verify: { params: ["entryId"], description: "entryId is required." },
   "still-true": { params: ["entryId"], description: "entryId is required. Allowed for the holder of the owning role; if that role is empty, the nearest seated Team Lead up the team tree." }
@@ -3505,7 +3524,7 @@ async function handleGet(entryId) {
   if (e.sourceExcerpt) lines.push(`**Source excerpt:** ${e.sourceExcerpt}`);
   const parsedOwner = parseOwnerOutput(e.owner);
   if (parsedOwner) lines.push(formatOwnerLine2(parsedOwner));
-  if (typeof e.pendingMove === "string") lines.push(sanitizeSingleLineField(e.pendingMove));
+  for (const line of [e.pendingMove, e.qualityOverride]) if (typeof line === "string") lines.push(sanitizeSingleLineField(line));
   const restsOnChanged = Array.isArray(e.restsOnChanged) ? e.restsOnChanged : [];
   for (const item of restsOnChanged) lines.push(sanitizeSingleLineField(restsOnLine(item)));
   if (epistemic) lines.push(formatEpistemicLine(epistemic));
@@ -3553,7 +3572,8 @@ async function handleGet(entryId) {
     ...e.sourceRef ? { sourceRef: String(e.sourceRef) } : {},
     ...e.sourceExcerpt ? { sourceExcerpt: String(e.sourceExcerpt) } : {},
     ...parsedOwner ? { owner: parsedOwner } : {},
-    ...typeof e.pendingMove === "string" ? { pendingMove: e.pendingMove } : {},
+    ...Object.fromEntries(["pendingMove", "qualityOverride"].filter((k) => typeof e[k] === "string").map((k) => [k, e[k]])),
+    // TASK-954
     ...restsOnChanged.length > 0 ? { restsOnChanged } : {},
     // TEN-2191: surface the WHY and its quality in the structured payload too.
     ...why.length > 0 ? { why } : {},
@@ -12890,4 +12910,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-A56KNDI2.js.map
+//# sourceMappingURL=chunk-HOGSWOSY.js.map