@shepai/cli 1.38.0 → 1.40.0

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/requirements.prompt.js

@@ -2,14 +2,40 @@
  * Requirements Phase Prompt
  *
  * Instructs the agent to build comprehensive requirements from the analysis,
- * including product questions with AI-recommended default answers.
+ * including product questions with structured QuestionOption[] arrays
+ * and AI-recommended default answers.
  * Writes back to spec.yaml with full requirements.
  */
+import yaml from 'js-yaml';
 import { readSpecFile } from '../node-helpers.js';
 export function buildRequirementsPrompt(state) {
     const specContent = readSpecFile(state.specDir, 'spec.yaml');
+    // Extract rejection feedback if present
+    let rejectionFeedbackSection = '';
+    try {
+        const specData = yaml.load(specContent);
+        const rejectionFeedback = specData?.rejectionFeedback;
+        if (rejectionFeedback && rejectionFeedback.length > 0) {
+            const entries = rejectionFeedback
+                .map((entry) => `- **Iteration ${entry.iteration}** (${entry.timestamp}): ${entry.message}`)
+                .join('\n');
+            rejectionFeedbackSection = `
+## Previous Rejection Feedback
+
+The user has previously rejected this PRD with the following feedback. You MUST address these concerns in your revised output:
+
+${entries}
+
+Focus on the most recent feedback (highest iteration number) while ensuring earlier feedback is still addressed.
+
+`;
+        }
+    }
+    catch {
+        // If YAML parsing fails, continue without rejection feedback
+    }
     return `You are a product analyst performing the REQUIREMENTS phase of feature development.
-
+${rejectionFeedbackSection}
 ## Your Task
 
 1. Read the current spec with codebase analysis below
@@ -29,13 +55,20 @@ ${specContent}
 - **Non-functional requirements (NFR-N)**: performance, security, UX, maintainability constraints
 - **Success criteria**: measurable checkboxes that define "done"
 
-### Product Questions with AI Defaults
+### Product Questions with Structured Options
 For every ambiguous product decision, create an openQuestion entry with:
 - A clear question
 - resolved: true (your recommendation is the default)
-- Your recommended answer with reasoning
+- Structured options array with EXACTLY ONE option marked selected: true
+- A selectionRationale explaining the recommendation
+- An answer field containing the text of the selected option (for backward compatibility)
+
+Each option MUST have:
+- \`option\`: Short label for the approach (2-5 words)
+- \`description\`: Explanation of this option's trade-offs and benefits
+- \`selected\`: boolean — exactly ONE option per question must be true
 
-The user can later review and override these defaults. This is the AI-recommended path — not the only option.
+The user can later review and override these defaults via interactive TUI.
 
 ## Output Instructions
 
@@ -63,32 +96,25 @@ Write the COMPLETE file. Preserve name/number/branch/technologies from analysis
     - question: 'Should X use approach A, B, or C?'
       resolved: true
       options:
-        - option: Approach A
-          description: (What approach A means and its trade-offs)
+        - option: 'Approach A'
+          description: 'Benefits of A, when to use it, trade-offs'
           selected: true
-        - option: Approach B
-          description: (What approach B means and its trade-offs)
-          selected: false
-        - option: Approach C
-          description: (What approach C means and its trade-offs)
+        - option: 'Approach B'
+          description: 'Benefits of B, when to use it, trade-offs'
           selected: false
-      selectionRationale: >
-        (Detailed reasoning for why approach A is recommended.
-        Reference codebase patterns where relevant.)
+      selectionRationale: 'Approach A is recommended because [detailed reasoning]'
+      answer: 'Approach A'
     - question: 'What level of Y is needed?'
       resolved: true
       options:
-        - option: Level X
-          description: (What level X means and its trade-offs)
-          selected: true
-        - option: Level Y
-          description: (What level Y means and its trade-offs)
-          selected: false
-        - option: Level Z
-          description: (What level Z means and its trade-offs)
+        - option: 'Basic'
+          description: 'Minimal implementation, fast to build'
           selected: false
-      selectionRationale: >
-        (Detailed reasoning for why level X is recommended.)
+        - option: 'Advanced'
+          description: 'Full-featured, handles edge cases'
+          selected: true
+      selectionRationale: 'Advanced is recommended because [reasoning]'
+      answer: 'Advanced'
 
   content: |
     ## Problem Statement
@@ -132,8 +158,8 @@ Write the COMPLETE file. Preserve name/number/branch/technologies from analysis
 ## Constraints
 
 - Write ONLY to ${state.specDir}/spec.yaml
-- Every open question MUST have an AI-recommended default answer (resolved: true)
-- Every open question MUST have exactly 3 options to choose from
+- Every open question MUST have structured options with exactly ONE option marked selected: true
+- Every open question MUST also have an \`answer\` field matching the selected option text
 - Requirements must be specific and testable, not vague
 - Do NOT create any other files
 - Do NOT modify any source code`;

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/phase-timing-context.d.ts

@@ -24,6 +24,9 @@ export declare function clearPhaseTimingContext(): void;
 /**
  * Record the start of a phase. Returns the timing record ID (for later update)
  * or null if context is not set or save fails.
+ *
+ * Automatically appends an iteration suffix (`:2`, `:3`, …) when the same
+ * phase has already been recorded for this run (e.g. after a rejection loop).
  */
 export declare function recordPhaseStart(phase: string): Promise<string | null>;
 /**

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/phase-timing-context.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"phase-timing-context.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/phase-timing-context.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAGH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,wEAAwE,CAAC;AAMrH;;GAEG;AACH,wBAAgB,eAAe,IAAI,MAAM,GAAG,IAAI,CAE/C;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,MAAM,EAAE,UAAU,EAAE,sBAAsB,GAAG,IAAI,CAG7F;AAED;;GAEG;AACH,wBAAgB,uBAAuB,IAAI,IAAI,CAI9C;AAED;;;GAGG;AACH,wBAAsB,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAqB5E;AAED;;;GAGG;AACH,wBAAsB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAW/F;AAED;;;;;;GAMG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,MAAM,EACb,aAAa,CAAC,EAAE,MAAM,EACtB,YAAY,CAAC,EAAE,sBAAsB,GACpC,OAAO,CAAC,IAAI,CAAC,CAoBf;AAED;;;GAGG;AACH,wBAAsB,uBAAuB,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAUpF"}
+{"version":3,"file":"phase-timing-context.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/phase-timing-context.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAGH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,wEAAwE,CAAC;AAMrH;;GAEG;AACH,wBAAgB,eAAe,IAAI,MAAM,GAAG,IAAI,CAE/C;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,MAAM,EAAE,UAAU,EAAE,sBAAsB,GAAG,IAAI,CAG7F;AAED;;GAEG;AACH,wBAAgB,uBAAuB,IAAI,IAAI,CAI9C;AA+BD;;;;;;GAMG;AACH,wBAAsB,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAsB5E;AAED;;;GAGG;AACH,wBAAsB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAW/F;AAED;;;;;;GAMG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,MAAM,EACb,aAAa,CAAC,EAAE,MAAM,EACtB,YAAY,CAAC,EAAE,sBAAsB,GACpC,OAAO,CAAC,IAAI,CAAC,CAoBf;AAED;;;GAGG;AACH,wBAAsB,uBAAuB,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAUpF"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/phase-timing-context.js

@@ -33,9 +33,35 @@ export function clearPhaseTimingContext() {
     contextRepository = undefined;
     lastTimingId = null;
 }
+/**
+ * Compute the iteration-aware phase name.
+ *
+ * Queries existing PhaseTiming rows for the current run, counts how many
+ * share the same base phase name, and appends `:N` for iterations > 1.
+ *
+ * Examples:
+ *   - First "requirements" execution  → "requirements"
+ *   - Second "requirements" execution → "requirements:2"
+ *   - Third "requirements" execution  → "requirements:3"
+ */
+async function resolveIterationPhase(phase, runId, repository) {
+    try {
+        const existing = await repository.findByRunId(runId);
+        // Count rows whose phase is the base name or matches the base:N pattern
+        const count = existing.filter((t) => t.phase === phase || t.phase.startsWith(`${phase}:`)).length;
+        return count === 0 ? phase : `${phase}:${count + 1}`;
+    }
+    catch {
+        // On error, fall back to bare phase name
+        return phase;
+    }
+}
 /**
  * Record the start of a phase. Returns the timing record ID (for later update)
  * or null if context is not set or save fails.
+ *
+ * Automatically appends an iteration suffix (`:2`, `:3`, …) when the same
+ * phase has already been recorded for this run (e.g. after a rejection loop).
  */
 export async function recordPhaseStart(phase) {
     if (!contextRunId || !contextRepository)
@@ -43,10 +69,11 @@ export async function recordPhaseStart(phase) {
     const id = randomUUID();
     const now = new Date();
     try {
+        const iterationPhase = await resolveIterationPhase(phase, contextRunId, contextRepository);
         await contextRepository.save({
             id,
             agentRunId: contextRunId,
-            phase,
+            phase: iterationPhase,
             startedAt: now,
             createdAt: now,
             updatedAt: now,

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/state.d.ts

@@ -38,6 +38,9 @@ export declare const FeatureAgentAnnotation: import("@langchain/langgraph").Anno
     validationRetries: import("@langchain/langgraph").BinaryOperatorAggregate<number, number>;
     lastValidationTarget: import("@langchain/langgraph").BinaryOperatorAggregate<string, string>;
     lastValidationErrors: import("@langchain/langgraph").BinaryOperatorAggregate<string[], string[]>;
+    _approvalAction: import("@langchain/langgraph").BinaryOperatorAggregate<string | null, string | null>;
+    _rejectionFeedback: import("@langchain/langgraph").BinaryOperatorAggregate<string | null, string | null>;
+    _needsReexecution: import("@langchain/langgraph").BinaryOperatorAggregate<boolean, boolean>;
     prUrl: import("@langchain/langgraph").BinaryOperatorAggregate<string | null, string | null>;
     prNumber: import("@langchain/langgraph").BinaryOperatorAggregate<number | null, number | null>;
     commitHash: import("@langchain/langgraph").BinaryOperatorAggregate<string | null, string | null>;

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/state.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAElE;;;;;;GAMG;AACH,eAAO,MAAM,sBAAsB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAwDjC,CAAC;AAEH,MAAM,MAAM,iBAAiB,GAAG,OAAO,sBAAsB,CAAC,KAAK,CAAC"}
+{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/state.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAElE;;;;;;GAMG;AACH,eAAO,MAAM,sBAAsB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAqEjC,CAAC;AAEH,MAAM,MAAM,iBAAiB,GAAG,OAAO,sBAAsB,CAAC,KAAK,CAAC"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/state.js

@@ -37,6 +37,19 @@ export const FeatureAgentAnnotation = Annotation.Root({
         reducer: (_prev, next) => next,
         default: () => [],
     }),
+    // --- Approval action channels (set by Command({update}) on resume) ---
+    _approvalAction: Annotation({
+        reducer: (_prev, next) => (next !== undefined ? next : _prev),
+        default: () => null,
+    }),
+    _rejectionFeedback: Annotation({
+        reducer: (_prev, next) => (next !== undefined ? next : _prev),
+        default: () => null,
+    }),
+    _needsReexecution: Annotation({
+        reducer: (_prev, next) => next,
+        default: () => false,
+    }),
     // --- Merge and workflow state channels ---
     prUrl: Annotation({
         reducer: (_prev, next) => (next !== undefined ? next : _prev),

package/dist/src/presentation/cli/commands/agent/reject.command.js

@@ -25,7 +25,7 @@ export function createRejectCommand() {
                 return;
             }
             const useCase = container.resolve(RejectAgentRunUseCase);
-            const result = await useCase.execute(resolved.run.id, opts.reason);
+            const result = await useCase.execute(resolved.run.id, opts.reason ?? 'Rejected by user');
             if (result.rejected) {
                 messages.success(`Rejected ${colors.accent(resolved.run.id.substring(0, 8))}: ${result.reason}`);
             }

package/dist/src/presentation/cli/commands/feat/reject.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reject.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/reject.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAQpC,wBAAgB,mBAAmB,IAAI,OAAO,CAsC7C"}
+{"version":3,"file":"reject.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/reject.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAQpC,wBAAgB,mBAAmB,IAAI,OAAO,CAwC7C"}

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

@@ -16,7 +16,7 @@ export function createRejectCommand() {
     return new Command('reject')
         .description('Reject a feature waiting for review')
         .argument('[id]', 'Feature ID (auto-resolves if omitted)')
-        .option('--reason <text>', 'Rejection reason')
+        .requiredOption('--reason <text>', 'Rejection feedback (required)')
         .action(async (featureId, options) => {
         try {
             const featureRepo = container.resolve('IFeatureRepository');
@@ -35,10 +35,12 @@ export function createRejectCommand() {
             }
             messages.newline();
             messages.warning(`Rejected: ${feature.name}`);
-            if (options.reason) {
-                console.log(`  ${colors.muted('Reason:')} ${options.reason}`);
+            console.log(`  ${colors.muted('Reason:')}    ${options.reason}`);
+            console.log(`  ${colors.muted('Iteration:')} ${result.iteration}`);
+            if (result.iterationWarning) {
+                messages.warning('Warning: 5+ iterations. Consider refining the prompt instead.');
             }
-            console.log(`  ${colors.muted('Agent:')}  cancelled`);
+            console.log(`  ${colors.muted('Agent:')}     re-running requirements`);
             messages.newline();
         }
         catch (error) {

package/dist/src/presentation/cli/commands/feat/review.command.d.ts

@@ -1,8 +1,9 @@
 /**
  * Feature Review Command
  *
- * Shows the current state of a feature waiting for approval,
- * including which phase triggered the interrupt and generated content.
+ * Interactive review of a feature waiting for approval.
+ * Routes to phase-specific TUI wizard (PRD, Plan, or Merge),
+ * then calls approve or reject use case based on user action.
  *
  * Usage:
  *   shep feat review [id]

package/dist/src/presentation/cli/commands/feat/review.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/review.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAOpC,wBAAgB,mBAAmB,IAAI,OAAO,CAqC7C"}
+{"version":3,"file":"review.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/review.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAiEpC,wBAAgB,mBAAmB,IAAI,OAAO,CA2E7C"}

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

@@ -1,19 +1,60 @@
 /**
  * Feature Review Command
  *
- * Shows the current state of a feature waiting for approval,
- * including which phase triggered the interrupt and generated content.
+ * Interactive review of a feature waiting for approval.
+ * Routes to phase-specific TUI wizard (PRD, Plan, or Merge),
+ * then calls approve or reject use case based on user action.
  *
  * Usage:
  *   shep feat review [id]
  */
 import { Command } from 'commander';
 import { container } from '../../../../../packages/core/src/infrastructure/di/container.js';
+import { ReviewFeatureUseCase } from '../../../../../packages/core/src/application/use-cases/agents/review-feature.use-case.js';
+import { ApproveAgentRunUseCase } from '../../../../../packages/core/src/application/use-cases/agents/approve-agent-run.use-case.js';
+import { RejectAgentRunUseCase } from '../../../../../packages/core/src/application/use-cases/agents/reject-agent-run.use-case.js';
 import { resolveWaitingFeature } from './resolve-waiting-feature.js';
+import { prdReviewWizard } from '../../../tui/wizards/prd-review.wizard.js';
+import { planReviewWizard } from '../../../tui/wizards/plan-review.wizard.js';
+import { mergeReviewWizard } from '../../../tui/wizards/merge-review.wizard.js';
 import { colors, messages } from '../../ui/index.js';
+function parsePhase(runResult) {
+    return runResult?.startsWith('node:') ? runResult.slice(5) : 'unknown';
+}
+async function runPhaseWizard(phase, featureId, repoPath) {
+    if (phase === 'requirements') {
+        const reviewUseCase = container.resolve(ReviewFeatureUseCase);
+        const reviewResult = await reviewUseCase.execute(featureId, repoPath);
+        if (!reviewResult.success) {
+            throw new Error(reviewResult.reason);
+        }
+        return prdReviewWizard(reviewResult.questions);
+    }
+    if (phase === 'plan') {
+        const result = await planReviewWizard();
+        return { action: result.action, feedback: result.feedback };
+    }
+    if (phase === 'merge') {
+        const result = await mergeReviewWizard();
+        return { action: result.action, feedback: result.feedback };
+    }
+    throw new Error(`Unknown approval phase: ${phase}`);
+}
+function phaseNextStep(phase) {
+    switch (phase) {
+        case 'requirements':
+            return 'research';
+        case 'plan':
+            return 'implementation';
+        case 'merge':
+            return 'merge';
+        default:
+            return 'next step';
+    }
+}
 export function createReviewCommand() {
     return new Command('review')
-        .description('Review a feature waiting for approval')
+        .description('Interactive review of a feature waiting for approval')
         .argument('[id]', 'Feature ID (auto-resolves if omitted)')
         .action(async (featureId) => {
         try {
@@ -26,19 +67,53 @@ export function createReviewCommand() {
                 featureRepo,
                 runRepo,
             });
-            const phase = run.result?.startsWith('node:') ? run.result.slice(5) : 'unknown';
+            const phase = parsePhase(run.result);
             messages.newline();
-            messages.info(`Feature waiting for approval: ${colors.accent(feature.name)}`);
-            console.log(`  ${colors.muted('ID:')}     ${feature.id.slice(0, 8)}`);
+            messages.info(`Reviewing: ${colors.accent(feature.name)}`);
             console.log(`  ${colors.muted('Phase:')}  ${colors.warning(phase)}`);
             console.log(`  ${colors.muted('Branch:')} ${colors.accent(feature.branch)}`);
-            if (feature.specPath) {
-                console.log(`  ${colors.muted('Specs:')}  ${feature.specPath}`);
-            }
-            messages.newline();
-            console.log(`  ${colors.muted('To approve:')} shep feat approve ${feature.id.slice(0, 8)}`);
-            console.log(`  ${colors.muted('To reject:')}  shep feat reject ${feature.id.slice(0, 8)}`);
             messages.newline();
+            const wizardResult = await runPhaseWizard(phase, feature.id, repoPath);
+            if (wizardResult.action === 'approve') {
+                const approveUseCase = container.resolve(ApproveAgentRunUseCase);
+                const payload = wizardResult.changedSelections && wizardResult.changedSelections.length > 0
+                    ? { approved: true, changedSelections: wizardResult.changedSelections }
+                    : undefined;
+                const approveResult = await approveUseCase.execute(run.id, payload);
+                if (approveResult.approved) {
+                    messages.newline();
+                    messages.success(`Approved: ${feature.name}`);
+                    if (wizardResult.changedSelections && wizardResult.changedSelections.length > 0) {
+                        console.log(`  ${colors.muted('Changes:')} ${wizardResult.changedSelections.length} selection(s) updated`);
+                    }
+                    if (wizardResult.feedback) {
+                        console.log(`  ${colors.muted('Comment:')} ${wizardResult.feedback}`);
+                    }
+                    console.log(`  ${colors.muted('Agent:')}   proceeding to ${phaseNextStep(phase)}`);
+                    messages.newline();
+                }
+                else {
+                    throw new Error(approveResult.reason);
+                }
+            }
+            else {
+                const rejectUseCase = container.resolve(RejectAgentRunUseCase);
+                const rejectResult = await rejectUseCase.execute(run.id, wizardResult.feedback);
+                if (rejectResult.rejected) {
+                    messages.newline();
+                    messages.warning(`Rejected: ${feature.name}`);
+                    console.log(`  ${colors.muted('Feedback:')} ${wizardResult.feedback}`);
+                    console.log(`  ${colors.muted('Iteration:')} ${rejectResult.iteration}`);
+                    if (rejectResult.iterationWarning) {
+                        messages.warning('Warning: 5+ iterations. Consider refining the prompt instead.');
+                    }
+                    console.log(`  ${colors.muted('Agent:')}    re-running ${phase}`);
+                    messages.newline();
+                }
+                else {
+                    throw new Error(rejectResult.reason);
+                }
+            }
         }
         catch (error) {
             const err = error instanceof Error ? error : new Error(String(error));

package/dist/src/presentation/cli/commands/feat/show.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"show.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/show.command.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAsTpC,wBAAgB,iBAAiB,IAAI,OAAO,CAwJ3C"}
+{"version":3,"file":"show.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/show.command.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA+VpC,wBAAgB,iBAAiB,IAAI,OAAO,CAyJ3C"}

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/agent-select.prompt.d.ts

@@ -20,7 +20,7 @@ export declare function createAgentSelectConfig(): {
     }, {
         readonly name: "Gemini CLI";
         readonly value: AgentType.GeminiCli;
-        readonly disabled: "(Coming Soon)";
+        readonly description: "Google Gemini CLI";
     }, {
         readonly name: "Aider";
         readonly value: AgentType.Aider;

package/dist/src/presentation/tui/prompts/agent-select.prompt.js

@@ -24,7 +24,7 @@ export function createAgentSelectConfig() {
             {
                 name: 'Gemini CLI',
                 value: AgentType.GeminiCli,
-                disabled: '(Coming Soon)',
+                description: 'Google Gemini CLI',
             },
             {
                 name: 'Aider',

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"}