feat: sync missing evm-private changes into public evm - #21
Merged
Merged
Conversation
* chore(erc20/v2): align ack validation with ibc-go. (#1232) * After code review * Apply suggestion from @dianab-cl * After code review (cherry picked from commit 8be065c7c75c39a3fa30fcc57b256b733776edef)
align v2 with main so failed native-ERC20 conversion surfaces as failure result (cherry picked from commit 0b7b50212ba0948ae91229376d591f07f9db5fd5)
…43) (#1256) * fix: Move tests to use eoa instead of module account as deployer * Lint * Apply suggestion from @dianab-cl Co-authored-by: dianab-cl <diana@cosmoslabs.io> (cherry picked from commit de08b71040cad98cb11ce46f1ebb3f038e83cb59)
…) (#1253) * fix: harden statedb balance and event amount handling (#1176) * fix: harden statedb balance and event amount handling Guard StateDB balance subtraction against underflow and make precompile balance-event parsing denom-aware for base vs extended denom paths. Also add regression tests and document that only 18-decimal EVM gas-token chains are supported. * fix(vm): enforce 18-decimal coin configuration Reject non-18 decimal EVM coin configs in both runtime and test configurators, align affected tests, and temporarily exclude precisebank packages from root unit-test targets until precisebank removal lands. * chore: remove obsolete precisebank test package filters Now that contrib/x/precisebank is removed on main, package selection no longer needs explicit exclusions and can rely on the standard simulation/e2e filters. * chore: fix formatter ordering in scaling tests Apply golangci formatter output for scaling tests so gci/gofumpt checks pass in CI. * test: align integration suites with 18-decimal-only config Remove non-18-decimal integration cases and fee checks that now fail by design under enforced 18-decimal EVM coin configuration. --------- (cherry picked from commit 264aa70f18f4217b0354855a0c056e19c2f51b51) # Conflicts: # README.md # x/vm/statedb/state_object.go # x/vm/types/denom_config.go # x/vm/wrappers/feemarket_test.go * fix conflicts * fix formatting * remove the requirement that the EVM coin be configured with 18 decimals * sum the base and extended denom amounts when parsing balance change events --------- Co-authored-by: Alex | Cosmos Labs <alex@cosmoslabs.io> Co-authored-by: Matt Acciai <matt@cosmoslabs.io> (cherry picked from commit 82b3ef6c80887f5b3fb95b1f6ecb519f06202a7e)
(cherry picked from commit 01541fc783eeaefe43c51715eacee107af695b33)
Record SubBalance underflow (#1253) and StateDB Commit atomicity alongside the 0.6.2 cherry-picks.
…faces with corresponding mock implementations and tests
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 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 |
mattkii
force-pushed
the
feat/sync-evm
branch
2 times, most recently
from
September 23, 2026 16:42
96046ca to
8f31a33
Compare
jhelison
approved these changes
Sep 23, 2026
4 tasks
mattkii
added a commit
to KiiChain/kiichain
that referenced
this pull request
Sep 24, 2026
# Description Bump `cosmos/evm` replace to public [`KiiChain/evm v0.6.2-fork.2`](https://github.com/KiiChain/evm/releases/tag/v0.6.2-fork.2) (August 2026 security backports synced in KiiChain/evm#21). ## Type of change - [x] chore (Updates on dependencies, gitignore, etc) # How Has This Been Tested? - [x] `go mod tidy` / module resolves at `v0.6.2-fork.2` # PR Checklist: Make sure each step was done: - [x] Updated changelog with PR's intent - [ ] Lint with `make lint-fix`
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Sync missing
evm-privatechanges into publicevm(ERC20/IBC ack alignment, StateDB balance hardening, overflow guards, andIsBaseAccountOrEmpty).Closes: N/A
Author Checklist
I have...
mainbranch