@shepai/cli 1.38.0 → 1.39.0

package/dist/packages/core/src/application/ports/output/agents/feature-agent-process.interface.d.ts

@@ -30,6 +30,7 @@ export interface IFeatureAgentProcessService {
         resumeFromInterrupt?: boolean;
         push?: boolean;
         openPr?: boolean;
+        resumePayload?: string;
     }): number;
     /**
      * Check if a process is still alive.

package/dist/packages/core/src/application/ports/output/agents/feature-agent-process.interface.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-agent-process.interface.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/agents/feature-agent-process.interface.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wCAAwC,CAAC;AAE5E;;GAEG;AACH,MAAM,WAAW,2BAA2B;IAC1C;;;;;;;;OAQG;IACH,KAAK,CACH,SAAS,EAAE,MAAM,EACjB,KAAK,EAAE,MAAM,EACb,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,MAAM,EACf,YAAY,CAAC,EAAE,MAAM,EACrB,OAAO,CAAC,EAAE;QACR,aAAa,CAAC,EAAE,aAAa,CAAC;QAC9B,MAAM,CAAC,EAAE,OAAO,CAAC;QACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,mBAAmB,CAAC,EAAE,OAAO,CAAC;QAC9B,IAAI,CAAC,EAAE,OAAO,CAAC;QACf,MAAM,CAAC,EAAE,OAAO,CAAC;KAClB,GACA,MAAM,CAAC;IAEV;;;;;OAKG;IACH,OAAO,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;IAE9B;;;;;OAKG;IACH,mBAAmB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;CACnD"}
+{"version":3,"file":"feature-agent-process.interface.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/agents/feature-agent-process.interface.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wCAAwC,CAAC;AAE5E;;GAEG;AACH,MAAM,WAAW,2BAA2B;IAC1C;;;;;;;;OAQG;IACH,KAAK,CACH,SAAS,EAAE,MAAM,EACjB,KAAK,EAAE,MAAM,EACb,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,MAAM,EACf,YAAY,CAAC,EAAE,MAAM,EACrB,OAAO,CAAC,EAAE;QACR,aAAa,CAAC,EAAE,aAAa,CAAC;QAC9B,MAAM,CAAC,EAAE,OAAO,CAAC;QACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,mBAAmB,CAAC,EAAE,OAAO,CAAC;QAC9B,IAAI,CAAC,EAAE,OAAO,CAAC;QACf,MAAM,CAAC,EAAE,OAAO,CAAC;QACjB,aAAa,CAAC,EAAE,MAAM,CAAC;KACxB,GACA,MAAM,CAAC;IAEV;;;;;OAKG;IACH,OAAO,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;IAE9B;;;;;OAKG;IACH,mBAAmB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;CACnD"}

package/dist/packages/core/src/application/use-cases/agents/approve-agent-run.use-case.d.ts

@@ -3,20 +3,21 @@
  *
  * Approves a paused agent run (waiting_approval status) and
  * spawns a new resume worker to continue graph execution.
+ * Optionally accepts a PrdApprovalPayload to update spec.yaml
+ * selections before resuming.
  */
 import type { IAgentRunRepository } from '../../ports/output/agents/agent-run-repository.interface.js';
 import type { IFeatureAgentProcessService } from '../../ports/output/agents/feature-agent-process.interface.js';
 import type { IPhaseTimingRepository } from '../../ports/output/agents/phase-timing-repository.interface.js';
 import type { IFeatureRepository } from '../../ports/output/repositories/feature-repository.interface.js';
