webext-messenger 0.33.0-2 → 0.33.0

package/distribution/receiver.js

@@ -95,7 +95,7 @@ async function prepareResponse(message, action, meta) {
     const localHandler = handlers.get(type);
     if (localHandler) {
         if ("extensionId" in target && !externalMethods.has(type)) {
-            throw new MessengerError(`The ${type} is registered in ${getContextName()} for internal use only`);
+            throw new MessengerError(`The ${type} handler is registered in ${getContextName()} for internal use only`);
         }
         return localHandler.apply(meta, args);
     }

package/distribution/sender.js

@@ -1,7 +1,7 @@
 import pRetry from "p-retry";
-import { isBackground } from "webext-detect";
+import { isBackground, isExtensionContext } from "webext-detect";
 import { deserializeError } from "serialize-error";
-import { isObject, MessengerError, __webextMessenger } from "./shared.js";
+import { isObject, MessengerError, ExtensionNotFoundError, __webextMessenger, } from "./shared.js";
 import { log } from "./logging.js";
 import { handlers } from "./handlers.js";
 import { events } from "./events.js";
@@ -10,6 +10,7 @@ const _errorTargetClosedEarly = "A listener indicated an asynchronous response b
 export const errorTargetClosedEarly = "The target was closed before receiving a response";
 export const errorTabDoesntExist = "The tab doesn't exist";
 export const errorTabWasDiscarded = "The tab was discarded";
+const errorExtensionNotFound = "Extension $ID is not installed or externally connectable";
 function isMessengerResponse(response) {
     return isObject(response) && response["__webextMessenger"] === true;
 }
@@ -78,7 +79,12 @@ async function manageMessage(type, target, seq, retry, sendMessage) {
                     attemptCount: error.attemptNumber,
                 },
             }));
-            if (wasContextInvalidated()) {
+            if ("extensionId" in target &&
+                error.message === _errorNonExistingTarget) {
+                // The extension is not available and it will not be. Do not retry.
+                throw new ExtensionNotFoundError(errorExtensionNotFound.replace("$ID", target.extensionId));
+            }
+            if (isExtensionContext() && wasContextInvalidated()) {
                 // The error matches the native context invalidated error
                 // *.sendMessage() might fail with a message-specific error that is less useful,
                 // like "Sender closed without responding"
@@ -142,6 +148,9 @@ function messenger(type, options, target, ...args) {
     options.seq = globalSeq++;
     const { seq } = options;
     if ("extensionId" in target) {
+        if (!globalThis.chrome?.runtime?.sendMessage) {
+            throw new ExtensionNotFoundError(errorExtensionNotFound.replace("$ID", target.extensionId));
+        }
         const sendMessage = async (attemptCount) => {
             log.debug(type, seq, "↗️ sending message to extension", attemptLog(attemptCount));
             return chrome.runtime.sendMessage(target.extensionId, makeMessage(type, args, target, options));

package/distribution/shared.d.ts

@@ -10,6 +10,9 @@ export declare function isObject(value: unknown): value is Record<string, unknow
 export declare class MessengerError extends Error {
     name: string;
 }
+export declare class ExtensionNotFoundError extends MessengerError {
+    name: string;
+}
 export declare function isErrorObject(error: unknown): error is ErrorObject;
 export declare function delay(milliseconds: number): Promise<void>;
 export declare function once<Callback extends (...arguments_: unknown[]) => unknown>(function_: Callback): Callback;

package/distribution/shared.js

@@ -6,6 +6,9 @@ export function isObject(value) {
 export class MessengerError extends Error {
     name = "MessengerError";
 }
+export class ExtensionNotFoundError extends MessengerError {
+    name = "ExtensionNotFoundError";
+}
 addKnownErrorConstructor(MessengerError);
 export function isErrorObject(error) {
     // eslint-disable-next-line @typescript-eslint/no-explicit-any -- This is a type guard function and it uses ?.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "webext-messenger",
-  "version": "0.33.0-2",
+  "version": "0.33.0",
   "description": "Browser Extension component messaging framework",
   "keywords": [],
   "repository": "pixiebrix/webext-messenger",
@@ -29,7 +29,7 @@
     "p-event": "^6.0.1",
     "p-retry": "^6.2.1",
     "serialize-error": "^12.0.0",
-    "type-fest": "^4.36.0",
+    "type-fest": "^4.38.0",
     "webext-detect": "^5.3.2"
   },
   "@parcel/resolver-default": {
@@ -38,7 +38,7 @@
   "devDependencies": {
     "@parcel/config-webextension": "^2.11.0",
     "@sindresorhus/tsconfig": "^7.0.0",
-    "@types/chrome": "^0.0.307",
+    "@types/chrome": "^0.0.313",
     "@types/tape": "^5.8.1",
     "buffer": "^6.0.3",
     "eslint": "^8.57.0",
@@ -51,7 +51,7 @@
     "stream-browserify": "^3.0.0",
     "tape": "^5.9.0",
     "typescript": "^5.8.2",
-    "vitest": "^3.0.7"
+    "vitest": "^3.1.1"
   },
   "targets": {
     "main": false,