@shepai/cli 1.38.0 → 1.39.0

package/dist/src/presentation/cli/commands/feat/show.command.js

@@ -6,7 +6,10 @@
  *
  * Usage: shep feat show <id>
  */
+import { readFileSync } from 'node:fs';
+import { join } from 'node:path';
 import { Command } from 'commander';
+import yaml from 'js-yaml';
 import { container } from '../../../../../packages/core/src/infrastructure/di/container.js';
 import { ShowFeatureUseCase } from '../../../../../packages/core/src/application/use-cases/features/show-feature.use-case.js';
 import { colors, symbols, messages, renderDetailView } from '../../ui/index.js';
@@ -94,6 +97,23 @@ function isProcessAlive(pid) {
         return false;
     }
 }
+/**
+ * Load rejection feedback entries from spec.yaml (best-effort).
+ */
+function loadRejectionFeedback(specPath) {
+    if (!specPath)
+        return [];
+    try {
+        const content = readFileSync(join(specPath, 'spec.yaml'), 'utf-8');
+        const spec = yaml.load(content);
+        return Array.isArray(spec?.rejectionFeedback)
+            ? spec.rejectionFeedback
+            : [];
+    }
+    catch {
+        return [];
+    }
+}
 /**
  * Group timings by agentRunId, preserving order.
  */
