InsanityLab-web/.superpowers/sdd/task-1-report.md

Task 1 — Report: migrazioni DB (ruoli, guid, author_id)

Nota: questo file conteneva un report di un run precedente (task diverso, "content_blocks

• role") — sovrascritto con il report corretto del Task 1 corrente ("migrazioni ruoli, guid contenuti, posts.author_id").

Cosa è stato fatto

Seguito TDD come da brief task-1-brief.md, codice usato verbatim.

1. Test scritti in tests/content-db.test.ts: aggiunto import createUser da ../src/lib/auth e il blocco describe('migrazioni ruoli/guid/autore', ...) con le 3 it: guid non vuoto/univoco, invariante SQL editor→superuser, colonna posts.author_id.
2. Verificato fallimento atteso: npm test -- tests/content-db.test.ts → 2 test falliti (no such column: guid, author_id assente), 6 preesistenti passati. Comportamento pre-migrazione confermato.
3. Migrazioni implementate in src/lib/db.ts:
   • import randomUUID da node:crypto.
   • dopo il blocco che aggiunge users.role, inserita: unificazione role='editor' → 'superuser' (UPDATE idempotente), aggiunta colonna additiva content_blocks.guid TEXT (con check PRAGMA table_info), aggiunta colonna additiva posts.author_id INTEGER (stesso check).
   • blocco seed finale sostituito: insert ora include guid generato con randomUUID(); aggiunto backfill per righe pre-esistenti con guid nullo/vuoto; creato indice univoco idx_content_guid DOPO il backfill (per non fallire su righe nulle transitorie).
4. syncSeed in src/lib/content.ts aggiornata allo stesso schema: import randomUUID, insert con guid generato per ogni riga.
5. Verificato successo: npm test -- tests/content-db.test.ts e poi npm test completo (tutta la suite) → tutti verdi.
6. Build: npm run build → completata senza errori.
7. Commit: 5cea895 feat(db): migrazioni ruoli editor->superuser, guid contenuti, posts.author_id (3 file: src/lib/db.ts, src/lib/content.ts, tests/content-db.test.ts).

File toccati

• src/lib/db.ts — migrazioni additive/idempotenti + seed con guid + backfill + indice univoco.
• src/lib/content.ts — syncSeed allineata (guid su ogni insert).
• tests/content-db.test.ts — nuovo describe block per le 3 verifiche del task.

Output test (finale, suite completa)

 Test Files  11 passed (11)
      Tests  51 passed (51)
   Start at  21:40:18
   Duration  2.62s

Output build

[build] Server built in 434ms
[build] Complete!

Dubbi / note

• Nessun dubbio bloccante. Rispettato il vincolo "migrazioni SOLO additive e idempotenti, PRIMA del seed, con verifica PRAGMA table_info prima di ogni ALTER" — verificato a mano leggendo db.ts dopo l'edit.
• Il test importa createUser da ../src/lib/auth ma non lo invoca in nessuna it (fedele al brief verbatim); nessun errore di lint/build osservato per l'import inutilizzato. Se in futuro un linter con regola no-unused-vars stretta viene attivato su questo file, andrà rivisto (fuori scope Task 1).
• La migrazione editor→superuser end-to-end su DB persistente non è testata qui (per design, :memory: non persiste tra due createDb) — demandata allo smoke di produzione previsto in un task successivo del piano (Task 14/15), come indicato nel brief stesso.