webext-messenger 0.22.0-4 → 0.22.0

package/distribution/handlers.d.ts

@@ -1,5 +1,5 @@
 import { __getTabData } from "./thisTarget.js";
-import { Method } from "./types.js";
+import { type Method } from "./types.js";
 declare global {
     interface MessengerMethods {
         __getTabData: typeof __getTabData;

package/distribution/index.d.ts

@@ -1,4 +1,4 @@
 export * from "./receiver.js";
 export * from "./sender.js";
 export * from "./types.js";
-export { getThisTarget } from "./thisTarget.js";
+export { getThisFrame, getTopLevelFrame } from "./thisTarget.js";

package/distribution/index.js

@@ -2,6 +2,6 @@
 export * from "./receiver.js";
 export * from "./sender.js";
 export * from "./types.js";
-export { getThisTarget } from "./thisTarget.js";
+export { getThisFrame, getTopLevelFrame } from "./thisTarget.js";
 import { initPrivateApi } from "./thisTarget.js";
 initPrivateApi();

package/distribution/receiver.d.ts

@@ -1,3 +1,3 @@
-import { Message } from "./types.js";
+import { type Message } from "./types.js";
 export declare function isMessengerMessage(message: unknown): message is Message;
 export declare function registerMethods(methods: Partial<MessengerMethods>): void;

package/distribution/sender.d.ts

@@ -1,5 +1,5 @@
-import { PublicMethod, PublicMethodWithTarget, Options, Target, PageTarget } from "./types.js";
-import { SetReturnType } from "type-fest";
+import { type PublicMethod, type PublicMethodWithTarget, type Options, type Target, type PageTarget } from "./types.js";
+import { type SetReturnType } from "type-fest";
 export declare const errorTargetClosedEarly = "The target was closed before receiving a response";
 export declare const errorTabDoesntExist = "The tab doesn't exist";
 declare function messenger<Type extends keyof MessengerMethods, Method extends MessengerMethods[Type]>(type: Type, options: {

package/distribution/shared.d.ts

@@ -1,5 +1,5 @@
-import { JsonObject } from "type-fest";
-declare type ErrorObject = {
+import { type JsonObject } from "type-fest";
+type ErrorObject = {
     name?: string;
     stack?: string;
     message?: string;

package/distribution/thisTarget.d.ts

@@ -1,5 +1,6 @@
-import { AnyTarget, KnownTarget, Message, MessengerMeta, Sender } from "./types.js";
+import { type AnyTarget, type TopLevelFrame, type Message, type MessengerMeta, type Sender, type FrameTarget } from "./types.js";
 export declare function getActionForMessage(from: Sender, message: Message): "respond" | "forward" | "ignore";
 export declare function __getTabData(this: MessengerMeta): AnyTarget;
-export declare function getThisTarget(): Promise<KnownTarget>;
+export declare function getThisFrame(): Promise<FrameTarget>;
+export declare function getTopLevelFrame(): Promise<TopLevelFrame>;
 export declare function initPrivateApi(): void;

package/distribution/thisTarget.js

@@ -29,16 +29,17 @@ const thisTarget = isBackground()
     ? { page: "background" }
     : {
         get page() {
-            return location.pathname + location.search;
+            // Extension pages have relative URLs to simplify comparison
+            const origin = location.protocol.startsWith("http")
+                ? location.origin
+                : "";
+            // Don't use the hash
+            return origin + location.pathname + location.search;
         },
     };
 let tabDataStatus = 
 // The background page doesn't have a tab
-isBackground() ||
-    // Content scripts don't use named targets yet
-    isContentScript()
-    ? "not-needed"
-    : "needed";
+isBackground() ? "not-needed" : "needed";
 function compareTargets(to, thisTarget) {
     for (const [key, value] of Object.entries(to)) {
         if (thisTarget[key] === value) {
@@ -112,9 +113,21 @@ const storeTabData = once(async () => {
 export function __getTabData() {
     return { tabId: this.trace[0]?.tab?.id, frameId: this.trace[0]?.frameId };
 }
-export async function getThisTarget() {
+export async function getThisFrame() {
     await storeTabData(); // It should already have been called by we still need to await it
-    return thisTarget;
+    const { tabId, frameId } = thisTarget;
+    if (typeof tabId !== "number" || typeof frameId !== "number") {
+        throw new TypeError("This target is not in a frame");
+    }
+    // Rebuild object to return exactly these two properties and nothing more
+    return { tabId, frameId };
+}
+export async function getTopLevelFrame() {
+    const { tabId } = await getThisFrame();
+    return {
+        tabId,
+        frameId: 0,
+    };
 }
 export function initPrivateApi() {
     if (isExtensionContext()) {

package/distribution/types.d.ts

@@ -1,30 +1,35 @@
-import { Runtime } from "webextension-polyfill";
-import { Asyncify, ValueOf } from "type-fest";
-import { ErrorObject } from "serialize-error";
+import { type Runtime } from "webextension-polyfill";
+import { type Asyncify, type ValueOf } from "type-fest";
+import { type ErrorObject } from "serialize-error";
+/**
+ * @file Target types are a bit overlapping. That's because some are "request" targets
+ * and some are "known" targets. The difference is that you could "request" `{tabId: 1}`, but you "know" that a specific target is exactly `{tabId: 1, frameId: 5}`
+ * TODO: Cleanup, clarify, deduplicate Target types
+ */
 declare global {
     interface MessengerMethods {
         _: Method;
     }
 }
-declare type WithTarget<Method> = Method extends (...args: infer PreviousArguments) => infer TReturnValue ? (target: Target | PageTarget, ...args: PreviousArguments) => TReturnValue : never;
-declare type ActuallyOmitThisParameter<T> = T extends (...args: infer A) => infer R ? (...args: A) => R : T;
+type WithTarget<Method> = Method extends (...args: infer PreviousArguments) => infer TReturnValue ? (target: Target | PageTarget, ...args: PreviousArguments) => TReturnValue : never;
+type ActuallyOmitThisParameter<T> = T extends (...args: infer A) => infer R ? (...args: A) => R : T;
 /** Removes the `this` type and ensure it's always Promised */
-export declare type PublicMethod<Method extends ValueOf<MessengerMethods>> = Asyncify<ActuallyOmitThisParameter<Method>>;
-export declare type PublicMethodWithTarget<Method extends ValueOf<MessengerMethods>> = WithTarget<PublicMethod<Method>>;
+export type PublicMethod<Method extends ValueOf<MessengerMethods>> = Asyncify<ActuallyOmitThisParameter<Method>>;
+export type PublicMethodWithTarget<Method extends ValueOf<MessengerMethods>> = WithTarget<PublicMethod<Method>>;
 export interface MessengerMeta {
     trace: Sender[];
 }
-declare type RawMessengerResponse = {
+type RawMessengerResponse = {
     value: unknown;
 } | {
     error: ErrorObject;
 };
-export declare type MessengerResponse = RawMessengerResponse & {
+export type MessengerResponse = RawMessengerResponse & {
     /** Guarantees that the message was handled by this library */
     __webextMessenger: true;
 };
-declare type Arguments = any[];
-export declare type Method = (this: MessengerMeta, ...args: Arguments) => Promise<unknown>;
+type Arguments = any[];
+export type Method = (this: MessengerMeta, ...args: Arguments) => Promise<unknown>;
 export interface Options {
     /**
      * "Notifications" won't await the response, return values, attempt retries, nor throw errors
@@ -33,17 +38,17 @@ export interface Options {
     isNotification?: boolean;
     trace?: Sender[];
 }
-export declare type Message<LocalArguments extends Arguments = Arguments> = {
+export type Message<LocalArguments extends Arguments = Arguments> = {
     type: keyof MessengerMethods;
     args: LocalArguments;
     target: Target | PageTarget;
     /** If the message is being sent to an intermediary receiver, also set the options */
     options?: Options;
 };
-export declare type Sender = Runtime.MessageSender & {
+export type Sender = Runtime.MessageSender & {
     origin?: string;
 };
-export declare type MessengerMessage = Message & {
+export type MessengerMessage = Message & {
     /** Guarantees that a message is meant to be handled by this library */
     __webextMessenger: true;
 };
@@ -52,6 +57,14 @@ export interface AnyTarget {
     frameId?: number;
     page?: string;
 }
+export interface TopLevelFrame {
+    tabId: number;
+    frameId: 0;
+}
+export interface FrameTarget {
+    tabId: number;
+    frameId: number;
+}
 export interface KnownTarget {
     tabId?: number;
     frameId?: number;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "webext-messenger",
-  "version": "0.22.0-4",
+  "version": "0.22.0",
   "description": "Browser Extension component messaging framework",
   "keywords": [],
   "repository": "pixiebrix/webext-messenger",
@@ -21,25 +21,19 @@
   "dependencies": {
     "p-retry": "^5.1.1",
     "serialize-error": "^11.0.0",
-    "type-fest": "^2.13.0",
+    "type-fest": "^3.2.0",
     "webext-detect-page": "^4.0.1",
     "webext-tools": "^1.1.0"
   },
   "devDependencies": {
-    "@parcel/config-webextension": "^2.6.0",
+    "@parcel/config-webextension": "^2.8.0",
     "@sindresorhus/tsconfig": "^3.0.1",
-    "@types/chrome": "^0.0.188",
+    "@types/chrome": "^0.0.203",
     "@types/tape": "^4.13.2",
     "@types/webextension-polyfill": "^0.9.0",
-    "@typescript-eslint/eslint-plugin": "^5.27.0",
-    "@typescript-eslint/parser": "^5.27.0",
     "buffer": "^6.0.3",
-    "eslint": "^8.17.0",
-    "eslint-config-prettier": "^8.5.0",
-    "eslint-config-xo": "^0.41.0",
-    "eslint-config-xo-typescript": "^0.51.1",
-    "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-unicorn": "^42.0.0",
+    "eslint": "^8.28.0",
+    "eslint-config-pixiebrix": "^0.20.0",
     "events": "^3.3.0",
     "npm-run-all": "^4.1.5",
     "parcel": "^2.6.0",
@@ -47,9 +41,9 @@
     "process": "^0.11.10",
     "stream-browserify": "^3.0.0",
     "tape": "^5.5.3",
-    "typescript": "^4.7.3",
+    "typescript": "^4.9.3",
     "webext-content-scripts": "^1.0.2",
-    "webextension-polyfill": "^0.9.0"
+    "webextension-polyfill": "^0.10.0"
   },
   "alias": {
     "./this-stuff-is-just-for-local-parcel-tests": "https://github.com/parcel-bundler/parcel/issues/4936",