Changeset 013e16c in Klonkt for src/routes

Timestamp:

08/13/2026 01:09:24 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f50a84b

Parents:

f59a46c

git-author:

Robin <roboburr@…> (08/13/2026 01:01:18 PM)

git-committer:

Robin <roboburr@…> (08/13/2026 01:09:24 PM)

Message:

Naar SEO & social, want sameAs is schema.org (Robins vraag)

Hij vroeg of het niet onder SEO en social hoort, "omdat het ook een soort
koppeling is". Dat is het, en sterker dan dat: op die pagina staat een blok
"Uitgever (JSON-LD / rich results)", en sameAs IS een schema.org-eigenschap.
Zelfde vocabulaire, zelfde plek. Het paneel staat nu direct onder Uitgever.

DAARDOOR VOEDT EEN KOPPELING NU TWEE UITGANGEN. Naast schema:sameAs op de
fediverse-actor komt hij in de JSON-LD van de pagina zelf, in _publisher. Dat
is hetzelfde feit, verteld aan de andere lezer -- en het was gratis, want het
blok stond er al.

BETERE LABELS, want de oude waren geschreven voor het audioscherm. "Ben jij dit
in MusicBrainz?" werd "MusicBrainz (dezelfde artiest elders)" als legenda, met
een hint die zegt wat het OPLEVERT in plaats van alleen wat het is: je komt als
sameAs in de JSON-LD en in je fediverse-profiel, en er verandert niets bij
MusicBrainz zelf. Dat laatste is de geruststelling die er hoort te staan. Plus
een veldlabel ("Zoek jezelf op") met de toelichting "de naam waaronder je muziek
uitbrengt" -- want dat is de vraag, niet je sitenaam. Drie talen.

Het zoekveld begint nu op publisher_name en valt terug op de sitetitel: op deze
pagina heb je die naam net ingevuld.

Meeverhuisd: het endpoint (/admin/seo/api/musicbrainz) en het clientblok, in een
nieuwe mod/admin-seo.js. Die pagina had nog geen JavaScript; opzoeken vraagt er
wel om. Het profielscherm is weer helemaal schoon.

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

Location:

src/routes

Files:

• admin-seo.js (modified) (5 diffs)
• admin-sites.js (modified) (5 diffs)

src/routes/admin-seo.js

@@ -19 +19 @@
 import { requireGod } from '../middleware/auth.js';
 import { getPrimarySite } from '../middleware/site.js';
+import { isMbid } from '../services/ap-core.js';
+import MusicBrainz from '../services/MusicBrainzService.js';
 
 const router = express.Router();
@@ -37 +39 @@
   renderPage(req, res, 'pages/admin-seo', {
     pageTitleKey: 'admin.t_seo',
+    pageJs: 'admin-seo',
     bodyClass: 'on-admin',
     site,
@@ -42 +45 @@
     error: req.query.error || null,
   });
+});
+
+/**
+ * "Zoek jezelf op" -- kandidaten uit MusicBrainz (shaer-mbz).
+ *
+ * De zoekopdracht draait HIER en niet in de browser: MusicBrainz staat een
+ * verzoek per seconde toe per APPLICATIE, en dat is alleen af te dwingen als
+ * alles langs een plek gaat. Bovendien eisen ze een User-Agent met contact, en
+ * die kan een browser niet zetten.
+ *
+ * Wij kiezen NIET. Ook niet als er precies een treffer is: een verkeerd geraden
+ * MBID zet jouw naam onder andermans werk.
+ */
+router.get('/api/musicbrainz', requireGod, async (req, res) => {
+  const site = getPrimarySite(req);
+  const q = String(req.query.q || (site && (site.publisher_name || site.title)) || '').trim();
+  if (!q) return res.json({ ok: true, q: '', kandidaten: [] });
+  res.json({ ok: true, q, kandidaten: await MusicBrainz.zoekArtiesten(q) });
 });
 
@@ -72 +93 @@
       publisher_url = ?,
       publisher_logo = ?,
