Skip to content

Move sei-iavl into sei-chain mono repo#2624

Merged
masih merged 98 commits intomainfrom
masih/mono-iavl
Dec 17, 2025
Merged

Move sei-iavl into sei-chain mono repo#2624
masih merged 98 commits intomainfrom
masih/mono-iavl

Conversation

@masih
Copy link
Copy Markdown
Collaborator

@masih masih commented Dec 17, 2025

The sei-iavl fork has a direct dependency to the original tendermint module. This blocks the go mod flattening effort in sei-chain, because flattening sei-tendermint will otherwise leave behind transient dependency to the original tendermint module in the go module dependency tree.

The work here:

  • moves the current head of sei-iavl into sei-chain, i.e. commit ff17b3473ee2438caa1777930a0bf73d267527fa.
  • Adjusts proto generation as needed, and removes redundant duplicate protos.
    • Note that the existing pb.go files in sei-iavl were out of date and are re-generated.
  • Directly flattens sei-iavl module into sei-chain while at it.
  • Preserves the original commit history from sei-iavl, which requires rebase-merge as opposed to the usual squash merge.

Upon the merge of this PR, sei-iavl repo should be archived.

We need to do this in order to get rid of the transient dependency to tendermint in `sei-iavl`. Otherwise, it blocks the upcoming flattening of `sei-tendermint` into `sei-chain`.
Fix the buf-based generation to integrate `sei-avl`.

Update out of date pb.go files, and remove now redundant duplicate protos.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 17, 2025

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedDec 17, 2025, 4:39 PM

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 17, 2025

Codecov Report

❌ Patch coverage is 59.87385% with 1654 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.62%. Comparing base (10539a2) to head (665a792).
⚠️ Report is 69 commits behind head on main.

Files with missing lines Patch % Lines
sei-iavl/mutable_tree.go 66.90% 154 Missing and 119 partials ⚠️
sei-iavl/nodedb.go 60.48% 151 Missing and 96 partials ⚠️
sei-iavl/mock/db_mock.go 0.00% 213 Missing ⚠️
sei-iavl/cmd/iaviewer/main.go 0.00% 141 Missing ⚠️
sei-iavl/benchmarks/cosmos-exim/main.go 0.00% 115 Missing ⚠️
sei-iavl/node.go 73.65% 62 Missing and 46 partials ⚠️
sei-iavl/immutable_tree.go 52.31% 62 Missing and 10 partials ⚠️
sei-iavl/internal/rand/random.go 55.46% 56 Missing and 1 partial ⚠️
sei-iavl/internal/encoding/encoding.go 26.66% 54 Missing and 1 partial ⚠️
sei-iavl/proof_range.go 83.38% 29 Missing and 21 partials ⚠️
... and 18 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2624      +/-   ##
==========================================
+ Coverage   43.15%   43.62%   +0.46%     
==========================================
  Files        1867     1899      +32     
  Lines      154112   158389    +4277     
==========================================
+ Hits        66509    69090    +2581     
- Misses      81637    82954    +1317     
- Partials     5966     6345     +379     
Flag Coverage Δ
sei-chain 45.52% <59.87%> (+0.91%) ⬆️
sei-cosmos 38.03% <ø> (+<0.01%) ⬆️
sei-db 69.06% <ø> (ø)
sei-tendermint 47.50% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cmd/seid/cmd/debug.go 0.00% <ø> (ø)
sei-cosmos/store/iavl/store.go 58.11% <ø> (ø)
sei-cosmos/store/iavl/tree.go 65.62% <ø> (ø)
sei-cosmos/store/rootmulti/store.go 69.48% <ø> (ø)
sei-cosmos/storev2/commitment/store.go 9.37% <ø> (ø)
sei-db/sc/memiavl/db.go 63.14% <ø> (ø)
sei-db/sc/memiavl/multitree.go 79.55% <ø> (ø)
sei-db/sc/memiavl/proof.go 90.17% <ø> (ø)
sei-db/sc/memiavl/tree.go 79.38% <ø> (ø)
sei-db/ss/test/storage_bench_suite.go 0.00% <ø> (ø)
... and 41 more

... and 13 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.

Copy link
Copy Markdown

@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 enabled auto-merge (rebase) December 17, 2025 17:33
@masih masih merged commit 217ae36 into main Dec 17, 2025
41 of 70 checks passed
@masih masih deleted the masih/mono-iavl branch December 17, 2025 17:51
masih added a commit to sei-protocol/sei-iavl that referenced this pull request Dec 17, 2025
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.

7 participants