Repository navigation
Add metrics collection, typed events, transaction tracking, and Asset Pool integration - #1215
Merged
Kingsman-99 merged 5 commits intoOct 21, 2026
Conversation
|
@ubermensh-323 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
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
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.
Summary
Add metrics collection, typed events, transaction tracking, and Asset Pool integration
What was solved
#1140 — Add performance metrics
Add a performance metrics subsystem to the SDK: a lightweight metrics collector that records RPC request latency, cache hit/miss rates, and batch operation performance, plus an export API producing standard-format (e.g., Prometheus/JSON) metrics for monitoring integration. Wire collection into existing RPC/cache/batch paths with minimal overhead and expose historical/aggregated metrics.
Addressed:
#1141 — Implement event emitter
Add a typed event emitter to the SDK that emits state-change and transaction lifecycle events, supports registration/deregistration, filtering, and async handlers without leaking listeners.
Addressed:
#1142 — Add transaction event tracking
Add a transaction event tracking system that emits structured events for each transaction lifecycle step (created, simulated, signed, submitted, confirmed) plus error events, so the UI can track transaction progress. This requires a small event schema/emitter module and wiring it into the existing transaction flow.
Addressed:
#1159 — Implement Stellar Asset Pool integration for liquidity
Add a Stellar Asset Pool integration layer to the SDK covering pool interaction, liquidity provider support, swap routing, slippage calculation, and pool monitoring. The repo already has an AMM calculator (src/ammCalculator.ts) and price oracle, so the contribution should build on those patterns with a focused pool module plus tests, staying within the issue's stated scope.
Addressed:
Changes
src/analyticsTracking.ts(modify)src/analyticsDashboardExporter.ts(modify)src/events/eventEmitter.ts(create)src/events/types.ts(create)src/transactionEvents.ts(create)src/transactionBuilder.ts(create)src/transactionSigner.ts(create)src/ammCalculator.ts(modify)Approach
Issues
Closes #1140
Closes #1141
Closes #1142
Closes #1159