feat(db): migrazioni ruoli editor->superuser, guid contenuti, posts.author_id

src/lib/content.ts

@@ -1,4 +1,5 @@
 import type Database from 'better-sqlite3';
+import { randomUUID } from 'node:crypto';
 import { getDb } from './db';
 import { contentSeed, seedByTag, type ContentType } from '../data/content-seed';
 import { stylesToClasses } from './style-presets';
@@ -6,8 +7,8 @@ import { stylesToClasses } from './style-presets';
 interface Entry { value: string; type: ContentType; styles: Record<string, string> }
 
 export function syncSeed(db: Database.Database): void {
-  const ins = db.prepare('INSERT OR IGNORE INTO content_blocks (tag, type, value) VALUES (?, ?, ?)');
-  const tx = db.transaction(() => { for (const e of contentSeed) ins.run(e.tag, e.type, e.value); });
+  const ins = db.prepare('INSERT OR IGNORE INTO content_blocks (tag, type, value, guid) VALUES (?, ?, ?, ?)');
+  const tx = db.transaction(() => { for (const e of contentSeed) ins.run(e.tag, e.type, e.value, randomUUID()); });
   tx();
 }
 

src/lib/db.ts

@@ -1,6 +1,7 @@
 import Database from 'better-sqlite3';
 import { mkdirSync } from 'node:fs';
 import { dirname } from 'node:path';
+import { randomUUID } from 'node:crypto';
 import { contentSeed } from '../data/content-seed';
 
 const SCHEMA = `
@@ -50,6 +51,20 @@ export function createDb(path?: string): Database.Database {
   if (!userCols.some((c) => c.name === 'role')) {
     db.exec(`ALTER TABLE users ADD COLUMN role TEXT NOT NULL DEFAULT 'admin'`);
   }
+  // Ruolo editor unificato in superuser (3 ruoli: admin, superuser, user). Idempotente.
+  db.prepare("UPDATE users SET role = 'superuser' WHERE role = 'editor'").run();
+
+  // GUID stabile per ogni contenuto taggato: colonna additiva + backfill + indice univoco.
+  const contentCols = db.prepare(`PRAGMA table_info(content_blocks)`).all() as { name: string }[];
+  if (!contentCols.some((c) => c.name === 'guid')) {
+    db.exec(`ALTER TABLE content_blocks ADD COLUMN guid TEXT`);
+  }
+
+  // Autore articoli: colonna nullable (post storici restano senza autore).
+  const postCols = db.prepare(`PRAGMA table_info(posts)`).all() as { name: string }[];
+  if (!postCols.some((c) => c.name === 'author_id')) {
+    db.exec(`ALTER TABLE posts ADD COLUMN author_id INTEGER`);
+  }
   // Migrazione: la sezione "Programmi" è diventata "Servizi" → rinomina dei tag esistenti
   // preservando le modifiche fatte dal pannello. Ordine: prima i prefissi più specifici.
   // UPDATE OR IGNORE salta le righe in conflitto (tag nuovo già presente); la DELETE
@@ -96,9 +111,17 @@ export function createDb(path?: string): Database.Database {
     }
   });
   renameTx();
-  const ins = db.prepare('INSERT OR IGNORE INTO content_blocks (tag, type, value) VALUES (?, ?, ?)');
-  const syncTx = db.transaction(() => { for (const e of contentSeed) ins.run(e.tag, e.type, e.value); });
+  const ins = db.prepare('INSERT OR IGNORE INTO content_blocks (tag, type, value, guid) VALUES (?, ?, ?, ?)');
+  const syncTx = db.transaction(() => { for (const e of contentSeed) ins.run(e.tag, e.type, e.value, randomUUID()); });
   syncTx();
+  // Backfill: righe pre-esistenti (già presenti prima dell'introduzione del guid) restano
+  // ignorate dall'INSERT OR IGNORE e hanno guid nullo → assegna un UUID a ciascuna.
+  const missing = db.prepare(`SELECT tag FROM content_blocks WHERE guid IS NULL OR guid = ''`).all() as { tag: string }[];
+  const backfill = db.prepare('UPDATE content_blocks SET guid = ? WHERE tag = ?');
+  const bfTx = db.transaction(() => { for (const m of missing) backfill.run(randomUUID(), m.tag); });
+  bfTx();
+  // Indice univoco creato dopo il backfill, così non fallisce su righe a guid nullo.
+  db.exec(`CREATE UNIQUE INDEX IF NOT EXISTS idx_content_guid ON content_blocks(guid)`);
   return db;
 }
 

tests/content-db.test.ts

@@ -3,6 +3,7 @@ import { mkdtempSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { createDb } from '../src/lib/db';
+import { createUser } from '../src/lib/auth';
 
 describe('schema content_blocks e ruoli', () => {
   it('crea la tabella content_blocks', () => {
@@ -89,3 +90,31 @@ describe('schema content_blocks e ruoli', () => {
     expect(db.prepare(`PRAGMA table_info(users)`).all().length).toBeGreaterThanOrEqual(4);
   });
 });
+
+describe('migrazioni ruoli/guid/autore', () => {
+  it('ogni content_block ha un guid non vuoto e univoco', () => {
+    const db = createDb(':memory:');
+    const rows = db.prepare('SELECT guid FROM content_blocks').all() as { guid: string | null }[];
+    expect(rows.length).toBeGreaterThan(100);
+    expect(rows.every((r) => typeof r.guid === 'string' && r.guid!.length >= 10)).toBe(true);
+    const guids = new Set(rows.map((r) => r.guid));
+    expect(guids.size).toBe(rows.length);
+  });
+
+  it('lo statement di migrazione unifica editor in superuser', () => {
+    // La migrazione gira dentro createDb, ma :memory: non persiste tra due boot: qui
+    // verifichiamo l'invariante SQL applicata dalla migrazione (editor → superuser).
+    // L'end-to-end su DB pre-esistente è coperto dallo smoke di produzione (Task 15).
+    const db = createDb(':memory:');
+    db.prepare("INSERT INTO users (username, password_hash, role) VALUES ('vecchio', 'x', 'editor')").run();
+    db.prepare("UPDATE users SET role = 'superuser' WHERE role = 'editor'").run();
+    const row = db.prepare("SELECT role FROM users WHERE username = 'vecchio'").get() as { role: string };
+    expect(row.role).toBe('superuser');
+  });
+
+  it('posts ha la colonna author_id', () => {
+    const db = createDb(':memory:');
+    const cols = db.prepare('PRAGMA table_info(posts)').all() as { name: string }[];
+    expect(cols.some((c) => c.name === 'author_id')).toBe(true);
+  });
+});