@fedify/fedify 2.4.0-pr.936.41 → 2.5.0-dev.2271

package/dist/federation/retry.test.mjs

@@ -1,9 +1,10 @@
 import { Temporal } from "temporal-polyfill";
 import "urlpattern-polyfill";
 globalThis.addEventListener = () => {};
-import { l as AssertionError, t as assertEquals } from "../assert_equals-BV7oLMxm.mjs";
-import { t as assertNotEquals } from "../assert_not_equals-EPJPoxuf.mjs";
-import { t as createExponentialBackoffPolicy } from "../retry-DgFqeS5k.mjs";
+import { t as AssertionError } from "../assertion_error-B-cx0fFA.mjs";
+import { t as assertEquals } from "../assert_equals-Z6IcEbWU.mjs";
+import { t as assertNotEquals } from "../assert_not_equals-CpPK2t89.mjs";
+import { t as createExponentialBackoffPolicy } from "../retry-zX5xw5e0.mjs";
 import { test } from "@fedify/fixture";
 //#region src/federation/retry.test.ts
 test("createExponentialBackoffPolicy()", () => {

package/dist/federation/router.test.mjs

@@ -1,164 +1,14 @@
 import "temporal-polyfill";
 import "urlpattern-polyfill";
 globalThis.addEventListener = () => {};
-import { t as assertEquals } from "../assert_equals-BV7oLMxm.mjs";
-import "../std__assert-B2NnY9bq.mjs";
-import { r as assertFalse } from "../assert_rejects-EgstcQy5.mjs";
-import { t as assertThrows } from "../assert_throws-NiFfqbnH.mjs";
-import { t as assert } from "../assert-CJ1KXcvv.mjs";
-import { getLogger } from "@logtape/logtape";
-import { Router, RouterError, assertPath, isPath } from "@fedify/uri-template";
+import { t as assertEquals } from "../assert_equals-Z6IcEbWU.mjs";
+import { t as assertFalse } from "../assert_false-CPdRnc3E.mjs";
+import { t as assertThrows } from "../assert_throws-BYITOvPO.mjs";
+import "../std__assert-DvMDRPes.mjs";
+import { t as assert } from "../assert-D1Eafmd_.mjs";
+import { n as RouterError$1, t as Router$1 } from "../router-u5O5KCQr.mjs";
+import { RouterError } from "@fedify/uri-template";
 import { test } from "@fedify/fixture";
-//#region src/federation/router.ts
-const logger = getLogger([
-	"fedify",
-	"federation",
-	"router",
-	"deprecated"
-]);
-let deprecationWarned = false;
-function warnDeprecated() {
-	if (deprecationWarned) return;
-	deprecationWarned = true;
-	logger.warn("The `Router` and `RouterError` classes from `@fedify/fedify` are deprecated.  Please use `Router` from `@fedify/uri-template` instead.");
-}
-/**
-* URL router and constructor based on URI Template
-* ([RFC 6570](https://tools.ietf.org/html/rfc6570)).
-*
-* @deprecated Import `Router` from `@fedify/uri-template` instead.  This class
-*             remains only for compatibility with older Fedify code.  The
-*             `@fedify/uri-template` router is the replacement implementation
-*             and should be used directly in new code.
-*/
-var Router$1 = class Router$1 {
-	#router;
-	/**
-	* Create a new {@link Router}.
-	* @param options Options for the router.
-	* @deprecated Use `new Router(options)` from `@fedify/uri-template`
-	*             instead.
-	*/
-	constructor(options) {
-		this.#router = convertRouterError(() => new Router(options));
-	}
-	/**
-	* Whether to ignore trailing slashes when matching paths.
-	* @deprecated Use `Router` from `@fedify/uri-template` instead.  This
-	*             accessor forwards to the underlying `@fedify/uri-template`
-	*             router so that post-construction mutation keeps working as
-	*             in older Fedify code.
-	*/
-	get trailingSlashInsensitive() {
-		return this.#router.trailingSlashInsensitive;
-	}
-	set trailingSlashInsensitive(value) {
-		this.#router.trailingSlashInsensitive = value;
-	}
-	/**
-	* Clones this router.
-	* @deprecated Use `Router` from `@fedify/uri-template` instead.
-	*/
-	clone() {
-		return convertRouterError(() => {
-			const clone = new Router$1();
-			clone.#router = this.#router.clone();
-			return clone;
-		});
-	}
-	/**
-	* Checks if a path name exists in the router.
-	* @param name The name of the path.
-	* @returns `true` if the path name exists, otherwise `false`.
-	* @deprecated Use `Router` from `@fedify/uri-template` instead.
-	*/
-	has(name) {
-		return convertRouterError(() => this.#router.has(name));
-	}
-	/**
-	* Adds a new path rule to the router.
-	* @param template The path pattern.
-	* @param name The name of the path.
-	* @returns The names of the variables in the path pattern.
-	* @deprecated Use `Router` from `@fedify/uri-template` instead.  In this
-	*             compatibility class, `add()` both registers the route and
-	*             returns the variables in the path pattern.  In
-	*             `@fedify/uri-template`, these two responsibilities are split:
-	*             `router.add(template, name)` registers the route and returns
-	*             `void`, while the pure static method
-	*             `Router.variables(template)` returns the variable names.  To
-	*             migrate, call `Router.variables(template)` when variables are
-	*             needed, then call `router.add(template, name)` to register the
-	*             route.
-	*/
-	add(template, name) {
-		return convertRouterError(() => {
-			assertPath(template);
-			this.#router.add(template, name);
-			return Router.variables(template);
-		});
-	}
-	/**
-	* Resolves a path name and values from a URL, if any match.
-	* @param url The URL to resolve.
-	* @returns The name of the path and its values, if any match.  Otherwise,
-	*          `null`.
-	* @deprecated Use `Router` from `@fedify/uri-template` instead.  Unlike the
-	*             stricter `@fedify/uri-template` router, this compatibility
-	*             method keeps the old Fedify 2.x contract of returning `null`
-	*             (rather than throwing) for inputs that are not router paths.
-	*/
-	route(url) {
-		return convertRouterError(() => {
-			if (!isPath(url)) return null;
-			return this.#router.route(url);
-		});
-	}
-	/**
-	* Constructs a URL/path from a path name and values.
-	* @param name The name of the path.
-	* @param values The values to expand the path with.
-	* @returns The URL/path, if the name exists.  Otherwise, `null`.
-	* @deprecated Use `Router` from `@fedify/uri-template` instead.
-	*/
-	build(name, values) {
-		return convertRouterError(() => this.#router.build(name, values));
-	}
-};
-/**
-* An error thrown by the {@link Router}.
-* @deprecated Import `RouterError` from `@fedify/uri-template` instead.
-*/
-var RouterError$1 = class extends RouterError {
-	/**
-	* Treats every `RouterError` from `@fedify/uri-template` as an instance of
-	* this deprecated class.
-	*
-	* @deprecated Import `RouterError` from `@fedify/uri-template` instead.
-	*/
-	static [Symbol.hasInstance](instance) {
-		return instance instanceof RouterError;
-	}
-	/**
-	* Create a new {@link RouterError}.
-	* @param message The error message.
-	* @deprecated Import `RouterError` from `@fedify/uri-template` instead.
-	*/
-	constructor(message) {
-		super(message);
-		warnDeprecated();
-	}
-};
-function convertRouterError(func) {
-	try {
-		warnDeprecated();
-		return func();
-	} catch (error) {
-		if (error instanceof RouterError) throw new RouterError$1(error.message);
-		throw error;
-	}
-}
-//#endregion
 //#region src/federation/router.test.ts
 function setUp(options = {}) {
 	const router = new Router$1(options);
@@ -182,8 +32,8 @@ test("Router.clone()", () => {
 test("Router.add()", () => {
 	const router = new Router$1();
 	assertEquals(router.add("/users", "users"), /* @__PURE__ */ new Set());
-	assertEquals(router.add("/users/{name}", "user"), new Set(["name"]));
-	assertEquals(router.add("/users/{name}/posts/{postId}", "post"), new Set(["name", "postId"]));
+	assertEquals(router.add("/users/{name}", "user"), /* @__PURE__ */ new Set(["name"]));
+	assertEquals(router.add("/users/{name}/posts/{postId}", "post"), /* @__PURE__ */ new Set(["name", "postId"]));
 	assertThrows(() => router.add("foo", "name"), RouterError$1);
 });
 test("Router.route()", () => {

package/dist/federation/send.test.mjs

@@ -1,20 +1,23 @@
 import "temporal-polyfill";
 import "urlpattern-polyfill";
 globalThis.addEventListener = () => {};
-import { t as esm_default } from "../esm-yYQfPhIY.mjs";
-import { t as assertEquals } from "../assert_equals-BV7oLMxm.mjs";
-import "../std__assert-B2NnY9bq.mjs";
-import { n as assertGreaterOrEqual, r as assertFalse, t as assertRejects } from "../assert_rejects-EgstcQy5.mjs";
-import { t as assertInstanceOf } from "../assert_instance_of-Q9KTBnrf.mjs";
-import { t as assertNotEquals } from "../assert_not_equals-EPJPoxuf.mjs";
-import { t as assert } from "../assert-CJ1KXcvv.mjs";
-import { l as verifyRequest } from "../http-CPVMUgsW.mjs";
-import { i as rsaPrivateKey2, n as ed25519PrivateKey, s as rsaPublicKey2, t as ed25519Multikey } from "../keys-Dbyd7bhl.mjs";
-import { t as doesActorOwnKey } from "../owner-Br9YYzn_.mjs";
-import { n as extractInboxes, r as sendActivity, t as SendActivityError } from "../send-CxHKUNzv.mjs";
+import { t as esm_default } from "../esm-BHXXx4Wx.mjs";
+import { t as assertEquals } from "../assert_equals-Z6IcEbWU.mjs";
+import { t as assertFalse } from "../assert_false-CPdRnc3E.mjs";
+import { t as assertGreaterOrEqual } from "../assert_greater_or_equal-BlafOF4-.mjs";
+import { t as assertInstanceOf } from "../assert_instance_of-mrXWWNY-.mjs";
+import { t as assertNotEquals } from "../assert_not_equals-CpPK2t89.mjs";
+import "../std__assert-DvMDRPes.mjs";
+import { t as assertRejects } from "../assert_rejects-CTptdYck.mjs";
+import { t as assert } from "../assert-D1Eafmd_.mjs";
+import { f as verifyRequest } from "../http-siciWkmx.mjs";
+import { i as rsaPrivateKey2, n as ed25519PrivateKey, s as rsaPublicKey2, t as ed25519Multikey } from "../keys-DIJNHW-Y.mjs";
+import { t as doesActorOwnKey } from "../owner-DYgfTFHy.mjs";
+import { n as extractInboxes, r as sendActivity, t as SendActivityError } from "../send-C4dDpAxm.mjs";
 import { Activity, Application, Endpoints, Group, Person, Service } from "@fedify/vocab";
+import { FetchError, UrlError } from "@fedify/vocab-runtime";
 import { createTestMeterProvider, createTestTracerProvider, mockDocumentLoader, test } from "@fedify/fixture";
-import { FetchError } from "@fedify/vocab-runtime";
+import dns from "node:dns/promises";
 import { AggregationTemporality, InMemoryMetricExporter, MeterProvider, PeriodicExportingMetricReader } from "@opentelemetry/sdk-metrics";
 //#region src/federation/send.test.ts
 test("extractInboxes()", () => {
@@ -42,19 +45,19 @@ test("extractInboxes()", () => {
 	let inboxes = extractInboxes({ recipients });
 	assertEquals(inboxes, {
 		"https://example.com/alice/inbox": {
-			actorIds: new Set(["https://example.com/alice"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.com/alice"]),
 			sharedInbox: false
 		},
 		"https://example.com/app/inbox": {
-			actorIds: new Set(["https://example.com/app"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.com/app"]),
 			sharedInbox: false
 		},
 		"https://example.org/group/inbox": {
-			actorIds: new Set(["https://example.org/group"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.org/group"]),
 			sharedInbox: false
 		},
 		"https://example.net/service/inbox": {
-			actorIds: new Set(["https://example.net/service"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.net/service"]),
 			sharedInbox: false
 		}
 	});
@@ -64,15 +67,15 @@ test("extractInboxes()", () => {
 	});
 	assertEquals(inboxes, {
 		"https://example.com/inbox": {
-			actorIds: new Set(["https://example.com/alice", "https://example.com/app"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.com/alice", "https://example.com/app"]),
 			sharedInbox: true
 		},
 		"https://example.org/group/inbox": {
-			actorIds: new Set(["https://example.org/group"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.org/group"]),
 			sharedInbox: false
 		},
 		"https://example.net/inbox": {
-			actorIds: new Set(["https://example.net/service"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.net/service"]),
 			sharedInbox: true
 		}
 	});
@@ -82,19 +85,19 @@ test("extractInboxes()", () => {
 	});
 	assertEquals(inboxes, {
 		"https://example.com/alice/inbox": {
-			actorIds: new Set(["https://example.com/alice"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.com/alice"]),
 			sharedInbox: false
 		},
 		"https://example.com/app/inbox": {
-			actorIds: new Set(["https://example.com/app"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.com/app"]),
 			sharedInbox: false
 		},
 		"https://example.org/group/inbox": {
-			actorIds: new Set(["https://example.org/group"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.org/group"]),
 			sharedInbox: false
 		},
 		"https://example.net/service/inbox": {
-			actorIds: new Set(["https://example.net/service"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.net/service"]),
 			sharedInbox: false
 		}
 	});
@@ -104,11 +107,11 @@ test("extractInboxes()", () => {
 	});
 	assertEquals(inboxes, {
 		"https://example.org/group/inbox": {
-			actorIds: new Set(["https://example.org/group"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.org/group"]),
 			sharedInbox: false
 		},
 		"https://example.net/service/inbox": {
-			actorIds: new Set(["https://example.net/service"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.net/service"]),
 			sharedInbox: false
 		}
 	});
@@ -119,11 +122,11 @@ test("extractInboxes()", () => {
 	});
 	assertEquals(inboxes, {
 		"https://example.org/group/inbox": {
-			actorIds: new Set(["https://example.org/group"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.org/group"]),
 			sharedInbox: false
 		},
 		"https://example.net/inbox": {
-			actorIds: new Set(["https://example.net/service"]),
+			actorIds: /* @__PURE__ */ new Set(["https://example.net/service"]),
 			sharedInbox: true
 		}
 	});
@@ -463,6 +466,7 @@ test("sendActivity() records OpenTelemetry delivery metrics", async (t) => {
 			activityId: "https://example.com/activity",
 			activityType: "https://www.w3.org/ns/activitystreams#Create",
 			keys: [],
+			allowPrivateAddress: true,
 			inbox: new URL("https://metrics.example:8443/inbox/path?x=1"),
 			meterProvider
 		});
@@ -500,6 +504,7 @@ test("sendActivity() records OpenTelemetry delivery metrics", async (t) => {
 			activityId: "https://example.com/follow",
 			activityType: "https://www.w3.org/ns/activitystreams#Follow",
 			keys: [],
+			allowPrivateAddress: true,
 			inbox: new URL("https://metrics.example/inbox"),
 			meterProvider
 		}), SendActivityError);
@@ -537,6 +542,7 @@ test("sendActivity() exports delivery metrics through OpenTelemetry SDK", async
 			activityId: "https://example.com/activity",
 			activityType: "https://www.w3.org/ns/activitystreams#Create",
 			keys: [],
+			allowPrivateAddress: true,
 			inbox: new URL("https://sdk-metrics.example/inbox"),
 			meterProvider
 		});
@@ -553,5 +559,217 @@ test("sendActivity() exports delivery metrics through OpenTelemetry SDK", async
 		}
 	}
 });
+for (const signed of [false, true]) {
+	const keys = [{
+		privateKey: signed ? rsaPrivateKey2 : ed25519PrivateKey,
+		keyId: signed ? rsaPublicKey2.id : ed25519Multikey.id
+	}];
+	test(`sendActivity() validates destinations (signed: ${signed})`, async (t) => {
+		const activity = {
+			type: "Create",
+			id: "https://example.com/activity"
+		};
+		const publicInbox = "https://8.8.8.8/inbox";
+		const privateInbox = "http://127.0.0.1/inbox";
+		for (const target of [
+			privateInbox,
+			"http://169.254.169.254/",
+			"http://[::1]/"
+		]) await t.step(`rejects ${target} before fetching`, async () => {
+			esm_default.mockGlobal().catch(202);
+			try {
+				await assertRejects(() => sendActivity({
+					activity,
+					keys,
+					inbox: new URL(target)
+				}), UrlError);
+				assertEquals(esm_default.callHistory.calls().length, 0);
+			} finally {
+				esm_default.hardReset();
+			}
+		});
+		for (const status of [
+			301,
+			302,
+			303,
+			307,
+			308
+		]) for (const allowPrivateAddress of [false, true]) await t.step(`${status} private redirect, opt-in: ${allowPrivateAddress}`, async () => {
+			esm_default.mockGlobal();
+			esm_default.route(publicInbox, {
+				status,
+				headers: { Location: privateInbox }
+			});
+			esm_default.route(privateInbox, 202);
+			try {
+				const send = () => sendActivity({
+					activity,
+					keys,
+					inbox: new URL(publicInbox),
+					allowPrivateAddress
+				});
+				if (allowPrivateAddress) {
+					await send();
+					const calls = esm_default.callHistory.calls(privateInbox);
+					assertEquals(calls.length, 1);
+					const request = calls[0].request;
+					const preservesBody = signed || status === 307 || status === 308;
+					assertEquals(request.method, preservesBody ? "POST" : "GET");
+					assertEquals(await request.clone().text(), preservesBody ? JSON.stringify(activity) : "");
+				} else {
+					await assertRejects(send, UrlError);
+					assertEquals(esm_default.callHistory.calls(privateInbox).length, 0);
+				}
+			} finally {
+				esm_default.hardReset();
+			}
+		});
+		await t.step("rejects a private destination after a public redirect", async () => {
+			esm_default.mockGlobal().route(publicInbox, {
+				status: 307,
+				headers: { Location: "/next" }
+			}).route("https://8.8.8.8/next", {
+				status: 308,
+				headers: { Location: privateInbox }
+			}).route(privateInbox, 202);
+			try {
+				await assertRejects(() => sendActivity({
+					activity,
+					keys,
+					inbox: new URL(publicInbox)
+				}), UrlError);
+				assertEquals(esm_default.callHistory.calls(privateInbox).length, 0);
+			} finally {
+				esm_default.hardReset();
+			}
+		});
+		await t.step("limits redirect loops", async () => {
+			esm_default.mockGlobal().route(publicInbox, {
+				status: 307,
+				headers: { Location: publicInbox }
+			});
+			try {
+				await assertRejects(() => sendActivity({
+					activity,
+					keys,
+					inbox: new URL(publicInbox)
+				}));
+				assert(esm_default.callHistory.calls().length <= 21);
+			} finally {
+				esm_default.hardReset();
+			}
+		});
+		await t.step("allows a direct private inbox with opt-in", async () => {
+			esm_default.mockGlobal().route(privateInbox, 202);
+			try {
+				await sendActivity({
+					activity,
+					keys,
+					inbox: new URL(privateInbox),
+					allowPrivateAddress: true
+				});
+				assertEquals(esm_default.callHistory.calls(privateInbox).length, 1);
+			} finally {
+				esm_default.hardReset();
+			}
+		});
+		await t.step("allows relative public redirects", async () => {
+			esm_default.mockGlobal().route(publicInbox, {
+				status: 307,
+				headers: { Location: "/next" }
+			}).route("https://8.8.8.8/next", 202);
+			try {
+				await sendActivity({
+					activity,
+					keys,
+					inbox: new URL(publicInbox)
+				});
+				assertEquals(esm_default.callHistory.calls("https://8.8.8.8/next").length, 1);
+			} finally {
+				esm_default.hardReset();
+			}
+		});
+	});
+}
+for (const signed of [false, true]) test({
+	name: `sendActivity() classifies DNS failures (signed: ${signed})`,
+	ignore: "Deno" in globalThis && (await Deno.permissions.query({ name: "net" })).state !== "granted",
+	async fn(t) {
+		const activity = {
+			type: "Create",
+			id: "https://example.com/activity"
+		};
+		const destination = "https://delivery.invalid/inbox";
+		const publicInbox = "https://8.8.8.8/inbox";
+		const keys = signed ? [{
+			privateKey: rsaPrivateKey2,
+			keyId: rsaPublicKey2.id
+		}] : [];
+		for (const result of [
+			"throws",
+			"empty",
+			"cname",
+			"private"
+		]) for (const redirected of [false, true]) await t.step(`${result}, redirected: ${redirected}`, async () => {
+			const [meterProvider, recorder] = createTestMeterProvider();
+			const originalLookup = dns.lookup;
+			const resolverError = /* @__PURE__ */ new Error("Resolver unavailable");
+			const lookups = [];
+			dns.lookup = ((hostname, options) => {
+				lookups.push(hostname);
+				assertEquals(hostname, "delivery.invalid");
+				assertEquals(options, { all: true });
+				if (result === "throws") return Promise.reject(resolverError);
+				return Promise.resolve(result === "empty" ? [] : [{
+					address: result === "private" ? "127.0.0.1" : "alias.invalid.",
+					family: 4
+				}]);
+			});
+			try {
+				esm_default.mockGlobal().catch(202);
+				if (redirected) esm_default.route(publicInbox, {
+					status: 307,
+					headers: { Location: destination }
+				});
+				const send = () => sendActivity({
+					activity,
+					activityType: "https://www.w3.org/ns/activitystreams#Create",
+					meterProvider,
+					keys,
+					inbox: new URL(redirected ? publicInbox : destination)
+				});
+				if (result === "private") assertEquals((await assertRejects(send, UrlError)).reason, "disallowed");
+				else {
+					const error = await assertRejects(send, FetchError);
+					assertEquals(error.url.href, destination);
+					assertInstanceOf(error.cause, UrlError);
+					assertEquals(error.cause.reason, "dns");
+					assertEquals(error.cause.cause, result === "throws" ? resolverError : void 0);
+				}
+				const expectedCount = result === "private" && !redirected ? 0 : 1;
+				const sent = recorder.getMeasurements("activitypub.delivery.sent");
+				const durations = recorder.getMeasurements("activitypub.delivery.duration");
+				assertEquals(sent.length, expectedCount);
+				assertEquals(durations.length, expectedCount);
+				if (expectedCount === 1) {
+					assertEquals(sent[0].value, 1);
+					assertEquals(sent[0].attributes, {
+						"activitypub.remote.host": redirected ? "8.8.8.8" : "delivery.invalid",
+						"activitypub.activity.type": "https://www.w3.org/ns/activitystreams#Create",
+						"activitypub.delivery.success": false
+					});
+					assertEquals(durations[0].attributes, sent[0].attributes);
+					assertGreaterOrEqual(durations[0].value, 0);
+				}
+				assertEquals(lookups, ["delivery.invalid"]);
+				assertEquals(esm_default.callHistory.calls(destination).length, 0);
+				assertEquals(esm_default.callHistory.calls().length, redirected ? 1 : 0);
+			} finally {
+				dns.lookup = originalLookup;
+				esm_default.hardReset();
+			}
+		});
+	}
+});
 //#endregion
 export {};

package/dist/federation/tasks/codec.test.mjs

@@ -1,7 +1,7 @@
 import { Temporal } from "temporal-polyfill";
 import "urlpattern-polyfill";
 globalThis.addEventListener = () => {};
-import { t as TaskCodec } from "../../codec-BBga-FWI.mjs";
+import { t as TaskCodec } from "../../codec-C6rTrpIp.mjs";
 import { Create, Link, Note, Person } from "@fedify/vocab";
 import { mockDocumentLoader, test } from "@fedify/fixture";
 import { deepStrictEqual, ok, rejects, strictEqual } from "node:assert/strict";
@@ -45,8 +45,8 @@ test("TaskCodec.serialize() / deserialize()", async (t) => {
 			big: 1234567890123456789n,
 			url: new URL("https://example.com/some/path"),
 			list: [person, link],
-			map: new Map([["create", create], ["n", 42]]),
-			set: new Set([
+			map: /* @__PURE__ */ new Map([["create", create], ["n", 42]]),
+			set: /* @__PURE__ */ new Set([
 				1,
 				2,
 				3
@@ -72,7 +72,7 @@ test("TaskCodec.serialize() / deserialize()", async (t) => {
 		const map = decoded.map;
 		ok(map.get("create") instanceof Create);
 		strictEqual(map.get("n"), 42);
-		deepStrictEqual(decoded.set, new Set([
+		deepStrictEqual(decoded.set, /* @__PURE__ */ new Set([
 			1,
 			2,
 			3

package/dist/federation/tasks/enqueue.test.mjs

@@ -1,10 +1,11 @@
 import { Temporal } from "temporal-polyfill";
 import "urlpattern-polyfill";
 globalThis.addEventListener = () => {};
-import { t as MemoryKvStore } from "../../kv-CXBAok3k.mjs";
-import { a as numberSchema, i as makeSchema, n as baseOptions, o as stringSchema, t as MockQueue } from "../../testing-CO7ftO8A.mjs";
-import { o as createFederation } from "../../middleware-CvwTUgnP.mjs";
-import { n as ParallelMessageQueue } from "../../mq-llHKivST.mjs";
+import { t as MemoryKvStore } from "../../kv-DCHkaJ9-.mjs";
+import { a as numberSchema, i as makeSchema, n as baseOptions, o as stringSchema, t as MockQueue } from "../../tasks-IG3pHfvZ.mjs";
+import "../../testing-t8xDeJZ5.mjs";
+import { o as createFederation } from "../../middleware-D7_LO-Vs.mjs";
+import { n as ParallelMessageQueue } from "../../mq-Dm1Wmtm-.mjs";
 import { configure, reset } from "@logtape/logtape";
 import { delay } from "es-toolkit";
 import { createTestMeterProvider, test } from "@fedify/fixture";
@@ -760,6 +761,43 @@ test("a multi-item batch dedup without enqueueMany is rejected on the cas path",
 	await ctx.enqueueTaskMany(task, ["single"], { deduplicationKey: "single" });
 	strictEqual(queue.enqueued.length, 1);
 });
+test("a non-atomic enqueueMany is rejected for batch deduplication", async () => {
+	class NonAtomicBatchQueue extends MockQueue {
+		atomicEnqueueMany = false;
+		constructor() {
+			super({ supportsEnqueueMany: true });
+		}
+	}
+	const queue = new NonAtomicBatchQueue();
+	const kv = new MemoryKvStore();
+	const federation = createFederation({
+		...baseOptions,
+		kv,
+		queue: { task: queue }
+	});
+	const task = federation.defineTask("cas-non-atomic-batch", {
+		schema: stringSchema,
+		handler: () => {}
+	});
+	const ctx = federation.createContext(new URL("https://example.com/"), void 0);
+	await rejects(() => ctx.enqueueTaskMany(task, [
+		"a",
+		"b",
+		"c"
+	], { deduplicationKey: "batch" }), {
+		name: "TypeError",
+		message: /atomically/
+	});
+	strictEqual(queue.enqueued.length, 0);
+	strictEqual(queue.enqueuedMany.length, 0);
+	deepStrictEqual(await collectKeys(kv, TASK_DEDUP_PREFIX), []);
+	await ctx.enqueueTaskMany(task, [
+		"a",
+		"b",
+		"c"
+	]);
+	strictEqual(queue.enqueuedMany.length, 1);
+});
 test("a failed rollback is swallowed; the original enqueue error reaches the caller", async () => {
 	class ClearFailingKvStore {
 		inner = new MemoryKvStore();

package/dist/federation/tasks/tasks.test.mjs

@@ -1,12 +1,13 @@
 import { Temporal } from "temporal-polyfill";
 import "urlpattern-polyfill";
 globalThis.addEventListener = () => {};
-import { t as MemoryKvStore } from "../../kv-CXBAok3k.mjs";
-import { a as numberSchema, i as makeSchema, n as baseOptions, o as stringSchema, t as MockQueue } from "../../testing-CO7ftO8A.mjs";
-import { o as createFederation } from "../../middleware-CvwTUgnP.mjs";
-import { r as createFederationBuilder } from "../../builder-DonuxRl_.mjs";
-import { t as TaskCodec } from "../../codec-BBga-FWI.mjs";
-import { t as InProcessMessageQueue } from "../../mq-llHKivST.mjs";
+import { t as MemoryKvStore } from "../../kv-DCHkaJ9-.mjs";
+import { a as numberSchema, i as makeSchema, n as baseOptions, o as stringSchema, t as MockQueue } from "../../tasks-IG3pHfvZ.mjs";
+import "../../testing-t8xDeJZ5.mjs";
+import { o as createFederation } from "../../middleware-D7_LO-Vs.mjs";
+import { r as createFederationBuilder } from "../../builder-Bda6LK7A.mjs";
+import { t as TaskCodec } from "../../codec-C6rTrpIp.mjs";
+import { t as InProcessMessageQueue } from "../../mq-Dm1Wmtm-.mjs";
 import { W3CTraceContextPropagator } from "@opentelemetry/core";
 import { Note } from "@fedify/vocab";
 import { SpanStatusCode, propagation } from "@opentelemetry/api";