@planu/cli 5.3.62 → 5.3.63

package/CHANGELOG.md

@@ -1,3 +1,26 @@
+## [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

package/dist/.planu-build.json

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

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}"`,

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