6b7d434c3c
The 20260507164347_add_workspace_time migration shipped as
ALTER TABLE \`workspace\` ADD \`time_used\` integer NOT NULL;
bun:sqlite (and standard SQLite) refuse to add a NOT NULL column
without a DEFAULT to a non-empty table. Anyone upgrading from
v1.14.41 (Hono) to v1.14.42 with populated workspace rows hit
"Cannot add a NOT NULL column with default value NULL" at startup
and the app crashed before reaching the TUI.
Dev users hadn't noticed because they pulled the migration when
their workspace table was empty (or under a fresh fixture); the
ALTER then succeeded and was journaled. State C (empty table at
migration time = success), state A/B (populated table at migration
time = crash) — only state A/B users hit the bug, and only on
upgrade from v1.14.41.
Replace with a table-recreate pattern that is idempotent across
both states. The SELECT pulls only columns from the v1.14.41
schema, so it works whether time_used exists or not. The new
table defines time_used with DEFAULT 0; existing rows reset to 0,
which is acceptable because time_used is a usage counter the app
re-derives anyway and is brand new in v1.14.42.
Verified locally on bun:sqlite for both states.
|
||
|---|---|---|
| .. | ||
| 20260127222353_familiar_lady_ursula | ||
| 20260211171708_add_project_commands | ||
| 20260213144116_wakeful_the_professor | ||
| 20260225215848_workspace | ||
| 20260227213759_add_session_workspace_id | ||
| 20260228203230_blue_harpoon | ||
| 20260303231226_add_workspace_fields | ||
| 20260309230000_move_org_to_state | ||
| 20260312043431_session_message_cursor | ||
| 20260323234822_events | ||
| 20260410174513_workspace-name | ||
| 20260413175956_chief_energizer | ||
| 20260423070820_add_icon_url_override | ||
| 20260427172553_slow_nightmare | ||
| 20260428004200_add_session_path | ||
| 20260501142318_next_venus | ||
| 20260504145000_add_sync_owner | ||
| 20260507164347_add_workspace_time | ||