@kaddo/cli 3.25.0 → 3.26.0

package/README.md

@@ -515,6 +515,8 @@ create --from roadmap → owners → guard → explain`.
 | v3.23.2 | Impact report defaults to `all` scope (accumulated impact), graph built in memory; `--scope active`; `scope_source`/`default_scope` in JSON |
 | v3.24 | Estimated Savings Model: `kaddo savings` (+ `savings init`, `.kaddo/savings.yml`); evidence-based time/value estimates, confidence; MCP resource + tool |
 | v3.25 | Guard history & drift trend: `kaddo guard --record`, `kaddo drift`; feeds impact Guard Activity + savings Drift Prevention; MCP drift/guard-history resources + tool |
+| v3.25.1 | Savings guard-history messaging fix: distinguishes "no history" from "history with 0 resolved warnings" (drift prevention available at 0 h) |
+| v3.26 | Open-questions readiness gate: `kaddo questions`/`readiness`, blocking/important/deferred classification; MCP open-questions + roadmap-readiness resources + tool; roadmap/work-item/implementation/bootstrap agents check it |
 
 **Optional modules (installed with `kaddo add`):**
 

package/dist/index.js

@@ -696,6 +696,10 @@ var COMMAND_HELP = {
   drift: {
     question: "Are code and knowledge drifting apart over time?",
     next: "Review open warnings; `kaddo guard --record` before commits"
+  },
+  questions: {
+    question: "Are there open questions to decide before the roadmap?",
+    next: "Resolve, assume or defer blocking questions, then generate the roadmap"
   }
 };
 function commandFooterLines(name) {
@@ -2401,6 +2405,21 @@ open question):
 - \`knowledge/knowledge.md\`
 - business priorities
 
+## Readiness Gate (check first)
+
+Before generating the roadmap, check **roadmap readiness** for open questions that affect scope,
+architecture or the MVP. Read \`kaddo://roadmap-readiness\` (MCP) or run \`kaddo questions\`.
+
+If readiness is \`needs_decisions\` (there are **blocking** open questions), do **not** generate the
+roadmap yet. Instead, list the blocking questions, propose reasonable assumptions for each, and ask
+the user to confirm, e.g.:
+
+> Before generating the roadmap I found blocking open questions that affect the MVP scope.
+> I can proceed with these assumptions: \u2026 Confirm and continue?
+
+Only generate the roadmap once the user confirms the assumptions (or resolves/defers the
+questions). Record confirmed assumptions explicitly in the roadmap.
+
 ## Expected Output
 
 A single Markdown artifact intended to be saved as \`knowledge/delivery/roadmap.md\`.
@@ -2709,6 +2728,13 @@ Work Items into clear, traceable units of work.
 You do not write code. You sharpen the problem, validate the Knowledge Level and make the
 Work Item actionable for a human.
 
+## Readiness Gate (check first)
+
+For high-impact Work Items, check \`kaddo://roadmap-readiness\` (or \`kaddo questions\`) for
+**blocking** open questions related to this Work Item's scope. If any are unresolved, surface them
+and propose assumptions for the user to confirm before refining \u2014 don't bake in invisible
+assumptions.
+
 ## When to Use
 
 Use this agent after a roadmap exists (\`knowledge/delivery/roadmap.md\`) or when an existing Work
@@ -3210,6 +3236,13 @@ You are the Kaddo Bootstrap Agent. You guide the transition from business defini
 initial architecture direction, quality attributes, a roadmap and first Work Items for a
 new project. You propose; the human decides.
 
+## Readiness Gate
+
+Bootstrap surfaces \`## Open Questions\` in the knowledge files. Before moving to the roadmap,
+explicitly recommend reviewing them (\`kaddo questions\` / \`kaddo://roadmap-readiness\`) and turning
+**blocking** ones into resolved decisions or confirmed assumptions \u2014 so the roadmap isn't built on
+invisible assumptions.
+
 ## When to Use
 
 Use this agent after \`kaddo bootstrap\` and after the business artifacts are drafted.
@@ -3321,6 +3354,13 @@ may suggest a Git branch, and only by respecting the project's Git strategy.
 You never run Git yourself. The Kaddo CLI never runs Git either. Every git action is the
 human's, and commits/pushes/merges happen only with explicit human confirmation.
 
+## Readiness Gate (check first)
+
+Before implementing a high-impact Work Item, check \`kaddo://roadmap-readiness\` (or
+\`kaddo questions\`) for **blocking** open questions about stack, architecture, persistence,
+authentication or the Work Item's scope. If any are unresolved, pause and ask the user to confirm
+assumptions or resolve them before writing code.
+
 ## When to Use
 
 Use this agent after the work-item-agent has produced a clear, traceable Work Item under
@@ -8162,6 +8202,224 @@ function renderDeliveryLifecycle(wi) {
   ];
 }
 
+// src/core/open-questions.ts
+var SOURCES = [
+  "knowledge/business/business.md",
+  "knowledge/product/product.md",
+  "knowledge/tech/codebase.md",
+  "knowledge/delivery/roadmap.md"
+];
+var BLOCKING = [
+  "mvp",
+  "stack",
+  "arquitect",
+  "architecture",
+  "api",
+  "web app",
+  "webapp",
+  "cli",
+  "backend",
+  "auth",
+  "autenticaci",
+  "persist",
+  "base de datos",
+  "database",
+  "roadmap",
+  "alcance",
+  "scope",
+  "primera versi\xF3n",
+  "primera version",
+  "first version",
+  "fastify",
+  "nestjs",
+  "framework"
+];
+var DEFERRED = [
+  "integraci",
+  "integration",
+  "analytics",
+  "notificaci",
+  "notification",
+  "google calendar",
+  "eventbrite",
+  "meetup",
+  "dashboard",
+  "multi-tenant",
+  "multitenant",
+  "mobile",
+  "m\xF3vil",
+  "pago",
+  "payment",
+  "ticketing",
+  "multiple tracks",
+  "m\xFAltiples tracks",
+  "webhook"
+];
+var IMPORTANT = [
+  "validaci",
+  "validation",
+  "paginaci",
+  "pagination",
+  "soft delete",
+  "sponsor",
+  "agenda",
+  "ubicaci",
+  "location",
+  "rol",
+  "role",
+  "permis",
+  "permission"
+];
+function matchesAny(text3, keywords) {
+  return keywords.some((k) => text3.includes(k));
+}
+function classifyQuestion(question) {
+  const t = question.toLowerCase();
+  if (matchesAny(t, BLOCKING)) {
+    return { classification: "blocking", reason: "Affects scope, architecture or the first Work Items." };
+  }
+  if (matchesAny(t, DEFERRED) && !matchesAny(t, IMPORTANT)) {
+    return { classification: "deferred", reason: "Can be moved to a later phase." };
+  }
+  if (matchesAny(t, IMPORTANT)) {
+    return { classification: "important", reason: "Relevant, but a temporary assumption can unblock progress." };
+  }
+  return { classification: "important", reason: "Relevant, but a temporary assumption can unblock progress." };
+}
+function extractFromMarkdown(md) {
+  const lines = md.split(/\r?\n/);
+  const start = lines.findIndex((l) => /^#{1,6}\s+open questions\s*$/i.test(l.trim()) || /^#{1,6}\s+preguntas abiertas\s*$/i.test(l.trim()));
+  if (start < 0) return [];
+  const out = [];
+  for (let i = start + 1; i < lines.length; i++) {
+    const line = lines[i];
+    if (/^#{1,6}\s+/.test(line)) break;
+    const m = line.match(/^\s*(?:[-*]|\d+\.)\s+(.+?)\s*$/);
+    if (m) {
+      const q = m[1].trim();
+      if (q && !/^_.*_$/.test(q)) out.push(q);
+    }
+  }
+  return out;
+}
+function suggestedAssumption(question) {
+  const t = question.toLowerCase();
+  if (matchesAny(t, ["api", "web app", "webapp", "cli", "backend"])) {
+    return "Start as a backend API to keep the initial scope small.";
+  }
+  if (matchesAny(t, ["auth", "autenticaci"])) {
+    return "No authentication in the MVP.";
+  }
+  if (matchesAny(t, ["fastify", "nestjs", "framework", "stack"])) {
+    return "Choose the lighter stack option to minimize MVP scope.";
+  }
+  if (matchesAny(t, ["persist", "base de datos", "database"])) {
+    return "Use a simple local datastore (e.g. SQLite) for the MVP.";
+  }
+  return "Proceed with the smallest reasonable MVP interpretation and revisit later.";
+}
+function buildOpenQuestionsReport(dir, now = /* @__PURE__ */ new Date()) {
+  const project = loadConfig(dir)?.project.name ?? "unknown";
+  const questions = [];
+  let seq = 0;
+  for (const source of SOURCES) {
+    const p2 = join(dir, source);
+    if (!exists(p2)) continue;
+    let md;
+    try {
+      md = readFile(p2);
+    } catch {
+      continue;
+    }
+    for (const q of extractFromMarkdown(md)) {
+      seq += 1;
+      const { classification, reason } = classifyQuestion(q);
+      questions.push({
+        id: `OQ-${String(seq).padStart(3, "0")}`,
+        source,
+        section: "Open Questions",
+        question: q,
+        classification,
+        reason,
+        ...classification === "blocking" ? { suggested_assumption: suggestedAssumption(q) } : {}
+      });
+    }
+  }
+  const blocking = questions.filter((q) => q.classification === "blocking");
+  const important = questions.filter((q) => q.classification === "important");
+  const deferred = questions.filter((q) => q.classification === "deferred");
+  let readiness;
+  if (questions.length === 0) readiness = "unknown";
+  else if (blocking.length > 0) readiness = "needs_decisions";
+  else readiness = "ready";
+  const suggested_assumptions = blocking.map((q) => q.suggested_assumption).filter(Boolean);
+  const recommended_next_step = readiness === "needs_decisions" ? "Confirm suggested assumptions with the user (resolve, assume or defer blocking questions) before generating the roadmap." : readiness === "ready" ? "No blocking questions \u2014 you can proceed to the roadmap (review important questions when convenient)." : "No open questions found. Run `kaddo bootstrap` to capture early questions, or proceed.";
+  return {
+    generated_at: now.toISOString(),
+    project,
+    summary: {
+      open_questions: questions.length,
+      roadmap_readiness: readiness,
+      blocking: blocking.length,
+      important: important.length,
+      deferred: deferred.length
+    },
+    questions,
+    blocking_questions: blocking,
+    important_questions: important,
+    deferred_questions: deferred,
+    suggested_assumptions,
+    recommended_next_step
+  };
+}
+function renderOpenQuestionsMarkdown(r) {
+  const L = [];
+  L.push("# Kaddo Open Questions Readiness Report", "");
+  L.push(`Generated at: ${r.generated_at}`);
+  L.push(`Project: ${r.project}`);
+  L.push("");
+  L.push("## Summary", "");
+  L.push(`- Open questions found: ${r.summary.open_questions}`);
+  L.push(`- Roadmap readiness: ${r.summary.roadmap_readiness}`);
+  L.push(`- Blocking: ${r.summary.blocking}`);
+  L.push(`- Important: ${r.summary.important}`);
+  L.push(`- Deferred: ${r.summary.deferred}`);
+  L.push("");
+  if (r.summary.open_questions === 0) {
+    L.push("No `## Open Questions` sections found in business / product / codebase / roadmap.", "");
+    L.push("## Recommended Next Step", "", r.recommended_next_step, "");
+    return L.join("\n");
+  }
+  L.push("## Blocking Questions", "");
+  if (r.blocking_questions.length === 0) L.push("_None._", "");
+  for (const q of r.blocking_questions) {
+    L.push(`### ${q.id}`, "");
+    L.push(`- Source: \`${q.source}\``);
+    L.push(`- Question: ${q.question}`);
+    L.push(`- Reason: ${q.reason}`);
+    if (q.suggested_assumption) L.push(`- Suggested assumption: ${q.suggested_assumption}`);
+    L.push("");
+  }
+  L.push("## Important Questions", "");
+  if (r.important_questions.length === 0) L.push("_None._");
+  for (const q of r.important_questions) L.push(`- ${q.question}`);
+  L.push("");
+  L.push("## Deferred Questions", "");
+  if (r.deferred_questions.length === 0) L.push("_None._");
+  for (const q of r.deferred_questions) L.push(`- ${q.question}`);
+  L.push("");
+  if (r.suggested_assumptions.length > 0) {
+    L.push("## Suggested Assumptions", "");
+    for (const a of r.suggested_assumptions) L.push(`- ${a}`);
+    L.push("");
+  }
+  L.push("## Recommended Next Step", "", r.recommended_next_step, "");
+  return L.join("\n");
+}
+function serializeOpenQuestionsJson(r) {
+  return JSON.stringify(r, null, 2) + "\n";
+}
+
 // src/commands/understand.ts
 function runUnderstand() {
   const dir = cwd();
@@ -8234,6 +8492,12 @@ function runUnderstand() {
     console.log("  \u2192 Review graph hints before continuing with implementation.");
     console.log("  Suggested agent: graph-agent (see .kaddo/graph-hints.md)");
   }
+  const oq = buildOpenQuestionsReport(dir);
+  if (oq.summary.blocking > 0) {
+    console.log("");
+    console.log(`Open questions: ${oq.summary.blocking} blocking decision(s) before the roadmap (roadmap readiness: ${oq.summary.roadmap_readiness}).`);
+    console.log("  \u2192 Run `kaddo questions` to review them, then resolve, assume or defer before `kaddo create --from roadmap`.");
+  }
   const active = activeWorkItems(dir);
   if (active.length > 0) {
     console.log("");
@@ -8291,35 +8555,35 @@ var DEP_PATTERNS = [
 function normalizePath2(p2) {
   return p2.split(path5.sep).join("/");
 }
-function matchesAny(file, patterns) {
+function matchesAny2(file, patterns) {
   const normalized = normalizePath2(file);
   return patterns.some((p2) => p2.test(normalized));
 }
 function detectSignals(touchedFiles) {
   const signals = [];
   for (const file of touchedFiles) {
-    if (matchesAny(file, MIGRATION_PATTERNS)) {
+    if (matchesAny2(file, MIGRATION_PATTERNS)) {
       signals.push({
         name: "DB migration",
         file,
         implication: "Data schema change \u2014 typically K4 or Migration type",
         suggestedLevel: "K4"
       });
-    } else if (matchesAny(file, CONTRACT_PATTERNS)) {
+    } else if (matchesAny2(file, CONTRACT_PATTERNS)) {
       signals.push({
         name: "API/event contract",
         file,
         implication: "Contract change affects consumers \u2014 typically K3/K4",
         suggestedLevel: "K3"
       });
-    } else if (matchesAny(file, INFRA_PATTERNS)) {
+    } else if (matchesAny2(file, INFRA_PATTERNS)) {
       signals.push({
         name: "Infrastructure",
         file,
         implication: "Infrastructure change \u2014 typically K3/K4",
         suggestedLevel: "K3"
       });
-    } else if (matchesAny(file, DEP_PATTERNS)) {
+    } else if (matchesAny2(file, DEP_PATTERNS)) {
       signals.push({
         name: "Dependency",
         file,
@@ -11656,11 +11920,23 @@ function buildSavingsReport(dir, opts = {}, now = /* @__PURE__ */ new Date()) {
   const graphMult = GRAPH_MULT[graphQuality] ?? 0;
   const archH = round2(a.architecture_discovery_hours_saved_when_graph_good * graphMult);
   const guard = buildGuardHistory(dir);
-  const driftSavings = guard.available && guard.resolved > 0 ? {
-    available: true,
-    hours: round2(guard.resolved * a.rework_hours_avoided_per_resolved_drift),
-    formula: `${guard.resolved} resolved drift warnings \xD7 ${a.rework_hours_avoided_per_resolved_drift} h`
-  } : { available: false, hours: 0, reason: "Guard history is not persisted yet." };
+  let driftSavings;
+  if (!guard.available) {
+    driftSavings = { available: false, hours: 0, reason: "Guard history is not persisted yet." };
+  } else if (guard.resolved > 0) {
+    driftSavings = {
+      available: true,
+      hours: round2(guard.resolved * a.rework_hours_avoided_per_resolved_drift),
+      formula: `${guard.resolved} resolved drift warnings \xD7 ${a.rework_hours_avoided_per_resolved_drift} h`
+    };
+  } else {
+    driftSavings = {
+      available: true,
+      hours: 0,
+      formula: `0 resolved drift warnings \xD7 ${a.rework_hours_avoided_per_resolved_drift} h`,
+      reason: "Guard history exists, but no resolved drift warnings have been recorded yet."
+    };
+  }
   const totalHours = round2(ctxPrepH + reviewH + clarH + onboardH + archH + driftSavings.hours);
   const totalValue = Math.round(totalHours * a.hourly_cost);
   const reasons = [];
@@ -11675,15 +11951,27 @@ function buildSavingsReport(dir, opts = {}, now = /* @__PURE__ */ new Date()) {
     level = "Medium";
     reasons.push("Strong knowledge and graph evidence.");
   }
-  if (!guard.available) reasons.push("Guard history is not persisted yet \u2014 avoided rework is not estimated.");
-  else reasons.push(`Guard history available (${guard.resolved} resolved drift warning(s)).`);
+  if (!guard.available) {
+    reasons.push("Guard history is not persisted yet \u2014 avoided rework is not estimated.");
+  } else if (guard.resolved > 0) {
+    reasons.push(`Guard history available (${guard.resolved} resolved drift warning(s)).`);
+  } else {
+    reasons.push("Guard history available, but no resolved drift warnings have been recorded yet.");
+  }
   reasons.push(source === "file" ? "Assumptions come from `.kaddo/savings.yml`." : "Savings assumptions are defaults and should be calibrated.");
   const actions = [];
   if (source === "default") actions.push("Run `kaddo savings init` to customize assumptions.");
   actions.push("Calibrate hourly cost and team assumptions with real team data.");
   actions.push("Run `kaddo impact` to verify the underlying evidence quality.");
   actions.push("Use this report as directional evidence, not accounting data.");
-  actions.push("Future: persist Guard history to estimate avoided rework.");
+  if (!guard.available) {
+    actions.push("Future: persist Guard history to estimate avoided rework.");
+  } else if (guard.resolved === 0) {
+    actions.push("Continue running `kaddo guard --record` before important commits to capture resolved drift warnings.");
+    if (guard.open > 0) actions.push("Review open drift warnings with `kaddo drift`.");
+  } else {
+    actions.push("Use drift history to calibrate `rework_hours_avoided_per_resolved_drift`.");
+  }
   return {
     generated_at: now.toISOString(),
     project: impact.project,
@@ -11709,7 +11997,9 @@ function buildSavingsReport(dir, opts = {}, now = /* @__PURE__ */ new Date()) {
       graph_nodes: impact.graph_quality.available ? impact.graph_quality.nodes : null,
       graph_edges: impact.graph_quality.available ? impact.graph_quality.edges : null,
       context_readiness: readiness,
-      guard_history_available: guard.available
+      guard_history_available: guard.available,
+      guard_runs_recorded: guard.total_runs,
+      resolved_drift_warnings: guard.resolved
     },
     estimated_savings: {
       context_preparation: { hours: ctxPrepH, formula: `${completed} Work Items \xD7 ${a.context_preparation_minutes_saved_per_work_item} min` },
@@ -11759,7 +12049,13 @@ function renderSavingsMarkdown(r) {
   if (e.graph_nodes !== null) L.push(`- Graph nodes: ${e.graph_nodes}`);
   if (e.graph_edges !== null) L.push(`- Graph edges: ${e.graph_edges}`);
   L.push(`- Context readiness: ${e.context_readiness}`);
-  L.push("- Guard history: not available");
+  if (e.guard_history_available) {
+    L.push("- Guard history: available");
+    L.push(`- Guard runs recorded: ${e.guard_runs_recorded}`);
+    L.push(`- Resolved drift warnings: ${e.resolved_drift_warnings}`);
+  } else {
+    L.push("- Guard history: not available");
+  }
   L.push("");
   L.push("## Estimated Savings", "");
   const s = r.estimated_savings;
@@ -11776,7 +12072,9 @@ function renderSavingsMarkdown(r) {
   L.push("### Drift Prevention", "");
   if (s.drift_prevention.available) {
     L.push(`- Formula: ${s.drift_prevention.formula}`);
-    L.push(`- Estimated: ${s.drift_prevention.hours} h`, "");
+    L.push(`- Estimated: ${s.drift_prevention.hours} h`);
+    if (s.drift_prevention.reason) L.push(`- Reason: ${s.drift_prevention.reason}`);
+    L.push("");
   } else {
     L.push("- Not available yet.");
     L.push(`- Reason: ${s.drift_prevention.reason}`, "");
@@ -11961,6 +12259,42 @@ function runDrift(opts = {}) {
   console.log(content);
 }
 
+// src/commands/questions.ts
+function runQuestions(opts = {}) {
+  const dir = cwd();
+  requireConfig(dir);
+  const report = buildOpenQuestionsReport(dir);
+  if (opts.output) {
+    intro2("kaddo questions");
+    writeFile(join(dir, opts.output), opts.json ? serializeOpenQuestionsJson(report) : renderOpenQuestionsMarkdown(report));
+    log2.success(`Wrote ${opts.output.replace(/\\/g, "/")}`);
+    printCommandFooter("questions");
+    outro2("Questions report written.");
+    return;
+  }
+  if (opts.json) {
+    console.log(serializeOpenQuestionsJson(report));
+    return;
+  }
+  console.log("");
+  console.log(`Open questions detected: ${report.summary.open_questions}`);
+  console.log(`Roadmap readiness: ${report.summary.roadmap_readiness === "needs_decisions" ? "needs decisions" : report.summary.roadmap_readiness}`);
+  const list2 = (title, qs) => {
+    if (qs.length === 0) return;
+    console.log("");
+    console.log(`${title}:`);
+    for (const q of qs.slice(0, 8)) console.log(`- ${q.question}`);
+    if (qs.length > 8) console.log(`  \u2026and ${qs.length - 8} more`);
+  };
+  list2("Blocking", report.blocking_questions);
+  list2("Important", report.important_questions);
+  list2("Deferred", report.deferred_questions);
+  console.log("");
+  console.log("Suggested next:");
+  console.log(report.recommended_next_step);
+  printCommandFooter("questions");
+}
+
 // src/index.ts
 var require2 = createRequire(import.meta.url);
 var { version } = require2("../package.json");
@@ -12011,6 +12345,9 @@ reportCmd.command("drift").description("Drift Trend Report from recorded guard h
 program.command("drift").description("Drift Trend Report from recorded `kaddo guard --record` history").option("--json", "Output JSON instead of Markdown").option("--output <path>", "Write the report to a file").action((opts) => {
   runDrift(opts);
 });
+var questionsAction = (opts) => runQuestions(opts);
+program.command("questions").description("Open-questions readiness gate: blocking/important/deferred decisions before the roadmap").option("--json", "Output JSON instead of a summary").option("--output <path>", "Write the report to a file (e.g. .kaddo/reports/questions-report.md)").action(questionsAction);
+program.command("readiness").description("Alias for `kaddo questions`").option("--json", "Output JSON instead of a summary").option("--output <path>", "Write the report to a file").action(questionsAction);
 program.command("guard").description("Check if modified code has related artifacts that were not updated").option("--staged", "Check only staged files").option("--no-interactive", "Disable interactive ignore prompts").option("--ci", "CI mode: output JSON, no prompts, non-blocking").option("--json", "Output JSON (alias for --ci)").option("--workspace", "Also check local mapped module repos from .kaddo/modules.yml (opt-in)").option("--include-archived", "Include archived Work Items in ownership matching (excluded by default)").option("--record", "Record this run to .kaddo/history/ for drift trend reporting").action(async (opts) => {
   await runGuard(opts);
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kaddo/cli",
-  "version": "3.25.0",
+  "version": "3.26.0",
   "description": "Knowledge Driven Development toolkit",
   "license": "MIT",
   "repository": {