Skip to content

Fix streaming usage accounting#143

Merged
kzu merged 1 commit into
mainfrom
copilotd/issue-142-a398
May 5, 2026
Merged

Fix streaming usage accounting#143
kzu merged 1 commit into
mainfrom
copilotd/issue-142-a398

Conversation

@kzu
Copy link
Copy Markdown
Member

@kzu kzu commented May 5, 2026

Summary

  • fix streamed chat usage accounting by emitting usage deltas and handling counter resets across tool-driven streaming segments
  • add regression tests for streamed usage aggregation and reset handling
  • add a live integration test that compares multi-turn streaming and non-streaming usage totals

Closes #142

@kzu
Fix streaming usage accounting
14bbbb9 
Closes #142

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@kzu kzu linked an issue May 5, 2026 that may be closed by this pull request
Verify total token usage reported matches between streaming and non-streaming responses #142
Closed
@kzu
Copy link
Copy Markdown
Member Author

kzu commented May 5, 2026

67 passed 67 passed 37 skipped

🧪 Details on Ubuntu 24.04.4 LTS

from retest v1.1.0 on .NET 10.0.7 with 💜 by @devlooped

@kzu kzu added the bug Something isn't working label May 5, 2026
@kzu kzu merged commit 3f71e78 into main May 5, 2026
4 checks passed
@kzu kzu deleted the copilotd/issue-142-a398 branch May 5, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Verify total token usage reported matches between streaming and non-streaming responses

1 participant

@kzu