fix: request completions with "__complete" so a double dash can't run the command - #2397

Draft

suzuki-shunsuke wants to merge 38 commits into

mainfrom

fix/completion-request-marker

Commits on Aug 4, 2026

fix: name the shell completion request so "--" cannot hide it
suzuki-shunsuke
committed
e1e772c

Commits on Aug 6, 2026

fix: answer each completion request on its own terms
suzuki-shunsuke
committed
746f1c9
fix: suggest nothing past a "--" whatever the completion func
suzuki-shunsuke
committed
f002167
fix: build the bash request from the words bash-completion reassembled
suzuki-shunsuke
committed
6646190
fix: take the quotes off the zsh request and drop the command's stderr
suzuki-shunsuke
committed
489d2f1
fix: read the pwsh word being completed from the cursor
suzuki-shunsuke
committed
3974dfa
fix: send the same word from zsh as from bash and PowerShell
suzuki-shunsuke
committed
b32595e
test: run the completion scripts in the shells they are written for
suzuki-shunsuke
committed
582e040
test: drop the -short guard that panicked in the full run
suzuki-shunsuke
committed
2bc0af4
docs: say what "__complete" reserves and what an old script now does
suzuki-shunsuke
committed
288c900
test: cover a disabled app and a nested command
suzuki-shunsuke
committed
a8df3be
test: give the bash driver the words bash actually produces
suzuki-shunsuke
committed
f506b18
ci: install the shells the completion scripts are run in
suzuki-shunsuke
committed
6df5887
fix: take the quotes off the pwsh command name too
suzuki-shunsuke
committed
4383d6e
fix: do not run Before for a request past a "--"
suzuki-shunsuke
committed
e27be3e
docs: say that nothing is completed after a "--"
suzuki-shunsuke
committed
da7275b
ci: require the shells the job installs, not every shell
suzuki-shunsuke
committed
77c2a0c
test: skip bash when its bash-completion cannot be used, and check the required names
suzuki-shunsuke
committed
893749c
test: keep looking when a bash-completion cannot be used
suzuki-shunsuke
committed
dfdb97a
fix: run a bash command typed as "~/bin/app" as the path it stands for
suzuki-shunsuke
committed
4af9af4
docs: say what a script generated before this change actually does
suzuki-shunsuke
committed
24813db
test: complete a quoted command word in every shell
suzuki-shunsuke
committed
8ffc739
docs: drop the claim that the flag it replaces could be renamed
suzuki-shunsuke
committed
36f874b
docs: drop an exception that does not exist
suzuki-shunsuke
committed
a5d0e71
test: leave bash out of a line it never completes
suzuki-shunsuke
committed
f5e5a9e
test: say why bash is left out rather than leaving a field empty
suzuki-shunsuke
committed
725b677
docs: leave what an old script's request runs into to the app
suzuki-shunsuke
committed
2cfa4da
docs: stop the paragraph from denying what it goes on to say
suzuki-shunsuke
committed
53a836b
test: point at the granularity, not at a directory
suzuki-shunsuke
committed
b9686bb
fix: keep the app name out of the bash variable names
suzuki-shunsuke
committed
9b3db57
style: satisfy gofumpt in the syntax check
suzuki-shunsuke
committed
897334f
fix: run a command typed as "~/bin/app" in the other three shells too
suzuki-shunsuke
committed
0c5896f
fix: keep the word being completed when a shell cannot read it back
suzuki-shunsuke
committed
64142e2
test: follow the bash variables to their fixed names
suzuki-shunsuke
committed
3c51951
docs: say what is deprecated about the completion flag
suzuki-shunsuke
committed
0e6f99b
fix: leave the tilde to PowerShell, which already resolves it
suzuki-shunsuke
committed
2f52ee8
test: check the tilde in every shell, not only in bash
suzuki-shunsuke
committed
91f3a39
test: let a required shell fail when it cannot run with HOME moved
suzuki-shunsuke
committed
980383c