@productbrain/mcp 0.0.1-beta.2110 → 0.0.1-beta.2122

package/dist/{chunk-QTY3D7P4.js → chunk-3CONR5XB.js}

@@ -675,117 +675,12 @@ async function isGoverned(slug) {
 }
 
 // src/tools/smart-capture.ts
-var COMMON_CHECKS = {
-  clearName: {
-    id: "clear-name",
-    label: "Clear, specific name (not vague)",
-    check: (ctx) => ctx.name.length > 10 && !["new tension", "new entry", "untitled", "test"].includes(ctx.name.toLowerCase()),
-    suggestion: () => "Rename to something specific \u2014 describe the actual problem or concept."
-  },
-  hasDescription: {
-    id: "has-description",
-    label: "Description provided (>50 chars)",
-    check: (ctx) => ctx.description.length > 50,
-    suggestion: () => "Add a fuller description explaining context and impact."
-  },
-  hasRelations: {
-    id: "has-relations",
-    label: "At least 1 relation created",
-    check: (ctx) => ctx.linksCreated.length >= 1,
-    suggestion: () => "Use `graph action=suggest` and `relations action=create` to add more connections."
-  },
-  diverseRelations: {
-    id: "diverse-relations",
-    label: "Relations span multiple collections",
-    check: (ctx) => {
-      const colls = new Set(ctx.linksCreated.map((l) => l.targetCollection));
-      return colls.size >= 2;
-    },
-    suggestion: () => "Try linking to entries in different collections (glossary, business-rules, strategy)."
-  },
-  hasType: {
-    id: "has-type",
-    label: "Has canonical type",
-    check: (ctx) => !!ctx.data?.canonicalKey || !!ctx.canonicalKey,
-    suggestion: () => "Classify this entry with a canonical type for better context assembly. Use update-entry to set canonicalKey."
-  }
-};
 var GENERIC_PROFILE = {
   governedDraft: false,
   descriptionField: "description",
   defaults: [],
-  recommendedRelationTypes: ["related_to", "references"],
-  qualityChecks: [
-    COMMON_CHECKS.clearName,
-    COMMON_CHECKS.hasDescription,
-    COMMON_CHECKS.hasRelations,
-    COMMON_CHECKS.hasType
-  ]
+  recommendedRelationTypes: ["related_to", "references"]
 };
