claude-issue-solver 1.21.0 → 1.22.0

package/README.md

@@ -59,6 +59,7 @@ Starting 2 issue(s)...
 - ✨ **Create and solve** - Create new issues and start solving them immediately
 - 🌿 **Worktree isolation** - Each issue gets its own worktree, work on multiple issues in parallel
 - 🤖 **Real-time PR creation** - Automatically creates/updates PR as Claude commits changes
+- 🔍 **AI code review** - Review PRs with Claude, posts suggestions you can commit directly on GitHub
 - 🧹 **Smart cleanup** - Auto-clean merged PRs, close VS Code/terminal windows on macOS
 - 📁 **Monorepo support** - Recursively copies all `.env*` files, symlinks `node_modules`
 - 💻 **Cross-platform terminals** - iTerm2, Terminal.app (macOS), gnome-terminal, xterm, konsole (Linux)
@@ -127,6 +128,9 @@ claude-issue new "Fix crash" -l bug -l priority
 # Create PR for a solved issue (if you skipped it earlier)
 claude-issue pr 42
 
+# Review a PR with AI (posts suggestions you can commit on GitHub)
+claude-issue review 42
+
 # Clean up worktree and branch
 claude-issue clean 42        # Clean specific issue
 claude-issue clean           # Interactive selection
@@ -151,6 +155,7 @@ claude-issue --help
 | `claude-issue list` | `ls` | List open issues |
 | `claude-issue show <number>` | - | Show full issue details |
 | `claude-issue pr <number>` | - | Create PR for solved issue |
+| `claude-issue review <number>` | - | Review PR with AI suggestions |
 | `claude-issue clean [number]` | `rm` | Remove worktree and branch |
 | `claude-issue go [number]` | - | Navigate to worktree |
 | `claude-issue init` | - | Setup wizard for requirements |

package/dist/commands/review.d.ts

@@ -0,0 +1 @@
+export declare function reviewCommand(issueNumber: number): Promise<void>;

package/dist/commands/review.js

