@planu/cli 5.3.2 → 5.3.4

package/dist/tools/update-status/transition-guard.js

@@ -7,7 +7,7 @@ import { scoreAmbiguityFromPath } from '../../engine/ambiguity-scorer.js';
 import { checkSpecReadiness } from '../../engine/readiness-checker.js';
 import { validateEnglishOnlySpecText } from '../../engine/spec-language/english-only.js';
 import { checkGroundedSpecContract } from '../../engine/spec-grounding/contract.js';
-import { checkGenericSpecOutput } from '../../engine/spec-quality/generic-output-gate.js';
+import { checkGenericSpecOutput, STRUCTURAL_INTERPOLATION_KIND, } from '../../engine/spec-quality/generic-output-gate.js';
 import { formatKeyValue } from '../output-formatter.js';
 import { getResolvedChallengeEvidence, requiredResolvedChallenges, } from '../challenge-spec/challenge-report.js';
 import { evaluateSpecDependencies } from '../../engine/dependency-evaluator.js';
@@ -319,14 +319,22 @@ export async function checkReadinessGate(spec, newStatus, forceApprove) {
         };
     }
     const genericOutputGate = checkGenericSpecOutput(body);
-    if (!genericOutputGate.passed) {
+    // SPEC-1406: structural-interpolation findings warn instead of block here — the
+    // `create_spec` call site is where this new detector is blocking; ~1100 already-persisted
+    // specs must stay transitionable at update_status even if their rendered body still
+    // contains an interpolated-list pattern.
+    const blockingGenericOutputIssues = genericOutputGate.issues.filter((issue) => issue.kind !== STRUCTURAL_INTERPOLATION_KIND);
+    const structuralInterpolationWarnings = genericOutputGate.issues
+        .filter((issue) => issue.kind === STRUCTURAL_INTERPOLATION_KIND)
+        .map((issue) => `${issue.phrase}: ${issue.reason}`);
+    if (blockingGenericOutputIssues.length > 0) {
         return {
             blockResult: {
                 content: [
                     {
                         type: 'text',
                         text: `Spec quality gate blocked transition to ${newStatus}. ` +
-                            genericOutputGate.issues
+                            blockingGenericOutputIssues
                                 .slice(0, 3)
                                 .map((issue) => `${issue.phrase}: ${issue.reason}`)
                                 .join('; '),
@@ -336,11 +344,15 @@ export async function checkReadinessGate(spec, newStatus, forceApprove) {
                 structuredContent: {
                     error: 'GENERIC_SPEC_OUTPUT_BLOCKED',
                     sddConstitutionRuleId: 'sdd.no-generic-output',
-                    issues: genericOutputGate.issues,
+                    issues: blockingGenericOutputIssues,
                     fixHint: 'Replace generic criteria or placeholder references with grounded, testable behavior before review or approval.',
                 },
             },
-            qualityWarnings: [],
+            // SPEC-1406 (AC3, round 2): the offending section must be named in the output on
+            // every return path, not only the success path — this blockResult already fired
+            // for a different reason, but any structural-interpolation warnings computed above
+            // are still real findings and must not be dropped.
+            qualityWarnings: structuralInterpolationWarnings,
         };
     }
     let readiness;
@@ -350,7 +362,7 @@ export async function checkReadinessGate(spec, newStatus, forceApprove) {
     catch (error) {
         /* reliability-optional: READINESS_EVALUATION_FAILED — transition fails closed below */
         reportClassifiedDegradation('READINESS_EVALUATION_FAILED', error);
-        return readinessUnavailable('READINESS_EVALUATION_FAILED', error instanceof Error ? error.message : 'readiness evaluation failed');
+        return readinessUnavailable('READINESS_EVALUATION_FAILED', error instanceof Error ? error.message : 'readiness evaluation failed', structuralInterpolationWarnings);
     }
     const { score } = readiness;
     const blockers = [...readiness.issues.blockers];
@@ -377,7 +389,7 @@ export async function checkReadinessGate(spec, newStatus, forceApprove) {
                         fixHint: 'Add at least 3 testable acceptance criteria before moving to review.',
                     },
                 },
-                qualityWarnings: [],
+                qualityWarnings: structuralInterpolationWarnings,
             };
         }
         const question = {
@@ -412,11 +424,11 @@ export async function checkReadinessGate(spec, newStatus, forceApprove) {
                     interactiveQuestions: [question],
                 },
             },
-            qualityWarnings: [],
+            qualityWarnings: structuralInterpolationWarnings,
         };
     }
     if (readiness.ready) {
-        return { blockResult: null, qualityWarnings: [] };
+        return { blockResult: null, qualityWarnings: structuralInterpolationWarnings };
     }
     const diagnostic = score < 70
         ? `score ${String(score)}/100 is below the 70 threshold`
