greptile 2.2.5 → 2.2.7

package/greptile-fix.applescript

@@ -41,15 +41,15 @@ on open location theURL
 			-- so we launch bash and source the runner script to keep the shell session alive
 			-- after the IDE command (claude/codex/cursor) finishes.
 			-- Unset CLAUDECODE so the new session doesn't think it's nested inside an existing one.
-			set cleanEnv to "unset CLAUDECODE; source " & quoted form of runnerPath & " ; exec bash"
+			set cleanEnv to "unset CLAUDECODE; source " & runnerPath & " ; exec zsh"
 			if termApp is "Ghostty" then
-				do shell script "open -na Ghostty --args -e bash -c '" & cleanEnv & "' &>/dev/null &"
+				do shell script "open -na Ghostty --args -e zsh -li -c '" & cleanEnv & "' &>/dev/null &"
 			else if termApp is "kitty" then
-				do shell script "open -na kitty --args bash -c '" & cleanEnv & "' &>/dev/null &"
+				do shell script "open -na kitty --args zsh -li -c '" & cleanEnv & "' &>/dev/null &"
 			else if termApp is "WezTerm" then
-				do shell script "open -na WezTerm --args start -- bash -c '" & cleanEnv & "' &>/dev/null &"
+				do shell script "open -na WezTerm --args start -- zsh -li -c '" & cleanEnv & "' &>/dev/null &"
 			else if termApp is "Alacritty" then
-				do shell script "open -na Alacritty --args -e bash -c '" & cleanEnv & "' &>/dev/null &"
+				do shell script "open -na Alacritty --args -e zsh -li -c '" & cleanEnv & "' &>/dev/null &"
 			else
 				-- Terminal.app, iTerm, and Warp natively execute scripts via "open -a"
 				do shell script "open -a " & quoted form of termApp & " " & quoted form of runnerPath

package/health-server.js

@@ -5,19 +5,33 @@ const path = require('path')
 
 const PORT = 4747
 
-// Self-destruct: if the package has been uninstalled (package.json gone),
-// clean up the LaunchAgent and exit so the health server doesn't linger.
+// Self-destruct: if the package has been uninstalled, clean up and exit.
+// Checks that package.json next to this script still exists.
+// Note: we intentionally don't check `command -v greptile-fix` because
+// launchd runs with a minimal PATH that doesn't include npm global bin.
 const PACKAGE_MARKER = path.join(__dirname, 'package.json')
 const SELF_CHECK_INTERVAL_MS = 30_000
 
 function selfCheck() {
-  if (!fs.existsSync(PACKAGE_MARKER)) {
+  const markerGone = !fs.existsSync(PACKAGE_MARKER)
+
+  if (markerGone) {
     console.log('Package uninstalled — cleaning up and exiting.')
-    const plistPath = path.join(process.env.HOME || '', 'Library/LaunchAgents/com.greptile.health.plist')
+    const { execSync } = require('child_process')
+    const home = process.env.HOME || ''
+    const plistPath = path.join(home, 'Library/LaunchAgents/com.greptile.health.plist')
+    // Remove .app bundle and plist file
+    try {
+      fs.rmSync(path.join(home, 'Applications/Greptile Fix.app'), { recursive: true, force: true })
+    } catch {}
     try {
       fs.unlinkSync(plistPath)
     } catch {}
     server.close()
+    // Remove the launchd job by label so KeepAlive won't restart us
+    try {
+      execSync('launchctl remove com.greptile.health 2>/dev/null')
+    } catch {}
     process.exit(0)
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "greptile",
-  "version": "2.2.5",
+  "version": "2.2.7",
   "description": "Bridge for Greptile code review 'Fix in Claude Code' and 'Fix in Codex' links",
   "bin": {
     "greptile-fix": "bin/greptile-fix.js"