@shepai/cli 1.121.0-pr388.bf8e9d3 → 1.121.0-pr391.c78e14b

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

@@ -19,6 +19,8 @@ export declare class ResumeFeatureUseCase {
     private readonly processService;
     private readonly worktreeService;
     constructor(featureRepo: IFeatureRepository, runRepo: IAgentRunRepository, processService: IFeatureAgentProcessService, worktreeService: IWorktreeService);
-    execute(featureId: string): Promise<ResumeFeatureResult>;
+    execute(featureId: string, options?: {
+        promptPrefix?: string;
+    }): Promise<ResumeFeatureResult>;
 }
 //# sourceMappingURL=resume-feature.use-case.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"resume-feature.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/features/resume-feature.use-case.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,qCAAqC,CAAC;AAE7E,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAC1G,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,8DAA8D,CAAC;AAChH,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,2DAA2D,CAAC;AAQlG,MAAM,WAAW,mBAAmB;IAClC,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,QAAQ,CAAC;CAClB;AAED,qBACa,oBAAoB;IAG7B,OAAO,CAAC,QAAQ,CAAC,WAAW;IAE5B,OAAO,CAAC,QAAQ,CAAC,OAAO;IAExB,OAAO,CAAC,QAAQ,CAAC,cAAc;IAE/B,OAAO,CAAC,QAAQ,CAAC,eAAe;gBANf,WAAW,EAAE,kBAAkB,EAE/B,OAAO,EAAE,mBAAmB,EAE5B,cAAc,EAAE,2BAA2B,EAE3C,eAAe,EAAE,gBAAgB;IAG9C,OAAO,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,mBAAmB,CAAC;CAqG/D"}
+{"version":3,"file":"resume-feature.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/features/resume-feature.use-case.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,qCAAqC,CAAC;AAE7E,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAC1G,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,8DAA8D,CAAC;AAChH,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,2DAA2D,CAAC;AAQlG,MAAM,WAAW,mBAAmB;IAClC,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,QAAQ,CAAC;CAClB;AAED,qBACa,oBAAoB;IAG7B,OAAO,CAAC,QAAQ,CAAC,WAAW;IAE5B,OAAO,CAAC,QAAQ,CAAC,OAAO;IAExB,OAAO,CAAC,QAAQ,CAAC,cAAc;IAE/B,OAAO,CAAC,QAAQ,CAAC,eAAe;gBANf,WAAW,EAAE,kBAAkB,EAE/B,OAAO,EAAE,mBAAmB,EAE5B,cAAc,EAAE,2BAA2B,EAE3C,eAAe,EAAE,gBAAgB;IAG9C,OAAO,CACX,SAAS,EAAE,MAAM,EACjB,OAAO,CAAC,EAAE;QAAE,YAAY,CAAC,EAAE,MAAM,CAAA;KAAE,GAClC,OAAO,CAAC,mBAAmB,CAAC;CAuGhC"}

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

@@ -35,7 +35,7 @@ let ResumeFeatureUseCase = class ResumeFeatureUseCase {
         this.processService = processService;
         this.worktreeService = worktreeService;
     }
