Changeset 68a4d1b in Klonkt for test/reactions-characterization.test.js

Timestamp:

08/06/2026 09:15:46 AM (5 weeks ago)

Author:

Claude (agent) <aiclaude@…>

Branches:

main

Children:

6bc5ebd

Parents:

14f7cb2

git-author:

Robin <roboburr@…> (08/06/2026 09:15:45 AM)

git-committer:

Claude (agent) <aiclaude@…> (08/06/2026 09:15:46 AM)

Message:

De halve schrijfactie kan niet meer (shaer-9e9 fase 3, deel 1)

markLiked, unmarkLiked, markBoosted, unmarkBoosted, setMyReaction,
getMyReactions en getTimelineReaction staan niet langer op het default-object
van ActivityPubService. Routes importeren dat object, dus daarmee is de helft
schrijven vanaf een route onmogelijk geworden in plaats van afgeraden. De named
exports blijven bestaan voor intern gebruik en voor de tests. Een test bewaakt
het oppervlak, want een afspraak in een comment overleeft geen half jaar.

Meteen ook de laatste halve schrijfactie weg, en die was van mijzelf: in de
interact-boost stond nog een kale setMyReaction "zodat de knop klopt voordat de
resolve terug is". Dat is nu setReaction, dus ook dat korte moment schrijft
beide bronnen; de resolve werkt hem daarna bij met de note zodat de post in je
tijdlijn belandt.

De kolommen zijn gedocumenteerd als afgeleide, op de plek waar iemand ze
tegenkomt. Met het onderscheid erbij: ap_timeline.boosted verdient zijn bestaan
als index (Cirkel-WHERE, boostedCount), ap_timeline.liked niet.

WAT IK BEWUST NIET DOE: ap_timeline.liked weggooien. Het plan zegt dat het kan,
en dat klopt, maar zolang die kolom gevuld blijft is terugdraaien een
code-revert en geen dataherstel. Weggooien hoort pas als fase 2 een tijd goed
heeft gestaan -- niet in dezelfde sessie waarin het leespad is omgezet.

Suite 431/431.

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

File:

• test/reactions-characterization.test.js (modified) (1 diff)

test/reactions-characterization.test.js

@@ -254 +254 @@
   assert.equal(AP.getReactionsFor('', [uri('g3')]).size, 0);
 });
+
+// ── Fase 3: het publieke oppervlak is versmald ───────────────────────────
+
+test('de primitieven zijn niet meer bereikbaar via het service-object', async () => {
+  // Routes doen `import ActivityPubService from ...` en werken dus met het
+  // default-object. Zolang markLiked daar in staat, kan een aanroeper de helft
+  // schrijven -- en dat is niet hypothetisch: precies zo bleef shaer:liked
+  // maandenlang false. De named exports blijven bestaan voor intern gebruik en
+  // voor deze tests.
+  const svc = (await import('../src/services/ActivityPubService.js')).default;
+  for (const naam of ['markLiked', 'unmarkLiked', 'markBoosted', 'unmarkBoosted',
+                      'setMyReaction', 'getMyReactions', 'getTimelineReaction']) {
+    assert.equal(svc[naam], undefined, `${naam} hoort niet op het publieke oppervlak te staan`);
+  }
+  // Wat er WEL hoort te staan: het ene schrijfpad en het ene leespad.
+  for (const naam of ['setReaction', 'getReaction', 'getReactionsFor']) {
+    assert.equal(typeof svc[naam], 'function', `${naam} hoort er wel te zijn`);
+  }
+});