feat: budget simulation, analytics pagination, wallet balance hook, agent events - #192
Merged
Cjay-Cyber-2 merged 5 commits intoSep 1, 2026
Conversation
Add the resource methods requested for @astroid/budget: getBudget and listBudgets as documented aliases of the existing get/list, and a new simulateBudgetCheck that POSTs a prospective spend to the budget simulate endpoint. Include the BudgetSimulationRequest/Result types in @astroid/types and unit tests covering allowed spends and limit breaches with a mocked transport. Closes ASTROIDX556#117 Generated with Codebuff 🤖 Co-Authored-By: Codebuff <noreply@codebuff.com>
Extend AnalyticsQueryParams with the shared PaginationParams (cursor, limit, order) and serialize them in getMetrics and getVolumeSummary. Add unit tests verifying query-string serialization and omission of undefined pagination args. Closes ASTROIDX556#127 Generated with Codebuff 🤖 Co-Authored-By: Codebuff <noreply@codebuff.com>
Expose useWalletBalance from @astroid/react, backed by the wallet balance endpoint through the AstroidProvider context. Cache under a dedicated query key, default to a 30s stale time, and support optional polling. Include hook and query-key tests. Closes ASTROIDX556#129 Generated with Codebuff 🤖 Co-Authored-By: Codebuff <noreply@codebuff.com>
Define agent lifecycle event payload, filter, and subscription option types in @astroid/types, and add listEvents, subscribe and unsubscribe methods to the agent resource client. Add unit tests verifying request path construction and query serialization. Closes ASTROIDX556#133 Generated with Codebuff 🤖 Co-Authored-By: Codebuff <noreply@codebuff.com>
|
MergeKeeper review Scope: in scope for linked issue The PR successfully implements budget simulation, analytics pagination, the wallet balance React hook, and agent lifecycle event subscriptions with comprehensive tests and type safety. Reviewed commit: |
|
MergeKeeper merge status Status: blocked Reason: One or more required CI checks failed. Failing checks:
Next steps:
|
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.
Summary
useWalletBalanceReact hook with tests.Closes #117
Closes #127
Closes #129
Closes #133