Skip to content

feat(app): show only the tokens a bot spent, without the reread conversation - #88

Merged
httpsphl merged 2 commits into
mainfrom
feat/tokens-spent-only
Oct 1, 2026
Merged

httpsphl merged 2 commits into
mainfrom
feat/tokens-spent-only

Conversation

@httpsphl

@httpsphl httpsphl commented Oct 1, 2026

Copy link
Copy Markdown
Owner

On top of #87 (both touch the same strings): until #87 merges, its commit shows here too. Merge #87 first, with a merge commit.

The reread count next to the spent one confused people. Botloft now shows only what the bot spent.

  • Usage dialog: each row reads "Ops · Worked 2 times", with no "48k reread". The intro no longer talks about the reread conversation.
  • Chat: the turn line was already only the spent tokens; its tooltip drops "reread X of the conversation".
  • The conversation sent again after a pause ("reloaded") still shows apart, as before.
  • cacheRead is still stored in the turn item and summed by usage.tokens; only the app stops showing it. Spec 8.7 says so, with the reason.

Tests

  • Chat and usage tests updated to the new text.
  • pnpm check passes locally. No Rust changes.

Manual check

Nothing depends on the real Claude Code. After a bot replies, hover the turn line in the chat and open Usage: neither mentions reread tokens.

Two crews can have bots with the same name, and the usage dialog only
showed the name. usage.tokens now carries the crew's name, and each row
reads "Ops · Worked 2 times · ...".
…rsation

The reread count next to the spent one confused people. The chat's turn
line tooltip and the usage dialog drop it; the conversation sent again
after a pause still shows apart. cacheRead is still stored and summed.
@httpsphl
httpsphl enabled auto-merge October 1, 2026 12:06
@httpsphl
httpsphl merged commit 2f5cce8 into main Oct 1, 2026
3 checks passed
@httpsphl
httpsphl deleted the feat/tokens-spent-only branch October 1, 2026 12:24
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.

1 participant