claude-issue-solver 1.6.4 → 1.6.6

package/dist/commands/clean.js

@@ -124,30 +124,57 @@ function getIssueWorktrees() {
     const projectName = (0, git_1.getProjectName)();
     const parentDir = path.dirname(projectRoot);
     const worktrees = [];
+    const foundPaths = new Set();
     // Get all worktrees from git
     const output = (0, git_1.exec)('git worktree list --porcelain', projectRoot);
-    if (!output)
-        return worktrees;
-    const lines = output.split('\n');
-    let currentPath = '';
-    let currentBranch = '';
-    for (const line of lines) {
-        if (line.startsWith('worktree ')) {
-            currentPath = line.replace('worktree ', '');
+    if (output) {
+        const lines = output.split('\n');
+        let currentPath = '';
+        let currentBranch = '';
+        for (const line of lines) {
+            if (line.startsWith('worktree ')) {
+                currentPath = line.replace('worktree ', '');
+            }
+            else if (line.startsWith('branch refs/heads/')) {
+                currentBranch = line.replace('branch refs/heads/', '');
+                // Check if this is an issue branch
+                const match = currentBranch.match(/^issue-(\d+)-/);
+                if (match && currentPath.includes(`${projectName}-issue-`)) {
+                    worktrees.push({
+                        path: currentPath,
+                        branch: currentBranch,
+                        issueNumber: match[1],
+                    });
+                    foundPaths.add(currentPath);
+                }
+            }
         }
-        else if (line.startsWith('branch refs/heads/')) {
-            currentBranch = line.replace('branch refs/heads/', '');
-            // Check if this is an issue branch
-            const match = currentBranch.match(/^issue-(\d+)-/);
-            if (match && currentPath.includes(`${projectName}-issue-`)) {
-                worktrees.push({
-                    path: currentPath,
-                    branch: currentBranch,
-                    issueNumber: match[1],
-                });
+    }
+    // Also check for orphaned folders (folders that exist but aren't in git worktree list)
+    // This can happen when git worktree remove fails but the folder remains
+    try {
+        const folderPattern = `${projectName}-issue-`;
+        const entries = fs.readdirSync(parentDir, { withFileTypes: true });
+        for (const entry of entries) {
+            if (entry.isDirectory() && entry.name.startsWith(folderPattern)) {
+                const folderPath = path.join(parentDir, entry.name);
+                if (!foundPaths.has(folderPath)) {
+                    // Extract issue number from folder name (e.g., "project-issue-38-slug")
+                    const match = entry.name.match(new RegExp(`${projectName}-issue-(\\d+)-`));
+                    if (match) {
+                        worktrees.push({
+                            path: folderPath,
+                            branch: '', // No branch known for orphaned folders
+                            issueNumber: match[1],
+                        });
+                    }
+                }
             }
         }
     }
+    catch {
+        // Ignore errors reading parent directory
+    }
     return worktrees;
 }
 async function cleanAllCommand() {
@@ -159,7 +186,7 @@ async function cleanAllCommand() {
     }
     console.log(chalk_1.default.bold('\n🧹 Found issue worktrees:\n'));
     for (const wt of worktrees) {
-        console.log(`  ${chalk_1.default.cyan(`#${wt.issueNumber}`)}\t${wt.branch}`);
+        console.log(`  ${chalk_1.default.cyan(`#${wt.issueNumber}`)}\t${wt.branch || chalk_1.default.yellow('(orphaned folder)')}`);
         console.log(chalk_1.default.dim(`  \t${wt.path}`));
         console.log();
     }
@@ -188,35 +215,54 @@ async function cleanAllCommand() {
             catch {
                 // Ignore errors closing windows
             }
-            // Remove worktree
+            // Remove worktree/folder
+            const isOrphaned = !wt.branch;
             if (fs.existsSync(wt.path)) {
-                try {
-                    (0, child_process_1.execSync)(`git worktree remove "${wt.path}" --force`, {
-                        cwd: projectRoot,
-                        stdio: 'pipe',
-                    });
+                // Try git worktree remove first (only if not orphaned)
+                if (!isOrphaned) {
+                    try {
+                        (0, child_process_1.execSync)(`git worktree remove "${wt.path}" --force`, {
+                            cwd: projectRoot,
+                            stdio: 'pipe',
+                        });
+                    }
+                    catch {
+                        // Ignore - we'll force delete below if needed
+                    }
                 }
-                catch {
-                    // If git worktree remove fails, try removing directory manually with rm -rf
-                    // This handles locked files better than fs.rmSync
+                // If folder still exists, force delete it
+                if (fs.existsSync(wt.path)) {
                     try {
-                        (0, child_process_1.execSync)(`rm -rf "${wt.path}"`, { stdio: 'pipe' });
+                        (0, child_process_1.execSync)(`/bin/rm -rf "${wt.path}"`, { stdio: 'pipe' });
                     }
                     catch {
-                        fs.rmSync(wt.path, { recursive: true, force: true });
+                        try {
+                            fs.rmSync(wt.path, { recursive: true, force: true, maxRetries: 3, retryDelay: 100 });
+                        }
+                        catch {
+                            // Ignore - will check at end
+                        }
                     }
+                }
+                // Prune git worktrees
+                try {
                     (0, child_process_1.execSync)('git worktree prune', { cwd: projectRoot, stdio: 'pipe' });
                 }
+                catch {
+                    // Ignore
+                }
             }
-            // Delete branch
-            try {
-                (0, child_process_1.execSync)(`git branch -D "${wt.branch}"`, {
-                    cwd: projectRoot,
-                    stdio: 'pipe',
-                });
-            }
-            catch {
-                // Branch may already be deleted
+            // Delete branch (if we have one)
+            if (wt.branch) {
+                try {
+                    (0, child_process_1.execSync)(`git branch -D "${wt.branch}"`, {
+                        cwd: projectRoot,
+                        stdio: 'pipe',
+                    });
+                }
+                catch {
+                    // Branch may already be deleted
+                }
             }
             spinner.succeed(`Cleaned issue #${wt.issueNumber}`);
         }
@@ -232,7 +278,9 @@ async function cleanAllCommand() {
 async function cleanCommand(issueNumber) {
     const projectRoot = (0, git_1.getProjectRoot)();
     const projectName = (0, git_1.getProjectName)();
+    const parentDir = path.dirname(projectRoot);
     // Find the worktree for this issue number (don't need to fetch from GitHub)
+    // This now also includes orphaned folders
     const worktrees = getIssueWorktrees();
     const worktree = worktrees.find((wt) => wt.issueNumber === String(issueNumber));
     if (!worktree) {
@@ -242,7 +290,7 @@ async function cleanCommand(issueNumber) {
         const branches = output.split('\n').map((b) => b.trim().replace('* ', ''));
         const matchingBranch = branches.find((b) => b.startsWith(branchPattern));
         if (!matchingBranch) {
-            console.log(chalk_1.default.red(`\n❌ No worktree or branch found for issue #${issueNumber}`));
+            console.log(chalk_1.default.red(`\n❌ No worktree, folder, or branch found for issue #${issueNumber}`));
             return;
         }
         // Found a branch but no worktree - just delete the branch
@@ -270,16 +318,22 @@ async function cleanCommand(issueNumber) {
     }
     const branchName = worktree.branch;
     const worktreePath = worktree.path;
+    const isOrphaned = !branchName;
     console.log();
     console.log(chalk_1.default.bold(`🧹 Cleaning up issue #${issueNumber}`));
-    console.log(chalk_1.default.dim(`   Branch: ${branchName}`));
-    console.log(chalk_1.default.dim(`   Worktree: ${worktreePath}`));
+    if (isOrphaned) {
+        console.log(chalk_1.default.yellow(`   (Orphaned folder - no git worktree reference)`));
+    }
+    else {
+        console.log(chalk_1.default.dim(`   Branch: ${branchName}`));
+    }
+    console.log(chalk_1.default.dim(`   Folder: ${worktreePath}`));
     console.log();
     const { confirm } = await inquirer_1.default.prompt([
         {
             type: 'confirm',
             name: 'confirm',
-            message: 'Remove worktree and delete branch?',
+            message: isOrphaned ? 'Remove orphaned folder?' : 'Remove worktree and delete branch?',
             default: false,
         },
     ]);
@@ -298,46 +352,66 @@ async function cleanCommand(issueNumber) {
     catch {
         windowSpinner.warn('Could not close some windows');
     }
-    // Remove worktree
+    // Remove worktree/folder
     if (fs.existsSync(worktreePath)) {
         const worktreeSpinner = (0, ora_1.default)('Removing worktree...').start();
-        try {
-            (0, child_process_1.execSync)(`git worktree remove "${worktreePath}" --force`, {
-                cwd: projectRoot,
-                stdio: 'pipe',
-            });
-            worktreeSpinner.succeed('Worktree removed');
+        // Try git worktree remove first (only if not orphaned)
+        if (!isOrphaned) {
+            try {
+                (0, child_process_1.execSync)(`git worktree remove "${worktreePath}" --force`, {
+                    cwd: projectRoot,
+                    stdio: 'pipe',
+                });
+            }
+            catch {
+                // Ignore - we'll force delete below if needed
+            }
         }
-        catch {
-            // If git worktree remove fails, try removing directory manually with rm -rf
+        // If folder still exists, force delete it
+        if (fs.existsSync(worktreePath)) {
             try {
-                (0, child_process_1.execSync)(`rm -rf "${worktreePath}"`, { stdio: 'pipe' });
-                (0, child_process_1.execSync)('git worktree prune', { cwd: projectRoot, stdio: 'pipe' });
-                worktreeSpinner.succeed('Worktree removed (manually)');
+                // Use rm -rf with full path
+                (0, child_process_1.execSync)(`/bin/rm -rf "${worktreePath}"`, { stdio: 'pipe' });
             }
             catch {
+                // Try Node's rmSync as fallback
                 try {
-                    fs.rmSync(worktreePath, { recursive: true, force: true });
-                    (0, child_process_1.execSync)('git worktree prune', { cwd: projectRoot, stdio: 'pipe' });
-                    worktreeSpinner.succeed('Worktree removed (manually)');
+                    fs.rmSync(worktreePath, { recursive: true, force: true, maxRetries: 3, retryDelay: 100 });
                 }
                 catch {
-                    worktreeSpinner.warn('Could not remove worktree directory');
+                    // Last resort - try with sudo hint
+                    worktreeSpinner.warn(`Could not remove directory. Try manually: rm -rf "${worktreePath}"`);
                 }
             }
         }
+        // Prune git worktrees
+        try {
+            (0, child_process_1.execSync)('git worktree prune', { cwd: projectRoot, stdio: 'pipe' });
+        }
+        catch {
+            // Ignore
+        }
+        // Check final result
+        if (fs.existsSync(worktreePath)) {
+            worktreeSpinner.warn(`Could not fully remove directory: ${worktreePath}`);
+        }
+        else {
+            worktreeSpinner.succeed(isOrphaned ? 'Folder removed' : 'Worktree removed');
+        }
     }
-    // Delete branch
-    const branchSpinner = (0, ora_1.default)('Deleting branch...').start();
-    try {
-        (0, child_process_1.execSync)(`git branch -D "${branchName}"`, {
-            cwd: projectRoot,
-            stdio: 'pipe',
-        });
-        branchSpinner.succeed('Branch deleted');
-    }
-    catch {
-        branchSpinner.warn('Could not delete branch (may already be deleted)');
+    // Delete branch (if we have one)
+    if (branchName) {
+        const branchSpinner = (0, ora_1.default)('Deleting branch...').start();
+        try {
+            (0, child_process_1.execSync)(`git branch -D "${branchName}"`, {
+                cwd: projectRoot,
+                stdio: 'pipe',
+            });
+            branchSpinner.succeed('Branch deleted');
+        }
+        catch {
+            branchSpinner.warn('Could not delete branch (may already be deleted)');
+        }
     }
     console.log();
     console.log(chalk_1.default.green('✅ Cleanup complete!'));

package/package.json

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