improve integration within who am i chrome extension (#52)

Author: yooper

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "your-rapport",
-  "version": "0.29.0",
+  "version": "0.30.0",
   "description": "A chrome extension for saving screenshots and making them searchable.",
   "license": "Commercial",
   "scripts": {

package.json

@@ -9,7 +9,7 @@
     "default_title": "Your Rapport is a screenshot tool with auto scroll capabilities. Right click, select Collect or Alt+S to save. Alt+A for auto scrolling captures. Alt+X open the dashboard. Alt+Q for doing a quick scan.",
     "default_popup": "popup.html"
   },
-  "version": "0.29.0",
+  "version": "0.30.0",
   "icons": {
     "128": "icon-128.png",
     "48": "icon-48.png",

src/manifest.json

@@ -24,6 +24,7 @@ import { db } from '../../models/db/dexieDb';
 import { Rapport } from '../../models/schemas/Rapport';
 import { ScheduledAutomation } from '../../models/schemas/ScheduledAutomation';
 import { getActivePageInfo } from '../Content/scripts/pageInfo';
+import { requestAllSitesAccess } from '../../services/manifest_permissions';
 
 /**
  * Initialize services when the extension is installed / activated
@@ -220,11 +221,16 @@ chrome.runtime.onMessage.addListener((message, sender, sendResponse) => {
 chrome.runtime.onMessageExternal.addListener((message, sender, sendResponse) => {
   debug('onMessageExternal:start', {message, sender});
 
+  let hasPermission = false;
   try {
     (async () => {
       switch (message.cmd) {
-        case 'singleCollect': // TODO: Deprecated remove after 1/1/26
         case 'deepSave':
+          hasPermission = await requestAllSitesAccess()
+          if(!hasPermission){
+            sendResponse({completed: false, message: 'You must accept the permissions change in order to enable this automation.'})
+            break;
+          }
           await createTab(message.url);
           const activeTab = await getActiveTab();
           // wait for page contents to load
@@ -233,36 +239,56 @@ chrome.runtime.onMessageExternal.addListener((message, sender, sendResponse) =>
           // TODO: make this configurable or dynamic based on the domain
           await capture(activeTab, pageInfo, true);
           sendResponse({completed: true})
+          try{
+            await chrome.tabs.remove(activeTab.id);
+          }
+          catch(error){
+            // could not close the tab
+          }
           break;
         case AUTO_COLLECT_STARTING:
-        case 'autoscrollCollect': // TODO: Deprecated remove after 1/1/26
+          hasPermission = await requestAllSitesAccess()
+          if(!hasPermission){
+            sendResponse({completed: false, message: 'You must accept the permissions change in order to enable this automation.'})
+            break;
+          }
           await createTab(message.url);
           await sleep(3000);
           sendResponse({completed: true})
-
           chrome.tabs.sendMessage((await getActiveTab()).id, { cmd: ACTIVATE_CAPTURE, requestId: crypto.randomUUID() })
             .then(response => {
               debug(ACTIVATE_CAPTURE + ':', response);
             })
           break;
         case ENQUEUE_BULK_AUTOMATION_URL:
         case 'enqueueBulkAutomation':
+          hasPermission = await requestAllSitesAccess()
+          if(!hasPermission){
+            sendResponse({completed: false, message: 'You must accept the permissions change in order to enable this automation.'})
+            break;
+          }
           const record = await BulkAutomationUrl.createBulkAutomationJob(message.url);
           await db.bulkAutomation.add(record);
           break;
         case 'ping':
           sendResponse({completed: true});
           break;
         case 'monitorHourly':
+          hasPermission = await requestAllSitesAccess()
+          if(!hasPermission){
+            sendResponse({completed: false, message: 'You must accept the permissions change in order to enable this automation.'})
+            break;
+          }
           await ScheduledAutomation.addMonitor(message.url, '0 0 * * * *');
+          sendResponse({completed: true})
           break;
         default:
           return false;
       }
     })()
     return false;
   } catch (e) {
-    debug('onMessageExternal:failure', { message, sender })
+    debug('onMessageExternal:failure', { message, sender, e })
   }
 })