@kaddo/mcp 3.65.0 → 3.66.0

package/dist/index.js

@@ -521,7 +521,14 @@ function parseArtifact(filePath, raw) {
       sourceId: String(data.source_id ?? ""),
       rawFrontmatter: data,
       decisions: Array.isArray(data.decisions) ? data.decisions.map(String).filter(Boolean) : [],
-      capsules: Array.isArray(data.capsules) ? data.capsules.map(String).filter(Boolean) : []
+      capsules: Array.isArray(data.capsules) ? data.capsules.map(String).filter(Boolean) : [],
+      implementationStatus: String(data.implementation_status ?? ""),
+      validationStatus: String(data.validation_status ?? ""),
+      releaseStatus: String(data.release_status ?? ""),
+      affectedModules: Array.isArray(data.affected_modules) ? data.affected_modules.map(String).filter(Boolean) : [],
+      refinedBy: String(data.refined_by ?? ""),
+      implementedBy: String(data.implemented_by ?? ""),
+      closedBy: String(data.closed_by ?? "")
     };
   } catch {
     return null;
@@ -2413,6 +2420,7 @@ function buildGraph(dir, config, opts = {}, now = /* @__PURE__ */ new Date()) {
   const selectedWIs = workItems.filter((a2) => a2.lifecycle && includedSet.has(a2.lifecycle));
   for (const wi of selectedWIs) {
     const id = wi.id || wi.title;
+    if (!id || !id.trim()) continue;
     const wiNodeId = `wi:${id}`;
     addNode({
       id: wiNodeId,
@@ -2423,16 +2431,19 @@ function buildGraph(dir, config, opts = {}, now = /* @__PURE__ */ new Date()) {
       knowledge_level: wi.knowledgeLevel || void 0
     });
     for (const glob of wi.codeGlobs) {
+      if (!glob || !glob.trim()) continue;
       const codeId = `code:${glob}`;
       addNode({ id: codeId, type: "code-glob", label: glob });
       addEdge(wiNodeId, codeId, "owns");
     }
     for (const cap of wi.capabilities) {
+      if (!cap || !cap.trim()) continue;
       const capId = `capability:${slug(cap) || cap}`;
       addNode({ id: capId, type: "capability", label: cap });
       addEdge(wiNodeId, capId, "implements");
     }
     for (const dec of wi.decisions) {
+      if (!dec || !dec.trim()) continue;
       const adrId = `adr:${dec}`;
       addNode({ id: adrId, type: "decision", label: dec });
       addEdge(wiNodeId, adrId, "depends_on");
@@ -2449,16 +2460,19 @@ function buildGraph(dir, config, opts = {}, now = /* @__PURE__ */ new Date()) {
     }
   }
   for (const adr of all.filter(isAdr)) {
-    const adrId = `adr:${adr.id || adr.title}`;
+    const adrLabel = adr.id || adr.title;
+    if (!adrLabel || !adrLabel.trim()) continue;
+    const adrId = `adr:${adrLabel}`;
     const referenced = nodes.has(adrId);
     if (scope === "all" || referenced) {
       nodes.set(adrId, {
         id: adrId,
         type: "decision",
-        label: `${adr.id} ${adr.title}`.trim() || adr.id || adr.title,
+        label: `${adr.id} ${adr.title}`.trim() || adrLabel,
         path: adr.relPath
       });
       for (const glob of adr.codeGlobs) {
+        if (!glob || !glob.trim()) continue;
         const codeId = `code:${glob}`;
         addNode({ id: codeId, type: "code-glob", label: glob });
         addEdge(adrId, codeId, "governs");
@@ -2514,13 +2528,20 @@ function renderGraphMermaid(graph) {
     safeIds.set(id, candidate);
     return candidate;
   };
-  const escapeLabel = (s) => s.replace(/"/g, "'");
+  const escapeLabel = (s) => s.replace(/"/g, "'").replace(/\[/g, "(").replace(/\]/g, ")").replace(/\n/g, " ");
+  const validNodes = graph.nodes.filter(
+    (n) => n.id && n.id.trim() !== "" && n.label && n.label.trim() !== ""
+  );
+  const validNodeIds = new Set(validNodes.map((n) => n.id));
   const lines = ["flowchart LR"];
-  for (const node of graph.nodes) {
+  for (const node of validNodes) {
     lines.push(`  ${safe(node.id)}["${escapeLabel(node.label)}"]`);
   }
-  if (graph.edges.length > 0) lines.push("");
-  for (const edge of graph.edges) {
+  const validEdges = graph.edges.filter(
+    (e) => validNodeIds.has(e.from) && validNodeIds.has(e.to)
+  );
+  if (validEdges.length > 0) lines.push("");
+  for (const edge of validEdges) {
     lines.push(`  ${safe(edge.from)} -->|${edge.type}| ${safe(edge.to)}`);
   }
   return lines.join("\n") + "\n";
@@ -8030,6 +8051,33 @@ function analyzeMetadataHealth(dir) {
     if (fileDrifted) drifted++;
     else healthy++;
   }
+  const wiDir = join(dir, "knowledge/delivery/work-items");
+  if (exists(wiDir)) {
+    const walkWIs = (d3) => {
+      for (const entry of readDir(d3)) {
+        const p = join(d3, entry);
+        if (isFile(p) && entry.endsWith(".md")) {
+          try {
+            const raw = readFile(p);
+            const fm = matter7(raw).data;
+            if (fm.status === "done") {
+              const rel = p.replace(dir + "/", "").replace(dir + "\\", "").replace(/\\/g, "/");
+              findings.push({
+                file: rel,
+                field: "status",
+                issue: "inconsistent",
+                detail: "Legacy status `done` detected. Canonical status is `completed`."
+              });
+            }
+          } catch {
+          }
+        } else if (!isFile(p) && !entry.startsWith(".")) {
+          walkWIs(p);
+        }
+      }
+    };
+    walkWIs(wiDir);
+  }
   return { findings, healthy, drifted };
 }
 
@@ -8319,7 +8367,30 @@ function buildProjectExplanation(dir) {
     projectRoute: buildProjectRoute(dir),
     scanSignals: loadScanSignals(dir),
     metadataHealth: analyzeMetadataHealth(dir),
-    isModuleRepo: config != null && isModule(config)
+    isModuleRepo: config != null && isModule(config),
+    implementationEvidence: workItemArtifacts.filter((a2) => a2.affectedModules.length > 0).map((a2) => {
+      const fm = a2.rawFrontmatter;
+      const evidence = fm.implementation_evidence;
+      const repos = evidence?.repositories ?? {};
+      const repoEvidence = {};
+      for (const [repoId, data] of Object.entries(repos)) {
+        repoEvidence[repoId] = {
+          role: String(data.role ?? (repoId === "core" ? "core" : "module")),
+          status: String(data.status ?? "unknown")
+        };
+      }
+      return {
+        id: a2.id || a2.title,
+        lifecycle: lifecycleStateOf({ status: a2.status, filePath: a2.filePath }),
+        implementationStatus: a2.implementationStatus || "not-started",
+        validationStatus: a2.validationStatus || "not-started",
+        releaseStatus: a2.releaseStatus || "not-assessed",
+        affectedModules: a2.affectedModules,
+        releaseGates: Array.isArray(fm.release_gates) ? fm.release_gates : [],
+        completionExceptions: Array.isArray(fm.completion_exceptions) ? fm.completion_exceptions : [],
+        repoEvidence
+      };
+    })
   };
 }
 function stateLabel(state) {
@@ -8445,6 +8516,36 @@ function renderExplanationHuman(exp) {
     }
     lines.push("");
   }
+  if (exp.implementationEvidence.length > 0) {
+    lines.push("## Implementation Evidence");
+    for (const ev of exp.implementationEvidence) {
+      lines.push("");
+      lines.push(`### ${ev.id}`);
+      lines.push(`- Lifecycle: ${ev.lifecycle.charAt(0).toUpperCase() + ev.lifecycle.slice(1)}`);
+      lines.push(`- Implementation: ${ev.implementationStatus}`);
+      lines.push(`- Validation: ${ev.validationStatus}`);
+      lines.push(`- Release: ${ev.releaseStatus}`);
+      if (Object.keys(ev.repoEvidence).length > 0) {
+        lines.push("Repositories:");
+        for (const [repoId, info] of Object.entries(ev.repoEvidence)) {
+          lines.push(`- ${repoId} \u2014 ${info.status}`);
+        }
+      } else if (ev.affectedModules.length > 0) {
+        lines.push(`Affected modules: ${ev.affectedModules.join(", ")}`);
+      }
+      const pendingGates = ev.releaseGates.filter((g) => g.status === "blocked" || g.status === "pending");
+      if (pendingGates.length > 0) {
+        lines.push("Pending release gates:");
+        for (const g of pendingGates) lines.push(`- ${g.id}${g.reason ? ` \u2014 ${g.reason}` : ""}`);
+      }
+      const acceptedExceptions = ev.completionExceptions.filter((e) => e.status === "accepted" || e.status === "deferred");
+      if (acceptedExceptions.length > 0) {
+        lines.push("Completion exceptions:");
+        for (const e of acceptedExceptions) lines.push(`- ${e.id}: ${e.status}${e.reason ? ` \u2014 ${e.reason}` : ""}`);
+      }
+    }
+    lines.push("");
+  }
   if (exp.domains.length > 0) {
     lines.push("## Domains");
     lines.push(`- ${exp.domains.join(", ")}`);
@@ -9641,7 +9742,16 @@ function listWorkItems(root) {
       capsules: strArray(data.capsules),
       source,
       domains,
-      readiness
+      readiness,
+      ...data.implementation_status ? { implementation_status: String(data.implementation_status) } : {},
+      ...data.validation_status ? { validation_status: String(data.validation_status) } : {},
+      ...data.release_status ? { release_status: String(data.release_status) } : {},
+      ...Array.isArray(data.affected_modules) && data.affected_modules.length > 0 ? { affected_modules: data.affected_modules.map(String) } : {},
+      ...data.refined_by ? { refined_by: String(data.refined_by) } : {},
+      ...data.implemented_by ? { implemented_by: String(data.implemented_by) } : {},
+      ...data.closed_by ? { closed_by: String(data.closed_by) } : {},
+      ...Array.isArray(data.release_gates) && data.release_gates.length > 0 ? { release_gates: data.release_gates } : {},
+      ...Array.isArray(data.completion_exceptions) && data.completion_exceptions.length > 0 ? { completion_exceptions: data.completion_exceptions } : {}
     });
   }
   return items.sort((a2, b2) => a2.id.localeCompare(b2.id));
@@ -10766,7 +10876,7 @@ var OPERATING_RULES = [
   "Kaddo itself never calls an LLM and never runs git \u2014 every git action is the human\u2019s."
 ];
 function toContextWorkItem(a2) {
-  return {
+  const wi = {
     id: a2.id,
     type: a2.type,
     title: a2.title,
@@ -10776,6 +10886,11 @@ function toContextWorkItem(a2) {
     domains: a2.domains,
     source: parseWorkItemSource(a2.rawFrontmatter)
   };
+  if (a2.implementationStatus) wi.implementationStatus = a2.implementationStatus;
+  if (a2.validationStatus) wi.validationStatus = a2.validationStatus;
+  if (a2.releaseStatus) wi.releaseStatus = a2.releaseStatus;
+  if (a2.affectedModules.length > 0) wi.affectedModules = a2.affectedModules;
+  return wi;
 }
 function toContextArtifact(a2) {
   return { id: a2.id, type: a2.type, title: a2.title, summary: a2.summary, codeGlobs: a2.codeGlobs };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kaddo/mcp",
-  "version": "3.65.0",
+  "version": "3.66.0",
   "description": "MCP server for Kaddo project knowledge: read-only resources/tools/prompts plus safe derived generation under .kaddo/.",
   "license": "MIT",
   "repository": {