From 7c028e8c09bbc380e6513444e7a174697a9cb83a Mon Sep 17 00:00:00 2001
From: NK <nk@NKs-Mac-mini.local>
Date: Tue, 17 Feb 2026 22:04:16 -0800
Subject: [PATCH] fix: respect canceled_by_user and replaced_with_devtools
 detach reasons

Skip re-attach when user explicitly dismisses debugger bar or opens
DevTools. Prevents frustrating re-attach loop that fights user intent.

Addresses review feedback from greptile-apps.
---
 assets/chrome-extension/background.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/assets/chrome-extension/background.js b/assets/chrome-extension/background.js
index 294d9f87b2b..5ebe4008af3 100644
--- a/assets/chrome-extension/background.js
+++ b/assets/chrome-extension/background.js
@@ -653,15 +653,18 @@ async function onDebuggerDetach(source, reason) {
   if (!tabId) return
   if (!tabs.has(tabId)) return
 
+  // User explicitly cancelled or DevTools replaced the connection — respect their intent
   if (reason === 'canceled_by_user' || reason === 'replaced_with_devtools') {
     void detachTab(tabId, reason)
     return
   }
 
+  // Check if tab still exists — distinguishes navigation from tab close
   let tabInfo
   try {
     tabInfo = await chrome.tabs.get(tabId)
   } catch {
+    // Tab is gone (closed) — normal cleanup
     void detachTab(tabId, reason)
     return
   }