@@ -117,7 +137,7 @@ function renderNodeTiming(t, isLast, isWaiting, isRunTerminal, run, maxDurationM
     const lines = [];
     const isSubPhase = t.phase.includes(':');
     const label = isSubPhase
-        ? `  \u21b3 ${t.phase.split(':')[1]}`.padEnd(16)
+        ? `  \u21b3 rev ${t.phase.split(':')[1]}`.padEnd(16)
         : (NODE_TO_PHASE[t.phase] ?? t.phase).padEnd(16);
     // Completed phase - green bar with duration
     if (t.completedAt && t.durationMs != null) {
@@ -185,7 +205,7 @@ function renderNodeTiming(t, isLast, isWaiting, isRunTerminal, run, maxDurationM
 /**
  * Render the full phase timing section, grouped by agent run with lifecycle events.
  */
-function renderPhaseTimings(timings, currentRun) {
+function renderPhaseTimings(timings, currentRun, rejections = []) {
     const groups = groupTimingsByRun(timings);
     const multiRun = groups.length > 1;
     // Compute max duration across ALL timings for consistent bar scaling
@@ -220,12 +240,26 @@ function renderPhaseTimings(timings, currentRun) {
             currentRun?.status === 'failed' ||
             currentRun?.status === 'cancelled';
         const isWaiting = isCurrentRun && currentRun?.status === 'waiting_approval';
+        let resumedIdx = 0;
         for (const [idx, t] of group.timings.entries()) {
             const isLast = isLastGroup && idx === group.timings.length - 1;
             // Lifecycle event - render as a marker line
             if (isLifecycleEvent(t.phase)) {
                 const event = LIFECYCLE_EVENTS[t.phase];
                 if (event) {
+                    // For resumed events, show rejection feedback if available
+                    if (t.phase === 'run:resumed' && rejections.length > 0) {
+                        const feedback = rejections[resumedIdx];
+                        resumedIdx++;
+                        if (feedback?.message) {
+                            const maxLen = 60;
+                            const msg = feedback.message.length > maxLen
+                                ? `${feedback.message.slice(0, maxLen)}…`
+                                : feedback.message;
+                            lines.push(`  ${colors.error(`↩ rejected: "${msg}"`)}`);
+                            continue;
+                        }
+                    }
                     const sym = t.phase === 'run:completed'
                         ? symbols.success
                         : t.phase === 'run:failed' || t.phase === 'run:crashed'
@@ -292,7 +326,8 @@ export function createShowCommand() {
                 });
             }
             if (timings.length > 0) {
-                const lines = renderPhaseTimings(timings, run);
+                const rejections = loadRejectionFeedback(feature.specPath);
+                const lines = renderPhaseTimings(timings, run, rejections);
                 textBlocks.push({ title: 'Phase Timing', content: lines.join('\n') });
             }
             if (run?.status === 'waiting_approval') {

package/dist/src/presentation/tui/prompts/prd-review-question.prompt.d.ts

@@ -0,0 +1,30 @@
+/**
+ * PRD Review Question Prompt Configuration
+ *
+ * Creates an @inquirer/select config for reviewing a single open question
+ * from the PRD spec.yaml. Users select from the available options.
+ */
+import type { QuestionOption } from '../../../../packages/core/src/domain/generated/output.js';
+/**
+ * Creates the @inquirer/select configuration for a single open question.
+ */
+export declare function createQuestionSelectConfig(question: string, options: QuestionOption[], currentAnswer?: string): {
+    message: string;
+    choices: {
+        name: string;
+        value: string;
+        description: string;
+    }[];
+    default: string | undefined;
+    theme: {
+        readonly prefix: {
+            readonly idle: string;
+            readonly done: string;
+        };
+        readonly style: {
+            readonly highlight: import("picocolors/types.js").Formatter;
+            readonly answer: import("picocolors/types.js").Formatter;
+        };
+    };
+};
+//# sourceMappingURL=prd-review-question.prompt.d.ts.map

package/dist/src/presentation/tui/prompts/prd-review-question.prompt.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prd-review-question.prompt.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/tui/prompts/prd-review-question.prompt.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAEnE;;GAEG;AACH,wBAAgB,0BAA0B,CACxC,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,cAAc,EAAE,EACzB,aAAa,CAAC,EAAE,MAAM;;;;;;;;;;;;;;;;;;EAcvB"}

package/dist/src/presentation/tui/prompts/prd-review-question.prompt.js

@@ -0,0 +1,23 @@
+/**
+ * PRD Review Question Prompt Configuration
+ *
+ * Creates an @inquirer/select config for reviewing a single open question
+ * from the PRD spec.yaml. Users select from the available options.
+ */
+import { shepTheme } from '../themes/shep.theme.js';
+/**
+ * Creates the @inquirer/select configuration for a single open question.
+ */
+export function createQuestionSelectConfig(question, options, currentAnswer) {
+    const choices = options.map((opt) => ({
+        name: opt.option,
+        value: opt.option,
+        description: opt.description,
+    }));
+    return {
+        message: question,
+        choices,
+        default: currentAnswer,
+        theme: shepTheme,
+    };
+}

package/dist/src/presentation/tui/prompts/prd-review-summary.prompt.d.ts

@@ -0,0 +1,29 @@
+/**
+ * PRD Review Summary Prompt Configuration
+ *
+ * Creates an @inquirer/select config for the final approve/reject
+ * action after reviewing open questions.
+ */
+export type ReviewAction = 'approve' | 'reject';
+/**
+ * Creates the @inquirer/select configuration for the review action prompt.
+ */
+export declare function createReviewActionConfig(changeCount: number): {
+    message: string;
+    choices: {
+        name: string;
+        value: ReviewAction;
+        description: string;
+    }[];
+    theme: {
+        readonly prefix: {
+            readonly idle: string;
+            readonly done: string;
+        };
+        readonly style: {
+            readonly highlight: import("picocolors/types.js").Formatter;
+            readonly answer: import("picocolors/types.js").Formatter;
+        };
+    };
+};
+//# sourceMappingURL=prd-review-summary.prompt.d.ts.map

package/dist/src/presentation/tui/prompts/prd-review-summary.prompt.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prd-review-summary.prompt.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/tui/prompts/prd-review-summary.prompt.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,MAAM,MAAM,YAAY,GAAG,SAAS,GAAG,QAAQ,CAAC;AAEhD;;GAEG;AACH,wBAAgB,wBAAwB,CAAC,WAAW,EAAE,MAAM;;;;eAShC,YAAY;;;;;;;;;;;;;EAWvC"}

package/dist/src/presentation/tui/prompts/prd-review-summary.prompt.js

@@ -0,0 +1,29 @@
+/**
+ * PRD Review Summary Prompt Configuration
+ *
+ * Creates an @inquirer/select config for the final approve/reject
+ * action after reviewing open questions.
+ */
+import { shepTheme } from '../themes/shep.theme.js';
+/**
+ * Creates the @inquirer/select configuration for the review action prompt.
+ */
+export function createReviewActionConfig(changeCount) {
+    const changeLabel = changeCount > 0 ? ` (${changeCount} change${changeCount !== 1 ? 's' : ''})` : '';
+    return {
+        message: `Review complete${changeLabel}. What would you like to do?`,
+        choices: [
+            {
+                name: 'Approve and continue',
+                value: 'approve',
+                description: 'Accept the requirements and proceed to research',
+            },
+            {
+                name: 'Reject and iterate',
+                value: 'reject',
+                description: 'Provide feedback and re-run requirements',
+            },
+        ],
+        theme: shepTheme,
+    };
+}

package/dist/src/presentation/tui/wizards/merge-review.wizard.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Merge Review Wizard
+ *
+ * Placeholder TUI wizard for the merge approval phase.
+ * Shows approve/reject action with optional comment.
+ */
+export type MergeReviewAction = 'approve' | 'reject';
+export interface MergeReviewWizardResult {
+    action: MergeReviewAction;
+    feedback?: string;
+}
+export declare function mergeReviewWizard(): Promise<MergeReviewWizardResult>;
+//# sourceMappingURL=merge-review.wizard.d.ts.map

package/dist/src/presentation/tui/wizards/merge-review.wizard.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"merge-review.wizard.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/tui/wizards/merge-review.wizard.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAKH,MAAM,MAAM,iBAAiB,GAAG,SAAS,GAAG,QAAQ,CAAC;AAErD,MAAM,WAAW,uBAAuB;IACtC,MAAM,EAAE,iBAAiB,CAAC;IAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,wBAAsB,iBAAiB,IAAI,OAAO,CAAC,uBAAuB,CAAC,CAkC1E"}

package/dist/src/presentation/tui/wizards/merge-review.wizard.js

@@ -0,0 +1,43 @@
+/**
+ * Merge Review Wizard
+ *
+ * Placeholder TUI wizard for the merge approval phase.
+ * Shows approve/reject action with optional comment.
+ */
+import { select, input } from '@inquirer/prompts';
+import { shepTheme } from '../themes/shep.theme.js';
+export async function mergeReviewWizard() {
+    const action = await select({
+        message: 'Review the merge. What would you like to do?',
+        choices: [
+            {
+                name: 'Approve and merge',
+                value: 'approve',
+                description: 'Accept the changes and merge the branch',
+            },
+            {
+                name: 'Reject',
+                value: 'reject',
+                description: 'Provide feedback — implementation will iterate',
+            },
+        ],
+        theme: shepTheme,
+    });
+    let feedback;
+    if (action === 'reject') {
+        feedback = await input({
+            message: 'What needs to change before merging?',
+            theme: shepTheme,
+            validate: (value) => value.trim().length > 0 || 'Feedback is required for rejection',
+        });
+    }
+    else {
+        feedback = await input({
+            message: 'Any comments? (optional, press Enter to skip)',
+            theme: shepTheme,
+        });
+        if (feedback?.trim().length === 0)
+            feedback = undefined;
+    }
+    return { action, feedback };
+}

package/dist/src/presentation/tui/wizards/plan-review.wizard.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Plan Review Wizard
+ *
+ * Placeholder TUI wizard for the plan approval phase.
+ * Shows approve/reject action with optional comment.
+ */
+export type PlanReviewAction = 'approve' | 'reject';
+export interface PlanReviewWizardResult {
+    action: PlanReviewAction;
+    feedback?: string;
+}
+export declare function planReviewWizard(): Promise<PlanReviewWizardResult>;
+//# sourceMappingURL=plan-review.wizard.d.ts.map

package/dist/src/presentation/tui/wizards/plan-review.wizard.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"plan-review.wizard.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/tui/wizards/plan-review.wizard.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAKH,MAAM,MAAM,gBAAgB,GAAG,SAAS,GAAG,QAAQ,CAAC;AAEpD,MAAM,WAAW,sBAAsB;IACrC,MAAM,EAAE,gBAAgB,CAAC;IACzB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,wBAAsB,gBAAgB,IAAI,OAAO,CAAC,sBAAsB,CAAC,CAkCxE"}

package/dist/src/presentation/tui/wizards/plan-review.wizard.js

@@ -0,0 +1,43 @@
+/**
+ * Plan Review Wizard
+ *
+ * Placeholder TUI wizard for the plan approval phase.
+ * Shows approve/reject action with optional comment.
+ */
+import { select, input } from '@inquirer/prompts';
+import { shepTheme } from '../themes/shep.theme.js';
+export async function planReviewWizard() {
+    const action = await select({
+        message: 'Review the implementation plan. What would you like to do?',
+        choices: [
+            {
+                name: 'Approve and continue',
+                value: 'approve',
+                description: 'Accept the plan and proceed to implementation',
+            },
+            {
+                name: 'Reject and iterate',
+                value: 'reject',
+                description: 'Provide feedback and re-run planning',
+            },
+        ],
+        theme: shepTheme,
+    });
+    let feedback;
+    if (action === 'reject') {
+        feedback = await input({
+            message: 'What needs to change in the plan?',
+            theme: shepTheme,
+            validate: (value) => value.trim().length > 0 || 'Feedback is required for rejection',
+        });
+    }
+    else {
+        feedback = await input({
+            message: 'Any comments? (optional, press Enter to skip)',
+            theme: shepTheme,
+        });
+        if (feedback?.trim().length === 0)
+            feedback = undefined;
+    }
+    return { action, feedback };
+}

package/dist/src/presentation/tui/wizards/prd-review.wizard.d.ts

@@ -0,0 +1,26 @@
+/**
+ * PRD Review Wizard
+ *
+ * Interactive TUI wizard that walks users through open questions
+ * from a PRD spec, lets them change selections, and approve or
+ * reject with feedback.
+ */
+import { type ReviewAction } from '../prompts/prd-review-summary.prompt.js';
+import type { OpenQuestion } from '../../../../packages/core/src/application/use-cases/agents/review-feature.use-case.js';
+import type { QuestionSelectionChange } from '../../../../packages/core/src/domain/generated/output.js';
+export interface PrdReviewWizardResult {
+    action: ReviewAction;
+    changedSelections: QuestionSelectionChange[];
+    feedback?: string;
+}
+/**
+ * Runs the interactive PRD review wizard.
+ *
+ * Steps:
+ * 1. For each open question, show select prompt with current options
+ * 2. Track which selections changed from the original
+ * 3. Show approve/reject action prompt
+ * 4. If reject, prompt for feedback
+ */
+export declare function prdReviewWizard(questions: OpenQuestion[]): Promise<PrdReviewWizardResult>;
+//# sourceMappingURL=prd-review.wizard.d.ts.map

package/dist/src/presentation/tui/wizards/prd-review.wizard.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prd-review.wizard.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/tui/wizards/prd-review.wizard.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAIH,OAAO,EAEL,KAAK,YAAY,EAClB,MAAM,yCAAyC,CAAC;AACjD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2DAA2D,CAAC;AAC9F,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,8BAA8B,CAAC;AAG5E,MAAM,WAAW,qBAAqB;IACpC,MAAM,EAAE,YAAY,CAAC;IACrB,iBAAiB,EAAE,uBAAuB,EAAE,CAAC;IAC7C,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED;;;;;;;;GAQG;AACH,wBAAsB,eAAe,CAAC,SAAS,EAAE,YAAY,EAAE,GAAG,OAAO,CAAC,qBAAqB,CAAC,CA4B/F"}

package/dist/src/presentation/tui/wizards/prd-review.wizard.js

@@ -0,0 +1,42 @@
+/**
+ * PRD Review Wizard
+ *
+ * Interactive TUI wizard that walks users through open questions
+ * from a PRD spec, lets them change selections, and approve or
+ * reject with feedback.
+ */
+import { select, input } from '@inquirer/prompts';
+import { createQuestionSelectConfig } from '../prompts/prd-review-question.prompt.js';
+import { createReviewActionConfig, } from '../prompts/prd-review-summary.prompt.js';
+import { shepTheme } from '../themes/shep.theme.js';
+/**
+ * Runs the interactive PRD review wizard.
+ *
+ * Steps:
+ * 1. For each open question, show select prompt with current options
+ * 2. Track which selections changed from the original
+ * 3. Show approve/reject action prompt
+ * 4. If reject, prompt for feedback
+ */
+export async function prdReviewWizard(questions) {
+    const changedSelections = [];
+    for (const q of questions) {
+        const selected = await select(createQuestionSelectConfig(q.question, q.options, q.selectedOption));
+        if (selected !== q.selectedOption) {
+            changedSelections.push({
+                questionId: q.question,
+                selectedOption: selected,
+            });
+        }
+    }
+    const action = await select(createReviewActionConfig(changedSelections.length));
+    let feedback;
+    if (action === 'reject') {
+        feedback = await input({
+            message: 'What needs to change? (feedback for re-iteration)',
+            theme: shepTheme,
+            validate: (value) => value.trim().length > 0 || 'Feedback is required for rejection',
+        });
+    }
+    return { action, changedSelections, feedback };
+}