Changeset 4424af7 in Klonkt

Timestamp:

07/16/2026 12:20:51 PM (8 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

df9da7d

Parents:

ae05747

git-author:

Robin <roboburr@…> (07/12/2026 10:41:10 PM)

git-committer:

Robin <roboburr@…> (07/16/2026 12:20:51 PM)

Message:

Fix: linkify #hashtags and bare URLs in post bodies for on-site display

The website showed raw '#tag' and URL text in post bodies; only the
federated (Mastodon) copy got links, via buildNote. Add a shared
linkifyBody() that reuses the exact same linkHashtags()/linkUrls() rules and
apply it in the post render pipeline, so the site and the federated copy
agree. Idempotent (splits out existing <a>), so editor links and autoembeds
are never double-wrapped; hashtags point at the site's own /tag/<slug>.
@mention display-linking is deferred (needs a per-render webfinger lookup).

This is the first step of the compose/display pipeline unification; buildNote
should later call linkifyBody too so there is a single content pipeline.

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

Location:

src

Files:

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

src/routes/posts.js

@@ -1070 +1070 @@
     html = html.replace(/\[\[(track|album|playlist):[^\]]+\]\]/gi, '');
   }
+  // Linkify inline #hashtags and bare URLs for on-site display — same rules as the
+  // federated copy (buildNote). Was: only the Mastodon copy got links; the website
+  // showed raw "#tag"/URL text. Idempotent, so editor links + embeds are untouched.
+  html = ActivityPubService.linkifyBody('', html);
   post.content_html = html;
 

src/services/ActivityPubService.js

@@ -1422 +1422 @@
   return parts.join('');
 }
+// Linkify inline #hashtags and bare URLs in BODY html for on-site DISPLAY, using the
+// EXACT same rules as the federated copy (linkHashtags/linkUrls), so the website and the
+// Mastodon copy agree instead of the website showing raw text. Idempotent: existing
+// <a>…</a> (editor links, embeds, shortcode buttons) are split out and left untouched, so
+// nothing is double-wrapped. Pass base='' → root-relative /tag/<slug> links.
+export function linkifyBody(base, html) {
+  const withTags = String(html || '')
+    .split(/(<a\b[^>]*>[\s\S]*?<\/a>)/gi)
+    .map((seg) => (/^<a\b/i.test(seg) ? seg : linkHashtags(base, seg)))
+    .join('');
+  return linkUrls(withTags);
+}
+
 // Extract the AP Hashtag tag objects from already-linked reply content.
 function hashtagTags(base, content) {
@@ -2421 +2434 @@
   deliverWithRetry, enqueueDelivery, processDeliveryQueue, startDeliveryWorker,
   getReplyUris, markNotificationsSeen, countUnseenNotifications, hasPlayableAudio,
+  linkifyBody,
 };