Changeset ed5e7ab in Klonkt for src/services/ArchiveImportService.js

Timestamp:

08/07/2026 05:41:20 PM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

c25eff7

Parents:

ba76bf5 (diff), a4fea5e (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge de drie nieuwste van GitHub, met hun exporter-fix boven de mijne

De parallelle sessie pushte drie commits terwijl deze kant aan het werk was. Een
ervan, 6248497, repareert dezelfde exporter-tijdzonebug die hier een uur eerder in
ba76bf5 was gerepareerd. Twee keer hetzelfde werk, langs twee routes gevonden: daar
via een vraag van Bart, hier via de test die in Europe/Amsterdam omviel.

Het enige conflict zat dan ook in ArchiveExportService.js, op precies die functie.
Hun versie is aangehouden, en dat is geen hoffelijkheid maar een meting: hun regex
accepteert ook een stempel zonder seconden ("2026-07-01 12:56"), waar de mijne
seconden eist. Ze pakten bovendien export-archive.mjs, import-archive.mjs en een
nieuwe instance-env.mjs mee, plus een test in archive-export.test.js. Mijn variant is
daarmee vervallen en verdwijnt uit de boom; de bug blijft gedekt.

Nagemeten dat die keuze de dekking niet kost: archive-import 21 groen en
archive-export 16 groen, in UTC en in Europe/Amsterdam. Zonder een fix valt de
import-test alleen buiten UTC om, dus de tijdzone-afhankelijkheid is echt weg.

Changed files:
src/services/ArchiveExportService.js

• conflict opgelost ten gunste van 6248497; de eigen toISO is eruit

remarks: dit is de derde keer vandaag dat de GitHub-mirror faalde omdat GitHub
verder stond dan prutfolio.git. De VPS is volgens de deploy-afspraak canoniek,
maar de andere sessie pusht rechtstreeks naar GitHub, dus het divergeert elke paar
uur opnieuw en dat kostte nu dubbel werk. Eén bron aanwijzen is een besluit voor
Robin, geen technische keuze.

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

File:

• src/services/ArchiveImportService.js (modified) (1 diff)

src/services/ArchiveImportService.js

@@ -143 +143 @@
 
   const site = db.prepare('SELECT * FROM sites WHERE slug = ?').get(opts.slug);
-  if (!site) throw new Error(`onbekende site: ${opts.slug}`);
+  if (!site) {
+    // De naam van de INSTANCE (de map, de unit) en de slug van de SITE in zijn
+    // database zijn twee dingen. Ze vallen vaak samen en soms niet, en dan zat je
+    // met een foutmelding die je liet raden. Zeg dus wat er wel in staat.
+    let bestaand = [];
+    try { bestaand = db.prepare('SELECT slug FROM sites ORDER BY rowid').all().map((r) => r.slug); } catch { /* geen sites-tabel */ }
+    const wat = opts.slug ? `onbekende site: ${opts.slug}` : 'geen site opgegeven';
+    throw new Error(bestaand.length
+      ? `${wat}. In deze database staat: ${bestaand.join(', ')}`
+      : `${wat}. In deze database staat geen enkele site -- wijst DATABASE_PATH naar de juiste?`);
+  }
   const eigenOrigin = (opts.origin || process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
   rapport.origin = manifest.origin || null;