@kaddo/mcp 3.37.1 → 3.39.0

package/dist/index.js

@@ -1502,6 +1502,9 @@ function isPlaceholderLine(line) {
   if (/^[-*]\s+(\[[^\]]+\]\s*)?_.+_$/.test(t)) return true;
   if (/^[-*]\s+\[[^\]]+\]\s*$/.test(t)) return true;
   if (/^_(?:Describe|List|Document|What|Which|Who|Use|To be defined|No production code)\b/i.test(t)) return true;
+  if (/<[^>]+>/.test(t)) return true;
+  if (/^[-*]\s+[^:]+:\s*$/.test(t)) return true;
+  if (/^[-*]\s+[^:]+:\s*\S.*\s\|\s/.test(t)) return true;
   return false;
 }
 function isStructuralLine(line, inFrontMatter) {
@@ -1916,10 +1919,10 @@ function extractFromMarkdown(md) {
     }
     const m = line.match(/^\s*(?:[-*]|\d+\.)\s+(.+?)\s*$/);
     if (m) {
-      const raw = m[1].trim();
-      if (!raw || /^_.*_$/.test(raw)) continue;
-      const { text: text2, resolution_status } = parseResolution(raw);
-      if (text2) out.push({ text: text2, resolution_status });
+      const bullet = m[1].trim();
+      if (!bullet || /^_.*_$/.test(bullet)) continue;
+      const { text: text2, resolution_status } = parseResolution(bullet);
+      if (text2) out.push({ text: text2, resolution_status, line: i + 1, raw: line.trim() });
     }
   }
   return out;
@@ -1959,12 +1962,15 @@ function buildOpenQuestionsReport(dir, now = /* @__PURE__ */ new Date()) {
       questions.push({
         id: `OQ-${String(seq).padStart(3, "0")}`,
         source,
+        sourcePath: source,
+        line: eq.line,
+        raw: eq.raw,
         section: "Open Questions",
         question: eq.text,
         classification,
         resolution_status: eq.resolution_status,
         reason,
-        ...eq.resolution_note ? { resolution_note: eq.resolution_note } : {},
+        ...eq.resolution_note ? { resolution_note: eq.resolution_note, note: eq.resolution_note } : {},
         ...classification === "blocking" && eq.resolution_status === "open" ? { suggested_assumption: suggestedAssumption(eq.text) } : {}
       });
     }
@@ -2034,17 +2040,23 @@ function renderOpenQuestionsMarkdown(r) {
   L.push("## Blocking Open Questions", "");
   if (blockingOpen.length === 0) L.push("_None \u2014 nothing blocks readiness._", "");
   for (const q of blockingOpen) {
-    L.push(`### ${q.id}`, "");
-    L.push(`- Source: \`${q.source}\``);
-    L.push(`- Question: ${q.question}`);
+    L.push(`### ${q.question}`, "");
+    L.push(`- Status: ${q.resolution_status}`);
+    L.push(`- Severity: ${q.classification}`);
+    L.push(`- Source: \`${q.sourcePath}:${q.line}\``);
     L.push(`- Reason: ${q.reason}`);
-    if (q.suggested_assumption) L.push(`- Suggested assumption: ${q.suggested_assumption}`);
+    L.push("");
+    L.push("Suggested action:", "");
+    L.push("```md");
+    L.push(`- [assumed] ${q.question}`);
+    L.push(`  - note: ${q.suggested_assumption ?? "<editable assumption for the MVP>"}`);
+    L.push("```");
     L.push("");
   }
   const renderStatusList = (title, items) => {
     L.push(`## ${title}`, "");
     if (items.length === 0) L.push("_None._");
-    for (const q of items) L.push(`- ${q.question}${q.resolution_note ? ` \u2014 _${q.resolution_note}_` : ""}`);
+    for (const q of items) L.push(`- ${q.question} \u2014 \`${q.sourcePath}:${q.line}\`${q.note ? ` \u2014 _${q.note}_` : ""}`);
     L.push("");
   };
   renderStatusList("Resolved", r.resolved_questions);
@@ -2281,10 +2293,11 @@ function resolveNextStep(dir, now = /* @__PURE__ */ new Date()) {
   if (!exists(join(dir, ".kaddo", "understand.md"))) {
     return { id: "understand", phase: "Discovery", label: "Run `kaddo understand` to summarize the project context.", command: "kaddo understand", reason: "No understand handoff has been generated yet." };
   }
-  const refine = (id, agent, target, quality) => ({
+  const discovery = state === "pre-ai" || state === "legacy";
+  const refine = (id, agent, target, quality, verb = "complete") => ({
     id,
     phase: "Knowledge Refinement",
-    label: `Use ${agent} to complete \`${target}\`.`,
+    label: `Use ${agent} to ${verb} \`${target}\`.`,
     agent,
     target,
     reason: `${target} is ${quality === "missing" ? "missing" : "still a bootstrap placeholder or too thin"}.`,
@@ -2296,8 +2309,10 @@ function resolveNextStep(dir, now = /* @__PURE__ */ new Date()) {
   });
   if (qBusiness !== "useful") return refine("refine-business", "business-agent", B, qBusiness);
   if (qProduct !== "useful" || qCap !== "useful") {
-    const target = qCap !== "useful" ? CAP : P;
-    return refine("refine-product", "capability-agent", target, qCap !== "useful" ? qCap : qProduct);
+    if (qCap !== "useful") {
+      return refine("refine-product", "capability-agent", CAP, qCap, discovery ? "discover and document existing system capabilities in" : "complete");
+    }
+    return refine("refine-product", "capability-agent", P, qProduct);
   }
   if (qCurrentState !== "useful") return refine("refine-current-state", "architecture-agent", CS, qCurrentState);
   if (qCodebase !== "useful") {
@@ -2306,7 +2321,7 @@ function resolveNextStep(dir, now = /* @__PURE__ */ new Date()) {
   }
   const oq = buildOpenQuestionsReport(dir, now);
   if (oq.summary.blocking_open > 0) {
-    return { id: "questions", phase: "Knowledge Refinement", label: "Resolve, assume or defer the blocking open questions (`kaddo questions`).", command: "kaddo questions", reason: `${oq.summary.blocking_open} blocking open question(s) remain.` };
+    return { id: "questions", phase: "Knowledge Refinement", label: "Run `kaddo questions` to see source locations and resolution guidance, then resolve, assume or defer the blocking open questions.", command: "kaddo questions", reason: `${oq.summary.blocking_open} blocking open question(s) remain.` };
   }
   const roadmap = roadmapSignal(dir);
   if (roadmap !== "has-candidates") {

package/package.json

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