Skip to content

Remove feegrant module - #3958

Merged
masih merged 6 commits into
mainfrom
masih/feegrant-be-gone
Aug 21, 2026
Merged

Remove feegrant module#3958
masih merged 6 commits into
mainfrom
masih/feegrant-be-gone

Clean up merges from main

0fa29cb
Select commit
Loading
Failed to load commit list.
Seidroid / AI Review succeeded Aug 20, 2026 in 0s

Claude + Codex + Cursor Review: success

Clean, thorough removal of the feegrant module: the fee-granter rejection is correctly placed at a single choke point (Tx.ValidateBasic, reached by both CosmosStatelessChecks and ValidateBasicDecorator), the v6.7 module-version delete is needed because SetModuleVersionMap is additive, and no dangling references remain. No blockers; a few non-blocking notes on flatkv store deletion, ante-level test coverage, historical tx decodability, and the 0x1010 "deterministic revert" claim.