greptile 2.2.8 → 2.2.10

package/health-server.js

@@ -17,6 +17,9 @@ const UPDATE_CHECK_INTERVAL_MS = 6 * 60 * 60 * 1000 // 6 hours
 let isUpdating = false
 
 function selfCheck() {
+  // Don't self-destruct during an update — npm temporarily removes the package dir
+  if (isUpdating) return
+
   const markerGone = !fs.existsSync(PACKAGE_MARKER)
 
   if (markerGone) {
@@ -41,6 +44,7 @@ function selfCheck() {
 }
 
 // Auto-update: periodically check npm registry for a newer version and install it.
+// After a successful update, the process exits so launchd restarts it with the new code.
 function checkForUpdate() {
   if (isUpdating) return
   let localVersion
@@ -52,29 +56,49 @@ function checkForUpdate() {
   }
 
   const req = https.get('https://registry.npmjs.org/greptile/latest', { timeout: 10_000 }, (res) => {
+    if (res.statusCode !== 200) {
+      res.resume()
+      return
+    }
     let data = ''
-    res.on('data', (chunk) => { data += chunk })
+    const MAX_BODY = 100_000 // 100KB safety limit
+    res.on('data', (chunk) => {
+      data += chunk
+      if (data.length > MAX_BODY) {
+        req.destroy()
+        return
+      }
+    })
     res.on('end', () => {
       try {
         const latest = JSON.parse(data)
         if (latest.version && latest.version !== localVersion) {
           console.log(`Update available: ${localVersion} → ${latest.version}. Installing...`)
-          isUpdating = true
           // Resolve npm path — launchd has minimal PATH
           const npmPaths = ['/opt/homebrew/bin/npm', '/usr/local/bin/npm']
           let npmBin = 'npm'
           for (const p of npmPaths) {
-            if (fs.existsSync(p)) { npmBin = p; break }
+            if (fs.existsSync(p)) {
+              npmBin = p
+              break
+            }
           }
-          const child = spawn(npmBin, ['install', '-g', 'greptile@latest'], {
-            stdio: 'ignore',
-            detached: true,
-          })
-          child.unref()
+          let child
+          try {
+            child = spawn(npmBin, ['install', '-g', 'greptile@latest'], {
+              stdio: 'inherit',
+            })
+          } catch (err) {
+            console.error('Failed to spawn npm:', err.message)
+            return
+          }
+          isUpdating = true
           child.on('close', (code) => {
             isUpdating = false
             if (code === 0) {
-              console.log(`Updated to greptile@${latest.version}`)
+              console.log(`Updated to greptile@${latest.version}. Restarting...`)
+              // Exit so launchd restarts the server with the new code
+              setTimeout(() => process.exit(0), 2000)
             } else {
               console.error(`Update failed with exit code ${code}`)
             }
@@ -88,7 +112,9 @@ function checkForUpdate() {
     })
   })
   req.on('error', () => {}) // Silently ignore network errors
-  req.on('timeout', () => { req.destroy() })
+  req.on('timeout', () => {
+    req.destroy()
+  })
 }
 
 const ALLOWED_ORIGINS = ['https://app.greptile.com', 'https://app.staging.greptile.com', 'http://localhost:3000']

package/package.json

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