@planu/cli 5.3.3 → 5.3.5

package/dist/engine/universal-rules/rules/planu-debate-review.d.ts

@@ -0,0 +1,3 @@
+import type { UniversalRule } from '../../../types/universal-rules/index.js';
+export declare const planuDebateReviewRule: UniversalRule;
+//# sourceMappingURL=planu-debate-review.d.ts.map

package/dist/engine/universal-rules/rules/planu-debate-review.js

@@ -0,0 +1,38 @@
+function buildBody() {
+    return `# Planu Debate Review
+
+Auto-generated by \`init_project\`. Do not edit manually.
+
+This rule assumes \`planu-approval-gates\` (reviewer independence, self-approval prohibitions) and \`planu-sdd-model-routing\` (arbitration) and restates neither.
+
+## Reviewer Plurality And Provider Diversity
+
+Every spec and every implementation is reviewed by two independent reviewers from DIFFERENT providers, in ONE parallel round.
+
+Prohibited: running a single reviewer, or running two reviewers from the same provider, and calling the result a debate round.
+
+Replacement action: launch both reviewers in the same message so they run concurrently, and pick two different providers (for example one Claude-family model and one external model).
+
+## One Parallel Round, No Serial Confirmation Loops
+
+Prohibited: re-launching a reviewer for pass 2, pass 3, or a "confirm the fix" loop after the first round already produced verdicts.
+
+Replacement action: cross the two verdicts once — coincident findings apply directly, divergent findings are arbitrated against the source code — then close the round. Re-open a reviewer only for a specific disagreement the arbiter cannot resolve by reading the code directly.
+
+## Degradation Ladder
+
+Prohibited: silently reviewing with only one provider when a second is unavailable and presenting it as a debate round.
+
+Replacement action: when only one provider is available, run two isolated sessions of the same model instead of one, so two independent verdicts still exist before arbitration.
+`;
+}
+export const planuDebateReviewRule = {
+    id: 'planu-debate-review',
+    name: 'Planu Debate Review',
+    description: 'Requires two-provider parallel review rounds with no serial confirmation loops and a single-provider degradation ladder.',
+    category: 'quality',
+    applicableHosts: ['all'],
+    defaultEnabled: true,
+    buildContent: (_host) => buildBody(),
+};
+//# sourceMappingURL=planu-debate-review.js.map

package/dist/engine/universal-rules/rules/planu-dogfood-bugs.js

