Skip to content

Remove unused wasmd precompile#271

Merged
Thaleszh merged 5 commits intomainfrom
chore/remove-wasmd-precompile
Feb 5, 2026
Merged

Remove unused wasmd precompile#271
Thaleszh merged 5 commits intomainfrom
chore/remove-wasmd-precompile

Conversation

@Thaleszh
Copy link
Contributor

@Thaleszh Thaleszh commented Feb 5, 2026

Description

Strips out code from wasmd precompile that is no longer used.

Type of change

  • chore (Updates on dependencies, gitignore, etc)

How Has This Been Tested?

With pipelines

PR Checklist:

Make sure each step was done:

  • Updated changelog with PR's intent
  • Lint with make lint-fix

@Thaleszh Thaleszh requested a review from jhelison as a code owner February 5, 2026 18:04
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

This pull request removes the Wasmd precompile feature entirely from the codebase. The changes include deletion of the Solidity interface (IWasmd.sol), all Go implementation files in precompiles/wasmd (including event emission, transaction handling, query methods, and types), corresponding unit and integration tests, mock contract bindings, and E2E test files. A new v7.1.0 upgrade handler is added that filters the Wasmd precompile address (0x0000000000000000000000000000000000001001) from the EVM's active static precompiles list. The precompile registration is removed from app/keepers/precompiles.go, and genesis configurations are updated accordingly. The changelog documents the removal.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Remove unused wasmd precompile' directly and accurately describes the primary change: removal of the wasmd precompile code throughout the codebase.
Description check ✅ Passed The description clearly relates to the changeset, stating the intent to strip out unused wasmd precompile code and noting it was tested via pipelines and changelog was updated.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-wasmd-precompile

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.

❤️ Share

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

@Thaleszh Thaleszh merged commit 7db5a2b into main Feb 5, 2026
9 checks passed
@Thaleszh Thaleszh deleted the chore/remove-wasmd-precompile branch February 5, 2026 18:20
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