-var NON_DATA_CRITERION_FIELDS = /* @__PURE__ */ new Set(["grounding", "governanceWhy", "governanceSteering"]);
-function buildFieldQualityChecks(col) {
-  const checks = [];
-  const fields = col.fields ?? [];
-  const seen = /* @__PURE__ */ new Set();
-  const addCheck = (check) => {
-    if (seen.has(check.id)) return;
-    seen.add(check.id);
-    checks.push(check);
-  };
-  const labelFor = (fieldKey) => {
-    return fields.find((field) => field.key === fieldKey)?.label ?? fieldKey;
-  };
-  for (const field of fields) {
-    if (field.required === true) {
-      addCheck({
-        id: `field-${field.key}-required`,
-        label: `${field.key} provided`,
-        check: (ctx) => isNonEmptyValue(ctx.data[field.key]),
-        suggestion: () => `Fill in ${labelFor(field.key)}.`
-      });
-    }
-    if (typeof field.minLength === "number") {
-      addCheck({
-        id: `field-${field.key}-min-length`,
-        label: `${field.key} min ${field.minLength} chars`,
-        check: (ctx) => {
-          const value = ctx.data[field.key];
-          return typeof value === "string" && value.length >= field.minLength;
-        },
-        suggestion: () => `Add more detail to ${labelFor(field.key)} (minimum ${field.minLength} characters).`
-      });
-    }
-    if (typeof field.maxLength === "number") {
-      addCheck({
-        id: `field-${field.key}-max-length`,
-        label: `${field.key} max ${field.maxLength} chars`,
-        check: (ctx) => {
-          const value = ctx.data[field.key];
-          return typeof value !== "string" || value.length <= field.maxLength;
-        },
-        suggestion: () => `Shorten ${labelFor(field.key)} to ${field.maxLength} characters or fewer.`
-      });
-    }
-  }
-  for (const criterion of col.qualityCriteria ?? []) {
-    if (criterion.active === false) continue;
-    if (NON_DATA_CRITERION_FIELDS.has(criterion.field)) continue;
-    const id = criterion.rule === "required" ? `field-${criterion.field}-required` : criterion.rule === "min_length" ? `field-${criterion.field}-min-length` : `quality-${criterion.field}-${criterion.rule}`;
-    addCheck({
-      id,
-      label: `${criterion.field} ${criterion.rule === "required" ? "provided" : criterion.rule === "min_length" ? `min ${criterion.value} chars` : `matches ${criterion.value}`}`,
-      check: (ctx) => {
-        const val = ctx.data[criterion.field];
-        if (criterion.rule === "required") return isNonEmptyValue(val);
-        if (criterion.rule === "min_length") return typeof val === "string" && val.length >= Number(criterion.value ?? 0);
-        if (criterion.rule === "pattern") return typeof val === "string" && new RegExp(criterion.value ?? "").test(val);
-        return true;
-      },
-      suggestion: () => `Fill in ${labelFor(criterion.field)} (${criterion.rule}).`
-    });
-  }
-  return checks;
-}
 async function buildRuntimeProfile(slug) {
   const col = await getCollectionBySlug(slug);
   if (!col) return GENERIC_PROFILE;
@@ -794,20 +689,11 @@ async function buildRuntimeProfile(slug) {
     (field) => field.zone === "body" || field.displayHint === "textarea" || field.displayHint === "section"
   )?.key;
   const descriptionField = col.descriptionFieldKey || layoutDescriptionField || "description";
-  const fieldChecks = buildFieldQualityChecks(col);
-  const qualityChecks = [
-    COMMON_CHECKS.clearName,
-    COMMON_CHECKS.hasDescription,
-    ...fieldChecks,
-    COMMON_CHECKS.hasRelations,
-    COMMON_CHECKS.hasType
-  ];
   return {
     governedDraft: col.governed === true,
     descriptionField,
     defaults: [],
-    recommendedRelationTypes: GENERIC_PROFILE.recommendedRelationTypes,
-    qualityChecks
+    recommendedRelationTypes: GENERIC_PROFILE.recommendedRelationTypes
   };
 }
 var profileCache = new ScopedCache(COLLECTION_CACHE_TTL_MS);
@@ -879,85 +765,6 @@ function inferRelationType(_sourceCollection, _targetCollection, profile) {
   const reason = `profile default (${type})`;
   return { type, reason };
 }
-function scoreQuality(ctx, profile) {
-  const checks = profile.qualityChecks.map((qc) => {
-    const passed2 = qc.check(ctx);
-    return {
-      id: qc.id,
-      label: qc.label,
-      passed: passed2,
-      suggestion: passed2 ? void 0 : qc.suggestion?.(ctx)
-    };
-  });
-  const passed = checks.filter((c) => c.passed).length;
-  const total = checks.length;
-  const score = total > 0 ? Math.round(passed / total * 10) : 10;
-  return { score, maxScore: 10, checks };
-}
-function formatQualityReport(result) {
-  const failed = result.checks.filter((c) => !c.passed);
-  const reason = failed.length > 0 ? ` because ${failed.map((c) => c.suggestion ?? c.label.toLowerCase()).join("; ")}` : "";
-  const lines = [`## Quality: ${result.score}/${result.maxScore}${reason}`];
-  for (const check of result.checks) {
-    const icon = check.passed ? "[x]" : "[ ]";
-    const suggestion = check.passed ? "" : ` \u2014 ${check.suggestion ?? check.label}`;
-    lines.push(`${icon} ${check.label}${suggestion}`);
-  }
-  return lines.join("\n");
-}
-async function checkEntryQuality(entryId) {
-  const entry = await kernelQuery("chain.getEntry", { entryId });
-  if (!entry) {
-    return {
-      text: `Entry \`${entryId}\` not found. Try search to find the right ID.`,
-      quality: { score: 0, maxScore: 10, checks: [] }
-    };
-  }
-  const collections = await getCollections();
-  const collMap = /* @__PURE__ */ new Map();
-  for (const c of collections) collMap.set(c._id, c.slug);
-  const collectionSlug = collMap.get(entry.collectionId) ?? "unknown";
-  const profile = await getProfile(collectionSlug);
-  const relations = await kernelQuery("chain.listEntryRelations", { entryId });
-  const linksCreated = [];
-  for (const r of relations) {
-    const otherId = r.fromId === entry._id ? r.toId : r.fromId;
-    linksCreated.push({
-      targetEntryId: otherId,
-      targetName: "",
-      targetCollection: "",
-      relationType: r.type
-    });
-  }
-  const descField = profile.descriptionField;
-  const description = typeof entry.data?.[descField] === "string" ? entry.data[descField] : "";
-  const ctx = {
-    collection: collectionSlug,
-    name: entry.name,
-    description,
-    data: entry.data ?? {},
-    entryId: entry.entryId ?? "",
-    canonicalKey: entry.canonicalKey,
-    linksCreated,
-    linksSuggested: [],
-    collectionFields: []
-  };
-  const quality = scoreQuality(ctx, profile);
-  const lines = [
-    `# Quality Check: ${entry.entryId ?? entry.name}`,
-    `**${entry.name}** in \`${collectionSlug}\` [${entry.status}]`,
-    "",
-    formatQualityReport(quality)
-  ];
-  if (quality.score < 10) {
-    const failedChecks = quality.checks.filter((c) => !c.passed && c.suggestion);
-    if (failedChecks.length > 0) {
-      lines.push("");
-      lines.push(`_To improve: use \`update-entry\` to fill missing fields, or \`relations action=create\` to add connections._`);
-    }
-  }
-  return { text: lines.join("\n"), quality };
-}
 var AUTO_LINK_CONFIDENCE_THRESHOLD = 35;
 var MAX_AUTO_LINKS = 5;
 var MAX_SUGGESTIONS = 5;
