@rmdes/indiekit-endpoint-activitypub 1.0.14 → 1.0.16

package/lib/batch-refollow.js

@@ -233,23 +233,32 @@ async function processOneFollow(options, entry) {
       throw new Error("Could not resolve remote actor");
     }
 
-    // Send Follow activity
+    // Use the canonical actor URL (may differ from imported URL)
+    const canonicalUrl = remoteActor.id?.href || entry.actorUrl;
+
+    // Send Follow activity using canonical URL
     const follow = new Follow({
       actor: ctx.getActorUri(handle),
-      object: new URL(entry.actorUrl),
+      object: new URL(canonicalUrl),
     });
 
     await ctx.sendActivity({ identifier: handle }, remoteActor, follow);
 
-    // Mark as sent
+    // Mark as sent — update actorUrl to canonical form so Accept handler
+    // can match when the remote server responds
+    const updateFields = {
+      source: "refollow:sent",
+      refollowLastAttempt: new Date().toISOString(),
+      refollowError: null,
+    };
+    if (canonicalUrl !== entry.actorUrl) {
+      updateFields.actorUrl = canonicalUrl;
+    }
+
     await collections.ap_following.updateOne(
       { _id: entry._id },
       {
-        $set: {
-          source: "refollow:sent",
-          refollowLastAttempt: new Date().toISOString(),
-          refollowError: null,
-        },
+        $set: updateFields,
         $inc: { refollowAttempts: 1 },
       },
     );

package/lib/inbox-listeners.js

@@ -120,15 +120,15 @@ export function registerInboxListeners(inboxChain, options) {
       }
     })
     .on(Accept, async (ctx, accept) => {
-      // Handle Accept(Follow) — remote server accepted our Follow request
+      // Handle Accept(Follow) — remote server accepted our Follow request.
+      // We don't inspect the inner object type because Fedify often resolves
+      // it to a Person (the Follow's target) rather than the Follow itself.
+      // Instead, we match directly against ap_following — if we have a
+      // pending follow for this actor, any Accept from them confirms it.
       const actorObj = await accept.getActor();
       const actorUrl = actorObj?.id?.href || "";
       if (!actorUrl) return;
 
-      const inner = await accept.getObject();
-      if (!(inner instanceof Follow)) return;
-
-      // Match against our following list for refollow or microsub-reader follows
       const result = await collections.ap_following.findOneAndUpdate(
         {
           actorUrl,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rmdes/indiekit-endpoint-activitypub",
-  "version": "1.0.14",
+  "version": "1.0.16",
   "description": "ActivityPub federation endpoint for Indiekit via Fedify. Adds full fediverse support: actor, inbox, outbox, followers, following, syndication, and Mastodon migration.",
   "keywords": [
     "indiekit",