@@ -442,13 +454,13 @@ export async function checkReadinessGate(spec, newStatus, forceApprove) {
                     fixHint: 'Complete the spec before moving from draft to review.',
                 },
             },
-            qualityWarnings: [],
+            qualityWarnings: structuralInterpolationWarnings,
         };
     }
     // Score < 70
     if (forceApprove) {
         // Caller wants to force through — proceed with warnings attached
-        return { blockResult: null, qualityWarnings: issues };
+        return { blockResult: null, qualityWarnings: [...issues, ...structuralInterpolationWarnings] };
     }
     const question = {
         header: 'Readiness gate',
@@ -486,7 +498,7 @@ export async function checkReadinessGate(spec, newStatus, forceApprove) {
                 interactiveQuestions: [question],
             },
         },
-        qualityWarnings: [],
+        qualityWarnings: structuralInterpolationWarnings,
     };
 }
 function gateUnavailableResult(code, message) {
@@ -496,8 +508,8 @@ function gateUnavailableResult(code, message) {
         structuredContent: { error: code, message, retryable: true },
     };
 }
-function readinessUnavailable(code, message) {
-    return { blockResult: gateUnavailableResult(code, message), qualityWarnings: [] };
+function readinessUnavailable(code, message, qualityWarnings = []) {
+    return { blockResult: gateUnavailableResult(code, message), qualityWarnings };
 }
 /**
  * SPEC-964: Challenge gate — block 'review' if challenge_spec was never run

package/dist/tools/update-status-actions.js

@@ -54,11 +54,13 @@ export async function runImplementingSideEffects(projectId, specId, transitionPr
         let branchResult;
         try {
             const { handleCreateBranch } = await import('./git/branch-ops.js');
-            branchResult = await withAudit(projectPath, 'update_status(implementing)', 'handleCreateBranch', () => handleCreateBranch(projectId, specId, undefined, projectPath), (r) => ({ isError: r.isError }));
+            branchResult = await withAudit(projectPath, 'update_status(implementing)', 'handleCreateBranch', () => handleCreateBranch(projectId, specId, undefined, projectPath, { mutateCheckout: false }), (r) => ({ isError: r.isError }));
         }
         catch {
             const { handleCreateBranch } = await import('./git/branch-ops.js');
-            branchResult = await handleCreateBranch(projectId, specId, undefined, projectPath);
+            branchResult = await handleCreateBranch(projectId, specId, undefined, projectPath, {
+                mutateCheckout: false,
+            });
         }
         if (!branchResult.isError) {
             try {

package/dist/types/git.d.ts

@@ -144,6 +144,10 @@ export interface GitSetupResult {
     data: Record<string, unknown>;
     branch: string;
 }
+/** SPEC-1396: automatic callers set `mutateCheckout: false` so the shared checkout never moves. */
+export interface GitAutoSetupOptions {
+    mutateCheckout?: boolean;
+}
 export interface AutoSetupResult {
     action: 'auto-setup';
     gitFlowType: GitFlowType;

package/dist/types/outbox-worker.d.ts

@@ -12,6 +12,8 @@ export interface OutboxFenceRecord {
     readonly state: 'leased' | 'failed' | 'dead-letter' | 'delivered';
     readonly attempts?: number;
     readonly lastError?: string;
+    /** Durable retry deadline (ISO timestamp). Only set for `failed` fences; optional for schemaVersion 1 compatibility. */
+    readonly nextAttemptAt?: string;
 }
 export declare const SPEC_CREATED_POST_COMMIT_TASKS: readonly ['git-setup', 'analysis', 'quality-score', 'duplicate-detection', 'complexity-advice', 'prior-decisions', 'suggestions', 'challenge-readiness'];
 export type SpecCreatedPostCommitTask = (typeof SPEC_CREATED_POST_COMMIT_TASKS)[number];

package/dist/types/runtime-policy.d.ts

@@ -47,5 +47,14 @@ export interface RuntimePolicy {
         maximumRememberedTransitions: number;
         minimumHeartbeatMs: number;
     };
+    outbox: {
+        sliceRecordBudget: number;
+        sliceTimeBudgetMs: number;
+        backoffBaseMs: number;
+        backoffCapMs: number;
+        maxAttempts: number;
+        diagnosticMaxLength: number;
+        continuationDelayMs: number;
+    };
 }
 //# sourceMappingURL=runtime-policy.d.ts.map

package/dist/types/spec-quality.d.ts

@@ -35,7 +35,12 @@ export interface SpecQualityReport {
         risk: QualityDimensionDetail;
     };
 }