@@ -1269,7 +1076,8 @@ var captureSuccessOutputSchema = z2.object({
   // accept it or a refused response fails to validate on `status` (codex review, completing the
   // coherencyRefusal contract fix).
   status: z2.enum(["draft", "committed", "proposed", "draft_on_failure"]),
-  qualityScore: z2.number(),
+  // WP-480 S1: `qualityScore` (required, client N/10) deleted — a named, accepted
+  // breaking output-contract change. The server verdict stays as `qualityVerdict`.
   qualityVerdict: z2.record(z2.unknown()).optional(),
   classifier: captureClassifierSchema.optional(),
   studioUrl: z2.string().optional(),
@@ -1901,19 +1709,6 @@ Use \`entries action=get\` to inspect the existing entry, or \`update-entry\` to
       if (advisedLinks.length > 0) {
         entryWarnings.push(`${advisedLinks.length} auto-link(s) advised, not written (untypeable) \u2014 accept with \`relations action=create\` to promote.`);
       }
-      const captureCtx = {
-        collection: resolvedCollection,
-        name,
-        description,
-        context,
-        data,
-        entryId: finalEntryId,
-        canonicalKey,
-        linksCreated,
-        linksSuggested,
-        collectionFields: col.fields ?? []
-      };
-      const quality = scoreQuality(captureCtx, profile);
       const tAfterQuality = Date.now();
       const cardinalityWarning = cardinalityCheck?.warning ?? null;
       if (contradictionWarnings.length > 0) {
@@ -2108,13 +1903,6 @@ Use \`entries action=get\` to inspect the existing entry, or \`update-entry\` to
           lines.push(`_Grounding: ${parts.join(", ")}. Review and link if relevant._`);
         }
       }
-      lines.push("");
-      lines.push(formatQualityReport(quality));
-      const failedChecks = quality.checks.filter((c) => !c.passed);
-      if (failedChecks.length > 0) {
-        lines.push("");
-        lines.push(`_To improve: \`update-entry entryId="${finalEntryId}"\` to fill missing fields._`);
-      }
       const fieldGuidanceSection = formatFieldGuidance(col.fields ?? []);
       if (fieldGuidanceSection) {
         lines.push("");
@@ -2194,17 +1982,18 @@ Use \`entries action=get\` to inspect the existing entry, or \`update-entry\` to
         lines.push("");
         lines.push(coachingSection);
       }
+      const hasQualityGaps = verdictResult?.verdict != null && verdictResult.verdict.passed === false || formativeHints.length > 0;
       lines.push("");
       lines.push("## Next Steps");
       const eid = finalEntryId || "(check entry ID)";
       if (finalStatus === "committed") {
         lines.push(`1. **Connect it:** \`graph action=suggest entryId="${eid}"\` \u2014 discover additional links`);
