actionFlags: multipart completion for flags as default

example/cmd/root_test.go

@@ -98,33 +98,33 @@ func TestRoot(t *testing.T) {
 				"-p", "Help message for persistentFlag",
 				"--toggle", "Help message for toggle",
 				"-t", "Help message for toggle",
-			).Tag("flags"))
+			).NoSpace('.').Tag("flags"))
 
 		s.Run("--").
 			Expect(carapace.ActionValuesDescribed(
 				"--array", "multiflag",
 				"--persistentFlag", "Help message for persistentFlag",
 				"--toggle", "Help message for toggle",
-			).Tag("flags"))
+			).NoSpace('.').Tag("flags"))
 
 		s.Run("--a").
 			Expect(carapace.ActionValuesDescribed(
 				"--array", "multiflag",
-			).Tag("flags"))
+			).NoSpace('.').Tag("flags"))
 
 		s.Run("--array").
 			Expect(carapace.ActionValuesDescribed(
 				"--array", "multiflag",
-			).Tag("flags"))
+			).NoSpace('.').Tag("flags"))
 
 		s.Run("--array", "", "--a").
 			Expect(carapace.ActionValuesDescribed(
 				"--array", "multiflag",
-			).Tag("flags"))
+			).NoSpace('.').Tag("flags"))
 
 		s.Run("-a", "", "--a").
 			Expect(carapace.ActionValuesDescribed(
 				"--array", "multiflag",
-			).Tag("flags"))
+			).NoSpace('.').Tag("flags"))
 	})
 }

internalActions.go

@@ -115,12 +115,7 @@ func actionFlags(cmd *cobra.Command) Action {
 		if isShorthandSeries {
 			return ActionValuesDescribed(vals...).Invoke(c).Prefix(c.CallbackValue).ToA().NoSpace('*')
 		}
-		for i := 0; i < len(vals); i = i + 2 { // TODO experimental - hardcoded multiparts completion if flags are "grouped" with `.`
-			if strings.Contains(vals[i], ".") {
-				return ActionValuesDescribed(vals...).Invoke(c).ToMultiPartsA(".")
-			}
-		}
-		return ActionValuesDescribed(vals...)
+		return ActionValuesDescribed(vals...).Invoke(c).ToMultiPartsA(".") // multiparts completion for flags grouped with `.`
 	}).Tag("flags")
 }