@oxy.so/federation 2.2.0 → 2.3.0

package/src/node/actorResolver.ts

@@ -314,9 +314,11 @@ function sameOriginUrl(a: string, b: string): boolean {
   try {
     const urlA = new URL(a);
     const urlB = new URL(b);
-    return urlA.protocol === urlB.protocol
-      && urlA.port === urlB.port
-      && isSameFederationHost(urlA.hostname, urlB.hostname);
+    return (
+      urlA.protocol === urlB.protocol &&
+      urlA.port === urlB.port &&
+      isSameFederationHost(urlA.hostname, urlB.hostname)
+    );
   } catch {
     return false;
   }
@@ -387,9 +389,7 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
     try {
       const data = await this.config.fetchWebFinger(url);
       if (!data) return null;
-      const link = data.links?.find(
-        (l) => l.rel === 'self' && isApActorContentType(l.type),
-      );
+      const link = data.links?.find((l) => l.rel === 'self' && isApActorContentType(l.type));
       return link?.href || null;
     } catch (err) {
       this.config.logger.warn(`WebFinger resolution failed for ${acct}:`, err);
@@ -419,7 +419,9 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
       // throws here and is handled by the catch below.
       const requestedHost = new URL(currentUri).hostname.toLowerCase();
       if (this.config.isBlockedDomain(requestedHost)) {
-        this.config.logger.info(`[FedSync] fetchRemoteActor skipping own/blocked domain ${requestedHost} for ${currentUri}`);
+        this.config.logger.info(
+          `[FedSync] fetchRemoteActor skipping own/blocked domain ${requestedHost} for ${currentUri}`,
+        );
         return null;
       }
 
@@ -431,20 +433,27 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
         // through to the WebFinger fallback (which recovers a STALE/wrong URI on a
         // transient failure, not a permanent removal). Only 410 does this.
         if (res.status === 410) {
-          this.config.logger.info(`[FedSync] fetchRemoteActor 410 Gone for ${currentUri} — tombstoning actor`);
+          this.config.logger.info(
+            `[FedSync] fetchRemoteActor 410 Gone for ${currentUri} — tombstoning actor`,
+          );
           await this.tombstoneGoneActor(currentUri);
           return null;
         }
         const body = await readBoundedResponseBody(res, ERROR_BODY_MAX_BYTES).catch(() => '');
-        this.config.logger.info(`[FedSync] fetchRemoteActor HTTP ${res.status} ${res.statusText} for ${currentUri} body=${body.slice(0, 500)}`);
+        this.config.logger.info(
+          `[FedSync] fetchRemoteActor HTTP ${res.status} ${res.statusText} for ${currentUri} body=${body.slice(0, 500)}`,
+        );
 
         // If direct fetch failed, try WebFinger to resolve the canonical actor URI.
         // Some servers (e.g., Threads) use numeric IDs in AP URIs that differ from
         // the username-based URI we may have stored.
         const parsed = new URL(currentUri);
         const pathUsername = parsed.pathname.split('/').filter(Boolean).pop();
-        const acct = canonicalAcctHint
-          || (pathUsername ? this.config.normalizeFederatedAcct(`${pathUsername}@${parsed.hostname}`) : undefined);
+        const acct =
+          canonicalAcctHint ||
+          (pathUsername
+            ? this.config.normalizeFederatedAcct(`${pathUsername}@${parsed.hostname}`)
+            : undefined);
         if (acct) {
           this.config.logger.info(`[FedSync] attempting WebFinger fallback for ${acct}`);
           const resolved = await this.resolveWebFinger(acct);
@@ -457,16 +466,24 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
               // A 410 on the WebFinger-RESOLVED URI is just as definitive. Tombstone
               // against the stored URI (not reassigned on this branch).
               if (res.status === 410) {
-                this.config.logger.info(`[FedSync] fetchRemoteActor 410 Gone for resolved ${resolved} — tombstoning actor ${currentUri}`);
+                this.config.logger.info(
+                  `[FedSync] fetchRemoteActor 410 Gone for resolved ${resolved} — tombstoning actor ${currentUri}`,
+                );
                 await this.tombstoneGoneActor(currentUri);
                 return null;
               }
-              const body2 = await readBoundedResponseBody(res, ERROR_BODY_MAX_BYTES).catch(() => '');
-              this.config.logger.info(`[FedSync] fetchRemoteActor HTTP ${res.status} for resolved ${resolved} body=${body2.slice(0, 500)}`);
+              const body2 = await readBoundedResponseBody(res, ERROR_BODY_MAX_BYTES).catch(
+                () => '',
+              );
+              this.config.logger.info(
+                `[FedSync] fetchRemoteActor HTTP ${res.status} for resolved ${resolved} body=${body2.slice(0, 500)}`,
+              );
               return null;
             }
           } else {
-            this.config.logger.info(`[FedSync] WebFinger returned ${resolved ?? 'null'} for ${acct}`);
+            this.config.logger.info(
+              `[FedSync] WebFinger returned ${resolved ?? 'null'} for ${acct}`,
+            );
             return null;
           }
         } else {
@@ -478,42 +495,53 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
       const actorId = asString(actor.id);
       const actorInbox = asString(actor.inbox);
       if (!actorId || !actorInbox) {
-        this.config.logger.info(`[FedSync] fetchRemoteActor missing fields for ${currentUri}: id=${!!actor.id} inbox=${!!actor.inbox} type=${String(actor.type)} keys=${Object.keys(actor).join(',')}`);
+        this.config.logger.info(
+          `[FedSync] fetchRemoteActor missing fields for ${currentUri}: id=${!!actor.id} inbox=${!!actor.inbox} type=${String(actor.type)} keys=${Object.keys(actor).join(',')}`,
+        );
         return null;
       }
 
       if (!sameOriginUrl(currentUri, actorId)) {
-        this.config.logger.warn(`[FedSync] rejecting actor ${currentUri}: fetched URI is not authoritative for claimed id ${actorId}`);
+        this.config.logger.warn(
+          `[FedSync] rejecting actor ${currentUri}: fetched URI is not authoritative for claimed id ${actorId}`,
+        );
         return null;
       }
 
       if (!actorPublicKeyIsSelfConsistent(actor, actorId)) {
-        this.config.logger.warn(`[FedSync] rejecting actor ${currentUri}: publicKey is not self-consistent for claimed id ${actorId}`);
+        this.config.logger.warn(
+          `[FedSync] rejecting actor ${currentUri}: publicKey is not self-consistent for claimed id ${actorId}`,
+        );
         return null;
       }
 
       const actorHost = new URL(actorId).hostname.toLowerCase();
-      const username = this.config.text.inlineField(actor.preferredUsername)
-        || this.config.text.inlineField(actor.name)
-        || 'unknown';
-      const actorWebfinger = typeof actor.webfinger === 'string'
-        ? this.config.normalizeFederatedAcct(actor.webfinger)
-        : undefined;
+      const username =
+        this.config.text.inlineField(actor.preferredUsername) ||
+        this.config.text.inlineField(actor.name) ||
+        'unknown';
+      const actorWebfinger =
+        typeof actor.webfinger === 'string'
+          ? this.config.normalizeFederatedAcct(actor.webfinger)
+          : undefined;
       const verifiedAcctHint = this.acctMatchesActorHost(canonicalAcctHint, actorHost)
         ? canonicalAcctHint
         : undefined;
       const verifiedActorWebfinger = this.acctMatchesActorHost(actorWebfinger, actorHost)
         ? actorWebfinger
         : undefined;
-      const acct = verifiedAcctHint
-        || verifiedActorWebfinger
-        || this.config.normalizeFederatedAcct(`${username}@${actorHost}`)
-        || `${username.toLowerCase()}@${actorHost}`;
+      const acct =
+        verifiedAcctHint ||
+        verifiedActorWebfinger ||
+        this.config.normalizeFederatedAcct(`${username}@${actorHost}`) ||
+        `${username.toLowerCase()}@${actorHost}`;
       const domain = this.config.domainFromAcct(acct) || actorHost;
       // Re-check against the RESOLVED host/acct (post-redirect / WebFinger), which
       // can differ from the originally-requested URI host the early guard screened.
       if (this.config.isBlockedDomain(domain) || this.config.isBlockedDomain(actorHost)) {
-        this.config.logger.info(`[FedSync] fetchRemoteActor blocked domain ${domain} actorHost=${actorHost} for ${currentUri}`);
+        this.config.logger.info(
+          `[FedSync] fetchRemoteActor blocked domain ${domain} actorHost=${actorHost} for ${currentUri}`,
+        );
         return null;
       }
 
@@ -536,9 +564,10 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
           const attRecord = asRecord(att);
           if (!attRecord || attRecord.type !== 'PropertyValue') continue;
           const fieldName = this.config.text.inlineField(attRecord.name);
-          const fieldValue = typeof attRecord.value === 'string'
-            ? this.config.text.sanitizeFieldValue(attRecord.value)
-            : '';
+          const fieldValue =
+            typeof attRecord.value === 'string'
+              ? this.config.text.sanitizeFieldValue(attRecord.value)
+              : '';
           if (!fieldName || !fieldValue) continue;
           fields.push({
             name: fieldName,
@@ -552,7 +581,8 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
       const headerUrl = this.config.firstStringUrl(actor.image);
       // `summary` is the actor's bio — a BODY, so its line breaks are the author's
       // and must survive; `htmlToPlainText` normalizes it as multiline.
-      const summary = typeof actor.summary === 'string' ? this.config.text.htmlToPlainText(actor.summary) : '';
+      const summary =
+        typeof actor.summary === 'string' ? this.config.text.htmlToPlainText(actor.summary) : '';
       // The display name is one line. Entity-decode FIRST (an encoded `&#10;` or
       // `&nbsp;` only becomes whitespace once decoded), THEN collapse.
       const rawDisplayName = typeof actor.name === 'string' ? actor.name : '';
@@ -617,7 +647,8 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
         featuredTagsUrl: asString(actor.featuredTags) || undefined,
         alsoKnownAs,
         networkAcct: networkIdentity?.federatedUsername,
-        remoteCreatedAt: typeof actor.published === 'string' ? new Date(actor.published) : undefined,
+        remoteCreatedAt:
+          typeof actor.published === 'string' ? new Date(actor.published) : undefined,
         // Omitted, not written as `undefined`, when this refresh could not tell:
         // an absent key is what tells the store to keep the value it has.
         ...(followersCount !== undefined && { followersCount }),
@@ -663,7 +694,9 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
             postsCount: postsCount ?? undefined,
             oxyUserId: fedActor.oxyUserId ?? undefined,
           };
-          const oxyId = await this.config.identity.resolveExternalUser(normalized, { forceAvatarRefresh });
+          const oxyId = await this.config.identity.resolveExternalUser(normalized, {
+            forceAvatarRefresh,
+          });
           if (oxyId && fedActor.oxyUserId !== oxyId && fedActor._id != null) {
             await this.config.store.setActorOxyUserId(fedActor._id, oxyId);
           }
@@ -691,9 +724,7 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
    * actor. The refusal is logged: it is a bug in the app's rule, not a normal
    * outcome, and it must not pass silently.
    */
-  private resolveNetworkIdentity(
-    candidate: NetworkIdentityCandidate,
-  ): NetworkIdentity | undefined {
+  private resolveNetworkIdentity(candidate: NetworkIdentityCandidate): NetworkIdentity | undefined {
     const derived = this.config.deriveNetworkIdentity?.(candidate);
     if (!derived) return undefined;
 
@@ -707,13 +738,13 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
     const atIndex = federatedUsername.indexOf('@');
     const localPart = atIndex > 0 ? federatedUsername.slice(0, atIndex) : '';
     if (
-      domain.length === 0
-      || localPart.length === 0
-      || federatedUsername !== `${localPart}@${domain}`
+      domain.length === 0 ||
+      localPart.length === 0 ||
+      federatedUsername !== `${localPart}@${domain}`
     ) {
       this.config.logger.warn(
-        `[FedSync] refusing network identity for ${candidate.actorUri}: `
-        + `"${derived.federatedUsername}" is not bindable to domain "${derived.instanceDomain}"`,
+        `[FedSync] refusing network identity for ${candidate.actorUri}: ` +
+          `"${derived.federatedUsername}" is not bindable to domain "${derived.instanceDomain}"`,
       );
       return undefined;
     }
@@ -733,7 +764,9 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
     try {
       const actor = await this.config.store.tombstoneActor(actorUri);
       if (!actor) {
-        this.config.logger.info(`[FedSync] 410 Gone for ${actorUri} — no stored actor row to tombstone`);
+        this.config.logger.info(
+          `[FedSync] 410 Gone for ${actorUri} — no stored actor row to tombstone`,
+        );
         return;
       }
 
@@ -741,7 +774,9 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
 
       if (actor.oxyUserId) {
         const outcome = await this.config.identity.reportActorGone(actor.oxyUserId);
-        this.config.logger.info(`[FedSync] actor-gone report for ${actorUri} (oxyUserId ${actor.oxyUserId}) → ${outcome}`);
+        this.config.logger.info(
+          `[FedSync] actor-gone report for ${actorUri} (oxyUserId ${actor.oxyUserId}) → ${outcome}`,
+        );
       }
     } catch (err) {
       this.config.logger.warn(`[FedSync] failed to tombstone gone actor ${actorUri}:`, err);
@@ -793,12 +828,15 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
     // instance that has ever reached us), blocking its domain changed nothing. That
     // made the blocklist inert for exactly the hosts it is added for.
     if (this.isBlockedActorUri(actorUri)) {
-      this.config.logger.info(`[FedSync] getOrFetchActor refusing own/blocked domain for ${actorUri}`);
+      this.config.logger.info(
+        `[FedSync] getOrFetchActor refusing own/blocked domain for ${actorUri}`,
+      );
       return null;
     }
     const existing = await this.config.store.findActorByUri(actorUri);
     if (existing) {
-      const isStale = !existing.lastFetchedAt || Date.now() - existing.lastFetchedAt.getTime() > ACTOR_STALE_MS;
+      const isStale =
+        !existing.lastFetchedAt || Date.now() - existing.lastFetchedAt.getTime() > ACTOR_STALE_MS;
       if (isStale) {
         // Refresh in the background — never block the caller on remote I/O.
         this.refreshActorInBackground(actorUri, existing);
@@ -820,8 +858,9 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
 
     const missingProfile = !existing || !existing.avatarUrl || !existing.headerUrl;
     const lastFetchedMs = existing?.lastFetchedAt?.getTime();
-    const refreshedRecently = typeof lastFetchedMs === 'number'
-      && Date.now() - lastFetchedMs < ACTOR_REFRESH_MIN_INTERVAL_MS;
+    const refreshedRecently =
+      typeof lastFetchedMs === 'number' &&
+      Date.now() - lastFetchedMs < ACTOR_REFRESH_MIN_INTERVAL_MS;
 
     // Skip if we refreshed recently AND the cached profile is already complete.
     if (refreshedRecently && !missingProfile) return;
@@ -835,7 +874,9 @@ export class ActorResolver<TActor extends FederatedActorRecordBase> {
         await this.fetchRemoteActor(actorUri, forceAvatarRefresh, existing?.acct);
       } catch (err) {
         const message = err instanceof Error ? err.message : String(err);
-        this.config.logger.warn(`[FedSync] background actor refresh failed for ${actorUri}: ${message}`);
+        this.config.logger.warn(
+          `[FedSync] background actor refresh failed for ${actorUri}: ${message}`,
+        );
       } finally {
         this.inFlightActorRefreshes.delete(actorUri);
       }

package/src/node/actorRouter.ts

@@ -104,9 +104,15 @@ export interface ActorRouterConfig {
     /** Whether to trust `X-Forwarded-Host` when reconstructing the signed host line. */
     trustForwardedHost: boolean;
     /** Enqueue a verified inbound activity for async processing (false ⇒ process inline). */
-    enqueueInboxActivity(job: { activity: Record<string, unknown>; verifiedActorUri: string }): Promise<boolean>;
+    enqueueInboxActivity(job: {
+      activity: Record<string, unknown>;
+      verifiedActorUri: string;
+    }): Promise<boolean>;
     /** The inbound dispatcher (the inline-fallback + post-enqueue processor). */
-    processInboxActivity(activity: Record<string, unknown>, verifiedActorUri: string): Promise<void>;
+    processInboxActivity(
+      activity: Record<string, unknown>,
+      verifiedActorUri: string,
+    ): Promise<void>;
   };
   /** Fetch one page of a user's Oxy follow graph (followers OR following). */
   fetchFollowPage(
@@ -161,7 +167,11 @@ export function createActorRouter(config: ActorRouterConfig): Router {
   async function handleInbox(req: InboxRequest, res: Response): Promise<Response> {
     try {
       // Verify HTTP signature (use originalUrl to avoid proxy path mangling).
-      const { verified, actorUri, reason: signatureError } = await verifyHttpSignature(
+      const {
+        verified,
+        actorUri,
+        reason: signatureError,
+      } = await verifyHttpSignature(
         {
           method: req.method,
           path: req.originalUrl || req.path,
@@ -186,7 +196,8 @@ export function createActorRouter(config: ActorRouterConfig): Router {
       }
 
       // Verify the actor in the activity matches the signature.
-      const activityActor = typeof activity.actor === 'string' ? activity.actor : activity.actor?.id;
+      const activityActor =
+        typeof activity.actor === 'string' ? activity.actor : activity.actor?.id;
       if (activityActor !== actorUri) {
         logger.debug(`Inbox: Actor mismatch. Signed: ${actorUri}, Activity: ${activityActor}`);
         return res.status(403).json({ error: 'Actor mismatch' });
@@ -199,7 +210,10 @@ export function createActorRouter(config: ActorRouterConfig): Router {
       // is never dropped.
       let enqueued = false;
       try {
-        enqueued = await config.inbound.enqueueInboxActivity({ activity, verifiedActorUri: actorUri });
+        enqueued = await config.inbound.enqueueInboxActivity({
+          activity,
+          verifiedActorUri: actorUri,
+        });
       } catch (err) {
         logger.error('Failed to enqueue inbox activity — processing inline:', err);
         enqueued = false;
@@ -243,7 +257,8 @@ export function createActorRouter(config: ActorRouterConfig): Router {
 
       const userId = String(user._id || user.id);
 
-      const rawCount: unknown = direction === 'followers' ? user._count?.followers : user._count?.following;
+      const rawCount: unknown =
+        direction === 'followers' ? user._count?.followers : user._count?.following;
       const profileTotal = typeof rawCount === 'number' ? rawCount : undefined;
 
       if (!page) {
@@ -256,7 +271,9 @@ export function createActorRouter(config: ActorRouterConfig): Router {
             totalItems = (await config.fetchFollowPage(userId, direction, 0, 1)).total;
           } catch (err) {
             logger.warn('[Federation] follow-collection summary total lookup failed', {
-              username, direction, error: err,
+              username,
+              direction,
+              error: err,
             });
           }
         }
@@ -277,7 +294,12 @@ export function createActorRouter(config: ActorRouterConfig): Router {
       let total = profileTotal ?? 0;
       let hasMore = false;
       try {
-        const pageResult = await config.fetchFollowPage(userId, direction, offset, FOLLOW_PAGE_SIZE);
+        const pageResult = await config.fetchFollowPage(
+          userId,
+          direction,
+          offset,
+          FOLLOW_PAGE_SIZE,
+        );
         members = pageResult.members;
         total = pageResult.total;
         hasMore = pageResult.hasMore;
@@ -285,7 +307,10 @@ export function createActorRouter(config: ActorRouterConfig): Router {
         // Fail-soft: never 500 the whole collection on an Oxy graph hiccup — serve
         // an empty page against the best-known total rather than crashing.
         logger.warn('[Federation] follow-collection Oxy graph list failed, serving empty page', {
-          username, direction, offset, error: err,
+          username,
+          direction,
+          offset,
+          error: err,
         });
       }
 
@@ -293,9 +318,10 @@ export function createActorRouter(config: ActorRouterConfig): Router {
         .map((member) => memberActorUri(member, urls))
         .filter((uri): uri is string => uri !== null);
 
-      const pageId = offset > 0
-        ? `${collectionUrl(username)}?page=true&offset=${offset}`
-        : `${collectionUrl(username)}?page=true`;
+      const pageId =
+        offset > 0
+          ? `${collectionUrl(username)}?page=true&offset=${offset}`
+          : `${collectionUrl(username)}?page=true`;
 
       const pageResponse: Record<string, unknown> = {
         '@context': AP_CONTEXT,

package/src/node/delivery.ts

@@ -81,7 +81,10 @@ export interface DeliverSingleHopInit {
  * response WITHOUT following redirects. Mention adapts its `@oxy.so/core/server`-
  * backed `fetchUpstreamSingleHop` into this shape.
  */
-export type DeliverSingleHop = (url: string, init: DeliverSingleHopInit) => Promise<DeliverSingleHopResult>;
+export type DeliverSingleHop = (
+  url: string,
+  init: DeliverSingleHopInit,
+) => Promise<DeliverSingleHopResult>;
 
 /** A durable-delivery job body (BullMQ + the fallback queue share this shape). */
 export interface DeliveryQueueJob {
@@ -94,9 +97,8 @@ export interface DeliveryQueueJob {
  * The durable-delivery fallback (written when BullMQ is unavailable).
  *
  * App-supplied, like every other store in this package: the engine never names
- * a database. Mention backs it with Postgres; the method names below are the
- * ones its original Mongoose collection exposed and are kept only so the
- * adapter shape stays stable for consumers.
+ * a database. Mention backs it with Postgres; the method names below are kept
+ * stable so the adapter shape stays stable for consumers.
  */
 export interface DeliveryFallbackQueue {
   /** Insert one fallback delivery row (`queueDelivery`). */
@@ -134,7 +136,9 @@ export interface DeliveryActorStore<TActor extends DeliveryActorFields> {
   /** One cached actor by uri (`resolveActorInbox` / `sendFollow` / `sendUndoFollow` / `sendAccept`). */
   findActorByUri(uri: string): Promise<TActor | null>;
   /** Inbox fields for many actor uris (`deliverToFollowers`, step 2). */
-  findActorInboxesByUris(uris: string[]): Promise<Array<Pick<DeliveryActorFields, 'sharedInboxUrl' | 'inboxUrl'>>>;
+  findActorInboxesByUris(
+    uris: string[],
+  ): Promise<Array<Pick<DeliveryActorFields, 'sharedInboxUrl' | 'inboxUrl'>>>;
 }
 
 /** Bring-your-own-store: the AP follow records stay in the app DB behind this adapter. */
@@ -142,9 +146,16 @@ export interface DeliveryFollowStore {
   /** Accepted inbound followers' remote actor uris (`deliverToFollowers`, step 1). */
   listAcceptedInboundFollowerActorUris(localOxyUserId: string): Promise<string[]>;
   /** Upsert an outbound pending follow with its activity id (`sendFollow`). */
-  upsertOutboundPending(localOxyUserId: string, remoteActorUri: string, activityId: string): Promise<void>;
+  upsertOutboundPending(
+    localOxyUserId: string,
+    remoteActorUri: string,
+    activityId: string,
+  ): Promise<void>;
   /** The outbound follow row for `(localOxyUserId, remoteActorUri)` (`sendUndoFollow`). */
-  findOutbound(localOxyUserId: string, remoteActorUri: string): Promise<{ _id: unknown; activityId?: string } | null>;
+  findOutbound(
+    localOxyUserId: string,
+    remoteActorUri: string,
+  ): Promise<{ _id: unknown; activityId?: string } | null>;
   /** Delete a follow row by id (`sendUndoFollow`). */
   deleteById(id: unknown): Promise<void>;
 }
@@ -356,7 +367,9 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
         ...sigHeaders,
       };
 
-      logger.debug(`[FedDeliver] POST ${targetInbox} body=${body} sig-headers=${sigHeaders.Signature?.match(/headers="([^"]+)"/)?.[1]}`);
+      logger.debug(
+        `[FedDeliver] POST ${targetInbox} body=${body} sig-headers=${sigHeaders.Signature?.match(/headers="([^"]+)"/)?.[1]}`,
+      );
 
       const { response, status } = await config.deliverSingleHop(targetInbox, {
         method: 'POST',
@@ -372,7 +385,9 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
       }
 
       const responseBody = await readResponsePreview(response);
-      logger.debug(`Activity delivery failed to ${targetInbox}: ${status} body=${responseBody.slice(0, 500)}`);
+      logger.debug(
+        `Activity delivery failed to ${targetInbox}: ${status} body=${responseBody.slice(0, 500)}`,
+      );
       return false;
     } catch (err) {
       logger.debug(`Activity delivery error to ${targetInbox}:`, err);
@@ -402,7 +417,9 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
       .enqueueDelivery({ activityJson: activity, targetInbox, senderOxyUserId })
       .catch((err) => {
         const message = err instanceof Error ? err.message : String(err);
-        logger.warn(`[FedDeliver] enqueue failed for ${targetInbox}, falling back to the durable queue: ${message}`);
+        logger.warn(
+          `[FedDeliver] enqueue failed for ${targetInbox}, falling back to the durable queue: ${message}`,
+        );
         return false;
       });
 
@@ -473,12 +490,19 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
         .enqueueDelivery({ activityJson: activity, targetInbox: inbox, senderOxyUserId })
         .catch((err) => {
           const message = err instanceof Error ? err.message : String(err);
-          logger.warn(`[FedDeliver] follower enqueue failed for ${inbox}, falling back to the durable queue: ${message}`);
+          logger.warn(
+            `[FedDeliver] follower enqueue failed for ${inbox}, falling back to the durable queue: ${message}`,
+          );
           return false;
         });
 
       if (!enqueued) {
-        durableFallback.push({ activityJson: activity, targetInbox: inbox, senderOxyUserId, nextAttemptAt: now });
+        durableFallback.push({
+          activityJson: activity,
+          targetInbox: inbox,
+          senderOxyUserId,
+          nextAttemptAt: now,
+        });
       }
     }
 
@@ -512,7 +536,9 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
         }
       } catch (err) {
         const message = err instanceof Error ? err.message : String(err);
-        logger.warn(`[FedSync] deferred follow delivery setup failed for ${remoteActorUri}: ${message}`);
+        logger.warn(
+          `[FedSync] deferred follow delivery setup failed for ${remoteActorUri}: ${message}`,
+        );
       }
     })();
   }
@@ -541,9 +567,7 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
     const localActorUri = urls.actor(localUsername);
     // Use the actor _id when known, otherwise a stable hash of the URI so the
     // activity ID is deterministic across retries before the actor is cached.
-    const activityIdSuffix = cached?._id
-      ? String(cached._id)
-      : encodeURIComponent(canonicalUri);
+    const activityIdSuffix = cached?._id ? String(cached._id) : encodeURIComponent(canonicalUri);
     const activityId = `${localActorUri}/follows/${activityIdSuffix}`;
 
     // Create or update the follow record
@@ -567,7 +591,9 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
         })
         .catch((err) => {
           const message = err instanceof Error ? err.message : String(err);
-          logger.warn(`[FedSync] background follow delivery failed for ${canonicalUri}: ${message}`);
+          logger.warn(
+            `[FedSync] background follow delivery failed for ${canonicalUri}: ${message}`,
+          );
         });
     } else {
       // No cached inbox yet — resolve the actor's inbox in the background and
@@ -623,10 +649,14 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
         })
         .catch((err) => {
           const message = err instanceof Error ? err.message : String(err);
-          logger.warn(`[FedSync] background undo-follow delivery failed for ${remoteActorUri}: ${message}`);
+          logger.warn(
+            `[FedSync] background undo-follow delivery failed for ${remoteActorUri}: ${message}`,
+          );
         });
     } else if (targetInbox) {
-      logger.warn(`[FedDeliver] skipping Undo(Follow) delivery to blocked origin ${remoteActorUri}`);
+      logger.warn(
+        `[FedDeliver] skipping Undo(Follow) delivery to blocked origin ${remoteActorUri}`,
+      );
     }
 
     return true;
@@ -649,7 +679,9 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
       return;
     }
     if (isBlockedUrl(targetInbox) || isBlockedUrl(remoteActorUri)) {
-      logger.warn(`[FedDeliver] refusing Accept(Follow) delivery to blocked origin ${remoteActorUri}`);
+      logger.warn(
+        `[FedDeliver] refusing Accept(Follow) delivery to blocked origin ${remoteActorUri}`,
+      );
       return;
     }
 
@@ -681,7 +713,9 @@ export function createDeliveryService<TActor extends DeliveryActorFields>(
     try {
       const user = await config.identity.resolveUserByUsername(username);
       if (!user) {
-        logger.warn(`[FedDeliver] cannot federate actor update for ${username}: user not resolvable`);
+        logger.warn(
+          `[FedDeliver] cannot federate actor update for ${username}: user not resolvable`,
+        );
         return;
       }