Skip to content

Commit de05740

Browse files
committed
fix(fish-completions): don't execute with env
Using `env` broke Fish completions because `command` is a Fish builtin while `env` is an external program unaware of what Fish does and doesn't provide: env: ‘command’: No such file or directory Fish supports `FIELD=val` natively, so we can just remove `env`. Assisted-by: Claude Sonnet 4.5 via Crush <[email protected]>
1 parent 5f7387d commit de05740

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

cmd/wtp/completion_config.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ function __fish_wtp_dynamic_complete --description 'wtp dynamic completion helpe
117117
return
118118
end
119119
120-
set -l raw (env WTP_SHELL_COMPLETION=1 command wtp $args)
120+
set -l raw (WTP_SHELL_COMPLETION=1 command wtp $args)
121121
for line in $raw
122122
if test -z "$line"
123123
continue

cmd/wtp/testdata/completion/fish_expected.fish

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ function __fish_wtp_dynamic_complete --description 'wtp dynamic completion helpe
2222
return
2323
end
2424

25-
set -l raw (env WTP_SHELL_COMPLETION=1 command wtp $args)
25+
set -l raw (WTP_SHELL_COMPLETION=1 command wtp $args)
2626
for line in $raw
2727
if test -z "$line"
2828
continue

0 commit comments

Comments
 (0)