Skip to content

feat: use hex validator addresses in precompiles#1141

Open
luchenqun wants to merge 7 commits into
cosmos:mainfrom
luchenqun:feat/staking-hex-addresses
Open

feat: use hex validator addresses in precompiles#1141
luchenqun wants to merge 7 commits into
cosmos:mainfrom
luchenqun:feat/staking-hex-addresses

Conversation

@luchenqun

Copy link
Copy Markdown

Summary

  • use EVM hex validator addresses for staking and distribution precompile inputs
  • convert hex validator addresses to SDK bech32 internally
  • update Solidity helpers, ABI artifacts, and related tests

Reference: evmos/evmos#2122

Testing

  • go test ./precompiles/distribution ./precompiles/staking ./precompiles/testutil/contracts -tags=test -count=1
  • go test ./tests/integration/precompiles/distribution ./tests/integration/precompiles/staking -tags=test -run TestDoesNotExist -count=1
  • cd contracts && npx hardhat compile --force

@luchenqun luchenqun requested a review from a team as a code owner April 28, 2026 05:20
@greptile-apps

greptile-apps Bot commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

PR author is not in the allowed authors list.

@aljo242 aljo242 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ci has not run on this pr: all workflows (tests, lint, solhint, slither, solidity-test, json-rpc compatibility) are in action_required state -- fork pr, needs maintainer approval to trigger. the only green check is the mergify gate summary, not actual test results.

branch is also 4 commits behind main (missing #1176 and #1187, both statedb fixes). needs a rebase before ci can be meaningfully evaluated.

given the breaking api surface (all staking and distribution precompile callers need new abis and redeployment), this needs ci green + rebase before review.

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.

2 participants