Changeset 7881080 in Klonkt for src/routes/admin.js

Timestamp:

06/18/2026 03:28:34 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

8398935

Parents:

98ecf51

Message:

Hub #3: explicit is_primary flag + shared getPrimarySite() (DRY)

The "oldest site = primary site" assumption was duplicated independently in
resolveSite, hub.js, account.js and admin.js (fragile: if the oldest happened
to be an artist site, the hub home would be wrong). Now:

• sites.is_primary column + backfill (marks the oldest if none is primary yet; ensurePrimarySite sets it on fresh installs) → existing behaviour exactly preserved.
• one getPrimarySite() helper (is_primary, fallback oldest) replaces the 4 copies.
• god can CHOOSE the primary/main site: ★ button + "primary" badge on /admin/sites (exactly one primary via a transaction).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@…>

File:

• src/routes/admin.js (modified) (2 diffs)

src/routes/admin.js

@@ -10 +10 @@
 import { requireAuth } from '../middleware/auth.js';
 import { getTenancy } from '../services/SettingsService.js';
+import { getPrimarySite } from '../middleware/site.js';
 
 const router = express.Router();
@@ -59 +60 @@
   const tenancy = getTenancy();
 
-  // De primaire/owner-site — in solo dé site, in hub de hoofdsite. Geeft de
+  // De primaire/hoofd-site — in solo dé site, in hub de hoofdsite. Geeft de
   // "Uiterlijk"-tegel z'n edit-link + de posts/concepten-lijst.
-  const primarySite = db.prepare(
-    'SELECT id, slug, title FROM sites ORDER BY created_at ASC LIMIT 1'
-  ).get() || null;
+  const primarySite = getPrimarySite();
 
   const stats = {