Open
Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
sergeytimoshin
force-pushed
the
sergey/interfaces
branch
4 times, most recently
from
cfc1a75 to
4289f14
Compare
sergeytimoshin
force-pushed
the
sergey/interfaces-infra
branch
2 times, most recently
from
c6ef964 to
7106bcf
Compare
* feat: infrastructure updates - migrations, ingester, prover API key, DAO schema, monitor, CLI args, dep updates - Add prover_api_key parameter to PhotonApi and validity proof methods - Add max_connections parameter to run_server - Add new migrations: onchain_pubkey, ata_owner, backfill_mint, ata_owner_index - Update ingester with startup cleanup, improved parsing, and persist logic - Update DAO generated models (accounts, token_accounts, queue_hash_chains, transactions) - Update monitor with queue monitoring and tree metadata sync improvements - Consolidate OpenAPI specs into single api.yaml - Fix OpenAPI AllOf handling and iterator usage - Update dependencies in Cargo.toml/Cargo.lock - Add token_layout module and update token_data types - Update snapshot loader/snapshotter and tree validator * refactor: remove unused backfill migration for mint onchain pubkey * refactor: rename compressed mint PDA constants to light mint PDA for clarity * refactor: remove parse_account_discriminator function and update usages for clarity * feat: add discriminator_v2 field and update related logic for account processing
…abs#341) Bumps all light-* dependencies to compatible versions: - light-event: 0.21 -> 0.23.1 (includes OOB panic fix) - light-concurrent-merkle-tree: 4 -> 5 - light-batched-merkle-tree: 0.10 -> 0.11 - light-merkle-tree-metadata: 0.10 -> 0.11 - light-compressed-account: 0.10.1 -> 0.11 - light-indexed-merkle-tree: 4 -> 5 - light-token-interface: 0.4 -> 0.5 - light-sdk-types: 0.20 -> 0.23 Fixes the index-out-of-bounds panic in create_nullifier_queue_indices() that halted both pitt and fra ingestors at slot 407265370. Upstream fix: Lightprotocol/light-protocol#2353 cargo check passes. Co-authored-by: i-am-bert <i-am-bert@users.noreply.github.com>
When the indexer encounters a tree not in tree_metadata during transaction parsing, fetch the tree's on-chain account via RPC, add it to tree_metadata, and proceed with indexing instead of skipping. This prevents data gaps for legitimate trees created after startup.
…DAO schema, monitor, CLI args, dep updates - Add prover_api_key parameter to PhotonApi and validity proof methods - Add max_connections parameter to run_server - Add new migrations: onchain_pubkey, ata_owner, backfill_mint, ata_owner_index - Update ingester with startup cleanup, improved parsing, and persist logic - Update DAO generated models (accounts, token_accounts, queue_hash_chains, transactions) - Update monitor with queue monitoring and tree metadata sync improvements - Consolidate OpenAPI specs into single api.yaml - Fix OpenAPI AllOf handling and iterator usage - Update dependencies in Cargo.toml/Cargo.lock - Add token_layout module and update token_data types - Update snapshot loader/snapshotter and tree validator
…ountInterfaces) - Add getAccountInterface and getMultipleAccountInterfaces RPC methods - Implement racing logic for compressed vs on-chain account resolution - Add interface types (AccountInterface, SolanaAccountData) - Register new methods in RPC server and OpenAPI spec - Add integration tests with snapshot testing - Add test transaction data for indexer interface
…ted test structure
sergeytimoshin
force-pushed
the
sergey/interfaces
branch
from
141d527 to
eadaf11
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
No description provided.