-import type { IWorktreeService } from '../../ports/output/services/worktree-service.interface.js';
+import type { PrdApprovalPayload } from '../../../domain/generated/output.js';
 export declare class ApproveAgentRunUseCase {
     private readonly agentRunRepository;
     private readonly processService;
     private readonly featureRepository;
-    private readonly worktreeService;
     private readonly phaseTimingRepository;
-    constructor(agentRunRepository: IAgentRunRepository, processService: IFeatureAgentProcessService, featureRepository: IFeatureRepository, worktreeService: IWorktreeService, phaseTimingRepository: IPhaseTimingRepository);
-    execute(id: string): Promise<{
+    constructor(agentRunRepository: IAgentRunRepository, processService: IFeatureAgentProcessService, featureRepository: IFeatureRepository, phaseTimingRepository: IPhaseTimingRepository);
+    execute(id: string, payload?: PrdApprovalPayload): Promise<{
         approved: boolean;
         reason: string;
     }>;

package/dist/packages/core/src/application/use-cases/agents/approve-agent-run.use-case.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"approve-agent-run.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/agents/approve-agent-run.use-case.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,8DAA8D,CAAC;AAChH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,gEAAgE,CAAC;AAC7G,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAC1G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,2DAA2D,CAAC;AAGlG,qBACa,sBAAsB;IAG/B,OAAO,CAAC,QAAQ,CAAC,kBAAkB;IAEnC,OAAO,CAAC,QAAQ,CAAC,cAAc;IAE/B,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IAElC,OAAO,CAAC,QAAQ,CAAC,eAAe;IAEhC,OAAO,CAAC,QAAQ,CAAC,qBAAqB;gBARrB,kBAAkB,EAAE,mBAAmB,EAEvC,cAAc,EAAE,2BAA2B,EAE3C,iBAAiB,EAAE,kBAAkB,EAErC,eAAe,EAAE,gBAAgB,EAEjC,qBAAqB,EAAE,sBAAsB;IAG1D,OAAO,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;CA8D1E"}
+{"version":3,"file":"approve-agent-run.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/agents/approve-agent-run.use-case.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,8DAA8D,CAAC;AAChH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,gEAAgE,CAAC;AAC7G,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAE1G,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAG9E,qBACa,sBAAsB;IAG/B,OAAO,CAAC,QAAQ,CAAC,kBAAkB;IAEnC,OAAO,CAAC,QAAQ,CAAC,cAAc;IAE/B,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IAElC,OAAO,CAAC,QAAQ,CAAC,qBAAqB;gBANrB,kBAAkB,EAAE,mBAAmB,EAEvC,cAAc,EAAE,2BAA2B,EAE3C,iBAAiB,EAAE,kBAAkB,EAErC,qBAAqB,EAAE,sBAAsB;IAG1D,OAAO,CACX,EAAE,EAAE,MAAM,EACV,OAAO,CAAC,EAAE,kBAAkB,GAC3B,OAAO,CAAC;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;CAqFlD"}

package/dist/packages/core/src/application/use-cases/agents/approve-agent-run.use-case.js

@@ -3,6 +3,8 @@
  *
  * Approves a paused agent run (waiting_approval status) and
  * spawns a new resume worker to continue graph execution.
+ * Optionally accepts a PrdApprovalPayload to update spec.yaml
+ * selections before resuming.
  */
 var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
     var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
@@ -17,21 +19,23 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
     return function (target, key) { decorator(target, key, paramIndex); }
 };
 import { injectable, inject } from 'tsyringe';
+import yaml from 'js-yaml';
+import { readFileSync } from 'node:fs';
+import { join } from 'node:path';
 import { AgentRunStatus } from '../../../domain/generated/output.js';
