@planu/cli 5.3.61 → 5.3.62

package/CHANGELOG.md

@@ -1,3 +1,12 @@
+## [5.3.62] - 2026-08-26
+
+### Bug Fixes
+- fix(SPEC-1639,1640,1641,1642): four dogfood defects in challenge, gates, coverage and sync
+
+### Chores
+- chore(planu): close SPEC-1639, SPEC-1640, SPEC-1641 and SPEC-1642
+
+
 ## [5.3.61] - 2026-08-26
 
 ### Refactoring

package/dist/.planu-build.json

@@ -1 +1 @@
-{"schemaVersion":1,"commit":"6e2fa966b00066e50f9458f35dc2716e3780763f"}
+{"schemaVersion":1,"commit":"653317c22ebdf21ad6b7f82f5d86dd042b05aaab"}

package/dist/engine/challenge-prioritizer.js

@@ -72,6 +72,7 @@ export function buildPrioritizedSummary(top3) {
         return '';
     }
     const lines = top3.map((s) => `${s.rank}. ${s.scenario}`);
-    return `Here are the 3 things you MUST address before coding:\n${lines.join('\n')}`;
+    const noun = top3.length === 1 ? 'thing' : 'things';
+    return `Here are the ${String(top3.length)} ${noun} you MUST address before coding:\n${lines.join('\n')}`;
 }
 //# sourceMappingURL=challenge-prioritizer.js.map

package/dist/tools/challenge-spec.js

@@ -314,7 +314,7 @@ export async function handleChallengeSpec(args, server) {
             totalScenarios: failureScenarios.length,
             actionableScenarios: failureScenariosScored.length,
             suppressedLowRelevanceScenarios: suppressedScenarioCount + concurrencySuppressedCount,
-            shownByDefault: 3,
+            shownByDefault: prioritizedBlocking.length,
             mustAddressBeforeCoding: prioritizedSummary,
             remainingAdvisories: advisoryIdentities,
             gateStatus: gateState.gateStatus,

package/dist/tools/update-status/dod-gates.js

@@ -634,14 +634,20 @@ export async function checkSpecReviewGate(specId, projectId, _forceApprove) {
                 fixHint: 'Resolve the spec review blockers and have planu-spec-reviewer write updated evidence, then retry approval.',
             });
         }
-        if (review.reviewer.kind !== 'spec-review-agent' ||
-            review.reviewer.agent !== 'planu-spec-reviewer') {
+        const specReviewerBlockers = [];
+        if (review.reviewer.kind !== 'spec-review-agent') {
+            specReviewerBlockers.push(`reviewer.kind was ${review.reviewer.kind}. Expected spec-review-agent.`);
+        }
+        if (review.reviewer.agent !== 'planu-spec-reviewer') {
+            specReviewerBlockers.push(`reviewer.agent was ${review.reviewer.agent}. Expected planu-spec-reviewer.`);
+        }
+        if (specReviewerBlockers.length > 0) {
             return specReviewGateError({
                 specId,
                 error: 'spec_review_wrong_reviewer',
                 message: 'Approval requires a dedicated spec reviewer. The implementation reviewer cannot approve the spec plan.',
-                blockers: [`Reviewer was ${review.reviewer.agent}. Expected planu-spec-reviewer.`],
-                fixHint: 'Have planu-spec-reviewer (not the implementation reviewer) write the review artifact, then retry approval.',
+                blockers: specReviewerBlockers,
+                fixHint: 'Have planu-spec-reviewer write the review artifact with reviewer.kind spec-review-agent and reviewer.agent planu-spec-reviewer, then retry approval.',
             });
         }
         return null;
@@ -698,16 +704,20 @@ export async function checkImplementationReviewGate(specId, projectId, _force) {
                 fixHint: 'Resolve the implementation review blockers and have planu-implementation-reviewer write updated evidence, then retry done.',
             });
         }
