@planu/cli 5.3.61 → 5.3.63

package/CHANGELOG.md

@@ -1,3 +1,35 @@
+## [5.3.63] - 2026-08-27
+
+### Bug Fixes
+- fix(SPEC-1644): stop emitting the default architecture rule and every line-cap instruction
+- fix(SPEC-1643): remove the empty typescript-patterns builtin skill
+- fix(SPEC-1371): compute branch cleanup against main with literal protected matching
+- fix(SPEC-1646): discriminate contradiction findings by offending criterion
+- fix(SPEC-1646): discriminate contradiction findings by offending criterion
+- fix(SPEC-1645): widen the load-bearing separator class per dual review
+- fix(SPEC-1645): stop treating load-bearing as a scale signal
+
+### Chores
+- chore(planu): file goal stop-hook loop dogfood bug
+- chore(SPEC-1644): record done transition
+- chore(planu): file SPEC-1651 and SPEC-1652 dogfood bugs
+- chore(SPEC-1644): record approved status
+- chore(SPEC-1643): mark acceptance criteria done
+- chore(SPEC-1643): lifecycle transition to approved
+- chore(planu): record SPEC-1371 approval and SPEC-1649 filing
+- chore(SPEC-1646): record approval transition
+- chore(SPEC-1645): record done transition
+
+
+## [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":"ae29e281086a9333b51c28dc6fd1c297538efe4d"}

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/engine/rules-generator/index.js

@@ -27,11 +27,9 @@ export const ARCHITECTURE_RULES = {
         ],
         conventions: [
             'Never put business logic in route handlers',
-            'Route handlers should be under 30 lines',
             'Services are framework-agnostic (no NextRequest/NextResponse)',
             'Repositories return plain objects, never ORM entities',
         ],
-        maxLinesPerHandler: 30,
     },
     express: {
         title: 'Express Architecture',
@@ -58,7 +56,6 @@ export const ARCHITECTURE_RULES = {
             'Services are stateless',
             'One file per resource',
         ],
-        maxLinesPerHandler: 40,
     },
     fastapi: {
         title: 'FastAPI Architecture',
@@ -85,34 +82,6 @@ export const ARCHITECTURE_RULES = {
             'Pydantic models for all I/O',
             'Async by default',
         ],
-        maxLinesPerHandler: 30,
-    },
-    default: {
-        title: 'Clean Architecture',
-        description: 'Generic clean architecture rules',
-        layers: [
-            {
-                name: 'Handler',
-                path: 'src/handlers/',
-                responsibility: 'Input/output boundary',
-            },
-            {
-                name: 'Service',
-                path: 'src/services/',
-                responsibility: 'Business logic',
-            },
-            {
-                name: 'Repository',
-                path: 'src/repositories/',
-                responsibility: 'Data access',
-            },
-        ],
-        conventions: [
-            'Separate concerns by layer',
-            'Dependencies point inward',
-            'No framework code in business logic',
-        ],
-        maxLinesPerHandler: 50,
     },
 };
 /** Detect stack from project files. Returns a key into ARCHITECTURE_RULES. */
@@ -187,9 +156,6 @@ export function generateRulesContent(rule, tool) {
                 '## Conventions',
                 conventions,
                 '',
-                `## Handler Size Limit`,
-                `Max ${rule.maxLinesPerHandler} lines per handler/controller.`,
-                '',
             ].join('\n');
         }
         case 'copilot': {
@@ -205,9 +171,6 @@ export function generateRulesContent(rule, tool) {
                 '## Coding Conventions',
                 conventions,
                 '',
-                `## Size Constraints`,
-                `- Max ${rule.maxLinesPerHandler} lines per handler`,
-                '',
             ].join('\n');
         }
         case 'aider': {
@@ -223,9 +186,6 @@ export function generateRulesContent(rule, tool) {
                 '## Conventions',
                 conventions,
                 '',
-                `## Constraints`,
-                `- Handlers/controllers: max ${rule.maxLinesPerHandler} lines`,
-                '',
             ].join('\n');
         }
         case 'claude':
@@ -244,11 +204,6 @@ export function generateRulesContent(rule, tool) {
                 '',
                 conventions,
                 '',
-                '## Size Budget',
-                '',
-                `- Max **${rule.maxLinesPerHandler} lines** per handler/controller`,
-                '- Extract helpers when exceeding this limit',
-                '',
             ].join('\n');
         }
     }
