Show enrichment provider credits in CLI receipts - #290
Merged
Merged
Conversation
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.
A successful
firecrawl/enrichcall can report 30 provider credits while the CLI printsCredits: 0, because the receipt only recognizes SQL's separately billed costs. Include enrichment'sproviderCreditsin the existing separate-cost receipt field and displayed total, retaining the outer request's zero charge.Incomplete or invalid enrichment cost reports are labeled as known credits, with a notice that additional costs may have been incurred.
providerCostsCompletereflects completeness of the API's reported amounts, not independent confirmation of billing settlement. The change does not modify API responses, charge users or sum step costs a second time. Existing SQL and search receipt behavior remains intact.Validation: all 28 receipt tests pass, TypeScript build passes, formatting and diff checks pass. The built formatter reproduces the reported case as
Credits: 30 (0 outer request + 30 separately billed provider calls)without making a paid request.Summary by cubic
Fixes CLI receipts showing
Credits: 0after enrichment calls that report provider credits. Enrichment'sproviderCreditsnow count toward the separately billed total and displayed total, while the outer request stays at zero.providerCostsCompletereflects API-reported completeness, not billing confirmation.Written for commit a2151d4. Summary will update on new commits.