Changeset 0677bac in Klonkt for src/services/music/index.js

Timestamp:

08/09/2026 08:24:16 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

d60c498

Parents:

a5d14c7

Message:

De tags kwamen er als onzin uit, en de tekst helemaal niet (shaer-38y)

Live nagekeken op dev, en twee dingen klopten niet.

DE TAGS. posts.tags staat als JSON-ARRAY in de database, niet als kommalijst.
Mijn eigen split leverde #["Doen we Niet" en #"music" op -- geen fout die
omvalt, een die onzin uitlevert. Nu met normalizeTags en tagParts, dezelfde
parser als de rest, dus ook dezelfde CamelCase voor tags van meerdere woorden.

Die twee stonden in ActivityPubService en music/ mag daar niet uit importeren,
dus ze zijn naar ap-core verhuisd -- de plek voor precies dit soort gedeelde
grondstof. Een kopie maken zou hetzelfde uit elkaar lopen opleveren als bij de
afleidingsregel.

DE TEKST. De excerpt bleek in de praktijk vaak leeg, en dan kwam er geen
content mee terwijl de post wel tekst heeft. Nu: excerpt eerst -- die is
geschreven om samen te vatten -- en anders het lijf zelf, zonder shortcodes
(die zijn de muziek, niet het verhaal erover) en zonder opmaak. Blijft er niets
over, dan geen veld: dat is beter dan een leeg veld.

De test gebruikte de kommavorm en testte dus de terugval in plaats van de echte
weg. Nu JSON, zoals het er staat.

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

File:

• src/services/music/index.js (modified) (3 diffs)

src/services/music/index.js

@@ -14 +14 @@
 
 import db from '../../config/database.js';
-import { AP_CONTEXT, PUBLIC, actorId, noteId, safeUrl, guessMediaType } from '../ap-core.js';
+import { AP_CONTEXT, PUBLIC, actorId, noteId, safeUrl, guessMediaType, normalizeTags, tagParts } from '../ap-core.js';
 import { afleidenUitInsluitingen, ingeslotenPlaylists } from '../../assets/js/shared/post-music-type.js';
 
@@ -368 +368 @@
   // samenvatting en content het lijf. Artiest en jaar blijven dus in summary
   // staan -- dat is een samenvatting, en de posttekst is dat niet.
-  const tekst = (post.excerpt || '').trim();
+  const tekst = tekstVanPost(post);
   if (tekst) obj.content = tekst;
 
@@ -388 +388 @@
 }
 
-/** De tags van een post als AS2 Hashtags. Zelfde vorm als buildHashtagList. */
+/**
+ * De tags van een post als AS2 Hashtags -- zelfde vorm als buildHashtagList.
+ *
+ * Met normalizeTags en niet met een eigen split: het veld staat als JSON-array
+ * in de database, en op komma's splitsen leverde live `#["Doen we Niet"` op.
+ * Dat is het soort fout dat niet omvalt maar onzin uitlevert.
+ */
 function hashtagsVanPost(base, tagsField) {
-  const ruw = Array.isArray(tagsField)
-    ? tagsField
-    : String(tagsField || '').split(',');
   const uit = [], gezien = new Set();
-  for (const t of ruw) {
-    const label = String(t || '').trim().replace(/^#/, '');
-    if (!label) continue;
-    const slug = label.toLowerCase().replace(/\s+/g, '-');
-    if (gezien.has(slug)) continue;
-    gezien.add(slug);
-    uit.push({ type: 'Hashtag', href: `${base}/tag/${encodeURIComponent(slug)}`, name: '#' + label });
+  for (const t of normalizeTags(tagsField)) {
+    const p = tagParts(t);
+    if (!p || gezien.has(p.slug)) continue;
+    gezien.add(p.slug);
+    uit.push({ type: 'Hashtag', href: `${base}/tag/${encodeURIComponent(p.slug)}`, name: '#' + p.label });
   }
   return uit;
+}
+
+/**
+ * De tekst van een post, als er een is. De excerpt heeft voorrang -- die is
+ * geschreven om samen te vatten. Staat die leeg, dan het lijf zelf: zonder
+ * shortcodes (die zijn de muziek, niet het verhaal erover) en zonder opmaak.
+ * Levert null als er niets overblijft, want een leeg veld is slechter dan geen.
+ */
+function tekstVanPost(post) {
+  const excerpt = String(post.excerpt || '').trim();
+  if (excerpt) return excerpt;
+  const kaal = String(post.content || '')
+    .replace(/\[\[[a-z]+:[^\]]*\]\]/gi, ' ')
+    .replace(/<[^>]+>/g, ' ')
+    .replace(/&nbsp;/gi, ' ')
+    .replace(/&[a-z#0-9]+;/gi, ' ')
+    .replace(/\s+/g, ' ')
+    .trim();
+  return kaal || null;
 }