@@ -273,8 +228,10 @@ function resolveRulesFilePath(projectPath, tool, stackKey) {
 /** Main entry: detect stack + tools, generate all rules objects. */
 export function generateProjectRules(projectPath) {
     const stackKey = detectArchitectureStack(projectPath);
-    const defaultRule = ARCHITECTURE_RULES.default;
-    const rule = ARCHITECTURE_RULES[stackKey] ?? defaultRule;
+    if (stackKey === 'default') {
+        return [];
+    }
+    const rule = ARCHITECTURE_RULES[stackKey];
     if (!rule) {
         return [];
     }

package/dist/engine/skill-registry/installer.js

@@ -79,35 +79,24 @@ function buildFrontmatter(entry) {
 /** Prepare a built-in skill from the local catalog, generating a SKILL.md. */
 function prepareFromBuiltIn(skillName) {
     const entry = getBuiltInSkillEntry(skillName);
-    let content;
-    if (entry) {
-        const frontmatter = buildFrontmatter(entry);
-        content = [
-            frontmatter,
-            `# ${entry.name}`,
-            ``,
-            entry.description,
-            ``,
-            `## When to use`,
-            ``,
-            entry.trigger,
-            ``,
-            `## Tags`,
-            ``,
-            entry.tags.join(', '),
-        ].join('\n');
-    }
-    else {
-        content = [
-            `# ${skillName}`,
-            ``,
-            `Built-in Planu skill.`,
-            ``,
-            `## When to use`,
-            ``,
-            `Consult the Planu documentation.`,
-        ].join('\n');
+    if (!entry) {
+        throw new Error(`Built-in skill '${skillName}' has no catalog entry; refusing to install a placeholder`);
     }
+    const frontmatter = buildFrontmatter(entry);
+    const content = [
+        frontmatter,
+        `# ${entry.name}`,
+        ``,
+        entry.description,
+        ``,
+        `## When to use`,
+        ``,
+        entry.trigger,
+        ``,
+        `## Tags`,
+        ``,
+        entry.tags.join(', '),
+    ].join('\n');
     return { content, hasScripts: false };
 }
 /** Prepare a skill placeholder for skillssh/agentskill sources (no direct file API yet). */

package/dist/engine/skills/skills-fetcher.js

@@ -89,15 +89,6 @@ async function fetchSkillsShCatalog(stack, language) {
 function buildFallbackSkills(knowledge) {
     const skills = [];
     const stackLower = knowledge.stack.map((s) => s.toLowerCase());
-    const lang = knowledge.language.toLowerCase();
-    if (lang === technologyValue('technology-typescript-969545') || lang === 'javascript') {
-        skills.push({
-            name: 'typescript-patterns',
-            description: 'TypeScript best practices, strict mode patterns, and type safety conventions',
-            installCommand: 'npx skills add typescript-patterns',
-            justification: `Project uses ${knowledge.language}`,
-        });
-    }
     if (stackLower.some((s) => s.includes(technologyValue('technology-react-275976')))) {
         skills.push({
             name: 'react-patterns',

package/dist/engine/skills-reconciler.js

@@ -1,4 +1,3 @@
-import { technologyValue } from './technology-registry.js';
 // engine/skills-reconciler.ts — SPEC-189: Detect stack changes and reconcile skills
 // SPEC-395: autoFixSkills — install missing, remove stale, repair corrupt paths.
 // SPEC-607: Detect old-format skills and new agents, trigger regeneration.
@@ -32,7 +31,7 @@ export async function reconcileSkills(projectPath, knowledge) {
     const newSkillsDetected = recommendedSkills
         .filter((s) => !installedSet.has(s.name))
         .map((s) => s.name);
-    const staleSkills = installedNames.filter((name) => !recommendedNames.has(name) && !isProtectedBuiltInSkill(name, knowledge));
+    const staleSkills = installedNames.filter((name) => !recommendedNames.has(name));
     const totalRelevant = recommendedNames.size;
     const alreadyInSync = installedNames.filter((name) => recommendedNames.has(name)).length;
     const healthScore = computeHealthScore(totalRelevant, alreadyInSync, staleSkills.length);
@@ -167,17 +166,6 @@ function computeHealthScore(totalRelevant, alreadyInSync, staleCount) {
     const penalty = staleCount * 5;
     return Math.max(0, base - penalty);
 }
-function isProtectedBuiltInSkill(name, knowledge) {
-    if (name !== 'typescript-patterns') {
-        return false;
-    }
-    const language = knowledge.language.toLowerCase();
-    const stack = knowledge.stack.map((item) => item.toLowerCase());
-    return (language === technologyValue('technology-typescript-969545') ||
-        language === 'javascript' ||
-        stack.some((item) => item.includes(technologyValue('technology-typescript-969545')) ||
-            item.includes('javascript')));
-}
 /** Return a neutral reconciliation when prerequisites are missing. */
 function buildEmptyReconciliation(_reason) {
     return {

package/dist/tools/challenge-spec/scenarios-utils.js

@@ -33,7 +33,7 @@ const CAPABILITY_SIGNALS = {
         /\bbatch\s+upload\b/i,
     ],
     scale: [
-        /\b(?:traffic|load|throughput|requests?\s+per\s+second|concurrent\s+users?)\b/i,
+        /\b(?:traffic|load\b(?![\s‐-―-]+bearing)|throughput|requests?\s+per\s+second|concurrent\s+users?)\b/i,
         /\b(?:auto[ -]?scal|load\s+test|capacity\s+plan)/i,
         /\b(?:data\s+growth|growing\s+dataset|query\s+performance\s+at\s+scale)\b/i,
     ],

package/dist/tools/challenge-spec.js

@@ -2,6 +2,7 @@
 // Analyzes a spec from adversarial perspectives: failure scenarios,
 // concurrency issues, scale limits, security holes, and data consistency.
 import { readFile } from 'node:fs/promises';
+import { createHash } from 'node:crypto';
 import { specStore, knowledgeStore } from '../storage/index.js';
 // SPEC-1011 Bug F: fallback resolver using disk fingerprints
 import { resolveProjectFromPath } from '../storage/project-resolver.js';
@@ -29,6 +30,14 @@ const ALL_FOCUS_AREAS = [
     'security',
     'data-consistency',
 ];
+function truncateCriterionDiscriminator(criterion) {
+    const codePoints = Array.from(criterion);
+    if (codePoints.length <= 80) {
+        return criterion;
+    }
+    const digest = createHash('sha256').update(criterion).digest('hex').slice(0, 8);
+    return `${codePoints.slice(0, 79).join('')}… [${digest}]`;
+}
 function extractOutOfScopeSection(content) {
     const headingPattern = /^(#{1,6})[ \t]+(.+?)[ \t]*#*[ \t]*$/;
     const fencePattern = /^[ \t]{0,3}(`{3,}|~{3,})/;
@@ -224,7 +233,7 @@ export async function handleChallengeSpec(args, server) {
         const contradictionResult = checkScopeContradictions(resolvedOutOfScope, criteriaTexts);
         for (const c of contradictionResult.contradictions) {
             failureScenarios.push({
-                scenario: `Criterion contradicts outOfScope declaration: "${c.contradicts}"`,
+                scenario: `Criterion contradicts outOfScope declaration: "${c.contradicts}" (criterion: "${truncateCriterionDiscriminator(c.criterion)}")`,
                 probability: 'medium',
                 impact: 'low',
                 currentHandling: `Criterion: "${c.criterion}"`,
@@ -314,7 +323,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/suggest-tooling/skills-catalog.js

@@ -209,15 +209,6 @@ async function fetchSkillsShCatalog(stack, language) {
 function buildFallbackSkills(knowledge) {
     const skills = [];
     const stackLower = knowledge.stack.map((s) => s.toLowerCase());
-    const lang = knowledge.language.toLowerCase();
-    if (lang === technologyValue('technology-typescript-969545') || lang === 'javascript') {
-        skills.push({
-            name: 'typescript-patterns',
-            description: 'TypeScript best practices, strict mode patterns, and type safety conventions',
-            installCommand: 'npx skills add typescript-patterns',
-            justification: `Project uses ${knowledge.language}`,
-        });
-    }
     if (stackLower.some((s) => s.includes(technologyValue('technology-react-275976')))) {
         skills.push({
             name: 'react-patterns',

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/dist/types/ai-tool-rules.d.ts

@@ -19,7 +19,6 @@ export interface ArchitectureRule {
     description: string;
     layers: ArchitectureLayer[];
     conventions: string[];
-    maxLinesPerHandler: number;
 }
 export interface GeneratedAiToolRules {
     tool: AiToolDetection;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planu/cli",
-  "version": "5.3.61",
+  "version": "5.3.63",
   "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.63",
   "icon": "assets/plugin/icon.svg",
   "command": ["npx", "@planu/cli@latest"],
   "packageName": "@planu/cli",