fix(derive)!: Remove value_parser/action defaulted attributes

CHANGELOG.md

@@ -5,6 +5,12 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
 
+## 5.0.0 - Upcoming
+
+### Breaking Changes
+
+- *(derive)* Removed `#[clap(value_parser)]` and `#[clap(action)]` defaulted attributes (its the default) (#3976)
+
 ## 4.0.0 - Upcoming
 
 ### Breaking Changes

clap_complete/examples/completion-derive.rs

@@ -26,34 +26,34 @@ use std::path::PathBuf;
 )]
 struct Opt {
     /// If provided, outputs the completion file for given shell
-    #[clap(long = "generate", arg_enum, value_parser)]
+    #[clap(long = "generate", arg_enum)]
     generator: Option<Shell>,
     // Showcasing all possible ValueHints:
-    #[clap(long, value_hint = ValueHint::Unknown, value_parser)]
+    #[clap(long, value_hint = ValueHint::Unknown)]
     unknown: Option<String>,
-    #[clap(long, value_hint = ValueHint::Other, value_parser)]
+    #[clap(long, value_hint = ValueHint::Other)]
     other: Option<String>,
-    #[clap(short, long, value_hint = ValueHint::AnyPath, value_parser)]
+    #[clap(short, long, value_hint = ValueHint::AnyPath)]
     path: Option<PathBuf>,
-    #[clap(short, long, value_hint = ValueHint::FilePath, value_parser)]
+    #[clap(short, long, value_hint = ValueHint::FilePath)]
     file: Option<PathBuf>,
-    #[clap(short, long, value_hint = ValueHint::DirPath, value_parser)]
+    #[clap(short, long, value_hint = ValueHint::DirPath)]
     dir: Option<PathBuf>,
-    #[clap(short, long, value_hint = ValueHint::ExecutablePath, value_parser)]
+    #[clap(short, long, value_hint = ValueHint::ExecutablePath)]
     exe: Option<PathBuf>,
-    #[clap(long, value_hint = ValueHint::CommandName, value_parser)]
+    #[clap(long, value_hint = ValueHint::CommandName)]
     cmd_name: Option<OsString>,
-    #[clap(short, long, value_hint = ValueHint::CommandString, value_parser)]
+    #[clap(short, long, value_hint = ValueHint::CommandString)]
     cmd: Option<String>,
-    #[clap(value_hint = ValueHint::CommandWithArguments, value_parser)]
+    #[clap(value_hint = ValueHint::CommandWithArguments)]
     command_with_args: Vec<String>,
-    #[clap(short, long, value_hint = ValueHint::Username, value_parser)]
+    #[clap(short, long, value_hint = ValueHint::Username)]
     user: Option<String>,
-    #[clap(short, long, value_hint = ValueHint::Hostname, value_parser)]
+    #[clap(short, long, value_hint = ValueHint::Hostname)]
     host: Option<String>,
-    #[clap(long, value_hint = ValueHint::Url, value_parser)]
+    #[clap(long, value_hint = ValueHint::Url)]
     url: Option<String>,
-    #[clap(long, value_hint = ValueHint::EmailAddress, value_parser)]
+    #[clap(long, value_hint = ValueHint::EmailAddress)]
     email: Option<String>,
 }
 

clap_derive/src/attrs.rs

@@ -429,11 +429,13 @@ impl Attrs {
                     self.push_method(ident, self.name.clone().translate(*self.casing));
                 }
 
+                #[cfg(not(feature = "unstable-v5"))]
                 ValueParser(ident) => {
                     use crate::attrs::ValueParser;
                     self.value_parser = Some(ValueParser::Implicit(ident));
                 }
 