-        if (failedChecks.length > 0) {
+        if (hasQualityGaps) {
           lines.push(`2. **Improve quality:** \`update-entry entryId="${eid}"\` \u2014 fill missing fields`);
         }
       } else if (finalStatus === "proposed") {
         lines.push(`1. **Connect it:** \`graph action=suggest entryId="${eid}"\` \u2014 discover additional links to support the proposal`);
-        if (failedChecks.length > 0) {
+        if (hasQualityGaps) {
           lines.push(`2. **Improve quality:** \`update-entry entryId="${eid}"\` \u2014 strengthen the entry before approval`);
         }
       } else {
@@ -2212,7 +2001,7 @@ Use \`entries action=get\` to inspect the existing entry, or \`update-entry\` to
           lines.push(`1. **Connect it:** \`graph action=suggest entryId="${eid}"\` \u2014 discover what this should link to`);
         }
         lines.push(`${userLinkResults.length === 0 ? "2" : "1"}. **Accept it:** \`commit-entry entryId="${eid}"\` \u2014 promote from draft to SSOT on the Chain`);
-        if (failedChecks.length > 0) {
+        if (hasQualityGaps) {
           lines.push(`${userLinkResults.length === 0 ? "3" : "2"}. **Improve quality:** \`update-entry entryId="${eid}"\` \u2014 fill missing fields`);
         }
       }
