@oxy.so/federation 2.4.2 → 2.5.1

package/dist/cjs/actorObject.js

@@ -22,6 +22,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = exports.AP_ACTOR_TYPES = void 0;
 exports.isApActorType = isApActorType;
 exports.localActorTypeForAccountKind = localActorTypeForAccountKind;
+exports.accountKindForActor = accountKindForActor;
 exports.normalizeAlsoKnownAs = normalizeAlsoKnownAs;
 exports.createLocalActorBuilder = createLocalActorBuilder;
 const contracts_1 = require("@oxy.so/contracts");
@@ -46,7 +47,10 @@ function isApActorType(value) {
     return typeof value === 'string' && exports.AP_ACTOR_TYPES.includes(value);
 }
 /**
- * Oxy account kind → the AS2 actor type the fediverse is told about it.
+ * Oxy account kind → the AS2 actor type the fediverse is told about it — the
+ * ONE table both directions read: what Oxy announces for its own accounts here,
+ * and, through {@link accountKindForActor}, what it records a remote actor as.
+ * Changing how a kind federates is an edit to this table and nowhere else.
  *
  * `satisfies Record<AccountKind, LocalActorType>` is the load-bearing part: a
  * kind added to `@oxy.so/contracts` fails THIS build rather than silently
@@ -78,6 +82,8 @@ function isApActorType(value) {
  *    accepted by all four implementations' whitelists and compared in none of
  *    them — neither `bot?` nor `group?` in Mastodon, `bot_account = false` in
  *    Lemmy, `isBot` false in Misskey, an ordinary account in PeerTube.
+ *  - **`community` → `Group`.** A forum its members post into and that
+ *    re-broadcasts them: exactly what Lemmy, Mbin and Friendica mean by `Group`.
  *
  * What this does NOT do: it does not stop a remote instance offering a reply box
  * under a channel's post. NO actor type gates that in any of the four — Mastodon
@@ -92,6 +98,7 @@ exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = {
     project: 'Organization',
     bot: 'Service',
     channel: 'Organization',
+    community: 'Group',
 };
 /**
  * The AS2 actor type for an Oxy account kind, defaulting to `Person`.
@@ -108,6 +115,49 @@ exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = {
 function localActorTypeForAccountKind(kind) {
     return (0, contracts_1.isAccountKind)(kind) ? exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND[kind] : 'Person';
 }
+/**
+ * The inverse of {@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}, derived from it. Three
+ * kinds share `Organization`, so a type reads back as the FIRST kind the table
+ * lists for it (`Organization` → `organization`); nothing on the wire tells a
+ * remote project from a remote organization.
+ */
+const ACCOUNT_KIND_BY_LOCAL_ACTOR_TYPE = (() => {
+    const inverse = new Map();
+    for (const [kind, type] of Object.entries(exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND)) {
+        if (!inverse.has(type))
+            inverse.set(type, kind);
+    }
+    return inverse;
+})();
+/**
+ * AS2 actor types Oxy never announces but receives, and the announced type each
+ * is read as. `Application` is a non-human software actor — what `Service`
+ * already says, and half of Mastodon's own `bot?`.
+ */
+const INBOUND_ACTOR_TYPE_ALIASES = new Map([
+    ['Application', 'Service'],
+]);
+/**
+ * The kind a `Group` that only its moderators may post into is recorded as.
+ * Such a group forwards nobody: it PUBLISHES (a PeerTube video channel, a
+ * WordPress multi-author blog), which is what an Oxy channel is. The one
+ * inbound signal the table above has no column for.
+ */
+const MODERATED_GROUP_KIND = 'channel';
+/**
+ * The Oxy account kind a REMOTE actor is recorded as, read from the same table
+ * the local announcement uses ({@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}). A type
+ * the table does not know (absent, misspelt, an extension type) is `personal`,
+ * the same default {@link localActorTypeForAccountKind} announces.
+ */
+function accountKindForActor(actor) {
+    if (typeof actor.type !== 'string')
+        return 'personal';
+    const type = INBOUND_ACTOR_TYPE_ALIASES.get(actor.type) ?? actor.type;
+    if (type === exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND.community && actor.postingRestrictedToMods === true)
+        return MODERATED_GROUP_KIND;
+    return ACCOUNT_KIND_BY_LOCAL_ACTOR_TYPE.get(type) ?? 'personal';
+}
 /** Map common image extensions to a MIME type for an actor image `mediaType`. */
 const IMAGE_MEDIA_TYPE_BY_EXT = {
     png: 'image/png',

package/dist/cjs/index.js

@@ -22,7 +22,7 @@
  * knowledge of any app's post shape.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = exports.AP_ACTOR_TYPES = exports.isApActorType = exports.localActorTypeForAccountKind = exports.normalizeAlsoKnownAs = exports.createLocalActorBuilder = exports.createDomainPolicy = exports.extractActorUriFromActivityId = exports.isSameFederationHost = exports.canonicalFederationHost = exports.AP_CONTEXT = exports.readProxyDeclarations = exports.upstreamHandleFromProxyOf = exports.upstreamHandleFromPreferredUsername = exports.upstreamHandleFromAutomatedActor = exports.upstreamHandleFromAlsoKnownAs = exports.upstreamHandleFromProfileField = exports.federatedUsernameFromUpstreamUrl = exports.parseUpstreamProfileUrl = exports.upstreamProfileUrl = exports.stripBridgeBoilerplate = exports.createBridgeRelabeller = exports.blueskyUsernameFromHandle = exports.BSKY_NETWORK_DOMAIN = exports.FEDERATION_NETWORKS = exports.INSTANCE_ACTOR_USERNAME = exports.normalizeActorUsername = exports.createUrlBuilders = exports.DEFAULT_SIGNED_CONTENT_TYPE = exports.HTTP_SIGNATURE_ALGORITHM = exports.verifyHttpSignature = exports.signRequest = void 0;
+exports.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = exports.AP_ACTOR_TYPES = exports.isApActorType = exports.accountKindForActor = exports.localActorTypeForAccountKind = exports.normalizeAlsoKnownAs = exports.createLocalActorBuilder = exports.createDomainPolicy = exports.extractActorUriFromActivityId = exports.isSameFederationHost = exports.canonicalFederationHost = exports.AP_CONTEXT = exports.readProxyDeclarations = exports.upstreamHandleFromProxyOf = exports.upstreamHandleFromPreferredUsername = exports.upstreamHandleFromAutomatedActor = exports.upstreamHandleFromAlsoKnownAs = exports.upstreamHandleFromProfileField = exports.federatedUsernameFromUpstreamUrl = exports.parseUpstreamProfileUrl = exports.upstreamProfileUrl = exports.stripBridgeBoilerplate = exports.createBridgeRelabeller = exports.blueskyUsernameFromHandle = exports.BSKY_NETWORK_DOMAIN = exports.FEDERATION_NETWORKS = exports.INSTANCE_ACTOR_USERNAME = exports.normalizeActorUsername = exports.createUrlBuilders = exports.DEFAULT_SIGNED_CONTENT_TYPE = exports.HTTP_SIGNATURE_ALGORITHM = exports.verifyHttpSignature = exports.signRequest = void 0;
 /**
  * HTTP Signatures (draft-cavage) — the pure sign/verify crypto every Oxy app's
  * ActivityPub federation shares. Private-key custody is injected; the key never
@@ -95,6 +95,7 @@ var actorObject_1 = require("./actorObject");
 Object.defineProperty(exports, "createLocalActorBuilder", { enumerable: true, get: function () { return actorObject_1.createLocalActorBuilder; } });
 Object.defineProperty(exports, "normalizeAlsoKnownAs", { enumerable: true, get: function () { return actorObject_1.normalizeAlsoKnownAs; } });
 Object.defineProperty(exports, "localActorTypeForAccountKind", { enumerable: true, get: function () { return actorObject_1.localActorTypeForAccountKind; } });
+Object.defineProperty(exports, "accountKindForActor", { enumerable: true, get: function () { return actorObject_1.accountKindForActor; } });
 Object.defineProperty(exports, "isApActorType", { enumerable: true, get: function () { return actorObject_1.isApActorType; } });
 Object.defineProperty(exports, "AP_ACTOR_TYPES", { enumerable: true, get: function () { return actorObject_1.AP_ACTOR_TYPES; } });
 Object.defineProperty(exports, "LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND", { enumerable: true, get: function () { return actorObject_1.LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND; } });

package/dist/cjs/node/actorResolver.js

@@ -24,6 +24,7 @@ exports.ActorResolver = void 0;
 exports.createActorResolver = createActorResolver;
 const apUri_1 = require("../apUri");
 const networkIdentity_1 = require("../networkIdentity");
+const boundedBody_1 = require("./boundedBody");
 /**
  * Minimum interval between background actor refreshes for the same actor.
  * Prevents refresh storms when a profile is viewed repeatedly in a short window.
@@ -43,38 +44,26 @@ const COLLECTION_BODY_MAX_BYTES = 64 * 1024;
  */
 const COLLECTION_WITHHELD_STATUSES = new Set([401, 403, 404, 410]);
 const ERROR_BODY_MAX_BYTES = 4 * 1024;
+/**
+ * Deadline for reading a whole actor/collection/error body once its headers
+ * arrived. `signedFetch`'s timeout ends at the headers, so without this a remote
+ * that stalls mid-body would hold the resolver open.
+ */
+const BODY_TIMEOUT_MS = 10000;
 async function readBoundedResponseBody(res, maxBytes) {
     const contentLength = res.headers.get('content-length');
     if (contentLength) {
         const declaredBytes = Number(contentLength);
         if (Number.isFinite(declaredBytes) && declaredBytes > maxBytes) {
             await res.body?.cancel().catch(() => { });
-            throw new Error(`Remote response exceeds ${maxBytes} byte limit`);
+            throw new boundedBody_1.ResponseBodyTooLargeError(maxBytes);
         }
     }
     if (!res.body)
         return '';
     const reader = res.body.getReader();
-    const decoder = new TextDecoder('utf-8', { fatal: true });
-    let bytesRead = 0;
-    let text = '';
-    try {
-        while (true) {
-            const { done, value } = await reader.read();
-            if (done)
-                break;
-            bytesRead += value.byteLength;
-            if (bytesRead > maxBytes) {
-                await reader.cancel().catch(() => { });
-                throw new Error(`Remote response exceeds ${maxBytes} byte limit`);
-            }
-            text += decoder.decode(value, { stream: true });
-        }
-        return text + decoder.decode();
-    }
-    finally {
-        reader.releaseLock();
-    }
+    const body = await (0, boundedBody_1.readBodyChunks)({ read: () => reader.read(), cancel: () => void reader.cancel().catch(() => { }) }, { maxBytes, timeoutMs: BODY_TIMEOUT_MS });
+    return new TextDecoder('utf-8', { fatal: true }).decode(body);
 }
 async function readBoundedJson(res, maxBytes) {
     const body = await readBoundedResponseBody(res, maxBytes);

package/dist/cjs/node/boundedBody.js

@@ -0,0 +1,77 @@
+"use strict";
+/**
+ * The one bounded, deadlined body read behind every federation response body
+ * (an actor or collection `Response`, a failed delivery's preview).
+ *
+ * The transports bound a request up to its response headers; from there the
+ * reader owns the stream, and a remote that sends headers and then stalls (or
+ * trickles) the body would hold a plain read loop forever. This caps the bytes
+ * AND gives the whole body one deadline, cancelling the source on every early
+ * exit. It reads through a {@link BodyChunkSource} so a WHATWG reader and a
+ * Node async iterator share it.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ResponseBodyTimeoutError = exports.ResponseBodyTooLargeError = void 0;
+exports.readBodyChunks = readBodyChunks;
+/** Thrown when a body exceeds its byte cap under `overflow: 'reject'`. */
+class ResponseBodyTooLargeError extends Error {
+    constructor(maxBytes) {
+        super(`Remote response exceeds ${maxBytes} byte limit`);
+        this.maxBytes = maxBytes;
+        this.name = 'ResponseBodyTooLargeError';
+    }
+}
+exports.ResponseBodyTooLargeError = ResponseBodyTooLargeError;
+/** Thrown when a body does not end within its deadline; this attempt failed, a retry may not. */
+class ResponseBodyTimeoutError extends Error {
+    constructor(timeoutMs) {
+        super(`Remote response body did not end within ${timeoutMs} ms`);
+        this.timeoutMs = timeoutMs;
+        this.name = 'ResponseBodyTimeoutError';
+    }
+}
+exports.ResponseBodyTimeoutError = ResponseBodyTimeoutError;
+/**
+ * Read `source` into one Buffer, at most `maxBytes` and within `timeoutMs`.
+ * The deadline timer is cleared on every path; the source is cancelled on a
+ * cap, a deadline, a read error, or a truncated preview.
+ */
+async function readBodyChunks(source, limits) {
+    const { maxBytes, timeoutMs, overflow = 'reject' } = limits;
+    const chunks = [];
+    let total = 0;
+    let timer;
+    const deadline = new Promise((_, reject) => {
+        timer = setTimeout(() => reject(new ResponseBodyTimeoutError(timeoutMs)), timeoutMs);
+    });
+    try {
+        for (;;) {
+            // `Promise.race` subscribes to both, so neither the deadline nor a read
+            // that rejects once the source is cancelled goes unhandled.
+            const { done, value } = await Promise.race([source.read(), deadline]);
+            if (done || !value)
+                break;
+            const buffer = Buffer.isBuffer(value)
+                ? value
+                : Buffer.from(value.buffer, value.byteOffset, value.byteLength);
+            if (total + buffer.length > maxBytes) {
+                if (overflow === 'reject')
+                    throw new ResponseBodyTooLargeError(maxBytes);
+                chunks.push(buffer.subarray(0, maxBytes - total));
+                total = maxBytes;
+                source.cancel();
+                break;
+            }
+            total += buffer.length;
+            chunks.push(buffer);
+        }
+    }
+    catch (err) {
+        source.cancel();
+        throw err;
+    }
+    finally {
+        clearTimeout(timer);
+    }
+    return Buffer.concat(chunks, total);
+}

package/dist/cjs/node/delivery.js

@@ -28,11 +28,14 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DeliveryAuthorizationDeniedError = void 0;
 exports.createDeliveryService = createDeliveryService;
 const apContext_1 = require("../apContext");
+const boundedBody_1 = require("./boundedBody");
 const httpSignature_1 = require("../httpSignature");
 /** Total time budget for a single delivery POST (connect + response headers). */
 const DELIVER_ACTIVITY_TIMEOUT_MS = 15000;
 /** How many bytes of a failed-delivery response body are read for the debug log. */
 const DELIVERY_RESPONSE_PREVIEW_MAX_BYTES = 1024;
+/** Deadline for reading that preview once the response headers arrived. */
+const DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS = 5000;
 /** The ActivityStreams public collection — the `to` addressee of a public activity. */
 const AP_PUBLIC = 'https://www.w3.org/ns/activitystreams#Public';
 /** A definitive refusal: workers must discard, not retry or count a delivery. */
@@ -44,18 +47,20 @@ class DeliveryAuthorizationDeniedError extends Error {
     }
 }
 exports.DeliveryAuthorizationDeniedError = DeliveryAuthorizationDeniedError;
-/** Read a bounded prefix of a failed-delivery response body for the debug log. */
+/**
+ * Read a bounded prefix of a failed-delivery response body for the debug log,
+ * within {@link DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS}: an inbox that stalls its
+ * body after the headers costs the worker that long, not forever.
+ */
 async function readResponsePreview(response) {
-    const chunks = [];
-    let totalBytes = 0;
+    const iterator = response[Symbol.asyncIterator]();
     try {
-        for await (const chunk of response) {
-            const buffer = Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk);
-            totalBytes += buffer.byteLength;
-            chunks.push(buffer);
-            if (totalBytes >= DELIVERY_RESPONSE_PREVIEW_MAX_BYTES)
-                break;
-        }
+        const preview = await (0, boundedBody_1.readBodyChunks)({ read: () => iterator.next(), cancel: () => response.destroy() }, {
+            maxBytes: DELIVERY_RESPONSE_PREVIEW_MAX_BYTES,
+            timeoutMs: DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS,
+            overflow: 'truncate',
+        });
+        return preview.toString('utf8');
     }
     catch {
         return '';
@@ -63,7 +68,6 @@ async function readResponsePreview(response) {
     finally {
         response.destroy();
     }
-    return Buffer.concat(chunks).toString('utf8', 0, DELIVERY_RESPONSE_PREVIEW_MAX_BYTES);
 }
 /** Build the outbound delivery + follow-lifecycle service from an app's adapters. */
 function createDeliveryService(config) {

package/dist/esm/actorObject.js

@@ -39,7 +39,10 @@ export function isApActorType(value) {
     return typeof value === 'string' && AP_ACTOR_TYPES.includes(value);
 }
 /**
- * Oxy account kind → the AS2 actor type the fediverse is told about it.
+ * Oxy account kind → the AS2 actor type the fediverse is told about it — the
+ * ONE table both directions read: what Oxy announces for its own accounts here,
+ * and, through {@link accountKindForActor}, what it records a remote actor as.
+ * Changing how a kind federates is an edit to this table and nowhere else.
  *
  * `satisfies Record<AccountKind, LocalActorType>` is the load-bearing part: a
  * kind added to `@oxy.so/contracts` fails THIS build rather than silently
@@ -71,6 +74,8 @@ export function isApActorType(value) {
  *    accepted by all four implementations' whitelists and compared in none of
  *    them — neither `bot?` nor `group?` in Mastodon, `bot_account = false` in
  *    Lemmy, `isBot` false in Misskey, an ordinary account in PeerTube.
+ *  - **`community` → `Group`.** A forum its members post into and that
+ *    re-broadcasts them: exactly what Lemmy, Mbin and Friendica mean by `Group`.
  *
  * What this does NOT do: it does not stop a remote instance offering a reply box
  * under a channel's post. NO actor type gates that in any of the four — Mastodon
@@ -85,6 +90,7 @@ export const LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = {
     project: 'Organization',
     bot: 'Service',
     channel: 'Organization',
+    community: 'Group',
 };
 /**
  * The AS2 actor type for an Oxy account kind, defaulting to `Person`.
@@ -101,6 +107,49 @@ export const LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = {
 export function localActorTypeForAccountKind(kind) {
     return isAccountKind(kind) ? LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND[kind] : 'Person';
 }
+/**
+ * The inverse of {@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}, derived from it. Three
+ * kinds share `Organization`, so a type reads back as the FIRST kind the table
+ * lists for it (`Organization` → `organization`); nothing on the wire tells a
+ * remote project from a remote organization.
+ */
+const ACCOUNT_KIND_BY_LOCAL_ACTOR_TYPE = (() => {
+    const inverse = new Map();
+    for (const [kind, type] of Object.entries(LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND)) {
+        if (!inverse.has(type))
+            inverse.set(type, kind);
+    }
+    return inverse;
+})();
+/**
+ * AS2 actor types Oxy never announces but receives, and the announced type each
+ * is read as. `Application` is a non-human software actor — what `Service`
+ * already says, and half of Mastodon's own `bot?`.
+ */
+const INBOUND_ACTOR_TYPE_ALIASES = new Map([
+    ['Application', 'Service'],
+]);
+/**
+ * The kind a `Group` that only its moderators may post into is recorded as.
+ * Such a group forwards nobody: it PUBLISHES (a PeerTube video channel, a
+ * WordPress multi-author blog), which is what an Oxy channel is. The one
+ * inbound signal the table above has no column for.
+ */
+const MODERATED_GROUP_KIND = 'channel';
+/**
+ * The Oxy account kind a REMOTE actor is recorded as, read from the same table
+ * the local announcement uses ({@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}). A type
+ * the table does not know (absent, misspelt, an extension type) is `personal`,
+ * the same default {@link localActorTypeForAccountKind} announces.
+ */
+export function accountKindForActor(actor) {
+    if (typeof actor.type !== 'string')
+        return 'personal';
+    const type = INBOUND_ACTOR_TYPE_ALIASES.get(actor.type) ?? actor.type;
+    if (type === LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND.community && actor.postingRestrictedToMods === true)
+        return MODERATED_GROUP_KIND;
+    return ACCOUNT_KIND_BY_LOCAL_ACTOR_TYPE.get(type) ?? 'personal';
+}
 /** Map common image extensions to a MIME type for an actor image `mediaType`. */
 const IMAGE_MEDIA_TYPE_BY_EXT = {
     png: 'image/png',

package/dist/esm/index.js

@@ -62,4 +62,4 @@ export { canonicalFederationHost, isSameFederationHost, extractActorUriFromActiv
  * byte-identical across apps, with media resolution injected. The actor `type`
  * follows the Oxy account kind ({@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}).
  */
-export { createLocalActorBuilder, normalizeAlsoKnownAs, localActorTypeForAccountKind, isApActorType, AP_ACTOR_TYPES, LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND, } from './actorObject.js';
+export { createLocalActorBuilder, normalizeAlsoKnownAs, localActorTypeForAccountKind, accountKindForActor, isApActorType, AP_ACTOR_TYPES, LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND, } from './actorObject.js';

package/dist/esm/node/actorResolver.js

@@ -20,6 +20,7 @@
  */
 import { isSameFederationHost } from '../apUri.js';
 import { readProxyDeclarations, } from '../networkIdentity.js';
+import { readBodyChunks, ResponseBodyTooLargeError } from './boundedBody.js';
 /**
  * Minimum interval between background actor refreshes for the same actor.
  * Prevents refresh storms when a profile is viewed repeatedly in a short window.
@@ -39,38 +40,26 @@ const COLLECTION_BODY_MAX_BYTES = 64 * 1024;
  */
 const COLLECTION_WITHHELD_STATUSES = new Set([401, 403, 404, 410]);
 const ERROR_BODY_MAX_BYTES = 4 * 1024;
+/**
+ * Deadline for reading a whole actor/collection/error body once its headers
+ * arrived. `signedFetch`'s timeout ends at the headers, so without this a remote
+ * that stalls mid-body would hold the resolver open.
+ */
+const BODY_TIMEOUT_MS = 10000;
 async function readBoundedResponseBody(res, maxBytes) {
     const contentLength = res.headers.get('content-length');
     if (contentLength) {
         const declaredBytes = Number(contentLength);
         if (Number.isFinite(declaredBytes) && declaredBytes > maxBytes) {
             await res.body?.cancel().catch(() => { });
-            throw new Error(`Remote response exceeds ${maxBytes} byte limit`);
+            throw new ResponseBodyTooLargeError(maxBytes);
         }
     }
     if (!res.body)
         return '';
     const reader = res.body.getReader();
-    const decoder = new TextDecoder('utf-8', { fatal: true });
-    let bytesRead = 0;
-    let text = '';
-    try {
-        while (true) {
-            const { done, value } = await reader.read();
-            if (done)
-                break;
-            bytesRead += value.byteLength;
-            if (bytesRead > maxBytes) {
-                await reader.cancel().catch(() => { });
-                throw new Error(`Remote response exceeds ${maxBytes} byte limit`);
-            }
-            text += decoder.decode(value, { stream: true });
-        }
-        return text + decoder.decode();
-    }
-    finally {
-        reader.releaseLock();
-    }
+    const body = await readBodyChunks({ read: () => reader.read(), cancel: () => void reader.cancel().catch(() => { }) }, { maxBytes, timeoutMs: BODY_TIMEOUT_MS });
+    return new TextDecoder('utf-8', { fatal: true }).decode(body);
 }
 async function readBoundedJson(res, maxBytes) {
     const body = await readBoundedResponseBody(res, maxBytes);

package/dist/esm/node/boundedBody.js

@@ -0,0 +1,71 @@
+/**
+ * The one bounded, deadlined body read behind every federation response body
+ * (an actor or collection `Response`, a failed delivery's preview).
+ *
+ * The transports bound a request up to its response headers; from there the
+ * reader owns the stream, and a remote that sends headers and then stalls (or
+ * trickles) the body would hold a plain read loop forever. This caps the bytes
+ * AND gives the whole body one deadline, cancelling the source on every early
+ * exit. It reads through a {@link BodyChunkSource} so a WHATWG reader and a
+ * Node async iterator share it.
+ */
+/** Thrown when a body exceeds its byte cap under `overflow: 'reject'`. */
+export class ResponseBodyTooLargeError extends Error {
+    constructor(maxBytes) {
+        super(`Remote response exceeds ${maxBytes} byte limit`);
+        this.maxBytes = maxBytes;
+        this.name = 'ResponseBodyTooLargeError';
+    }
+}
+/** Thrown when a body does not end within its deadline; this attempt failed, a retry may not. */
+export class ResponseBodyTimeoutError extends Error {
+    constructor(timeoutMs) {
+        super(`Remote response body did not end within ${timeoutMs} ms`);
+        this.timeoutMs = timeoutMs;
+        this.name = 'ResponseBodyTimeoutError';
+    }
+}
+/**
+ * Read `source` into one Buffer, at most `maxBytes` and within `timeoutMs`.
+ * The deadline timer is cleared on every path; the source is cancelled on a
+ * cap, a deadline, a read error, or a truncated preview.
+ */
+export async function readBodyChunks(source, limits) {
+    const { maxBytes, timeoutMs, overflow = 'reject' } = limits;
+    const chunks = [];
+    let total = 0;
+    let timer;
+    const deadline = new Promise((_, reject) => {
+        timer = setTimeout(() => reject(new ResponseBodyTimeoutError(timeoutMs)), timeoutMs);
+    });
+    try {
+        for (;;) {
+            // `Promise.race` subscribes to both, so neither the deadline nor a read
+            // that rejects once the source is cancelled goes unhandled.
+            const { done, value } = await Promise.race([source.read(), deadline]);
+            if (done || !value)
+                break;
+            const buffer = Buffer.isBuffer(value)
+                ? value
+                : Buffer.from(value.buffer, value.byteOffset, value.byteLength);
+            if (total + buffer.length > maxBytes) {
+                if (overflow === 'reject')
+                    throw new ResponseBodyTooLargeError(maxBytes);
+                chunks.push(buffer.subarray(0, maxBytes - total));
+                total = maxBytes;
+                source.cancel();
+                break;
+            }
+            total += buffer.length;
+            chunks.push(buffer);
+        }
+    }
+    catch (err) {
+        source.cancel();
+        throw err;
+    }
+    finally {
+        clearTimeout(timer);
+    }
+    return Buffer.concat(chunks, total);
+}

package/dist/esm/node/delivery.js

@@ -24,11 +24,14 @@
  * call `deliverToFollowers` / `deliverActivity` / `queueDelivery` here.
  */
 import { AP_CONTEXT } from '../apContext.js';
+import { readBodyChunks } from './boundedBody.js';
 import { signRequest } from '../httpSignature.js';
 /** Total time budget for a single delivery POST (connect + response headers). */
 const DELIVER_ACTIVITY_TIMEOUT_MS = 15000;
 /** How many bytes of a failed-delivery response body are read for the debug log. */
 const DELIVERY_RESPONSE_PREVIEW_MAX_BYTES = 1024;
+/** Deadline for reading that preview once the response headers arrived. */
+const DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS = 5000;
 /** The ActivityStreams public collection — the `to` addressee of a public activity. */
 const AP_PUBLIC = 'https://www.w3.org/ns/activitystreams#Public';
 /** A definitive refusal: workers must discard, not retry or count a delivery. */
@@ -39,18 +42,20 @@ export class DeliveryAuthorizationDeniedError extends Error {
         this.name = 'DeliveryAuthorizationDeniedError';
     }
 }
-/** Read a bounded prefix of a failed-delivery response body for the debug log. */
+/**
+ * Read a bounded prefix of a failed-delivery response body for the debug log,
+ * within {@link DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS}: an inbox that stalls its
+ * body after the headers costs the worker that long, not forever.
+ */
 async function readResponsePreview(response) {
-    const chunks = [];
-    let totalBytes = 0;
+    const iterator = response[Symbol.asyncIterator]();
     try {
-        for await (const chunk of response) {
-            const buffer = Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk);
-            totalBytes += buffer.byteLength;
-            chunks.push(buffer);
-            if (totalBytes >= DELIVERY_RESPONSE_PREVIEW_MAX_BYTES)
-                break;
-        }
+        const preview = await readBodyChunks({ read: () => iterator.next(), cancel: () => response.destroy() }, {
+            maxBytes: DELIVERY_RESPONSE_PREVIEW_MAX_BYTES,
+            timeoutMs: DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS,
+            overflow: 'truncate',
+        });
+        return preview.toString('utf8');
     }
     catch {
         return '';
@@ -58,7 +63,6 @@ async function readResponsePreview(response) {
     finally {
         response.destroy();
     }
-    return Buffer.concat(chunks).toString('utf8', 0, DELIVERY_RESPONSE_PREVIEW_MAX_BYTES);
 }
 /** Build the outbound delivery + follow-lifecycle service from an app's adapters. */
 export function createDeliveryService(config) {

package/dist/types/actorObject.d.ts

@@ -36,28 +36,29 @@ export declare function isApActorType(value: unknown): value is ApActorType;
 /**
  * The AS2 actor types this engine will announce for a LOCAL user actor.
  *
- * Narrower than AS2's five actor types, and narrow ON PURPOSE — the union names
- * what the builder can emit, so the two absences are documented decisions rather
- * than oversights. Both were checked against real receiving implementations
- * (mastodon `a3649295`, lemmy `4ce92433`, misskey `b95e4841`, peertube
- * `fe0da961`), not against the spec:
+ * Narrower than AS2's five actor types, and narrow ON PURPOSE. Both decisions
+ * were checked against real receiving implementations (mastodon `a3649295`,
+ * lemmy `4ce92433`, misskey `b95e4841`, peertube `fe0da961`), not against the
+ * spec:
  *
  *  - **`Application`** is the INSTANCE actor's type (see the actor router's
  *    `/ap/users/instance` branch), reserved by convention for the software
  *    itself. No account is the software.
- *  - **`Group`** is refused because in the deployed fediverse it is not read as
- *    "a collective of actors" but as a FORWARDING actor, and the two failure
- *    modes are concrete. Lemmy reclassifies a remote `Group` as a COMMUNITY:
- *    it is followable and the Follow is Accepted, so it looks like it worked,
- *    and then it never shows a single post — we emit no `Announce`, and our
- *    Notes do not resolve to a community. PeerTube is worse and louder: it
- *    REJECTS a `Group` actor outright unless it carries `attributedTo` naming a
- *    `Person`. An Oxy account authors its own posts and forwards nothing, so
- *    `Group` would advertise a protocol this engine does not implement.
+ *  - **`Group`** is announced ONLY for a `community`, because in the deployed
+ *    fediverse it is not read as "a collective of actors" but as a FORWARDING
+ *    actor. Lemmy reclassifies a remote `Group` as a community: followable, the
+ *    Follow is Accepted, and then it never shows a post unless the actor
+ *    `Announce`s its members' posts. PeerTube rejects a `Group` outright unless
+ *    it carries `attributedTo` naming a `Person`. An account that authors its
+ *    own posts (a project, a channel) must therefore never claim it; a community
+ *    is the one kind whose nature IS forwarding.
  */
-export type LocalActorType = Extract<ApActorType, 'Person' | 'Organization' | 'Service'>;
+export type LocalActorType = Extract<ApActorType, 'Person' | 'Organization' | 'Service' | 'Group'>;
 /**
- * Oxy account kind → the AS2 actor type the fediverse is told about it.
+ * Oxy account kind → the AS2 actor type the fediverse is told about it — the
+ * ONE table both directions read: what Oxy announces for its own accounts here,
+ * and, through {@link accountKindForActor}, what it records a remote actor as.
+ * Changing how a kind federates is an edit to this table and nowhere else.
  *
  * `satisfies Record<AccountKind, LocalActorType>` is the load-bearing part: a
  * kind added to `@oxy.so/contracts` fails THIS build rather than silently
@@ -89,6 +90,8 @@ export type LocalActorType = Extract<ApActorType, 'Person' | 'Organization' | 'S
  *    accepted by all four implementations' whitelists and compared in none of
  *    them — neither `bot?` nor `group?` in Mastodon, `bot_account = false` in
  *    Lemmy, `isBot` false in Misskey, an ordinary account in PeerTube.
+ *  - **`community` → `Group`.** A forum its members post into and that
+ *    re-broadcasts them: exactly what Lemmy, Mbin and Friendica mean by `Group`.
  *
  * What this does NOT do: it does not stop a remote instance offering a reply box
  * under a channel's post. NO actor type gates that in any of the four — Mastodon
@@ -103,6 +106,7 @@ export declare const LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND: {
     readonly project: "Organization";
     readonly bot: "Service";
     readonly channel: "Organization";
+    readonly community: "Group";
 };
 /**
  * The AS2 actor type for an Oxy account kind, defaulting to `Person`.
@@ -117,6 +121,19 @@ export declare const LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND: {
  * existed.
  */
 export declare function localActorTypeForAccountKind(kind: unknown): LocalActorType;
+/** The fields of a remote actor document that decide its kind. */
+export interface ActorKindSignals {
+    type?: unknown;
+    /** Lemmy's and PeerTube's flag: only moderators may post into this `Group`. */
+    postingRestrictedToMods?: unknown;
+}
+/**
+ * The Oxy account kind a REMOTE actor is recorded as, read from the same table
+ * the local announcement uses ({@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}). A type
+ * the table does not know (absent, misspelt, an extension type) is `personal`,
+ * the same default {@link localActorTypeForAccountKind} announces.
+ */
+export declare function accountKindForActor(actor: ActorKindSignals): AccountKind;
 /**
  * App-supplied media resolution for the actor `icon`/`image`. Each function
  * resolves a stored reference (Oxy file id or URL) to a FINAL, ready-to-serve

package/dist/types/index.d.ts

@@ -62,7 +62,7 @@ export { canonicalFederationHost, isSameFederationHost, extractActorUriFromActiv
  * byte-identical across apps, with media resolution injected. The actor `type`
  * follows the Oxy account kind ({@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}).
  */
-export { createLocalActorBuilder, normalizeAlsoKnownAs, localActorTypeForAccountKind, isApActorType, AP_ACTOR_TYPES, LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND, type ApActorType, type LocalActorType, type LocalActorBuilder, type LocalActorBuilderConfig, type BuildLocalActorParams, type ActorMediaResolver, } from './actorObject';
+export { createLocalActorBuilder, normalizeAlsoKnownAs, localActorTypeForAccountKind, accountKindForActor, type ActorKindSignals, isApActorType, AP_ACTOR_TYPES, LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND, type ApActorType, type LocalActorType, type LocalActorBuilder, type LocalActorBuilderConfig, type BuildLocalActorParams, type ActorMediaResolver, } from './actorObject';
 /**
  * Supported external networks, keyed by the transport a connector speaks.
  * `instagram-graph` is Meta's Graph API Business Discovery: a read-only

package/dist/types/node/boundedBody.d.ts

@@ -0,0 +1,50 @@
+/**
+ * The one bounded, deadlined body read behind every federation response body
+ * (an actor or collection `Response`, a failed delivery's preview).
+ *
+ * The transports bound a request up to its response headers; from there the
+ * reader owns the stream, and a remote that sends headers and then stalls (or
+ * trickles) the body would hold a plain read loop forever. This caps the bytes
+ * AND gives the whole body one deadline, cancelling the source on every early
+ * exit. It reads through a {@link BodyChunkSource} so a WHATWG reader and a
+ * Node async iterator share it.
+ */
+/** One `read()` step: a WHATWG `ReadableStreamDefaultReader` and an async iterator both return it. */
+interface BodyChunk {
+    done?: boolean;
+    value?: Uint8Array;
+}
+/** Where the chunks come from, and how to release the stream when a read ends early. */
+export interface BodyChunkSource {
+    read(): Promise<BodyChunk>;
+    cancel(): void;
+}
+/** Limits for {@link readBodyChunks}. */
+export interface BodyReadLimits {
+    /** Largest body accepted, in bytes. */
+    maxBytes: number;
+    /** Deadline for the WHOLE body, from the call until the source ends, in milliseconds. */
+    timeoutMs: number;
+    /**
+     * `reject` (the default) throws {@link ResponseBodyTooLargeError} past the cap;
+     * `truncate` stops there and returns the first `maxBytes` (a preview).
+     */
+    overflow?: 'reject' | 'truncate';
+}
+/** Thrown when a body exceeds its byte cap under `overflow: 'reject'`. */
+export declare class ResponseBodyTooLargeError extends Error {
+    readonly maxBytes: number;
+    constructor(maxBytes: number);
+}
+/** Thrown when a body does not end within its deadline; this attempt failed, a retry may not. */
+export declare class ResponseBodyTimeoutError extends Error {
+    readonly timeoutMs: number;
+    constructor(timeoutMs: number);
+}
+/**
+ * Read `source` into one Buffer, at most `maxBytes` and within `timeoutMs`.
+ * The deadline timer is cleared on every path; the source is cancelled on a
+ * cap, a deadline, a read error, or a truncated preview.
+ */
+export declare function readBodyChunks(source: BodyChunkSource, limits: BodyReadLimits): Promise<Buffer>;
+export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oxy.so/federation",
-  "version": "2.4.2",
+  "version": "2.5.1",
   "description": "Oxy Federation — the app-agnostic ActivityPub identity + follow engine substrate: the network-connector contract, normalized cross-network DTOs, HTTP signatures, actor resolution, the outbound delivery transport + follow lifecycle, the inbound dispatcher, and the webfinger/actor/inbox Express routers. Domain-parameterized so every Oxy app backend federates under its own domain.",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",

package/src/__tests__/actorObject.test.ts

@@ -7,6 +7,7 @@ import {
   isApActorType,
   LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND,
   localActorTypeForAccountKind,
+  accountKindForActor,
   type ActorMediaResolver,
 } from '../index';
 
@@ -198,6 +199,7 @@ describe('actor type by Oxy account kind', () => {
     expect(localActorTypeForAccountKind('project')).toBe('Organization');
     expect(localActorTypeForAccountKind('bot')).toBe('Service');
     expect(localActorTypeForAccountKind('channel')).toBe('Organization');
+    expect(localActorTypeForAccountKind('community')).toBe('Group');
   });
 
   it('never announces a human-curated channel as automated', () => {
@@ -207,10 +209,11 @@ describe('actor type by Oxy account kind', () => {
     // A channel is curated by people; only `bot` may claim automation.
     expect(localActorTypeForAccountKind('channel')).not.toBe('Service');
     expect(localActorTypeForAccountKind('bot')).toBe('Service');
-    // `Group` is never emitted for anything: Lemmy would reclassify the actor as
-    // a community that silently never receives content, and PeerTube rejects a
-    // `Group` lacking `attributedTo` outright.
-    expect(Object.values(LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND)).not.toContain('Group');
+    // `Group` is announced for a community alone: Lemmy reads it as a forwarding
+    // community, and PeerTube rejects a `Group` lacking `attributedTo` outright.
+    expect(ACCOUNT_KINDS.filter((kind) => localActorTypeForAccountKind(kind) === 'Group')).toEqual([
+      'community',
+    ]);
   });
 
   it('covers every account kind contracts defines, with more than one answer', () => {
@@ -219,13 +222,30 @@ describe('actor type by Oxy account kind', () => {
     // Vacuity floor: a mapping that returned one constant would pass every
     // single-kind assertion above and fail this.
     const answers = ACCOUNT_KINDS.map((kind) => localActorTypeForAccountKind(kind));
-    expect(new Set(answers).size).toBe(3);
+    expect(new Set(answers).size).toBe(4);
     // And specifically: the two kinds either side of the channel decision differ.
     expect(localActorTypeForAccountKind('channel')).not.toBe(
       localActorTypeForAccountKind('personal'),
     );
   });
 
+  it('reads every announced type back as a kind announced the same way', () => {
+    for (const kind of ACCOUNT_KINDS) {
+      const type = localActorTypeForAccountKind(kind);
+      expect(localActorTypeForAccountKind(accountKindForActor({ type }))).toBe(type);
+    }
+  });
+
+  it('reads the actor types and flags Oxy never announces', () => {
+    expect(accountKindForActor({ type: 'Application' })).toBe('bot');
+    expect(accountKindForActor({ type: 'Group', postingRestrictedToMods: true })).toBe('channel');
+    expect(accountKindForActor({ type: 'Group', postingRestrictedToMods: false })).toBe(
+      'community',
+    );
+    expect(accountKindForActor({ type: 'Tombstone' })).toBe('personal');
+    expect(accountKindForActor({})).toBe('personal');
+  });
+
   it('falls back to Person for an absent, null, or unrecognized kind', () => {
     expect(localActorTypeForAccountKind(undefined)).toBe('Person');
     expect(localActorTypeForAccountKind(null)).toBe('Person');

package/src/__tests__/boundedBody.test.ts

@@ -0,0 +1,57 @@
+import { PassThrough } from 'node:stream';
+import {
+  type BodyChunkSource,
+  readBodyChunks,
+  ResponseBodyTimeoutError,
+  ResponseBodyTooLargeError,
+} from '../node/boundedBody';
+
+function streamSource(): { stream: PassThrough; source: BodyChunkSource } {
+  const stream = new PassThrough();
+  const iterator = stream[Symbol.asyncIterator]();
+  return { stream, source: { read: () => iterator.next(), cancel: () => stream.destroy() } };
+}
+
+describe('readBodyChunks', () => {
+  it('reads a body that ends within its limits', async () => {
+    const { stream, source } = streamSource();
+    const pending = readBodyChunks(source, { maxBytes: 8, timeoutMs: 1_000 });
+    stream.write('ab');
+    stream.end('cd');
+    expect((await pending).toString()).toBe('abcd');
+  });
+
+  it('rejects one byte past the cap (or truncates a preview) and destroys the stream', async () => {
+    const rejected = streamSource();
+    const pending = readBodyChunks(rejected.source, { maxBytes: 3, timeoutMs: 1_000 });
+    rejected.stream.write('abcd');
+    await expect(pending).rejects.toBeInstanceOf(ResponseBodyTooLargeError);
+    expect(rejected.stream.destroyed).toBe(true);
+
+    const truncated = streamSource();
+    const preview = readBodyChunks(truncated.source, {
+      maxBytes: 3,
+      timeoutMs: 1_000,
+      overflow: 'truncate',
+    });
+    truncated.stream.write('abcd');
+    expect((await preview).toString()).toBe('abc');
+    expect(truncated.stream.destroyed).toBe(true);
+  });
+
+  it('throws ResponseBodyTimeoutError for a body that stalls and destroys the stream', async () => {
+    jest.useFakeTimers();
+    try {
+      const { stream, source } = streamSource();
+      const pending = readBodyChunks(source, { maxBytes: 1_024, timeoutMs: 5_000 });
+      stream.write('{"partial":');
+      const settled = expect(pending).rejects.toBeInstanceOf(ResponseBodyTimeoutError);
+      await jest.advanceTimersByTimeAsync(5_000);
+      await settled;
+      expect(stream.destroyed).toBe(true);
+      expect(jest.getTimerCount()).toBe(0);
+    } finally {
+      jest.useRealTimers();
+    }
+  });
+});

package/src/actorObject.ts

@@ -49,29 +49,30 @@ export function isApActorType(value: unknown): value is ApActorType {
 /**
  * The AS2 actor types this engine will announce for a LOCAL user actor.
  *
- * Narrower than AS2's five actor types, and narrow ON PURPOSE — the union names
- * what the builder can emit, so the two absences are documented decisions rather
- * than oversights. Both were checked against real receiving implementations
- * (mastodon `a3649295`, lemmy `4ce92433`, misskey `b95e4841`, peertube
- * `fe0da961`), not against the spec:
+ * Narrower than AS2's five actor types, and narrow ON PURPOSE. Both decisions
+ * were checked against real receiving implementations (mastodon `a3649295`,
+ * lemmy `4ce92433`, misskey `b95e4841`, peertube `fe0da961`), not against the
+ * spec:
  *
  *  - **`Application`** is the INSTANCE actor's type (see the actor router's
  *    `/ap/users/instance` branch), reserved by convention for the software
  *    itself. No account is the software.
- *  - **`Group`** is refused because in the deployed fediverse it is not read as
- *    "a collective of actors" but as a FORWARDING actor, and the two failure
- *    modes are concrete. Lemmy reclassifies a remote `Group` as a COMMUNITY:
- *    it is followable and the Follow is Accepted, so it looks like it worked,
- *    and then it never shows a single post — we emit no `Announce`, and our
- *    Notes do not resolve to a community. PeerTube is worse and louder: it
- *    REJECTS a `Group` actor outright unless it carries `attributedTo` naming a
- *    `Person`. An Oxy account authors its own posts and forwards nothing, so
- *    `Group` would advertise a protocol this engine does not implement.
+ *  - **`Group`** is announced ONLY for a `community`, because in the deployed
+ *    fediverse it is not read as "a collective of actors" but as a FORWARDING
+ *    actor. Lemmy reclassifies a remote `Group` as a community: followable, the
+ *    Follow is Accepted, and then it never shows a post unless the actor
+ *    `Announce`s its members' posts. PeerTube rejects a `Group` outright unless
+ *    it carries `attributedTo` naming a `Person`. An account that authors its
+ *    own posts (a project, a channel) must therefore never claim it; a community
+ *    is the one kind whose nature IS forwarding.
  */
-export type LocalActorType = Extract<ApActorType, 'Person' | 'Organization' | 'Service'>;
+export type LocalActorType = Extract<ApActorType, 'Person' | 'Organization' | 'Service' | 'Group'>;
 
 /**
- * Oxy account kind → the AS2 actor type the fediverse is told about it.
+ * Oxy account kind → the AS2 actor type the fediverse is told about it — the
+ * ONE table both directions read: what Oxy announces for its own accounts here,
+ * and, through {@link accountKindForActor}, what it records a remote actor as.
+ * Changing how a kind federates is an edit to this table and nowhere else.
  *
  * `satisfies Record<AccountKind, LocalActorType>` is the load-bearing part: a
  * kind added to `@oxy.so/contracts` fails THIS build rather than silently
@@ -103,6 +104,8 @@ export type LocalActorType = Extract<ApActorType, 'Person' | 'Organization' | 'S
  *    accepted by all four implementations' whitelists and compared in none of
  *    them — neither `bot?` nor `group?` in Mastodon, `bot_account = false` in
  *    Lemmy, `isBot` false in Misskey, an ordinary account in PeerTube.
+ *  - **`community` → `Group`.** A forum its members post into and that
+ *    re-broadcasts them: exactly what Lemmy, Mbin and Friendica mean by `Group`.
  *
  * What this does NOT do: it does not stop a remote instance offering a reply box
  * under a channel's post. NO actor type gates that in any of the four — Mastodon
@@ -117,6 +120,7 @@ export const LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND = {
   project: 'Organization',
   bot: 'Service',
   channel: 'Organization',
+  community: 'Group',
 } as const satisfies Record<AccountKind, LocalActorType>;
 
 /**
@@ -135,6 +139,58 @@ export function localActorTypeForAccountKind(kind: unknown): LocalActorType {
   return isAccountKind(kind) ? LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND[kind] : 'Person';
 }
 
+/**
+ * The inverse of {@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}, derived from it. Three
+ * kinds share `Organization`, so a type reads back as the FIRST kind the table
+ * lists for it (`Organization` → `organization`); nothing on the wire tells a
+ * remote project from a remote organization.
+ */
+const ACCOUNT_KIND_BY_LOCAL_ACTOR_TYPE: ReadonlyMap<string, AccountKind> = (() => {
+  const inverse = new Map<string, AccountKind>();
+  for (const [kind, type] of Object.entries(LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND)) {
+    if (!inverse.has(type)) inverse.set(type, kind as AccountKind);
+  }
+  return inverse;
+})();
+
+/**
+ * AS2 actor types Oxy never announces but receives, and the announced type each
+ * is read as. `Application` is a non-human software actor — what `Service`
+ * already says, and half of Mastodon's own `bot?`.
+ */
+const INBOUND_ACTOR_TYPE_ALIASES: ReadonlyMap<string, LocalActorType> = new Map([
+  ['Application', 'Service'],
+]);
+
+/**
+ * The kind a `Group` that only its moderators may post into is recorded as.
+ * Such a group forwards nobody: it PUBLISHES (a PeerTube video channel, a
+ * WordPress multi-author blog), which is what an Oxy channel is. The one
+ * inbound signal the table above has no column for.
+ */
+const MODERATED_GROUP_KIND: AccountKind = 'channel';
+
+/** The fields of a remote actor document that decide its kind. */
+export interface ActorKindSignals {
+  type?: unknown;
+  /** Lemmy's and PeerTube's flag: only moderators may post into this `Group`. */
+  postingRestrictedToMods?: unknown;
+}
+
+/**
+ * The Oxy account kind a REMOTE actor is recorded as, read from the same table
+ * the local announcement uses ({@link LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND}). A type
+ * the table does not know (absent, misspelt, an extension type) is `personal`,
+ * the same default {@link localActorTypeForAccountKind} announces.
+ */
+export function accountKindForActor(actor: ActorKindSignals): AccountKind {
+  if (typeof actor.type !== 'string') return 'personal';
+  const type = INBOUND_ACTOR_TYPE_ALIASES.get(actor.type) ?? actor.type;
+  if (type === LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND.community && actor.postingRestrictedToMods === true)
+    return MODERATED_GROUP_KIND;
+  return ACCOUNT_KIND_BY_LOCAL_ACTOR_TYPE.get(type) ?? 'personal';
+}
+
 /** Map common image extensions to a MIME type for an actor image `mediaType`. */
 const IMAGE_MEDIA_TYPE_BY_EXT: Record<string, string> = {
   png: 'image/png',

package/src/index.ts

@@ -119,6 +119,8 @@ export {
   createLocalActorBuilder,
   normalizeAlsoKnownAs,
   localActorTypeForAccountKind,
+  accountKindForActor,
+  type ActorKindSignals,
   isApActorType,
   AP_ACTOR_TYPES,
   LOCAL_ACTOR_TYPE_BY_ACCOUNT_KIND,

package/src/node/actorResolver.ts

@@ -27,6 +27,7 @@ import {
   type NetworkIdentityCandidate,
 } from '../networkIdentity';
 import type { NormalizedExternalActor } from '../index';
+import { readBodyChunks, ResponseBodyTooLargeError } from './boundedBody';
 import type { SignedFetch } from './signedFetch';
 import type { ReportActorGoneOutcome } from './identityBridge';
 
@@ -53,37 +54,31 @@ const COLLECTION_BODY_MAX_BYTES = 64 * 1024;
 const COLLECTION_WITHHELD_STATUSES: ReadonlySet<number> = new Set([401, 403, 404, 410]);
 const ERROR_BODY_MAX_BYTES = 4 * 1024;
 
+/**
+ * Deadline for reading a whole actor/collection/error body once its headers
+ * arrived. `signedFetch`'s timeout ends at the headers, so without this a remote
+ * that stalls mid-body would hold the resolver open.
+ */
+const BODY_TIMEOUT_MS = 10_000;
+
 async function readBoundedResponseBody(res: Response, maxBytes: number): Promise<string> {
   const contentLength = res.headers.get('content-length');
   if (contentLength) {
     const declaredBytes = Number(contentLength);
     if (Number.isFinite(declaredBytes) && declaredBytes > maxBytes) {
       await res.body?.cancel().catch(() => {});
-      throw new Error(`Remote response exceeds ${maxBytes} byte limit`);
+      throw new ResponseBodyTooLargeError(maxBytes);
     }
   }
 
   if (!res.body) return '';
 
   const reader = res.body.getReader();
-  const decoder = new TextDecoder('utf-8', { fatal: true });
-  let bytesRead = 0;
-  let text = '';
-  try {
-    while (true) {
-      const { done, value } = await reader.read();
-      if (done) break;
-      bytesRead += value.byteLength;
-      if (bytesRead > maxBytes) {
-        await reader.cancel().catch(() => {});
-        throw new Error(`Remote response exceeds ${maxBytes} byte limit`);
-      }
-      text += decoder.decode(value, { stream: true });
-    }
-    return text + decoder.decode();
-  } finally {
-    reader.releaseLock();
-  }
+  const body = await readBodyChunks(
+    { read: () => reader.read(), cancel: () => void reader.cancel().catch(() => {}) },
+    { maxBytes, timeoutMs: BODY_TIMEOUT_MS },
+  );
+  return new TextDecoder('utf-8', { fatal: true }).decode(body);
 }
 
 async function readBoundedJson(res: Response, maxBytes: number): Promise<Record<string, unknown>> {

package/src/node/boundedBody.ts

@@ -0,0 +1,96 @@
+/**
+ * The one bounded, deadlined body read behind every federation response body
+ * (an actor or collection `Response`, a failed delivery's preview).
+ *
+ * The transports bound a request up to its response headers; from there the
+ * reader owns the stream, and a remote that sends headers and then stalls (or
+ * trickles) the body would hold a plain read loop forever. This caps the bytes
+ * AND gives the whole body one deadline, cancelling the source on every early
+ * exit. It reads through a {@link BodyChunkSource} so a WHATWG reader and a
+ * Node async iterator share it.
+ */
+
+/** One `read()` step: a WHATWG `ReadableStreamDefaultReader` and an async iterator both return it. */
+interface BodyChunk {
+  done?: boolean;
+  value?: Uint8Array;
+}
+
+/** Where the chunks come from, and how to release the stream when a read ends early. */
+export interface BodyChunkSource {
+  read(): Promise<BodyChunk>;
+  cancel(): void;
+}
+
+/** Limits for {@link readBodyChunks}. */
+export interface BodyReadLimits {
+  /** Largest body accepted, in bytes. */
+  maxBytes: number;
+  /** Deadline for the WHOLE body, from the call until the source ends, in milliseconds. */
+  timeoutMs: number;
+  /**
+   * `reject` (the default) throws {@link ResponseBodyTooLargeError} past the cap;
+   * `truncate` stops there and returns the first `maxBytes` (a preview).
+   */
+  overflow?: 'reject' | 'truncate';
+}
+
+/** Thrown when a body exceeds its byte cap under `overflow: 'reject'`. */
+export class ResponseBodyTooLargeError extends Error {
+  constructor(public readonly maxBytes: number) {
+    super(`Remote response exceeds ${maxBytes} byte limit`);
+    this.name = 'ResponseBodyTooLargeError';
+  }
+}
+
+/** Thrown when a body does not end within its deadline; this attempt failed, a retry may not. */
+export class ResponseBodyTimeoutError extends Error {
+  constructor(public readonly timeoutMs: number) {
+    super(`Remote response body did not end within ${timeoutMs} ms`);
+    this.name = 'ResponseBodyTimeoutError';
+  }
+}
+
+/**
+ * Read `source` into one Buffer, at most `maxBytes` and within `timeoutMs`.
+ * The deadline timer is cleared on every path; the source is cancelled on a
+ * cap, a deadline, a read error, or a truncated preview.
+ */
+export async function readBodyChunks(
+  source: BodyChunkSource,
+  limits: BodyReadLimits,
+): Promise<Buffer> {
+  const { maxBytes, timeoutMs, overflow = 'reject' } = limits;
+  const chunks: Buffer[] = [];
+  let total = 0;
+  let timer: ReturnType<typeof setTimeout> | undefined;
+  const deadline = new Promise<never>((_, reject) => {
+    timer = setTimeout(() => reject(new ResponseBodyTimeoutError(timeoutMs)), timeoutMs);
+  });
+  try {
+    for (;;) {
+      // `Promise.race` subscribes to both, so neither the deadline nor a read
+      // that rejects once the source is cancelled goes unhandled.
+      const { done, value } = await Promise.race([source.read(), deadline]);
+      if (done || !value) break;
+      const buffer = Buffer.isBuffer(value)
+        ? value
+        : Buffer.from(value.buffer, value.byteOffset, value.byteLength);
+      if (total + buffer.length > maxBytes) {
+        if (overflow === 'reject') throw new ResponseBodyTooLargeError(maxBytes);
+        chunks.push(buffer.subarray(0, maxBytes - total));
+        total = maxBytes;
+        source.cancel();
+        break;
+      }
+      total += buffer.length;
+      chunks.push(buffer);
+    }
+  } catch (err) {
+    source.cancel();
+    throw err;
+  } finally {
+    clearTimeout(timer);
+  }
+  return Buffer.concat(chunks, total);
+}

package/src/node/delivery.ts

@@ -26,6 +26,7 @@
 
 import type { AccountKind } from '@oxy.so/contracts';
 import { AP_CONTEXT } from '../apContext';
+import { readBodyChunks } from './boundedBody';
 import { signRequest, type HttpSignatureSigner } from '../httpSignature';
 import type { UrlBuilders } from '../urls';
 import type { LocalActorBuilder } from '../actorObject';
@@ -34,6 +35,8 @@ import type { LocalActorBuilder } from '../actorObject';
 const DELIVER_ACTIVITY_TIMEOUT_MS = 15000;
 /** How many bytes of a failed-delivery response body are read for the debug log. */
 const DELIVERY_RESPONSE_PREVIEW_MAX_BYTES = 1024;
+/** Deadline for reading that preview once the response headers arrived. */
+const DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS = 5_000;
 
 /** The ActivityStreams public collection — the `to` addressee of a public activity. */
 const AP_PUBLIC = 'https://www.w3.org/ns/activitystreams#Public';
@@ -278,25 +281,28 @@ export interface DeliveryServiceConfig<TActor extends DeliveryActorFields> {
   isBlockedDomain(host: string): boolean;
 }
 
-/** Read a bounded prefix of a failed-delivery response body for the debug log. */
+/**
+ * Read a bounded prefix of a failed-delivery response body for the debug log,
+ * within {@link DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS}: an inbox that stalls its
+ * body after the headers costs the worker that long, not forever.
+ */
 async function readResponsePreview(response: DeliveryResponseStream): Promise<string> {
-  const chunks: Buffer[] = [];
-  let totalBytes = 0;
-
+  const iterator = response[Symbol.asyncIterator]();
   try {
-    for await (const chunk of response) {
-      const buffer = Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk);
-      totalBytes += buffer.byteLength;
-      chunks.push(buffer);
-      if (totalBytes >= DELIVERY_RESPONSE_PREVIEW_MAX_BYTES) break;
-    }
+    const preview = await readBodyChunks(
+      { read: () => iterator.next(), cancel: () => response.destroy() },
+      {
+        maxBytes: DELIVERY_RESPONSE_PREVIEW_MAX_BYTES,
+        timeoutMs: DELIVERY_RESPONSE_PREVIEW_TIMEOUT_MS,
+        overflow: 'truncate',
+      },
+    );
+    return preview.toString('utf8');
   } catch {
     return '';
   } finally {
     response.destroy();
   }
-
-  return Buffer.concat(chunks).toString('utf8', 0, DELIVERY_RESPONSE_PREVIEW_MAX_BYTES);
 }
 
 /** The outbound delivery + follow-lifecycle service. */