Repository navigation
fix(app): make the turn navigator thin, instant and obvious - #57672
Closed
kitlangton wants to merge 2 commits into
Closed
kitlangton wants to merge 2 commits into
kitlangton wants to merge 2 commits into
Conversation
kitlangton
force-pushed
the
turn-navigator-polish
branch
from
October 21, 2026 09:16
9153122 to
105cd06
Compare
Contributor
Author
|
Session-load coordination: #57712 adds includeFiles=false only to the navigator user-index message.list request, preserving full prompt text and metadata. Its controlled handler/client benchmark is 22.64→0.57 MB and 520→91 ms. Please preserve that flag when combining turn-navigator-polish; no preview truncation or hover/render changes are part of #57712. |
Contributor
Author
|
Closing: the turn navigator was removed from beta in ee8221c, and we're not bringing it back. |
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
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.
Kit tried the merged turn navigator (#57572) in OpenCode Beta and found it confusing. The ticks read as a thick grey bar, hovering made a bulge, the preview came late, and nothing explained what the rail does. #57682 has since replaced the fade on the prompt with a 4-line clamp; this PR keeps that.
Changes
Proof (headless mocked app, dark, 1200×760, real time; before is beta)
before-after.mp4
The clip shows a long session (200 turns), then a short one (5 turns), hovering up and down and clicking a middle tick and the bottom tick.
A prompt longer than three lines, before and after:
timeline-history.spec.tsand one insettings.spec.ts.Performance
session-timeline-benchmarkstreaming, with 320 history turns, 160 deltas and a 30× CPU throttle. The benchmark fixture had the rail turned on for this run; the setting is off by default. Two rounds on Studio, alternating beta, this head, and this head with the rail off as a control. Medians:With the rail mounted, streaming costs the same as with it off; there is no regression. An earlier, noisier run on a loaded Pro, before rebasing onto #57682, pointed the same way: beta 78 s and this PR 33 s, with the rail off at 41 s.
LOC: code +100/−50 (
turn-navigator.tsx), plus one English string. No test changes: the keepers cover the rail's contract. Sampling only takes effect at sizes those fixtures don't reach, so the proof above covers it.