fix: handle no active tab scenarios (#164)

* fix: improve getActiveWindowTab function to handle no active tab scenarios

* fix: handle undefined active tab in heartbeat

* Lint :)

src/background/heartbeat.ts

@@ -8,7 +8,7 @@ import deepEqual from 'deep-equal'
 
 async function heartbeat(
   client: AWClient,
-  tab: browser.Tabs.Tab,
+  tab: browser.Tabs.Tab | undefined,
   tabCount: number,
 ) {
   const enabled = await getEnabled()
@@ -17,10 +17,13 @@ async function heartbeat(
     return
   }
 
+  if (!tab) {
+    console.warn('Ignoring heartbeat because no active tab was found')
+    return
+  }
+
   if (!tab.url || !tab.title) {
-    console.warn(
-      'Ignoring heartbeat because tab is missing required properties',
-    )
+    console.warn('Ignoring heartbeat because tab is missing URL or title')
     return
   }
 

src/background/helpers.ts

@@ -6,15 +6,33 @@ export const getTab = (id: number) => browser.tabs.get(id)
 export const getTabs = (query: browser.Tabs.QueryQueryInfoType = {}) =>
   browser.tabs.query(query)
 
-export const getActiveWindowTab = () =>
-  getTabs({
+export const getActiveWindowTab = async (): Promise<
+  browser.Tabs.Tab | undefined
+> => {
+  const tabs = await getTabs({
     active: true,
     currentWindow: true,
-  }).then((tabs) => {
-    if (tabs.length === 0) throw Error('No active window tab found')
+  })
+
+  if (tabs.length > 0) {
     return tabs[0]
+  }
+
+  console.debug('No active tab found in current window')
+
+  const allTabs = await getTabs({
+    active: true,
   })
 
+  if (allTabs.length > 0) {
+    return allTabs[0]
+  }
+
+  console.debug('No active tab found in any window')
+
+  return undefined
+}
+
 export function emitNotification(title: string, message: string) {
   browser.notifications.create({
     type: 'basic',