+import { writeSpecFileAtomic } from '../../../infrastructure/services/agents/feature-agent/nodes/node-helpers.js';
 let ApproveAgentRunUseCase = class ApproveAgentRunUseCase {
     agentRunRepository;
     processService;
     featureRepository;
-    worktreeService;
     phaseTimingRepository;
-    constructor(agentRunRepository, processService, featureRepository, worktreeService, phaseTimingRepository) {
+    constructor(agentRunRepository, processService, featureRepository, phaseTimingRepository) {
         this.agentRunRepository = agentRunRepository;
         this.processService = processService;
         this.featureRepository = featureRepository;
-        this.worktreeService = worktreeService;
         this.phaseTimingRepository = phaseTimingRepository;
     }
-    async execute(id) {
+    async execute(id, payload) {
         const run = await this.agentRunRepository.findById(id);
         if (!run) {
             return { approved: false, reason: 'Agent run not found' };
@@ -42,12 +46,31 @@ let ApproveAgentRunUseCase = class ApproveAgentRunUseCase {
                 reason: `Agent run is not waiting for approval (status: ${run.status})`,
             };
         }
-        // Look up the feature to get the branch for worktree path derivation
+        // Look up the feature to get specPath
         const feature = run.featureId ? await this.featureRepository.findById(run.featureId) : null;
-        const repoPath = run.repositoryPath ?? '';
-        const worktreePath = feature
-            ? this.worktreeService.getWorktreePath(repoPath, feature.branch)
-            : repoPath;
+        // Write updated selections to spec.yaml if changedSelections provided
+        if (payload?.changedSelections && payload.changedSelections.length > 0 && feature?.specPath) {
+            try {
+                const specDir = feature.specPath;
+                const specContent = readFileSync(join(specDir, 'spec.yaml'), 'utf-8');
+                const spec = yaml.load(specContent);
+                if (Array.isArray(spec?.openQuestions)) {
+                    for (const change of payload.changedSelections) {
+                        const question = spec.openQuestions.find((q) => q.question === change.questionId);
+                        if (question && Array.isArray(question.options)) {
+                            for (const opt of question.options) {
+                                opt.selected = opt.option === change.selectedOption;
+                            }
+                            question.answer = change.selectedOption;
+                        }
+                    }
+                }
+                writeSpecFileAtomic(specDir, 'spec.yaml', yaml.dump(spec));
+            }
+            catch {
+                // Non-fatal: selection update failure should not block approval
+            }
+        }
         const now = new Date();
         await this.agentRunRepository.updateStatus(id, AgentRunStatus.running, {
             updatedAt: now,
@@ -69,14 +92,14 @@ let ApproveAgentRunUseCase = class ApproveAgentRunUseCase {
         catch {
             // Non-fatal: approval wait timing failure should not block approval
         }
-        this.processService.spawn(run.featureId ?? '', id, repoPath, worktreePath, // specDir = worktree path (same as initial spawn)
-        worktreePath, {
+        this.processService.spawn(run.featureId ?? '', id, feature?.repositoryPath ?? run.repositoryPath ?? '', feature?.specPath ?? '', feature?.worktreePath, {
             resume: true,
             approvalGates: run.approvalGates,
             threadId: run.threadId,
             resumeFromInterrupt: true,
             push: feature?.push ?? false,
             openPr: feature?.openPr ?? false,
+            ...(payload ? { resumePayload: JSON.stringify(payload) } : {}),
         });
         return { approved: true, reason: 'Approved and resumed' };
     }
@@ -86,8 +109,7 @@ ApproveAgentRunUseCase = __decorate([
     __param(0, inject('IAgentRunRepository')),
     __param(1, inject('IFeatureAgentProcessService')),
     __param(2, inject('IFeatureRepository')),
-    __param(3, inject('IWorktreeService')),
-    __param(4, inject('IPhaseTimingRepository')),
-    __metadata("design:paramtypes", [Object, Object, Object, Object, Object])
+    __param(3, inject('IPhaseTimingRepository')),
+    __metadata("design:paramtypes", [Object, Object, Object, Object])
 ], ApproveAgentRunUseCase);
 export { ApproveAgentRunUseCase };

package/dist/packages/core/src/application/use-cases/agents/reject-agent-run.use-case.d.ts

@@ -2,15 +2,24 @@
  * Reject Agent Run Use Case
  *
  * Rejects a paused agent run (waiting_approval status) and
- * marks it as cancelled with the rejection reason.
+ * spawns a resume worker to iterate with user feedback.
+ * Appends rejection feedback to spec.yaml for tracking.
  */
 import type { IAgentRunRepository } from '../../ports/output/agents/agent-run-repository.interface.js';
+import type { IFeatureAgentProcessService } from '../../ports/output/agents/feature-agent-process.interface.js';
+import type { IPhaseTimingRepository } from '../../ports/output/agents/phase-timing-repository.interface.js';
+import type { IFeatureRepository } from '../../ports/output/repositories/feature-repository.interface.js';
 export declare class RejectAgentRunUseCase {
     private readonly agentRunRepository;
-    constructor(agentRunRepository: IAgentRunRepository);
-    execute(id: string, reason?: string): Promise<{
+    private readonly processService;
+    private readonly featureRepository;
+    private readonly phaseTimingRepository;
+    constructor(agentRunRepository: IAgentRunRepository, processService: IFeatureAgentProcessService, featureRepository: IFeatureRepository, phaseTimingRepository: IPhaseTimingRepository);
+    execute(id: string, feedback: string): Promise<{
         rejected: boolean;
         reason: string;
+        iteration?: number;
+        iterationWarning?: boolean;
     }>;
 }
 //# sourceMappingURL=reject-agent-run.use-case.d.ts.map

package/dist/packages/core/src/application/use-cases/agents/reject-agent-run.use-case.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reject-agent-run.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/agents/reject-agent-run.use-case.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AAGvG,qBACa,qBAAqB;IAG9B,OAAO,CAAC,QAAQ,CAAC,kBAAkB;gBAAlB,kBAAkB,EAAE,mBAAmB;IAGpD,OAAO,CAAC,EAAE,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;CAsB3F"}
+{"version":3,"file":"reject-agent-run.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/agents/reject-agent-run.use-case.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAMH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,8DAA8D,CAAC;AAChH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,gEAAgE,CAAC;AAC7G,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAQ1G,qBACa,qBAAqB;IAG9B,OAAO,CAAC,QAAQ,CAAC,kBAAkB;IAEnC,OAAO,CAAC,QAAQ,CAAC,cAAc;IAE/B,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IAElC,OAAO,CAAC,QAAQ,CAAC,qBAAqB;gBANrB,kBAAkB,EAAE,mBAAmB,EAEvC,cAAc,EAAE,2BAA2B,EAE3C,iBAAiB,EAAE,kBAAkB,EAErC,qBAAqB,EAAE,sBAAsB;IAG1D,OAAO,CACX,EAAE,EAAE,MAAM,EACV,QAAQ,EAAE,MAAM,GACf,OAAO,CAAC;QACT,QAAQ,EAAE,OAAO,CAAC;QAClB,MAAM,EAAE,MAAM,CAAC;QACf,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,gBAAgB,CAAC,EAAE,OAAO,CAAC;KAC5B,CAAC;CAuGH"}

package/dist/packages/core/src/application/use-cases/agents/reject-agent-run.use-case.js

@@ -2,7 +2,8 @@
  * Reject Agent Run Use Case
  *
  * Rejects a paused agent run (waiting_approval status) and
- * marks it as cancelled with the rejection reason.
+ * spawns a resume worker to iterate with user feedback.
+ * Appends rejection feedback to spec.yaml for tracking.
  */
 var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
     var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
@@ -17,13 +18,23 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
     return function (target, key) { decorator(target, key, paramIndex); }
 };
 import { injectable, inject } from 'tsyringe';
+import yaml from 'js-yaml';
+import { readFileSync } from 'node:fs';
+import { join } from 'node:path';
 import { AgentRunStatus } from '../../../domain/generated/output.js';
+import { writeSpecFileAtomic } from '../../../infrastructure/services/agents/feature-agent/nodes/node-helpers.js';
 let RejectAgentRunUseCase = class RejectAgentRunUseCase {
     agentRunRepository;
-    constructor(agentRunRepository) {
+    processService;
+    featureRepository;
+    phaseTimingRepository;
+    constructor(agentRunRepository, processService, featureRepository, phaseTimingRepository) {
         this.agentRunRepository = agentRunRepository;
+        this.processService = processService;
+        this.featureRepository = featureRepository;
+        this.phaseTimingRepository = phaseTimingRepository;
     }
-    async execute(id, reason) {
+    async execute(id, feedback) {
         const run = await this.agentRunRepository.findById(id);
         if (!run) {
             return { rejected: false, reason: 'Agent run not found' };
@@ -34,18 +45,87 @@ let RejectAgentRunUseCase = class RejectAgentRunUseCase {
                 reason: `Agent run is not waiting for approval (status: ${run.status})`,
             };
         }
+        // Validate non-empty feedback
+        if (!feedback?.trim()) {
+            return { rejected: false, reason: 'Feedback is required for rejection' };
+        }
+        // Look up feature for spec path
+        const feature = run.featureId ? await this.featureRepository.findById(run.featureId) : null;
+        if (!feature?.specPath) {
+            return { rejected: false, reason: 'Feature has no spec path' };
+        }
+        // Read and update spec.yaml with rejection feedback
+        const specDir = feature.specPath;
+        let iteration = 1;
+        try {
+            const specContent = readFileSync(join(specDir, 'spec.yaml'), 'utf-8');
+            const spec = yaml.load(specContent);
+            const existingFeedback = Array.isArray(spec?.rejectionFeedback)
+                ? spec.rejectionFeedback
+                : [];
+            iteration = existingFeedback.length + 1;
+            const newEntry = {
+                iteration,
+                message: feedback,
+                timestamp: new Date().toISOString(),
+            };
+            spec.rejectionFeedback = [...existingFeedback, newEntry];
+            writeSpecFileAtomic(specDir, 'spec.yaml', yaml.dump(spec));
+        }
+        catch {
+            // If spec.yaml can't be read, still proceed with iteration 1
+        }
+        // Update run status to running (NOT cancelled)
         const now = new Date();
-        await this.agentRunRepository.updateStatus(id, AgentRunStatus.cancelled, {
-            error: reason ?? 'Rejected by user',
-            completedAt: now,
+        await this.agentRunRepository.updateStatus(id, AgentRunStatus.running, {
             updatedAt: now,
         });
-        return { rejected: true, reason: 'Rejected and cancelled' };
+        // Record approval wait duration
+        try {
+            const timings = await this.phaseTimingRepository.findByRunId(id);
+            const waitingTiming = timings.find((t) => t.waitingApprovalAt && !t.approvalWaitMs);
+            if (waitingTiming) {
+                const waitStart = waitingTiming.waitingApprovalAt instanceof Date
+                    ? waitingTiming.waitingApprovalAt.getTime()
+                    : Number(waitingTiming.waitingApprovalAt);
+                const approvalWaitMs = BigInt(now.getTime() - waitStart);
+                await this.phaseTimingRepository.updateApprovalWait(waitingTiming.id, {
+                    approvalWaitMs,
+                });
+            }
+        }
+        catch {
+            // Non-fatal
+        }
+        // Spawn worker with rejection payload
+        const rejectionPayload = {
+            rejected: true,
+            feedback,
+            iteration,
+        };
+        this.processService.spawn(run.featureId ?? '', id, feature.repositoryPath, feature.specPath, feature.worktreePath, {
+            resume: true,
+            approvalGates: run.approvalGates,
+            threadId: run.threadId,
+            resumeFromInterrupt: true,
+            push: feature?.push ?? false,
+            openPr: feature?.openPr ?? false,
+            resumePayload: JSON.stringify(rejectionPayload),
+        });
+        return {
+            rejected: true,
+            reason: 'Rejected and iterating',
+            iteration,
+            iterationWarning: iteration >= 5,
+        };
     }
 };
 RejectAgentRunUseCase = __decorate([
     injectable(),
     __param(0, inject('IAgentRunRepository')),
-    __metadata("design:paramtypes", [Object])
+    __param(1, inject('IFeatureAgentProcessService')),
+    __param(2, inject('IFeatureRepository')),
+    __param(3, inject('IPhaseTimingRepository')),
+    __metadata("design:paramtypes", [Object, Object, Object, Object])
 ], RejectAgentRunUseCase);
 export { RejectAgentRunUseCase };

package/dist/packages/core/src/application/use-cases/agents/review-feature.use-case.d.ts

@@ -0,0 +1,35 @@
+/**
+ * Review Feature Use Case
+ *
+ * Reads spec.yaml open questions for a feature waiting for approval,
+ * returning structured data for the TUI review wizard.
+ */
+import type { IFeatureRepository } from '../../ports/output/repositories/feature-repository.interface.js';
+import type { IAgentRunRepository } from '../../ports/output/agents/agent-run-repository.interface.js';
+import type { QuestionOption } from '../../../domain/generated/output.js';
+export interface OpenQuestion {
+    question: string;
+    options: QuestionOption[];
+    selectedOption: string;
+    selectionRationale?: string;
+    answer?: string;
+}
+export interface ReviewFeatureResult {
+    success: true;
+    questions: OpenQuestion[];
+    featureName: string;
+    phase: string;
+    runId: string;
+    repoPath: string;
+}
+export interface ReviewFeatureError {
+    success: false;
+    reason: string;
+}
+export declare class ReviewFeatureUseCase {
+    private readonly featureRepository;
+    private readonly agentRunRepository;
+    constructor(featureRepository: IFeatureRepository, agentRunRepository: IAgentRunRepository);
+    execute(featureId: string, repoPath: string): Promise<ReviewFeatureResult | ReviewFeatureError>;
+}
+//# sourceMappingURL=review-feature.use-case.d.ts.map

package/dist/packages/core/src/application/use-cases/agents/review-feature.use-case.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-feature.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/agents/review-feature.use-case.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAMH,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAC1G,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AAEvG,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qCAAqC,CAAC;AAE1E,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,cAAc,EAAE,CAAC;IAC1B,cAAc,EAAE,MAAM,CAAC;IACvB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,mBAAmB;IAClC,OAAO,EAAE,IAAI,CAAC;IACd,SAAS,EAAE,YAAY,EAAE,CAAC;IAC1B,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,kBAAkB;IACjC,OAAO,EAAE,KAAK,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,qBACa,oBAAoB;IAEC,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IACjC,OAAO,CAAC,QAAQ,CAAC,kBAAkB;gBADnB,iBAAiB,EAAE,kBAAkB,EACpC,kBAAkB,EAAE,mBAAmB;IAGnF,OAAO,CACX,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,GACf,OAAO,CAAC,mBAAmB,GAAG,kBAAkB,CAAC;CAqDrD"}

package/dist/packages/core/src/application/use-cases/agents/review-feature.use-case.js

@@ -0,0 +1,88 @@
+/**
+ * Review Feature Use Case
+ *
+ * Reads spec.yaml open questions for a feature waiting for approval,
+ * returning structured data for the TUI review wizard.
+ */
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+var __metadata = (this && this.__metadata) || function (k, v) {
+    if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
+};
+var __param = (this && this.__param) || function (paramIndex, decorator) {
+    return function (target, key) { decorator(target, key, paramIndex); }
+};
+import { injectable, inject } from 'tsyringe';
+import yaml from 'js-yaml';
+import { readFileSync } from 'node:fs';
+import { join } from 'node:path';
+import { AgentRunStatus } from '../../../domain/generated/output.js';
+let ReviewFeatureUseCase = class ReviewFeatureUseCase {
+    featureRepository;
+    agentRunRepository;
+    constructor(featureRepository, agentRunRepository) {
+        this.featureRepository = featureRepository;
+        this.agentRunRepository = agentRunRepository;
+    }
+    async execute(featureId, repoPath) {
+        // Find feature (try exact match, then prefix)
+        const feature = (await this.featureRepository.findById(featureId)) ??
+            (await this.featureRepository.findByIdPrefix(featureId));
+        if (!feature)
+            return { success: false, reason: 'Feature not found' };
+        // Find agent run
+        if (!feature.agentRunId)
+            return { success: false, reason: 'Feature has no agent run' };
+        const run = await this.agentRunRepository.findById(feature.agentRunId);
+        if (!run || run.status !== AgentRunStatus.waitingApproval) {
+            return {
+                success: false,
+                reason: `Feature is not waiting for approval (status: ${run?.status ?? 'unknown'})`,
+            };
+        }
+        // Read spec.yaml using feature's stored specPath
+        if (!feature.specPath) {
+            return { success: false, reason: 'Feature has no spec path' };
+        }
+        const specPath = join(feature.specPath, 'spec.yaml');
+        let specContent;
+        try {
+            specContent = readFileSync(specPath, 'utf-8');
+        }
+        catch {
+            return { success: false, reason: 'Could not read spec.yaml' };
+        }
+        // Parse openQuestions
+        const spec = yaml.load(specContent);
+        const openQuestions = spec?.openQuestions;
+        if (!Array.isArray(openQuestions) || openQuestions.length === 0) {
+            return { success: false, reason: 'No open questions found in spec.yaml' };
+        }
+        const questions = openQuestions.map((q) => ({
+            question: String(q.question ?? ''),
+            options: Array.isArray(q.options) ? q.options : [],
+            selectedOption: String(q.answer ?? ''),
+            selectionRationale: q.selectionRationale ? String(q.selectionRationale) : undefined,
+            answer: q.answer ? String(q.answer) : undefined,
+        }));
+        return {
+            success: true,
+            questions,
+            featureName: feature.name,
+            phase: 'requirements',
+            runId: run.id,
+            repoPath: run.repositoryPath ?? repoPath,
+        };
+    }
+};
+ReviewFeatureUseCase = __decorate([
+    injectable(),
+    __param(0, inject('IFeatureRepository')),
+    __param(1, inject('IAgentRunRepository')),
+    __metadata("design:paramtypes", [Object, Object])
+], ReviewFeatureUseCase);
+export { ReviewFeatureUseCase };

package/dist/packages/core/src/domain/generated/output.d.ts

@@ -791,6 +791,23 @@ export type TechDecision = {
      */
     rationale: string;
 };
+/**
+ * Rejection feedback entry for PRD iteration tracking
+ */
+export type RejectionFeedbackEntry = {
+    /**
+     * Iteration number (1-based)
+     */
+    iteration: number;
+    /**
+     * User's feedback message explaining what needs to change
+     */
+    message: string;
+    /**
+     * When the rejection occurred
+     */
+    timestamp: any;
+};
 /**
  * Implementation phase grouping related tasks
  */
@@ -898,6 +915,10 @@ export type FeatureArtifact = SpecArtifactBase & {
      * Size estimate: XS, S, M, L, or XL
      */
     sizeEstimate: string;
+    /**
+     * Rejection feedback history for PRD iterations (append-only)
+     */
+    rejectionFeedback?: RejectionFeedbackEntry[];
 };
 /**
  * Research artifact documenting technical analysis and decisions
@@ -1597,6 +1618,87 @@ export type PhaseTiming = BaseEntity & {
      */
     approvalWaitMs?: bigint;
 };
+/**
+ * Change to a question's selected option during PRD review
+ */
+export type QuestionSelectionChange = {
+    /**
+     * ID of the open question being changed (the question text)
+     */
+    questionId: string;
+    /**
+     * The option text that the user selected
+     */
+    selectedOption: string;
+};
+/**
+ * Payload sent when user approves a PRD with optional selection changes
+ */
+export type PrdApprovalPayload = {
+    /**
+     * Always true for approval payloads
+     */
+    approved: boolean;
+    /**
+     * List of selection changes the user made during review (empty if no changes)
+     */
+    changedSelections?: QuestionSelectionChange[];
+};
+/**
+ * Payload sent when user rejects a PRD with feedback for iteration
+ */
+export type PrdRejectionPayload = {
+    /**
+     * Always true for rejection payloads
+     */
+    rejected: boolean;
+    /**
+     * User's feedback explaining what needs to change
+     */
+    feedback: string;
+    /**
+     * Iteration number (1-based, derived from PhaseTiming row count)
+     */
+    iteration: number;
+};
+/**
+ * A single question with its options as presented in the review TUI
+ */
+export type ReviewQuestion = {
+    /**
+     * The question text
+     */
+    question: string;
+    /**
+     * Available options with selection state
+     */
+    options: QuestionOption[];
+    /**
+     * The option text that was selected by the user
+     */
+    selectedOption: string;
+    /**
+     * Whether the user changed the selection from the AI default
+     */
+    changed: boolean;
+};
+/**
+ * Result of the PRD review TUI interaction
+ */
+export type PrdReviewResult = {
+    /**
+     * All questions with their final selection state
+     */
+    questions: ReviewQuestion[];
+    /**
+     * User action: approve or reject
+     */
+    action: string;
+    /**
+     * Rejection feedback (only present when action is 'reject')
+     */
+    feedback?: string;
+};
 /**
  * A selectable option within a PRD questionnaire question
  */