Skip to content

feat: add spend-request list command#108

Open
harryguo-nyc wants to merge 4 commits into
mainfrom
harry/spend-request-list
Open

feat: add spend-request list command#108
harryguo-nyc wants to merge 4 commits into
mainfrom
harry/spend-request-list

Conversation

@harryguo-nyc
Copy link
Copy Markdown

Implements spend-request list subcommand that calls GET /spend_requests, unwraps the response, and renders active spend requests grouped by status with color coding. Includes SDK interface update, implementation, Ink UI component, subcommand registration, and unit/integration tests.
Screenshot 2026-05-20 at 5 15 53 PM

Screenshot 2026-05-20 at 5 19 16 PM

Committed-By-Agent: claude

Implements `spend-request list` subcommand that calls GET /spend_requests,
unwraps the response, and renders active spend requests grouped by status
with color coding. Includes SDK interface update, implementation, Ink UI
component, subcommand registration, and unit/integration tests.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed-By-Agent: claude
@harryguo-nyc harryguo-nyc requested a review from a team as a code owner May 20, 2026 21:19
harryguo-nyc and others added 2 commits May 20, 2026 17:23
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed-By-Agent: claude
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed-By-Agent: claude
Copy link
Copy Markdown
Collaborator

@kreese-stripe kreese-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good, but I'm getting the following error after running the command in interactive mode:
Image

from debugging with claude it appears the issue might be that we're not explicitly calling exit() from the useApp Ink hook once the component is done/once we handle completion (other commands use useInput which handles exiting automatically. let's update to include that to fix this in interactive mode

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed-By-Agent: claude
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants