Idrinth / habit-evaluator / 22285520396

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>

2631 of 6734 branches covered (39.07%)

Branch coverage included in aggregate %.

9276 of 21011 relevant lines covered (44.15%)

2.23 hits per line