Changeset d54dade in Klonkt for src/routes/posts.js

Timestamp:

06/15/2026 03:39:35 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

52badd3

Parents:

7f46817d

Message:

fix: hub mode — related posts + Newer/Older across ALL users

In hub mode "Related posts" and the Newer/Older navigation only pulled from
the current site; now from all sites combined (newest->oldest), with links to
/user/<site-slug>/<post-slug>. Cross-user links use full navigation (the
chrome/header differs per artist); solo + same-context keep the HTMX partial.
Pinned nav stays per-site but now gets the correct siteUrlBase prefix.
Solo mode unchanged.

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

File:

• src/routes/posts.js (modified) (2 diffs)

src/routes/posts.js

@@ -501 +501 @@
   // Prev / next chronological (kept for back-compat — "post-nav" feature
   // below the article still uses these as a simple linear navigation).
-  const prevPost = db.prepare(`
-    SELECT slug, title FROM posts
-    WHERE site_id = ? AND status = 'published' AND published_at < ? AND id != ?
-    ORDER BY published_at DESC LIMIT 1
-  `).get(site.id, post.published_at, post.id);
-
-  const nextPost = db.prepare(`
-    SELECT slug, title FROM posts
-    WHERE site_id = ? AND status = 'published' AND published_at > ? AND id != ?
-    ORDER BY published_at ASC LIMIT 1
-  `).get(site.id, post.published_at, post.id);
+  // Hub-modus: Gerelateerde posts + Newer/Older trekken uit ALLE users (alle
+  // sites), nieuwste->oudste. Solo-modus: binnen de huidige site (oud gedrag).
+  const isHub = res.locals.tenancy === 'hub';
+  // Per-post URL-basis: in hub wijst een link naar /user/<site-slug>/<post-slug>.
+  const urlBaseFor = (p) => (isHub && p && p.site_slug) ? `/user/${p.site_slug}` : '';
+
+  const prevPost = isHub
+    ? db.prepare(`
+        SELECT p.slug, p.title, s.slug AS site_slug
+        FROM posts p JOIN sites s ON s.id = p.site_id
+        WHERE p.status = 'published' AND p.published_at < ? AND p.id != ?
+        ORDER BY p.published_at DESC LIMIT 1
+      `).get(post.published_at, post.id)
+    : db.prepare(`
+        SELECT slug, title FROM posts
+        WHERE site_id = ? AND status = 'published' AND published_at < ? AND id != ?
+        ORDER BY published_at DESC LIMIT 1
+      `).get(site.id, post.published_at, post.id);
+
+  const nextPost = isHub
+    ? db.prepare(`
+        SELECT p.slug, p.title, s.slug AS site_slug
+        FROM posts p JOIN sites s ON s.id = p.site_id
+        WHERE p.status = 'published' AND p.published_at > ? AND p.id != ?
+        ORDER BY p.published_at ASC LIMIT 1
+      `).get(post.published_at, post.id)
+    : db.prepare(`
+        SELECT slug, title FROM posts
+        WHERE site_id = ? AND status = 'published' AND published_at > ? AND id != ?
+        ORDER BY published_at ASC LIMIT 1
+      `).get(site.id, post.published_at, post.id);
+  if (prevPost) prevPost._urlBase = urlBaseFor(prevPost);
+  if (nextPost) nextPost._urlBase = urlBaseFor(nextPost);
 
   // ── Related posts: same-tag matching with recency fallback ─────
   // Fetch ~50 candidates, score by tag overlap, take top 3.
   // Excluding self via `id != ?`.
-  const candidates = db.prepare(`
-    SELECT id, slug, title, cover_image_url, published_at, tags
-    FROM posts
-    WHERE site_id = ? AND status = 'published' AND id != ?
-    ORDER BY published_at DESC
-    LIMIT 50
-  `).all(site.id, post.id);
+  const candidates = isHub
+    ? db.prepare(`
+        SELECT p.id, p.slug, p.title, p.cover_image_url, p.published_at, p.tags, s.slug AS site_slug
+        FROM posts p JOIN sites s ON s.id = p.site_id
+        WHERE p.status = 'published' AND p.id != ?
+        ORDER BY p.published_at DESC LIMIT 50
+      `).all(post.id)
+    : db.prepare(`
+        SELECT id, slug, title, cover_image_url, published_at, tags
+        FROM posts
+        WHERE site_id = ? AND status = 'published' AND id != ?
+        ORDER BY published_at DESC LIMIT 50
+      `).all(site.id, post.id);
 
   // Parse tags JSON safely; missing/malformed → empty array.
@@ -558 +586 @@
   }
   // Strip the internal _overlap field before sending to view
-  relatedPosts = relatedPosts.map(({ _overlap, tags, ...rest }) => rest);
+  relatedPosts = relatedPosts.map(({ _overlap, tags, ...rest }) => ({ ...rest, _urlBase: urlBaseFor(rest) }));
 
   // ── Pinned navigation: prev/next pinned post ───────────────────