Changeset 0677bac in Klonkt

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@…>

Files:

• src/services/ActivityPubService.js (modified) (2 diffs)
• src/services/ap-core.js (modified) (1 diff)
• src/services/music/index.js (modified) (3 diffs)
• test/music-uitgave.test.js (modified) (3 diffs)

src/services/ActivityPubService.js

@@ -29 +29 @@
 import Blocklist from './BlocklistService.js';
 import * as Guardianship from './guardianship/index.js';
-import { PUBLIC, AP_CONTEXT, safeUrl, actorId, noteId, guessMediaType } from './ap-core.js';
+import { PUBLIC, AP_CONTEXT, safeUrl, actorId, noteId, guessMediaType, normalizeTags, tagParts } from './ap-core.js';
 // Doorgeven wat hier altijd vandaan kwam, zodat elke bestaande aanroep blijft werken.
 export { AP_CONTEXT, actorId, noteId, guessMediaType };
@@ -2973 +2973 @@
 
 // Normalise a post's tags field (array, JSON-string, or comma-string) to an array.
-function normalizeTags(t) {
-  if (Array.isArray(t)) return t;
-  if (typeof t === 'string') {
-    const s = t.trim(); if (!s) return [];
-    if (s[0] === '[') { try { const a = JSON.parse(s); return Array.isArray(a) ? a : []; } catch { /* fall through */ } }
-    return s.split(',').map((x) => x.trim()).filter(Boolean);
-  }
-  return [];
-}
+// normalizeTags en tagParts staan sinds shaer-38y in ap-core: music/ heeft ze
+// ook nodig en mag hier niet uit importeren.
 // A tag → { label, slug }. Multi-word tags become CamelCase (#LiveMusic) for the display
 // name (Mastodon hashtags can't contain spaces; CamelCase is the accessibility norm); the
 // slug/href stays lowercase ("livemusic").
-function tagParts(raw) {
-  const words = String(raw || '').trim().split(/[\s_]+/).map((w) => w.replace(/[^\p{L}\p{M}\p{N}]/gu, '')).filter(Boolean);
-  if (!words.length) return null;
-  const slug = words.join('').toLowerCase();
-  if (!slug) return null;
-  const label = words.length > 1 ? words.map((w) => w[0].toUpperCase() + w.slice(1)).join('') : words[0];
-  return { label, slug };
-}
 // Merge a post's tags field + the #hashtags linked inline in its body into one deduped
 // Hashtag tag list (with hrefs to our /tag page).

src/services/ap-core.js

@@ -93 +93 @@
   })[(e || '').toLowerCase()] || 'image/jpeg';
 }
+
+/**
+ * Het tagveld van een post als lijst. Het staat in de database als JSON-ARRAY
+ * en niet als kommalijst -- op komma's splitsen levert `#["Doen we Niet"` op,
+ * en dat faalt niet, het liegt. Vandaar een echte parser, met de kommavorm als
+ * terugval voor wat er handmatig is ingevuld.
+ */
+export function normalizeTags(t) {
+  if (Array.isArray(t)) return t;
+  if (typeof t === 'string') {
+    const s = t.trim(); if (!s) return [];
+    if (s[0] === '[') { try { const a = JSON.parse(s); return Array.isArray(a) ? a : []; } catch { /* dan toch als kommalijst */ } }
+    return s.split(',').map((x) => x.trim()).filter(Boolean);
+  }
+  return [];
+}
+
+/**
+ * Een tag -> { label, slug }. Tags van meerdere woorden worden CamelCase
+ * (#LiveMusic) voor de weergavenaam -- een Mastodon-hashtag mag geen spaties
+ * bevatten en CamelCase is daar de toegankelijkheidsnorm; de slug en de href
+ * blijven kleingeschreven ("livemusic").
+ */
+export function tagParts(raw) {
+  const words = String(raw || '').trim().split(/[\s_]+/).map((w) => w.replace(/[^\p{L}\p{M}\p{N}]/gu, '')).filter(Boolean);
+  if (!words.length) return null;
+  const slug = words.join('').toLowerCase();
+  if (!slug) return null;
+  const label = words.length > 1 ? words.map((w) => w[0].toUpperCase() + w.slice(1)).join('') : words[0];
+  return { label, slug };
+}

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;
 }
 

test/music-uitgave.test.js

@@ -51 +51 @@
   excerpt: 'Opgenomen op zolder, in een week.',
   cover: '/media/hoes.jpg',
-  tags: 'lofi, zolder',
+  tags: '["lofi","zolder tapes"]',
 });
 
@@ -74 +74 @@
   assert.equal(col.image.type, 'Image');
   assert.equal(col.image.url, `${BASE}/media/hoes.jpg`);
-  assert.deepEqual(col.tag.map((t) => t.name), ['#lofi', '#zolder']);
+  // Het tagveld is JSON in de database, en een tag van twee woorden wordt
+  // CamelCase -- zelfde regel als elders, want het is nu dezelfde parser.
+  assert.deepEqual(col.tag.map((t) => t.name), ['#lofi', '#ZolderTapes']);
+  assert.equal(col.tag[1].href, `${BASE}/tag/zoldertapes`);
   assert.equal(col.url, `${BASE}/drie-nieuwe`);
   assert.equal(col.context, `${BASE}/ap/notes/p-los`);
@@ -153 +156 @@
   assert.equal(M.uitgavePost(SITE, 'tweede'), null);
 });
+
+test('zonder excerpt leent hij de tekst uit het lijf van de post', () => {
+  // Live bleek de excerpt vaak leeg; dan is `content` weglaten slechter dan de
+  // tekst zelf pakken. Shortcodes gaan eruit -- die zijn de muziek, niet het
+  // verhaal erover.
+  const p = maakPost({
+    id: 'p-tekst', slug: 'zonder-excerpt', titel: 'Zonder excerpt',
+    content: '<p>Drie schetsen van vorige week.</p>[[track:t-een]]<div>&nbsp;</div>',
+  });
+  const col = M.buildPostTrackCollection(BASE, site, p);
+  assert.equal(col.content, 'Drie schetsen van vorige week.');
+});
+
+test('en een post die alleen uit shortcodes bestaat leent geen lege tekst', () => {
+  const p = maakPost({
+    id: 'p-kaal', slug: 'alleen-muziek', titel: 'Alleen muziek',
+    content: '[[track:t-twee]]<div> </div>',
+  });
+  const col = M.buildPostTrackCollection(BASE, site, p);
+  assert.equal(col.content, undefined, 'een leeg veld is slechter dan geen veld');
+});