claude-issue-solver 1.43.6 → 1.44.0

package/dist/utils/helpers.d.ts

@@ -3,11 +3,6 @@ export declare function checkRequirements(): {
     ok: boolean;
     missing: string[];
 };
-/**
- * Generate AppleScript for opening a script in iTerm2.
- * Creates a new window and runs the command directly to avoid session restoration issues.
- */
-export declare function generateITermOpenScript(script: string): string;
 /**
  * Generate AppleScript for opening a script in Terminal.app.
  * Changes to the script's directory first so the session path matches the worktree.

package/dist/utils/helpers.js

@@ -35,15 +35,14 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.slugify = slugify;
 exports.checkRequirements = checkRequirements;
-exports.generateITermOpenScript = generateITermOpenScript;
 exports.generateTerminalOpenScript = generateTerminalOpenScript;
 exports.openInNewTerminal = openInNewTerminal;
 exports.copyEnvFiles = copyEnvFiles;
 exports.symlinkNodeModules = symlinkNodeModules;
 const child_process_1 = require("child_process");
-const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const os = __importStar(require("os"));
+const fs = __importStar(require("fs"));
 function slugify(text) {
     // Remove common prefixes in brackets like [FAQ], [Bug], etc.
     const withoutBrackets = text.replace(/^\[.*?\]\s*/, '');
@@ -73,26 +72,6 @@ function checkRequirements() {
     }
     return { ok: missing.length === 0, missing };
 }