-export type GenericSpecOutputIssueKind = 'generic-criterion' | 'unsupported-verification' | 'placeholder-reference' | 'generic-technical-reference';
+export type GenericSpecOutputIssueKind = 'generic-criterion' | 'unsupported-verification' | 'placeholder-reference' | 'generic-technical-reference'
+/** SPEC-1406: dedicated kind for structural-interpolation findings (checkInterpolatedSections).
+ * Kept distinct from 'generic-technical-reference' — severity routing in transition-guard.ts
+ * and sdd-rules-registry.ts filters BY KIND, so aliasing the two would silently downgrade
+ * whichever kind collides the day something else emits the shared value. */
+ | 'structural-interpolation';
 export interface GenericSpecOutputIssue {
     kind: GenericSpecOutputIssueKind;
     phrase: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@planu/cli",
-  "version": "5.3.2",
+  "version": "5.3.4",
   "description": "Planu — MCP Server for Spec Driven Development with native Rust acceleration for hot paths. Cross-platform (Linux/macOS/Windows, x64/arm64, glibc/musl).",
   "type": "module",
   "main": "dist/index.js",
@@ -35,14 +35,14 @@
     "packageName": "@planu/core"
   },
   "optionalDependencies": {
-    "@planu/core-darwin-arm64": "5.3.2",
-    "@planu/core-darwin-x64": "5.3.2",
-    "@planu/core-linux-arm64-gnu": "5.3.2",
-    "@planu/core-linux-arm64-musl": "5.3.2",
-    "@planu/core-linux-x64-gnu": "5.3.2",
-    "@planu/core-linux-x64-musl": "5.3.2",
-    "@planu/core-win32-arm64-msvc": "5.3.2",
-    "@planu/core-win32-x64-msvc": "5.3.2"
+    "@planu/core-darwin-arm64": "5.3.4",
+    "@planu/core-darwin-x64": "5.3.4",
+    "@planu/core-linux-arm64-gnu": "5.3.4",
+    "@planu/core-linux-arm64-musl": "5.3.4",
+    "@planu/core-linux-x64-gnu": "5.3.4",
+    "@planu/core-linux-x64-musl": "5.3.4",
+    "@planu/core-win32-arm64-msvc": "5.3.4",
+    "@planu/core-win32-x64-msvc": "5.3.4"
   },
   "engines": {
     "node": ">=24.0.0"

package/planu-native.json

@@ -1,7 +1,7 @@
 {
   "name": "dev.planu.native",
   "displayName": "Planu Native Lightweight Surface",
-  "version": "5.3.2",
+  "version": "5.3.4",
   "packageName": "@planu/cli",
   "modes": {
     "lightweight": {

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