@planu/cli 4.10.11 → 4.11.0

package/dist/engine/spec-format/value-only-spec-serializer.js

@@ -0,0 +1,18 @@
+import { generateLeanSpecContent } from './lean-spec-generator.js';
+/**
+ * Authority for secondary new-spec writers. Frontmatter always flows through
+ * the value-only serializer; callers may add only source-backed body content.
+ */
+export function serializeValueOnlySpecContent(input) {
+    const leanSpec = generateLeanSpecContent({
+        spec: input.spec,
+        description: input.description,
+        criteriaOverride: input.criteria ? [...input.criteria] : [],
+    });
+    const technical = input.technical?.trim();
+    if (!technical || /^##\s+Technical\b/m.test(input.description)) {
+        return leanSpec;
+    }
+    return `${leanSpec.trimEnd()}\n\n## Technical\n\n${technical}\n`;
+}
+//# sourceMappingURL=value-only-spec-serializer.js.map

package/dist/engine/spec-generator/fallback-generator.js

@@ -14,8 +14,10 @@ export class FallbackGenerator {
         return Promise.resolve({
             specBody,
             technicalSection,
-            generatedWithModel: 'deterministic-fallback',
-            generatedAt: new Date().toISOString(),
+            generation: {
+                method: 'deterministic',
+                generatedAt: new Date().toISOString(),
+            },
             qualityWarnings: [
                 'Fallback generator used. Source description was preserved; technical ownership must come from explicit source paths or local project analysis.',
             ],

package/dist/engine/spec-generator/opus-generator.js

@@ -89,8 +89,11 @@ function parseGeneratedText(text, model) {
     return {
         specBody: text,
         technicalSection: technicalMatch?.[1]?.trim() ?? '',
-        generatedWithModel: model,
-        generatedAt: new Date().toISOString(),
+        generation: {
+            method: 'internal-model',
+            modelId: model,
+            generatedAt: new Date().toISOString(),
+        },
         qualityWarnings: [],
     };
 }

package/dist/engine/spec-migrator/lean-migration.js

@@ -35,19 +35,31 @@ export async function isOldFormat(specDir) {
         return false;
     }
 }
-/** Check if a spec is already in lean format (has estimation in YAML frontmatter). */
+/** Check if a spec already uses the lean frontmatter contract. */
 function isLeanFormat(content) {
     const fmMatch = /^---\n([\s\S]*?)\n---/m.exec(content);
     if (!fmMatch?.[1]) {
         return false;
     }
-    return fmMatch[1].includes('estimation:') && fmMatch[1].includes('criteria:');
+    return (/^spec_format_version:\s*["']?1\.0["']?\s*$/m.test(fmMatch[1]) ||
+        /^(?:criteria|scenarios):\s*$/m.test(fmMatch[1]));
+}
+function isTerminalStatus(status) {
+    return status === 'done' || status === 'discarded';
+}
+function readTopLevelStatus(content) {
+    const frontmatter = /^---\n([\s\S]*?)\n---/m.exec(content)?.[1];
+    if (!frontmatter) {
+        return undefined;
+    }
+    const value = /^status:\s*["']?([^\s"']+)["']?\s*$/m.exec(frontmatter)?.[1];
+    return value;
 }
 /** Extract the user description from old spec.md (skip generic sections, keep real content). */
 function extractDescription(content) {
     // Remove frontmatter
     const withoutFm = content.replace(/^---\n[\s\S]*?\n---\n/, '').trim();
-    // Remove the heading line (# SPEC-XXX: Title)
+    // Remove the heading line containing a spec ID and title.
     const lines = withoutFm.split('\n');
     const startIdx = lines.findIndex((l) => /^#\s+SPEC-\d+/.test(l));
     const afterHeading = startIdx >= 0 ? lines.slice(startIdx + 1) : lines;
@@ -320,7 +332,9 @@ function buildSpecFromMetadata(metadata, id, title, specStatus, estimation, spec
         tags: Array.isArray(metadata.tags) ? metadata.tags : [],
         dependencies: [],
         blockedBy: [],
-        gitBranch: typeof metadata.branch === 'string' ? metadata.branch : '',
+        gitBranch: typeof metadata.branch === 'string' && metadata.branch.trim().length > 0
+            ? metadata.branch
+            : `feat/${id.toLowerCase()}`,
         impactAnalysis: null,
     };
 }
@@ -369,18 +383,23 @@ export async function migrateSpecToLean(specDir, projectPath) {
         catch {
             return 'skipped'; // No spec.md — not a valid spec dir
         }
+        // Parse frontmatter
+        const { metadata } = parseFrontmatterLocal(specContent);
+        const status = readTopLevelStatus(specContent) ?? 'draft';
+        // Terminal specs are immutable during automatic migration. This guard runs before
+        // backups, writes, or cleanup so init_project cannot alter completed decisions.
+        if (isTerminalStatus(status)) {
+            return 'skipped';
+        }
         // Skip if already lean
         if (isLeanFormat(specContent)) {
             return 'skipped';
         }
-        // Parse frontmatter
-        const { metadata } = parseFrontmatterLocal(specContent);
         const id = resolveSpecId(metadata, specDir, specContent);
         if (!id) {
             return 'missing valid spec id';
         }
         const title = typeof metadata.title === 'string' ? metadata.title : 'Untitled';
-        const status = typeof metadata.status === 'string' ? metadata.status : 'draft';
         // Read old technical.md first (needed for estimation extraction)
         let oldTechContent = '';
         let files = { create: [], modify: [], test: [] };
@@ -396,12 +415,6 @@ export async function migrateSpecToLean(specDir, projectPath) {
         const criteria = extractOldCriteria(specContent);
         const estimation = buildEstimationFromOld(metadata, oldTechContent);
         const spec = buildSpecFromMetadata(metadata, id, title, status, estimation, specPath, techPath);
-        // If status is done, mark all criteria as done
-        if (status === 'done') {
-            for (const c of criteria) {
-                c.done = true;
-            }
-        }
         // Generate lean spec.md with extracted criteria
         const leanSpecLines = generateLeanSpecContent({ spec, description, estimation });
         // Inject real criteria (replace the default one)

package/dist/storage/spec-store.js

@@ -118,8 +118,8 @@ export async function createSpec(projectId, spec) {
             ...spec,
             healthScore: computeHealthScore(spec, criteria.length > 0 ? criteria : undefined).score,
         };
-        specs.push(specWithScore);
-        await saveAll(projectId, specs);
+        const nextSpecs = [...specs, specWithScore];
+        await saveAll(projectId, nextSpecs);
         return specWithScore;
     });
 }
@@ -158,8 +158,8 @@ async function __internalUpdateSpec(projectId, specId, updates) {
             ...merged,
             healthScore: computeHealthScore(merged, criteria.length > 0 ? criteria : undefined).score,
         };
-        specs[idx] = updated;
-        await saveAll(projectId, specs);
+        const nextSpecs = specs.map((current, currentIdx) => (currentIdx === idx ? updated : current));
+        await saveAll(projectId, nextSpecs);
         return updated;
     });
 }
@@ -204,8 +204,8 @@ export async function deleteSpec(projectId, specId) {
         if (idx === -1) {
             return false;
         }
-        specs.splice(idx, 1);
-        await saveAll(projectId, specs);
+        const nextSpecs = specs.filter((_, currentIdx) => currentIdx !== idx);
+        await saveAll(projectId, nextSpecs);
         return true;
     });
 }