-
Notifications
You must be signed in to change notification settings - Fork 108
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
feat(ton): TON observer 💎 #2896
Conversation
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 📝 Walkthrough📝 Walkthrough📝 Walkthrough📝 Walkthrough📝 Walkthrough📝 Walkthrough📝 Walkthrough📝 Walkthrough📝 WalkthroughWalkthroughThe pull request introduces a series of enhancements and modifications to the ZetaChain node, focusing on the integration of stateful precompiled contracts, support for various blockchain functionalities, and improvements in testing frameworks. Key changes include the addition of a new RPC package, enhancements to Bitcoin chain support, and the introduction of new test cases for the TON blockchain. Refactoring efforts address code quality, while the changelog documents significant updates across multiple versions, reflecting a comprehensive improvement in functionality and structure. Changes
Assessment against linked issues
Possibly related PRs
Suggested labels
Suggested reviewers
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
* Use chainParams from base observer * Improve base observer Start() semantics * Add pkg/ticker options * Add TON to pkg/chains & protos * Add liteapi wrapper client; locate first tx * Add TON tx scraper * Add client.GetTransactionsUntil * Gateway WIP * Implement Gateway inbound tx parsing * Add Gateway tx filtration * Add GetBlockHeader cache; Add masterchain seqno. Implement watchInbound * Improve ton contracts pkg * Add IsTONChain() * Refactor Gateway package * Implement samples for TON * Add liteClient mocks * Add unit tests for inbound TON observer * Localnet: add TON ZRC20 * Wire the TON observer into the orchestrator * Add TON chain params of the fly! * TON deposits E2E wip * Fix bugs during cctx; * TON Deposits E2E 🫡✅ * TON Deposit And Call E2E * Merge fixes * Update changelog * gosec * Improve testing * Simplify ticker.Stop(). Leverage ctx.cancel() * Simplify E2E * Simplify liteapi semantics * Fix comments * Update zetaclient/chains/base/observer.go Co-authored-by: Francisco de Borja Aranda Castillejo <borja@zetachain.com> * Add gw explanatory comments; address PR comments * Apply fixes for AI suggestions --------- Co-authored-by: Francisco de Borja Aranda Castillejo <borja@zetachain.com>
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.
looks good
Description
This PR adds TON observer with all necessary dependencies. It does NOT cover gas economics
Closes #2808, #2757
Unit tests
E2E tests
Transaction Parsing
TON tools
liteapi
wrapperclient.GetFirstTransaction()
client.GetTransactionsUntil()
Zetacore
Misc
pkg/ticker
package. Stop ticker by external sig chanStop()
,ChainParams()
,Chain()
,SetChainParam()
from base observerHow Has This Been Tested?
Summary by CodeRabbit
New Features
Bug Fixes
Tests
Documentation