[MINOR] syncStatus optioneel in type maar verplicht in database #10

New issue

Open

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

eddy commented

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

Owner

Probleem

src/types.ts:14 declareert syncStatus?: string (optioneel), maar de database schema (src/database/db.ts:17) definieert syncStatus TEXT NOT NULL DEFAULT 'pending'. De kolom is dus altijd gevuld — syncStatus is nooit daadwerkelijk undefined voor persisted records.

Dit is een kleine type-inconsistentie. In getPendingRecords() (db.ts:114) wordt syncStatus gebruikt zonder null-check, wat aangeeft dat het in de praktijk altijd aanwezig is.

Oplossing

Maak syncStatus verplicht in het type:

type Measurement = {
  ...
- syncStatus?: string;
+ syncStatus: string;
  deletedAt?: string;
}

deletedAt blijft optioneel — die is pas gezet bij een soft delete.

## Probleem `src/types.ts:14` declareert `syncStatus?: string` (optioneel), maar de database schema (`src/database/db.ts:17`) definieert `syncStatus TEXT NOT NULL DEFAULT 'pending'`. De kolom is dus altijd gevuld — `syncStatus` is nooit daadwerkelijk `undefined` voor persisted records. Dit is een kleine type-inconsistentie. In `getPendingRecords()` (db.ts:114) wordt `syncStatus` gebruikt zonder null-check, wat aangeeft dat het in de praktijk altijd aanwezig is. ## Oplossing Maak `syncStatus` verplicht in het type: ```diff type Measurement = { ... - syncStatus?: string; + syncStatus: string; deletedAt?: string; } ``` `deletedAt` blijft optioneel — die is pas gezet bij een soft delete.

eddy added this to the Codereview augustus 2025 milestone

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