@@ -0,0 +1,264 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.reviewCommand = reviewCommand;
+const chalk_1 = __importDefault(require("chalk"));
+const ora_1 = __importDefault(require("ora"));
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const child_process_1 = require("child_process");
+const github_1 = require("../utils/github");
+const git_1 = require("../utils/git");
+const helpers_1 = require("../utils/helpers");
+async function reviewCommand(issueNumber) {
+    const spinner = (0, ora_1.default)(`Fetching issue #${issueNumber}...`).start();
+    const issue = (0, github_1.getIssue)(issueNumber);
+    if (!issue) {
+        spinner.fail(`Could not find issue #${issueNumber}`);
+        process.exit(1);
+    }
+    spinner.succeed(`Found issue #${issueNumber}`);
+    const projectRoot = (0, git_1.getProjectRoot)();
+    const projectName = (0, git_1.getProjectName)();
+    const baseBranch = (0, git_1.getDefaultBranch)();
+    const branchSlug = (0, helpers_1.slugify)(issue.title);
+    const branchName = `issue-${issueNumber}-${branchSlug}`;
+    const worktreePath = path.join(path.dirname(projectRoot), `${projectName}-${branchName}`);
+    // Check if there's a PR for this issue
+    const prCheckSpinner = (0, ora_1.default)('Checking for PR...').start();
+    let prNumber = null;
+    try {
+        const prOutput = (0, child_process_1.execSync)(`gh pr list --head "${branchName}" --json number --jq '.[0].number'`, {
+            cwd: projectRoot,
+            encoding: 'utf-8',
+            stdio: ['pipe', 'pipe', 'pipe'],
+        }).trim();
+        if (prOutput) {
+            prNumber = prOutput;
+            prCheckSpinner.succeed(`Found PR #${prNumber}`);
+        }
+        else {
+            prCheckSpinner.fail('No PR found for this issue');
+            console.log(chalk_1.default.yellow('\nA PR must exist before you can review it.'));
+            console.log(chalk_1.default.dim(`First solve the issue: claude-issue ${issueNumber}`));
+            process.exit(1);
+        }
+    }
+    catch {
+        prCheckSpinner.fail('Could not check for PR');
+        process.exit(1);
+    }
+    console.log();
+    console.log(chalk_1.default.bold(`📌 Reviewing: ${issue.title}`));
+    console.log(chalk_1.default.dim(`🔗 PR: https://github.com/${getRepoName(projectRoot)}/pull/${prNumber}`));
+    console.log();
+    // Fetch latest
+    const fetchSpinner = (0, ora_1.default)(`Fetching latest changes...`).start();
+    try {
+        (0, child_process_1.execSync)(`git fetch origin ${branchName} --quiet`, { cwd: projectRoot, stdio: 'pipe' });
+        fetchSpinner.succeed('Fetched latest changes');
+    }
+    catch {
+        fetchSpinner.warn('Could not fetch branch');
+    }
+    // Check if worktree already exists
+    if (fs.existsSync(worktreePath)) {
+        console.log(chalk_1.default.yellow(`\n🌿 Using existing worktree at: ${worktreePath}`));
+        // Pull latest changes
+        try {
+            (0, child_process_1.execSync)('git pull --quiet', { cwd: worktreePath, stdio: 'pipe' });
+        }
+        catch {
+            // Ignore pull errors
+        }
+    }
+    else {
+        const worktreeSpinner = (0, ora_1.default)(`Creating worktree for review...`).start();
+        try {
+            if ((0, git_1.branchExists)(branchName)) {
+                (0, child_process_1.execSync)(`git worktree add "${worktreePath}" "${branchName}"`, {
+                    cwd: projectRoot,
+                    stdio: 'pipe',
+                });
+            }
+            else {
+                // Branch should exist if PR exists, but handle edge case
+                (0, child_process_1.execSync)(`git worktree add "${worktreePath}" "origin/${branchName}"`, {
+                    cwd: projectRoot,
+                    stdio: 'pipe',
+                });
+            }
+            worktreeSpinner.succeed(`Created worktree at: ${worktreePath}`);
+        }
+        catch (error) {
+            worktreeSpinner.fail('Failed to create worktree');
+            console.error(error);
+            process.exit(1);
+        }
+        // Copy env files and symlink node_modules
+        const setupSpinner = (0, ora_1.default)('Setting up worktree...').start();
+        (0, helpers_1.copyEnvFiles)(projectRoot, worktreePath);
+        (0, helpers_1.symlinkNodeModules)(projectRoot, worktreePath);
+        setupSpinner.succeed('Worktree setup complete');
+    }
+    // Get the diff for context
+    let diffContent = '';
+    try {
+        diffContent = (0, child_process_1.execSync)(`gh pr diff ${prNumber}`, {
+            cwd: projectRoot,
+            encoding: 'utf-8',
+            stdio: ['pipe', 'pipe', 'pipe'],
+            maxBuffer: 10 * 1024 * 1024, // 10MB buffer for large diffs
+        });
+    }
+    catch {
+        console.log(chalk_1.default.yellow('Could not fetch PR diff, Claude will review the files directly.'));
+    }
+    // Build the review prompt
+    const prompt = `You are reviewing PR #${prNumber} for issue #${issueNumber}: ${issue.title}
+
+## Issue Description
+${issue.body}
+
+## Your Task
+Review the code changes in this PR. Look for:
+1. Bugs and logic errors
+2. Security vulnerabilities
+3. Missing error handling
+4. Code quality issues
+5. Missing tests
+6. Performance problems
+
+## How to Leave Feedback
+Use the gh CLI to post review comments with suggestions. For each issue you find:
+
+\`\`\`bash
+gh pr review ${prNumber} --comment --body "**File: path/to/file.ts**
+
+Description of the issue...
+
+\\\`\\\`\\\`suggestion
+// Your suggested fix here
+\\\`\\\`\\\`
+"
+\`\`\`
+
+The \`suggestion\` code block will create a "Commit suggestion" button on GitHub.
+
+For a final review summary, use:
+\`\`\`bash
+gh pr review ${prNumber} --comment --body "## Review Summary
+
+- Issue 1: ...
+- Issue 2: ...
+
+Overall: [APPROVE/REQUEST_CHANGES/COMMENT]"
+\`\`\`
+
+Or to approve/request changes formally:
+\`\`\`bash
+gh pr review ${prNumber} --approve --body "LGTM! Code looks good."
+gh pr review ${prNumber} --request-changes --body "Please address the issues above."
+\`\`\`
+
+## PR Diff
+${diffContent ? `\n\`\`\`diff\n${diffContent.slice(0, 50000)}\n\`\`\`\n` : 'Run `gh pr diff ' + prNumber + '` to see the changes.'}
+
+Start by examining the diff and the changed files, then provide your review.`;
+    // Write prompt to a file
+    const promptFile = path.join(worktreePath, '.claude-review-prompt.txt');
+    fs.writeFileSync(promptFile, prompt);
+    // Create runner script for review
+    const runnerScript = path.join(worktreePath, '.claude-review-runner.sh');
+    const runnerContent = `#!/bin/bash
+cd "${worktreePath}"
+
+# Set terminal title
+echo -ne "\\033]0;Review PR #${prNumber}: ${issue.title.replace(/"/g, '\\"').slice(0, 50)}\\007"
+
+echo "🔍 Claude Code Review - PR #${prNumber}"
+echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+echo ""
+echo "Issue #${issueNumber}: ${issue.title.replace(/"/g, '\\"')}"
+echo ""
+echo "Claude will review the PR and post suggestions."
+echo "You can commit suggestions directly on GitHub."
+echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+echo ""
+
+# Run Claude interactively
+claude --dangerously-skip-permissions "$(cat '${promptFile}')"
+
+# Clean up prompt file
+rm -f '${promptFile}'
+
+echo ""
+echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+echo "Review session ended."
+echo ""
+echo "View PR: gh pr view ${prNumber} --web"
+echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+echo ""
+
+# Keep terminal open
+exec bash
+`;
+    fs.writeFileSync(runnerScript, runnerContent, { mode: 0o755 });
+    console.log();
+    console.log(chalk_1.default.cyan('🔍 Opening new terminal for code review...'));
+    console.log();
+    (0, helpers_1.openInNewTerminal)(`'${runnerScript}'`);
+    console.log(chalk_1.default.green(`✅ Review session started for PR #${prNumber}`));
+    console.log(chalk_1.default.dim(`   Claude is reviewing in a new terminal window.`));
+    console.log();
+    console.log(chalk_1.default.dim(`   View PR: gh pr view ${prNumber} --web`));
+    console.log(chalk_1.default.dim(`   To clean up later: claude-issue clean ${issueNumber}`));
+}
+function getRepoName(projectRoot) {
+    try {
+        const output = (0, child_process_1.execSync)('gh repo view --json nameWithOwner --jq .nameWithOwner', {
+            cwd: projectRoot,
+            encoding: 'utf-8',
+            stdio: ['pipe', 'pipe', 'pipe'],
+        });
+        return output.trim();
+    }
+    catch {
+        return '';
+    }
+}

package/dist/index.js

@@ -17,6 +17,7 @@ const go_1 = require("./commands/go");
 const new_1 = require("./commands/new");
 const init_1 = require("./commands/init");
 const show_1 = require("./commands/show");
+const review_1 = require("./commands/review");
 // eslint-disable-next-line @typescript-eslint/no-var-requires
 const packageJson = require('../package.json');
 const program = new commander_1.Command();
@@ -150,4 +151,16 @@ program
     .action(async () => {
     await (0, init_1.initCommand)();
 });
+// Review command - AI code review for PRs
+program
+    .command('review <issue>')
+    .description('Review a PR with Claude and post suggestions')
+    .action(async (issue) => {
+    const issueNumber = parseInt(issue, 10);
+    if (isNaN(issueNumber)) {
+        console.log(chalk_1.default.red(`❌ Invalid issue number: ${issue}`));
+        process.exit(1);
+    }
+    await (0, review_1.reviewCommand)(issueNumber);
+});
 program.parse();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-issue-solver",
-  "version": "1.21.0",
+  "version": "1.22.0",
   "description": "Automatically solve GitHub issues using Claude Code",
   "main": "dist/index.js",
   "bin": {