@@ -1,5 +1,3 @@
-// engine/universal-rules/rules/planu-dogfood-bugs.ts — SPEC-779
-// Universal rule: Planu Dogfood Bugs (mandatory for all Planu-using projects)
 /** Shared markdown body (same for all hosts). */
 function buildBody() {
     return `# Planu Dogfood Bugs (MANDATORY — STRICT)

package/dist/engine/universal-rules/rules/planu-english-specs.js

@@ -1,4 +1,3 @@
-// engine/universal-rules/rules/planu-english-specs.ts — Universal rule: specs are written in English
 function buildBody() {
     return `# Planu Artifacts Must Be Written in English
 

package/dist/engine/universal-rules/rules/planu-minimal-change.js

@@ -1,5 +1,3 @@
-// engine/universal-rules/rules/planu-minimal-change.ts
-// Universal rule: minimal-change implementation discipline.
 function buildBody() {
     return `# Planu Minimal Change
 

package/dist/engine/universal-rules/rules/planu-modes.js

@@ -1,5 +1,3 @@
-// engine/universal-rules/rules/planu-modes.ts — SPEC-779
-// Universal rule: Claude Code Mode Hints per spec phase (SPEC-494)
 function buildBody(host) {
     if (host === 'claude-code') {
         return `# Claude Code Mode Hints (MANDATORY — Planu SDD)

package/dist/engine/universal-rules/rules/planu-release-policy.js

@@ -1,4 +1,3 @@
-// engine/universal-rules/rules/planu-release-policy.ts — Universal rule: release decision
 function buildBody() {
     return `# Planu Release Policy
 

package/dist/engine/universal-rules/rules/planu-revert-proof-tests.d.ts

@@ -0,0 +1,3 @@
+import type { UniversalRule } from '../../../types/universal-rules/index.js';
+export declare const planuRevertProofTestsRule: UniversalRule;
+//# sourceMappingURL=planu-revert-proof-tests.d.ts.map

package/dist/engine/universal-rules/rules/planu-revert-proof-tests.js

@@ -0,0 +1,34 @@
+function buildBody() {
+    return `# Planu Revert-Proof Tests
+
+Auto-generated by \`init_project\`. Do not edit manually.
+
+A new test that never fails against the unfixed source proves nothing.
+
+## The Rule
+
+Prohibited: reporting a new test as passing evidence without ever having watched it fail. This includes an assertion that compares a value to itself, a mock that echoes back what it was given, and a negative assertion that holds vacuously regardless of the source under test.
+
+Replacement action: before reporting the test as evidence, revert only the source change it covers — invert the specific guard line in place, or run the test against the pre-change body read with \`git show <ref>:<path>\` — run the test, and observe it fail. Then restore the source and confirm the test passes again. Report the observed failure output, not just the final green run.
+
+Prohibited: using \`git stash\`, \`git checkout\` or \`git restore\` on a shared checkout to stage the revert. Stashes are a process-wide LIFO stack, so a concurrent agent can pop work that is not its own.
+
+A guard's regression test must fail against the unguarded variant specifically, not merely against an unrelated point in history such as HEAD.
+
+## Hard Blocks
+
+- Do not accept a test whose only run is against already-fixed source.
+- Do not accept an assertion that compares a value to itself.
+- Do not accept an assertion that cannot fail for any input.
+`;
+}
+export const planuRevertProofTestsRule = {
+    id: 'planu-revert-proof-tests',
+    name: 'Planu Revert-Proof Tests',
+    description: 'Requires new tests to be proven to fail against the unfixed source before being reported as evidence.',
+    category: 'quality',
+    applicableHosts: ['all'],
+    defaultEnabled: true,
+    buildContent: (_host) => buildBody(),
+};
+//# sourceMappingURL=planu-revert-proof-tests.js.map

package/dist/engine/universal-rules/rules/planu-sdd-model-routing.js

@@ -1,4 +1,3 @@
-// engine/universal-rules/rules/planu-sdd-model-routing.ts — Universal rule: model routing + context continuity
 function buildBody() {
     return `# Planu SDD Model Routing and Context Guarantee
 

package/dist/engine/universal-rules/rules/planu-workflow.js

@@ -1,5 +1,3 @@
-// engine/universal-rules/rules/planu-workflow.ts — SPEC-779
-// Universal rule: Planu Workflow (migrated from rules-generator.ts)
 function buildBody() {
     return `# Planu SDD Workflow (MANDATORY)
 

package/dist/engine/validator/validation-report-writer.js

@@ -39,7 +39,7 @@ export async function writeImplementationReviewReport(input) {
         return {
             written: true,
             path: artifact.path,
-            sha: artifact.sha,
+            sha: `sha256:${artifact.sha}`,
             reviewer,
             passed,
             gates: mergedGates,

package/dist/engine/workers/handlers/auto-drift.js

@@ -154,14 +154,6 @@ async function buildDriftFindings(projectDir, effectiveChanged, activeSpecs, pro
                 driftedFiles,
                 cachedAt: new Date().toISOString(),
             });
-            // Wire drift:detected event so autopilot trigger-rules can react
-            emitAutopilotEvent({
-                name: 'drift:detected',
-                projectPath: projectDir,
-                projectId,
-                specId: spec.id,
-                payload: { driftScore, driftLevel: driftScore >= 0.7 ? 'high' : 'warning' },
-            });
             findings.push({
                 severity: driftScore >= 0.7 ? 'high' : 'warning',
                 file: '',
@@ -171,6 +163,28 @@ async function buildDriftFindings(projectDir, effectiveChanged, activeSpecs, pro
             });
         }
     }
+    if (driftScore >= threshold) {
+        // Wire drift:detected event so autopilot trigger-rules can react.
+        // Emitted once per project run — driftScore is project-wide, not per-spec,
+        // so emitting inside the loop would fire N identical events for N specs.
+        //
+        // MUST run after the cache-persistence loop above: emitAutopilotEvent
+        // dispatches handlers in a microtask (event-bus.ts), and the enabled
+        // rule drift-detected:resolve_drift_violations calls resolveDriftViolations,
+        // which loads the drift cache as its first action. Emitting before the
+        // cache entries are persisted lets that microtask race ahead and read an
+        // empty cache, silently resolving nothing.
+        emitAutopilotEvent({
+            name: 'drift:detected',
+            projectPath: projectDir,
+            projectId,
+            payload: {
+                driftScore,
+                driftLevel: driftScore >= 0.7 ? 'high' : 'warning',
+                specIds: activeSpecs.map((s) => s.id),
+            },
+        });
+    }
     return findings;
 }
 // ---------------------------------------------------------------------------

package/dist/tools/create-spec.js

@@ -1030,6 +1030,11 @@ async function prepareCreateSpecCandidate(initialParams, server) {
         },
     }));
     spec.generation = generatedSpec.generation;
+    // SPEC-1406 (DEFECT 4, round 2): FallbackGenerator no longer emits an unconditional
+    // warning, so qualityWarnings only needs the non-empty guard now — the redundant
+    // constant-string filter was removed from create-spec.ts, fallback-generator.ts and
+    // opus-generator.ts (dead code, reverted) so the literal doesn't have to stay
+    // byte-identical across three files.
     spec.qualityWarnings =
         generatedSpec.qualityWarnings.length > 0 ? generatedSpec.qualityWarnings : undefined;
     const baseCriteria = extractCriteria(generatedSpec.specBody).map((criterion) => criterion.text);

package/dist/tools/reconcile-spec.d.ts

@@ -1,4 +1,4 @@
 import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-import type { ReconcileSpecInput, ToolResult } from '../types/index.js';
+import { type ReconcileSpecInput, type ToolResult } from '../types/index.js';
 export declare function handleReconcileSpec(params: ReconcileSpecInput, server?: McpServer): Promise<ToolResult>;
 //# sourceMappingURL=reconcile-spec.d.ts.map