webext-messenger 0.24.0 → 0.25.0-0

package/distribution/index.js

@@ -5,4 +5,6 @@ export * from "./types.js";
 export { getThisFrame, getTopLevelFrame } from "./thisTarget.js";
 export { toggleLogging } from "./logging.js";
 import { initPrivateApi } from "./thisTarget.js";
+// Required side effect to better track errors:
+// https://github.com/pixiebrix/webext-messenger/pull/80
 initPrivateApi();

package/distribution/shared.js

@@ -4,15 +4,7 @@ export function isObject(value) {
     return typeof value === "object" && value !== null;
 }
 export class MessengerError extends Error {
-    constructor() {
-        super(...arguments);
-        Object.defineProperty(this, "name", {
-            enumerable: true,
-            configurable: true,
-            writable: true,
-            value: "MessengerError"
-        });
-    }
+    name = "MessengerError";
 }
 // @ts-expect-error Wrong `errorConstructors` types
 errorConstructors.set("MessengerError", MessengerError);

package/distribution/thisTarget.js

@@ -106,9 +106,7 @@ const storeTabData = once(async () => {
     }
     catch (error) {
         tabDataStatus = "error";
-        throw new MessengerError("Tab registration failed. This page won’t be able to receive messages that require tab information", 
-        // @ts-expect-error TODO: update lib to accept Error#cause
-        { cause: error });
+        throw new MessengerError("Tab registration failed. This page won’t be able to receive messages that require tab information", { cause: error });
     }
 });
 export function __getTabData() {
@@ -131,6 +129,16 @@ export async function getTopLevelFrame() {
     };
 }
 export function initPrivateApi() {
+    // Improve DX by informing the developer that it's being loaded the wrong way
+    // https://github.com/pixiebrix/webext-messenger/issues/88
+    if (globalThis.__webextMessenger) {
+        // TODO: Use Error#cause after https://bugs.chromium.org/p/chromium/issues/detail?id=1211260
+        console.log(globalThis.__webextMessenger.replace(/^Error/, "webext-messenger"));
+        console.error("webext-messenger: Duplicate execution. This is a fatal error.\nhttps://github.com/pixiebrix/webext-messenger/issues/88");
+        return;
+    }
+    // Use Error to capture the stack and make it easier to find the cause
+    globalThis.__webextMessenger = new Error("First execution").stack;
     if (isExtensionContext()) {
         // Only `runtime` pages can handle this message but I can't remove it because its listener
         // also serves the purpose of throwing a specific error when no methods have been registered.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "webext-messenger",
-  "version": "0.24.0",
+  "version": "0.25.0-0",
   "description": "Browser Extension component messaging framework",
   "keywords": [],
   "repository": "pixiebrix/webext-messenger",
@@ -27,25 +27,25 @@
   "dependencies": {
     "p-retry": "^6.0.0",
     "serialize-error": "^11.0.2",
-    "type-fest": "^4.3.1",
+    "type-fest": "^4.6.0",
     "webext-detect-page": "^4.1.1"
   },
   "devDependencies": {
     "@parcel/config-webextension": "^2.6.2",
-    "@sindresorhus/tsconfig": "^4.0.0",
-    "@types/chrome": "^0.0.245",
-    "@types/tape": "^5.6.1",
-    "@types/webextension-polyfill": "^0.10.2",
+    "@sindresorhus/tsconfig": "^5.0.0",
+    "@types/chrome": "^0.0.248",
+    "@types/tape": "^5.6.3",
+    "@types/webextension-polyfill": "^0.10.5",
     "buffer": "^6.0.3",
-    "eslint": "^8.50.0",
-    "eslint-config-pixiebrix": "^0.27.2",
+    "eslint": "^8.52.0",
+    "eslint-config-pixiebrix": "^0.29.1",
     "events": "^3.3.0",
     "npm-run-all": "^4.1.5",
     "parcel": "^2.6.2",
     "path-browserify": "^1.0.1",
     "process": "^0.11.10",
     "stream-browserify": "^3.0.0",
-    "tape": "^5.7.0",
+    "tape": "^5.7.2",
     "typescript": "^5.2.2",
     "webext-content-scripts": "^2.5.5",
     "webextension-polyfill": "^0.10.0"