Changeset 7e9d0ea in Klonkt

Timestamp:

08/11/2026 07:18:51 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

bac4bf3

Parents:

960f015

git-author:

Robin <roboburr@…> (08/11/2026 07:12:33 PM)

git-committer:

Robin <roboburr@…> (08/11/2026 07:18:51 PM)

Message:

De laatste tenancy-resten, en een testbestand dat alleen zijn naam kwijt was (shaer-x7c0)

Het meeste van de bead was al gedaan door 72ec6a4 (Hub-modus en guardian-lite
eruit), dezelfde dag nog. Wat er lag:

• database.js beschreef nog een tenancy-modus met hub, en zette bij elke boot een app_settings-rij 'tenancy' die niemand meer leest. Beide weg. Bestaande rijen blijven staan; die opruimen is een aparte beslissing.
• ensurePrimarySite.js noemde solo/hub/circle op twee plekken.

En het belangrijkste, precies wat de bead verkeerd had: hub-permissions.test.js
test GEEN verwijderde modus. Het dekt canAdminSite (inclusief de site_members-tak
die ooit stil kapot was), canEditPost en getPrimarySite -- allemaal springlevend,
en CLAUDE.md wijst dit bestand aan als de vangrail daarvoor. Weggooien had die
dekking gesloopt. Alleen de naam verwees nog naar de hub, dus die is aangepast,
met een kop die uitlegt waarom het bestand blijft.

Files:

• CLAUDE.md (modified) (1 diff)
• src/config/database.js (modified) (2 diffs)
• src/services/ensurePrimarySite.js (modified) (2 diffs)
• test/site-permissions.test.js (moved) (moved from test/hub-permissions.test.js ) (1 diff)

CLAUDE.md

@@ -63 +63 @@
 Tests live in `test/*.test.js` and run against an in-memory SQLite
 (`DATABASE_PATH=':memory:'`), so they never touch real data. Cover new
-permission/tenancy logic with tests — `PermissionsService.canAdminSite` was once
-silently broken; see `test/hub-permissions.test.js`.
+permission logic with tests — `PermissionsService.canAdminSite` was once
+silently broken; see `test/site-permissions.test.js`.
 
 ## Architecture Overview

src/config/database.js

@@ -266 +266 @@
   `);
 
-  // Global app settings (key/value singleton). Includes the tenancy mode
-  // (solo = one site, hub = company site + /user/). Default = solo.
+  // Global app settings (key/value singleton). One instance is one owner, so
+  // there is no tenancy mode here anymore — see SettingsService.
+  // Oudere installaties dragen nog een dode rij key='tenancy' ('solo' of
+  // 'circle'). Niets leest hem; bewust laten staan (shaer-x7c0) in plaats van
+  // er opruimcode voor te schrijven die na één ronde zelf dood is.
   db.exec(`
     CREATE TABLE IF NOT EXISTS app_settings (
@@ -275 +278 @@
     );
   `);
-  db.prepare("INSERT OR IGNORE INTO app_settings (key, value) VALUES ('tenancy', 'solo')").run();
 
   // ── Statistics (premium) — cookie-free ──────────────────────

src/services/ensurePrimarySite.js

@@ -3 +3 @@
 
 // A Klonkt instance should ALWAYS have a primary site — it carries the identity
-// (title, theme, profile) and is the anchor point in solo/hub/circle mode.
+// (title, theme, profile) and is the anchor point for the whole instance.
 // The register flow already creates one, but an admin created via a script
 // (or an empty sites table for any reason) left the instance without a site:
@@ -10 +10 @@
 // This helper runs at boot (and is idempotent): as soon as there is an admin
 // but no site yet, it creates a default site owned by the first god/admin.
-// Tenancy-agnostic — applies to solo, hub, and circle.
 
 function defaultTitle() {

test/site-permissions.test.js

@@ -1 @@
-// Unit-tests voor de hub-permissielaag — juist `canAdminSite` was lang stil kapot
+// Unit-tests voor de site-permissielaag — juist `canAdminSite` was lang stil kapot
 // (las een nooit-gevulde user.siteRoles), dus dit dekt 'm nu af. Draait op de
 // ingebouwde node:test (geen extra deps). In-memory SQLite → raakt geen echte data.
+//
+// Heette hub-permissions.test.js: de hub-modus is weg (72ec6a4), maar wat hier
+// getest wordt niet. canAdminSite, canEditPost en getPrimarySite zijn springlevend
+// en dit bestand is hun enige dekking — alleen de naam verwees nog naar de modus.
 //
 // Run: npm test   (= node --test)