Repository navigation
Avoid double-counting retained reasoning in token usage - #55991
Merged
copyberry[bot] merged 1 commit intoOct 21, 2026
Merged
copyberry[bot] merged 1 commit into
copyberry[bot] merged 1 commit into
Conversation
## Why Responses Lite includes retained reasoning in its reported token usage even without an `X-Reasoning-Included` header. Adding a separate reasoning estimate can trigger automatic compaction prematurely. Locally recomputed usage also already includes retained reasoning. ## What changed Initialize reasoning accounting from `use_responses_lite` before each sampling or compaction request, while allowing server events to override it. Mark locally recomputed token usage as including reasoning. ## Testing Update the compaction regression test to verify that Responses Lite usage without the header stays below an 81-token limit at 80 tokens and triggers compaction at 81 tokens. GitOrigin-RevId: 06f9272cb3c259e927941f1d2a1d7096310ffb2c
copyberry
Bot
force-pushed
the
copyberry/codex-internal-to-codex-oss/06f9272cb3c259e927941f1d2a1d7096310ffb2c
branch
from
October 21, 2026 21:23
4679e11 to
07cc649
Compare
Contributor
|
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
copyberry
Bot
deleted the
copyberry/codex-internal-to-codex-oss/06f9272cb3c259e927941f1d2a1d7096310ffb2c
branch
October 21, 2026 21:23
Contributor
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
juancarlosroca
temporarily deployed
to
issue-triage
October 21, 2026 21:31 — with 
GitHub Actions
Inactive
GitHub Actions
Inactive
juancarlosroca
temporarily deployed
to
issue-triage
October 21, 2026 21:31 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
juancarlosroca
temporarily deployed
to
issue-triage
October 21, 2026 21:31 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
This branch was previously deployed
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Avoid double-counting retained reasoning in token usage
Why
Responses Lite includes retained reasoning in its reported token usage even without an
X-Reasoning-Includedheader. Adding a separate reasoning estimate can trigger automatic compaction prematurely. Locally recomputed usage also already includes retained reasoning.What changed
Initialize reasoning accounting from
use_responses_litebefore each sampling or compaction request, while allowing server events to override it. Mark locally recomputed token usage as including reasoning.Testing
Update the compaction regression test to verify that Responses Lite usage without the header stays below an 81-token limit at 80 tokens and triggers compaction at 81 tokens.