Skip to content

Handle prototype migration identifiers safely - #16

Merged
DX23876 merged 1 commit into
mainfrom
fix/prototype-migration-compat
Aug 25, 2026
Merged

DX23876 merged 1 commit into
mainfrom
fix/prototype-migration-compat

Conversation

@DX23876

@DX23876 DX23876 commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • recognize the unpublished v42 body-weight migration only when its exact legacy marker exists
  • recognize the unpublished v43 energy-coverage migration only when its exact legacy marker exists
  • preserve existing rows and record the released v43/v44 migration identifiers without masking unrelated schema collisions
  • add a regression test that recreates the prototype database and verifies a data-preserving upgrade

Validation

  • WhoopStore: 485 tests passed, 1 skipped
  • prototype migration regression: passed
  • concurrent fresh-file opens: 16/16 passed
  • universal macOS build: passed
  • generic iOS build including Watch and Widgets: passed
  • i18n audit and git diff --check: passed

Copilot AI balanced review requested due to automatic review settings August 25, 2026 01:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@DX23876
DX23876 merged commit fca98fb into main Aug 25, 2026
19 checks passed
@DX23876
DX23876 deleted the fix/prototype-migration-compat branch August 25, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants