@productbrain/mcp 0.0.1-beta.8770 → 0.0.1-beta.8779

package/dist/{chunk-KKDDIKQU.js → chunk-USZ2527R.js}

@@ -36,14 +36,13 @@ import {
   trackClassifierDivergence,
   trackCollectionClassified,
   trackCommitErrorByCode,
-  trackFieldGuidanceApplied,
   trackKnowledgeGap,
   trackQualityCheck,
   trackQualityVerdict,
   trackSessionCaptureRate,
   trackZeroCaptureAuditFired,
   validateCallerSession
-} from "./chunk-7GE3P7XW.js";
+} from "./chunk-EQSIIMRC.js";
 
 // src/server.ts
 import { McpServer as McpServer2 } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -905,13 +904,6 @@ async function captureVersionOrPreview(preview, params) {
   return preview ? buildVersionPreviewResult(params.groundingEntry, params.name) : runVersionCapture(params);
 }
 
-// src/lib/bodyTemplate.ts
-function readBodyTemplateGuidance(bodyTemplate) {
-  if (!bodyTemplate || bodyTemplate.length === 0) return void 0;
-  return bodyTemplate.map((entry) => `## ${entry.heading}
-${entry.guidance}`).join("\n\n");
-}
-
 // src/lib/inferSourceDate.ts
 function inferSourceDate(...values) {
   const combined = values.filter((value) => typeof value === "string" && value.trim().length > 0).join("\n");
@@ -1171,29 +1163,23 @@ function verdictQualityPhrase(result2) {
   if (!v || criteria.length === 0) {
     return "Quality: not evaluated (no rubric for this type).";
   }
-  const gating = criteria.filter((c) => c.gating !== false);
-  const total = gating.length;
-  const passed = gating.filter((c) => c.passed).length;
-  return v.passed ? `Quality: all ${total} gating criteria pass.` : `Quality: ${passed}/${total} gating criteria pass \u2014 see the coaching above.`;
-}
-function hintLines(criteria) {
-  const hints = (criteria ?? []).filter((c) => c.gating === false && !c.passed);
-  return hints.length === 0 ? [] : ["", ...hints.map((c) => `hint (does not block): ${c.id} \u2014 ${c.hint}`)];
+  const total = criteria.length;
+  const passed = criteria.filter((c) => c.passed).length;
+  return v.passed ? `Quality: all ${total} rubric criteria pass.` : `Quality: ${passed}/${total} rubric criteria pass \u2014 see the coaching above.`;
 }
 function formatRubricCoaching(result2) {
   const { verdict, rogerMartin } = result2;
   if (!verdict || verdict.criteria.length === 0) return "";
   const lines = ["## Semantic Quality"];
-  const gating = (verdict.criteria ?? []).filter((c) => c.gating !== false);
-  const failed = gating.filter((c) => !c.passed);
-  const total = gating.length;
+  const failed = (verdict.criteria ?? []).filter((c) => !c.passed);
+  const total = verdict.criteria?.length ?? 0;
   const passedCount = total - failed.length;
   if (verdict.passed) {
-    lines.push(`All ${total} gating criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier).`);
+    lines.push(`All ${total} rubric criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier).`);
   } else {
-    lines.push(`${passedCount}/${total} gating criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier)`);
+    lines.push(`${passedCount}/${total} criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier)`);
     lines.push("");
-    for (const c of gating) {
+    for (const c of verdict.criteria) {
       const icon = c.passed ? "[x]" : "[ ]";
       const extra = c.passed ? "" : ` \u2014 ${c.hint}`;
       lines.push(`${icon} ${c.id}${extra}`);
@@ -1204,7 +1190,6 @@ function formatRubricCoaching(result2) {
       lines.push(`_Question to consider:_ ${verdict.weakest.questionTemplate}`);
     }
   }
-  lines.push(...hintLines(verdict.criteria));
   if (rogerMartin) {
     lines.push("");
     lines.push("### Roger Martin Test");
@@ -1222,18 +1207,17 @@ function formatRubricCoaching(result2) {
 function formatRubricVerdictSection(verdict) {
   if (!verdict || !verdict.criteria || verdict.criteria.length === 0) return "";
   const lines = ["## Semantic Quality"];
-  const gating = verdict.criteria.filter((c) => c.gating !== false);
-  const failed = gating.filter((c) => !c.passed);
-  const total = gating.length;
+  const failed = verdict.criteria.filter((c) => !c.passed);
+  const total = verdict.criteria.length;
   const passedCount = total - failed.length;
   const durationSuffix = verdict.llmDurationMs ? ` in ${(verdict.llmDurationMs / 1e3).toFixed(1)}s` : "";
   const statusNote = verdict.llmStatus === "pending" ? " \u2014 LLM evaluation in progress..." : verdict.llmStatus === "failed" ? ` \u2014 LLM evaluation failed${verdict.llmError ? `: ${verdict.llmError}` : ""}, showing heuristic results` : verdict.source === "llm" && durationSuffix ? ` \u2014 evaluated${durationSuffix}` : "";
   if (failed.length === 0) {
-    lines.push(`All ${total} gating criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier, ${verdict.source} evaluation).${statusNote}`);
+    lines.push(`All ${total} rubric criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier, ${verdict.source} evaluation).${statusNote}`);
   } else {
-    lines.push(`${passedCount}/${total} gating criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier, ${verdict.source} evaluation)${statusNote}`);
+    lines.push(`${passedCount}/${total} criteria pass for \`${verdict.canonicalKey}\` (${verdict.tier} tier, ${verdict.source} evaluation)${statusNote}`);
     lines.push("");
-    for (const c of gating) {
+    for (const c of verdict.criteria) {
       const icon = c.passed ? "[x]" : "[ ]";
       const extra = c.passed ? "" : ` \u2014 ${c.hint}`;
       lines.push(`${icon} ${c.id}${extra}`);
@@ -1243,7 +1227,6 @@ function formatRubricVerdictSection(verdict) {
       lines.push(`**Top improvement:** ${verdict.weakest.hint}`);
     }
   }
-  lines.push(...hintLines(verdict.criteria));
   if (verdict.rogerMartin) {
     const rm = verdict.rogerMartin;
     lines.push("");
@@ -2311,7 +2294,7 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
     try {
       const wsForTracking = await getWorkspaceContext();
       const v = verdictResult.verdict;
-      const failedCount = (v.criteria ?? []).filter((c) => !c.passed && c.gating !== false).length;
+      const failedCount = (v.criteria ?? []).filter((c) => !c.passed).length;
       trackQualityVerdict(wsForTracking.workspaceId, {
         entry_id: finalEntryId,
         entry_type: v.canonicalKey ?? resolvedCollection,
@@ -2319,7 +2302,7 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
         context: "capture",
         passed: v.passed,
         source: verdictResult.source ?? "heuristic",
-        criteria_total: (v.criteria ?? []).filter((c) => c.gating !== false).length,
+        criteria_total: v.criteria?.length ?? 0,
         criteria_failed: failedCount,
         llm_scheduled: v.tier !== "passive",
         methodology_version: v.methodologyVersion
@@ -2422,15 +2405,6 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
       lines.push(`_Grounding: ${parts.join(", ")}. Review and link if relevant._`);
     }
   }
-  const bodyGuidance = readBodyTemplateGuidance(col.bodyTemplate);
-  if (bodyGuidance) {
-    lines.push("");
-    lines.push(bodyGuidance);
-    trackFieldGuidanceApplied(wsCtx.workspaceId, {
-      collection: resolvedCollection,
-      guided_heading_count: col.bodyTemplate?.length ?? 0
-    });
-  }
   if (formativeHints.length > 0) {
     lines.push("");
     lines.push("## Quality Hints (advisory)");
@@ -2534,8 +2508,7 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
       key: f.key,
       required: f.required === true,
       type: f.type
-    })),
-    ...bodyGuidance ? { guidance: bodyGuidance } : {}
+    }))
   };
   const totalMs = Date.now() - timingStart;
   const timing = {
@@ -3008,7 +2981,7 @@ No DB writes \u2014 call without \`preview:true\` to accept for real.` }],
   if (coachingResult?.verdict) {
     try {
       const v = coachingResult.verdict;
-      const failedCount = (v.criteria ?? []).filter((c) => !c.passed && c.gating !== false).length;
+      const failedCount = (v.criteria ?? []).filter((c) => !c.passed).length;
       trackQualityVerdict(wsCtx.workspaceId, {
         entry_id: entryId,
         entry_type: v.canonicalKey ?? entry.canonicalKey ?? "",
@@ -3016,7 +2989,7 @@ No DB writes \u2014 call without \`preview:true\` to accept for real.` }],
         context: "commit",
         passed: v.passed,
         source: coachingResult.source ?? "heuristic",
-        criteria_total: (v.criteria ?? []).filter((c) => c.gating !== false).length,
+        criteria_total: v.criteria?.length ?? 0,
         criteria_failed: failedCount,
         llm_scheduled: v.tier !== "passive",
         methodology_version: v.methodologyVersion
@@ -5947,6 +5920,13 @@ function formatTimeAgo(ms) {
 // src/tools/collections.ts
 import { z as z17 } from "zod/v3";
 
+// src/lib/bodyTemplate.ts
+function readBodyTemplateGuidance(bodyTemplate) {
+  if (!bodyTemplate || bodyTemplate.length === 0) return void 0;
+  return bodyTemplate.map((entry) => `## ${entry.heading}
+${entry.guidance}`).join("\n\n");
+}
+
 // src/tools/labels.ts
 import { z as z16 } from "zod/v3";
 var labelsSchema = z16.object({
@@ -10540,9 +10520,7 @@ var qualityCheckOutputSchema = z21.object({
   criteria: z21.array(z21.object({
     id: z21.string(),
     passed: z21.boolean(),
-    hint: z21.string().optional(),
-    /** TASK-588 / DEC-1908: false = a coaching hint; an unmet one never fails the verdict. */
-    gating: z21.literal(false).optional()
+    hint: z21.string().optional()
   }))
 });
 var qualityReevaluateOutputSchema = z21.object({
@@ -10672,11 +10650,10 @@ Entry \`${entryId}\` not found. Use \`search\` to find the right ID.`,
     });
   } catch {
   }
-  const gatingCriteria = criteria.filter((c) => c.gating !== false);
-  const passedCount = gatingCriteria.filter((c) => c.passed).length;
+  const passedCount = criteria.filter((c) => c.passed).length;
   return successResult(
     lines.join("\n"),
-    `Quality check for ${entryId}: ${verdict.tier ?? "unknown"} tier, ${passedCount}/${gatingCriteria.length} gating criteria pass.` + (res.stale ? ` STALE \u2014 ${staleWhy}; re-evaluate for a current one.` : ""),
+    `Quality check for ${entryId}: ${verdict.tier ?? "unknown"} tier, ${passedCount}/${criteria.length} criteria pass.` + (res.stale ? ` STALE \u2014 ${staleWhy}; re-evaluate for a current one.` : ""),
     {
       entryId,
       entryFound: true,
@@ -10684,7 +10661,7 @@ Entry \`${entryId}\` not found. Use \`search\` to find the right ID.`,
       stale: res.stale ?? false,
       tier: verdict.tier,
       passed: verdict.passed,
-      criteria: criteria.map((c) => ({ id: c.id, passed: c.passed, hint: c.hint, ...c.gating === false ? { gating: false } : {} }))
+      criteria: criteria.map((c) => ({ id: c.id, passed: c.passed, hint: c.hint }))
     },
     [
       { tool: "relations", description: "Suggest connections", parameters: { action: "suggest", entryId } }
@@ -10723,15 +10700,14 @@ async function handleReEvaluate(entryId, context) {
       context,
       passed: result2.verdict.passed ?? false,
       source: "heuristic",
-      // TASK-588 / DEC-1908: totals and fails count gating criteria; a gating:false one is a hint.
-      criteria_total: result2.verdict.criteria?.filter((c) => c.gating !== false).length ?? 0,
-      criteria_failed: result2.verdict.criteria?.filter((c) => !c.passed && c.gating !== false).length ?? 0,
+      criteria_total: result2.verdict.criteria?.length ?? 0,
+      criteria_failed: result2.verdict.criteria?.filter((c) => !c.passed).length ?? 0,
       llm_scheduled: llmScheduled,
       methodology_version: result2.verdict.methodologyVersion
     });
   } catch {
   }
-  const criteria = (result2.verdict.criteria ?? []).filter((c) => c.gating !== false);
+  const criteria = result2.verdict.criteria ?? [];
   const reEvalScore = criteria.filter((c) => c.passed).length;
   const reEvalMaxScore = criteria.length;
   return {
@@ -14157,4 +14133,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-KKDDIKQU.js.map
+//# sourceMappingURL=chunk-USZ2527R.js.map