Moved cli frontend of the cairo linter to scarb command

Author: wawel37

Cargo.lock

@@ -47,6 +47,7 @@ repository = "https://github.com/software-mansion/scarb"
 # To keep our Cargo.toml following this contract, always use `cargo xtask upgrade`
 # for manipulating these dependencies.
 [workspace.dependencies]
+annotate-snippets = "0.11.5"
 anyhow = "1"
 assert_fs = "1"
 async-trait = "0.1"
@@ -71,6 +72,7 @@ cairo-lang-test-plugin = "*"
 cairo-lang-test-runner = "*"
 cairo-lang-utils = { version = "*", features = ["env_logger"] }
 cairo-language-server = "*"
+cairo-lint-core = {git = "https://github.com/software-mansion/cairo-lint", rev = "3cd07f36a6d509c8844284dd88a17535b6b1e265"}
 camino = { version = "1", features = ["serde1"] }
 cargo_metadata = ">=0.18"
 clap = { version = "4", features = ["derive", "env", "string"] }

Cargo.toml

@@ -14,6 +14,7 @@ readme.workspace = true
 repository.workspace = true
 
 [dependencies]
+annotate-snippets.workspace = true
 anyhow.workspace = true
 async-trait.workspace = true
 cairo-lang-compiler.workspace = true
@@ -34,6 +35,7 @@ cairo-lang-starknet.workspace = true
 cairo-lang-syntax.workspace = true
 cairo-lang-test-plugin.workspace = true
 cairo-lang-utils.workspace = true
+cairo-lint-core.workspace = true
 camino.workspace = true
 clap.workspace = true
 convert_case.workspace = true

scarb/Cargo.toml

