BAM writer service: BAM writer service #27
Merged
Conversation
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
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new BAM writer service to track BAM validator metrics per epoch and adds support for jito pool eligibility fields in the validators table. Key changes include:
- New BAM writer service that monitors BAM delegations and calculates available delegation stake based on JIP-28 specification
- New API endpoint
/api/v1/bam_epoch_metricfor retrieving BAM epoch metrics - Added
jito_pool_eligibleandjito_pool_directed_stake_targetfields to validators
Reviewed Changes
Copilot reviewed 45 out of 46 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| bam-writer-service/* | New service to track BAM validator metrics including total network stake weight, available delegation stake, and eligible validator count per epoch |
| core/src/db_models/bam_epoch_metric.rs | New database model for storing BAM epoch metrics with CRUD operations |
| core/src/fetcher.rs | Added logic to calculate jito pool eligibility based on voting epochs and stake thresholds, and check directed stake targets |
| core/src/validators_app.rs | Added Localhost cluster variant support |
| api/src/schemas/bam_epoch_metric.rs | New schema for BAM epoch metric API responses |
| api/src/resolvers/query_resolver.rs | Added resolver and endpoint for BAM epoch metrics |
| api/src/schemas/validator.rs | Added jito pool eligibility fields to validator schema |
| writer-service/src/stake_pool_manager.rs | Updated to pass jito steward configuration for pool eligibility checks |
| api-client/* | New Kobe API client library with comprehensive async support |
| Cargo.toml | Updated dependencies including stakenet-sdk revision and reqwest version |
| core/src/rpc_utils.rs | Replaced solana-rpc-client-api with solana-client error types |
| Dockerfile | Added bam-writer-service container image |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 48 out of 49 changed files in this pull request and generated 18 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
API
validatorsto add new fieldsjito_pool_eligible,jito_pool_directed_stake_targetBam Writer Service
Kobe Writer Service
jito_pool_eligible,jito_pool_directed_stake_targetin validators table