@@ -2239,7 +2028,8 @@ Use \`entries action=get\` to inspect the existing entry, or \`update-entry\` to
         }
         next.push({ tool: "commit-entry", description: "Accept onto Chain", parameters: { entryId: finalEntryId } });
       }
-      const summary = finalStatus === "committed" ? `Captured and accepted ${finalEntryId} (${name}) onto ${resolvedCollection}. Quality ${quality.score}/10.` : finalStatus === "proposed" ? `Captured ${finalEntryId} (${name}) in ${resolvedCollection} and created a proposal for acceptance. Quality ${quality.score}/10.` : `Captured ${finalEntryId} (${name}) as draft in ${resolvedCollection}. Quality ${quality.score}/10.`;
+      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 expectedFields = (col.fields ?? []).map((f) => ({
         key: f.key,
         type: f.type,
@@ -2275,7 +2065,8 @@ Use \`entries action=get\` to inspect the existing entry, or \`update-entry\` to
               status: finalStatus,
               // WP-465 surface parity: thread full refusal (routes + mode) for coherency-refused auto-commits.
               ...commitRefusal ? { coherencyRefusal: commitRefusal } : {},
-              qualityScore: quality.score,
+              // 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,
               ...classifierMeta && { classifier: classifierMeta },
               ...studioUrl && { studioUrl },
@@ -3012,6 +2803,16 @@ async function runSemanticConflictPreflight(name, description, collectionHint) {
     return [];
   }
 }
+function verdictQualityPhrase(result) {
+  const v = result?.verdict;
+  const criteria = v?.criteria ?? [];
+  if (!v || criteria.length === 0) {
+    return "Quality: not evaluated (no rubric for this type).";
+  }
+  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(result) {
   const { verdict, rogerMartin } = result;
   if (!verdict || verdict.criteria.length === 0) return "";
@@ -3630,6 +3431,20 @@ function renderWhyLine(e) {
   }
   return { line: null, why, whySourceKey: null };
 }
+function renderVerificationLine(e) {
+  if (!e.verificationStatus) return null;
+  const verifiedBy = typeof e.verifiedBy === "string" ? e.verifiedBy : void 0;
+  const attestation = e.attestation && typeof e.attestation === "object" ? e.attestation : void 0;
+  const strength = attestation && typeof attestation.strength === "string" ? attestation.strength : void 0;
+  const basis = attestation && typeof attestation.basis === "string" ? attestation.basis : void 0;
+  let who;
+  if (basis && basis !== "attested") {
+    who = verifiedBy ? ` (${verifiedBy})` : "";
+  } else {
+    who = verifiedBy ? ` \u2014 by ${verifiedBy}${strength ? ` (${strength})` : ""}` : "";
+  }
+  return `**Verification:** ${String(e.verificationStatus)}${who}`;
+}
 var ENTRIES_ACTIONS = ["list", "get", "batch", "search"];
 var entriesSchema = z4.object({
   action: z4.enum(ENTRIES_ACTIONS).describe(
@@ -3656,6 +3471,14 @@ var entriesGetOutputSchema = z4.object({
   origin: z4.string().optional(),
   originDetail: z4.string().optional(),
   verificationStatus: z4.string().optional(),
+  // Attestation-model finding (PR #341 review): the server's honest verifier label and
+  // derived attestation strength/basis — connectors NEVER re-derive strength (spec §5),
+  // they only render what chain.getEntry ships. Mirrors packages/cli EntryFromApi.
+  verifiedBy: z4.string().optional(),
+  attestation: z4.object({
+    strength: z4.enum(["human-direct", "delegated", "system", "unattested"]),
+    basis: z4.string().optional()
+  }).optional(),
   sourceRef: z4.string().optional(),
   sourceExcerpt: z4.string().optional(),
   why: z4.string().optional(),
@@ -3700,6 +3523,13 @@ var entriesBatchOutputSchema = z4.object({
     origin: z4.string().optional(),
     originDetail: z4.string().optional(),
     verificationStatus: z4.string().optional(),
+    // Attestation-model finding (PR #341 review): mirror entriesGetOutputSchema — batch
+    // entries now carry the same honest verifiedBy/attestation fields.
+    verifiedBy: z4.string().optional(),
+    attestation: z4.object({
+      strength: z4.enum(["human-direct", "delegated", "system", "unattested"]),
+      basis: z4.string().optional()
+    }).optional(),
     sourceRef: z4.string().optional(),
     sourceExcerpt: z4.string().optional(),
     // TEN-2191: mirror entriesGetOutputSchema — batch entries now carry why/whyQuality.
@@ -3786,7 +3616,8 @@ async function handleGet(entryId) {
     const detail = e.originDetail ? ` (${e.originDetail})` : "";
     lines.push(`**Origin:** ${e.origin}${detail}`);
   }
-  if (e.verificationStatus) lines.push(`**Verification:** ${e.verificationStatus}`);
+  const verificationLine = renderVerificationLine(e);
+  if (verificationLine) lines.push(verificationLine);
   if (e.sourceRef) lines.push(`**Source ref:** ${e.sourceRef}`);
   if (e.sourceExcerpt) lines.push(`**Source excerpt:** ${e.sourceExcerpt}`);
   if (epistemic) {
@@ -3837,6 +3668,11 @@ async function handleGet(entryId) {
     ...e.origin ? { origin: String(e.origin) } : {},
     ...e.originDetail ? { originDetail: String(e.originDetail) } : {},
     ...e.verificationStatus ? { verificationStatus: String(e.verificationStatus) } : {},
+    // Attestation-model finding (PR #341 review): forward the server's honest verifier
+    // label + derived attestation strength/basis on the structured payload too — the
+    // agent surface must see the same fields the text render displays (spec §5).
+    ...typeof e.verifiedBy === "string" ? { verifiedBy: e.verifiedBy } : {},
+    ...e.attestation && typeof e.attestation === "object" ? { attestation: e.attestation } : {},
     ...e.sourceRef ? { sourceRef: String(e.sourceRef) } : {},
     ...e.sourceExcerpt ? { sourceExcerpt: String(e.sourceExcerpt) } : {},
     // TEN-2191: surface the WHY and its quality in the structured payload too.
@@ -3897,9 +3733,8 @@ async function handleBatch(entryIds) {
       const detail = entry.originDetail ? ` (${entry.originDetail})` : "";
       lines.push(`**Origin:** ${entry.origin}${detail}`);
     }
-    if (entry.verificationStatus) {
-      lines.push(`**Verification:** ${entry.verificationStatus}`);
-    }
+    const entryVerificationLine = renderVerificationLine(entry);
+    if (entryVerificationLine) lines.push(entryVerificationLine);
     if (entry.sourceRef) {
       lines.push(`**Source ref:** ${entry.sourceRef}`);
     }
@@ -3955,6 +3790,10 @@ async function handleBatch(entryIds) {
       ...entry.origin ? { origin: String(entry.origin) } : {},
       ...entry.originDetail ? { originDetail: String(entry.originDetail) } : {},
       ...entry.verificationStatus ? { verificationStatus: String(entry.verificationStatus) } : {},
+      // Attestation-model finding (PR #341 review): mirror handleGet — forward the
+      // honest verifier label + derived attestation on batch's structured payload too.
+      ...typeof entry.verifiedBy === "string" ? { verifiedBy: entry.verifiedBy } : {},
+      ...entry.attestation && typeof entry.attestation === "object" ? { attestation: entry.attestation } : {},
       ...entry.sourceRef ? { sourceRef: String(entry.sourceRef) } : {},
       ...entry.sourceExcerpt ? { sourceExcerpt: String(entry.sourceExcerpt) } : {},
       ...entry.workflowStatus ? { workflowStatus: String(entry.workflowStatus) } : {},
@@ -5787,8 +5626,10 @@ import { z as z8 } from "zod/v3";
 var COLLECTIONS_ACTIONS = ["list", "create", "update", "describe", "audit", "export"];
 var qualityCriterionSchema = z8.object({
   field: z8.string().describe("Entry data field key this criterion applies to, e.g. 'description', 'owner'"),
-  rule: z8.enum(["required", "min_length", "pattern"]).describe("'required': field must be non-empty (blocks accept). 'min_length': minimum string length (warns). 'pattern': regex match (warns)."),
-  value: z8.string().optional().describe("For min_length: the minimum length as a string integer. For pattern: the regex string. Unused for 'required'.")
+  // WP-480 S1: `max_length` — must mirror the Convex rule union, or the tool rejects a
+  // valid criterion before the request ever reaches the server.
+  rule: z8.enum(["required", "min_length", "max_length", "pattern"]).describe("'required': field must be non-empty (blocks accept). 'min_length': minimum string length (warns). 'max_length': maximum string length (warns). 'pattern': regex match (warns)."),
+  value: z8.string().optional().describe("For min_length/max_length: the length bound as a string integer. For pattern: the regex string. Unused for 'required'.")
 });
 var fieldSchema = z8.object({
   key: z8.string().describe("Field key, e.g. 'description', 'severity', 'status'"),
@@ -7635,20 +7476,24 @@ import { z as z12 } from "zod/v3";
 var QUALITY_ACTIONS = ["check", "re-evaluate"];
 var qualitySchema = z12.object({
   action: z12.enum(QUALITY_ACTIONS).describe(
-    "'check': score an entry against quality criteria. 're-evaluate': trigger fresh evaluation."
+    "'check': read the entry's server quality verdict (tier + criteria). 're-evaluate': trigger fresh evaluation."
   ),
   entryId: z12.string().describe("Entry ID, e.g. 'TEN-graph-db', '<PREFIX>-<n>'"),
   context: z12.enum(["capture", "commit", "review"]).default("review").optional().describe("For re-evaluate: evaluation context")
 });
 var qualityCheckOutputSchema = z12.object({
   entryId: z12.string(),
-  score: z12.number(),
-  maxScore: z12.number(),
+  /** WP-480 S1: false = the ID does not resolve to an entry (typo/deleted) — distinct from "no verdict yet". */
+  entryFound: z12.boolean().optional(),
+  hasVerdict: z12.boolean(),
+  /** WP-480 S1: the verdict was judged against content the entry no longer has — re-evaluate for a current one. */
+  stale: z12.boolean().optional(),
+  tier: z12.string().optional(),
+  passed: z12.boolean().optional(),
   criteria: z12.array(z12.object({
-    name: z12.string(),
-    score: z12.number(),
-    maxScore: z12.number(),
-    met: z12.boolean()
+    id: z12.string(),
+    passed: z12.boolean(),
+    hint: z12.string().optional()
   }))
 });
 var qualityReevaluateOutputSchema = z12.object({
@@ -7663,7 +7508,7 @@ function registerQualityTools(server) {
     "quality",
     {
       title: "Quality",
-      description: "Assess and re-evaluate entry quality. Two actions:\n\n- **check**: Score an entry against collection-specific quality criteria. Returns a scorecard with actionable suggestions. Includes link suggestions when relations are missing.\n- **re-evaluate**: Trigger a fresh quality evaluation. Returns heuristic verdict immediately, schedules LLM evaluation in background.",
+      description: "Assess and re-evaluate entry quality. Two actions:\n\n- **check**: Read the entry's server quality verdict (tier + criteria) \u2014 it does not re-score. Suggests links when the entry has no relations.\n- **re-evaluate**: Trigger a fresh quality evaluation. Returns heuristic verdict immediately, schedules LLM evaluation in background.",
       inputSchema: qualitySchema,
       annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: true, openWorldHint: false }
     },
@@ -7684,70 +7529,100 @@ function registerQualityTools(server) {
   );
   trackWriteTool(qualityTool);
 }
-async function handleCheck(entryId) {
-  const result = await checkEntryQuality(entryId);
-  const needsRelations = result.quality.checks.some(
-    (c) => !c.passed && (c.id === "has-relations" || c.id === "diverse-relations")
-  );
-  if (needsRelations) {
-    try {
-      const suggestions = await kernelQuery("chain.graphSuggestLinks", {
-        entryId,
-        maxHops: 2,
-        limit: 3
-      });
-      if ((suggestions?.suggestions?.length ?? 0) > 0) {
-        const linkHints = (suggestions.suggestions ?? []).map((s) => `  \u2192 \`relations action=create from="${entryId}" to="${s.entryId}" type="${s.recommendedRelationType}"\` \u2014 ${s.name} [${s.collectionSlug}] (${s.score}/100)`).join("\n");
-        result.text += `
-
+async function appendLinkSuggestions(entryId, lines) {
+  let relationCount = -1;
+  try {
+    const relations = await kernelQuery("chain.listEntryRelations", { entryId });
+    relationCount = relations?.length ?? 0;
+  } catch {
+    return;
+  }
+  if (relationCount !== 0) return;
+  try {
+    const suggestions = await kernelQuery("chain.graphSuggestLinks", {
+      entryId,
+      maxHops: 2,
+      limit: 3
+    });
+    if ((suggestions?.suggestions?.length ?? 0) > 0) {
+      const linkHints = (suggestions.suggestions ?? []).map((s) => `  \u2192 \`relations action=create from="${entryId}" to="${s.entryId}" type="${s.recommendedRelationType}"\` \u2014 ${s.name} [${s.collectionSlug}] (${s.score}/100)`).join("\n");
+      lines.push(`
 ## Suggested Links to Improve Quality
-${linkHints}`;
-      }
-    } catch {
+${linkHints}`);
     }
+  } catch {
   }
+}
+async function handleCheck(entryId) {
+  const res = await kernelQuery("quality.getLatestVerdictForEntry", { entryId });
+  if (!res || !res.entryFound) {
+    return successResult(
+      `# Quality Check: ${entryId}
+
+Entry \`${entryId}\` not found. Use \`search\` to find the right ID.`,
+      `Entry ${entryId} not found \u2014 search for the correct ID.`,
+      { entryId, entryFound: false, hasVerdict: false, criteria: [] },
+      [{ tool: "search", description: "Find the entry", parameters: { query: entryId } }]
+    );
+  }
+  const verdict = res.verdict;
+  if (!verdict) {
+    const lines2 = [
+      `# Quality Check: ${entryId}`,
+      "",
+      `No quality verdict yet for \`${entryId}\`. Run \`quality action=re-evaluate entryId="${entryId}"\` to evaluate it.`
+    ];
+    await appendLinkSuggestions(entryId, lines2);
+    return successResult(
+      lines2.join("\n"),
+      `No quality verdict yet for ${entryId} \u2014 re-evaluate to produce one.`,
+      { entryId, entryFound: true, hasVerdict: false, criteria: [] },
+      [{ tool: "quality", description: "Evaluate now", parameters: { action: "re-evaluate", entryId } }]
+    );
+  }
+  const criteria = verdict.criteria ?? [];
+  const lines = [`# Quality Check: ${entryId}`, ""];
+  if (res.stale) {
+    lines.push(
+      `> STALE \u2014 this verdict was judged against an earlier version of \`${entryId}\`, which has been edited since. Run \`quality action=re-evaluate entryId="${entryId}"\` for a current verdict.`,
+      ""
+    );
+  }
+  const verdictSection = formatRubricVerdictSection(verdict);
+  lines.push(verdictSection || `Verdict tier \`${verdict.tier ?? "unknown"}\` for \`${entryId}\` \u2014 no rubric criteria to report.`);
+  await appendLinkSuggestions(entryId, lines);
   try {
-    const verdict = await kernelQuery("quality.getLatestVerdictForEntry", { entryId });
-    if (verdict && verdict.criteria && verdict.criteria.length > 0) {
-      result.text += "\n\n" + formatRubricVerdictSection(verdict);
-      try {
-        const wsForTracking = await getWorkspaceContext();
-        trackQualityCheck(wsForTracking.workspaceId, {
-          entry_id: entryId,
-          entry_type: verdict.canonicalKey ?? "",
-          tier: verdict.tier ?? "",
-          passed: verdict.passed ?? false,
-          source: verdict.source ?? "",
-          llm_status: verdict.llmStatus,
-          llm_duration_ms: verdict.llmDurationMs,
-          llm_error: verdict.llmError,
-          has_roger_martin: !!verdict.rogerMartin
-        });
-      } catch {
-      }
-    }
+    const wsForTracking = await getWorkspaceContext();
+    trackQualityCheck(wsForTracking.workspaceId, {
+      entry_id: entryId,
+      entry_type: verdict.canonicalKey ?? "",
+      tier: verdict.tier ?? "",
+      passed: verdict.passed ?? false,
+      source: verdict.source ?? "",
+      llm_status: verdict.llmStatus,
+      llm_duration_ms: verdict.llmDurationMs,
+      llm_error: verdict.llmError,
+      has_roger_martin: !!verdict.rogerMartin
+    });
   } catch {
   }
