@kaddo/cli 3.25.0 → 3.25.1

package/README.md

@@ -515,6 +515,7 @@ 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) |
 
 **Optional modules (installed with `kaddo add`):**
 

package/dist/index.js

@@ -11656,11 +11656,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 +11687,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 +11733,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 +11785,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 +11808,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}`, "");

package/package.json

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