Remove live IBC transfer module - #3981
Conversation
Remove ICS-20 module wiring, APIs, CLI commands, and CosmWasm integration. Retire the live IBC precompile while preserving legacy implementations and codecs for historical tracing and transaction decoding. Keep the transfer store and module account mounted for historical state safety, and remove its module version during the v6.7 upgrade.
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3981 +/- ##
==========================================
- Coverage 59.08% 57.58% -1.50%
==========================================
Files 2305 2225 -80
Lines 197021 185842 -11179
==========================================
- Hits 116402 107011 -9391
+ Misses 69858 69048 -810
+ Partials 10761 9783 -978
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
PR SummaryHigh Risk Overview State compatibility is preserved by keeping the Operator and client impact is documented in Reviewed by Cursor Bugbot for commit 5817e97. Bugbot is set up for automated code reviews on this repo. Configure here. |
6f8297d to
cd3c443
Compare
Remove the remaining transfer keeper, codecs, protobufs, metrics, and functional legacy IBC precompiles. Route pre-v6.7 transfer history to v6.6 freeze nodes and map all IBC precompile versions to the deterministic retirement tombstone. Retain the transfer store and module account for state compatibility, along with raw Wasm packet handling and callbacks.
# Conflicts: # sei-ibc-go/modules/apps/transfer/keeper/grpc_query.go
Remove ICS-20 module wiring, APIs, CLI commands, and CosmWasm integration.
Retire the live IBC precompile while preserving legacy implementations and codecs for historical tracing and transaction decoding. Keep the transfer store and module account mounted for historical state safety, and remove its module version during the v6.7 upgrade.