[vidigoat]

What type of PR is this?

• bug

What this PR does / why we need it:

TakesFile is a field of every flag type (FlagBase), and its only reader is the fish completion script, but fishAddFileFlag only checks *StringFlag and *StringSliceFlag. Any other flag with TakesFile: true still gets -f, so fish completes no files for it:

&cli.StringFlag{Name: "config", TakesFile: true},
&cli.GenericFlag{Name: "input", Value: &p, TakesFile: true},
&cli.StringMapFlag{Name: "env-file", TakesFile: true},

complete -c app -n '__fish_app_no_subcommand' -l config -r ...
complete -c app -n '__fish_app_no_subcommand' -f -l input -r ...
complete -c app -n '__fish_app_no_subcommand' -f -l env-file -r ...

In fish 4.9.3, complete -C "app --config " lists the files in the directory, while complete -C "app --input " lists nothing. v2's fishAddFileFlag also handled GenericFlag (and PathFlag, which v3 replaced with StringFlag + TakesFile).

FlagBase now has an unexported takesFile() (TakesFile and the flag takes a value), and fishAddFileFlag checks for it instead of the two concrete types. So every flag type works, there is no API change (make v3diff is clean), and a BoolFlag with TakesFile keeps -f, as it does now, since it has no value to complete.

Which issue(s) this PR fixes:

None, found while testing the generated fish script in fish.

Testing

• New TestFishCompletionTakesFileForAllFlagTypes (StringFlag, GenericFlag, StringMapFlag, a plain StringFlag and a BoolFlag). It fails on main.
• Checked the generated script in fish 4.9.3: --input and --env-file now complete files like --config.
• make (generate, vet, tests, binary size, gfmrun), make lint and make v3diff pass. go test -race passes, as do the tests with urfave_cli_no_template, urfave_cli_no_docs and urfave_cli_no_suggest.

The change and the test were written with Claude Code (AI).

Release Notes

TakesFile now enables file completion in the fish script for every flag type, not only StringFlag and StringSliceFlag.

🤖 Generated with Claude Code