From 8ab6d005583590c472f4e51247f7051a3970da96 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Sat, 22 Aug 2026 00:42:16 +0000
Subject: [PATCH] Extract chrome.tabs.onUpdated and onActivated listener
 callbacks in sidepanel.js

Extracted deeply nested anonymous functions into named functions `handleTabActivated` and `handleTabUpdated` to improve readability and code health. No functionality was changed.

Co-authored-by: savvides <1580637+savvides@users.noreply.github.com>
---
 extension/sidepanel/sidepanel.js | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/extension/sidepanel/sidepanel.js b/extension/sidepanel/sidepanel.js
index 7b383c9..52514e9 100644
--- a/extension/sidepanel/sidepanel.js
+++ b/extension/sidepanel/sidepanel.js
@@ -460,18 +460,22 @@ if (saveSettingsBtn && apiKeyInput) {
 })();
 
 // Listen for tab switching / updates
+function handleTabActivated() {
+  refreshActiveTab();
+}
+
+function handleTabUpdated(tabId, changeInfo, tab) {
+  if (changeInfo.status === 'complete' && tab.active) {
+    refreshActiveTab();
+  }
+}
+
 if (typeof chrome !== 'undefined' && chrome.tabs) {
   if (chrome.tabs.onActivated) {
-    chrome.tabs.onActivated.addListener(() => {
-      refreshActiveTab();
-    });
+    chrome.tabs.onActivated.addListener(handleTabActivated);
   }
   if (chrome.tabs.onUpdated) {
-    chrome.tabs.onUpdated.addListener((tabId, changeInfo, tab) => {
-      if (changeInfo.status === 'complete' && tab.active) {
-        refreshActiveTab();
-      }
-    });
+    chrome.tabs.onUpdated.addListener(handleTabUpdated);
   }
 }