Skip to content

Conversation

@masih
Copy link
Collaborator

@masih masih commented Sep 26, 2025

Merge git commit history from sei-wasmd into sei-chain and update the replace directive in root of sei-chain to use relative path.

Note

The merged commits from sei-wasmd point to v0.3.9, consistent with the version bubbled up to sei-chain prior to this migration. Note that 4 commits from sei-wasmd are excluded, where 2 of which carry functional significance. Due to issues detected during the bubble up related to high gas usage the work here explicitly excluded those commits. A seprate set of PRs will re-introduce the excluded commits to isolate risk from mono-repo migration.
The PRs that need to be seprately bubbled up to sei-chain are:

codchen and others added 30 commits February 6, 2023 15:01
Add mutex around WasmVM write calls
Use HexBytes without custom Marshal/UnmarshalJSON functions
Replace cosmos/tendermint/iavl/tmdb/ibc dependencies with forked versions
refactor tx position and update usages
* Smart Contract Query Metrics

* Add more keys
* Remove Non-Multiplier Gas Meter

* Remove go mod changes

* Remove go sum changes
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@masih masih force-pushed the masih/mono-ish-wasmd branch from 6709d60 to 9045827 Compare September 26, 2025 14:14
@codecov
Copy link

codecov bot commented Sep 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.89%. Comparing base (d3e9077) to head (813c111).
⚠️ Report is 1 commits behind head on main.

❌ Your project status has failed because the head coverage (51.89%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2400      +/-   ##
==========================================
- Coverage   61.26%   51.89%   -9.37%     
==========================================
  Files         382      464      +82     
  Lines       35316    45388   +10072     
==========================================
+ Hits        21635    23556    +1921     
- Misses      12146    20015    +7869     
- Partials     1535     1817     +282     
Files with missing lines Coverage Δ
sei-wasmd/x/wasm/artifacts/v152/api/callbacks.go 55.17% <ø> (ø)
sei-wasmd/x/wasm/artifacts/v152/api/iterator.go 77.27% <ø> (ø)
sei-wasmd/x/wasm/artifacts/v152/api/lib.go 18.44% <ø> (ø)
sei-wasmd/x/wasm/artifacts/v152/api/memory.go 42.10% <ø> (ø)
...ei-wasmd/x/wasm/artifacts/v152/api/mock_failure.go 0.00% <ø> (ø)
sei-wasmd/x/wasm/artifacts/v152/api/mocks.go 0.00% <ø> (ø)
...ei-wasmd/x/wasm/artifacts/v152/api/testdb/memdb.go 0.00% <ø> (ø)
...x/wasm/artifacts/v152/api/testdb/memdb_iterator.go 0.00% <ø> (ø)
sei-wasmd/x/wasm/artifacts/v152/api/version.go 0.00% <ø> (ø)
sei-wasmd/x/wasm/artifacts/v152/lib.go 10.41% <ø> (ø)
... and 6 more

... and 74 files with indirect coverage changes

🚀 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.

Test prior to the work linked below were broken in seiwasmd:
* sei-protocol/sei-wasmd#80

Postponing the wasmd CI test runs until that work is independently
migrated over.
@masih masih marked this pull request as ready for review September 26, 2025 15:34
@masih masih merged commit 07f1420 into main Sep 26, 2025
71 of 74 checks passed
@masih masih deleted the masih/mono-ish-wasmd branch September 26, 2025 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.