@@ -187,6 +187,8 @@ pub enum Command {
         to a registry.
     ")]
     Publish(PublishArgs),
+    /// Run lint checker.
+    Lint(LintArgs),
     /// Run arbitrary package scripts.
     Run(ScriptsRunnerArgs),
     /// Execute all unit and integration tests of a local package.
@@ -531,6 +533,27 @@ pub struct PublishArgs {
     pub ignore_cairo_version: bool,
 }
 
+#[derive(Parser, Clone, Debug)]
+pub struct LintArgs {
+    /// Name of the package.
+    #[command(flatten)]
+    pub packages_filter: PackagesFilter,
+    /// Path to the file or project to analyze
+    pub path: Option<String>,
+    /// Logging verbosity.
+    #[command(flatten)]
+    pub verbose: VerbositySpec,
+    /// Comma separated list of target names to compile.
+    #[arg(long, value_delimiter = ',', env = "SCARB_TARGET_NAMES")]
+    pub target_names: Vec<String>,
+    /// Should lint the tests.
+    #[arg(short, long, default_value_t = false)]
+    pub test: bool,
+    /// Should fix the lint when it can.
+    #[arg(short, long, default_value_t = false)]
+    pub fix: bool,
+}
+
 /// Git reference specification arguments.
 #[derive(Parser, Clone, Debug)]
 #[group(requires = "git", multiple = false)]

scarb/src/bin/scarb/args.rs

@@ -0,0 +1,24 @@
+use scarb::ops::{self, LintOptions};
+
+use crate::args::LintArgs;
+use anyhow::Result;
+use scarb::core::Config;
+
+#[tracing::instrument(skip_all, level = "info")]
+pub fn run(args: LintArgs, config: &Config) -> Result<()> {
+    let ws = ops::read_workspace(config.manifest_path(), config)?;
+    let packages = args
+        .packages_filter
+        .match_many(&ws)?
+        .into_iter()
+        .collect::<Vec<_>>();
+
+    ops::lint(
+        LintOptions {
+            packages,
+            test: args.test,
+            fix: args.fix,
+        },
+        &ws,
+    )
+}

scarb/src/bin/scarb/commands/lint.rs

@@ -18,6 +18,7 @@ pub mod external;
 pub mod fetch;
 pub mod fmt;
 pub mod init;
+mod lint;
 pub mod manifest_path;
 pub mod metadata;
 pub mod new;
@@ -52,6 +53,7 @@ pub fn run(command: Command, config: &mut Config) -> Result<()> {
         Package(args) => package::run(args, config),
         ProcMacroServer => proc_macro_server::run(config),
         Publish(args) => publish::run(args, config),
+        Lint(args) => lint::run(args, config),
         Remove(args) => remove::run(args, config),
         Run(args) => run::run(args, config),
         Test(args) => test::run(args, config),

scarb/src/bin/scarb/commands/mod.rs

@@ -12,6 +12,7 @@ use cairo_lang_filesystem::db::{
     AsFilesGroupMut, CrateIdentifier, CrateSettings, DependencySettings, FilesGroup, FilesGroupEx,
 };
 use cairo_lang_filesystem::ids::CrateLongId;
+use cairo_lang_semantic::plugin::PluginSuite;
 use cairo_lang_utils::ordered_hash_map::OrderedHashMap;
 use smol_str::SmolStr;
 use std::path::PathBuf;
@@ -26,12 +27,13 @@ pub struct ScarbDatabase {
 pub(crate) fn build_scarb_root_database(
     unit: &CairoCompilationUnit,
     ws: &Workspace<'_>,
+    additional_plugins: Vec<PluginSuite>,
 ) -> Result<ScarbDatabase> {
     let mut b = RootDatabase::builder();
     b.with_project_config(build_project_config(unit)?);
     b.with_cfg(unit.cfg_set.clone());
     b.with_inlining_strategy(unit.compiler_config.inlining_strategy.clone().into());
-    let proc_macro_host = load_plugins(unit, ws, &mut b)?;
+    let proc_macro_host = load_plugins(unit, ws, &mut b, additional_plugins)?;
     if !unit.compiler_config.enable_gas {
         b.skip_auto_withdraw_gas();
     }
@@ -50,6 +52,7 @@ fn load_plugins(
     unit: &CairoCompilationUnit,
     ws: &Workspace<'_>,
     builder: &mut RootDatabaseBuilder,
+    additional_plugins: Vec<PluginSuite>,
 ) -> Result<Arc<ProcMacroHostPlugin>> {
     let mut proc_macros = ProcMacroHost::default();
     for plugin_info in &unit.cairo_plugins {
@@ -64,6 +67,9 @@ fn load_plugins(
             proc_macros.register_new(plugin_info.package.clone(), ws.config())?;
         }
     }
+    for plugin in additional_plugins {
+        builder.with_plugin_suite(plugin);
+    }
     let macro_host = Arc::new(proc_macros.into_plugin()?);
     builder.with_plugin_suite(ProcMacroHostPlugin::build_plugin_suite(macro_host.clone()));
     Ok(macro_host)

scarb/src/compiler/db.rs

@@ -219,7 +219,7 @@ fn compile_unit_inner(unit: CompilationUnit, ws: &Workspace<'_>) -> Result<()> {
             let ScarbDatabase {
                 mut db,
                 proc_macro_host,
-            } = build_scarb_root_database(&unit, ws)?;
+            } = build_scarb_root_database(&unit, ws, vec![])?;
             check_starknet_dependency(&unit, ws, &db, &package_name);
             let result = ws.config().compilers().compile(unit, &mut db, ws);
             proc_macro_host
@@ -284,7 +284,7 @@ fn check_unit(unit: CompilationUnit, ws: &Workspace<'_>) -> Result<()> {
     let result = match unit {
         CompilationUnit::ProcMacro(unit) => proc_macro::check_unit(unit, ws),
         CompilationUnit::Cairo(unit) => {
-            let ScarbDatabase { db, .. } = build_scarb_root_database(&unit, ws)?;
+            let ScarbDatabase { db, .. } = build_scarb_root_database(&unit, ws, vec![])?;
             let main_crate_ids = collect_main_crate_ids(&unit, &db);
             check_starknet_dependency(&unit, ws, &db, &package_name);
             let mut compiler_config = build_compiler_config(&db, &unit, &main_crate_ids, ws);

scarb/src/ops/compile.rs

@@ -168,7 +168,7 @@ fn do_expand(
     opts: ExpandOpts,
     ws: &Workspace<'_>,
 ) -> Result<()> {
-    let ScarbDatabase { db, .. } = build_scarb_root_database(compilation_unit, ws)?;
+    let ScarbDatabase { db, .. } = build_scarb_root_database(compilation_unit, ws, vec![])?;
     let name = compilation_unit.main_component().cairo_package_name();
     let main_crate_id = db.intern_crate(CrateLongId::Real {
         name,