@shepai/cli 1.121.0-pr375.de718bc → 1.121.0-pr381.1f06300

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;AAgBrD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,cAAc,IAGhD,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAkE7E"}
+{"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,6 +9,7 @@
  * 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';
@@ -54,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) : [];
@@ -119,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/merge/merge.node.d.ts

@@ -36,6 +36,11 @@ export interface MergeNodeDeps {
      * Resolve a branch ref to its current SHA.
      */
     revParse: (cwd: string, ref: string) => Promise<string>;
+    /**
+     * Stage all changes and commit in the given directory.
+     * Returns the commit hash, or undefined if there was nothing to commit.
+     */
+    commitAll: (cwd: string, message: string) => Promise<string | undefined>;
     gitPrService: IGitPrService;
     cleanupFeatureWorktreeUseCase: Pick<CleanupFeatureWorktreeUseCase, 'execute'>;
 }

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

@@ -1 +1 @@
-{"version":3,"file":"merge.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/merge/merge.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAGH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,yEAAyE,CAAC;AAClH,OAAO,KAAK,EACV,WAAW,EACX,aAAa,EACd,MAAM,iEAAiE,CAAC;AAmBzE,OAAO,KAAK,EAAE,6BAA6B,EAAE,MAAM,uEAAuE,CAAC;AAE3H,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,cAAc,CAAC;IACzB,cAAc,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,WAAW,CAAC,CAAC;IAC1E,SAAS,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IAC7C,gBAAgB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACnD,iBAAiB,EAAE,IAAI,CAAC,kBAAkB,EAAE,UAAU,GAAG,QAAQ,CAAC,CAAC;IACnE;;OAEG;IACH,gBAAgB,EAAE,CAChB,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,aAAa,EAAE,MAAM,EACrB,SAAS,CAAC,EAAE,OAAO,KAChB,OAAO,CAAC,IAAI,CAAC,CAAC;IACnB;;;OAGG;IACH,WAAW,EAAE,CACX,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,KACrB,OAAO,CAAC,OAAO,CAAC,CAAC;IACtB;;OAEG;IACH,QAAQ,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACxD,YAAY,EAAE,aAAa,CAAC;IAC5B,6BAA6B,EAAE,IAAI,CAAC,6BAA6B,EAAE,SAAS,CAAC,CAAC;CAC/E;AAED;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,aAAa,IAGnC,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CA+T7E"}
+{"version":3,"file":"merge.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/merge/merge.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAGH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,yEAAyE,CAAC;AAClH,OAAO,KAAK,EACV,WAAW,EACX,aAAa,EACd,MAAM,iEAAiE,CAAC;AAmBzE,OAAO,KAAK,EAAE,6BAA6B,EAAE,MAAM,uEAAuE,CAAC;AAE3H,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,cAAc,CAAC;IACzB,cAAc,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,WAAW,CAAC,CAAC;IAC1E,SAAS,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IAC7C,gBAAgB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACnD,iBAAiB,EAAE,IAAI,CAAC,kBAAkB,EAAE,UAAU,GAAG,QAAQ,CAAC,CAAC;IACnE;;OAEG;IACH,gBAAgB,EAAE,CAChB,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,aAAa,EAAE,MAAM,EACrB,SAAS,CAAC,EAAE,OAAO,KAChB,OAAO,CAAC,IAAI,CAAC,CAAC;IACnB;;;OAGG;IACH,WAAW,EAAE,CACX,GAAG,EAAE,MAAM,EACX,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,KACrB,OAAO,CAAC,OAAO,CAAC,CAAC;IACtB;;OAEG;IACH,QAAQ,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACxD;;;OAGG;IACH,SAAS,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CAAC;IACzE,YAAY,EAAE,aAAa,CAAC;IAC5B,6BAA6B,EAAE,IAAI,CAAC,6BAA6B,EAAE,SAAS,CAAC,CAAC;CAC/E;AAED;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,aAAa,IAGnC,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CA0V7E"}

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

