Skip to content

[WIP] Backport upstream changes#2

Draft
maxim-inj wants to merge 287 commits into
InjectiveLabs:developfrom
crypto-org-chain:develop
Draft

[WIP] Backport upstream changes#2
maxim-inj wants to merge 287 commits into
InjectiveLabs:developfrom
crypto-org-chain:develop

Conversation

@maxim-inj

@maxim-inj maxim-inj commented Sep 7, 2024

Copy link
Copy Markdown

Need to evaluate and backport some or all fixes from the upstream.

  • Problem: gasWanted is not aligned for process proposal mode (#773438) ⚠️
  • Problem: no way to disable nonce checking in benchmark (#f9cd15)
  • Problem: no MaxTx set from config for mempool (#e5b222)
  • Problem: insufficient invalid value check in eth transaction (#45ac08)
  • Problem: incarnation cache not enabled (#9c959a)
  • Problem: no overflow check when add gasLimit to gasWanted (#d874df) ⚠️
  • Problem: invalid chain id for signer (#f3e62c) ⚠️
  • Problem: benchmark logic don't increase nonce (#350955)
  • Problem: cometbft not up to date (#c36b26)
  • Problem: disable of create vesting account messages are not complete (#b0022e)
  • Optimize AnteHandle method to skip checks if disabledMsgs is empty (#ff4caa)
  • Problem: vote_extensions is not disabled in test (#0e0baa)
  • Problem: no debug files uploaded when timeout in ci (#e3b813)
  • Problem: nondeterministic account set occurs with more stm workers (#7ef8af)
  • Problem: unnecessary GetAccount in ante handlers (#fe3f4f)
  • Problem: handle for integer overflow is messy (#d26ada) ⚠️
  • Problem: method eth_chainId crashed occasionally(#452b0c) ⚠️
  • Problem: hash in subscribe newHeads differs from eth_getBlockByNumber (#3fabdbe) ⚠️
  • Problem: block-stm tx executor bad worst case performance (#79bb39e)
  • Problem: pre-estimation don't run in parallel (#56f8a5)
  • Problem: tx evm raw doesn't work under offline (#dbc7eb4)
  • Problem: get unnecessary block result when only need header (#7e30762)
  • Problem: trace block gas refund change not committed (#bcac9e2) ⚠️
  • Problem: db is double closed when shutdown (#f2a562b) ⚠️
  • Problem: no time in Transaction (#9755333) ⚠️
  • Problem: multisig account failed on threshold encode after send tx #33e3cc6
  • Problem: dependencies are outdated #5acb559
  • Problem: get unnecessary block result in header related api call #b16b489
  • Problem: unsuppored sign mode SIGN_MODE_TEXTUAL in bank transfer #c4cef0f
  • Problem: validation broke after transaction conversion with raw field #617eac9
  • Problem: opBlockhash broke after sdk50 #853e1e5 ⚠️
  • Problem: default headerHashNum is too large #580ce6e ⚠️
  • Problem: no header hash from fallback historicalInfo #a2ad87c ⚠️
  • Problem: node can't quit by signal #816389c ⚠️

Additional issues

  • Problem: reverted Evm call doesn't revert Tx, other msgs in Tx still execute ⚠️
  • Problem: fee collector account failed to refund fees: the check always considers that 100% of gas is refundable
  • Problem: need to support eth_getBlockReceipts see Get receipts by block number ethereum/go-ethereum#19634 ⚠️

@maxim-inj maxim-inj self-assigned this Sep 7, 2024
@coderabbitai

coderabbitai Bot commented Sep 7, 2024

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 95e860b7-2ccf-4d8e-8663-a3b92efcc199

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@maxim-inj maxim-inj changed the title Backport upstream changes [WIP] Backport upstream changes Sep 7, 2024
valli0x and others added 27 commits October 30, 2024 10:32
* graceful shutdown of websocket

* context for unsubscribe logs

* added a logger to the log subscription

* lint

---------

Co-authored-by: mmsqe <mavis@crypto.com>
* Problem: build fail without cgo

* Update CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

* revert

---------

Signed-off-by: yihuang <huang@crypto.com>
* Problem: rpc stream overhead even if not used

Solution:
- init the underlying subscriptions lazily.

* Update CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

* fix build

* ignore pending tx

---------

Signed-off-by: yihuang <huang@crypto.com>
* removed listener closing in gracefully shutdown json-rpc server

* just logging error from shutdown json-rpc server

---------

Co-authored-by: yihuang <huang@crypto.com>
* Problem: check-tx don't run in parallel

Solution:
- try with experimental flag

* fix test

---------

Co-authored-by: mmsqe <mavis@crypto.com>
* Problem: no trace detail on insufficient balance

* skip check

* simple reproduce
* Problem: incorrect balance when no tracer in predecessors

* lint
…trace tx (#559)

* Problem: wrong gas price when using basefee of minus one height when trace tx

* add basefee req

---------

Signed-off-by: mmsqe <tqd0800210105@gmail.com>
* Problem: nil pointer error with legacy tx format

* Update CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

---------

Signed-off-by: yihuang <huang@crypto.com>
* Problem: cometbft is outdated

* toolchain

* align gomod2nix

* update flake lock

---------

Co-authored-by: HuangYi <huang@crypto.com>
* Problem: nonce management in batch tx

* changelog
* Problem: latest sdk requires a minimum version of go 1.23.3

* align flake lock

---------

Co-authored-by: HuangYi <huang@crypto.com>
* Problem: nixpkgs are outdated

* use upstream
* Problem: CacheWrapWithTrace is not supported

* align nixpkgs

* deps

* fix build
* Problem: incorrect spendable balance when debug trace tx

* fix

* Update tests/integration_tests/test_tracers.py

Co-authored-by: mmsqe <tqd0800210105@gmail.com>
Signed-off-by: yihuang <huang@crypto.com>

---------

Signed-off-by: yihuang <huang@crypto.com>
Co-authored-by: yihuang <huang@crypto.com>
…c rpc (#576)

* Problem: no specific dir for the log from debug_goTrace

* lint

* allow any

* revert

* cleanup

* fix format

* fix config

* Apply suggestions from code review

---------

Co-authored-by: HuangYi <huang@crypto.com>
* Problem: eth_getLogs miss logs with batch transactions.

* Update CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

---------

Signed-off-by: yihuang <huang@crypto.com>
* build(deps): bump golang.org/x/net from 0.12.0 to 0.15.0

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.12.0 to 0.15.0.
- [Commits](golang/net@v0.12.0...v0.15.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: vulncheck

* feat: upgrade go version in nix

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xinyu Zhao <xinyu.zhao@crypto.com>
* update deps and fix script

* fix labeler

* upgrade deps

---------

Co-authored-by: Xinyu <91446598+XinyuCRO@users.noreply.github.com>
* build(deps): bump github.com/onsi/gomega from 1.26.0 to 1.27.10

Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.26.0 to 1.27.10.
- [Release notes](https://github.com/onsi/gomega/releases)
- [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md)
- [Commits](onsi/gomega@v1.26.0...v1.27.10)

---
updated-dependencies:
- dependency-name: github.com/onsi/gomega
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* ci: use codecov/codecov-action@v5

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xinyu Zhao <xinyu.zhao@crypto.com>
* build(deps): bump github.com/onsi/ginkgo/v2 from 2.7.0 to 2.12.0

Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.7.0 to 2.12.0.
- [Release notes](https://github.com/onsi/ginkgo/releases)
- [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md)
- [Commits](onsi/ginkgo@v2.7.0...v2.12.0)

---
updated-dependencies:
- dependency-name: github.com/onsi/ginkgo/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: typo

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xinyu Zhao <xinyu.zhao@crypto.com>
…102358-957c09536969 to 0.3.0 (#334)

* build(deps): bump github.com/status-im/keycard-go

Bumps [github.com/status-im/keycard-go](https://github.com/status-im/keycard-go) from 0.0.0-20200402102358-957c09536969 to 0.3.0.
- [Release notes](https://github.com/status-im/keycard-go/releases)
- [Commits](https://github.com/status-im/keycard-go/commits/v0.3.0)

---
updated-dependencies:
- dependency-name: github.com/status-im/keycard-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: type

to acquire codecov token in old branch

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xinyu Zhao <xinyu.zhao@crypto.com>
Bumps [github.com/cosmos/cosmos-db](https://github.com/cosmos/cosmos-db) from 1.0.3-0.20240408151834-e75f6e4b28d8 to 1.1.3.
- [Changelog](https://github.com/cosmos/cosmos-db/blob/main/CHANGELOG.md)
- [Commits](https://github.com/cosmos/cosmos-db/commits/v1.1.3)

---
updated-dependencies:
- dependency-name: github.com/cosmos/cosmos-db
  dependency-version: 1.1.3
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
JayT106 and others added 30 commits May 12, 2026 23:43
v5/v6 requires changed-files key with glob objects instead of flat glob list.
Bumps the gh-actions group with 2 updates: [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action) and [actions/create-github-app-token](https://github.com/actions/create-github-app-token).


Updates `anthropics/claude-code-action` from 1.0.120 to 1.0.121
- [Release notes](https://github.com/anthropics/claude-code-action/releases)
- [Commits](anthropics/claude-code-action@dde2242...f4fb5c6)

Updates `actions/create-github-app-token` from 3.1.1 to 3.2.0
- [Release notes](https://github.com/actions/create-github-app-token/releases)
- [Changelog](https://github.com/actions/create-github-app-token/blob/main/CHANGELOG.md)
- [Commits](actions/create-github-app-token@1b10c78...bcd2ba4)

---
updated-dependencies:
- dependency-name: anthropics/claude-code-action
  dependency-version: 1.0.121
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: gh-actions
- dependency-name: actions/create-github-app-token
  dependency-version: 3.2.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: gh-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…-indexing (#939)

* extend cli

* feat(server): add range option to index-eth-tx for targeted block re-indexing

Add `index-eth-tx range <start> <end>` subcommand to allow re-indexing a
specific inclusive block range in the KV indexer, overwriting any existing
entries. This enables fixing missing block-gas-exceeded tx entries without
requiring a full DB wipe and re-index from scratch.

* changelog: add entry for #939 index-eth-tx range subcommand

---------

Co-authored-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
* build(deps): bump the pip group across 1 directory with 3 updates

---
updated-dependencies:
- dependency-name: protobuf
  dependency-version: 5.29.6
  dependency-type: direct:production
  dependency-group: pip
- dependency-name: aiohttp
  dependency-version: 3.13.3
  dependency-type: indirect
  dependency-group: pip
- dependency-name: urllib3
  dependency-version: 2.6.3
  dependency-type: indirect
  dependency-group: pip
...

Signed-off-by: dependabot[bot] <support@github.com>

* update gomod2nix

* update poetry2nix

* update poetry2nix in nix instead of flake

* patch poetry2nix

* revert irrelevant changes

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Randy Ang <randy.ang@cronoslabs.org>
… updates + update to ethers v6 (#942)

* build(deps): bump the npm_and_yarn group across 2 directories with 11 updates + update to ethers v6

Bumps the npm_and_yarn group with 8 updates in the /tests/integration_tests/hardhat directory:

| Package | From | To |
| --- | --- | --- |
| [brace-expansion](https://github.com/juliangruber/brace-expansion) | `1.1.12` | `1.1.14` |
| [brace-expansion](https://github.com/juliangruber/brace-expansion) | `2.0.2` | `2.1.0` |
| [follow-redirects](https://github.com/follow-redirects/follow-redirects) | `1.15.9` | `1.16.0` |
| [immutable](https://github.com/immutable-js/immutable-js) | `4.3.4` | `4.3.8` |
| [lodash](https://github.com/lodash/lodash) | `4.17.23` | `4.18.1` |
| [minimatch](https://github.com/isaacs/minimatch) | `3.1.2` | `3.1.5` |
| [minimatch](https://github.com/isaacs/minimatch) | `5.1.6` | `5.1.9` |
| [picomatch](https://github.com/micromatch/picomatch) | `2.3.1` | `2.3.2` |
| [serialize-javascript](https://github.com/yahoo/serialize-javascript) | `6.0.2` | `removed` |
| [undici](https://github.com/nodejs/undici) | `5.29.0` | `6.25.0` |

Bumps the npm_and_yarn group with 6 updates in the /tests/solidity directory:

| Package | From | To |
| --- | --- | --- |
| [brace-expansion](https://github.com/juliangruber/brace-expansion) | `1.1.12` | `1.1.14` |
| [follow-redirects](https://github.com/follow-redirects/follow-redirects) | `1.15.9` | `1.16.0` |
| [picomatch](https://github.com/micromatch/picomatch) | `2.3.1` | `2.3.2` |
| [protobufjs](https://github.com/protobufjs/protobuf.js) | `6.11.4` | `6.11.6` |
| [protocol-buffers-schema](https://github.com/mafintosh/protocol-buffers-schema) | `3.6.0` | `3.6.1` |
| [yaml](https://github.com/eemeli/yaml) | `1.10.2` | `1.10.3` |



Updates `brace-expansion` from 1.1.12 to 1.1.14
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@v1.1.12...v1.1.14)

Updates `brace-expansion` from 2.0.2 to 2.1.0
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@v1.1.12...v1.1.14)

Updates `follow-redirects` from 1.15.9 to 1.16.0
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.9...v1.16.0)

Updates `immutable` from 4.3.4 to 4.3.8
- [Release notes](https://github.com/immutable-js/immutable-js/releases)
- [Changelog](https://github.com/immutable-js/immutable-js/blob/main/CHANGELOG.md)
- [Commits](immutable-js/immutable-js@v4.3.4...v4.3.8)

Updates `lodash` from 4.17.23 to 4.18.1
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.23...4.18.1)

Updates `minimatch` from 3.1.2 to 3.1.5
- [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md)
- [Commits](isaacs/minimatch@v3.1.2...v3.1.5)

Updates `minimatch` from 5.1.6 to 5.1.9
- [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md)
- [Commits](isaacs/minimatch@v3.1.2...v3.1.5)

Updates `picomatch` from 2.3.1 to 2.3.2
- [Release notes](https://github.com/micromatch/picomatch/releases)
- [Changelog](https://github.com/micromatch/picomatch/blob/master/CHANGELOG.md)
- [Commits](micromatch/picomatch@2.3.1...2.3.2)

Removes `serialize-javascript`

Updates `undici` from 5.29.0 to 6.25.0
- [Release notes](https://github.com/nodejs/undici/releases)
- [Commits](nodejs/undici@v5.29.0...v6.25.0)

Updates `brace-expansion` from 1.1.12 to 1.1.14
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@v1.1.12...v1.1.14)

Updates `follow-redirects` from 1.15.9 to 1.16.0
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.9...v1.16.0)

Updates `picomatch` from 2.3.1 to 2.3.2
- [Release notes](https://github.com/micromatch/picomatch/releases)
- [Changelog](https://github.com/micromatch/picomatch/blob/master/CHANGELOG.md)
- [Commits](micromatch/picomatch@2.3.1...2.3.2)

Updates `protobufjs` from 6.11.4 to 6.11.6
- [Release notes](https://github.com/protobufjs/protobuf.js/releases)
- [Changelog](https://github.com/protobufjs/protobuf.js/blob/master/CHANGELOG.md)
- [Commits](protobufjs/protobuf.js@v6.11.4...v6.11.6)

Updates `protocol-buffers-schema` from 3.6.0 to 3.6.1
- [Commits](mafintosh/protocol-buffers-schema@v3.6.0...v3.6.1)

Updates `yaml` from 1.10.2 to 1.10.3
- [Release notes](https://github.com/eemeli/yaml/releases)
- [Commits](eemeli/yaml@v1.10.2...v1.10.3)

---
updated-dependencies:
- dependency-name: brace-expansion
  dependency-version: 1.1.14
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: brace-expansion
  dependency-version: 2.1.0
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: follow-redirects
  dependency-version: 1.16.0
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: immutable
  dependency-version: 4.3.8
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: lodash
  dependency-version: 4.18.1
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: minimatch
  dependency-version: 3.1.5
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: minimatch
  dependency-version: 5.1.9
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: picomatch
  dependency-version: 2.3.2
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: serialize-javascript
  dependency-version: 
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: undici
  dependency-version: 6.25.0
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: brace-expansion
  dependency-version: 1.1.14
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: follow-redirects
  dependency-version: 1.16.0
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: picomatch
  dependency-version: 2.3.2
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: protobufjs
  dependency-version: 6.11.6
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: protocol-buffers-schema
  dependency-version: 3.6.1
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: yaml
  dependency-version: 1.10.3
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>

* update gomod2nix

* upgrade to ethers v6 to ensure that hardhat can be upgraded properly

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: JayT106 <JayT106@users.noreply.github.com>
Co-authored-by: Randy Ang <randy.ang@cronoslabs.org>
…up (#947)

Bumps the gh-actions group with 1 update: [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action).


Updates `anthropics/claude-code-action` from 1.0.121 to 1.0.122
- [Release notes](https://github.com/anthropics/claude-code-action/releases)
- [Commits](anthropics/claude-code-action@f4fb5c6...86eb26b)

---
updated-dependencies:
- dependency-name: anthropics/claude-code-action
  dependency-version: 1.0.122
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: gh-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…up (#950)

Bumps the gh-actions group with 1 update: [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action).


Updates `anthropics/claude-code-action` from 1.0.122 to 1.0.123
- [Release notes](https://github.com/anthropics/claude-code-action/releases)
- [Commits](anthropics/claude-code-action@86eb26b...51ea8ea)

---
updated-dependencies:
- dependency-name: anthropics/claude-code-action
  dependency-version: 1.0.123
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: gh-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* make JSON-RPC batch limits configurable

* add changelog

* Update CHANGELOG.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>

* support server flag

---------

Signed-off-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: JayT106 <JayT106@users.noreply.github.com>
Co-authored-by: songgaoye <217724508+songgaoye@users.noreply.github.com>
* chore(evm): preallocate slices in AccessList conversion functions

* add changelog

---------

Co-authored-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
…py (#948)

* fix(evm): fix SetCodeTx nil pointer panics and missing AuthList in Copy

* add changelog

* fix reviews

---------

Co-authored-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
…up (#958)

Bumps the gh-actions group with 1 update: [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action).


Updates `anthropics/claude-code-action` from 1.0.123 to 1.0.125
- [Release notes](https://github.com/anthropics/claude-code-action/releases)
- [Commits](anthropics/claude-code-action@51ea8ea...fd1877d)

---
updated-dependencies:
- dependency-name: anthropics/claude-code-action
  dependency-version: 1.0.125
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: gh-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…updates (#955)

Bumps the npm_and_yarn group with 3 updates in the /tests/integration_tests/hardhat directory: [bn.js](https://github.com/indutny/bn.js), [minimatch](https://github.com/isaacs/minimatch) and [semver](https://github.com/npm/node-semver).
Bumps the npm_and_yarn group with 4 updates in the /tests/solidity directory: [lodash](https://github.com/lodash/lodash), [ajv](https://github.com/ajv-validator/ajv), [micromatch](https://github.com/micromatch/micromatch) and [underscore](https://github.com/jashkenas/underscore).


Updates `bn.js` from 4.12.3 to 5.2.3
- [Release notes](https://github.com/indutny/bn.js/releases)
- [Changelog](https://github.com/indutny/bn.js/blob/master/CHANGELOG.md)
- [Commits](indutny/bn.js@v4.12.3...v5.2.3)

Updates `minimatch` from 3.1.5 to 9.0.9
- [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md)
- [Commits](isaacs/minimatch@v3.1.5...v9.0.9)

Updates `semver` from 5.7.2 to 7.8.0
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md)
- [Commits](npm/node-semver@v5.7.2...v7.8.0)

Updates `lodash` from 4.17.20 to 4.18.1
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.20...4.18.1)

Updates `ajv` from 6.12.6 to 8.20.0
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](ajv-validator/ajv@v6.12.6...v8.20.0)

Updates `micromatch` from 3.1.10 to 4.0.8
- [Release notes](https://github.com/micromatch/micromatch/releases)
- [Changelog](https://github.com/micromatch/micromatch/blob/master/CHANGELOG.md)
- [Commits](micromatch/micromatch@3.1.10...4.0.8)

Updates `underscore` from 1.9.1 to 1.13.8
- [Commits](jashkenas/underscore@1.9.1...1.13.8)

---
updated-dependencies:
- dependency-name: bn.js
  dependency-version: 5.2.3
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: minimatch
  dependency-version: 9.0.9
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: semver
  dependency-version: 7.8.0
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: lodash
  dependency-version: 4.18.1
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: ajv
  dependency-version: 8.20.0
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: micromatch
  dependency-version: 4.0.8
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: underscore
  dependency-version: 1.13.8
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: JayT106 <JayT106@users.noreply.github.com>
Bumps the npm_and_yarn group with 1 update in the /tests/solidity directory: [braces](https://github.com/micromatch/braces).


Updates `braces` from 2.3.2 to 3.0.3
- [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md)
- [Commits](https://github.com/micromatch/braces/commits/3.0.3)

---
updated-dependencies:
- dependency-name: braces
  dependency-version: 3.0.3
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…up (#960)

Bumps the gh-actions group with 1 update: [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action).


Updates `anthropics/claude-code-action` from 1.0.125 to 1.0.127
- [Release notes](https://github.com/anthropics/claude-code-action/releases)
- [Commits](anthropics/claude-code-action@fd1877d...1dc994e)

---
updated-dependencies:
- dependency-name: anthropics/claude-code-action
  dependency-version: 1.0.127
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: gh-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…d fork (#957)

* feat(geth): update go-ethereum version to `v1.16.9`, enable Osaka hardfork

* doc:

* fix: hash

* fix: test

* fix: enable precompile

* fix: lint

* fix: review
…ec (#962)

* fix(rpc): align eth_createAccessList response with execution-apis spec

* add changelog

* fmt

* rename AccessListResult.Accesslist to AccessList and add keeper test

* add unit test TestCreateAccessList_VmError

* fix review

* revert defer cancel() in mock helpers, add nolint

* update changelog
…nts (#972)

* test(integration): fix flaky test_tracecall_diff by using fixed accounts

* add changelog
* fix(rpc): align eth_feeHistory response with execution-apis spec

* add changelog

* handle zero-block edge case and fix omitempty in eth_feeHistory

* fix FeeHistory tests using invalid FeeHistoryCap=0 config
* fix(rpc): align eth_getLogs response with execution-apis spec

* return error for unknown block hash in eth_getLogs

* add changelog

* clean code

* use errUnknowBlock
…sh lookups (#970)

* fix(rpc): prevent nil panic in SignClient type assertion for block hash lookups

* add changelog

* fix GetBlockTransactionCountByHash
* feat: migrate to Cosmos SDK v0.54, IBC v11, and CometBFT 0.39

    Upgrade core deps (cosmos-sdk/store v2, log v2, ibc-go v11), restore
    crisis from contrib/x/crisis, and align app wiring (gov, NodeService,
    IBC keepers, ante). Refactor EVM StateDB native actions to nested
    cachemultistore layers instead of store clone/restore. Drop trace-store
    from AppCreator and update server/tests. Refresh tests for auth
    NextAccountNumber(ctx, AccountI) and gRPC expectations. Trim outdated
    comments in app, statedb, and server startup.

* chore(lint): fix lint issues after SDK v0.54 rebase

- Apply gofumpt formatting to all changed files
- Remove unused openTraceWriter (trace-store removed from AppCreator)
- Remove unused io import in server/start.go
- Fix filters_test.go to use cosmossdk.io/log/v2 Logger interface
- Add nolint:prealloc to ante decorator slices extended with variadic extra
- Preallocate active slice in simulate.go using defaultPrecompiles map len
- Remove nolint:staticcheck directives made redundant by golangci exclusion
- Add SA1019 exclusion to golangci.yml for SDK v0.54 deprecation warnings
- Add revive exclusion for rpc/namespaces/ethereum/net package name

* fix(statedb): add double-Commit guard and address security review findings

- Add committed bool guard to StateDB.Commit() - prevents double-flush of
  cachemultistore layers in block-STM or any re-entrant Commit() path
- Emit error log when --trace-store flag is set (removed in SDK v0.54 migration,
  flag registration kept for backward compat but no longer has any effect)
- Scope SA1019 suppression in golangci.yml to specific affected packages
  (ante, client, cmd, ethereum, evmd, server, testutil, evm/keeper, feemarket/keeper)
  instead of blanket global suppression across the entire repo

* ci: bump Go version to 1.25.9 and fix nix golangci-lint PATH

go.mod now requires go >= 1.25.9 (jayt106/cosmos-sdk fork constraint).
Update all CI workflow go-version fields from 1.25.8 to 1.25.9 so
GOTOOLCHAIN=local can find a compatible toolchain.

Also fix the lint workflow: nix profile add installs its own go binary
which overrides PATH, causing golangci-lint to use go 1.25.8 and fail
the go directive check. Capture SYSTEM_GO before nix runs and prepend
it back to PATH when invoking golangci-lint.

* chore(deps): upgrade to cosmos-sdk v0.54.3 and cometbft v0.39.3

Replace jayt106 fork replace directives with official releases:
- github.com/cosmos/cosmos-sdk: v0.54.0 (jayt106 fork) → v0.54.3
- github.com/cometbft/cometbft: v0.39.0 (jayt106 fork) → v0.39.3

v0.54.3 includes the gov proposal_messages fix and CometBFT v0.39.3.

* fix: address code review, security, and simplification findings

Code review findings:
- Add TestDoubleCommit to verify the double-Commit guard contract
- Update --trace-store flag help string to say unsupported since SDK v0.54
- Add --trace-store warning to startStandAlone path (was only in startInProcess)
- Quote \$SYSTEM_GO in lint.yml PATH fix for shell correctness

Security findings:
- simulate.go: log applyAuthorization errors at DEBUG instead of silently
  suppressing via //nolint:errcheck; matches state_transition.go pattern

Code simplification:
- statedb.go: remove degenerate commitMS function pointer, call
  flushNativeCacheLayers directly (was always the only assignee)
- statedb.go: collapse duplicated NewWithParams multistore branching into
  a single type-assertion with one panic path
- statedb.go: hoist keeper.GetState into named vars (origin/dirty/store)
  in conflict-detection loop for readability; add comments clarifying
  EVM-dirty-only scope and Write() vs Restore() semantic change

* chore: regenerate gomod2nix.toml for official sdk/cometbft/go-getter

* fix(ci): fix golangci-lint prealloc issues and vulncheck allowlist

Lint (golangci-lint 2.1.6 in CI):
- evmd/ante/evm_handler.go: convert decorator slice to make+append
  (capacity 15+len(extra)); removes now-unused //nolint:prealloc
- evmd/ante/handler_options.go: same pattern, capacity 16+len(extra)
- rpc/namespaces/ethereum/eth/filters/filters.go: preallocate filtersBz
  with capacity 1+len(topics) instead of nil slice

Vulncheck allowlist:
- Add GO-2024-2584 (cosmos-sdk slashing evasion, Fixed in: N/A) to
  the allowed set in the dependency-review workflow; fix the shell
  check to use grep -w so multiple IDs can be allowed

* fix(nix): patch gomod2nix vendor symlink for split-module monorepos; restore app.go ordering

- Add nix/gomod2nix-symlink-dedup.patch: skip already-present entries in
  populateVendorPath to handle split-module monorepos (google.golang.org/genproto
  + genproto/googleapis/api, go.opentelemetry.io/otel + otel/exporters). Sub-module
  symlinks are placed first by makeSymlinks reverse-alphabetical order, then
  populateVendorPath for the parent tried to re-symlink the same paths causing
  "file exists" panic. Upstream unfixed as of nix-community/gomod2nix@514283ec.
- Apply patch in nix/default.nix via bootstrapPkgs.applyPatches (same pattern
  as patchedPoetry2nix)
- Restore original field/setup ordering in evmd/app.go (CrisisKeeper field
  position, crisistypes.StoreKey position, skipGenesisInvariants placement)
  to minimize diff from develop
- Revert ante decorator slices to composite literal form; remove prealloc
  linter from golangci.yml (inconsistent behavior between local and CI
  golangci-lint 2.1.6 caused nolintlint false positives)

* chore(lint): restore prealloc linter; use make+append for ante decorator slices

Prealloc was removed in the previous commit because composite literals
triggered it when followed by variadic append. Restore the linter and
use make([]T, 0, N+len(extra)) + append form which satisfies prealloc
without needing //nolint directives.

* fix(evmd): restore original crisis module ordering in begin/end blockers

SetOrderBeginBlockers and SetOrderEndBlockers had crisistypes.ModuleName
moved to the end without reason. contrib/x/crisis BeginBlocker/EndBlocker
are no-ops so ordering doesn't matter functionally. Restore original
positions to minimize diff from develop.

* fix(evmd): restore original crisis.NewAppModule position in module manager

crisis.NewAppModule was moved from after bank.NewAppModule to after
mint.NewAppModule without reason. Restore original position (minimum diff).
Parameter stays as app.CrisisKeeper (not &app.CrisisKeeper) since
CrisisKeeper is now *crisiskeeper.Keeper in contrib/x/crisis.

* fix(ci): remove GO-2024-2584 from vulncheck allowlist

GO-2024-2584 affects cosmos-sdk before v0.47.10 only. We are on v0.54.3
which is greater than v0.47.10, so govulncheck does not report it.
The allowlist entry was added in error based on a misreading of the
GitHub advisory (which covered 0.47.x and 0.50.x branches separately).

* fix(nix): fix gomod2nix patch context to match actual source

Patch context had wrong semicolon and stale line numbers, causing
patch(1) to fail during nix build. Regenerated from actual
gomod2nix@514283ec builder/symlink/symlink.go content.

* refactor(lint): revert SA1019 golangci exclusion; add inline nolint:staticcheck

Replace the scoped SA1019 exclusion in golangci.yml with per-site
//nolint:staticcheck directives in affected files. Also replace the
revive path exclusion for rpc/namespaces/ethereum/net with a package-
level //nolint:revive annotation in api.go.

* fix(lint,nix): fix gofumpt formatting, remove unused nolint, switch nix gomod2nix patch to postPatch+python

- Run gofumpt on client/testnet.go, cmd/config/config.go, testutil/network/network.go
  (trailing-space alignment before nolint comments violated gofumpt)
- Remove unused //nolint:revive from rpc/namespaces/ethereum/net/api.go
  (revive does not flag 'package net' in this config; nolintlint caught it)
- Replace patch-file approach in nix/default.nix with postPatch+python substituteInPlace:
  patch context was fragile (tab/space mismatch with applyPatches -p1);
  Python string replace is exact and line-number independent

* fix(nix): replace python3 with substituteInPlace for gomod2nix postPatch

bootstrapPkgs.applyPatches runs in minimal stdenv without python3.
substituteInPlace uses bash string ops (always available) and handles
tab/newline literals via ANSI-C quoting ($'\t\n').

* fix(nix): pin nixpkgs to fe3afaa4 for go_1_25 1.25.9

cosmos-sdk v0.54.3 requires go >= 1.25.9. Advance the nixpkgs
release-25.11 pin to the commit that bumped go_1_25 from 1.25.8 to
1.25.9, so GOTOOLCHAIN=local no longer rejects the ethermintd build.

* fix(ante): use standard fee deduction for cosmos txs to fix simulation supply invariant

SDK v0.54 simulation calls FinalizeBlock with empty Txs (running BeginBlock/EndBlock
only), then delivers actual txs via SimDeliver outside FinalizeBlock. The EVM
DeductFeeDecorator calls SendCoinsToVirtual, which parks fees in the ObjectStore.
CreditVirtualAccounts (bank EndBlock) already ran inside FinalizeBlock before
SimDeliver, so virtual coins from SimDeliver-delivered txs are never credited.
SimWriteState only flushes the KV store, discarding the ObjectStore — leaving
supply > sum(balances) in committed state, causing bank.InitGenesis to panic
on import.

Fix: cosmos and legacy EIP-712 ante handlers now use ante.NewDeductFeeDecorator
(SendCoinsFromAccountToModule, non-virtual) instead of evm.NewDeductFeeDecorator.
Cosmos txs have no gas refund so virtual fee handling is unnecessary.

Also add missing revive exclusion for rpc/namespaces/ethereum/net package.

* fix(ci): fix integration test cache - use ubuntu for upload-cache and update nixos channel

- upload-cache ran on macos-latest, integration_tests on ubuntu-latest; nix store
  paths are platform-specific so the macOS cache produced 100% misses on Linux CI.
  Switch upload-cache to ubuntu-latest to match.
- Update nix_path from nixos-22.11 to nixos-25.11 to match the nixpkgs pin in
  nix/sources.json (fe3afaa4, release-25.11).

* fix(ante): add fee deduction regression test and trim changelog

- Add TestCosmosTxFeeDeductedInSimulate: verifies cosmos and legacy EIP-712 ante
  handlers credit fee_collector in KV store (not ObjectStore) when simulate=true.
  Guards against regression to evm.NewDeductFeeDecorator which parks fees in
  ObjectStore and causes bank supply invariant panic in SDK v0.54 simulations.
- Trim CHANGELOG PR #894 entry to match PR title.

* fix(statedb): set committed flag early to block re-entry on partial failure

Move s.committed = true immediately after the guard check so that any
mid-Commit failure (e.g. ErrStateConflict) leaves the flag set and
prevents a second Commit() from attempting a partial double-flush.

* fix(upgrades): add sdk54 upgrade handler for SDK v0.54 / IBC v11 migrations

Existing live chains need a named upgrade plan that calls RunMigrations to
execute IBC v10→v11 and SDK v0.53→v0.54 module consensus-version migrations.
Without this handler, chains upgrading via governance proposal would skip
the module migrations and risk state inconsistencies or startup panics.

* docs(statedb): clarify flushNativeCacheLayers flush direction and origCtx relationship

Add comment to flushNativeCacheLayers explaining that cacheLayers[0].Write()
propagates into the parent cachemulti.Store (not origCtx directly), and that
EVM-dirty writes in Commit() use origCtx which targets the same parent store.
Addresses review comment r3244878007.

* ci: break upload-cache/integration_tests bootstrap deadlock

Remove needs: ["integration_tests"] from upload-cache so it runs
unconditionally on push to develop. Previously, a cold nix cache caused
integration tests to compile OpenSSL from source (>60 min), timing out
before tests started. Tests can't pass without a warm cache, and cache
can't warm without tests passing — bootstrap deadlock.

With this change, the first push to develop after a nixpkgs channel bump
will warm the cache via upload-cache regardless of test outcome. Subsequent
runs will find packages pre-built in cachix and complete in normal time.

Also bump integration_tests timeout-minutes from 60 to 90 as a safety net
for the first cold-cache run.

* ci: extend integration_tests timeout to 360 min for cold-cache bootstrap

* fix(nix): override mbedtls to strip GCC 15-only CMAKE_C_FLAGS

mbedtls 3.6.5 in nixpkgs-25.11 rev fe3afaa4 unconditionally passes
-fzero-init-padding-bits=unions via CMAKE_C_FLAGS. That flag requires
GCC 15, but stdenv in this nixpkgs pin ships gcc 14.3.0, causing the
mbedtls build to fail and cascading to librist, ffmpeg, matplotlib, and
all python3.13 test-env packages.

Strip the incompatible flag via overrideAttrs so the nix integration
test environment can build cleanly with gcc 14.

* ci: allow GO-2024-2584 in govulncheck (unfixable cosmos-sdk vuln)

GO-2024-2584 is a slashing-evasion issue in github.com/cosmos/cosmos-sdk;
the govulndb entry has "Fixed in: N/A" — no upstream patch exists for the
v0.54 line. Add it to the ALLOWED list alongside GO-2026-4479.

* fix(nix): skip p11-kit build tests (test-server.sh fails in sandbox)

p11-kit 0.25.10 in nixpkgs-25.11 rev fe3afaa4 fails test 40/67
(test-server.sh) with exit status 1 because p11tool is not found on
PATH during the nix sandbox build phase. This cascades through
gnutls → cups/ffmpeg/gnupg/glib → matplotlib/poetry/pytest-regressions,
preventing the integration test environment from being built.

Disable doCheck to skip the failing sandbox-incompatible test.

* ci: add workflow_dispatch to Tests workflow for manual cache warming

Fork PRs lack CACHIX_SIGNING_KEY so integration_tests cannot push to
cachix; every run rebuilds nix from scratch and hits the 6h timeout.

- Add workflow_dispatch trigger to Tests workflow
- Extend upload-cache guard to also fire on workflow_dispatch

Maintainers can now run upload-cache on the fork branch via the Actions
UI to pre-warm cachix before integration tests run.

* ci: enable parallel nix builds (max-jobs=auto) in integration tests

Cold nix builds on fork PRs cannot push to cachix (no CACHIX_SIGNING_KEY)
so every run starts from scratch. ubuntu-latest runners have 4 vCPUs;
setting max-jobs=auto lets nix build 4 derivations in parallel, reducing
cold-build time from ~6h to ~1.5h and fitting within the 360-min timeout.

Also add timeout-minutes=360 to upload-cache job for consistency.

* fix(nix): remove p11-kit override — causes gnutls/node.js cascade rebuild

The p11-kit overrideAttrs (doCheck=false) creates a new derivation hash.
Since gnutls depends on p11-kit, gnutls also gets a new hash not in
cache.nixos.org, forcing a from-source build. Node.js depends on gnutls
transitively, so node.js V8 also rebuilds from source — taking 4-5h and
blowing the 6h CI timeout.

The original p11-kit for nixpkgs rev fe3afaa4 IS in cache.nixos.org
(built by Hydra). Without the override, p11-kit, gnutls, and node.js
all come from binary cache and require no source build.

The p11-kit test-server.sh sandbox failure only occurs when building
from source. With cache hits, no build happens and no tests run.

* nix: update nixpkgs pin to nixos-25.11 channel HEAD (d7a713c0)

New pin (2026-05-14) natively includes the GCC14 mbedtls fix, removing
the need for the overrideAttrs workaround. Eliminates the hash cascade
that forced ffmpeg and all codec dependencies to rebuild from source on
every CI run, cutting integration test nix-build time from 3+ hours to
cache hits.

* nix: fix gomod2nix vendor for split-module monorepos (aws-sdk-go-v2, btcd)

The previous skip-if-exists patch fixed the "file exists" panic but
silently dropped root module content when sub-modules had already
created the parent directory via MkdirAll.  Affected modules include
github.com/aws/aws-sdk-go-v2, github.com/btcsuite/btcd, and
google.golang.org/genproto — all monorepos where root + sub-modules
share a vendor path prefix.

Replace os.Symlink with symlinkOrMerge (added in new merge.go):
  - dst missing       → create symlink as usual
  - dst is a symlink  → skip (another module already owns the path)
  - dst is a directory → recurse and symlink each src entry individually

This allows the root module's packages (e.g. aws-sdk-go-v2/aws,
btcd/chaincfg, btcd/wire) to be accessible alongside sub-module
symlinks in the same vendor directory.

* fix(nix): append symlinkOrMerge into symlink.go instead of separate merge.go

gomod2nix-symlink.drv compiles symlink.go as a single-file build
(# command-line-arguments), so adding a separate merge.go is ignored.

Append the symlinkOrMerge function body directly into symlink.go via
cat >>, and use string concatenation instead of filepath.Join to avoid
any import dependencies.

* nix: drop redundant go_1_25 overlay; remove unreferenced go vendor patch

nixpkgs 25.11 already aliases `go = go_1_25` (1.25.9) and
`buildGoModule = buildGo125Module` (all-packages.nix:8815-8816), so
the overlay override is a no-op. `go_no_vendor_checks-1.23.patch` is
not referenced anywhere.

* build(deps): pin cosmos-sdk to crypto-org-chain fork backport/26023

Replace github.com/cosmos/cosmos-sdk with the fork branch
backport/26023-staking-queue-optimization (db8ba1def35d) to pull in
the staking queue optimization.

* build(deps): pin cosmos-sdk to crypto-org-chain fork PR evmos#1814

Pin to chore/staking-v6-migration-utility branch
(crypto-org-chain/cosmos-sdk@f2bcc1893cf0) which exposes v6 staking
queue migration as a utility while keeping consensus version at 5.

* build(deps): pin cosmos-sdk to upstream v0.54.3

Drop crypto-org-chain fork replace directive. The fork only adds
staking end-block queue optimization (PR evmos#1814) which requires
calling Keeper.PopulateQueuePendingSlots from the upgrade handler —
ethermint does not call it, so the fork had no effect. Downstream
chains (e.g. Cronos) can add the replace + migration call at their
app layer.

* build(deps): remove dead btree replace directive

* fix(ci): bump x/net and use store/v2 types for v8 migration

- bump golang.org/x/net to v0.55.0 to address GO-2026-5026 IDNA vuln
- switch x/evm/migrations/v8 store key imports to cosmos-sdk/store/v2/types
  to match testutil.DefaultContext signature
- use cosmossdk.io/log/v2 nop logger in filters_test to satisfy
  PublicFilterAPI.logger interface

---------

Signed-off-by: JayT106 <JayT106@users.noreply.github.com>
Co-authored-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
… tx (#973)

* fix: setting evm hooks prevent the stateDB from committing for failed tx

* Update CHANGELOG.md

Signed-off-by: Xinyu <91446598+XinyuCRO@users.noreply.github.com>

* fix: refund & evm events check

* fix: delay commit

* fix comments

---------

Signed-off-by: Xinyu <91446598+XinyuCRO@users.noreply.github.com>
Co-authored-by: songgaoye <217724508+songgaoye@users.noreply.github.com>
Co-authored-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
Co-authored-by: Thomas Nguy <thomas.nguy@crypto.com>
)

* fix(rpc): align `eth_simulateV1` response with execution-apis spec

* fix: review

* fix: tests

* fix: lint

* fix: doc

* fix: review

* fix: align changes with geth

---------

Signed-off-by: Xinyu <91446598+XinyuCRO@users.noreply.github.com>
Co-authored-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
Co-authored-by: songgaoye <217724508+songgaoye@users.noreply.github.com>
…pec (#965)

* fix(rpc): align `eth_getBlockReceipts` response with execution-apis spec.

* Update CHANGELOG.md

Signed-off-by: Xinyu <91446598+XinyuCRO@users.noreply.github.com>

* fix: review

* fix(rpc): propagate block lookup error in eth_getBlockReceipts

* omit effectiveGasPrice from receipt when baseFee unavailable for EIP-1559 txs

---------

Signed-off-by: Xinyu <91446598+XinyuCRO@users.noreply.github.com>
Co-authored-by: songgaoye <217724508+songgaoye@users.noreply.github.com>
* fix(rpc): align eth_sendRawTransaction with  execution-apis

* add changelog

* update tests
* fix(rpc): add blockTimestamp to transaction RPC responses

* add changelog

* lint

* test(integration): add blockTimestamp to EXPECTED_GET_TRANSACTION

* guard blockTimestamp against uint64 wrap-around on zero/negative block times

* check blockTimestamp value

* remove redundant blockTimestamp assignment in SimBlockResult

* fix reviews

* fmt
…971)

* fix(rpc): align eth_getBlockHash response with execution-apis spec

* fix invalid type

* propagate gas limit conversion error and use t.Run in TestFormatBlock

* add changelog

* rm IIFE and update changelog

* compute transactionsRoot from EVM-only transactions in RPCBlockFromTendermintBlock

* update changelog

* keep transactionsRoot consistent with transactions array in fullTx mode

* emit post-merge fields in newHeads and document TxHash inconsistency

* lint

* lint

* use common.Hash for block hash field in FormatBlock
* fix(evm): align debug_trace fee with virtual ante path

* fix issues

* revert behavior to match ethereum

* fix golang ci

* fix lint

* add changelog

* move defer

* improve tests

* Apply suggestion from @thomas-nguy

Signed-off-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>

* add tracer

---------

Signed-off-by: Thomas <81727899+thomas-nguy@users.noreply.github.com>
…thods (#990)

* fix(debug): apply restrict-user-input path check to all file-write methods

* add unit test TestValidatePath

* add changelog

* fix symlink

* remove symlink

* fix(debug): prevent overwriting existing files in restricted mode

* update unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants