Changeset 3bbf73d in Klonkt for src

Timestamp:

08/10/2026 02:43:16 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f0a33e1

Parents:

f549cf5

Message:

Alleen namen: het Nederlands van vandaag terug naar Engels

shaer-8ms zegt het al: Engelse namen, Nederlands commentaar waar dat helpt. Ik
ben er vandaag opnieuw van afgedreven en zet dat recht voordat er meer bovenop
komt -- gatesFor, messageItem, sentItem, conversationItems, ownHandle,
CONVERSATION_UNION, MESSAGE_COLUMNS, cursorOf, en de locals eromheen
(rows/more/oldest/direction/incoming/heads/self/size).

Geen gedrag, alleen namen, en daarom een eigen commit: die is na te lopen.
851/851 groen voor en na. Het commentaar blijft Nederlands.

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

Location:

src

Files:

• routes/activitypub.js (modified) (9 diffs)
• services/ActivityPubService.js (modified) (5 diffs)

src/routes/activitypub.js

@@ -289 +289 @@
 // dat evenveel kopieen worden -- en een poort die op een van die plekken
 // vergeten wordt, levert stil iets uit dat dicht hoorde te staan.
-function leesPoorten(site) {
+function gatesFor(site) {
   const isWard = (() => { try { return Guardianship.listGuardians(site.slug).length > 0; } catch { return false; } })();
   const embeds = Guardianship.externalEmbedsAllowed(site.external_embeds, isWard);
@@ -314 +314 @@
 
 /** De naam waaronder deze lezer zichzelf herkent in een Mention. */
-function eigenHandle(base, slug) {
+function ownHandle(base, slug) {
   try { return `@${slug}@${new URL(base).host}`; } catch { return `@${slug}`; }
 }
@@ -323 +323 @@
 // opschrijven is twee vormen die uit de pas kunnen lopen, en dat merk je pas
 // als een kaart ergens anders rendert dan waar je keek.
-function berichtItem(m, { base, me, myHandle, p }) {
+function messageItem(m, { base, me, myHandle, p }) {
   return {
     id: `${m.object_uri}#create`,
@@ -359 +359 @@
 
 /** Een eigen verzonden note als AS2-item, zelfde vorm als de inbox-leg. */
-function verzondenItem(n, { me, mine }) {
+function sentItem(n, { me, mine }) {
   return {
     id: `${n.id}#create`,
@@ -383 +383 @@
 // ophoudt.
 //
-// Beide lopen langs dezelfde poorten als de inbox-lezing (leesPoorten) en
-// dezelfde kaartvorm (berichtItem/verzondenItem). Messages dicht sluit ook
+// Beide lopen langs dezelfde poorten als de inbox-lezing (gatesFor) en
+// dezelfde kaartvorm (messageItem/sentItem). Messages dicht sluit ook
 // hier vreemden en vrienden, maar nooit het guardian-kanaal en nooit de boei.
-function gesprekItems(req, auth, refs) {
+function conversationItems(req, auth, refs) {
   const base = baseUrl(req);
-  const P = leesPoorten(auth.site);
+  const P = gatesFor(auth.site);
   const me = AP.actorId(base, auth.site.slug);
-  const ctx = { base, me, myHandle: eigenHandle(base, auth.site.slug), p: P };
+  const ctx = { base, me, myHandle: ownHandle(base, auth.site.slug), p: P };
   const mine = AP.selfAuthor(base, auth.site);
   const guardianUris = (() => { try { return new Set(Guardianship.listGuardians(auth.site.slug).map((g) => g.other_uri)); } catch { return new Set(); } })();
 
-  const binnen = new Map(AP.messageRowsByUri(auth.site.slug, refs.filter((r) => r.richting === 'in').map((r) => r.ref))
+  const incoming = new Map(AP.messageRowsByUri(auth.site.slug, refs.filter((r) => r.direction === 'in').map((r) => r.ref))
     .map((m) => [m.object_uri, m]));
-  const uit = [];
+  const items = [];
   for (const r of refs) {
-    if (r.richting === 'in') {
-      const m = binnen.get(r.ref);
+    if (r.direction === 'in') {
+      const m = incoming.get(r.ref);
       if (!m) continue;
       if (!(P.messagesAllowed || m.help_request || guardianUris.has(m.actor_uri))) continue;
-      uit.push(berichtItem(m, ctx));
+      items.push(messageItem(m, ctx));
     } else {
       const n = AP.getOutboxNote(base, r.ref);
       // Je eigen woorden blijven van jou: een dichte messages-poort verbergt
       // niet wat je zelf gezegd hebt.
-      if (n) uit.push(verzondenItem(n, { me, mine }));
+      if (n) items.push(sentItem(n, { me, mine }));
     }
   }
-  return uit;
+  return items;
 }
 
@@ -416 +416 @@
   const auth = OAuth.verifyBearer(req.headers.authorization);
   if (!auth || auth.site.slug !== req.params.slug) return res.status(403).end();
-  const koppen = AP.conversationHeads(auth.site.slug);
-  const items = gesprekItems(req, auth, koppen);
+  const heads = AP.conversationHeads(auth.site.slug);
+  const items = conversationItems(req, auth, heads);
   AP.sendAP(res, {
     '@context': AP.AP_CONTEXT,
@@ -433 +433 @@
   const other = String(req.query.with || '');
   if (!/^https?:\/\//i.test(other)) return res.status(400).json({ error: 'with must be an actor URI' });
-  const uit = AP.conversationHistory(auth.site.slug, other, {
+  const page = AP.conversationHistory(auth.site.slug, other, {
     before: req.query.before ? String(req.query.before) : null,
     limit: req.query.limit,
   });
-  const items = gesprekItems(req, auth, uit.rijen);
+  const items = conversationItems(req, auth, page.rows);
   // De paginagrootte reist mee in next: vroeg je om 30, dan hoort de volgende
   // pagina er ook 30 te zijn. Zonder dit wordt hij stilletjes de standaard, en
   // dan klopt het ritme van een 'load more' niet meer met wat de gebruiker ziet.
-  const maat = req.query.limit ? `&limit=${encodeURIComponent(String(req.query.limit))}` : '';
-  const zelf = `${baseUrl(req)}/ap/users/${encodeURIComponent(auth.site.slug)}/messages?with=${encodeURIComponent(other)}`;
+  const size = req.query.limit ? `&limit=${encodeURIComponent(String(req.query.limit))}` : '';
+  const self = `${baseUrl(req)}/ap/users/${encodeURIComponent(auth.site.slug)}/messages?with=${encodeURIComponent(other)}`;
   AP.sendAP(res, {
     '@context': AP.AP_CONTEXT,
-    id: req.query.before ? `${zelf}${maat}&before=${encodeURIComponent(String(req.query.before))}` : `${zelf}${maat}`,
+    id: req.query.before ? `${self}${size}&before=${encodeURIComponent(String(req.query.before))}` : `${self}${size}`,
     type: 'OrderedCollectionPage',
-    partOf: zelf,
+    partOf: self,
     orderedItems: items,
     // De volgende pagina is de standaardvorm van 'er is meer' (AS2). Ontbreekt
     // hij, dan is het gesprek op -- en dat mag de client weten zonder gokken,
     // want anders kan een 'load more' niet eerlijk verschijnen.
-    next: uit.meer && uit.oudste ? `${zelf}${maat}&before=${encodeURIComponent(uit.oudste)}` : undefined,
+    next: page.more && page.oldest ? `${self}${size}&before=${encodeURIComponent(page.oldest)}` : undefined,
   }, 'private, no-store');
 });
@@ -587 +587 @@
   // here, at serialisation: a blocked embed is never sent, because an embed the
   // client merely hides has still been delivered to the device.
-  // De poorten van deze lezer (leesPoorten): een plek waar ze berekend worden,
+  // De poorten van deze lezer (gatesFor): een plek waar ze berekend worden,
   // zodat de gesprekslezingen dezelfde stand eerbiedigen en niet hun eigen
   // kopie krijgen die kan gaan afwijken.
-  const P = leesPoorten(auth.site);
+  const P = gatesFor(auth.site);
   const {
     embedsAllowed, playbackAllowed, imagesAllowed, musicAllowed, quotesAllowed,
@@ -676 +676 @@
   // beschermt niemand. De hulpvraag zelf gaat aan de innamekant al altijd voor.
   const guardianUris = (() => { try { return new Set(Guardianship.listGuardians(auth.site.slug).map((g) => g.other_uri)); } catch { return new Set(); } })();
-  const berichtCtx = { base, me, myHandle, p: P };
+  const messageCtx = { base, me, myHandle, p: P };
   const messages = AP.getDirectMessages(auth.site.slug, 60)
     .filter((m) => messagesAllowed || m.help_request || guardianUris.has(m.actor_uri))
-    .map((m) => berichtItem(m, berichtCtx));
+    .map((m) => messageItem(m, messageCtx));
   // Inbound REPLIES on your own posts: stored as interactions (the web's
   // comment machinery), never as mentions, so this read missed them and a

src/services/ActivityPubService.js

@@ -4210 +4210 @@
 // visibility 'direct'. Een publiek antwoord is geen gesprek en hoort niet als
 // gezicht in de hemel.
-const GESPREK_UNIE = `
+const CONVERSATION_UNION = `
   SELECT m.actor_uri AS other, COALESCE(m.published, m.created_at) AS stamp,
-         'in' AS richting, m.object_uri AS ref
+         'in' AS direction, m.object_uri AS ref
     FROM ap_mentions m
    WHERE m.slug = @slug AND m.actor_uri IS NOT NULL AND m.actor_uri <> ''
   UNION ALL
   SELECT j.value AS other, o.created_at AS stamp,
-         'uit' AS richting, o.id AS ref
+         'out' AS direction, o.id AS ref
     FROM ap_outbox o
     JOIN json_each(COALESCE(NULLIF(o.to_actors, ''), json_array(o.to_actor))) j
@@ -4233 +4233 @@
   try {
     return db.prepare(`
-      SELECT other, stamp, richting, ref FROM (
+      SELECT other, stamp, direction, ref FROM (
         SELECT *, ROW_NUMBER() OVER (PARTITION BY other ORDER BY stamp DESC, ref DESC) AS rn
-          FROM (${GESPREK_UNIE})
+          FROM (${CONVERSATION_UNION})
       ) WHERE rn = 1
       ORDER BY stamp DESC, ref DESC`).all({ slug });
@@ -4260 +4260 @@
  * ontbreekt alleen iets in het midden.
  */
-const cursorVan = (r) => (r ? `${r.stamp}|${r.ref}` : null);
+const cursorOf = (r) => (r ? `${r.stamp}|${r.ref}` : null);
 
 export function conversationHistory(slug, other, { before = null, limit = 60 } = {}) {
   try {
     const n = Math.min(Math.max(parseInt(limit, 10) || 60, 1), 200);
-    const knip = String(before || '').indexOf('|');
-    const bStamp = before && knip > 0 ? String(before).slice(0, knip) : null;
-    const bRef = before && knip > 0 ? String(before).slice(knip + 1) : null;
-    const rijen = db.prepare(`
-      SELECT other, stamp, richting, ref FROM (${GESPREK_UNIE})
+    const sep = String(before || '').indexOf('|');
+    const bStamp = before && sep > 0 ? String(before).slice(0, sep) : null;
+    const bRef = before && sep > 0 ? String(before).slice(sep + 1) : null;
+    const rows = db.prepare(`
+      SELECT other, stamp, direction, ref FROM (${CONVERSATION_UNION})
        WHERE other = @other
          AND (@bStamp IS NULL OR stamp < @bStamp OR (stamp = @bStamp AND ref < @bRef))
        ORDER BY stamp DESC, ref DESC LIMIT @n`).all({ slug, other, bStamp, bRef, n: n + 1 });
-    const meer = rijen.length > n;
-    const uit = meer ? rijen.slice(0, n) : rijen;
-    return { rijen: uit, meer, oudste: cursorVan(uit[uit.length - 1]) };
-  } catch { return { rijen: [], meer: false, oudste: null }; }
+    const more = rows.length > n;
+    const page = more ? rows.slice(0, n) : rows;
+    return { rows: page, more, oldest: cursorOf(page[page.length - 1]) };
+  } catch { return { rows: [], more: false, oldest: null }; }
 }
 
 // De kolommen die een bericht tot kaart maken. Een constante, want de
-// gesprekslezing haalt dezelfde rijen op: twee lijsten die uiteenlopen leveren
+// gesprekslezing haalt dezelfde rows op: twee lijsten die uiteenlopen leveren
 // een kaart die op de ene plek een plaatje heeft en op de andere niet.
-const BERICHT_KOLOMMEN = `
+const MESSAGE_COLUMNS = `
       m.object_uri, m.note_url, m.actor_uri, m.actor_name, m.actor_handle, m.actor_icon, m.actor_url,
       m.content, m.published, m.created_at, m.wave, m.help_request,
@@ -4293 +4293 @@
   try {
     const gaten = lijst.map(() => '?').join(',');
-    return db.prepare(`SELECT ${BERICHT_KOLOMMEN} FROM ap_mentions m
+    return db.prepare(`SELECT ${MESSAGE_COLUMNS} FROM ap_mentions m
                         WHERE m.slug = ? AND m.object_uri IN (${gaten})`).all(slug, ...lijst);
   } catch { return []; }
@@ -4301 +4301 @@
   try {
     return db.prepare(`
-      SELECT ${BERICHT_KOLOMMEN}
+      SELECT ${MESSAGE_COLUMNS}
       FROM ap_mentions m
       WHERE m.slug = ?