feat(harness): Droid Phase 3 — thinking/cacheWrite tokens + cache-read cost - #59
Open
joshau wants to merge 2 commits into
Open
feat(harness): Droid Phase 3 — thinking/cacheWrite tokens + cache-read cost#59joshau wants to merge 2 commits into
joshau wants to merge 2 commits into
Conversation
…d cost Parse `[Session] Saving session settings` lines to extract cumulative thinkingTokens and cacheCreationTokens; delta-track per sessionId so each scan emits only the incremental count as new `thinking` and `cacheWrite` direction hook events. Add 0.1× cache-read multiplier to estimate_cost_usd and wire cacheReadInputTokens from `[Agent] Streaming result` into the call, closing the ~10% cost underestimate. Keep direction closed-enum in sync across Rust validate.rs and TS validate.ts; add two new default HookRules to defaults.rs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…her tests Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
Summary
[Session] Saving session settingslog lines to extract cumulativethinkingTokensandcacheCreationTokens. Delta-track persessionIdso each watcher scan emits only the incremental count asdroid.tokens.thinkinganddroid.tokens.cache_writehook events, mapping to the newthinkingandcacheWritedirection values.estimate_cost_usdand wirecacheReadInputTokensfrom[Agent] Streaming resultlines into the call, closing the ~10% cost underestimate.directionclosed-enum updated in bothvalidate.rs(Rust) andvalidate.ts(TS); two new defaultHookRuleentries added todefaults.rs.Test plan
cargo clippy --all-targets -- -D warningspassescargo test --lib— 695 tests pass (9 new: session delta parsing, multi-session tracking, payload emission, cache-read cost contribution)pnpm run build(Vite/TS) passes clean🤖 Generated with Claude Code