@planu/cli 5.3.63 → 5.3.65

package/dist/types/handoff-artifacts.d.ts

@@ -146,4 +146,23 @@ export interface ArtifactPayloadMap {
 }
 export type ArtifactPayload<K extends ArtifactKind> = ArtifactPayloadMap[K];
 import type { MinimalImplementationReport } from './minimal-implementation-gate.js';
+export type ImplementationReviewRejectionReason = 'missing' | 'unreadable' | 'schema_invalid' | 'stub_body' | 'wrong_agent_or_kind' | 'not_approved' | 'self_review' | 'actor_unreadable' | 'wrong_spec';
+export interface VerifiedImplementationReviewOk {
+    ok: true;
+    review: ImplementationReviewV1;
+    implementingActor: string | null;
+}
+export interface VerifiedImplementationReviewErr {
+    ok: false;
+    reason: ImplementationReviewRejectionReason;
+    review?: ImplementationReviewV1;
+    implementingActor?: string;
+    errors?: {
+        code: string;
+        path: string;
+        message: string;
+    }[];
+    readError?: string;
+}
+export type VerifiedImplementationReviewResult = VerifiedImplementationReviewOk | VerifiedImplementationReviewErr;
 //# sourceMappingURL=handoff-artifacts.d.ts.map

package/dist/types/index.d.ts

@@ -291,4 +291,5 @@ export * from './host-tool-filter.js';
 export * from './reconcile.js';
 export * from './release-pipeline.js';
 export * from './network-policy.js';
+export type { VerifiedImplementationReviewResult, VerifiedImplementationReviewOk, VerifiedImplementationReviewErr, ImplementationReviewRejectionReason, } from './handoff-artifacts.js';
 //# sourceMappingURL=index.d.ts.map

package/package.json

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

package/scripts/lib/portable-paths.mjs

