fix: Wrong completion context for chained commands in zsh
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 82/100
Direção de pesquisa
Start with lib/src/installer/shell_completion_configuration.dart and CompletionState.fromEnvironment, which the issue identifies as passing and reading the completion context. Check the existing completion tests and how the words passed after completion -- relate to COMP_CWORD. Done means commands after |, && or ; suggest the current command’s options, while existing bash completion still works.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Description
Thank you for this cool lib! I'm facing a minor bug when used with zsh.
In zsh, completing a command that isn't at the start of the command line (after |, && or ;) uses the wrong context. The zsh script passes COMP_LINE="$BUFFER", which is zsh's whole edit buffer, and CompletionState.fromEnvironment reads the arguments from COMP_LINE. So everything typed before the current command is treated as CLI arguments.
Steps To Reproduce
Take a CLI my_cli built on CompletionCommandRunner, with commands foo and bar that have different options, and install its completion in zsh with my_cli install-completion-files.
my_cli foo | my_cli bar <TAB>suggestsfoo's options. The request is parsed asfoo | my_cli bar '', sofoois taken as the command being completed.cd somewhere && my_cli bar <TAB>suggests nothing, so zsh falls back to file names.
Without a shell, this is the request zsh sends for step 1:
ZSH_NAME=zsh COMP_LINE='my_cli foo | my_cli bar ' COMP_POINT=24 COMP_CWORD=2 \
my_cli completion -- my_cli bar ''
It prints foo's options.
Expected Behavior
A command after |, && or ; completes the same way as at the start of the line. Both my_cli foo | my_cli bar <TAB> and cd somewhere && my_cli bar <TAB> suggest bar's options, just like my_cli bar <TAB>. Bash already behaves this way.
Additional Context
Cause
- The zsh template in
lib/src/installer/shell_completion_configuration.dartrunsCOMP_CWORD="$((CURRENT-1))" COMP_LINE="$BUFFER" COMP_POINT="$CURSOR" … completion -- "${words[@]}".$BUFFERis the whole edit buffer, while$wordsand$CURRENTcover only the current command. CompletionState.fromEnvironmenttakes the arguments fromCOMP_LINE(line.trimLeft().split(' ').skip(1)) and ignores the words passed aftercompletion --.- Bash isn't affected, because bash itself limits
COMP_LINEto the current command. With bash 3.2,echo x | mycmd a <TAB>setsCOMP_LINEtomycmd a.
Suggested fix
Build the arguments from the words the scripts already pass after completion --, up to COMP_CWORD, instead of from COMP_LINE. Both scripts pass only the current command's words there, so this fixes zsh without changing the scripts or making users reinstall them.
Workaround
Until this is fixed, a CLI can narrow COMP_LINE itself before the request is handled, in its CompletionCommandRunner subclass:
@override
Future<int?> runCommand(ArgResults topLevelResults) {
final command = topLevelResults.command;
if (command?.name == HandleCompletionRequestCommand.commandName) {
final words = command!.rest; // the words after `completion --`
final environment = Platform.environment;
final cword = int.tryParse(environment['COMP_CWORD'] ?? '');
if (cword != null && cword >= 0 && cword < words.length) {
final line = words.take(cword + 1).join(' ');
environmentOverride = {...environment, 'COMP_LINE': line, 'COMP_POINT': '${line.length}'};
}
}
return super.runCommand(topLevelResults);
}
Environment
- cli_completion 0.6.0
- Dart 3.13.3
- zsh 5.9, macOS (arm64)
- Linguagem predominante
- Dart
- Estrelas
- 54
- Forks
- 6
- Merge médio
- 24min
- PRs com merge (30d)
- 1
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Sem guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de VeryGoodOpenSource/cli_completion
-
feature p3
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 30/100
VeryGoodOpenSource/cli_completion#91 · 2 comentários · 1 reação ·
-
Better shell completionAbertafeature p3
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
VeryGoodOpenSource/cli_completion#90 · 1 comentário · 1 reação ·
-
feat: CLI-user configurable auto installTalvez já em andamento @vgvbot assumiu há 54 dias. Abertafeature p2
VeryGoodOpenSource/cli_completion#55 · 1 comentário · 1 reação · 1 responsável ·
-
feature p2
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
VeryGoodOpenSource/cli_completion#47 · 2 comentários · 2 reações ·
Todas as issues de VeryGoodOpenSource/cli_completion
Issues semelhantes
-
Mobile login: "Sign In" button is squashed to 16 px with its label cut while the keyboard is openAbertaai-driven-qa bug claude
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
linagora/tmail-flutter#4976 ·
Mantenedores costumam responder em até 1 dia
-
P2 package platform-ios team-ios
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
dart-lang/sdk#64568 · 1 comentário ·
Mantenedores costumam responder em até 4 dias
-
discover
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 91/100
DevDogsUGA/DogPack#14 ·
-
direct-code
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
aaronified/fluenough#500 ·
Mantenedores costumam responder em até 1 dia