fix(desktop): make migration 11 idempotent by dropping and recreating terminal_persistence column#822
Conversation
… terminal_persistence column
📝 WalkthroughWalkthroughThis PR bumps the desktop application version from 0.0.56 to 0.0.57 and introduces a database migration that drops and re-adds the terminal_persistence column, along with updating associated migration metadata timestamps and snapshot records. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🧹 Preview Cleanup CompleteThe following preview resources have been cleaned up:
Thank you for your contribution! 🎉 |
Fixes migration 11 failing in production by making it idempotent.
Problem
Migration 11 was failing in production, likely because the column was partially added in a previous failed migration attempt.
Solution
Modified the migration to:
terminal_persistencecolumn if it existsThis ensures the migration can run successfully even if previously partially applied.
Testing
desktop-canary(v0.0.57-canary.20260119025121)Migration Change
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.