-    async execute(featureId) {
+    async execute(featureId, options) {
         // Resolve feature by exact ID or prefix
         const feature = (await this.featureRepo.findById(featureId)) ??
             (await this.featureRepo.findByIdPrefix(featureId));
@@ -79,7 +79,9 @@ let ResumeFeatureUseCase = class ResumeFeatureUseCase {
             agentType: lastRun.agentType,
             agentName: lastRun.agentName,
             status: AgentRunStatus.pending,
-            prompt: lastRun.prompt,
+            prompt: options?.promptPrefix
+                ? `${options.promptPrefix}\n\n${lastRun.prompt}`
+                : lastRun.prompt,
             threadId: lastRun.threadId, // Same thread for checkpoint continuity
             featureId: feature.id,
             repositoryPath: feature.repositoryPath,

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence-output-parser.d.ts

@@ -4,6 +4,9 @@
  * Extracts structured Evidence records from free-form agent text output.
  * Looks for a fenced JSON code block containing an array of evidence objects.
  * Returns empty array gracefully on any parsing failure.
+ *
+ * Also provides validation to ensure UI-related evidence includes app-level
+ * proof (not just Storybook screenshots).
  */
 import type { Evidence } from '../../../../../domain/generated/output.js';
 /**
@@ -12,4 +15,22 @@ import type { Evidence } from '../../../../../domain/generated/output.js';
  * Returns empty array when no block found, JSON is malformed, or no valid records exist.
  */
 export declare function parseEvidenceRecords(output: string): Evidence[];
+export interface UiEvidenceValidationResult {
+    valid: boolean;
+    hasScreenshots: boolean;
+    hasAppScreenshots: boolean;
+    hasOnlyStorybookScreenshots: boolean;
+    warnings: string[];
+}
+/**
+ * Validate that UI-related evidence includes app-level proof, not just Storybook screenshots.
+ *
+ * Returns a validation result indicating whether the evidence set meets the requirement
+ * for app-level proof. Screenshots/videos are considered "Storybook-only" if their
+ * description or path contains storybook-related keywords and NO screenshots have
+ * app-level indicators.
+ *
+ * This validation is informational — it produces warnings but does not filter out records.
+ */
+export declare function validateUiEvidenceHasAppProof(evidence: Evidence[]): UiEvidenceValidationResult;
 //# sourceMappingURL=evidence-output-parser.d.ts.map

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence-output-parser.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"evidence-output-parser.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence-output-parser.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,2CAA2C,CAAC;AAwB1E;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,MAAM,GAAG,QAAQ,EAAE,CAc/D"}
+{"version":3,"file":"evidence-output-parser.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence-output-parser.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,2CAA2C,CAAC;AAwB1E;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,MAAM,GAAG,QAAQ,EAAE,CAc/D;AAwCD,MAAM,WAAW,0BAA0B;IACzC,KAAK,EAAE,OAAO,CAAC;IACf,cAAc,EAAE,OAAO,CAAC;IACxB,iBAAiB,EAAE,OAAO,CAAC;IAC3B,2BAA2B,EAAE,OAAO,CAAC;IACrC,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAED;;;;;;;;;GASG;AACH,wBAAgB,6BAA6B,CAAC,QAAQ,EAAE,QAAQ,EAAE,GAAG,0BAA0B,CAsC9F"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence-output-parser.js

@@ -4,6 +4,9 @@
  * Extracts structured Evidence records from free-form agent text output.
  * Looks for a fenced JSON code block containing an array of evidence objects.
  * Returns empty array gracefully on any parsing failure.
+ *
+ * Also provides validation to ensure UI-related evidence includes app-level
+ * proof (not just Storybook screenshots).
  */
 import { EvidenceType } from '../../../../../domain/generated/output.js';
 // Matches a fenced JSON code block: ```json ... ```
@@ -46,3 +49,71 @@ export function parseEvidenceRecords(output) {
         return [];
     return parsed.filter(isValidRecord);
 }
+/**
+ * Patterns that indicate a screenshot is from Storybook rather than the actual app.
+ * Checked case-insensitively against description and relativePath.
+ */
+const STORYBOOK_PATTERNS = [/storybook/i, /story\b/i, /stories/i, /\b6006\b/];
+/**
+ * Patterns that indicate a screenshot is from the actual running application.
+ * Checked case-insensitively against description and relativePath.
+ */
+const APP_PROOF_PATTERNS = [/\bapp[:\-\s]/i, /dev\s*server/i, /running\s*app/i, /actual\s*app/i];
+function isScreenshot(e) {
+    return e.type === EvidenceType.Screenshot || e.type === EvidenceType.Video;
+}
+function matchesAny(text, patterns) {
+    return patterns.some((p) => p.test(text));
+}
+function isStorybookEvidence(e) {
+    return (matchesAny(e.description, STORYBOOK_PATTERNS) || matchesAny(e.relativePath, STORYBOOK_PATTERNS));
+}
+function isAppEvidence(e) {
+    // Explicitly marked as app-level evidence
+    if (matchesAny(e.description, APP_PROOF_PATTERNS) ||
+        matchesAny(e.relativePath, APP_PROOF_PATTERNS)) {
+        return true;
+    }
+    // A screenshot that is NOT storybook is treated as app-level evidence
+    return !isStorybookEvidence(e);
+}
+/**
+ * Validate that UI-related evidence includes app-level proof, not just Storybook screenshots.
+ *
+ * Returns a validation result indicating whether the evidence set meets the requirement
+ * for app-level proof. Screenshots/videos are considered "Storybook-only" if their
+ * description or path contains storybook-related keywords and NO screenshots have
+ * app-level indicators.
+ *
+ * This validation is informational — it produces warnings but does not filter out records.
+ */
+export function validateUiEvidenceHasAppProof(evidence) {
+    const screenshots = evidence.filter(isScreenshot);
+    const warnings = [];
+    if (screenshots.length === 0) {
+        return {
+            valid: true,
+            hasScreenshots: false,
+            hasAppScreenshots: false,
+            hasOnlyStorybookScreenshots: false,
+            warnings,
+        };
+    }
+    const appScreenshots = screenshots.filter(isAppEvidence);
+    const storybookScreenshots = screenshots.filter(isStorybookEvidence);
+    const hasOnlyStorybook = storybookScreenshots.length > 0 && appScreenshots.length === 0;
+    if (hasOnlyStorybook) {
+        warnings.push('UI evidence contains only Storybook screenshots. App-level screenshots from the running application are REQUIRED for UI features.');
+    }
+    if (screenshots.length > 0 && appScreenshots.length === 0 && storybookScreenshots.length === 0) {
+        // Screenshots exist but none clearly identified as app or storybook — warn to be explicit
+        warnings.push('UI screenshots lack clear app-level indicators. Include "app" in the description or filename to confirm they are from the actual running application.');
+    }
+    return {
+        valid: !hasOnlyStorybook,
+        hasScreenshots: screenshots.length > 0,
+        hasAppScreenshots: appScreenshots.length > 0,
+        hasOnlyStorybookScreenshots: hasOnlyStorybook,
+        warnings,
+    };
+}

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

@@ -1 +1 @@
-{"version":3,"file":"evidence.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAGH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAerD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,cAAc,IAG3C,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAuE7E"}
+{"version":3,"file":"evidence.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAGH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAerD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,cAAc,IAG3C,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAkF7E"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence.node.js

@@ -22,7 +22,7 @@ import { reportNodeStart } from '../heartbeat.js';
 import { recordPhaseStart, recordPhaseEnd } from '../phase-timing-context.js';
 import { updateNodeLifecycle } from '../lifecycle-context.js';
 import { buildEvidencePrompt } from './prompts/evidence-prompts.js';
-import { parseEvidenceRecords } from './evidence-output-parser.js';
+import { parseEvidenceRecords, validateUiEvidenceHasAppProof } from './evidence-output-parser.js';
 import { hasSettings, getSettings } from '../../../settings.service.js';
 /**
  * Factory that creates the evidence collection node function.
@@ -70,6 +70,15 @@ export function createEvidenceNode(executor) {
                 log.error(`Warning: evidence parsing failed: ${msg} — continuing with empty evidence`);
                 evidence = [];
             }
+            // --- Validate UI evidence includes app-level proof ---
+            const validationResult = validateUiEvidenceHasAppProof(evidence);
+            const validationMessages = [];
+            if (validationResult.warnings.length > 0) {
+                for (const warning of validationResult.warnings) {
+                    log.error(`Evidence validation: ${warning}`);
+                    validationMessages.push(`[evidence] Warning: ${warning}`);
+                }
+            }
             await recordPhaseEnd(timingId, durationMs);
             markPhaseComplete(state.specDir, 'evidence', log);
             return {
@@ -80,6 +89,7 @@ export function createEvidenceNode(executor) {
                     ...(evidence.length === 0
                         ? ['[evidence] Warning: no evidence records parsed from agent output']
                         : []),
+                    ...validationMessages,
                 ],
                 _needsReexecution: false,
             };

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fast-implement.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAKH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAUrD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,cAAc,IAGhD,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAoD7E"}
+{"version":3,"file":"fast-implement.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAMH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAgBrD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,cAAc,IAGhD,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CA0E7E"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.js

@@ -9,10 +9,11 @@
  * Follows the same factory pattern as other nodes: takes executor
  * dependency, returns async (state) => Partial<FeatureAgentState>.
  */
+import { execSync } from 'node:child_process';
 import { mkdirSync, writeFileSync } from 'node:fs';
 import { join, dirname } from 'node:path';
 import { isGraphBubbleUp } from '@langchain/langgraph';
-import { createNodeLogger, buildExecutorOptions, retryExecute } from './node-helpers.js';
+import { createNodeLogger, buildExecutorOptions, retryExecute, getCompletedPhases, markPhaseComplete, } from './node-helpers.js';
 import { reportNodeStart } from '../heartbeat.js';
 import { recordPhaseStart, recordPhaseEnd } from '../phase-timing-context.js';
 import { updateNodeLifecycle } from '../lifecycle-context.js';
@@ -33,6 +34,16 @@ export function createFastImplementNode(executor) {
         log.info('Starting fast implementation');
         reportNodeStart('fast-implement');
         await updateNodeLifecycle('fast-implement');
+        // Skip if already completed (resume from error path)
+        const completedPhases = getCompletedPhases(state.specDir);
+        if (completedPhases.includes('fast-implement')) {
+            log.info('Phase already completed, skipping execution');
+            return {
+                currentNode: 'fast-implement',
+                messages: ['[fast-implement] already completed — skipping'],
+                _needsReexecution: false,
+            };
+        }
         const startTime = Date.now();
         const timingId = await recordPhaseStart('fast-implement');
         try {
@@ -44,6 +55,11 @@ export function createFastImplementNode(executor) {
             const durationMs = Date.now() - startTime;
             const elapsed = (durationMs / 1000).toFixed(1);
             log.info(`Complete (${result.result.length} chars, ${elapsed}s)`);
+            // Validate that the executor actually produced file changes
+            const cwd = state.worktreePath || state.repositoryPath;
+            if (!hasWorktreeChanges(cwd)) {
+                throw new Error('[fast-implement] Agent produced no file changes — it may have entered plan mode or asked questions instead of implementing. Retrying.');
+            }
             // --- Evidence sub-agent: capture proof of completion (settings-gated) ---
             const evidenceEnabled = hasSettings() && getSettings().workflow.enableEvidence;
             const evidence = evidenceEnabled ? await collectEvidence(executor, state, log) : [];
@@ -51,6 +67,8 @@ export function createFastImplementNode(executor) {
                 log.info('Evidence collection disabled via settings — skipping');
             }
             await recordPhaseEnd(timingId, durationMs);
+            // Mark phase complete so resume from error skips re-execution
+            markPhaseComplete(state.specDir, 'fast-implement', log);
             return {
                 currentNode: 'fast-implement',
                 evidence,
@@ -107,6 +125,21 @@ async function collectEvidence(executor, state, log) {
         return [];
     }
 }
+/**
+ * Check whether the worktree has any uncommitted changes (new, modified, or deleted files).
+ * Uses `git status --porcelain` which outputs one line per changed file, or empty if clean.
+ * Returns false if the git command fails (e.g. not a git repo).
+ */
+function hasWorktreeChanges(cwd) {
+    try {
+        const output = execSync('git status --porcelain', { cwd, encoding: 'utf-8' });
+        return output.trim().length > 0;
+    }
+    catch {
+        // If git command fails, assume no changes (conservative — will trigger the error)
+        return false;
+    }
+}
 /**
  * Save evidence manifest to the shep home evidence folder so the
  * merge review UI can read it without accessing graph state.

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

@@ -1 +1 @@
-{"version":3,"file":"evidence-prompts.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/evidence-prompts.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAIH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAExD,MAAM,WAAW,qBAAqB;IACpC,kEAAkE;IAClE,cAAc,EAAE,OAAO,CAAC;CACzB;AAED,wBAAgB,mBAAmB,CACjC,KAAK,EAAE,iBAAiB,EACxB,OAAO,GAAE,qBAAiD,GACzD,MAAM,CA8LR"}
+{"version":3,"file":"evidence-prompts.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/evidence-prompts.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAIH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAExD,MAAM,WAAW,qBAAqB;IACpC,kEAAkE;IAClE,cAAc,EAAE,OAAO,CAAC;CACzB;AAED,wBAAgB,mBAAmB,CACjC,KAAK,EAAE,iBAAiB,EACxB,OAAO,GAAE,qBAAiD,GACzD,MAAM,CA0MR"}

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

@@ -90,11 +90,17 @@ ${cwd}
 
 Review the completed tasks above and capture evidence for each.
 
-### MANDATORY Screenshot Rule
+### MANDATORY Screenshot Rule — App-Level Proof Required
 
-**If ANY task touches UI code (components, pages, views, styles, layouts, templates, JSX/TSX, HTML, CSS), you MUST capture at least one Screenshot.** This is non-negotiable. Changing a heading, button label, color, layout, or any visible element requires a screenshot proving the change renders correctly.
+**If ANY task touches UI code (components, pages, views, styles, layouts, templates, JSX/TSX, HTML, CSS), you MUST capture at least one Screenshot from the ACTUAL RUNNING APPLICATION (dev server).** This is non-negotiable. Changing a heading, button label, color, layout, or any visible element requires a screenshot proving the change renders correctly in the real app context.
 
-To capture screenshots:
+**CRITICAL: Storybook-only evidence is NOT sufficient for UI features.** Storybook screenshots may be included as supplementary evidence, but they do NOT replace app-level screenshots. You MUST capture screenshots from the actual running application (via the dev server) to prove the feature works in its real context with actual data, routing, and integration.
+
+For each UI change you MUST capture:
+1. **App screenshot (REQUIRED)**: A screenshot from the running dev server showing the feature in its actual app context. Include "app" or the actual page/route name in the screenshot file name (e.g., \`app-dashboard-new-toggle.png\`, \`app-settings-page.png\`).
+2. **Storybook screenshot (OPTIONAL, supplementary)**: If the component has a Storybook story, you may also capture a Storybook screenshot as supplementary evidence. Include "storybook" in the file name (e.g., \`storybook-toggle-component.png\`).
+
+To capture app-level screenshots:
 1. Install Playwright if not already available: \`npx playwright install chromium --with-deps 2>/dev/null || npx playwright install chromium\`
 2. Start the dev server in the background (e.g., \`npm run dev &\` or \`pnpm dev &\`)
 3. Wait for the server to be ready
@@ -114,11 +120,16 @@ To capture screenshots:
    \`\`\`
 5. Stop the dev server after capturing
 
+To capture Storybook screenshots (supplementary only):
+1. Start Storybook: \`npx storybook dev -p 6006 &\` or \`pnpm dev:storybook &\`
+2. Navigate to the relevant story and capture a screenshot
+3. Stop Storybook after capturing
+
 If Playwright installation fails, try alternative approaches (e.g., \`npx puppeteer\`, \`curl\` for API responses). Only skip screenshots if ALL browser automation approaches fail — and in that case, explicitly note the failure reason in the evidence description.
 
 ### Evidence by Task Type
 
-- **UI/Web tasks**: You MUST take screenshots (see mandatory rule above). Also run and capture test output.
+- **UI/Web tasks**: You MUST take screenshots FROM THE RUNNING APP (see mandatory rule above). Storybook-only screenshots are NOT acceptable as primary evidence. Also run and capture test output.
 - **Backend/API tasks**: Run the relevant test suite and capture test output showing passing tests.
 - **CLI tasks**: Run the CLI commands and capture terminal output demonstrating correct behavior.
 - **Refactoring/Infrastructure tasks**: Capture test output or build output confirming no regressions.
@@ -133,9 +144,10 @@ If Playwright installation fails, try alternative approaches (e.g., \`npx puppet
 ### Minimum Evidence Requirements
 
 Your output MUST include:
-1. At least one **Screenshot** if any task modifies UI/visual code (components, styles, templates, markup)
+1. At least one **Screenshot from the running app** (NOT Storybook) if any task modifies UI/visual code (components, styles, templates, markup). The screenshot description MUST clearly indicate it was captured from the actual application (e.g., "App: dashboard page showing new toggle", "App: settings page with updated layout"). Do NOT use descriptions that suggest Storybook-only proof.
 2. At least one **TestOutput** if the project has a test suite
-3. Do NOT output only TestOutput/TerminalRecording when the feature involves visible UI changes — screenshots are mandatory in that case
+3. Do NOT output only TestOutput/TerminalRecording when the feature involves visible UI changes — app-level screenshots are mandatory in that case
+4. Do NOT submit only Storybook screenshots for UI features — each UI screenshot MUST have a corresponding app-level screenshot proving the feature works in its real context
 
 ${storageSection}
 

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/fast-implement.prompt.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fast-implement.prompt.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/fast-implement.prompt.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAwIxD;;;;;;;;;;;GAWG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,iBAAiB,GAAG,MAAM,CAoEzE"}
+{"version":3,"file":"fast-implement.prompt.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/fast-implement.prompt.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAwIxD;;;;;;;;;;;GAWG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,iBAAiB,GAAG,MAAM,CAuEzE"}

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

@@ -211,6 +211,9 @@ ${dirListing}
 - Follow existing codebase conventions and patterns
 - Do NOT commit, push, or create pull requests
 - Do NOT modify any spec YAML files
-- Keep changes focused and minimal`);
+- Keep changes focused and minimal
+- Do NOT enter plan mode — implement directly without planning phases
+- Do NOT ask the user questions or use AskUserQuestion — make reasonable decisions and proceed
+- You MUST create or modify actual code files — a plan or summary alone is not acceptable output`);
     return sections.join('\n\n');
 }

package/dist/packages/core/src/infrastructure/services/git/git-pr.service.d.ts

@@ -19,6 +19,7 @@ export declare class GitPrService implements IGitPrService {
     push(cwd: string, branch: string, setUpstream?: boolean): Promise<void>;
     createPr(cwd: string, prYamlPath: string): Promise<PrCreateResult>;
     mergePr(cwd: string, prNumber: number, strategy?: MergeStrategy): Promise<void>;
+    private getPrHeadBranch;
     localMergeSquash(cwd: string, featureBranch: string, baseBranch: string, commitMessage: string, hasRemote?: boolean): Promise<void>;
     mergeBranch(cwd: string, sourceBranch: string, targetBranch: string): Promise<void>;
     getCiStatus(cwd: string, branch: string): Promise<CiStatusResult>;

package/dist/packages/core/src/infrastructure/services/git/git-pr.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"git-pr.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/git/git-pr.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wEAAwE,CAAC;AAC5G,OAAO,KAAK,EACV,cAAc,EAGd,WAAW,EACX,QAAQ,EACR,aAAa,EACb,cAAc,EACd,YAAY,EACb,MAAM,wEAAwE,CAAC;AAUhF,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAE1D,qBACa,YAAa,YAAW,aAAa;IACZ,OAAO,CAAC,QAAQ,CAAC,QAAQ;gBAAR,QAAQ,EAAE,YAAY;IAErE,SAAS,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAKxC,YAAY,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IAgBjD,gBAAgB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAgF9C,QAAQ,CAAC,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAKnD,qBAAqB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAKpD,SAAS,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAWxD,IAAI,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IAYvE,QAAQ,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,CAAC;IAuClE,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,GAAE,aAAwB,GAAG,OAAO,CAAC,IAAI,CAAC;IAczF,gBAAgB,CACpB,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,aAAa,EAAE,MAAM,EACrB,SAAS,UAAQ,GAChB,OAAO,CAAC,IAAI,CAAC;IA4EV,WAAW,CAAC,GAAG,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAUnF,WAAW,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,CAAC;IAsBjE,OAAO,CACX,GAAG,EAAE,MAAM,EACX,MAAM,EAAE,MAAM,EACd,SAAS,CAAC,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,GACvB,OAAO,CAAC,cAAc,CAAC;IA4DpB,YAAY,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IAWhF,gBAAgB,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,CAAC;IAevE,YAAY,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC;IAaxE,OAAO,CAAC,gBAAgB;IAexB,OAAO,CAAC,aAAa;IAmFf,cAAc,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,YAAY,EAAE,CAAC;IAoCpD,kBAAkB,CAAC,GAAG,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC;IAc/E,WAAW,CACf,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,GACvB,OAAO,CAAC,OAAO,CAAC;IA6CnB;;;OAGG;YACW,UAAU;IAmBlB,cAAc,CAClB,GAAG,EAAE,MAAM,EACX,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,MAAM,EACf,WAAW,SAAS,GACnB,OAAO,CAAC,MAAM,CAAC;IAWlB,OAAO,CAAC,WAAW;IAQnB,OAAO,CAAC,cAAc;IAMtB,OAAO,CAAC,gBAAgB;IAKxB,OAAO,CAAC,aAAa;IAcrB,OAAO,CAAC,YAAY;IAepB,OAAO,CAAC,oBAAoB;IAK5B,OAAO,CAAC,aAAa;CAiBtB"}
+{"version":3,"file":"git-pr.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/git/git-pr.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wEAAwE,CAAC;AAC5G,OAAO,KAAK,EACV,cAAc,EAGd,WAAW,EACX,QAAQ,EACR,aAAa,EACb,cAAc,EACd,YAAY,EACb,MAAM,wEAAwE,CAAC;AAUhF,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAE1D,qBACa,YAAa,YAAW,aAAa;IACZ,OAAO,CAAC,QAAQ,CAAC,QAAQ;gBAAR,QAAQ,EAAE,YAAY;IAErE,SAAS,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAKxC,YAAY,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IAgBjD,gBAAgB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAgF9C,QAAQ,CAAC,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAKnD,qBAAqB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAKpD,SAAS,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAWxD,IAAI,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IAYvE,QAAQ,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,CAAC;IAuClE,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,GAAE,aAAwB,GAAG,OAAO,CAAC,IAAI,CAAC;YA6BjF,eAAe;IASvB,gBAAgB,CACpB,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,aAAa,EAAE,MAAM,EACrB,SAAS,UAAQ,GAChB,OAAO,CAAC,IAAI,CAAC;IA4EV,WAAW,CAAC,GAAG,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAUnF,WAAW,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,CAAC;IAsBjE,OAAO,CACX,GAAG,EAAE,MAAM,EACX,MAAM,EAAE,MAAM,EACd,SAAS,CAAC,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,GACvB,OAAO,CAAC,cAAc,CAAC;IA4DpB,YAAY,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IAWhF,gBAAgB,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,CAAC;IAevE,YAAY,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC;IAaxE,OAAO,CAAC,gBAAgB;IAexB,OAAO,CAAC,aAAa;IAmFf,cAAc,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,YAAY,EAAE,CAAC;IAoCpD,kBAAkB,CAAC,GAAG,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC;IAc/E,WAAW,CACf,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,GACvB,OAAO,CAAC,OAAO,CAAC;IA6CnB;;;OAGG;YACW,UAAU;IAmBlB,cAAc,CAClB,GAAG,EAAE,MAAM,EACX,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,MAAM,EACf,WAAW,SAAS,GACnB,OAAO,CAAC,MAAM,CAAC;IAWlB,OAAO,CAAC,WAAW;IAQnB,OAAO,CAAC,cAAc;IAMtB,OAAO,CAAC,gBAAgB;IAKxB,OAAO,CAAC,aAAa;IAcrB,OAAO,CAAC,YAAY;IAepB,OAAO,CAAC,oBAAoB;IAK5B,OAAO,CAAC,aAAa;CAiBtB"}

package/dist/packages/core/src/infrastructure/services/git/git-pr.service.js

@@ -186,13 +186,32 @@ let GitPrService = class GitPrService {
     }
     async mergePr(cwd, prNumber, strategy = 'squash') {
         try {
-            await this.execFile('gh', ['pr', 'merge', String(prNumber), `--${strategy}`, '--auto', '--delete-branch'], { cwd });
+            await this.execFile('gh', ['pr', 'merge', String(prNumber), `--${strategy}`, '--auto'], {
+                cwd,
+            });
         }
         catch (error) {
             const message = error instanceof Error ? error.message : String(error);
             const cause = error instanceof Error ? error : undefined;
             throw new GitPrError(message, GitPrErrorCode.MERGE_FAILED, cause);
         }
+        // Try to delete the remote branch gracefully — not fatal if it fails
+        // (e.g. branch already deleted by GitHub auto-delete, or permissions)
+        try {
+            await this.execFile('gh', [
+                'api',
+                '--method',
+                'DELETE',
+                `repos/{owner}/{repo}/git/refs/heads/${await this.getPrHeadBranch(cwd, prNumber)}`,
+            ], { cwd });
+        }
+        catch {
+            // Branch deletion is best-effort — log-level concern, not an error
+        }
+    }
+    async getPrHeadBranch(cwd, prNumber) {
+        const { stdout } = await this.execFile('gh', ['pr', 'view', String(prNumber), '--json', 'headRefName', '--jq', '.headRefName'], { cwd });
+        return stdout.trim();
     }
     async localMergeSquash(cwd, featureBranch, baseBranch, commitMessage, hasRemote = false) {
         try {

package/dist/src/presentation/web/app/(dashboard)/get-graph-data.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-graph-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/(dashboard)/get-graph-data.ts"],"names":[],"mappings":"AAWA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAE5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAQ1C,wBAAsB,YAAY,IAAI,OAAO,CAAC;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAAC,CAuFxF"}
+{"version":3,"file":"get-graph-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/(dashboard)/get-graph-data.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAE5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAE1C,wBAAsB,YAAY,IAAI,OAAO,CAAC;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAAC,CA+CxF"}

package/dist/src/presentation/web/app/(dashboard)/get-graph-data.js

@@ -1,55 +1,14 @@
 import { resolve } from '../../lib/server-container.js';
-import { CiStatus } from '../../../../../packages/core/src/domain/generated/output.js';
 import { getSettings } from '../../../../../packages/core/src/infrastructure/services/settings.service.js';
 import { layoutWithDagre, CANVAS_LAYOUT_DEFAULTS } from '../../lib/layout-with-dagre.js';
 import { buildGraphNodes } from '../build-graph-nodes.js';
-const CI_STATUS_MAP = {
-    success: CiStatus.Success,
-    failure: CiStatus.Failure,
-    pending: CiStatus.Pending,
-};
 export async function getGraphData() {
     const listFeatures = resolve('ListFeaturesUseCase');
     const listRepos = resolve('ListRepositoriesUseCase');
     const agentRunRepo = resolve('IAgentRunRepository');
-    const gitPrService = resolve('IGitPrService');
-    const featureRepo = resolve('IFeatureRepository');
     const [features, repositories] = await Promise.all([listFeatures.execute(), listRepos.execute()]);
-    // Fetch live PR status for features with open PRs so the canvas
-    // reflects the current GitHub state without waiting for the background watcher.
-    await Promise.all(features
-        .filter((f) => f.pr?.number)
-        .map(async (feature) => {
-        try {
-            const updates = {};
-            // Live mergeable status (only if not yet known)
-            if (feature.pr.mergeable === undefined) {
-                const mergeable = await gitPrService.getMergeableStatus(feature.repositoryPath, feature.pr.number);
-                if (mergeable !== undefined)
-                    updates.mergeable = mergeable;
-            }
-            // Live CI status
-            if (feature.branch) {
-                const ciResult = await gitPrService
-                    .getCiStatus(feature.repositoryPath, feature.branch)
-                    .catch(() => undefined);
-                if (ciResult) {
-                    const ciStatus = CI_STATUS_MAP[ciResult.status] ?? CiStatus.Pending;
-                    if (ciStatus !== feature.pr.ciStatus)
-                        updates.ciStatus = ciStatus;
-                }
-            }
-            if (Object.keys(updates).length > 0) {
-                feature.pr = { ...feature.pr, ...updates };
-                featureRepo.update(feature).catch(() => {
-                    /* best-effort persist */
-                });
-            }
-        }
-        catch {
-            // GitHub fetch failed — use cached value
-        }
-    }));
+    // PR/CI status is kept fresh by PrSyncWatcher (30s background poll).
+    // No live GitHub calls here — use cached DB values for fast response.
     const featuresWithRuns = await Promise.all(features.map(async (feature) => {
         const run = feature.agentRunId ? await agentRunRepo.findById(feature.agentRunId) : null;
         return { feature, run };

package/dist/src/presentation/web/app/actions/approve-feature.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"approve-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/approve-feature.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,sCAAsC,CAAC;AAE/E,wBAAsB,cAAc,CAClC,SAAS,EAAE,MAAM,EACjB,OAAO,CAAC,EAAE,kBAAkB,GAC3B,OAAO,CAAC;IAAE,QAAQ,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CA6BhD"}
+{"version":3,"file":"approve-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/approve-feature.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,sCAAsC,CAAC;AAK/E,wBAAsB,cAAc,CAClC,SAAS,EAAE,MAAM,EACjB,OAAO,CAAC,EAAE,kBAAkB,GAC3B,OAAO,CAAC;IAAE,QAAQ,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAsChD"}

package/dist/src/presentation/web/app/actions/approve-feature.js

@@ -1,5 +1,7 @@
 'use server';
 import { resolve } from '../../lib/server-container.js';
+import { AgentRunStatus } from '../../../../../packages/core/src/domain/generated/output.js';
+const ERROR_STATUSES = new Set([AgentRunStatus.failed, AgentRunStatus.interrupted]);
 export async function approveFeature(featureId, payload) {
     if (!featureId.trim()) {
         return { approved: false, error: 'Feature id is required' };
@@ -13,6 +15,13 @@ export async function approveFeature(featureId, payload) {
         if (!feature.agentRunId) {
             return { approved: false, error: 'Feature has no agent run' };
         }
+        const runRepo = resolve('IAgentRunRepository');
+        const run = await runRepo.findById(feature.agentRunId);
+        if (run && ERROR_STATUSES.has(run.status)) {
+            const resumeUseCase = resolve('ResumeFeatureUseCase');
+            await resumeUseCase.execute(featureId, { promptPrefix: 'User approved. Please continue.' });
+            return { approved: true };
+        }
         const approveUseCase = resolve('ApproveAgentRunUseCase');
         const result = await approveUseCase.execute(feature.agentRunId, payload);
         if (!result.approved) {

package/dist/src/presentation/web/app/actions/reject-feature.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reject-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/reject-feature.ts"],"names":[],"mappings":"AAMA,wBAAsB,aAAa,CACjC,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,WAAW,CAAC,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC;IACT,QAAQ,EAAE,OAAO,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB,CAAC,CAqCD"}
+{"version":3,"file":"reject-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/reject-feature.ts"],"names":[],"mappings":"AAWA,wBAAsB,aAAa,CACjC,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,WAAW,CAAC,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC;IACT,QAAQ,EAAE,OAAO,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB,CAAC,CA8CD"}

package/dist/src/presentation/web/app/actions/reject-feature.js

@@ -1,5 +1,7 @@
 'use server';
 import { resolve } from '../../lib/server-container.js';
+import { AgentRunStatus } from '../../../../../packages/core/src/domain/generated/output.js';
+const ERROR_STATUSES = new Set([AgentRunStatus.failed, AgentRunStatus.interrupted]);
 export async function rejectFeature(featureId, feedback, attachments) {
     if (!featureId.trim()) {
         return { rejected: false, error: 'Feature id is required' };
@@ -16,6 +18,13 @@ export async function rejectFeature(featureId, feedback, attachments) {
         if (!feature.agentRunId) {
             return { rejected: false, error: 'Feature has no agent run' };
         }
+        const runRepo = resolve('IAgentRunRepository');
+        const run = await runRepo.findById(feature.agentRunId);
+        if (run && ERROR_STATUSES.has(run.status)) {
+            const resumeUseCase = resolve('ResumeFeatureUseCase');
+            await resumeUseCase.execute(featureId, { promptPrefix: feedback });
+            return { rejected: true, iteration: 1 };
+        }
         const rejectUseCase = resolve('RejectAgentRunUseCase');
         const result = await rejectUseCase.execute(feature.agentRunId, feedback, attachments);
         if (!result.rejected) {

package/dist/src/presentation/web/app/api/npm-version/route.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/npm-version/route.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAE3C,wBAAsB,GAAG,IAAI,OAAO,CAAC,YAAY,CAAC,CAejD"}
+{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/npm-version/route.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAK3C,wBAAsB,GAAG,IAAI,OAAO,CAAC,YAAY,CAAC,CAsBjD"}

package/dist/src/presentation/web/app/api/npm-version/route.js

@@ -1,5 +1,11 @@
 import { NextResponse } from 'next/server';
+const CACHE_TTL_MS = 5 * 60_000; // 5 minutes
+let cachedVersion = null;
 export async function GET() {
+    // Return cached value if fresh
+    if (cachedVersion && Date.now() - cachedVersion.fetchedAt < CACHE_TTL_MS) {
+        return NextResponse.json({ latest: cachedVersion.latest });
+    }
     const packageName = '@shepai/cli';
     try {
         const res = await fetch(`https://registry.npmjs.org/${packageName}/latest`, {
@@ -9,7 +15,9 @@ export async function GET() {
             return NextResponse.json({ error: `npm registry returned ${res.status}` }, { status: 502 });
         }
         const data = (await res.json());
-        return NextResponse.json({ latest: data.version ?? null });
+        const latest = data.version ?? null;
+        cachedVersion = { latest, fetchedAt: Date.now() };
+        return NextResponse.json({ latest });
     }
     catch (error) {
         const message = error instanceof Error ? error.message : 'Failed to check npm version';

package/dist/src/presentation/web/app/api/sessions/route.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/sessions/route.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAK3C,eAAO,MAAM,OAAO,kBAAkB,CAAC;AA4QvC;;;;;GAKG;AACH,wBAAsB,GAAG,CAAC,OAAO,EAAE,OAAO;;;;YA/QpC,MAAM;mBACC,MAAM;iBACR,MAAM,GAAG,IAAI;sBACR,MAAM;wBACJ,MAAM,GAAG,IAAI;uBACd,MAAM,GAAG,IAAI;mBACjB,MAAM,GAAG,IAAI;qBACX,MAAM;kBACT,MAAM;;IAoSjB"}
+{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/sessions/route.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAK3C,eAAO,MAAM,OAAO,kBAAkB,CAAC;AAgTvC;;;;;GAKG;AACH,wBAAsB,GAAG,CAAC,OAAO,EAAE,OAAO;;;;YAnTpC,MAAM;mBACC,MAAM;iBACR,MAAM,GAAG,IAAI;sBACR,MAAM;wBACJ,MAAM,GAAG,IAAI;uBACd,MAAM,GAAG,IAAI;mBACjB,MAAM,GAAG,IAAI;qBACX,MAAM;kBACT,MAAM;;IAwUjB"}