fix: correct flag handling in commands run by asdf exec (#2115)

Author: Stratus3D

internal/cli/cli.go

@@ -104,6 +104,10 @@ func Execute(version string) {
 			},
 			{
 				Name: "exec",
+				// We want all arguments to exec to remain unparsed so we can pass them
+				// directly to the command asdf whill exec on behalf of the shim/user.
+				// SkipFlagParsing tells urfave/cli to do this.
+				SkipFlagParsing: true,
 				Action: func(_ context.Context, cmd *cli.Command) error {
 					command := cmd.Args().Get(0)
 					args := cmd.Args().Slice()

test/shim_exec.bats

@@ -34,6 +34,15 @@ teardown() {
   [ "$status" -eq 0 ]
 }
 
+@test "asdf exec should pass all arguments including flags to executable" {
+  echo "dummy 1.0" >"$PROJECT_DIR/.tool-versions"
+  run asdf install
+
+  run asdf exec dummy --mytestflag hello
+  [ "$output" = "This is Dummy 1.0! hello --mytestflag" ]
+  [ "$status" -eq 0 ]
+}
+
 @test "asdf exec should pass all arguments to executable even if shim is not in PATH" {
   echo "dummy 1.0" >"$PROJECT_DIR/.tool-versions"
   run asdf install