+                #[cfg(not(feature = "unstable-v5"))]
                 Action(ident) => {
                     use crate::attrs::Action;
                     self.action = Some(Action::Implicit(ident));

clap_derive/src/parse.rs

@@ -25,7 +25,9 @@ pub enum ClapAttr {
     // single-identifier attributes
     Short(Ident),
     Long(Ident),
+    #[cfg(not(feature = "unstable-v5"))]
     ValueParser(Ident),
+    #[cfg(not(feature = "unstable-v5"))]
     Action(Ident),
     Env(Ident),
     Flatten(Ident),
@@ -143,7 +145,9 @@ impl Parse for ClapAttr {
             match name_str.as_ref() {
                 "long" => Ok(Long(name)),
                 "short" => Ok(Short(name)),
+                #[cfg(not(feature = "unstable-v5"))]
                 "value_parser" => Ok(ValueParser(name)),
+                #[cfg(not(feature = "unstable-v5"))]
                 "action" => Ok(Action(name)),
                 "env" => Ok(Env(name)),
                 "flatten" => Ok(Flatten(name)),

examples/cargo-example-derive.rs

@@ -10,7 +10,7 @@ enum Cargo {
 #[derive(clap::Args)]
 #[clap(author, version, about, long_about = None)]
 struct ExampleDerive {
-    #[clap(long, value_parser)]
+    #[clap(long)]
     manifest_path: Option<std::path::PathBuf>,
 }
 

examples/demo.rs

@@ -5,11 +5,11 @@ use clap::Parser;
 #[clap(author, version, about, long_about = None)]
 struct Args {
     /// Name of the person to greet
-    #[clap(short, long, value_parser)]
+    #[clap(short, long)]
     name: String,
 
     /// Number of times to greet
-    #[clap(short, long, value_parser, default_value_t = 1)]
+    #[clap(short, long, default_value_t = 1)]
     count: u8,
 }
 

examples/derive_ref/augment_args.rs

@@ -2,7 +2,7 @@ use clap::{arg, Args, Command, FromArgMatches as _};
 
 #[derive(Args, Debug)]
 struct DerivedArgs {
-    #[clap(short, long, action)]
+    #[clap(short, long)]
     derived: bool,
 }
 

examples/derive_ref/augment_subcommands.rs

@@ -3,7 +3,7 @@ use clap::{Command, FromArgMatches as _, Parser, Subcommand as _};
 #[derive(Parser, Debug)]
 enum Subcommands {
     Derived {
-        #[clap(short, long, action)]
+        #[clap(short, long)]
         derived_flag: bool,
     },
 }

examples/derive_ref/flatten_hand_args.rs

@@ -69,7 +69,7 @@ impl Args for CliArgs {
 
 #[derive(Parser, Debug)]
 struct Cli {
-    #[clap(short, long, action)]
+    #[clap(short, long)]
     top_level: bool,
     #[clap(flatten)]
     more_args: CliArgs,

examples/derive_ref/hand_subcommand.rs

@@ -3,14 +3,12 @@ use clap::{ArgMatches, Args as _, Command, FromArgMatches, Parser, Subcommand};
 
 #[derive(Parser, Debug)]
 struct AddArgs {
-    #[clap(value_parser)]
     name: Vec<String>,
 }
 #[derive(Parser, Debug)]
 struct RemoveArgs {
-    #[clap(short, long, action)]
+    #[clap(short, long)]
     force: bool,
-    #[clap(value_parser)]
     name: Vec<String>,
 }
 
@@ -69,7 +67,7 @@ impl Subcommand for CliSub {
 
 #[derive(Parser, Debug)]
 struct Cli {
-    #[clap(short, long, action)]
+    #[clap(short, long)]
     top_level: bool,
     #[clap(subcommand)]
     subcommand: CliSub,

examples/escaped-positional-derive.rs

@@ -3,13 +3,13 @@ use clap::Parser;
 #[derive(Parser)] // requires `derive` feature
 #[clap(author, version, about, long_about = None)]
 struct Cli {
-    #[clap(short = 'f', action)]
+    #[clap(short = 'f')]
     eff: bool,
 
-    #[clap(short = 'p', value_name = "PEAR", value_parser)]
+    #[clap(short = 'p', value_name = "PEAR")]
     pea: Option<String>,
 
-    #[clap(last = true, value_parser)]
+    #[clap(last = true)]
     slop: Vec<String>,
 }
 

examples/git-derive.rs

@@ -18,21 +18,19 @@ enum Commands {
     #[clap(arg_required_else_help = true)]
     Clone {
         /// The remote to clone
-        #[clap(value_parser)]
         remote: String,
     },
     /// pushes things
     #[clap(arg_required_else_help = true)]
     Push {
         /// The remote to target
-        #[clap(value_parser)]
         remote: String,
     },
     /// adds things
     #[clap(arg_required_else_help = true)]
     Add {
         /// Stuff to add
-        #[clap(required = true, value_parser)]
+        #[clap(required = true)]
         path: Vec<PathBuf>,
     },
     Stash(Stash),
@@ -53,19 +51,13 @@ struct Stash {
 #[derive(Debug, Subcommand)]
 enum StashCommands {
     Push(StashPush),
-    Pop {
-        #[clap(value_parser)]
-        stash: Option<String>,
-    },
-    Apply {
-        #[clap(value_parser)]
-        stash: Option<String>,
-    },
+    Pop { stash: Option<String> },
+    Apply { stash: Option<String> },
 }
 
 #[derive(Debug, Args)]
 struct StashPush {
-    #[clap(short, long, value_parser)]
+    #[clap(short, long)]
     message: Option<String>,
 }
 

examples/tutorial_derive/01_quick.rs

@@ -6,11 +6,10 @@ use clap::{Parser, Subcommand};
 #[clap(author, version, about, long_about = None)]
 struct Cli {
     /// Optional name to operate on
-    #[clap(value_parser)]
     name: Option<String>,
 
     /// Sets a custom config file
-    #[clap(short, long, value_parser, value_name = "FILE")]
+    #[clap(short, long, value_name = "FILE")]
     config: Option<PathBuf>,
 
     /// Turn debugging information on
@@ -26,7 +25,7 @@ enum Commands {
     /// does testing things
     Test {
         /// lists test values
-        #[clap(short, long, action)]
+        #[clap(short, long)]
         list: bool,
     },
 }

examples/tutorial_derive/02_app_settings.rs

@@ -4,9 +4,9 @@ use clap::Parser;
 #[clap(author, version, about, long_about = None)]
 #[clap(allow_negative_numbers = true)]
 struct Cli {
-    #[clap(long, value_parser)]
+    #[clap(long)]
     two: String,
-    #[clap(long, value_parser)]
+    #[clap(long)]
     one: String,
 }
 

examples/tutorial_derive/02_apps.rs

@@ -6,9 +6,9 @@ use clap::Parser;
 #[clap(version = "1.0")]
 #[clap(about = "Does awesome things", long_about = None)]
 struct Cli {
-    #[clap(long, value_parser)]
+    #[clap(long)]
     two: String,
-    #[clap(long, value_parser)]
+    #[clap(long)]
     one: String,
 }
 

examples/tutorial_derive/02_crate.rs

@@ -3,9 +3,9 @@ use clap::Parser;
 #[derive(Parser)]
 #[clap(author, version, about, long_about = None)] // Read from `Cargo.toml`
 struct Cli {
-    #[clap(long, value_parser)]
+    #[clap(long)]
     two: String,
-    #[clap(long, value_parser)]
+    #[clap(long)]
     one: String,
 }
 

examples/tutorial_derive/03_01_flag_bool.rs

@@ -3,7 +3,7 @@ use clap::Parser;
 #[derive(Parser)]
 #[clap(author, version, about, long_about = None)]
 struct Cli {
-    #[clap(short, long, action)]
+    #[clap(short, long)]
     verbose: bool,
 }
 

examples/tutorial_derive/03_02_option.rs

@@ -3,7 +3,7 @@ use clap::Parser;
 #[derive(Parser)]
 #[clap(author, version, about, long_about = None)]
 struct Cli {
-    #[clap(short, long, value_parser)]
+    #[clap(short, long)]
     name: Option<String>,
 }
 

examples/tutorial_derive/03_03_positional.rs

@@ -3,7 +3,6 @@ use clap::Parser;
 #[derive(Parser)]
 #[clap(author, version, about, long_about = None)]
 struct Cli {
-    #[clap(value_parser)]
     name: Option<String>,
 }
 

examples/tutorial_derive/03_04_subcommands.rs

@@ -11,10 +11,7 @@ struct Cli {
 #[derive(Subcommand)]
 enum Commands {
     /// Adds files to myapp
-    Add {
-        #[clap(value_parser)]
-        name: Option<String>,
-    },
+    Add { name: Option<String> },
 }
 
 fn main() {

examples/tutorial_derive/03_04_subcommands_alt.rs

@@ -16,7 +16,6 @@ enum Commands {
 
 #[derive(Args)]
 struct Add {
-    #[clap(value_parser)]
     name: Option<String>,
 }
 

examples/tutorial_derive/03_05_default_values.rs

@@ -3,7 +3,7 @@ use clap::Parser;
 #[derive(Parser)]
 #[clap(author, version, about, long_about = None)]
 struct Cli {
-    #[clap(default_value_t = String::from("alice"), value_parser)]
+    #[clap(default_value_t = String::from("alice"))]
     name: String,
 }
 

examples/tutorial_derive/04_01_enum.rs

@@ -4,7 +4,7 @@ use clap::{ArgEnum, Parser};
 #[clap(author, version, about, long_about = None)]
 struct Cli {
     /// What mode to run the program in
-    #[clap(arg_enum, value_parser)]
+    #[clap(arg_enum)]
     mode: Mode,
 }
 

examples/tutorial_derive/04_03_relations.rs

@@ -9,30 +9,30 @@ use clap::{ArgGroup, Parser};
         ))]
 struct Cli {
     /// set version manually
-    #[clap(long, value_name = "VER", value_parser)]
+    #[clap(long, value_name = "VER")]
     set_ver: Option<String>,
 
     /// auto inc major
-    #[clap(long, action)]
+    #[clap(long)]
     major: bool,
 
     /// auto inc minor
-    #[clap(long, action)]
+    #[clap(long)]
     minor: bool,
 
     /// auto inc patch
-    #[clap(long, action)]
+    #[clap(long)]
     patch: bool,
 
     /// some regular input
-    #[clap(group = "input", value_parser)]
+    #[clap(group = "input")]
     input_file: Option<String>,
 
     /// some special input argument
-    #[clap(long, group = "input", value_parser)]
+    #[clap(long, group = "input")]
     spec_in: Option<String>,
 
-    #[clap(short, requires = "input", value_parser)]
+    #[clap(short, requires = "input")]
     config: Option<String>,
 }