-        if (review.reviewer.kind !== 'implementation-review-agent' ||
-            review.reviewer.agent !== 'planu-implementation-reviewer') {
+        const implementationReviewerBlockers = [];
+        if (review.reviewer.kind !== 'implementation-review-agent') {
+            implementationReviewerBlockers.push(`reviewer.kind was ${review.reviewer.kind}. Expected implementation-review-agent.`);
+        }
+        if (review.reviewer.agent !== 'planu-implementation-reviewer') {
+            implementationReviewerBlockers.push(`reviewer.agent was ${review.reviewer.agent}. Expected planu-implementation-reviewer.`);
+        }
+        if (implementationReviewerBlockers.length > 0) {
             return implementationReviewGateError({
                 specId,
                 error: 'implementation_review_wrong_reviewer',
                 message: 'Done requires the recognized independent implementation reviewer identity.',
-                blockers: [
-                    `Reviewer was ${review.reviewer.agent}. Expected planu-implementation-reviewer.`,
-                ],
-                fixHint: 'Have planu-implementation-reviewer (with the recognized identity) write the review artifact, then retry done.',
+                blockers: implementationReviewerBlockers,
+                fixHint: 'Have planu-implementation-reviewer write the review artifact with reviewer.kind implementation-review-agent and reviewer.agent planu-implementation-reviewer, then retry done.',
             });
         }
         let implementerActor;

package/dist/tools/update-status/file-sync.js

@@ -250,13 +250,45 @@ function isLeanSpecContent(content) {
  * SPEC-630: Remove done criteria entries from lean spec and annotate the count on the criteria key.
  * Only call this after markAllCriteriaDone — assumes all completed criteria have `done: true`.
  */
+function indentWidth(line) {
+    return /^[ \t]*/.exec(line)?.[0].length ?? 0;
+}
+function isNestedUnder(line, itemIndent) {
+    return line.trim().length === 0 || indentWidth(line) > itemIndent.length;
+}
+function isDoneTrueField(line, fieldIndent) {
+    return new RegExp(`^${fieldIndent}done:[ \\t]*true[ \\t]*(?:#.*)?$`).test(line);
+}
 function stripDoneCriteria(content) {
+    const lines = content.split('\n');
+    const kept = [];
     let doneCount = 0;
-    const stripped = content.replace(/^(\s+- text:.*\n\s+done: true\n?)/gm, () => {
-        doneCount++;
-        return '';
-    });
-    return stripped.replace(/^(criteria:)(\s*#.*)?$/m, `$1 # ${String(doneCount)} done`);
+    let index = 0;
+    while (index < lines.length) {
+        const line = lines[index] ?? '';
+        const itemIndent = /^([ \t]+)- text:/.exec(line)?.[1];
+        if (itemIndent === undefined) {
+            kept.push(line);
+            index += 1;
+            continue;
+        }
+        let end = index + 1;
+        while (end < lines.length && isNestedUnder(lines[end] ?? '', itemIndent)) {
+            end += 1;
+        }
+        const item = lines.slice(index + 1, end);
+        const firstField = item.find((entry) => entry.trim().length > 0);
+        const fieldIndent = firstField === undefined ? '' : (/^[ \t]*/.exec(firstField)?.[0] ?? '');
+        const isDone = firstField !== undefined && item.some((entry) => isDoneTrueField(entry, fieldIndent));
+        if (isDone) {
+            doneCount += 1;
+        }
+        else {
+            kept.push(line, ...item);
+        }
+        index = end;
+    }
+    return kept.join('\n').replace(/^(criteria:)(\s*#.*)?$/m, `$1 # ${String(doneCount)} done`);
 }
 export async function tryReconcile(newStatus, specId, projectId) {
     if (newStatus !== 'done') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planu/cli",
-  "version": "5.3.61",
+  "version": "5.3.62",
   "description": "Planu — MCP Server for Spec Driven Development. Cross-platform (Linux/macOS/Windows, x64/arm64).",
   "type": "module",
   "main": "dist/index.js",

package/planu-plugin.json

@@ -2,7 +2,7 @@
   "name": "dev.planu.cli",
   "displayName": "Planu — Spec Driven Development",
   "description": "Manage software specs, estimations, and autonomous SDD workflows. Language-agnostic MCP server for Claude Code.",
-  "version": "5.3.61",
+  "version": "5.3.62",
   "icon": "assets/plugin/icon.svg",
   "command": ["npx", "@planu/cli@latest"],
   "packageName": "@planu/cli",