Skip to content

fix(completer): retain separators for escaped-space directories - #15398

Open
dexhunter wants to merge 1 commit into
ipython:mainfrom
dexhunter:fix/path-completion-directory-separators
Open

dexhunter wants to merge 1 commit into
ipython:mainfrom
dexhunter:fix/path-completion-directory-separators

Conversation

@dexhunter

Copy link
Copy Markdown

Fixes #14516.

Complete directories containing spaces with a trailing /, then allow completion to continue into their children. Directory detection now uses the filesystem path before display escaping, and POSIX CLI argument splitting keeps escaped spaces in the same path token.

Adds a regression test through the prompt-toolkit completion adapter for selecting a spaced directory and continuing into a spaced child directory, with ordinary-directory and file controls. The reproduction uses Linux/POSIX; the original macOS UI and Windows have not been exercised.

🤖🤖 AI-assisted with Codex (GPT-6 Astra and GPT-5.6 Luna).

Baseline and candidate trajectory

Fixes ipython#14516

Keep escaped POSIX CLI paths in one token and classify directories using
filesystem paths before display escaping. Cover completion and traversal
through the prompt-toolkit adapter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tab completion on path with space not working MacOS

1 participant