Idrinth / habit-evaluator / 22285520396 / 3

40%

the-one: 40%

push

github

Fix Android crash when loading data after migration to Room (#566)

The Kotlin/Compose migration (#559) introduced two crash scenarios:

1. Pre-Room database conflict: The old SQLiteHelper and new Room both used
   "habit_evaluator.db" (version 10). Room cannot open a database lacking
   its room_master_table metadata, causing IllegalStateException on first
   data access. Fix: detect pre-Room databases before Room initializes,
   rename them to a legacy file, add fallbackToDestructiveMigration() as
   safety net, and migrate data via new LegacySqliteToRoomMigration class.

2. Empty string parsing in EntityMappers: toEntity() methods stored "" for
   null time/date fields, but toModel() called LocalTime.parse("") which
   throws DateTimeParseException. Fix: add safe parsing helpers that return
   null for empty/invalid strings instead of crashing.

Also moved migration code off the main thread to prevent ANR during
database initialization.

https://claude.ai/code/session_01F33UgLjkt8TQ9h4TXBmmv9

Co-authored-by: Claude <noreply@anthropic.com>

60 of 75 branches covered (80.0%)

Branch coverage included in aggregate %.

143 of 154 relevant lines covered (92.86%)

20.53 hits per line