1use std::process::Command;
2
3use anyhow::{bail, Context, Result};
4use clap::{Parser, Subcommand};
5
6#[derive(Parser)]
7#[command(name = "cargo xtask")]
8struct Args {
9 #[command(subcommand)]
10 command: CliCommand,
11}
12
13#[derive(Subcommand)]
14enum CliCommand {
15 /// Runs `cargo clippy`.
16 Clippy(ClippyArgs),
17}
18
19fn main() -> Result<()> {
20 let args = Args::parse();
21
22 match args.command {
23 CliCommand::Clippy(args) => run_clippy(args),
24 }
25}
26
27#[derive(Parser)]
28struct ClippyArgs {
29 /// Automatically apply lint suggestions (`clippy --fix`).
30 #[arg(long)]
31 fix: bool,
32
33 /// The package to run Clippy against (`cargo -p <PACKAGE> clippy`).
34 #[arg(long, short)]
35 package: Option<String>,
36}
37
38fn run_clippy(args: ClippyArgs) -> Result<()> {
39 let cargo = std::env::var("CARGO").unwrap_or_else(|_| "cargo".to_string());
40
41 let mut clippy_command = Command::new(&cargo);
42 clippy_command.arg("clippy");
43
44 if let Some(package) = args.package.as_ref() {
45 clippy_command.args(["--package", package]);
46 } else {
47 clippy_command.arg("--workspace");
48 }
49
50 clippy_command
51 .arg("--release")
52 .arg("--all-targets")
53 .arg("--all-features");
54
55 if args.fix {
56 clippy_command.arg("--fix");
57 }
58
59 clippy_command.arg("--");
60
61 // Deny all warnings.
62 // We don't do this yet on Windows, as it still has some warnings present.
63 #[cfg(not(target_os = "windows"))]
64 clippy_command.args(["--deny", "warnings"]);
65
66 /// These are all of the rules that currently have violations in the Zed
67 /// codebase.
68 ///
69 /// We'll want to drive this list down by either:
70 /// 1. fixing violations of the rule and begin enforcing it
71 /// 2. deciding we want to allow the rule permanently, at which point
72 /// we should codify that separately in this task.
73 ///
74 /// This list shouldn't be added to; it should only get shorter.
75 const MIGRATORY_RULES_TO_ALLOW: &[&str] = &[
76 // There are a bunch of rules currently failing in the `style` group, so
77 // allow all of those, for now.
78 "clippy::style",
79 // Individual rules that have violations in the codebase:
80 "clippy::almost_complete_range",
81 "clippy::arc_with_non_send_sync",
82 "clippy::await_holding_lock",
83 "clippy::borrow_deref_ref",
84 "clippy::borrowed_box",
85 "clippy::cast_abs_to_unsigned",
86 "clippy::cmp_owned",
87 "clippy::derive_ord_xor_partial_ord",
88 "clippy::eq_op",
89 "clippy::implied_bounds_in_impls",
90 "clippy::let_underscore_future",
91 "clippy::map_entry",
92 "clippy::never_loop",
93 "clippy::non_canonical_clone_impl",
94 "clippy::non_canonical_partial_ord_impl",
95 "clippy::reversed_empty_ranges",
96 "clippy::single_range_in_vec_init",
97 "clippy::suspicious_to_owned",
98 "clippy::type_complexity",
99 "clippy::unnecessary_to_owned",
100 ];
101
102 // When fixing violations automatically for a single package we don't care
103 // about the rules we're already violating, since it may be possible to
104 // have them fixed automatically.
105 let ignore_suppressed_rules = args.fix && args.package.is_some();
106 if !ignore_suppressed_rules {
107 for rule in MIGRATORY_RULES_TO_ALLOW {
108 clippy_command.args(["--allow", rule]);
109 }
110 }
111
112 // Deny `dbg!` and `todo!`s.
113 clippy_command
114 .args(["--deny", "clippy::dbg_macro"])
115 .args(["--deny", "clippy::todo"]);
116
117 eprintln!(
118 "running: {cargo} {}",
119 clippy_command
120 .get_args()
121 .map(|arg| arg.to_str().unwrap())
122 .collect::<Vec<_>>()
123 .join(" ")
124 );
125
126 let exit_status = clippy_command
127 .spawn()
128 .context("failed to spawn child process")?
129 .wait()
130 .context("failed to wait for child process")?;
131
132 if !exit_status.success() {
133 bail!("clippy failed: {}", exit_status);
134 }
135
136 Ok(())
137}