Changeset ca25f360 in Klonkt

Timestamp:

06/25/2026 05:37:12 AM (3 months ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

2923a95

Parents:

4c9c60b

Message:

feat(fediverse): federate post edits as Update (Mastodon refreshes cached copy)

Editing an already-published, federated post now sends Update(Note) to followers
(with an 'updated' timestamp), so Mastodon refreshes the cached post instead of
keeping the stale original — e.g. fixes a post that federated before content
handling existed. New-publish still sends Create; fan_only never federates.

Co-Authored-By: Claude <noreply@…>

Location:

src

Files:

• routes/posts.js (modified) (1 diff)
• services/ActivityPubService.js (modified) (2 diffs)

src/routes/posts.js

@@ -354 +354 @@
   } catch (e) { /* FTS issues non-fatal */ }
 
-  // ActivityPub: federate when a post BECOMES published (draft/scheduled → published
-  // via the editor). A brand-new published post is handled in the create route.
-  if (finalStatus === 'published' && post.status !== 'published' && !fanOnly) {
-    ActivityPubService.deliverCreate(site, {
+  // ActivityPub: federate edits to followers. A post that BECOMES published →
+  // Create (new post); an already-published post that's edited → Update (so
+  // Mastodon refreshes its cached copy). fan_only posts don't federate.
+  if (finalStatus === 'published' && !fanOnly) {
+    const apPost = {
       id: post.id, slug: finalSlug, title: title || finalSlug,
       content: cleanContent, cover_image_url: cover_image_url || null,
       published_at: publishedAt, created_at: post.created_at,
-    }).catch(() => { /* best-effort */ });
+    };
+    if (post.status !== 'published') ActivityPubService.deliverCreate(site, apPost).catch(() => { /* best-effort */ });
+    else ActivityPubService.deliverUpdate(site, apPost).catch(() => { /* best-effort */ });
   }
 

src/services/ActivityPubService.js

@@ -598 +598 @@
   };
   for (const inbox of inboxes) deliverWithRetry(site.slug, inbox, del, `${me}#main-key`, keys.private_pem);
+}
+
+// Tell followers an already-published post changed (Update + edited Note) so
+// Mastodon refreshes the cached copy (e.g. after fixing content).
+export async function deliverUpdate(site, post) {
+  const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
+  if (!base || !site || !site.slug || !post || !post.id) return;
+  const followers = fStmts().list.all(site.slug);
+  if (!followers.length) return;
+  const inboxes = [...new Set(followers.map((f) => f.shared_inbox || f.inbox).filter(Boolean))];
+  const keys = getOrCreateKeys(site.slug);
+  const me = actorId(base, site.slug);
+  const note = buildNote(base, site, post);
+  note.updated = new Date().toISOString();
+  const update = {
+    '@context': 'https://www.w3.org/ns/activitystreams',
+    id: `${noteId(base, post.id)}#update-${Date.now()}`,
+    type: 'Update', actor: me, to: [PUBLIC], cc: [`${me}/followers`],
+    object: note,
+  };
+  for (const inbox of inboxes) deliverWithRetry(site.slug, inbox, update, `${me}#main-key`, keys.private_pem);
 }
 
@@ -930 +951 @@
   getOrCreateKeys, apWants, sendAP, actorId, noteId,
   buildActor, buildNote, buildCreate, buildOutbox, buildFollowers,
-  followerCount, deliver, fetchActor, verifyRequest, handleInbox, deliverCreate, deliverDelete,
+  followerCount, deliver, fetchActor, verifyRequest, handleInbox, deliverCreate, deliverDelete, deliverUpdate,
   getInteractions, getInteractionById, buildReplyNote, getOutboxNote, deliverReply, resolveRemoteNote,
   listOutbox, deliverOutboxDelete,