greptile 2.1.0 → 2.2.0

package/greptile-fix

@@ -215,8 +215,44 @@ log "Creating runner script for: $REPO_PATH (IDE: $IDE)"
 RUNNER=$(create_runner_script "$REPO_PATH" "$PROMPT" "$IDE")
 log "Runner script: $RUNNER"
 
-# Print the runner path to stdout — the AppleScript app reads this
-# and uses it to open Terminal (which requires Automation permission
-# that only the .app bundle has).
+# Detect the user's terminal app using lsappinfo (no permissions needed).
+# Prefer the frontmost app if it's a known terminal, else first running terminal, else Terminal.
+detect_terminal() {
+  # Map of process names to "open -a" app names
+  local process_names=("iTerm2"  "Ghostty" "Warp" "kitty" "WezTerm" "Alacritty" "Terminal")
+  local app_names=(    "iTerm"   "Ghostty" "Warp" "kitty" "WezTerm" "Alacritty" "Terminal")
+
+  # Check if frontmost app is a terminal
+  local front_asn
+  front_asn=$(lsappinfo front 2>/dev/null)
+  if [ -n "$front_asn" ]; then
+    local front_name
+    front_name=$(lsappinfo info -only name "$front_asn" 2>/dev/null | sed 's/.*="\(.*\)"/\1/')
+    for i in "${!process_names[@]}"; do
+      if [ "$front_name" = "${process_names[$i]}" ]; then
+        echo "${app_names[$i]}"
+        return
+      fi
+    done
+  fi
+
+  # Check for first running terminal
+  local running
+  running=$(lsappinfo list 2>/dev/null)
+  for i in "${!process_names[@]}"; do
+    if echo "$running" | grep -q "\"${process_names[$i]}\""; then
+      echo "${app_names[$i]}"
+      return
+    fi
+  done
+
+  echo "Terminal"
+}
+
+TERM_APP=$(detect_terminal)
+log "Detected terminal: $TERM_APP"
+
+# Print terminal app and runner path (two lines) — the AppleScript reads both.
+echo "$TERM_APP"
 echo "$RUNNER"
 log "Done"

package/greptile-fix.applescript

@@ -5,45 +5,19 @@
 -- Flow:
 -- 1. macOS sends us the greptile:// URL
 -- 2. We call greptile-fix CLI to parse the URL, resolve the repo path, and create a runner script
--- 3. greptile-fix prints the runner script path to stdout
--- 4. We detect the user's terminal and open the runner script with "open -a <terminal>"
+-- 3. greptile-fix detects the user's terminal (via lsappinfo) and prints two lines:
+--    line 1: terminal app name (e.g. "iTerm", "Ghostty", "Terminal")
+--    line 2: runner script path
+-- 4. We open the runner script with "open -a <terminal>"
 
 on open location theURL
 	try
-		-- Call greptile-fix to parse URL and create runner script; it prints the runner path to stdout
-		set runnerPath to do shell script "PATH=/opt/homebrew/bin:/usr/local/bin:$PATH greptile-fix " & quoted form of theURL & " 2>> /tmp/greptile-fix.log"
+		-- Call greptile-fix; it prints terminal app name (line 1) and runner path (line 2) to stdout
+		set outputLines to do shell script "PATH=/opt/homebrew/bin:/usr/local/bin:$PATH greptile-fix " & quoted form of theURL & " 2>> /tmp/greptile-fix.log"
+
+		set {termApp, runnerPath} to {paragraph 1 of outputLines, paragraph 2 of outputLines}
 
 		if runnerPath is not "" then
-			-- Detect terminal app: prefer frontmost if it's a terminal, else first running terminal, else Terminal.app
-			set termApp to "Terminal"
-			set knownTerminals to {"iTerm2", "Ghostty", "Warp", "kitty", "WezTerm", "Alacritty", "Terminal"}
-			-- Map process names to the app name expected by "open -a"
-			-- (iTerm2's process name is "iTerm2" but the app name is "iTerm")
-			set termAppNames to {"iTerm", "Ghostty", "Warp", "kitty", "WezTerm", "Alacritty", "Terminal"}
-			try
-				tell application "System Events"
-					set frontName to name of first application process whose frontmost is true
-					set runningApps to name of every application process whose background only is false
-				end tell
-				-- If the frontmost app is a known terminal, use it
-				set foundFront to false
-				repeat with i from 1 to count of knownTerminals
-					if frontName is equal to item i of knownTerminals then
-						set termApp to item i of termAppNames
-						set foundFront to true
-						exit repeat
-					end if
-				end repeat
-				-- Otherwise pick the first running terminal we recognize
-				if not foundFront then
-					repeat with i from 1 to count of knownTerminals
-						if item i of knownTerminals is in runningApps then
-							set termApp to item i of termAppNames
-							exit repeat
-						end if
-					end repeat
-				end if
-			end try
 			do shell script "open -a " & quoted form of termApp & " " & quoted form of runnerPath
 		end if
 	on error errMsg

package/package.json

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

package/preuninstall.sh

@@ -2,6 +2,11 @@
 #
 # preuninstall.sh — Removes the Greptile Fix .app bundle on npm uninstall.
 
+# Only run on macOS
+if [ "$(uname)" != "Darwin" ]; then
+  exit 0
+fi
+
 APP_PATH="$HOME/Applications/Greptile Fix.app"
 
 if [ -d "$APP_PATH" ]; then