chore(deps): bump sea-orm & sea-orm-migration to 2.0 - #2422
Conversation
Combines the two Dependabot bumps (sea-orm constructorfabric#2412, sea-orm-migration constructorfabric#2411) into one change and migrates the backend to the SeaORM 2.0 API: - ConnectionTrait::{execute,query_one,query_all} now take a StatementBuilder by reference; raw sea_orm::Statement values go through the new {execute,query_one,query_all}_raw methods (by value). - sea_query Value::Bytes now holds Option<Vec<u8>> instead of Option<Box<Vec<u8>>>; drop the Box wrapper. - DatabaseConnection::Disconnected moved onto DatabaseConnectionType; use DatabaseConnection::default() for the disconnected sentinel. - Drop ConnectionTrait imports that are no longer needed at call sites. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Anton Zelenov <antonz@constructor.tech>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (36)
📝 WalkthroughWalkthroughSeaORM and SeaORM migration dependencies were upgraded to 2.0. Analytics and identity-resolution database operations now use raw query and execution APIs. UUID byte bindings now use direct vectors. Existing SQL, control flow, and result handling remain unchanged. ChangesSeaORM 2.0 compatibility migration
Estimated code review effort: 3 (Moderate) | ~25 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Consolidates the two Dependabot bumps into a single PR and makes the backend compile against the new major:
Both must move together — a split bump leaves two
sea_ormversions in the graph and theStatementtype stops matching across the boundary.SeaORM 2.0 API migration
ConnectionTrait::{execute,query_one,query_all}now take an&impl StatementBuilder. Rawsea_orm::Statementvalues run through the new{execute,query_one,query_all}_rawmethods (by value).sea_queryValue::Bytesnow holdsOption<Vec<u8>>instead ofOption<Box<Vec<u8>>>; theBoxwrapper is dropped.DatabaseConnection::Disconnectedmoved ontoDatabaseConnectionType; the disconnected sentinel is nowDatabaseConnection::default().ConnectionTraitimports that are no longer needed at call sites.Verification
cargo build --workspace— cleancargo clippy --workspace --all-targets— clean (workspace denies pedantic)cargo test --workspace --no-run— all test binaries linkcargo fmt,cargo clippy) — pass🤖 Generated with Claude Code
Summary by CodeRabbit
Maintenance
Tests