-  return {
-    content: [{ type: "text", text: result.text }],
-    structuredContent: success(
-      `Quality check for ${entryId}: ${result.quality.score}/${result.quality.maxScore}.`,
-      {
-        entryId,
-        score: result.quality.score,
-        maxScore: result.quality.maxScore,
-        criteria: result.quality.checks.map((c) => ({
-          name: c.label,
-          score: c.passed ? 1 : 0,
-          maxScore: 1,
-          met: c.passed
-        }))
-      },
-      [
-        { tool: "graph", description: "Suggest connections", parameters: { action: "suggest", entryId } }
-      ]
-    )
-  };
+  const passedCount = criteria.filter((c) => c.passed).length;
+  return successResult(
+    lines.join("\n"),
+    `Quality check for ${entryId}: ${verdict.tier ?? "unknown"} tier, ${passedCount}/${criteria.length} criteria pass.` + (res.stale ? " STALE \u2014 the entry was edited after this verdict; re-evaluate for a current one." : ""),
+    {
+      entryId,
+      entryFound: true,
+      hasVerdict: true,
+      stale: res.stale ?? false,
+      tier: verdict.tier,
+      passed: verdict.passed,
+      criteria: criteria.map((c) => ({ id: c.id, passed: c.passed, hint: c.hint }))
+    },
+    [
+      { tool: "graph", description: "Suggest connections", parameters: { action: "suggest", entryId } }
+    ]
+  );
 }
 async function handleReEvaluate(entryId, context) {
   requireWriteAccess();
@@ -10088,8 +9963,19 @@ function registerVerifyTools(server) {
       annotations: { readOnlyHint: false, destructiveHint: false, openWorldHint: false }
     },
     thinWrapper(async ({ entryId }) => {
-      const result = await kernelMutation("chain.verifyEntry", { entryId });
-      return successResult(`Entry ${entryId} verified.`, `Entry ${entryId} verified.`, result);
+      const agentSessionId = getAgentSessionId();
+      const result = await kernelMutation(
+        "chain.verifyEntry",
+        {
+          entryId,
+          ...agentSessionId ? { sessionId: agentSessionId } : {}
+        }
+      );
+      const basis = result.attestation?.basis;
+      const strength = result.attestation?.strength;
+      const verifiedBySuffix = result.verifiedBy ? basis && basis !== "attested" ? ` (${result.verifiedBy})` : ` \u2014 by ${result.verifiedBy}${strength ? ` (${strength})` : ""}` : "";
+      const message = result.alreadyVerified ? `Entry ${entryId} is already verified${verifiedBySuffix}.` : `Entry ${entryId} verified${verifiedBySuffix}.`;
+      return successResult(message, message, result);
     })
   );
   trackWriteTool(verifyEntryTool);
@@ -14813,4 +14699,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-QTY3D7P4.js.map
+//# sourceMappingURL=chunk-3CONR5XB.js.map