@rmdes/indiekit-endpoint-activitypub 3.4.0 → 3.5.0

package/lib/inbox-handlers.js

@@ -984,6 +984,24 @@ export async function handleUpdate(item, collections, ctx, handle) {
 
   const existing = await collections.ap_followers.findOne({ actorUrl });
   if (existing) {
+    let updatedAvatar = "";
+    try {
+      updatedAvatar = actorObj.icon
+        ? (await actorObj.icon)?.url?.href || ""
+        : "";
+    } catch {
+      // Icon fetch failed
+    }
+
+    let updatedBanner = "";
+    try {
+      updatedBanner = actorObj.image
+        ? (await actorObj.image)?.url?.href || ""
+        : "";
+    } catch {
+      // Banner fetch failed
+    }
+
     await collections.ap_followers.updateOne(
       { actorUrl },
       {
@@ -993,9 +1011,8 @@ export async function handleUpdate(item, collections, ctx, handle) {
             actorObj.preferredUsername?.toString() ||
             actorUrl,
           handle: actorObj.preferredUsername?.toString() || "",
-          avatar: actorObj.icon
-            ? (await actorObj.icon)?.url?.href || ""
-            : "",
+          avatar: updatedAvatar,
+          banner: updatedBanner,
           updatedAt: new Date().toISOString(),
         },
       },

package/lib/inbox-listeners.js

@@ -66,13 +66,30 @@ export function registerInboxListeners(inboxChain, options) {
         followerActor.preferredUsername?.toString() ||
         followerUrl;
 
+      let followerAvatar = "";
+      try {
+        followerAvatar = followerActor.icon
+          ? (await followerActor.icon)?.url?.href || ""
+          : "";
+      } catch {
+        // Icon fetch failed
+      }
+
+      let followerBanner = "";
+      try {
+        followerBanner = followerActor.image
+          ? (await followerActor.image)?.url?.href || ""
+          : "";
+      } catch {
+        // Banner fetch failed
+      }
+
       const followerData = {
         actorUrl: followerUrl,
         handle: followerActor.preferredUsername?.toString() || "",
         name: followerName,
-        avatar: followerActor.icon
-          ? (await followerActor.icon)?.url?.href || ""
-          : "",
+        avatar: followerAvatar,
+        banner: followerBanner,
         inbox: followerActor.inbox?.id?.href || "",
         sharedInbox: followerActor.endpoints?.sharedInbox?.href || "",
       };

package/lib/mastodon/entities/notification.js

@@ -55,17 +55,26 @@ export function serializeNotification(notif, { baseUrl, statusMap, interactionSt
   );
 
   // Resolve the associated status (for favourite, reblog, mention types)
+  // For mention types, prefer the triggering post (notif.url) over the target post (notif.targetUrl)
+  // because targetUrl for replies points to the user's OWN post being replied to
   let status = null;
-  if (notif.targetUrl && statusMap) {
-    const timelineItem = statusMap.get(notif.targetUrl);
-    if (timelineItem) {
-      status = serializeStatus(timelineItem, {
-        baseUrl,
-        favouritedIds: interactionState?.favouritedIds || new Set(),
-        rebloggedIds: interactionState?.rebloggedIds || new Set(),
-        bookmarkedIds: interactionState?.bookmarkedIds || new Set(),
-        pinnedIds: new Set(),
-      });
+  if (statusMap) {
+    const isMentionType = mastodonType === "mention";
+    const lookupUrl = isMentionType
+      ? (notif.url || notif.targetUrl)
+      : (notif.targetUrl || notif.url);
+
+    if (lookupUrl) {
+      const timelineItem = statusMap.get(lookupUrl);
+      if (timelineItem) {
+        status = serializeStatus(timelineItem, {
+          baseUrl,
+          favouritedIds: interactionState?.favouritedIds || new Set(),
+          rebloggedIds: interactionState?.rebloggedIds || new Set(),
+          bookmarkedIds: interactionState?.bookmarkedIds || new Set(),
+          pinnedIds: new Set(),
+        });
+      }
     }
   }
 
@@ -82,7 +91,7 @@ export function serializeNotification(notif, { baseUrl, statusMap, interactionSt
       visibility: notif.type === "dm" ? "direct" : "public",
       language: null,
       uri: notif.uid || "",
-      url: notif.targetUrl || notif.uid || "",
+      url: notif.url || notif.targetUrl || notif.uid || "",
       replies_count: 0,
       reblogs_count: 0,
       favourites_count: 0,

package/lib/mastodon/routes/accounts.js

@@ -110,7 +110,7 @@ router.get("/api/v1/accounts/lookup", async (req, res, next) => {
     if (follower) {
       return res.json(
         serializeAccount(
-          { name: follower.name, url: follower.actorUrl, photo: follower.avatar, handle: follower.handle },
+          { name: follower.name, url: follower.actorUrl, photo: follower.avatar, handle: follower.handle, bannerUrl: follower.banner || "" },
           { baseUrl },
         ),
       );
@@ -277,7 +277,7 @@ router.get("/api/v1/accounts/:id/followers", async (req, res, next) => {
 
     const accounts = followers.map((f) =>
       serializeAccount(
-        { name: f.name, url: f.actorUrl, photo: f.avatar, handle: f.handle },
+        { name: f.name, url: f.actorUrl, photo: f.avatar, handle: f.handle, bannerUrl: f.banner || "" },
         { baseUrl },
       ),
     );
@@ -310,7 +310,7 @@ router.get("/api/v1/accounts/:id/following", async (req, res, next) => {
 
     const accounts = following.map((f) =>
       serializeAccount(
-        { name: f.name, url: f.actorUrl, photo: f.avatar, handle: f.handle },
+        { name: f.name, url: f.actorUrl, photo: f.avatar, handle: f.handle, bannerUrl: f.banner || "" },
         { baseUrl },
       ),
     );
@@ -712,23 +712,35 @@ async function resolveActorUrl(id, collections) {
  * Returns { actor, actorUrl } or { actor: null, actorUrl: null }.
  */
 async function resolveActorData(id, collections) {
-  // Check followers
+  // Check followers — pass through all stored fields for richer serialization
   const followers = await collections.ap_followers.find({}).toArray();
   for (const f of followers) {
     if (remoteActorId(f.actorUrl) === id) {
       return {
-        actor: { name: f.name, url: f.actorUrl, photo: f.avatar, handle: f.handle },
+        actor: {
+          name: f.name,
+          url: f.actorUrl,
+          photo: f.avatar,
+          handle: f.handle,
+          bannerUrl: f.banner || "",
+        },
         actorUrl: f.actorUrl,
       };
     }
   }
 
-  // Check following
+  // Check following — pass through all stored fields
   const following = await collections.ap_following.find({}).toArray();
   for (const f of following) {
     if (remoteActorId(f.actorUrl) === id) {
       return {
-        actor: { name: f.name, url: f.actorUrl, photo: f.avatar, handle: f.handle },
+        actor: {
+          name: f.name,
+          url: f.actorUrl,
+          photo: f.avatar,
+          handle: f.handle,
+          bannerUrl: f.banner || "",
+        },
         actorUrl: f.actorUrl,
       };
     }

package/lib/mastodon/routes/notifications.js

@@ -225,23 +225,25 @@ function resolveInternalTypes(mastodonTypes) {
 async function batchFetchStatuses(collections, notifications) {
   const statusMap = new Map();
 
-  const targetUrls = [
+  // Collect both targetUrl (the post being acted on) and url (the triggering post)
+  // For mention/reply notifications, the url is the actual mention/reply post
+  const allUrls = [
     ...new Set(
       notifications
-        .map((n) => n.targetUrl)
+        .flatMap((n) => [n.targetUrl, n.url])
         .filter(Boolean),
     ),
   ];
 
-  if (targetUrls.length === 0 || !collections.ap_timeline) {
+  if (allUrls.length === 0 || !collections.ap_timeline) {
     return statusMap;
   }
 
   const items = await collections.ap_timeline
     .find({
       $or: [
-        { uid: { $in: targetUrls } },
-        { url: { $in: targetUrls } },
+        { uid: { $in: allUrls } },
+        { url: { $in: allUrls } },
       ],
     })
     .toArray();

package/package.json

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