Fix bug: Login with token

The issue was in addListener. We had no sender and sendResponse defined
and thus the channel auto-closed without ack.
This fixes the bug and adds some debugging for future issues.

Closes #62

Author: mulbc

content.js

@@ -3,7 +3,12 @@
 // It will get the credentials via message passing from the popup
 // It is also responsible to copy strings to the clipboard
 
-browser.runtime.onMessage.addListener((request) => {
+console.log('VaultPass: Content script loaded');
+
+// We need sender and sendResponse so that the async communication works
+// otherwise users cannot login to vault using their token
+// eslint-disable-next-line no-unused-vars
+browser.runtime.onMessage.addListener((request, sender, sendResponse) => {
   switch (request.message) {
     case 'copy_to_clipboard':
       handleCopyToClipboard(request);
@@ -17,8 +22,12 @@ browser.runtime.onMessage.addListener((request) => {
       break;
     case 'fetch_token':
       handleFetchToken();
+      // If handleFetchToken is async or needs to send a response back,
+      // we should handle it here. For now, it sends a message back to runtime.
       break;
   }
+  // Return false because we don't need to keep the message channel open for an async response
+  return false;
 });
 
 function handleCopyToClipboard(request) {

options.js

@@ -311,6 +311,27 @@ async function tokenGrabberClick() {
     currentWindow: true,
   });
 
+  console.log('TokenGrabber: Target Tab', currentTab);
+
+  if (!currentTab) {
+    notify.error('No active tab found.');
+    return;
+  }
+
+  // Check if we are trying to grab token from the options page itself
+  if (
+    currentTab.url &&
+    (currentTab.url.startsWith('moz-extension://') ||
+      currentTab.url.startsWith('chrome-extension://'))
+  ) {
+    notify
+      .clear()
+      .error(
+        'You are on the Options page. Please go to the Vault tab, click the extension icon, click "Options", and then click "Get Token".'
+      );
+    return;
+  }
+
   try {
     await browser.tabs.sendMessage(currentTab.id, { message: 'fetch_token' });
   } catch (err) {
@@ -326,7 +347,8 @@ async function tokenGrabberClick() {
         );
       return;
     } else {
-      notify.clear().error(err.message);
+      console.error('TokenGrabber Error:', err);
+      notify.clear().error(`Error contacting Vault tab: ${err.message}`);
     }
   }
 }