@productbrain/mcp 0.0.1-beta.2394 → 0.0.1-beta.2407

package/dist/{chunk-Y52AWERD.js → chunk-DDZACG3Y.js}

@@ -39,7 +39,7 @@ import {
   trackSessionCaptureRate,
   trackWriteBackHintServed,
   trackZeroCaptureAuditFired
-} from "./chunk-DIMIOMXU.js";
+} from "./chunk-PXHW2YGB.js";
 
 // src/server.ts
 import { McpServer as McpServer2 } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -1606,7 +1606,7 @@ ${groundingReport.governance.map((g) => `- **${g.entryId}** ${g.name} [${g.colle
       // accepted shape as the CLI dying between create and re-eval).
       ...links && links.length > 0 ? { deferLLMSchedule: true } : {}
     });
-    if (result2?.preview) {
+    if (result2 && "preview" in result2 && result2.preview) {
       const hasGrounding = groundingReport.related.length > 0 || groundingReport.duplicates.length > 0 || groundingReport.governance.length > 0;
       const groundingSummary = hasGrounding ? `
 
@@ -1647,16 +1647,17 @@ No DB writes \u2014 call without \`preview:true\` to capture for real.${result2.
         structuredContent: previewEnvelope
       };
     }
-    internalId = result2.docId;
-    finalEntryId = result2.entryId;
-    entryWarnings.push(...result2.warnings ?? []);
-    normalizationMeta = result2.normalization;
-    relationSuggestionsFromCreate = result2.relationSuggestions ?? [];
-    formativeHints = result2.qualityHints ?? [];
-    authorityDomainFromCreate = result2.authorityDomain;
-    coachingFromCreate = result2.coaching;
-    resolveGapsForEntry(name, result2.entryId);
-    if (result2.status === "active") {
+    const created = result2;
+    internalId = created.docId;
+    finalEntryId = created.entryId;
+    entryWarnings.push(...created.warnings ?? []);
+    normalizationMeta = created.normalization;
+    relationSuggestionsFromCreate = created.relationSuggestions ?? [];
+    formativeHints = created.qualityHints ?? [];
+    authorityDomainFromCreate = created.authorityDomain;
+    coachingFromCreate = created.coaching;
+    resolveGapsForEntry(name, created.entryId);
+    if (created.status === "active") {
       wasAutoCommittedServerSide = true;
     }
   } catch (error) {
@@ -1932,11 +1933,11 @@ Use \`entries action=get\` to inspect the existing entry, or \`entries action=up
       const refusalReason = coherencyRefusalReason(commitResult);
       if (refusalReason) {
         commitError = `coherency gate refused this accept \u2014 ${refusalReason}`;
-        commitRefusal = commitResult?.refusal ?? null;
+        commitRefusal = commitResultField(commitResult, "refusal") ?? null;
         finalStatus = "draft_on_failure";
       } else {
-        finalStatus = commitResult?.status === "proposal_created" ? "proposed" : "committed";
-        contradictionAdvisory = commitResult?.contradictionAdvisory ?? null;
+        finalStatus = commitResultField(commitResult, "status") === "proposal_created" ? "proposed" : "committed";
+        contradictionAdvisory = commitResultField(commitResult, "contradictionAdvisory") ?? null;
         if (finalStatus === "committed") {
           await recordSessionActivity({ entryModified: internalId });
           trackChainEntryCommitted(wsCtx.workspaceId, {
@@ -2642,11 +2643,11 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
               const refusalReason = coherencyRefusalReason(commitResult);
               if (refusalReason) {
                 commitError = `coherency gate refused this accept \u2014 ${refusalReason}`;
-                commitRefusal = commitResult?.refusal ?? void 0;
+                commitRefusal = commitResultField(commitResult, "refusal") ?? void 0;
                 finalStatus = "draft_on_failure";
               } else {
-                finalStatus = commitResult?.status === "proposal_created" ? "proposed" : "committed";
-                contradictionAdvisory = commitResult?.contradictionAdvisory ?? void 0;
+                finalStatus = commitResultField(commitResult, "status") === "proposal_created" ? "proposed" : "committed";
+                contradictionAdvisory = commitResultField(commitResult, "contradictionAdvisory") ?? void 0;
                 if (finalStatus === "committed") {
                   await recordSessionActivity({ entryModified: internalId });
                   trackChainEntryCommitted(wsCtx.workspaceId, {
@@ -2948,8 +2949,15 @@ _Use \`entries action=move\` to correct any misclassified entries._`);
   };
 }
 function coherencyRefusalReason(commitResult) {
-  if (commitResult?.status !== "coherency_refused") return null;
-  return commitResult.refusal?.reason ?? "refused by the coherency gate";
+  if (!commitResult || typeof commitResult !== "object" || !("status" in commitResult) || commitResult.status !== "coherency_refused") {
+    return null;
+  }
+  const refusal = "refusal" in commitResult ? commitResult.refusal : void 0;
+  return refusal?.reason ?? "refused by the coherency gate";
+}
+function commitResultField(commitResult, field) {
+  if (!commitResult || typeof commitResult !== "object" || !(field in commitResult)) return void 0;
+  return commitResult[field];
 }
 async function recordCommitFailure({
   entryId,
@@ -3362,17 +3370,17 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
     responseLines.push("");
     responseLines.push(deprecationWarning);
   }
-  if (updateResult.normalizationWarnings.length > 0) {
+  if ((updateResult.normalizationWarnings ?? []).length > 0) {
     responseLines.push("");
     responseLines.push("\u26A0\uFE0F **Normalization warnings:**");
-    for (const warning of updateResult.normalizationWarnings) {
+    for (const warning of updateResult.normalizationWarnings ?? []) {
       responseLines.push(`  - ${warning}`);
     }
   }
-  if (updateResult.validationWarnings.length > 0) {
+  if ((updateResult.validationWarnings ?? []).length > 0) {
     responseLines.push("");
     responseLines.push("\u26A0\uFE0F **Validation warnings:**");
-    for (const warning of updateResult.validationWarnings) {
+    for (const warning of updateResult.validationWarnings ?? []) {
       responseLines.push(`  - ${warning}`);
     }
   }
@@ -3394,10 +3402,10 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
           rejected: updateResult.normalization.rejected
         }
       },
-      ...updateResult.normalizationWarnings.length > 0 && {
+      ...(updateResult.normalizationWarnings ?? []).length > 0 && {
         normalizationWarnings: updateResult.normalizationWarnings
       },
-      ...updateResult.validationWarnings.length > 0 && {
+      ...(updateResult.validationWarnings ?? []).length > 0 && {
         validationWarnings: updateResult.validationWarnings
       }
     }, next)
@@ -3412,7 +3420,8 @@ async function handleGetHistory({ entryId }) {
       { entryId, eventCount: 0, events: [] }
     );
   }
-  const formatted = history.map((h) => {
+  const formatted = history.map((rawH) => {
+    const h = rawH;
     const date = new Date(h.timestamp).toISOString();
     const changes = h.changes ? ` \u2014 ${JSON.stringify(h.changes)}` : "";
     const qualifier = h.legacyQualifier ? ` [${h.legacyQualifier}]` : "";
@@ -3971,28 +3980,29 @@ async function handleVerifyChain(server, { collection, mode }) {
     for (const [eid, driftedFields] of driftedByEntry) {
       const entry = scopedEntries.find((e) => (e.entryId ?? e.name) === eid);
       if (!entry?.entryId) continue;
+      const targetEntryId = entry.entryId;
       const updated = (entry.data?.codeMapping ?? []).map(
         (cm) => cm.status === "aligned" && driftedFields.has(cm.field) ? { ...cm, status: "drifted" } : cm
       );
       const updateResult = await kernelMutation("chain.updateEntry", {
-        entryId: entry.entryId,
+        entryId: targetEntryId,
         data: { codeMapping: updated }
       });
       if (updateResult.outcome === "refused") {
         fixRefusals.push({
-          entryId: entry.entryId,
+          entryId: targetEntryId,
           reason: updateResult.refusal?.reason ?? "coherency gate refused this update"
         });
         continue;
       }
       if (updateResult.normalizationWarnings.length > 0 || updateResult.validationWarnings.length > 0) {
         fixUpdateWarnings.push({
-          entryId: entry.entryId,
+          entryId: targetEntryId,
           normalizationWarnings: updateResult.normalizationWarnings,
           validationWarnings: updateResult.validationWarnings
         });
       }
-      fixes.push(entry.entryId);
+      fixes.push(targetEntryId);
     }
   }
   const verified = mappingChecks.filter((c) => c.result === "verified").length;
@@ -5586,7 +5596,7 @@ Use \`relations action=suggest\` to discover potential connections.`
     }
     const byCollection2 = /* @__PURE__ */ new Map();
     for (const entry of result3.context) {
-      const key = entry.collectionName;
+      const key = entry.collectionName ?? "Uncategorized";
       if (!byCollection2.has(key)) byCollection2.set(key, []);
       byCollection2.get(key).push(entry);
     }
@@ -5680,7 +5690,7 @@ _This helps future sessions start with better constraints and context._`
     const hasBindingGovernance = bindingGovernanceCollections.some((bucket) => bucket.entries.length > 0) || result3.bindingGovernance.principles.length > 0 || result3.bindingGovernance.standards.length > 0 || result3.bindingGovernance.businessRules.length > 0;
     const byCollection2 = /* @__PURE__ */ new Map();
     for (const entry of result3.supportingContext) {
-      const key = entry.collectionName;
+      const key = entry.collectionName ?? "Uncategorized";
       if (!byCollection2.has(key)) byCollection2.set(key, []);
       byCollection2.get(key).push(entry);
     }
@@ -5857,7 +5867,7 @@ Use \`relations action=suggest\` to discover potential connections, or \`relatio
   }
   const byCollection = /* @__PURE__ */ new Map();
   for (const entry of result2.related) {
-    const key = entry.collectionName;
+    const key = entry.collectionName ?? "Uncategorized";
     if (!byCollection.has(key)) byCollection.set(key, []);
     byCollection.get(key).push(entry);
   }
@@ -5951,7 +5961,7 @@ Add journey steps via MCP: \`map action=slot.add mapEntryId="${mapEntryId}" slot
     lines.push("");
     const byCollection = /* @__PURE__ */ new Map();
     for (const entry of result2.context) {
-      const key = entry.collectionName;
+      const key = entry.collectionName ?? "Uncategorized";
       if (!byCollection.has(key)) byCollection.set(key, []);
       byCollection.get(key).push(entry);
     }
@@ -8293,7 +8303,9 @@ async function buildOrientResponse(wsCtx, agentSessionId, errors, task) {
   let openTensions = [];
   try {
     const tensions = await kernelQuery("chain.listEntries", { collectionSlug: "tensions" });
-    openTensions = (tensions ?? []).filter((e) => e.workflowStatus === "open");
+    openTensions = (tensions ?? []).filter(
+      (e) => e.workflowStatus === "open"
+    );
   } catch {
   }
   let readiness = null;
@@ -11657,7 +11669,7 @@ async function suggestLinksForEntries(entries) {
   for (const result2 of results) {
     if (result2.status !== "fulfilled") continue;
     const { entry, suggestions: entrySuggestions } = result2.value;
-    for (const s of entrySuggestions.slice(0, 2)) {
+    for (const s of entrySuggestions.filter((x) => x.entryId != null).slice(0, 2)) {
       suggestions.push({
         fromEntryId: entry.entryId,
         toEntryId: s.entryId,
@@ -11710,7 +11722,8 @@ async function runWrapupReview() {
       const verdict = data.qualityVerdicts[draft.entryId];
       const conns = data.connectionCounts[draft.entryId] ?? 0;
       const qualityStr = verdict ? `${verdict.passed ? "Pass" : "Needs work"} (${verdict.source})` : "\u2014";
-      const coaching = verdict?.weakest ? verdict.weakest.hint : "\u2014";
+      const weakest = verdict?.weakest;
+      const coaching = weakest ? weakest.hint : "\u2014";
       lines.push(`| \`${draft.entryId}\` ${draft.name} | ${draft.collection} | ${qualityStr} | ${conns} | ${coaching} |`);
     }
     lines.push("");
@@ -11880,15 +11893,16 @@ async function runWrapupCommitAll(data, cachedSuggestions) {
         author: `agent:${sessionId}`,
         sessionId
       });
-      if (result2?.status === "coherency_refused") {
+      if (result2 && "status" in result2 && result2.status === "coherency_refused") {
+        const refusal = "refusal" in result2 ? result2.refusal : void 0;
         results.push({
           entryId: draft.entryId,
           ok: false,
-          error: `coherency gate refused this accept \u2014 ${result2.refusal?.reason ?? "refused by the coherency gate"}`
+          error: `coherency gate refused this accept \u2014 ${refusal?.reason ?? "refused by the coherency gate"}`
         });
         continue;
       }
-      const proposed = result2?.status === "proposal_created";
+      const proposed = !!(result2 && "status" in result2 && result2.status === "proposal_created");
       if (!proposed) {
         await recordSessionActivity({ entryModified: draft.docId });
         trackChainEntryCommitted(wsCtx.workspaceId, {
@@ -13720,13 +13734,13 @@ async function handleMapSuggest({ mapEntryId, slotId, query }) {
   for (const sid of emptySlots) {
     const def = slotDefs.find((s) => s.id === sid);
     const searchQuery = query ?? def?.label ?? sid;
-    const results = await kernelQuery("knowledge.search", {
-      query: searchQuery,
-      limit: 5
+    const allResults = await kernelQuery("chain.searchEntries", {
+      query: searchQuery
     });
-    if (results && results.length > 0) {
+    const results = (allResults ?? []).slice(0, 5);
+    if (results.length > 0) {
       const items = results.map(
-        (r) => `  - **${r.name}** (\`${r.entryId}\`, ${r.collectionName ?? "unknown"}) \u2014 ${r.status}`
+        (r) => `  - **${r.name}** (\`${r.entryId}\`, ${r.collectionSlug ?? "unknown"}) \u2014 ${r.status}`
       );
       suggestions.push(
         `### ${def?.label ?? sid}
@@ -14001,7 +14015,8 @@ async function fetchOrganisationHealth() {
       { fields: ["name", "description", "collectionSlug"] }
     );
     if (!allEntries || allEntries.length === 0) return null;
-    const classifyInput = allEntries.map((e) => ({
+    const projectedEntries = allEntries;
+    const classifyInput = projectedEntries.map((e) => ({
       name: e.name ?? "",
       description: typeof e.description === "string" ? e.description : ""
     }));
@@ -14009,7 +14024,7 @@ async function fetchOrganisationHealth() {
       "chain.batchClassifyHeuristic",
       { entries: classifyInput }
     );
-    return computeOrganisationHealth(allEntries, classifications);
+    return computeOrganisationHealth(projectedEntries, classifications);
   } catch (err) {
     process.stderr.write(`[MCP] fetchOrganisationHealth failed: ${err.message}
 `);
@@ -14914,7 +14929,18 @@ function registerResources(server) {
       return {
         contents: [{
           uri: uri.href,
-          text: buildOrientationMarkdown(collections, trackingEvents, standards, businessRules, principles),
+          // `chain.listEntries`'s real return is a union (raw EntryDoc[] vs Record<string,
+          // unknown>[] depending on field projection, see wrapup.ts's own doc comment on this
+          // recurring residual) — `EntryListItem` only reads a documented subset of fields
+          // present on both union branches; cast rather than widen the render helper's
+          // signature to the full server union (PR #395, Lane C S3b annotation pass).
+          text: buildOrientationMarkdown(
+            collections,
+            trackingEvents,
+            standards,
+            businessRules,
+            principles
+          ),
           mimeType: "text/markdown"
         }]
       };
@@ -15124,7 +15150,7 @@ ${entry.labels.map((l) => `- ${l.name ?? l.slug}`).join("\n")}`);
       ];
       const byCollection = /* @__PURE__ */ new Map();
       for (const entry of result2.related ?? []) {
-        const key = entry.collectionName;
+        const key = entry.collectionName ?? "Uncategorized";
         if (!byCollection.has(key)) byCollection.set(key, []);
         byCollection.get(key).push(entry);
       }
@@ -15227,7 +15253,7 @@ ${entry.labels.map((l) => `- ${l.name ?? l.slug}`).join("\n")}`);
       ];
       for (const entry of results) {
         const id = entry.entryId ? `**${entry.entryId}:** ` : "";
-        const coll = entry.collectionName ?? entry.collectionSlug ?? "";
+        const coll = entry.collectionSlug ?? "";
         lines.push(`- ${id}${entry.name} [${coll}] (${entry.status})`);
       }
       return {
@@ -15646,4 +15672,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-Y52AWERD.js.map
+//# sourceMappingURL=chunk-DDZACG3Y.js.map