Skip to content

feat: add token usage recording for responses blocking interceptor #124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
pawbana merged 1 commit into from
Jan 19, 2026
Merged

feat: add token usage recording for responses blocking interceptor #124

pawbana merged 1 commit into from
Jan 19, 2026
+267 −1

Conversation

@pawbana
Copy link
Contributor

@pawbana pawbana commented Jan 16, 2026

No description provided.

@pawbana Graphite App
Copy link
Contributor Author

pawbana commented Jan 16, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Jan 16, 2026
Merged
Merged
Merged
@pawbana pawbana force-pushed the pb/responses-record-tool-usage-streaming branch from a76a4d7 to 8e0756e Compare January 16, 2026 13:27
@pawbana pawbana force-pushed the 01-16-feat_add_token_usage_recording_for_responses_blocking_interceptor branch from a1f205c to 2102f13 Compare January 16, 2026 13:27
@pawbana pawbana force-pushed the pb/responses-record-tool-usage-streaming branch from 8e0756e to 8eff86e Compare January 16, 2026 13:32
@pawbana pawbana force-pushed the 01-16-feat_add_token_usage_recording_for_responses_blocking_interceptor branch from 2102f13 to cc6e1ee Compare January 16, 2026 13:32
@pawbana pawbana force-pushed the pb/responses-record-tool-usage-streaming branch from 8eff86e to 86a14ee Compare January 16, 2026 14:56
@pawbana pawbana force-pushed the 01-16-feat_add_token_usage_recording_for_responses_blocking_interceptor branch from cc6e1ee to 27b4f6e Compare January 16, 2026 14:56
@pawbana pawbana changed the base branch from pb/responses-record-tool-usage-streaming to graphite-base/124 January 16, 2026 14:59
@pawbana pawbana force-pushed the 01-16-feat_add_token_usage_recording_for_responses_blocking_interceptor branch from 27b4f6e to 3443c83 Compare January 16, 2026 15:01
@graphite-app graphite-app bot changed the base branch from graphite-base/124 to main January 16, 2026 15:01
@pawbana pawbana force-pushed the 01-16-feat_add_token_usage_recording_for_responses_blocking_interceptor branch 4 times, most recently from 7f3c6c2 to 01fede4 Compare January 16, 2026 15:17
@pawbana pawbana force-pushed the 01-16-feat_add_token_usage_recording_for_responses_blocking_interceptor branch from 01fede4 to ceeebcf Compare January 16, 2026 15:17
@pawbana pawbana marked this pull request as ready for review January 16, 2026 15:24
dannykopping
dannykopping approved these changes Jan 19, 2026
View reviewed changes
Copy link
Collaborator

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@pawbana Graphite App
Copy link
Contributor Author

pawbana commented Jan 19, 2026 •
edited
Loading

Merge activity

  • Jan 19, 8:31 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jan 19, 8:31 AM UTC: @pawbana merged this pull request with Graphite.

@pawbana pawbana merged commit 3eb6168 into main Jan 19, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dannykopping dannykopping dannykopping approved these changes

@mtojek mtojek Awaiting requested review from mtojek

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pawbana @dannykopping