@@ -47,6 +47,7 @@ export function portablePathPatterns({
 }
 
 export function redactPortablePaths(text, options = {}) {
+  if (typeof text !== 'string') return text;
   return portablePathPatterns(options).reduce(
     (redacted, { pattern, replacement }) => redacted.replace(pattern, replacement),
     text,

package/dist/engine/execution-plan/phases-b.d.ts

@@ -1,4 +0,0 @@
-import type { Spec, ProjectKnowledge, ExecutionStep } from '../../types/index.js';
-export declare function generateTestingPhase(spec: Spec, knowledge: ProjectKnowledge, startOrder: number): ExecutionStep[];
-export declare function generateIntegrationPhase(_spec: Spec, knowledge: ProjectKnowledge, startOrder: number): ExecutionStep[];
-//# sourceMappingURL=phases-b.d.ts.map

package/dist/engine/execution-plan/phases-b.js

@@ -1,88 +0,0 @@
-// engine/execution-plan/phases-b.ts — Execution phase generators (part B)
-// SRP: testing and integration phase generators.
-// OCP: extend without modifying the dispatcher in generate-execution-plan.ts.
-export function generateTestingPhase(spec, knowledge, startOrder) {
-    const steps = [];
-    let order = startOrder;
-    steps.push({
-        order: order++,
-        title: 'Write unit tests',
-        description: 'Test business logic, validators, and utilities in isolation',
-        files: [`tests/unit/${spec.slug}.test.ts`],
-        dependsOn: [startOrder - 1],
-        verification: `All unit tests pass: ${knowledge.testCommand ?? 'npm test'}`,
-        estimatedMinutes: 30,
-        canRollback: true,
-    });
-    if (spec.scope !== 'trivial') {
-        steps.push({
-            order: order++,
-            title: 'Write integration tests',
-            description: 'Test API endpoints and data layer together',
-            files: [`tests/integration/${spec.slug}.test.ts`],
-            dependsOn: [startOrder],
-            verification: 'Integration tests pass against test database',
-            estimatedMinutes: 30,
-            canRollback: true,
-        });
-    }
-    if (spec.target === 'frontend' || spec.target === 'fullstack') {
-        steps.push({
-            order: order,
-            title: 'Write E2E tests',
-            description: 'Test critical user flows end-to-end (Playwright/Cypress)',
-            files: [`tests/e2e/${spec.slug}.spec.ts`],
-            dependsOn: [startOrder],
-            verification: 'E2E tests pass in headless browser',
-            estimatedMinutes: 30,
-            canRollback: true,
-        });
-    }
-    return steps;
-}
-export function generateIntegrationPhase(_spec, knowledge, startOrder) {
-    const steps = [];
-    let order = startOrder;
-    steps.push({
-        order: order++,
-        title: 'Run linting and formatting',
-        description: 'Ensure code passes all lint rules and is properly formatted',
-        files: [],
-        dependsOn: [startOrder - 1],
-        verification: `Lint passes: ${knowledge.linting.detectedLinters[0]?.tool ?? 'eslint'} runs clean`,
-        estimatedMinutes: 10,
-        canRollback: true,
-    });
-    steps.push({
-        order: order++,
-        title: 'Update documentation',
-        description: 'Update README, API docs, and changelog if applicable',
-        files: [],
-        dependsOn: [startOrder],
-        verification: 'Documentation reflects the new changes',
-        estimatedMinutes: 15,
-        canRollback: true,
-    });
-    steps.push({
-        order: order++,
-        title: 'Full build and test suite',
-        description: 'Run complete build and all test suites to catch regressions',
-        files: [],
-        dependsOn: [order - 2, order - 1],
-        verification: `Full build succeeds: ${knowledge.buildCommand ?? 'npm run build'} && ${knowledge.testCommand ?? 'npm test'}`,
-        estimatedMinutes: 10,
-        canRollback: true,
-    });
-    steps.push({
-        order: order++,
-        title: 'Create pull request',
-        description: 'Push branch and create PR with spec reference and summary',
-        files: [],
-        dependsOn: [order - 2],
-        verification: 'PR created with passing CI checks',
-        estimatedMinutes: 10,
-        canRollback: true,
-    });
-    return steps;
-}
-//# sourceMappingURL=phases-b.js.map

package/dist/engine/execution-plan/phases.d.ts

@@ -1,7 +0,0 @@
-import type { Spec, ProjectKnowledge, ExecutionStep } from '../../types/index.js';
-export { generateTestingPhase, generateIntegrationPhase } from './phases-b.js';
-export declare function generateSetupPhase(spec: Spec, knowledge: ProjectKnowledge, startOrder: number): ExecutionStep[];
-export declare function generateDataLayerPhase(spec: Spec, knowledge: ProjectKnowledge, content: string, startOrder: number): ExecutionStep[];
-export declare function generateBusinessLogicPhase(spec: Spec, knowledge: ProjectKnowledge, content: string, startOrder: number): ExecutionStep[];
-export declare function generateUIPhase(spec: Spec, knowledge: ProjectKnowledge, content: string, startOrder: number): ExecutionStep[];
-//# sourceMappingURL=phases.d.ts.map

package/dist/engine/execution-plan/phases.js

@@ -1,216 +0,0 @@
-import { technologyValue } from '../technology-registry.js';
-import { inferDataLayerFiles, inferServiceFiles, inferApiFiles, inferPageFiles, inferComponentFiles, } from './plan-utils.js';
-export { generateTestingPhase, generateIntegrationPhase } from './phases-b.js';
-export function generateSetupPhase(spec, knowledge, startOrder) {
-    const steps = [];
-    let order = startOrder;
-    steps.push({
-        order: order++,
-        title: 'Create feature branch',
-        description: `Create branch from main: ${spec.gitBranch || `feature/${spec.slug}`}`,
-        files: [],
-        dependsOn: [],
-        verification: `Branch exists and is checked out: git branch --show-current`,
-        estimatedMinutes: 2,
-        canRollback: true,
-    });
-    if (knowledge.language === technologyValue('technology-typescript-969545') ||
-        knowledge.language === 'javascript') {
-        steps.push({
-            order: order++,
-            title: 'Define types and interfaces',
-            description: 'Create TypeScript interfaces for the new entities and API contracts',
-            files: [`src/types/${spec.slug}.ts`],
-            dependsOn: [startOrder],
-            verification: 'TypeScript compiles without errors: npm run build',
-            estimatedMinutes: 15,
-            canRollback: true,
-        });
-    }
-    if (spec.scope === 'cross-module' || spec.scope === 'architectural') {
-        steps.push({
-            order: order,
-            title: 'Update project configuration',
-            description: 'Add new configuration entries, feature flags, or environment variables',
-            files: ['.env.example'],
-            dependsOn: [startOrder],
-            verification: 'Configuration values are documented and have defaults',
-            estimatedMinutes: 10,
-            canRollback: true,
-        });
-    }
-    return steps;
-}
-export function generateDataLayerPhase(spec, knowledge, content, startOrder) {
-    const steps = [];
-    let order = startOrder;
-    const lower = content.toLowerCase();
-    if (lower.includes('table') ||
-        lower.includes('schema') ||
-        lower.includes('model') ||
-        lower.includes('entity') ||
-        lower.includes('migration') ||
-        knowledge.database !== 'unknown') {
-        steps.push({
-            order: order++,
-            title: 'Create database migration',
-            description: `Create migration for new tables/columns in ${knowledge.database}`,
-            files: [`migrations/${spec.slug}.sql`],
-            dependsOn: [startOrder - 1],
-            verification: 'Migration runs successfully: apply and rollback both work',
-            estimatedMinutes: 20,
-            canRollback: true,
-        });
-        steps.push({
-            order: order++,
-            title: 'Run migration on development database',
-            description: 'Apply migration and verify schema changes',
-            files: [],
-            dependsOn: [order - 2],
-            verification: 'Database schema matches expected state',
-            estimatedMinutes: 5,
-            canRollback: true,
-        });
-    }
-    const lastStep = steps[steps.length - 1];
-    steps.push({
-        order: order++,
-        title: 'Implement data access layer',
-        description: 'Create repository/model functions for CRUD operations',
-        files: inferDataLayerFiles(spec, knowledge),
-        dependsOn: lastStep ? [lastStep.order] : [startOrder - 1],
-        verification: 'Data access functions compile. Write unit tests for basic CRUD.',
-        estimatedMinutes: 30,
-        canRollback: true,
-    });
-    if (lower.includes('seed') || lower.includes('fixture') || lower.includes('sample data')) {
-        steps.push({
-            order: order++,
-            title: 'Create seed data',
-            description: 'Add seed/fixture data for development and testing',
-            files: [`seeds/${spec.slug}.ts`],
-            dependsOn: [order - 2],
-            verification: 'Seed command runs without errors',
-            estimatedMinutes: 15,
-            canRollback: true,
-        });
-    }
-    return steps;
-}
-export function generateBusinessLogicPhase(spec, knowledge, content, startOrder) {
-    const steps = [];
-    let order = startOrder;
-    const lower = content.toLowerCase();
-    steps.push({
-        order: order++,
-        title: 'Implement business logic',
-        description: 'Create service/use-case functions with core domain logic',
-        files: inferServiceFiles(spec, knowledge),
-        dependsOn: [startOrder - 1],
-        verification: 'Business logic functions pass unit tests with valid and invalid inputs',
-        estimatedMinutes: 45,
-        canRollback: true,
-    });
-    if (lower.includes('validat') || lower.includes('input') || lower.includes('form')) {
-        steps.push({
-            order: order++,
-            title: 'Implement input validation',
-            description: 'Add validation schemas for all inputs (Zod/Joi/class-validator)',
-            files: [`src/validators/${spec.slug}.ts`],
-            dependsOn: [startOrder],
-            verification: 'Validation rejects invalid inputs with clear error messages',
-            estimatedMinutes: 20,
-            canRollback: true,
-        });
-    }
-    if (lower.includes('api') ||
-        lower.includes('endpoint') ||
-        lower.includes('route') ||
-        spec.target === 'backend' ||
-        spec.target === 'fullstack') {
-        steps.push({
-            order: order++,
-            title: 'Create API endpoints',
-            description: 'Implement REST/GraphQL/tRPC endpoints with request handling',
-            files: inferApiFiles(spec, knowledge),
-            dependsOn: [startOrder],
-            verification: 'Endpoints respond correctly: test with curl or HTTP client',
-            estimatedMinutes: 30,
-            canRollback: true,
-        });
-    }
-    steps.push({
-        order: order++,
-        title: 'Add error handling',
-        description: 'Implement structured error handling, logging, and error responses',
-        files: [],
-        dependsOn: [order - 2],
-        verification: 'Error cases return proper status codes and error messages',
-        estimatedMinutes: 15,
-        canRollback: true,
-    });
-    return steps;
-}
-export function generateUIPhase(spec, knowledge, content, startOrder) {
-    const steps = [];
-    let order = startOrder;
-    const lower = content.toLowerCase();
-    const framework = knowledge.framework ?? technologyValue('technology-react-275976');
-    steps.push({
-        order: order++,
-        title: 'Create page layout and routing',
-        description: `Set up page structure and routes in ${framework}`,
-        files: inferPageFiles(spec, knowledge),
-        dependsOn: [startOrder - 1],
-        verification: 'Page renders without errors. Route is accessible.',
-        estimatedMinutes: 20,
-        canRollback: true,
-    });
-    steps.push({
-        order: order++,
-        title: 'Build UI components',
-        description: 'Implement reusable components (forms, lists, cards, etc.)',
-        files: inferComponentFiles(spec, knowledge),
-        dependsOn: [startOrder],
-        verification: 'Components render correctly with mock data',
-        estimatedMinutes: 45,
-        canRollback: true,
-    });
-    steps.push({
-        order: order++,
-        title: 'Connect data fetching',
-        description: 'Wire components to API, add loading and error states',
-        files: [],
-        dependsOn: [order - 2, order - 1],
-        verification: 'Data loads from API. Loading spinners show. Errors display correctly.',
-        estimatedMinutes: 30,
-        canRollback: true,
-    });
-    if (lower.includes('form') ||
-        lower.includes('create') ||
-        lower.includes('edit') ||
-        lower.includes('input')) {
-        steps.push({
-            order: order++,
-            title: 'Implement form handling and validation',
-            description: 'Add form state management, client-side validation, and submission',
-            files: [],
-            dependsOn: [order - 2],
-            verification: 'Form submits successfully. Validation errors show inline. Submit button disables during submission.',
-            estimatedMinutes: 25,
-            canRollback: true,
-        });
-    }
-    steps.push({
-        order: order++,
-        title: 'Add responsive design and accessibility',
-        description: 'Ensure UI works on mobile, tablet, and desktop. Add ARIA labels.',
-        files: [],
-        dependsOn: [order - 2],
-        verification: 'UI passes basic responsive checks. No accessibility violations on axe/Lighthouse.',
-        estimatedMinutes: 20,
-        canRollback: true,
-    });
-    return steps;
-}
-//# sourceMappingURL=phases.js.map