Skip to content

feat: add LunarBase protocol integration#1053

Open
ThomasAqu1nas wants to merge 9 commits into
propeller-heads:mainfrom
Lunarbase-Lab:update/lunarbase-integration
Open

feat: add LunarBase protocol integration#1053
ThomasAqu1nas wants to merge 9 commits into
propeller-heads:mainfrom
Lunarbase-Lab:update/lunarbase-integration

Conversation

@ThomasAqu1nas

Copy link
Copy Markdown

No description provided.

@claude claude Bot 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.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@ThomasAqu1nas ThomasAqu1nas changed the title Add LunarBase protocol integration feat: add LunarBase protocol integration May 28, 2026
Comment thread protocols/substreams/base-lunarbase/src/modules/1_map_protocol_components.rs Outdated
Comment thread protocols/substreams/base-lunarbase/src/modules/3_map_protocol_changes.rs Outdated
Comment thread protocols/substreams/base-lunarbase/src/tycho_mapper.rs Outdated
Comment thread protocols/substreams/base-lunarbase/src/lunarbase/events.rs Outdated
Comment thread protocols/substreams/base-lunarbase/src/lunarbase/indexed.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/decoder.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/decoder.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/mod.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/mod.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/mod.rs Outdated
@ThomasAqu1nas ThomasAqu1nas force-pushed the update/lunarbase-integration branch from 6069396 to 80e5fd0 Compare June 2, 2026 12:11
Comment thread protocols/substreams/base-lunarbase/src/lunarbase/events.rs Outdated
Comment thread protocols/substreams/base-lunarbase/src/lunarbase/indexed.rs Outdated
@ThomasAqu1nas ThomasAqu1nas force-pushed the update/lunarbase-integration branch from ae46904 to 69e6db4 Compare June 3, 2026 12:10
@ThomasAqu1nas

Copy link
Copy Markdown
Author

@zach030, pls check latest commit, fixed

Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/state.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/state.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/decoder.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/state.rs
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/decoder.rs Outdated
Comment thread crates/tycho-simulation/src/evm/protocol/lunarbase/decoder.rs Outdated
@ThomasAqu1nas ThomasAqu1nas force-pushed the update/lunarbase-integration branch from 69e6db4 to a68cb27 Compare June 5, 2026 19:02
@ThomasAqu1nas

Copy link
Copy Markdown
Author

I checked the failed RPC-dependent CI jobs. The logs show ETH_RPC_URL, BASE_RPC_URL, RPC_URL and the other RPC env vars are all empty. It looks like repository secrets are not available for this fork PR rather than a LunarBase-specific failure. Could you rerun these checks in a trusted context, or confirm if RPC-dependent checks are expected to fail on external fork PRs?

@zach030

zach030 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

I checked the failed RPC-dependent CI jobs. The logs show ETH_RPC_URL, BASE_RPC_URL, RPC_URL and the other RPC env vars are all empty. It looks like repository secrets are not available for this fork PR rather than a LunarBase-specific failure. Could you rerun these checks in a trusted context, or confirm if RPC-dependent checks are expected to fail on external fork PRs?

Thanks, I’ll review the code. This looks like a CI setup issue on our side, we’ll check it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants