[IMPORTANT] ALTER TABLE catch slikt alle fouten — ook serieuze #6

New issue

Open

opened 2026-08-05 21:11:31 +02:00 by eddy · 0 comments

eddy commented

2026-08-05 21:11:31 +02:00

Owner

Probleem

src/database/db.ts:28-30:

try {
  await db.execAsync('ALTER TABLE measurements ADD COLUMN deletedAt TEXT');
} catch {
  // Column already exists, ignore
}

De lege catch {} is bedoeld om de "column already exists" fout op te vangen bij herhaaldelijk initialiseren. Maar het vangt ook serieuze fouten:

Disk full
Corrupte database
Permission errors

Bij een corrupte database gaat de app gewoon door alsof alles in orde is, terwijl data-operaties vervolgens stilletjes falen.

Oplossing

Gebruik PRAGMA table_info om te checken of de kolom al bestaat vóór de ALTER:

const columns = await db.getAllAsync<{ name: string }>(
  'PRAGMA table_info(measurements)'
);
const hasDeletedAt = columns.some(col => col.name === 'deletedAt');
if (!hasDeletedAt) {
  await db.execAsync('ALTER TABLE measurements ADD COLUMN deletedAt TEXT');
}

Of als alternatief: check expliciet op "duplicate column" in de catch en re-throw bij andere fouten.

## Probleem `src/database/db.ts:28-30`: ```ts try { await db.execAsync('ALTER TABLE measurements ADD COLUMN deletedAt TEXT'); } catch { // Column already exists, ignore } ``` De lege `catch {}` is bedoeld om de "column already exists" fout op te vangen bij herhaaldelijk initialiseren. Maar het vangt ook serieuze fouten: - Disk full - Corrupte database - Permission errors Bij een corrupte database gaat de app gewoon door alsof alles in orde is, terwijl data-operaties vervolgens stilletjes falen. ## Oplossing Gebruik `PRAGMA table_info` om te checken of de kolom al bestaat vóór de ALTER: ```ts const columns = await db.getAllAsync<{ name: string }>( 'PRAGMA table_info(measurements)' ); const hasDeletedAt = columns.some(col => col.name === 'deletedAt'); if (!hasDeletedAt) { await db.execAsync('ALTER TABLE measurements ADD COLUMN deletedAt TEXT'); } ``` Of als alternatief: check expliciet op "duplicate column" in de catch en re-throw bij andere fouten.

eddy added this to the Codereview augustus 2025 milestone

2026-08-05 21:14:30 +02:00