fix: skip all hidden flags in shell completion, not just bool flags

help.go

@@ -216,7 +216,7 @@ func printFlagSuggestions(lastArg string, flags []Flag, writer io.Writer) {
 	// Trim to handle both "-short" and "--long" flags.
 	cur := strings.TrimLeft(lastArg, "-")
 	for _, flag := range flags {
-		if bflag, ok := flag.(*BoolFlag); ok && bflag.Hidden {
+		if vf, ok := flag.(VisibleFlag); ok && !vf.IsVisible() {
 			continue
 		}
 

help_test.go

@@ -1353,6 +1353,28 @@ func TestDefaultCompleteWithFlags(t *testing.T) {
 			env:      map[string]string{"SHELL": "bash"},
 			expected: "",
 		},
+		{
+			name: "typical-flag-suggestion-hidden-non-bool",
+			cmd: &Command{
+				Flags: []Flag{
+					&StringFlag{Name: "excellent", Hidden: true},
+					&StringFlag{Name: "excitement"},
+				},
+				parent: &Command{
+					Name: "cmd",
+					Flags: []Flag{
+						&BoolFlag{Name: "happiness"},
+						&Int64Flag{Name: "everybody-jump-on"},
+					},
+					Commands: []*Command{
+						{Name: "putz"},
+					},
+				},
+			},
+			argv:     []string{"cmd", "--e", completionFlag},
+			env:      map[string]string{"SHELL": "bash"},
+			expected: "--excitement\n",
+		},
 		{
 			name: "flag-suggestion-double-dash-shows-all-flags",
 			cmd: &Command{