Drop activeTab support (#78)

Author: fregante

.github/workflows/ci.yml

@@ -33,7 +33,8 @@ jobs:
       - run: npm run vitest
 
   Test:
-    runs-on: ubuntu-latest
+    # https://github.com/puppeteer/puppeteer/issues/12818#issuecomment-2415915338
+    runs-on: ubuntu-22.04
     strategy:
       fail-fast: false
       matrix:

.github/workflows/esm-lint.yml

@@ -58,7 +58,7 @@ jobs:
     needs: Pack
     steps:
       - uses: actions/download-artifact@v4
-      - run: npm install --omit=dev ./artifact rollup@2 @rollup/plugin-node-resolve
+      - run: npm install --omit=dev ./artifact rollup@4 @rollup/plugin-node-resolve
       - run: echo "$IMPORT_STATEMENT" > index.js
       - run: npx rollup -p node-resolve index.js
   Vite:
@@ -84,10 +84,11 @@ jobs:
     needs: Pack
     steps:
       - uses: actions/download-artifact@v4
+      - run: echo '{"type":"module"}' > package.json
       - run: npm install --omit=dev ./artifact @sindresorhus/tsconfig
-      - run: echo "$IMPORT_STATEMENT" > index.ts
+      - run: echo "$IMPORT_STATEMENT" > index.mts
       - run: >
-          echo '{"extends":"@sindresorhus/tsconfig","files":["index.ts"]}' >
+          echo '{"extends":"@sindresorhus/tsconfig","files":["index.mts"]}' >
           tsconfig.json
       - run: npx --package typescript -- tsc
-      - run: cat distribution/index.js
+      - run: cat distribution/index.mjs

.npmignore

@@ -2,4 +2,3 @@
 !/distribution/*
 *.test.*
 !/utils.*
-!/including-active-tab.*

package.json

@@ -1,7 +1,7 @@
 {
 	"name": "webext-dynamic-content-scripts",
 	"version": "10.0.4",
-	"description": "WebExtension module: Automatically registers your `content_scripts` on domains added via `permission.request` or on `activeTab`",
+	"description": "WebExtension module: Automatically registers your `content_scripts` on domains added via `permission.request`",
 	"keywords": [
 		"contentscript",
 		"register",
@@ -14,7 +14,6 @@
 		"chrome",
 		"firefox",
 		"browser",
-		"activetab",
 		"extension"
 	],
 	"repository": "fregante/webext-dynamic-content-scripts",
@@ -27,10 +26,6 @@
 			"types": "./distribution/index.d.ts",
 			"default": "./distribution/index.js"
 		},
-		"./including-active-tab.js": {
-			"types": "./distribution/including-active-tab.d.ts",
-			"default": "./distribution/including-active-tab.js"
-		},
 		"./utils.js": {
 			"types": "./distribution/utils.d.ts",
 			"default": "./distribution/utils.js"
@@ -137,9 +132,6 @@
 				"https://dynamic-ephiframe.vercel.app/Dynamic?iframe=./Inner",
 				"https://dynamic-ephiframe.vercel.app/Dynamic?iframe=https://static-ephiframe.vercel.app/Static-inner",
 				"https://static-ephiframe.vercel.app/Static?iframe=https://dynamic-ephiframe.vercel.app/Dynamic-inner",
-				"https://extra-ephiframe.vercel.app/Inject-via-context-menu-please",
-				"https://extra-ephiframe.vercel.app/Inject-via-context-menu-please?iframe=https://static-ephiframe.vercel.app/Static-inner",
-				"https://extra-ephiframe.vercel.app/Inject-via-context-menu-please?iframe=https://dynamic-ephiframe.vercel.app/Dynamic-inner",
 				"chrome://extensions/"
 			]
 		}

readme.md

@@ -59,19 +59,6 @@ navigator.importScripts('webext-dynamic-content-scripts.js');
 }
 ```
 
-### `activeTab` tracking
-
-By default, the module will only inject the content scripts into newly-permitted hosts, but it will ignore temporary permissions like `activeTab`. If you also want to automatically inject the content scripts into every frame of tabs as soon as they receive the `activeTab` permission, import a different entry point **instead of the default one.**
-
-```js
-import 'webext-dynamic-content-scripts/including-active-tab.js';
-```
-
-> **Note**
-> This does not work well in Firefox because of some compounding bugs:
-> - `activeTab` seems to be lost after a reload
-> - further `contextMenu` clicks receive a moz-extension URL rather than the current page’s URL
-
 ### Additional APIs
 
 #### `isContentScriptRegistered(url)`

source/active-tab.ts

@@ -1,15 +1,3 @@
-import 'webext-dynamic-content-scripts/including-active-tab.ts';
-
-chrome.contextMenus.create({
-	title: 'Enable activeTab permission',
-}, () => {
-	if (chrome.runtime.lastError) {
-		// Shush
-	}
-});
-
-chrome.contextMenus.onClicked.addListener(() => {
-	console.log('Context menu clicked');
-});
+import 'webext-dynamic-content-scripts';
 
 console.log('Background loaded');

source/including-active-tab.ts

@@ -3,9 +3,7 @@
 	"version": "0.0.0",
 	"manifest_version": 2,
 	"permissions": [
-		"activeTab",
 		"webNavigation",
-		"contextMenus",
 		"https://dynamic-ephiframe.vercel.app/*",
 		"https://accepted-ephiframe.vercel.app/*"
 	],