Skip to content

(WIP) feat: EIP-7702 #690

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from
Draft

(WIP) feat: EIP-7702 #690

wants to merge 4 commits into from

Conversation

XinyuCRO
Copy link

Closes: #XXX

Description

For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@XinyuCRO
feat: eip-7702
2a3629f 
# Conflicts:
#	x/evm/keeper/migrations.go
#	x/evm/migrations/v7/migrate.go
#	x/evm/module.go
#	x/evm/types/chain_config.go
@XinyuCRO
test: add test
e37e386
github-advanced-security[bot]
github-advanced-security bot found potential problems Aug 15, 2025
View reviewed changes
tests/integration_tests/test_set_code_tx.py

import pytest
from web3 import Web3
from eth_account import Account

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'Account' is not used.

Copilot Autofix

AI 1 day ago

To fix the problem, simply remove the unused import statement from eth_account import Account from line 5 of tests/integration_tests/test_set_code_tx.py. This will clean up the code, reduce unnecessary dependencies, and improve readability. No other changes are required, as the rest of the code does not reference Account.

Suggested changeset 1
tests/integration_tests/test_set_code_tx.py

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/tests/integration_tests/test_set_code_tx.py b/tests/integration_tests/test_set_code_tx.py
--- a/tests/integration_tests/test_set_code_tx.py
+++ b/tests/integration_tests/test_set_code_tx.py
@@ -2,7 +2,6 @@
 
 import pytest
 from web3 import Web3
-from eth_account import Account
 from hexbytes import HexBytes
 import time
 
EOF
@@ -2,7 +2,6 @@

import pytest
from web3 import Web3
from eth_account import Account
from hexbytes import HexBytes
import time

Copilot is powered by AI and may make mistakes. Always verify output.
tests/integration_tests/test_set_code_tx.py
from web3 import Web3
from eth_account import Account
from hexbytes import HexBytes
import time

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'time' is not used.

Copilot Autofix

AI 1 day ago

To fix the problem, simply remove the unused import statement import time from line 7 of tests/integration_tests/test_set_code_tx.py. This will clean up the code and remove an unnecessary dependency, improving readability and maintainability. No other changes are required, as the rest of the code does not depend on the time module.

Suggested changeset 1
tests/integration_tests/test_set_code_tx.py

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/tests/integration_tests/test_set_code_tx.py b/tests/integration_tests/test_set_code_tx.py
--- a/tests/integration_tests/test_set_code_tx.py
+++ b/tests/integration_tests/test_set_code_tx.py
@@ -4,7 +4,6 @@
 from web3 import Web3
 from eth_account import Account
 from hexbytes import HexBytes
-import time
 
 from .network import setup_custom_ethermint
 from .utils import (
EOF
@@ -4,7 +4,6 @@
from web3 import Web3
from eth_account import Account
from hexbytes import HexBytes
import time

from .network import setup_custom_ethermint
from .utils import (
Copilot is powered by AI and may make mistakes. Always verify output.
@socket-security Socket Security
Copy link

socket-security bot commented Aug 15, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Updatedpypi/​pydantic@​2.11.7 ⏵ 2.7.4100100100100100
Updatedpypi/​pydantic@​2.11.7 ⏵ 2.9.2100100100100100
Updatedpypi/​eth-utils@​3.0.0 ⏵ 5.3.0100100100100100
Updatedpypi/​parsimonious@​0.9.0 ⏵ 0.10.0100 +1100100100100
Updatedpypi/​pydantic-core@​2.39.0 ⏵ 2.18.4100100100100100
Updatedpypi/​pydantic-core@​2.39.0 ⏵ 2.23.4100100100100100
Updatedpypi/​eth-typing@​4.0.0 ⏵ 5.2.1100100100100100
Updatedpypi/​eth-abi@​5.0.0 ⏵ 5.2.0100100100100100
Updatedpypi/​eth-rlp@​1.0.1 ⏵ 2.2.0100100100100100
Updatedpypi/​hexbytes@​0.3.1 ⏵ 1.3.1100100100100100
Updatedpypi/​types-requests@​2.32.4.20250809 ⏵ 2.32.4.20250611100100100100100

View full report

@codecov Codecov
Copy link

codecov bot commented Aug 15, 2025

Codecov Report

❌ Patch coverage is 57.54717% with 135 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.23%. Comparing base (3a866eb) to head (bc0cddd).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
x/evm/keeper/set_code_authorizations.go 0.00% 38 Missing ⚠️
x/evm/types/auth_list.go 18.42% 28 Missing and 3 partials ⚠️
x/evm/types/set_code_tx.go 87.30% 16 Missing and 8 partials ⚠️
x/evm/types/tx_args.go 5.55% 17 Missing ⚠️
ante/interfaces/setup.go 0.00% 6 Missing and 1 partial ⚠️
x/evm/keeper/grpc_query.go 25.00% 4 Missing and 2 partials ⚠️
x/evm/keeper/state_transition.go 0.00% 4 Missing and 1 partial ⚠️
x/evm/types/utils.go 57.14% 2 Missing and 1 partial ⚠️
x/evm/types/tx_data.go 0.00% 2 Missing ⚠️
x/evm/types/codec.go 0.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #690      +/-   ##
===========================================
+ Coverage    46.07%   46.23%   +0.15%     
===========================================
  Files          186      189       +3     
  Lines        18646    18963     +317     
===========================================
+ Hits          8592     8768     +176     
- Misses        9314     9437     +123     
- Partials       740      758      +18
Files with missing lines Coverage Δ
x/evm/types/chain_config.go 50.00% <100.00%> (-2.41%) ⬇️
x/evm/types/codec.go 37.50% <0.00%> (-0.80%) ⬇️
x/evm/types/msg.go 56.95% <0.00%> (-0.26%) ⬇️
x/evm/types/tx_data.go 66.00% <0.00%> (-2.75%) ⬇️
x/evm/types/utils.go 33.33% <57.14%> (+1.11%) ⬆️
x/evm/keeper/state_transition.go 71.06% <0.00%> (-1.23%) ⬇️
x/evm/keeper/grpc_query.go 73.07% <25.00%> (-0.67%) ⬇️
ante/interfaces/setup.go 75.82% <0.00%> (-6.32%) ⬇️
x/evm/types/tx_args.go 89.88% <5.55%> (-10.12%) ⬇️
x/evm/types/set_code_tx.go 87.30% <87.30%> (ø)
... and 2 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@XinyuCRO