Rate-limit TUI frames from their actual emission time - #39057
Merged
copyberry[bot] merged 1 commit intoAug 17, 2026
Merged
Conversation
## Why Recording a late frame's requested deadline as its emission time can leave the rate limiter with a stale timestamp, allowing the next frame to be emitted immediately in a redraw burst. ## What changed Record the current time when the frame scheduler sends a draw notification so the 120 FPS limit is measured from the actual emission. ## Testing Add a paused-time regression test that submits an overdue draw and verifies the next frame waits for `MIN_FRAME_INTERVAL`. GitOrigin-RevId: 8c9a5ac0bf858b2ad87838f6d7c67c9470fdfc61
copyberry
Bot
force-pushed
the
copyberry/codex-internal-to-codex-oss/8c9a5ac0bf858b2ad87838f6d7c67c9470fdfc61
branch
from
August 17, 2026 19:35
2c95a56 to
796325f
Compare
copyberry
Bot
deleted the
copyberry/codex-internal-to-codex-oss/8c9a5ac0bf858b2ad87838f6d7c67c9470fdfc61
branch
August 17, 2026 19:36
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.
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.
Rate-limit TUI frames from their actual emission time
Why
Recording a late frame's requested deadline as its emission time can leave the
rate limiter with a stale timestamp, allowing the next frame to be emitted
immediately in a redraw burst.
What changed
Record the current time when the frame scheduler sends a draw notification so
the 120 FPS limit is measured from the actual emission.
Testing
Add a paused-time regression test that submits an overdue draw and verifies the
next frame waits for
MIN_FRAME_INTERVAL.