greptile 2.2.0 → 2.2.2

package/com.greptile.health.plist

@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+	<key>Label</key>
+	<string>com.greptile.health</string>
+	<key>ProgramArguments</key>
+	<array>
+		<string>/usr/local/bin/node</string>
+		<string>health-server.js</string>
+	</array>
+	<key>WorkingDirectory</key>
+	<string>__PACKAGE_DIR__</string>
+	<key>RunAtLoad</key>
+	<true/>
+	<key>KeepAlive</key>
+	<true/>
+	<key>StandardOutPath</key>
+	<string>/tmp/greptile-health.log</string>
+	<key>StandardErrorPath</key>
+	<string>/tmp/greptile-health.log</string>
+</dict>
+</plist>

package/greptile-fix

@@ -215,44 +215,8 @@ log "Creating runner script for: $REPO_PATH (IDE: $IDE)"
 RUNNER=$(create_runner_script "$REPO_PATH" "$PROMPT" "$IDE")
 log "Runner script: $RUNNER"
 
-# 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"
+# 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).
 echo "$RUNNER"
 log "Done"

package/greptile-fix.applescript

@@ -5,19 +5,45 @@
 -- 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 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>"
+-- 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>"
 
 on open location theURL
 	try
-		-- 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}
+		-- 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"
 
 		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/health-server.js

@@ -0,0 +1,60 @@
+#!/usr/bin/env node
+const http = require('http')
+const fs = require('fs')
+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.
+const PACKAGE_MARKER = path.join(__dirname, 'package.json')
+const SELF_CHECK_INTERVAL_MS = 30_000
+
+function selfCheck() {
+  if (!fs.existsSync(PACKAGE_MARKER)) {
+    console.log('Package uninstalled — cleaning up and exiting.')
+    const plistPath = path.join(
+      process.env.HOME || '',
+      'Library/LaunchAgents/com.greptile.health.plist'
+    )
+    try { fs.unlinkSync(plistPath) } catch {}
+    server.close()
+    process.exit(0)
+  }
+}
+
+const server = http.createServer((req, res) => {
+  res.setHeader('Access-Control-Allow-Origin', '*')
+  res.setHeader('Access-Control-Allow-Methods', 'GET, OPTIONS')
+  res.setHeader('Access-Control-Allow-Headers', 'Content-Type')
+
+  if (req.method === 'OPTIONS') {
+    res.writeHead(204)
+    res.end()
+    return
+  }
+
+  if (req.method === 'GET' && req.url === '/health') {
+    res.writeHead(200, { 'Content-Type': 'application/json' })
+    res.end(JSON.stringify({ status: 'ok' }))
+    return
+  }
+
+  res.writeHead(404)
+  res.end()
+})
+
+server.on('error', (err) => {
+  if (err.code === 'EADDRINUSE') {
+    console.error(`Port ${PORT} is already in use. Exiting.`)
+    process.exit(0) // Exit cleanly so launchd KeepAlive doesn't restart in a tight loop
+  }
+  console.error('Health server error:', err.message)
+  process.exit(1)
+})
+
+server.listen(PORT, '127.0.0.1', () => {
+  console.log(`Greptile health server listening on http://127.0.0.1:${PORT}`)
+  // Start self-check loop after server is up
+  setInterval(selfCheck, SELF_CHECK_INTERVAL_MS)
+})

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "greptile",
-  "version": "2.2.0",
+  "version": "2.2.2",
   "description": "Bridge for Greptile code review 'Fix in Claude Code' and 'Fix in Codex' links",
   "bin": {
     "greptile-fix": "bin/greptile-fix.js"
@@ -15,7 +15,9 @@
     "greptile-fix.applescript",
     "build-app.sh",
     "postinstall.sh",
-    "preuninstall.sh"
+    "preuninstall.sh",
+    "health-server.js",
+    "com.greptile.health.plist"
   ],
   "os": [
     "darwin"

package/postinstall.sh

@@ -22,6 +22,29 @@ mkdir -p "$APP_DIR"
 # Build the .app bundle into ~/Applications
 bash "$SCRIPT_DIR/build-app.sh" "$APP_DIR"
 
+# --- Health server LaunchAgent ---
+PLIST_NAME="com.greptile.health.plist"
+PLIST_SRC="$SCRIPT_DIR/$PLIST_NAME"
+PLIST_DST="$HOME/Library/LaunchAgents/$PLIST_NAME"
+
+if [ -f "$PLIST_SRC" ]; then
+  # Ensure LaunchAgents directory exists
+  mkdir -p "$HOME/Library/LaunchAgents"
+
+  # Resolve the node binary path
+  NODE_BIN="$(command -v node 2>/dev/null || echo "/usr/local/bin/node")"
+
+  # Create a configured copy with the correct paths
+  sed -e "s|__PACKAGE_DIR__|$SCRIPT_DIR|g" \
+      -e "s|/usr/local/bin/node|$NODE_BIN|g" \
+      "$PLIST_SRC" > "$PLIST_DST"
+
+  # Load the agent (unload first in case it's already loaded)
+  launchctl unload "$PLIST_DST" 2>/dev/null || true
+  launchctl load "$PLIST_DST"
+  echo "Greptile health server installed and started."
+fi
+
 echo ""
 echo "Repo path mappings are stored in ~/.greptile/repos.json."
 echo "The first time you click 'Fix in Claude Code' for a repo,"

package/preuninstall.sh

@@ -13,3 +13,11 @@ if [ -d "$APP_PATH" ]; then
   rm -rf "$APP_PATH"
   echo "Removed: $APP_PATH"
 fi
+
+# --- Health server LaunchAgent ---
+PLIST_PATH="$HOME/Library/LaunchAgents/com.greptile.health.plist"
+if [ -f "$PLIST_PATH" ]; then
+  launchctl unload "$PLIST_PATH" 2>/dev/null || true
+  rm -f "$PLIST_PATH"
+  echo "Removed Greptile health server LaunchAgent."
+fi