@@ -98,66 +98,94 @@ export function createMergeNode(deps) {
                 : undefined;
             // --- Agent Call 1: Commit + Push + PR (skip on approval resume) ---
             if (!isResumeAfterInterrupt) {
-                if (!remoteAvailable) {
-                    log.info('No git remote configured — skipping push and PR, will merge locally');
-                }
                 const effectiveState = remoteAvailable ? state : { ...state, push: false, openPr: false };
-                log.info('Agent call 1: commit + push + PR');
-                const commitPushPrPrompt = buildCommitPushPrPrompt(effectiveState, branch, baseBranch, repoUrl);
-                const commitResult = await retryExecute(executor, commitPushPrPrompt, options, {
-                    logger: log,
-                });
-                totalInputTokens += commitResult.usage?.inputTokens ?? 0;
-                totalOutputTokens += commitResult.usage?.outputTokens ?? 0;
-                totalCostUsd += commitResult.usage?.costUsd ?? 0;
-                totalNumTurns += commitResult.usage?.numTurns ?? 0;
-                totalDurationApiMs += commitResult.usage?.durationApiMs ?? 0;
-                commitHash = parseCommitHash(commitResult.result) ?? state.commitHash;
-                messages.push(`[merge] Agent completed commit/push/PR operations`);
-                if (effectiveState.openPr) {
-                    const prResult = parsePrUrl(commitResult.result);
-                    if (prResult) {
-                        prUrl = prResult.url;
-                        prNumber = prResult.number;
-                        // Cross-validate agent-parsed PR URL against authoritative source.
-                        // The agent may hallucinate the repo URL or PR number, so we look up
-                        // the real PR for this branch via the GitHub API.
-                        try {
-                            const prStatuses = await deps.gitPrService.listPrStatuses(cwd);
-                            const matchingPr = prStatuses.find((pr) => pr.headRefName === branch);
-                            if (matchingPr) {
-                                prUrl = matchingPr.url;
-                                prNumber = matchingPr.number;
-                            }
+                const needsAgentCall = effectiveState.push || effectiveState.openPr;
+                if (!needsAgentCall) {
+                    // Local-only: commit programmatically in worktree, no agent needed.
+                    // This avoids calling the agent executor just to run git commit,
+                    // which is slow/unreliable on some platforms (e.g. cursor on Windows hangs).
+                    log.info('Local-only mode — committing in worktree programmatically (no agent)');
+                    const worktreeCwd = state.worktreePath ?? cwd;
+                    try {
+                        const msg = `feat: ${branch}`;
+                        const hash = await deps.commitAll(worktreeCwd, msg);
+                        if (hash) {
+                            commitHash = hash;
+                            log.info(`Committed changes in worktree: ${commitHash}`);
+                            messages.push(`[merge] Programmatic commit: ${commitHash}`);
                         }
-                        catch {
-                            // gh CLI unavailable or API failure — fall back to agent-parsed URL
+                        else {
+                            log.info('No changes to commit in worktree');
+                            messages.push(`[merge] No changes to commit`);
                         }
-                        messages.push(`[merge] PR created: ${prUrl}`);
+                    }
+                    catch (commitErr) {
+                        const errMsg = commitErr instanceof Error ? commitErr.message : String(commitErr);
+                        log.info(`Programmatic commit failed: ${errMsg} — falling back to agent`);
+                        messages.push(`[merge] Programmatic commit failed, falling back to agent`);
+                        // Fall through — needsAgentCall stays false so agent won't run either.
+                        // The localMergeSquash will handle uncommitted changes via --squash.
                     }
                 }
-                // --- CI watch/fix loop (when push or openPr is enabled and CI watch is not disabled) ---
-                const ciWatchEnabled = getSettings().workflow?.ciWatchEnabled !== false;
-                if (ciWatchEnabled && (effectiveState.push || effectiveState.openPr)) {
-                    const ciResult = await runCiWatchFixLoop({
-                        executor,
-                        gitPrService: deps.gitPrService,
-                        featureRepository: deps.featureRepository,
-                    }, {
-                        cwd,
-                        branch,
-                        options,
-                        feature,
-                        prUrl,
-                        prNumber,
-                        existingAttempts: ciFixAttempts,
-                        messages,
-                        log,
+                if (needsAgentCall) {
+                    if (!remoteAvailable) {
+                        log.info('No git remote configured — skipping push and PR, will merge locally');
+                    }
+                    log.info('Agent call 1: commit + push + PR');
+                    const commitPushPrPrompt = buildCommitPushPrPrompt(effectiveState, branch, baseBranch, repoUrl);
+                    const commitResult = await retryExecute(executor, commitPushPrPrompt, options, {
+                        logger: log,
                     });
-                    ciStatus = ciResult.ciStatus;
-                    ciFixAttempts = ciResult.ciFixAttempts;
-                    ciFixHistory = ciResult.ciFixHistory;
-                    ciFixStatus = ciResult.ciFixStatus;
+                    totalInputTokens += commitResult.usage?.inputTokens ?? 0;
+                    totalOutputTokens += commitResult.usage?.outputTokens ?? 0;
+                    totalCostUsd += commitResult.usage?.costUsd ?? 0;
+                    totalNumTurns += commitResult.usage?.numTurns ?? 0;
+                    totalDurationApiMs += commitResult.usage?.durationApiMs ?? 0;
+                    commitHash = parseCommitHash(commitResult.result) ?? state.commitHash;
+                    messages.push(`[merge] Agent completed commit/push/PR operations`);
+                    if (effectiveState.openPr) {
+                        const prResult = parsePrUrl(commitResult.result);
+                        if (prResult) {
+                            prUrl = prResult.url;
+                            prNumber = prResult.number;
+                            // Cross-validate agent-parsed PR URL against authoritative source.
+                            try {
+                                const prStatuses = await deps.gitPrService.listPrStatuses(cwd);
+                                const matchingPr = prStatuses.find((pr) => pr.headRefName === branch);
+                                if (matchingPr) {
+                                    prUrl = matchingPr.url;
+                                    prNumber = matchingPr.number;
+                                }
+                            }
+                            catch {
+                                // gh CLI unavailable or API failure — fall back to agent-parsed URL
+                            }
+                            messages.push(`[merge] PR created: ${prUrl}`);
+                        }
+                    }
+                    // --- CI watch/fix loop (when push or openPr is enabled and CI watch is not disabled) ---
+                    const ciWatchEnabled = getSettings().workflow?.ciWatchEnabled !== false;
+                    if (ciWatchEnabled && (effectiveState.push || effectiveState.openPr)) {
+                        const ciResult = await runCiWatchFixLoop({
+                            executor,
+                            gitPrService: deps.gitPrService,
+                            featureRepository: deps.featureRepository,
+                        }, {
+                            cwd,
+                            branch,
+                            options,
+                            feature,
+                            prUrl,
+                            prNumber,
+                            existingAttempts: ciFixAttempts,
+                            messages,
+                            log,
+                        });
+                        ciStatus = ciResult.ciStatus;
+                        ciFixAttempts = ciResult.ciFixAttempts;
+                        ciFixHistory = ciResult.ciFixHistory;
+                        ciFixStatus = ciResult.ciFixStatus;
+                    }
                 }
                 // --- Persist lifecycle + PR data before approval gate ---
                 // Setting lifecycle to Review serves as the resume detection marker

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/src/presentation/tui/prompts/prd-review-question.prompt.d.ts

@@ -8,7 +8,7 @@ import type { QuestionOption } from '../../../../packages/core/src/domain/genera
 /**
  * Creates the @inquirer/select configuration for a single open question.
  */
-export declare function createQuestionSelectConfig(question: string, options: QuestionOption[], currentAnswer?: string): {
+export declare function createQuestionSelectConfig(question: string, options: QuestionOption[], currentAnswer?: string, questionNumber?: number, totalQuestions?: number): {
     message: string;
     choices: {
         name: string;

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

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

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

@@ -8,14 +8,17 @@ import { shepTheme } from '../themes/shep.theme.js';
 /**
  * Creates the @inquirer/select configuration for a single open question.
  */
-export function createQuestionSelectConfig(question, options, currentAnswer) {
+export function createQuestionSelectConfig(question, options, currentAnswer, questionNumber, totalQuestions) {
     const choices = options.map((opt) => ({
         name: opt.option,
         value: opt.option,
         description: opt.description,
     }));
+    const prefix = questionNumber != null && totalQuestions != null
+        ? `[${questionNumber}/${totalQuestions}] `
+        : '';
     return {
-        message: question,
+        message: `${prefix}${question}`,
         choices,
         default: currentAnswer,
         theme: shepTheme,

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

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

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

@@ -20,8 +20,10 @@ import { shepTheme } from '../themes/shep.theme.js';
  */
 export async function prdReviewWizard(questions) {
     const changedSelections = [];
-    for (const q of questions) {
-        const selected = await select(createQuestionSelectConfig(q.question, q.options, q.selectedOption));
+    const total = questions.length;
+    for (let i = 0; i < total; i++) {
+        const q = questions[i];
+        const selected = await select(createQuestionSelectConfig(q.question, q.options, q.selectedOption, i + 1, total));
         if (selected !== q.selectedOption) {
             changedSelections.push({
                 questionId: q.question,

package/dist/src/presentation/web/components/common/prd-questionnaire/prd-questionnaire.js

@@ -27,7 +27,7 @@ export function PrdQuestionnaire({ data, selections, onSelect, onApprove, onReje
     }, [onSelect, isLastStep, selectSound]);
     if (total === 0)
         return null;
-    return (_jsxs("div", { className: "flex min-h-0 flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [showHeader ? (_jsxs("div", { className: "border-border flex items-start gap-3 border-b pb-3", children: [_jsx("div", { className: "mt-1.5 h-2 w-2 shrink-0 rounded-full bg-amber-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h3", { className: "text-foreground mb-1.5 text-sm font-bold", children: question }), _jsx("p", { className: "text-muted-foreground text-xs leading-relaxed", children: context })] })] })) : null, _jsxs("div", { className: "space-y-3", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsx("label", { className: "text-foreground min-w-0 flex-1 text-sm font-semibold", children: currentQuestion.question }), _jsx("div", { className: "mt-1.5 flex shrink-0 gap-1", children: questions.map((q, idx) => (_jsx("button", { type: "button", "aria-label": `Go to question ${idx + 1}`, className: cn('h-1.5 rounded-full transition-all duration-200', idx === currentStep ? 'bg-primary w-4' : 'w-1.5', idx !== currentStep && selections[q.id] ? 'bg-primary/50' : '', idx !== currentStep && !selections[q.id] ? 'bg-muted-foreground/25' : ''), onClick: () => {
+    return (_jsxs("div", { className: "flex min-h-0 flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [showHeader ? (_jsxs("div", { className: "border-border flex items-start gap-3 border-b pb-3", children: [_jsx("div", { className: "mt-1.5 h-2 w-2 shrink-0 rounded-full bg-amber-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h3", { className: "text-foreground mb-1.5 text-sm font-bold", children: question }), _jsx("p", { className: "text-muted-foreground text-xs leading-relaxed", children: context })] })] })) : null, _jsxs("div", { className: "space-y-3", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsxs("label", { className: "text-foreground min-w-0 flex-1 text-sm font-semibold", children: [currentStep + 1, ". ", currentQuestion.question] }), _jsx("div", { className: "mt-1.5 flex shrink-0 gap-1", children: questions.map((q, idx) => (_jsx("button", { type: "button", "aria-label": `Go to question ${idx + 1}`, className: cn('h-1.5 rounded-full transition-all duration-200', idx === currentStep ? 'bg-primary w-4' : 'w-1.5', idx !== currentStep && selections[q.id] ? 'bg-primary/50' : '', idx !== currentStep && !selections[q.id] ? 'bg-muted-foreground/25' : ''), onClick: () => {
                                                 navigateSound.play();
                                                 setCurrentStep(idx);
                                             } }, q.id))) })] }), _jsx("div", { className: "space-y-2", children: currentQuestion.options.map((opt, optIdx) => {