-/**
- * Generate AppleScript for opening a script in iTerm2.
- * Creates a new window and runs the command directly to avoid session restoration issues.
- */
-function generateITermOpenScript(script) {
-    const escapedScript = script.replace(/"/g, '\\"').replace(/'/g, '');
-    // Extract directory from script path to set as working directory
-    const scriptDir = path.dirname(script.replace(/'/g, ''));
-    const escapedDir = scriptDir.replace(/"/g, '\\"');
-    // cd to the script's directory first, so session path matches worktree
-    const bashCommand = `cd "${escapedDir}" && /bin/bash "${escapedScript}"`;
-    const escapedBashCommand = bashCommand.replace(/\\/g, '\\\\').replace(/"/g, '\\"');
-    return `
-      tell application "iTerm"
-        activate
-        -- Create a new window with a command to avoid session restoration issues
-        create window with default profile command "/bin/bash -c '${bashCommand.replace(/'/g, "'\\''")}'"
-      end tell
-    `;
-}
 /**
  * Generate AppleScript for opening a script in Terminal.app.
  * Changes to the script's directory first so the session path matches the worktree.
@@ -115,27 +94,16 @@ function generateTerminalOpenScript(script) {
 function openInNewTerminal(script) {
     const platform = os.platform();
     if (platform === 'darwin') {
-        // macOS - try iTerm2 first, then Terminal
-        const iTermScript = generateITermOpenScript(script);
+        // macOS - use Terminal.app (always available, no session restoration issues)
         const terminalScript = generateTerminalOpenScript(script);
-        try {
-            // Check if iTerm is installed
-            if (fs.existsSync('/Applications/iTerm.app')) {
-                (0, child_process_1.execSync)(`osascript -e '${iTermScript}'`, { stdio: 'pipe' });
-            }
-            else {
-                (0, child_process_1.execSync)(`osascript -e '${terminalScript}'`, { stdio: 'pipe' });
-            }
-        }
-        catch {
-            // Fallback to Terminal
-            try {
-                (0, child_process_1.execSync)(`osascript -e '${terminalScript}'`, { stdio: 'pipe' });
-            }
-            catch {
-                console.log('Could not open new terminal. Run manually:');
-                console.log(script);
-            }
+        // Use spawnSync with stdin to avoid shell escaping issues
+        const result = (0, child_process_1.spawnSync)('osascript', [], {
+            input: terminalScript,
+            stdio: ['pipe', 'pipe', 'pipe'],
+        });
+        if (result.status !== 0) {
+            console.log('Could not open new terminal. Run manually:');
+            console.log(script);
         }
     }
     else if (platform === 'linux') {

package/dist/utils/helpers.test.js

@@ -22,35 +22,6 @@ const helpers_1 = require("./helpers");
             (0, vitest_1.expect)((0, helpers_1.slugify)(longTitle).length).toBeLessThanOrEqual(30);
         });
     });
-    (0, vitest_1.describe)('generateITermOpenScript', () => {
-        (0, vitest_1.it)('should generate valid AppleScript for iTerm', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/path/to/script.sh');
-            (0, vitest_1.expect)(script).toContain('tell application "iTerm"');
-            (0, vitest_1.expect)(script).toContain('create window with default profile command');
-        });
-        (0, vitest_1.it)('should use create window with command to avoid session restoration', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/path/to/script.sh');
-            // Should use command parameter to run directly without session restoration
-            (0, vitest_1.expect)(script).toContain('create window with default profile command');
-            (0, vitest_1.expect)(script).toContain('/bin/bash -c');
-        });
-        (0, vitest_1.it)('should use /bin/bash to run the script', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/path/to/script.sh');
-            (0, vitest_1.expect)(script).toContain('/bin/bash');
-            (0, vitest_1.expect)(script).toContain('/path/to/script.sh');
-        });
-        (0, vitest_1.it)('should handle double quotes in script path without breaking AppleScript', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/path/to/"quoted"/script.sh');
-            // Should still contain the path and be valid AppleScript structure
-            (0, vitest_1.expect)(script).toContain('tell application "iTerm"');
-            (0, vitest_1.expect)(script).toContain('/bin/bash');
-            (0, vitest_1.expect)(script).toContain('quoted');
-        });
-        (0, vitest_1.it)('should handle paths with spaces', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/path/with spaces/script.sh');
-            (0, vitest_1.expect)(script).toContain('/path/with spaces/script.sh');
-        });
-    });
     (0, vitest_1.describe)('generateTerminalOpenScript', () => {
         (0, vitest_1.it)('should generate valid AppleScript for Terminal.app', () => {
             const script = (0, helpers_1.generateTerminalOpenScript)('/path/to/script.sh');
@@ -79,12 +50,6 @@ const helpers_1 = require("./helpers");
         });
     });
     (0, vitest_1.describe)('oh-my-zsh bypass behavior', () => {
-        (0, vitest_1.it)('iTerm script should run command directly to avoid session restoration', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/test/script.sh');
-            // Should use create window with command to avoid session restoration issues
-            (0, vitest_1.expect)(script).toContain('create window with default profile command');
-            (0, vitest_1.expect)(script).toContain('/bin/bash');
-        });
         (0, vitest_1.it)('Terminal script should combine dismiss and command in single do script', () => {
             const script = (0, helpers_1.generateTerminalOpenScript)('/test/script.sh');
             // Should have "n; /bin/bash..." in a single do script call
@@ -92,12 +57,6 @@ const helpers_1 = require("./helpers");
         });
     });
     (0, vitest_1.describe)('working directory for terminal closing', () => {
-        (0, vitest_1.it)('iTerm script should cd to script directory before running', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/path/to/worktree/.claude-runner.sh');
-            // Should cd to the script's directory
-            (0, vitest_1.expect)(script).toContain('cd "/path/to/worktree"');
-            (0, vitest_1.expect)(script).toContain('/bin/bash');
-        });
         (0, vitest_1.it)('Terminal script should cd to script directory before running', () => {
             const script = (0, helpers_1.generateTerminalOpenScript)('/path/to/worktree/.claude-runner.sh');
             // Should cd to the script's directory first (quotes are escaped in AppleScript)
@@ -105,7 +64,7 @@ const helpers_1 = require("./helpers");
             (0, vitest_1.expect)(script).toContain('/bin/bash');
         });
         (0, vitest_1.it)('should handle script paths with quotes', () => {
-            const script = (0, helpers_1.generateITermOpenScript)('/path/to/work"tree/.script.sh');
+            const script = (0, helpers_1.generateTerminalOpenScript)('/path/to/work"tree/.script.sh');
             (0, vitest_1.expect)(script).toContain('/bin/bash');
             // Path should be escaped
             (0, vitest_1.expect)(script).toContain('work');

package/package.json

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