House keeping#10
Open
utx0 wants to merge 7 commits into
Open
Conversation
- Add agave-unstable-api feature to solana-transaction-status dependency This suppresses deprecation warnings about Solana 4.0 API changes - Remove unused 'debug' import from sologger_log_context These changes acknowledge that we're using Solana's unstable API surface and are prepared for potential breaking changes in future versions.
…coded_transaction - Change from_ui_confirmed_block to accept &UiConfirmedBlock instead of owned - Replace tx.meta.to_owned().unwrap() with tx.meta.as_ref().unwrap() to eliminate 552M String::clone calls per session - Update benchmark to pass reference Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…gregator) When ProgramsSelector filters by specific programs, parse_logs broke out of the loop on depth mismatch — e.g., when PumpSwap invoke [2] appeared inside Jupiter invoke [1] (not monitored). The depth tracker expected [1] but found [2], causing all CPI event data to be silently dropped. Now adjusts current_depth to match the actual invoke level when filtering, allowing CPI events to be captured normally. Adds tests for single and multiple CPI invocations through non-monitored aggregator programs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Use saturating_sub(1) for current_depth decrement to prevent "attempt to subtract with overflow" when CPI depth tracking gets misaligned (e.g., Jupiter CPI invocations) - Guard all call_ids[call_ids.len() - 1] accesses with .last() to prevent "index out of bounds" panic on empty vec - Guard call_stack access in program_return with .last() - Guard call_stack.pop().unwrap() in program_failed_result Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
1 participant
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.
agave-unstable-apifeature acknowledges we're using unstable APIs and are prepared for potential breaking changes