wiggum-cli 0.16.0 → 0.17.0

package/dist/tui/utils/pr-summary.d.ts

@@ -24,11 +24,12 @@ export interface IssueInfo {
 export declare function getPrForBranch(projectRoot: string, branchName: string): PrInfo | null;
 /**
  * Get linked issue for a branch by parsing the PR body for closing keywords
- * (Closes/Fixes/Resolves #N).
+ * (Closes/Fixes/Resolves #N), with fallbacks for spec file and feature name search.
  *
  * @param projectRoot - Root directory of the git repository
  * @param branchName - Branch name to look up
  * @param prInfo - Optional pre-fetched PR info to avoid redundant gh call
+ * @param featureName - Optional feature name for fallback issue detection
  * @returns Issue info object, or null if not found or gh not available
  */
-export declare function getLinkedIssue(projectRoot: string, branchName: string, prInfo?: PrInfo | null): IssueInfo | null;
+export declare function getLinkedIssue(projectRoot: string, branchName: string, prInfo?: PrInfo | null, featureName?: string): IssueInfo | null;

package/dist/tui/utils/pr-summary.js

@@ -2,6 +2,8 @@
  * PR and Issue utilities for enhanced run summary.
  */
 import { execFileSync } from 'node:child_process';
+import { existsSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
 import { logger } from '../../utils/logger.js';
 /**
  * Get PR information for a branch.
@@ -33,16 +35,51 @@ export function getPrForBranch(projectRoot, branchName) {
         title: pr.title,
     };
 }
+/**
+ * Extract issue number from PR body closing keywords or fallback strategies.
+ */
+function findIssueNumber(body, featureName, projectRoot) {
+    // Strategy 1: Closing keywords in PR body (Closes/Fixes/Resolves #N)
+    const issueMatch = body.match(/(?:closes|fixes|resolves)\s+#(\d+)/i);
+    if (issueMatch) {
+        return parseInt(issueMatch[1], 10);
+    }
+    // Strategy 2: Bare "#N" references in the body (e.g., "Closes #7" without keyword before it)
+    // Look for "# N" patterns that are likely issue references (not markdown headers)
+    const hashRefMatch = body.match(/(?:^|\s)#(\d+)\b/m);
+    if (hashRefMatch) {
+        return parseInt(hashRefMatch[1], 10);
+    }
+    // Strategy 3: Read spec file for issue reference
+    if (featureName && projectRoot) {
+        try {
+            const specPath = join(projectRoot, '.ralph', 'specs', `${featureName}.md`);
+            if (existsSync(specPath)) {
+                const specContent = readFileSync(specPath, 'utf-8');
+                // Look for "Issue: #N", "Source Issue: #N", "GitHub Issue: #N", or issue URLs
+                const specIssueMatch = specContent.match(/(?:issue|source|github)[:\s]*#(\d+)/i) || specContent.match(/github\.com\/[^/]+\/[^/]+\/issues\/(\d+)/i);
+                if (specIssueMatch) {
+                    return parseInt(specIssueMatch[1], 10);
+                }
+            }
+        }
+        catch {
+            // spec read failure is non-fatal
+        }
+    }
+    return null;
+}
 /**
  * Get linked issue for a branch by parsing the PR body for closing keywords
- * (Closes/Fixes/Resolves #N).
+ * (Closes/Fixes/Resolves #N), with fallbacks for spec file and feature name search.
  *
  * @param projectRoot - Root directory of the git repository
  * @param branchName - Branch name to look up
  * @param prInfo - Optional pre-fetched PR info to avoid redundant gh call
+ * @param featureName - Optional feature name for fallback issue detection
  * @returns Issue info object, or null if not found or gh not available
  */
-export function getLinkedIssue(projectRoot, branchName, prInfo) {
+export function getLinkedIssue(projectRoot, branchName, prInfo, featureName) {
     try {
         // Use provided PR info or fetch it
         const pr = prInfo !== undefined ? prInfo : getPrForBranch(projectRoot, branchName);
@@ -57,12 +94,10 @@ export function getLinkedIssue(projectRoot, branchName, prInfo) {
         }).trim();
         const prData = JSON.parse(prBody);
         const body = prData.body || '';
-        // Look for issue references in PR body (e.g., "Closes #123", "Fixes #456")
-        const issueMatch = body.match(/(?:closes|fixes|resolves)\s+#(\d+)/i);
-        if (!issueMatch) {
+        const issueNumber = findIssueNumber(body, featureName, projectRoot);
+        if (!issueNumber) {
             return null;
         }
-        const issueNumber = parseInt(issueMatch[1], 10);
         // Fetch the issue details
         const issueOutput = execFileSync('gh', ['issue', 'view', String(issueNumber), '--json', 'number,url,state,title'], {
             cwd: projectRoot,

package/dist/utils/config.d.ts

@@ -50,6 +50,13 @@ export interface LoopConfig {
     planningModel: string;
     reviewMode: 'manual' | 'auto' | 'merge';
 }
+/**
+ * Agent configuration in ralph.config.cjs
+ */
+export interface AgentSectionConfig {
+    defaultProvider: string;
+    defaultModel: string;
+}
 /**
  * Full ralph.config.cjs structure
  */
@@ -59,6 +66,7 @@ export interface RalphConfig {
     commands: CommandsConfig;
     paths: PathsConfig;
     loop: LoopConfig;
+    agent: AgentSectionConfig;
 }
 /**
  * Default configuration values

package/dist/utils/config.js

@@ -47,6 +47,10 @@ export const DEFAULT_CONFIG = {
         planningModel: 'opus',
         reviewMode: 'manual',
     },
+    agent: {
+        defaultProvider: '',
+        defaultModel: '',
+    },
 };
 /**
  * Load ralph.config.cjs from a project directory
@@ -104,6 +108,10 @@ export async function loadConfigWithDefaults(projectRoot) {
             ...DEFAULT_CONFIG.loop,
             ...config.loop,
         },
+        agent: {
+            ...DEFAULT_CONFIG.agent,
+            ...config?.agent,
+        },
     };
 }
 /**

package/dist/utils/github.d.ts

@@ -0,0 +1,32 @@
+export declare function isGhInstalled(): Promise<boolean>;
+export declare function _resetGhCache(): void;
+export interface GitHubIssueDetail {
+    title: string;
+    body: string;
+    labels: string[];
+}
+export declare function fetchGitHubIssue(owner: string, repo: string, number: number): Promise<GitHubIssueDetail | null>;
+export interface GitHubIssueListItem {
+    number: number;
+    title: string;
+    state: 'open' | 'closed';
+    labels: string[];
+    createdAt: string;
+}
+export interface ListIssuesResult {
+    issues: GitHubIssueListItem[];
+    error?: string;
+}
+export declare function listRepoIssues(owner: string, repo: string, search?: string, limit?: number): Promise<ListIssuesResult>;
+export declare function detectGitHubRemote(projectRoot: string): Promise<GitHubRepo | null>;
+export interface ParsedGitHubIssue {
+    owner: string;
+    repo: string;
+    number: number;
+}
+export declare function isGitHubIssueUrl(input: string): ParsedGitHubIssue | null;
+export interface GitHubRepo {
+    owner: string;
+    repo: string;
+}
+export declare function parseGitHubRemote(remoteUrl: string): GitHubRepo | null;

package/dist/utils/github.js

@@ -0,0 +1,106 @@
+import { execFile as execFileCb } from 'node:child_process';
+/**
+ * Safe command execution using execFile (no shell, array-based args).
+ */
+function safeExec(cmd, args, cwd) {
+    return new Promise((resolve, reject) => {
+        execFileCb(cmd, args, { cwd, timeout: 10000 }, (error, stdout) => {
+            if (error)
+                reject(error);
+            else
+                resolve(String(stdout));
+        });
+    });
+}
+let ghInstalledCache = null;
+export async function isGhInstalled() {
+    if (ghInstalledCache !== null)
+        return ghInstalledCache;
+    try {
+        await safeExec('gh', ['--version']);
+        ghInstalledCache = true;
+    }
+    catch {
+        ghInstalledCache = false;
+    }
+    return ghInstalledCache;
+}
+export function _resetGhCache() {
+    ghInstalledCache = null;
+}
+export async function fetchGitHubIssue(owner, repo, number) {
+    try {
+        const stdout = await safeExec('gh', [
+            'issue', 'view', String(number),
+            '--repo', `${owner}/${repo}`,
+            '--json', 'title,body,labels',
+        ]);
+        const data = JSON.parse(stdout);
+        return {
+            title: data.title ?? '',
+            body: data.body ?? '',
+            labels: (data.labels ?? []).map((l) => l.name),
+        };
+    }
+    catch {
+        return null;
+    }
+}
+export async function listRepoIssues(owner, repo, search, limit = 20) {
+    try {
+        const args = [
+            'issue', 'list',
+            '--repo', `${owner}/${repo}`,
+            '--limit', String(limit),
+            '--json', 'number,title,state,labels,createdAt',
+            '--state', 'open',
+        ];
+        if (search) {
+            args.push('--search', search);
+        }
+        const stdout = await safeExec('gh', args);
+        const data = JSON.parse(stdout);
+        const issues = data.map((item) => ({
+            number: item.number,
+            title: item.title,
+            state: item.state.toLowerCase(),
+            labels: (item.labels ?? []).map((l) => l.name),
+            createdAt: item.createdAt ?? '',
+        }));
+        return { issues };
+    }
+    catch (err) {
+        const msg = err instanceof Error ? err.message : String(err);
+        if (msg.includes('auth') || msg.includes('login') || msg.includes('not logged')) {
+            return { issues: [], error: 'Run "gh auth login" to enable issue browsing' };
+        }
+        return { issues: [] };
+    }
+}
+export async function detectGitHubRemote(projectRoot) {
+    try {
+        const stdout = await safeExec('git', ['remote', 'get-url', 'origin'], projectRoot);
+        return parseGitHubRemote(stdout.trim());
+    }
+    catch {
+        return null;
+    }
+}
+const GITHUB_ISSUE_RE = /^https?:\/\/github\.com\/([^/]+)\/([^/]+)\/(issues|pull)\/(\d+)\/?/;
+export function isGitHubIssueUrl(input) {
+    const match = input.match(GITHUB_ISSUE_RE);
+    if (!match)
+        return null;
+    return { owner: match[1], repo: match[2], number: parseInt(match[4], 10) };
+}
+const SSH_REMOTE_RE = /^git@github\.com:([^/]+)\/(.+?)(?:\.git)?$/;
+const HTTPS_REMOTE_RE = /^https?:\/\/github\.com\/([^/]+)\/([^/]+?)(?:\.git)?$/;
+export function parseGitHubRemote(remoteUrl) {
+    const ssh = remoteUrl.match(SSH_REMOTE_RE);
+    if (ssh)
+        return { owner: ssh[1], repo: ssh[2] };
+    const https = remoteUrl.match(HTTPS_REMOTE_RE);
+    if (https)
+        return { owner: https[1], repo: https[2] };
+    return null;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wiggum-cli",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "AI-powered feature development loop CLI",
   "type": "module",
   "main": "dist/index.js",
@@ -61,6 +61,9 @@
     "react": "^18.3.1",
     "zod": "^4.3.5"
   },
+  "overrides": {
+    "minimatch": ">=10.2.1"
+  },
   "devDependencies": {
     "@types/node": "^20.10.0",
     "@types/react": "^19.2.9",

package/src/templates/prompts/PROMPT.md.tmpl

@@ -1,5 +1,5 @@
 ## Context
-Study @.ralph/AGENTS.md for commands and patterns.
+If @.ralph/guides/AGENTS.md exists, study it for commands and patterns.
 Study @.ralph/specs/$FEATURE.md for feature specification.
 Study @.ralph/specs/$FEATURE-implementation-plan.md for current tasks.
 {{#if frameworkVariant}}For detailed architecture, see @{{appDir}}/.claude/CLAUDE.md{{/if}}
@@ -18,10 +18,10 @@ Key patterns: parallel fetches, direct imports, React.cache(), lazy loading.
 - Search codebase before assuming something doesn't exist
 
 ## Task
-Pick the next incomplete task from the implementation plan.
+Work through ALL incomplete tasks in the implementation plan in a single session.
 **Skip E2E tasks** (tasks starting with `E2E:`) - those are handled in a separate phase.
-Implement it following the patterns in AGENTS.md.
-Write tests for the implementation.
+For each task: implement it, write tests, validate, commit, then move to the next task.
+Do not stop after one task — keep going until all non-E2E tasks are complete.
 
 ## Validation
 After changes, ALL must pass:
@@ -36,7 +36,8 @@ If any validation fails, fix the issue before proceeding.
 Before committing, review your changes against @.ralph/guides/SECURITY.md:
 1. **Quick scan**: Input validation, injection prevention, auth checks, data exposure
 2. **Run**: `cd {{appDir}} && {{packageManager}} audit` (check for vulnerable dependencies)
-3. **Check**: `mcp__supabase__get_advisors` with type "security" (RLS policies)
+{{#if hasSupabase}}3. **Check**: `mcp__supabase__get_advisors` with type "security" (RLS policies)
+{{/if}}
 4. **Red team**: Can auth be bypassed? Can other users' data be accessed?
 
 Flag any security issues in the implementation plan and fix before committing.
@@ -54,7 +55,7 @@ If any check fails, fix before committing.
 
 ## Completion
 When ALL validations pass:
-1. Update @.ralph/specs/$FEATURE-implementation-plan.md - mark task done with commit hash
+1. Update @.ralph/specs/$FEATURE-implementation-plan.md — change the task's `- [ ]` to `- [x]` and append the commit hash (e.g., `- [x] Task description - abc1234`). The harness tracks progress by counting checkboxes, so this step is mandatory.
 2. `git -C {{appDir}} add -A`
 3. `git -C {{appDir}} commit -m "type(scope): description"`
 4. `git -C {{appDir}} push origin feat/$FEATURE`
@@ -69,9 +70,11 @@ If this iteration revealed something useful, append to @.ralph/LEARNINGS.md:
 Format: `- [YYYY-MM-DD] [$FEATURE] Brief description`
 
 ## Rules
-- One task per iteration
+- Complete ALL remaining non-E2E tasks before ending the session
+- Commit after each task so progress is preserved if the session is interrupted
 - Tests are mandatory - no task is complete without tests
 - Search codebase before assuming something doesn't exist
-- If blocked, document in implementation plan and move to next task
-- Use Supabase MCP for database operations
-- Use PostHog MCP for analytics queries
+- If blocked on a task, document in implementation plan and move to the next task
+{{#if hasSupabase}}- Use Supabase MCP for database operations
+{{/if}}{{#if hasPosthog}}- Use PostHog MCP for analytics queries
+{{/if}}

package/src/templates/prompts/PROMPT_e2e.md.tmpl

@@ -1,5 +1,5 @@
 ## Context
-Study @.ralph/AGENTS.md for commands and patterns.
+If @.ralph/guides/AGENTS.md exists, study it for commands and patterns.
 Study @.ralph/specs/$FEATURE.md for feature specification.
 Study @.ralph/specs/$FEATURE-implementation-plan.md for E2E test scenarios.
 {{#if frameworkVariant}}For detailed architecture, see @{{appDir}}/.claude/CLAUDE.md{{/if}}
@@ -21,6 +21,7 @@ If either fails, fix issues before proceeding with E2E tests.
 {{#if isTui}}
 ## Task
 Execute automated E2E tests for the completed TUI feature using the xterm.js bridge and agent-browser.
+Run ALL scenarios in a single session — do not end between scenarios.
 
 ### Step 1: Start Bridge
 Check the bridge is running:
@@ -149,11 +150,11 @@ When all scenarios are executed:
 2. Update the Implementation Summary status to `[PASSED]` if all passed
 3. **Commit the updated implementation plan:**
    ```bash
-   git add -A && git commit -m "test($FEATURE): E2E tests passed via agent-browser"
+   git -C {{appDir}} add -A && git -C {{appDir}} commit -m "test($FEATURE): E2E tests passed via agent-browser"
    ```
 4. **Push to remote:**
    ```bash
-   git push origin feat/$FEATURE
+   git -C {{appDir}} push origin feat/$FEATURE
    ```
 5. If all passed: signal ready for PR phase
 6. If any failed: failures documented, loop will retry after fix iteration
@@ -168,6 +169,7 @@ Format: `- [YYYY-MM-DD] [$FEATURE] Brief description`
 {{else}}
 ## Task
 Execute automated E2E tests for the completed feature using Playwright MCP tools.
+Run ALL scenarios in a single session — do not end between scenarios.
 
 ### Step 1: Check Dev Server
 Verify dev server is running at http://localhost:3000. If not accessible, start it:
@@ -176,7 +178,7 @@ cd {{appDir}} && {{devCommand}} &
 ```
 Wait ~10 seconds for server startup, then verify with a simple browser_navigate.
 
-### Step 1.5: Seed Test Data (if needed)
+{{#if hasSupabase}}### Step 1.5: Seed Test Data (if needed)
 
 Check if test scenarios require specific data volumes (e.g., pagination needs >10 rows).
 
@@ -199,6 +201,7 @@ query: "DELETE FROM table_name WHERE data->>'_test' = 'true';"
 ```
 
 **If seeding is impractical:** Document in implementation plan that E2E was skipped but unit tests provide coverage.
+{{/if}}
 
 ### Step 2: Parse E2E Test Scenarios
 Read E2E test scenarios from @.ralph/specs/$FEATURE-implementation-plan.md.
@@ -246,7 +249,7 @@ For each E2E test scenario:
    - Check console: `browser_console_messages` for JS errors
    - Document failure details
 
-### Step 4: Database Verification
+{{#if hasSupabase}}### Step 4: Database Verification
 For scenarios with database checks, use Supabase MCP:
 ```
 mcp__plugin_supabase_supabase__execute_sql
@@ -255,6 +258,7 @@ query: "SELECT * FROM survey_responses WHERE ..."
 ```
 
 Verify returned data matches expected state.
+{{/if}}
 
 ### Unique Test Data (for Parallel Execution)
 When creating test data, use unique identifiers to avoid conflicts with other loops:
@@ -315,11 +319,12 @@ Update @.ralph/specs/$FEATURE-implementation-plan.md for each scenario:
 2. Verify URL contains expected path/params
 ```
 
-### Database State
+{{#if hasSupabase}}### Database State
 ```
 1. mcp__plugin_supabase_supabase__execute_sql with SELECT query
 2. Verify row count, column values match expectations
 ```
+{{/if}}
 
 ## Browser State Management
 
@@ -364,8 +369,9 @@ If code changes don't appear in the browser:
 
 ### Stale Data
 - Clear browser storage: Use `browser_close` between scenarios
-- Check Supabase for stale test data from previous runs
+{{#if hasSupabase}}- Check Supabase for stale test data from previous runs
 - Delete test data: `DELETE FROM table WHERE data->>'_test' = 'true'`
+{{/if}}
 
 ## Rules
 - Always get a fresh `browser_snapshot` after actions before making assertions

package/src/templates/prompts/PROMPT_feature.md.tmpl

@@ -1,5 +1,5 @@
 ## Context
-Study @.ralph/AGENTS.md for commands and patterns.
+If @.ralph/guides/AGENTS.md exists, study it for commands and patterns.
 Study @.ralph/specs/README.md for spec structure.
 Study @.ralph/specs/$FEATURE.md for feature specification.
 {{#if frameworkVariant}}For detailed architecture, see @{{appDir}}/.claude/CLAUDE.md{{/if}}
@@ -96,9 +96,22 @@ Example E2E scenario:
 - [x] E2E: Scenario name - PASSED
 ```
 
+## CRITICAL CONSTRAINT — PLANNING ONLY
+**You are in the PLANNING phase. Your ONLY job is to produce an implementation plan.**
+- Do NOT write any source code, test code, or configuration files
+- Do NOT create, modify, or touch any files outside `.ralph/specs/`
+- Do NOT run build, test, or lint commands
+- Do NOT make git commits
+- If you feel the urge to "just implement a small piece", STOP — that is a phase violation
+- The implementation phase runs AFTER this session ends, in a separate session
+- Violation of this constraint wastes tokens and breaks the harness automation
+
 ## Rules
-- Plan only in this phase, do NOT implement
-- One task = one commit-sized unit of work
+- You MUST use `- [ ]` checkbox syntax for every task in the plan
+- Do NOT use heading-based task formats (e.g., `#### Task 1:`) for individual tasks
+- The harness parses `- [ ]` lines to track progress — other formats will break automation
+- Use `### Phase N:` headings only for phase grouping, not for individual tasks
+- One task = one commit-sized unit of work (but tasks can be grouped into phases for batch implementation)
 - Every implementation task needs a corresponding test task
 - Use Supabase MCP to check existing schema
 - Use PostHog MCP to check existing analytics setup

package/src/templates/prompts/PROMPT_review_auto.md.tmpl

@@ -1,5 +1,5 @@
 ## Context
-Study @.ralph/AGENTS.md for commands and patterns.
+If @.ralph/guides/AGENTS.md exists, study it for commands and patterns.
 Study @.ralph/specs/$FEATURE.md for feature specification.
 Study @.ralph/specs/$FEATURE-implementation-plan.md for completed tasks.
 
@@ -9,6 +9,7 @@ Capture any review feedback patterns for future iterations.
 
 ## Task
 All implementation and E2E tasks are complete. Create PR and request review.
+Complete ALL steps in a single pass — do not end the session between steps.
 
 ### Step 1: Verify Ready State
 1. Check all tasks are complete in implementation plan (no `- [ ]` items)
@@ -52,12 +53,14 @@ cd {{appDir}} && gh pr create --base main --head feat/$FEATURE \
 [Read from implementation plan - list completed phases]
 
 ## Testing
-- [x] Unit/integration tests: 97 passing
-- [x] E2E tests: All scenarios passed via Playwright MCP
+- [x] Unit/integration tests: passing
+- [x] E2E tests: All scenarios passed
 - [x] Build succeeds
 
 ## E2E Test Results
-[Copy from implementation plan Phase 9]
+[Copy from implementation plan E2E section]
+
+Closes #[Read the source issue number from the spec file metadata or context section]
 
 Generated with Claude Code
 EOF
@@ -88,17 +91,27 @@ Review the git diff against main and check:
 
 Run: git diff main
 
-Respond with:
-- APPROVED if everything looks good
-- Or list specific issues with file:line references that need to be fixed
-
-IMPORTANT: After posting any PR review comment, you MUST print your final verdict as the LAST line of your output. Print exactly one of:
-  VERDICT: APPROVED
-  VERDICT: NOT APPROVED
-This line is parsed by the automation — do not omit it."
+Then:
+1. Post your complete review as a comment on the PR using:
+   gh pr comment --body '<your review in markdown>'
+   Format the comment with: a summary, specific issues with file:line refs (if any), and the verdict.
+2. Print your final verdict as the LAST line of stdout. Print exactly one of:
+   VERDICT: APPROVED
+   VERDICT: NOT APPROVED
+   This line is parsed by the automation — do not omit it."
 fi
 ```
 
+After the review completes, check its output:
+- If it contains "VERDICT: APPROVED", echo that line so the automation detects it:
+  ```bash
+  echo "VERDICT: APPROVED"
+  ```
+- If issues were found, echo:
+  ```bash
+  echo "VERDICT: NOT APPROVED"
+  ```
+
 **Handle review feedback:**
 - If Claude outputs "VERDICT: APPROVED" -> Done. The PR is ready for manual merge by the user.
 - If Claude lists issues:
@@ -113,8 +126,15 @@ After review is complete (approved or max iterations reached):
 1. Post a summary comment on the PR with the review outcome
 2. Do NOT merge — the user will review and merge manually
 
+## IMPORTANT: Review scope
+If you discover that no implementation code exists (empty diff, no source files changed),
+do NOT implement the feature yourself. Instead, report "VERDICT: NOT APPROVED —
+no implementation found" so the harness can trigger a new implementation iteration.
+
 ## Rules
+- **NEVER approve if any tests are failing.** Output "VERDICT: NOT APPROVED — test failures" if any tests fail.
 - Do NOT merge the PR — auto mode only reviews, the user merges
+- Do NOT implement missing features — only review and fix minor issues
 - Address ALL review comments before marking as approved
 - If gh CLI fails, check authentication: `gh auth status`
 - Keep review conversation focused and professional

package/src/templates/prompts/PROMPT_review_manual.md.tmpl

@@ -1,5 +1,5 @@
 ## Context
-Study @.ralph/AGENTS.md for commands and patterns.
+If @.ralph/guides/AGENTS.md exists, study it for commands and patterns.
 Study @.ralph/specs/$FEATURE.md for feature specification.
 Study @.ralph/specs/$FEATURE-implementation-plan.md for completed tasks.
 
@@ -9,6 +9,7 @@ Capture any review feedback patterns for future iterations.
 
 ## Task
 All implementation and E2E tasks are complete. Create PR for manual review.
+Complete ALL steps in a single pass — do not end the session between steps.
 
 ### Step 1: Verify Ready State
 1. Check all tasks are complete in implementation plan (no `- [ ]` items)
@@ -59,6 +60,8 @@ cd {{appDir}} && gh pr create --base main --head feat/$FEATURE \
 ## E2E Test Results
 [Copy from implementation plan if E2E phase exists]
 
+Closes #[Read the source issue number from the spec file metadata or context section]
+
 Generated with Claude Code
 EOF
 )"