fix(parser): Don't duplicate requireds when usage disabled

Skipping a test for now because we don't have a mixed-feature testing
strategy.

Fixes #5312

Author: epage

clap_builder/src/parser/validator.rs

@@ -429,6 +429,8 @@ impl<'cmd> Validator<'cmd> {
                             "".to_owned()
                         }
                     })
+                    .collect::<FlatSet<_>>()
+                    .into_iter()
                     .collect::<Vec<_>>()
             }
         };