Fixed followers dispatcher and api endpoint

There was a bug in fedify with serialising Actors which mean we had to return
incompleter followers data, which broke our frontend, this has now been fixed
so we can update the followers dispatcher to return full Actor objects again

features/step_definitions/stepdefs.js

@@ -343,7 +343,7 @@ Then('{string} is in our Followers', async function (actorName) {
     const followers = await response.json();
     const actor = this.actors[actorName];
 
-    const found = followers.orderedItems.find(item => item === actor.id);
+    const found = this.followers.orderedItems.find(item => item.id === actor.id);
 
     assert(found);
 });
@@ -356,7 +356,7 @@ Then('{string} is in our Followers once only', async function (actorName) {
     });
     const followers = await response.json();
     const actor = this.actors[actorName];
-    const found = followers.orderedItems.filter(item => item === actor.id);
+    const found = this.followers.orderedItems.filter(item => item.id === actor.id);
 
     assert.equal(found.length, 1);
 });

src/dispatchers.ts

@@ -303,17 +303,6 @@ export async function inboxErrorHandler(
     console.error(error);
 }
 
-function convertJsonLdToRecipient(result: any): Recipient {
-    return {
-        ...result,
-        id: new URL(result.id),
-        inboxId: new URL(result.inbox),
-        endpoints: result.endpoints?.sharedInbox != null
-            ? { sharedInbox: new URL(result.endpoints.sharedInbox) }
-            : null,
-    };
-}
-
 export async function followersDispatcher(
     ctx: Context<ContextData>,
     handle: string,
@@ -322,14 +311,20 @@ export async function followersDispatcher(
     let items: Recipient[] = [];
     const fullResults = await ctx.data.db.get<any[]>(['followers', 'expanded']);
     if (fullResults) {
-        items = fullResults.map(convertJsonLdToRecipient)
+        items = await fullResults.reduce(async (list, item) => {
+            const obj = await APObject.fromJsonLd(item);
+            if (isActor(obj)) {
+                return (await list).concat(obj);
+            }
+            return list;
+        }, Promise.resolve([]));
     } else {
         const results = (await ctx.data.db.get<string[]>(['followers'])) || [];
         const actors = items = (await Promise.all(results.map((result) => lookupActor(ctx, result))))
             .filter((item): item is Actor => isActor(item))
         const toStore = await Promise.all(actors.map(actor => actor.toJsonLd() as any));
         await ctx.data.db.set(['followers', 'expanded'], toStore);
-        items = toStore.map(convertJsonLdToRecipient);
+        items = actors;
     }
     return {
         items,