feat: implement issues #882 #883 #884 #885 - #901
Merged
Kingsman-99 merged 3 commits intoSep 28, 2026
Merged
Kingsman-99 merged 3 commits into
Kingsman-99 merged 3 commits into
Conversation
…lit#884 Stellar-split#885 Stellar-split#882 - Add MockStellarSplitClient (src/mock/index.ts) - Full in-memory StellarSplitClient interface implementation - reset(), setInvoice(), getCallHistory(), simulateEvent() - subscribeToInvoice() returns unsubscribe fn (real client parity) - subscribeInvoice() returns Subscription object (test convenience) - Exported from ./mock entry point Stellar-split#883 - Add ConfigurationError with field/value/hint (src/configValidator.ts) - ConfigurationError class with structured field, value, hint fields - validateConfigStrict(): rpcUrl must be https, contractId must be 56-char C-address, network must be testnet|mainnet, passphrase must match network when both provided - NETWORK_PASSPHRASE_MAP exported for testnet/mainnet constants Stellar-split#884 - Add React hooks package (src/react/index.tsx) - StellarSplitProvider context provider - useInvoice(id): data/loading/error/refetch - useCreatorInvoices(creator, filter): data/loading/error/hasMore/loadMore - useProtocolStats(): data/loading/error - useInvoiceStream(id): latestEvent/isConnected/error with auto-cleanup - Exported from ./react entry point Stellar-split#885 - Add SDK telemetry module (src/telemetryModule.ts) - StellarSplitTelemetry: opt-in (disabled by default) - Per-method callCount/errorCount/p50/p95/p99 latency - getMetrics(), resetMetrics(), flush(), destroy() - Periodic flush to endpoint when configured - withTelemetry() wrapper helper - Exported from ./telemetry entry point Tests: 103 tests, all passing (34 mock + 28 configValidator + 23 telemetry + 18 react) Types: zero new TypeScript errors introduced Build: new entry points added to tsup.config.ts and package.json exports
|
@kurehkim334-lang 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
2 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.
#882 - Add MockStellarSplitClient (src/mock/index.ts)
#883 - Add ConfigurationError with field/value/hint (src/configValidator.ts)
#884 - Add React hooks package (src/react/index.tsx)
#885 - Add SDK telemetry module (src/telemetryModule.ts)
Tests: 103 tests, all passing (34 mock + 28 configValidator + 23 telemetry + 18 react)
Types: zero new TypeScript errors introduced
Build: new entry points added to tsup.config.ts and package.json exports
Closes
closes #882
closes #883
closes #884
closes #885