Repository navigation
fix(subagents): drop cost from subagent spend - #887
Merged
Merged
Conversation
Price is not wanted in the spend display, so remove the field rather than hide it: SubagentSpend keeps only the four token counters, the Pi backend stops copying stats.cost, and formatSubagentSpend returns just "<in> in (<cached> cached) · <out> out". Persisted result entries that still carry details.spend.cost keep rendering — the extra field is ignored — and tests now assert no "$" reaches details or subagent_check text. Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
github-actions
Bot
added
the
area:subagents
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
1 participant
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.
Problem
#886 showed a dollar cost next to subagent token spend. The maintainer does not want prices shown, and nothing else uses the cost field.
Value
Subagent surfaces show only token counts, and an unused field is gone.
Approach
costfromSubagentSpend, from what the Pi backend copies out ofgetSessionStats(), and from the formatter.21k in (10k cached) · 65 out.spend.costrender unchanged; the extra field is ignored, and a test covers it.Validation
bun run check: passed.bun run test: Node 2,359 passed / 0 failed; Vitest 1,498/1,498.Impact
· $…suffix disappears from subagent surfaces.subagent_checkline loses its cost suffix. Tools are unchanged.Generated with Devin