Changeset a4fea5e in Klonkt for scripts/instance-env.mjs

Timestamp:

08/07/2026 04:43:07 PM (5 weeks ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

ed5e7ab

Parents:

6248497

git-author:

roboburr <roboburr@…> (08/07/2026 04:43:06 PM)

git-committer:

roboburr <roboburr@…> (08/07/2026 04:43:07 PM)

Message:

De instance-naam is niet de site-slug, en vlagwaarden zijn geen argumenten

Vervolg op 6248497. Het instance-pad werkte, maar export-archive.mjs liz gaf
"onbekende site: liz" -- en liet je vervolgens raden.

## De naam van de instance is niet de slug van de site

De map onder de data-root en de systemd-unit heten liz; de SITE in die database
kan een andere slug hebben. Ze vallen vaak samen en soms niet, en dan stond je met
een foutmelding die niets prijsgaf.

Twee dingen veranderd:

• de foutmelding zegt nu WAT er wel in die database staat, en als er helemaal geen sites in staan vraagt hij of DATABASE_PATH wel klopt
• de site-slug is optioneel geworden. Staat er precies een site, dan is de keuze niet dubbelzinnig en hoef je hem niet te weten; hij meldt welke hij nam. Bij meerdere sites stopt hij en noemt ze.

node scripts/export-archive.mjs <instance> [site-slug]

## En een parseerfout die ik er zelf net in had gezet

args.filter(a => !a.startsWith('-')) pikt de WAARDE van een vlag op als
positioneel argument. Met liz --data-root /var/lib/klonkt werd /var/lib/klonkt
de site-slug. Gevonden bij het testen van het bovenstaande, en import-archive.mjs
had hem ook: daar zou het pad naar het archief verschoven zijn.

Nu een gedeelde splitsArgs() die weet welke vlaggen een waarde slikken.

Getoetst met een instance 'liz' waarvan de site 'lizzy' heet -- precies het geval
dat op productie omviel: zonder slug pakt hij lizzy en zegt dat, met de verkeerde
slug noemt hij wat er wel is, en met de juiste plus --out schrijft hij het archief.
Suite 577/577 onder UTC en Europe/Amsterdam.

File:

• scripts/instance-env.mjs (modified) (1 diff)

scripts/instance-env.mjs

@@ -94 +94 @@
   process.exit(1);
 }
+
+/**
+ * Argumenten splitsen in vrije woorden en vlaggen.
+ *
+ * Nodig omdat de naieve versie -- alles wat niet met een streepje begint is
+ * positioneel -- de WAARDE van een vlag als positioneel argument oppikt. Met
+ * `export-archive.mjs liz --data-root /var/lib/klonkt` werd /var/lib/klonkt de
+ * site-slug. Precies zo'n fout die pas opvalt als iemand hem gebruikt.
+ *
+ * @param {string[]} args
+ * @param {string[]} metWaarde  vlaggen die een waarde slikken
+ */
+export function splitsArgs(args, metWaarde) {
+  const vrij = [];
+  const vlaggen = {};
+  for (let i = 0; i < args.length; i += 1) {
+    const a = args[i];
+    if (!a.startsWith('-')) { vrij.push(a); continue; }
+    if (metWaarde.includes(a)) { vlaggen[a] = args[i + 1] ?? null; i += 1; continue; }
+    vlaggen[a] = true;
+  }
+  return { vrij, vlaggen };
+}