@shepai/cli 1.37.3 → 1.39.0

package/README.md

@@ -407,7 +407,7 @@ Breakpoints are configurable. In `--allow-all` mode, the agent handles everythin
 | TUI Framework   | [@inquirer/prompts](https://github.com/SBoudrias/Inquirer.js)                           |
 | Web UI          | Next.js 16 + React 19 + shadcn/ui ([@shepai/web](./src/presentation/web/))              |
 | Design System   | Storybook                                                                               |
-| Build Tool      | tsc + tsc-alias (prod), Vite (dev)                                                      |
+| Build Tool      | tsc + tsc-alias (prod), tsx (CLI dev), Next.js (web dev)                                |
 | Database        | SQLite                                                                                  |
 | Agent System    | [LangGraph](https://www.langchain.com/langgraph) (`@langchain/langgraph`)               |
 | Vector DB       | [LanceDB](https://lancedb.com/) (file-based)                                            |

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/application/use-cases/features/get-feature-artifact.use-case.d.ts

@@ -0,0 +1,19 @@
+/**
+ * Get Feature Artifact Use Case
+ *
+ * Retrieves the parsed FeatureArtifact (PRD) for a given feature by
+ * reading and parsing the spec.yaml from the feature's spec directory.
+ *
+ * Business Rules:
+ * - Throws if the feature does not exist
+ * - Throws if the feature has no specPath
+ * - Throws if spec.yaml cannot be read or parsed
+ */
+import type { FeatureArtifact } from '../../../domain/generated/output.js';
+import type { IFeatureRepository } from '../../ports/output/repositories/feature-repository.interface.js';
+export declare class GetFeatureArtifactUseCase {
+    private readonly featureRepo;
+    constructor(featureRepo: IFeatureRepository);
+    execute(featureId: string): Promise<FeatureArtifact>;
+}
+//# sourceMappingURL=get-feature-artifact.use-case.d.ts.map

package/dist/packages/core/src/application/use-cases/features/get-feature-artifact.use-case.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"get-feature-artifact.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/features/get-feature-artifact.use-case.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAKH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,qCAAqC,CAAC;AAE3E,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAE1G,qBACa,yBAAyB;IAGlC,OAAO,CAAC,QAAQ,CAAC,WAAW;gBAAX,WAAW,EAAE,kBAAkB;IAG5C,OAAO,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,CAAC;CAgB3D"}

package/dist/packages/core/src/application/use-cases/features/get-feature-artifact.use-case.js

@@ -0,0 +1,52 @@
+/**
+ * Get Feature Artifact Use Case
+ *
+ * Retrieves the parsed FeatureArtifact (PRD) for a given feature by
+ * reading and parsing the spec.yaml from the feature's spec directory.
+ *
+ * Business Rules:
+ * - Throws if the feature does not exist
+ * - Throws if the feature has no specPath
+ * - Throws if spec.yaml cannot be read or parsed
+ */
+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 { readFile } from 'node:fs/promises';
+import { join } from 'node:path';
+import { parseSpecYaml } from '../../../domain/factories/spec-yaml-parser.js';
+let GetFeatureArtifactUseCase = class GetFeatureArtifactUseCase {
+    featureRepo;
+    constructor(featureRepo) {
+        this.featureRepo = featureRepo;
+    }
+    async execute(featureId) {
+        const feature = (await this.featureRepo.findById(featureId)) ??
+            (await this.featureRepo.findByIdPrefix(featureId));
+        if (!feature) {
+            throw new Error(`Feature not found: "${featureId}"`);
+        }
+        if (!feature.specPath) {
+            throw new Error(`Feature "${featureId}" has no spec path`);
+        }
+        const specYamlPath = join(feature.specPath, 'spec.yaml');
+        const content = await readFile(specYamlPath, 'utf-8');
+        return parseSpecYaml(content);
+    }
+};
+GetFeatureArtifactUseCase = __decorate([
+    injectable(),
+    __param(0, inject('IFeatureRepository')),
+    __metadata("design:paramtypes", [Object])
+], GetFeatureArtifactUseCase);
+export { GetFeatureArtifactUseCase };

package/dist/packages/core/src/application/use-cases/features/get-research-artifact.use-case.d.ts

@@ -0,0 +1,19 @@
+/**
+ * Get Research Artifact Use Case
+ *
+ * Retrieves the parsed ResearchArtifact for a given feature by
+ * reading and parsing the research.yaml from the feature's spec directory.
+ *
+ * Business Rules:
+ * - Throws if the feature does not exist
+ * - Throws if the feature has no specPath
+ * - Throws if research.yaml cannot be read or parsed
+ */
+import type { ResearchArtifact } from '../../../domain/generated/output.js';
+import type { IFeatureRepository } from '../../ports/output/repositories/feature-repository.interface.js';
+export declare class GetResearchArtifactUseCase {
+    private readonly featureRepo;
+    constructor(featureRepo: IFeatureRepository);
+    execute(featureId: string): Promise<ResearchArtifact>;
+}
+//# sourceMappingURL=get-research-artifact.use-case.d.ts.map

package/dist/packages/core/src/application/use-cases/features/get-research-artifact.use-case.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"get-research-artifact.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/features/get-research-artifact.use-case.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAKH,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAE5E,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAE1G,qBACa,0BAA0B;IAGnC,OAAO,CAAC,QAAQ,CAAC,WAAW;gBAAX,WAAW,EAAE,kBAAkB;IAG5C,OAAO,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,gBAAgB,CAAC;CAgB5D"}

package/dist/packages/core/src/application/use-cases/features/get-research-artifact.use-case.js

@@ -0,0 +1,52 @@
+/**
+ * Get Research Artifact Use Case
+ *
+ * Retrieves the parsed ResearchArtifact for a given feature by
+ * reading and parsing the research.yaml from the feature's spec directory.
+ *
+ * Business Rules:
+ * - Throws if the feature does not exist
+ * - Throws if the feature has no specPath
+ * - Throws if research.yaml cannot be read or parsed
+ */
+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 { readFile } from 'node:fs/promises';
+import { join } from 'node:path';
+import { parseResearchYaml } from '../../../domain/factories/spec-yaml-parser.js';
+let GetResearchArtifactUseCase = class GetResearchArtifactUseCase {
+    featureRepo;
+    constructor(featureRepo) {
+        this.featureRepo = featureRepo;
+    }
+    async execute(featureId) {
+        const feature = (await this.featureRepo.findById(featureId)) ??
+            (await this.featureRepo.findByIdPrefix(featureId));
+        if (!feature) {
+            throw new Error(`Feature not found: "${featureId}"`);
+        }
+        if (!feature.specPath) {
+            throw new Error(`Feature "${featureId}" has no spec path`);
+        }
+        const researchYamlPath = join(feature.specPath, 'research.yaml');
+        const content = await readFile(researchYamlPath, 'utf-8');
+        return parseResearchYaml(content);
+    }
+};
+GetResearchArtifactUseCase = __decorate([
+    injectable(),
+    __param(0, inject('IFeatureRepository')),
+    __metadata("design:paramtypes", [Object])
+], GetResearchArtifactUseCase);
+export { GetResearchArtifactUseCase };

package/dist/packages/core/src/domain/factories/spec-yaml-parser.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"spec-yaml-parser.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/domain/factories/spec-yaml-parser.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAOH,OAAO,KAAK,EACV,eAAe,EACf,gBAAgB,EAChB,qBAAqB,EACrB,aAAa,EACb,aAAa,EACd,MAAM,qBAAqB,CAAC;AA8E7B;;GAEG;AACH,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,GAAG,eAAe,CAI9D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,GAAG,gBAAgB,CAInE;AAED;;GAEG;AACH,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,GAAG,qBAAqB,CAIpE;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,MAAM,GAAG,aAAa,CAI7D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,MAAM,GAAG,aAAa,CAIrE"}
+{"version":3,"file":"spec-yaml-parser.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/domain/factories/spec-yaml-parser.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAOH,OAAO,KAAK,EACV,eAAe,EACf,gBAAgB,EAChB,qBAAqB,EACrB,aAAa,EACb,aAAa,EACd,MAAM,qBAAqB,CAAC;AAgF7B;;GAEG;AACH,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,GAAG,eAAe,CAI9D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,GAAG,gBAAgB,CAInE;AAED;;GAEG;AACH,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,GAAG,qBAAqB,CAIpE;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,MAAM,GAAG,aAAa,CAI7D;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,MAAM,GAAG,aAAa,CAIrE"}

package/dist/packages/core/src/domain/factories/spec-yaml-parser.js

@@ -41,7 +41,9 @@ function validateSchema(schemaId, data) {
         throw new Error(`Schema not found: ${schemaId}`);
     }
     if (!validate(data)) {
-        const errors = validate.errors?.map((e) => `${e.instancePath} ${e.message}`).join('; ');
+        const errors = validate.errors
+            ?.map((e) => `${e.instancePath} ${e.message}`)
+            .join('; ');
         throw new Error(`Validation failed for ${schemaId}: ${errors}`);
     }
     return data;