Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork34.2k
gh-145448: Fix REPL tab completion cursor position in pending wrap state#145727
Open
mvanhorn wants to merge 1 commit intopython:mainfrom
Open
gh-145448: Fix REPL tab completion cursor position in pending wrap state#145727mvanhorn wants to merge 1 commit intopython:mainfrom
mvanhorn wants to merge 1 commit intopython:mainfrom
Conversation
…rap stateWhen __write_changed_line writes to the last column of the terminal,the cursor enters "pending wrap" state where it physically stays atwidth-1. However, posxy was recording width, causing subsequent CUB(Cursor Backward) calculations to overshoot by one cell.Cap posxy x-coordinate at width-1 to match the physical cursorposition in pending wrap state. This fixes the cursor moving backwardduring tab completion in xterm and Ghostty.Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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 freeto 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.
Uh oh!
There was an error while loading.Please reload this page.
Fixes#145448
Summary
When the PyREPL's
__write_changed_linewrites to the last column of the terminal, the cursor enters "pending wrap" state - it physically stays at columnwidth-1butposxywas recordingwidth. This caused subsequent CUB (Cursor Backward) escape sequences to overshoot by one cell, moving the cursor one position too far left.Root cause: In
unix_console.py, three code paths in__write_changed_linesetposxyto the full line width without accounting for pending wrap state. When the cursor is later moved back (e.g., to return to the input line after showing "[ complete but not unique ]"), the CUB count is calculated astarget_x - widthinstead oftarget_x - (width-1), producing one extra backward movement.Fix: Cap
posxy[0]atself.width - 1when writing reaches the terminal edge. This matches the physical cursor position in pending wrap state, so CUB calculations produce the correct count.Why Terminal.app isn't affected: Terminal.app compensates by subtracting 1 from CUB in pending wrap state. xterm and Ghostty do not compensate, matching the spec more strictly.
Test plan
test_pyrepltests passposxyassignment paths in__write_changed_lineThis contribution was developed with AI assistance (Claude Code).