fix: Prevent getCompletions from modifying os.Args

[jonathan-conder]

Possible fix for #2257.

Tested using this test case:

func TestModifiedArgs(t *testing.T) {
	cmd := &Command{TraverseChildren: true}
	args := []string{"__complete", ""}
	cmd.SetArgs(args)
	if err := cmd.Execute(); err != nil {
		t.Errorf("Unexpected error: %v", err)
	}
	if len(args) != 2 || args[1] != "" {
		t.Error("Modified args")
	}
}

[CLAassistant]

All committers have signed the CLA.

[marckhouzam]

Before going this route, let's see if we can improve pflag with the required API as suggested in #1877