Skip to content

fix(store): upgrade exact legacy v97 databases - #127

Merged
Qiyuanqiii merged 2 commits into
mainfrom
codex/rc2-v97-compatibility
Aug 22, 2026
Merged

Qiyuanqiii merged 2 commits into
mainfrom
codex/rc2-v97-compatibility

Conversation

@Qiyuanqiii

Copy link
Copy Markdown
Member

Summary

  • accept only the exact pre-final Windows preview checksum for migration v97
  • add schema v125 to transactionally restore the canonical Docker lifecycle cleanup trigger while preserving the historical v97 row
  • extend the cumulative historical migration fixture chain and document that the portable ZIP still uses the shared user data home

Safety boundary

  • no database deletion, reset, or checksum rewriting
  • unknown checksums, wrong names, migration gaps, and newer histories still fail closed
  • published v0.1.0-rc.2 assets remain immutable; this repair is intended for a later prerelease

Validation

  • targeted v97/v125 migration and rejection tests
  • go test ./internal/store -count=1 -timeout 30m (passed in 756.866s)
  • go vet ./internal/store
  • 289 frontend tests plus production renderer build
  • Desktop Go/Wails tests and reproducible production-tag EXE build
  • final EXE launched against a temporary copy of the affected legacy database, upgraded it to v125 with PRAGMA integrity_check = ok, preserved the v97 checksum, and left the source database hash unchanged

Release follow-up

After required PR checks pass and this PR is merged, publish v0.1.0-rc.3 only through the tag-triggered Desktop release workflow.

@Qiyuanqiii

Copy link
Copy Markdown
Member Author

CI follow-up: the first Go job had no assertion failure; internal/store was interrupted exactly at the existing 30-minute package timeout. The rc.2 main run completed Store in 1050.129s, while the v125 run reached 1800.133s under shared-runner contention. Commit b971634 raises the still-bounded complete-suite timeout to 45 minutes without skipping, caching, or narrowing any test. All other CI jobs and the full Desktop release candidate workflow passed on the first attempt.

@Qiyuanqiii
Qiyuanqiii marked this pull request as ready for review August 22, 2026 05:29
@Qiyuanqiii
Qiyuanqiii merged commit fe138da into main Aug 22, 2026
12 checks passed
@NanaseInori
NanaseInori deleted the codex/rc2-v97-compatibility branch September 16, 2026 09:09
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.

1 participant