Changeset e9ec5e4 in Klonkt for test/following-csv.test.js

Timestamp:

08/13/2026 06:16:49 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

fbda20e

Parents:

096f099

Message:

Volglijst: Featured is auto_boost, plus de knoppen om hem mee te nemen

Twee dingen: een correctie op de vorige commit, en de ingang die er nog niet was.

DE CORRECTIE. Ik had auto_boost in de kolom "Show boosts" gezet omdat in beide het
woord boost zit. Dat is fout. Bij Mastodon betekent die kolom "toon de reblogs van
deze persoon in mijn tijdlijn", en dat kent Klonkt niet; onze vlag gaat over hun
EIGEN posts in JOUW Cirkel. Dat is de vlag die in de UI al "Uitgelicht" heet
(tl.autoboost, in het Engels letterlijk "Featured"), dus hij hoort in de kolom
Featured en "Show boosts" blijft leeg.

Daarmee vervalt ook de kolom highlighted die ik in de vorige commit toevoegde:
die was een nieuw begrip naast een bestaand begrip met dezelfde naam. Eén ding, één
naam. Een bestand uit Mastodon levert nu featured: false, en dat is juist: hun
kolom 2 mag hier niet als uitgelicht binnenkomen.

DE INGANG. De vorige commit had de functies maar geen knop, en volgens de eigen
regel is een feature pas af als een eindgebruiker hem kan gebruiken. Op /following
staat nu een uitklapblok met een downloadknop en een plakveld.

De importroute geeft followActor mee als followFn, zodat er één pad naar een
volgrelatie blijft. Daar zat een val in: followActor meldt een fout als VELD en
niet als exception, dus zonder vertaling telde een onvindbaar account als geslaagd
mee. Die vertaling zit er nu in.

Changed files:
src/config/database.js

• kolom highlighted teruggedraaid; commentaar bij auto_boost zegt nu dat het "Uitgelicht"/"Featured" heet

src/services/ArchiveExportService.js

• auto_boost naar kolom Featured, Show boosts/Notify/Languages blijven leeg
• parseFollowingCsv leest alleen kolom 5, en levert featured in plaats van twee velden

src/services/ArchiveImportService.js

• de uitgelicht-stand gaat mee IN de Follow (derde argument), geen tweede UPDATE erna

src/routes/posts.js

• GET /news/following.csv, met private/no-store want dit is jouw volglijst
• POST /news/following/import, urlencoded, roept importFollowing aan met followActor

src/services/i18n.js

• vijf teksten in nl, en en de

src/views/pages/following.ejs

• het uitklapblok met de downloadknop en het plakveld

test/following-csv.test.js

• bijgewerkt naar de juiste kolomindeling; de Mastodon-test controleert nu expliciet dat hun "Show boosts: true" NIET als uitgelicht landt

remarks: 921 groen in UTC en Europe/Amsterdam. Server gestart en de drie routes
nagelopen: alle drie bestaan en sturen naar de login, geen serverfouten. De template
apart gerenderd in nl/en/de, geen onvertaalde sleutels. De pagina zelf achter de
login niet visueel gezien; daar is een account voor nodig. Onderweg per ongeluk
.claude/launch.json overschreven (die zette PUBLIC_BASE_URL) en weer hersteld.

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

File:

• test/following-csv.test.js (modified) (9 diffs)

test/following-csv.test.js

@@ -24 +24 @@
 const { importFollowing } = await import('../src/services/ArchiveImportService.js');
 
