-
Notifications
You must be signed in to change notification settings - Fork 75
feat(besu): add IBFT 2.0 and QBFT light clients #1092
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.
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
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
8f9c9c5
poc: Besu IBFT 2.0 / QBFT light clients
gjermundgaraba 7217605
fix(besu): correct fixture paths in proof-api tests and e2e generator
gjermundgaraba 729a757
fix(besu): match validator quorum calculation
gjermundgaraba ce2f05e
fix(besu): address CI validation failures
gjermundgaraba 82bda31
fix(besu): address review findings
gjermundgaraba b3952ba
refactor(besu): simplify branch changes
gjermundgaraba 1eb1e7d
Simplify Besu e2e infrastructure
gjermundgaraba 4bea83a
refactor(besu): reuse dependency helpers
gjermundgaraba 0110436
refactor: extract dummy client msgs and simplify proof-api config
gjermundgaraba 4ccf100
fix(e2e): format Besu fixture imports
gjermundgaraba 0ac6ae6
fix(e2e): stabilize Besu transaction submission
gjermundgaraba 785c79f
fix(e2e): recheck Besu after peer startup
gjermundgaraba e01e1cb
fix(ci): update generated Go bindings
gjermundgaraba 2d8c0c5
fix(besu): align fixtures, tests, and CI
gjermundgaraba 61e2829
test(e2e): align Besu suite with project conventions
gjermundgaraba bb13755
refactor: remove unused EVM dummy runtime
gjermundgaraba 506a73c
fix(e2e): configure Besu private-network sync peers
gjermundgaraba d01a969
refactor(besu): deduplicate fixture client deployment
gjermundgaraba 97f9cb6
test(besu): table-drive valid fixture updates
gjermundgaraba 6abf71c
refactor(besu): address proof API review feedback
gjermundgaraba d8ce113
test(besu): table-drive rejection and proof cases
srdtrk 03915ca
test(besu): separate membership verification cases
srdtrk e111c3f
test: refactor
srdtrk 08aa39b
test(besu): combine success and rejection tables
srdtrk d9025d4
test: refactor
srdtrk e6eb1b9
test: pure
srdtrk f1d818c
chore: forge fmt
srdtrk 77b444f
nix: update
srdtrk c85a614
lint
srdtrk 6e0602a
lint
srdtrk 13a1aa2
fix(nix): reuse root nixpkgs for natlint
srdtrk 93eac04
lint
srdtrk d4e3285
lint more
srdtrk 1109075
deps: bun updated
srdtrk 60d4232
lint
srdtrk 3f473ec
test(solidity): disable Foundry call isolation
srdtrk f70ad1d
imp: nix hash
srdtrk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| # AGENTS.md | ||
|
|
||
| This repo combines Solidity/Foundry contracts, a Rust workspace, Solana Anchor programs, and Go-based end-to-end tooling for IBC Eureka. | ||
|
|
||
| Look here first: | ||
| - `README.md` | ||
| - `justfile` | ||
| - `ibc-solidity/foundry.toml` | ||
| - `Cargo.toml` | ||
| - `ibc-solidity/contracts/README.md` | ||
|
|
||
| Use the smallest relevant validation from the repo root: | ||
| - Solidity: `just solidity::lint-solidity` and `just solidity::test-foundry` | ||
| - Rust workspace, relayer, operator, and shared packages: `just lint-rust` and `just test-cargo` | ||
| - Broad cross-stack changes: `just lint` | ||
| - Solana or e2e changes: use the subtree-specific commands in `ibc-solana/AGENTS.md` or `e2e/interchaintestv8/AGENTS.md` | ||
|
|
||
| Hard constraints: | ||
| - Do not hand-edit generated outputs in `packages/go-abigen/`, most of `packages/go-anchor/`, `e2e/interchaintestv8/solana/go-anchor/`, or protobuf outputs under `e2e/interchaintestv8/types/`; regenerate with `just solidity::generate-abi`, `just solana::generate-solana-types`, or `just generate-buf`. | ||
| - `packages/go-anchor/ics07_tendermint_patches/` is the hand-maintained exception inside `packages/go-anchor/`; preserve it across regeneration. | ||
| - If Solidity interfaces, ABI-exposed structs, or contract types change, run `just solidity::generate-abi` before validating Go or e2e code. | ||
| - If Solana program interfaces or IDLs change, run `just solana::generate-solana-types`. | ||
| - If `.proto` files change, run `just generate-buf`. |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| # AGENTS.md | ||
|
|
||
| This subtree contains the Go end-to-end test suites that drive local Ethereum, Cosmos, and Solana environments through interchaintest. | ||
|
|
||
| Look here first: | ||
| - `e2e/interchaintestv8/README.md` | ||
| - `e2e/interchaintestv8/go.mod` | ||
| - the relevant suite entrypoint (`ibc_eureka_test.go`, `relayer_test.go`, `sp1_ics07_test.go`, `solana_test.go`, etc.) | ||
|
|
||
| Use the smallest relevant validation from the repo root: | ||
| - Single targeted test: `just test-e2e TestWithSuite/Test_Name` | ||
| - Common wrappers: `just test-e2e-eureka`, `just test-e2e-relayer`, `just test-e2e-cosmos-relayer`, `just test-e2e-sp1-ics07` | ||
| - Solana e2e wrappers: `just test-e2e-solana`, `just test-e2e-solana-gmp`, `just test-e2e-solana-ift`, `just test-e2e-solana-upgrade`, `just test-e2e-solana-attestation` | ||
| - Go lint for this subtree lives in `just lint-go`; see `justfile` for the other suite wrappers | ||
|
|
||
| Local constraints: | ||
| - Prefer the smallest targeted test; full e2e runs are slow and environment-dependent. | ||
| - Local runs expect `.env` set up from `.env.example`; `just test-e2e` installs the relayer, and `just test-e2e-sp1-ics07` also installs the operator. | ||
| - If contract interfaces or ABI-exposed types change, run `just generate-abi` before e2e validation. | ||
| - If Solana IDLs change, run `just generate-solana-types`; if `.proto` files change, run `just generate-buf`. | ||
| - Do not hand-edit generated code under `e2e/interchaintestv8/types/` or `e2e/interchaintestv8/solana/go-anchor/`. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should probably use matrix generation but we can do that in a follow up PR