+      mb_artist_id = ?, mb_artist_name = ?,
       updated_at = CURRENT_TIMESTAMP
     WHERE id = ?
+  // De MusicBrainz-koppeling (shaer-mbz). Alleen een echte MBID komt de kolom
+  // in: zonder deze zeef sluipt er een URL of een handle in het veld dat naar
+  // buiten gaat, en het gaat naar TWEE uitgangen -- de JSON-LD en de actor.
+  // Leeg is een geldige keuze; dat is ontkoppelen.
+  const mbRuw = String(f.mb_artist_id || '').trim().toLowerCase();
+  const mbArtistId = isMbid(mbRuw) ? mbRuw : null;
+
   `).run(
     f.robots_index ? 1 : 0,
@@ -93 +122 @@
     trimOrNull(f.publisher_url, 200),
     trimOrNull(f.publisher_logo, 500),
+    mbArtistId,
+    mbArtistId ? (String(f.mb_artist_name || '').trim().slice(0, 200) || null) : null,
     primary.id,
   );

src/routes/admin-sites.js

@@ -27 +27 @@
 import { mediaDir } from '../config/paths.js';
 import AP from '../services/ActivityPubService.js';
-import MusicBrainz from '../services/MusicBrainzService.js';
 
 
@@ -323 +322 @@
 
 // ==================== SAVE ====================
-/**
- * "Ben jij dit?" -- kandidaten uit MusicBrainz (shaer-mbz).
- *
- * De zoekopdracht draait HIER en niet in de browser: MusicBrainz staat een
- * verzoek per seconde toe per APPLICATIE, en dat is alleen af te dwingen als
- * alles langs een plek gaat. Bovendien eisen ze een User-Agent met contact, en
- * die kan een browser niet zetten.
- *
- * Wij kiezen NIET. Ook niet als er precies een treffer is: een verkeerd geraden
- * MBID zet jouw naam onder andermans werk.
- */
-router.get('/:slug/api/musicbrainz', requireSiteManagerBySlug, async (req, res) => {
-  const site = db.prepare('SELECT title, mb_artist_id, mb_artist_name FROM sites WHERE slug = ?').get(req.params.slug);
-  if (!site) return res.status(404).json({ error: 'no_site' });
-  const q = String(req.query.q || site.title || '').trim();
-  if (!q) return res.json({ ok: true, q: '', kandidaten: [] });
-  res.json({ ok: true, q, kandidaten: await MusicBrainz.zoekArtiesten(q) });
-});
-
 router.post('/:slug/save', requireSiteManagerBySlug, async (req, res) => {
   const site = db.prepare('SELECT id, ap_aliases FROM sites WHERE slug = ?').get(req.params.slug);
@@ -366 +346 @@
   // accent: only accept colors from the curated ACCENTS list. Falls back to
   // the orange default if the submitted value isn't recognised.
-  // De MusicBrainz-koppeling (shaer-mbz). Alleen een echte MBID komt de kolom
-  // in: zonder deze zeef sluipt er een hele URL of een handle in het veld dat
-  // straks naar buiten gaat. Leeg is een geldige keuze -- dat is ontkoppelen.
-  const mbRuw = String(f.mb_artist_id || '').trim().toLowerCase();
-  const mbArtistId = MusicBrainz.isMbid(mbRuw) ? mbRuw : null;
-
   const accent = ThemeService.validateAccent(f.accent) || '#e8b04b';
 
@@ -381 +355 @@
       profile_links = ?,
       ap_aliases = ?,
-      mb_artist_id = ?, mb_artist_name = ?,
       is_public = ?, robots_index = ?, require_login_to_comment = ?,
       enable_audio_player = ?,
@@ -401 +374 @@
     profileLinksJson,
     apAliasesJson,
-    mbArtistId,
-    mbArtistId ? (String(f.mb_artist_name || '').trim().slice(0, 200) || null) : null,
     f.is_public ? 1 : 0,
     f.robots_index ? 1 : 0,