greptile 2.3.0 → 2.4.0

package/build-app.sh

@@ -45,7 +45,7 @@ TEMP_SCRIPT="$(mktemp /tmp/greptile-fix-XXXXXX.applescript)"
 trap 'rm -f "$TEMP_SCRIPT"' EXIT
 
 # Escape sed special characters in the directory path (& \ /)
-ESCAPED_DIR="$(printf '%s' "$GREPTILE_FIX_DIR" | sed 's/[&/\]/\\&/g')"
+ESCAPED_DIR="$(printf '%s' "$GREPTILE_FIX_DIR" | sed 's/[&\\/]/\\&/g')"
 sed "s|PATH=/opt/homebrew/bin:/usr/local/bin:|PATH=$ESCAPED_DIR:/opt/homebrew/bin:/usr/local/bin:|" \
   "$SCRIPT_DIR/greptile-fix.applescript" > "$TEMP_SCRIPT"
 

package/greptile-fix

@@ -187,6 +187,7 @@ esac
 # Parse URL parameters
 PROMPT=$(parse_url_param "$URL" "prompt")
 REPO=$(parse_url_param "$URL" "repo")
+ACK_ID=$(parse_url_param "$URL" "ack")
 
 if [ -z "$PROMPT" ]; then
   log "ERROR: No prompt parameter in URL"
@@ -234,5 +235,9 @@ 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).
+# Line 1: runner path, Line 2 (optional): ACK ID for auto-redirect
 echo "$RUNNER"
+if [ -n "$ACK_ID" ]; then
+  echo "$ACK_ID"
+fi
 log "Done"

package/greptile-fix.applescript

@@ -12,8 +12,16 @@ on open location theURL
 	try
 		-- Ensure log directory exists
 		do shell script "mkdir -p $HOME/.cache/greptile"
-		-- 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>> $HOME/.cache/greptile/greptile-fix.log"
+		-- Call greptile-fix to parse URL and create runner script
+		-- Output: line 1 = runner path, line 2 (optional) = ACK ID for auto-redirect
+		set fixOutput to do shell script "PATH=/opt/homebrew/bin:/usr/local/bin:$PATH greptile-fix " & quoted form of theURL & " 2>> $HOME/.cache/greptile/greptile-fix.log"
+
+		set outputLines to paragraphs of fixOutput
+		set runnerPath to item 1 of outputLines
+		set ackId to ""
+		if (count of outputLines) > 1 then
+			set ackId to item 2 of outputLines
+		end if
 
 		if runnerPath is not "" then
 			-- Detect terminal by checking running processes via shell (no accessibility permissions needed).
@@ -56,6 +64,13 @@ on open location theURL
 				-- Terminal.app, iTerm, and Warp natively execute scripts via "open -a"
 				do shell script "open -a " & quoted form of termApp & " " & quoted form of runnerPath
 			end if
+
+			-- Send ACK to health server so the web page can auto-redirect back to the PR
+			if ackId is not "" then
+				try
+					do shell script "curl -s -X POST http://127.0.0.1:4747/ack/" & quoted form of ackId & " --max-time 2 &>/dev/null &"
+				end try
+			end if
 		end if
 	on error errMsg
 		do shell script "echo '[ERROR] " & quoted form of errMsg & "' >> $HOME/.cache/greptile/greptile-fix.log"

package/health-server.js

@@ -95,11 +95,13 @@ function checkForUpdate() {
           const npmPaths = [
             '/opt/homebrew/bin/npm',
             '/usr/local/bin/npm',
-            ...(home ? [
-              path.join(home, '.volta/bin/npm'),
-              path.join(home, '.asdf/shims/npm'),
-              path.join(home, '.local/bin/npm'),
-            ] : []),
+            ...(home
+              ? [
+                  path.join(home, '.volta/bin/npm'),
+                  path.join(home, '.asdf/shims/npm'),
+                  path.join(home, '.local/bin/npm'),
+                ]
+              : []),
           ]
           // Also check nvm: find the current default node version's npm
           if (home) {
@@ -173,12 +175,26 @@ function checkForUpdate() {
   })
 }
 
+// --- ACK store for "Fix in X" auto-redirect ---
+const ackStore = new Map() // id → timestamp
+const ACK_TTL_MS = 60_000
+
+function pruneAcks() {
+  const now = Date.now()
+  for (const [id, ts] of ackStore) {
+    if (now - ts > ACK_TTL_MS) ackStore.delete(id)
+  }
+}
+setInterval(pruneAcks, 30_000)
+
+const ACK_PATH_RE = /^\/ack\/([a-zA-Z0-9_-]+)$/
+
 const ALLOWED_ORIGINS = ['https://app.greptile.com', 'https://app.staging.greptile.com', 'http://localhost:3000']
 
 function isAllowedOrigin(origin) {
   if (ALLOWED_ORIGINS.includes(origin)) return true
-  // Allow Vercel preview deployments (e.g. https://greptilia-xyz.vercel.app)
-  if (/^https:\/\/[a-z0-9-]+\.vercel\.app$/.test(origin)) return true
+  // Allow Vercel preview deployments (e.g. https://greptilia-abc123.vercel.app)
+  if (/^https:\/\/greptilia-[a-z0-9-]+\.vercel\.app$/.test(origin)) return true
   return false
 }
 
@@ -188,7 +204,7 @@ const server = http.createServer((req, res) => {
 
   if (allowed) {
     res.setHeader('Access-Control-Allow-Origin', origin)
-    res.setHeader('Access-Control-Allow-Methods', 'GET, OPTIONS')
+    res.setHeader('Access-Control-Allow-Methods', 'GET, POST, OPTIONS')
     res.setHeader('Access-Control-Allow-Headers', 'Content-Type')
     res.setHeader('Access-Control-Allow-Private-Network', 'true')
   }
@@ -205,6 +221,26 @@ const server = http.createServer((req, res) => {
     return
   }
 
+  // ACK endpoints for "Fix in X" auto-redirect
+  const urlPath = (req.url || '').split('?')[0]
+  const ackMatch = urlPath.match(ACK_PATH_RE)
+  if (ackMatch) {
+    const id = ackMatch[1]
+    if (req.method === 'POST') {
+      ackStore.set(id, Date.now())
+      res.writeHead(200, { 'Content-Type': 'application/json' })
+      res.end(JSON.stringify({ ok: true }))
+      return
+    }
+    if (req.method === 'GET') {
+      const acked = ackStore.has(id)
+      if (acked) ackStore.delete(id)
+      res.writeHead(200, { 'Content-Type': 'application/json' })
+      res.end(JSON.stringify({ acked }))
+      return
+    }
+  }
+
   res.writeHead(404)
   res.end()
 })

package/package.json

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