-const volg = db.prepare(`INSERT INTO ap_following (slug, actor_uri, handle, status, auto_boost, highlighted)
-                         VALUES (?,?,?,?,?,?)`);
+const volg = db.prepare(`INSERT INTO ap_following (slug, actor_uri, handle, status, auto_boost)
+                         VALUES (?,?,?,?,?)`);
 
 beforeEach(() => {
@@ -32 +32 @@
 
 test('de CSV heeft de Mastodon-koppen, met Featured erachter', () => {
-  volg.run('me', 'https://a.example/ap/users/jason', '@jason@a.example', 'accepted', 1, 1);
+  volg.run('me', 'https://a.example/ap/users/jason', '@jason@a.example', 'accepted', 1);
   const csv = followingCsv('me');
   const [kop, eerste] = csv.trim().split('\n');
   assert.equal(kop, 'Account address,Show boosts,Notify on new posts,Languages,Featured');
   // Zonder de leidende @, want zo schrijft Mastodon het adres.
-  assert.equal(eerste, 'jason@a.example,true,false,,true');
+  assert.equal(eerste, 'jason@a.example,,,,true');
 });
 
 test('een openstaand verzoek gaat NIET mee', () => {
-  volg.run('me', 'https://a.example/ap/users/wacht', '@wacht@a.example', 'pending', 0, 0);
-  volg.run('me', 'https://a.example/ap/users/ja', '@ja@a.example', 'accepted', 0, 0);
+  volg.run('me', 'https://a.example/ap/users/wacht', '@wacht@a.example', 'pending', 0);
+  volg.run('me', 'https://a.example/ap/users/ja', '@ja@a.example', 'accepted', 0);
   const regels = followingCsv('me').trim().split('\n').slice(1);
   assert.equal(regels.length, 1, 'een verzoek dat iemand bewust liet liggen mag niet opnieuw verstuurd worden');
@@ -49 +49 @@
 
 test('zonder handle valt hij terug op de actor-URI', () => {
-  volg.run('me', 'https://a.example/ap/users/naamloos', null, 'accepted', 0, 0);
+  volg.run('me', 'https://a.example/ap/users/naamloos', null, 'accepted', 0);
   assert.match(followingCsv('me'), /https:\/\/a\.example\/ap\/users\/naamloos/);
 });
@@ -58 +58 @@
 
 test('een export leest zichzelf terug', () => {
-  volg.run('me', 'https://a.example/ap/users/a', '@a@a.example', 'accepted', 1, 0);
-  volg.run('me', 'https://b.example/ap/users/b', '@b@b.example', 'accepted', 0, 1);
+  volg.run('me', 'https://a.example/ap/users/a', '@a@a.example', 'accepted', 1);
+  volg.run('me', 'https://b.example/ap/users/b', '@b@b.example', 'accepted', 0);
   const terug = parseFollowingCsv(followingCsv('me'));
   assert.deepEqual(terug, [
-    { address: 'a@a.example', autoBoost: true, highlighted: false },
-    { address: 'b@b.example', autoBoost: false, highlighted: true },
+    { address: 'a@a.example', featured: true },
+    { address: 'b@b.example', featured: false },
   ]);
 });
@@ -72 +72 @@
     + 'iemand@mastodon.social,true,false,\n',
   );
-  assert.deepEqual(uit, [{ address: 'iemand@mastodon.social', autoBoost: true, highlighted: false }]);
+  // Hun "Show boosts" staat op true, en dat mag hier NIET als uitgelicht landen.
+  assert.deepEqual(uit, [{ address: 'iemand@mastodon.social', featured: false }]);
 });
 
@@ -83 +84 @@
   const uit = parseFollowingCsv('Account address,Show boosts\n"raar,naam@a.example",true\n');
   assert.equal(uit[0].address, 'raar,naam@a.example');
-  assert.equal(uit[0].autoBoost, true);
 });
 
@@ -90 +90 @@
   const r = await importFollowing({ slug: 'me' },
     'Account address,Show boosts,Notify on new posts,Languages,Featured\n'
-    + 'a@a.example,true,false,,false\n'
-    + 'b@b.example,false,false,,true\n',
+    + 'a@a.example,,,,true\n'
+    + 'b@b.example,,,,false\n',
     { followFn: async (site, adres, boost) => { gezien.push([adres, boost]); return true; } });
 
-  assert.deepEqual(gezien, [['a@a.example', true], ['b@b.example', false]]);
+  assert.deepEqual(gezien, [['a@a.example', true], ['b@b.example', false]],
+    'de uitgelicht-stand moet als derde argument mee de Follow in');
   assert.equal(r.gevolgd, 2);
   assert.equal(r.mislukt.length, 0);
@@ -130 +131 @@
   db.prepare('INSERT OR IGNORE INTO sites (id, slug, title, owner_id) VALUES (?,?,?,?)')
     .run('s1', 'me', 'Mijn site', 'u1');
-  volg.run('me', 'https://a.example/ap/users/jason', '@jason@a.example', 'accepted', 1, 1);
+  volg.run('me', 'https://a.example/ap/users/jason', '@jason@a.example', 'accepted', 1);
 
   const { buildArchive } = await import('../src/services/ArchiveExportService.js');
@@ -138 +139 @@
   assert.equal(r.counts.following, 1);
   assert.ok(r.manifest.files['following.csv'], 'en hij hoort in het manifest, anders telt hij niet mee bij de controle');
-  assert.match(String(r.files.get('following.csv')), /jason@a\.example,true,false,,true/);
+  assert.match(String(r.files.get('following.csv')), /jason@a\.example,,,,true/);
 });