.#114
Conversation
|
@CollinsKRO is attempting to deploy a commit to the emrekayat's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thanks for the PR. I reviewed the diff from the maintainer account, but this is not merge-ready. The PR claims to close #55, but the current diff only changes: There is no code, docs, or test change that satisfies #55, and the PR title is just |
|
@CollinsKRO 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! 🚀 |
|
@emrekayat pls review this again |
closes #55
Summary
Adds
--receipt(and--jsonalias) CLI flag forsearch,news, andscrapecommands. When set, the flag suppresses the verbose text summary and instead prints a structured JSON receipt containing only the key payment/query metadata — ideal for terminal demos, SCF follow-up, and investor presentations.Receipt shape
{ "command": "search", "provider": "search.basic", "input": "latest soroban updates", "price": 0.01, "traceId": "trace_abc123def456" }commandsearch/news/scrape)provider--providerargument or defaultinput...)priceresult.priceUsdfrom the paid responsetraceIdresult.traceIdfrom the paid responseUsage
Changes
apps/agent-client/src/cli.ts— AddedhasFlag(),redactInput(),buildReceipt()helpers;--receipt/--jsonflag parsing; receipt-only output path; guardedmain()call so module can be imported in testsapps/agent-client/src/cli.test.ts— Added 6 tests covering flag error handling, input redaction, and receipt structureVerification
npm run test)npm run typecheck)