fix(linter): do not output number of rules with nested configs

apps/oxlint/src/lint.rs

@@ -60,6 +60,11 @@ impl Runner for LintRunner {
             ..
         } = self.options;
 
+        let use_nested_config = experimental_nested_config &&
+            // If the `--config` option is explicitly passed, we should not search for nested config files
+            // as the passed config file takes absolute precedence.
+            basic_options.config.is_none();
+
         let mut paths = paths;
         let provided_path_count = paths.len();
         let now = Instant::now();
@@ -151,7 +156,7 @@ impl Runner for LintRunner {
             if provided_path_count > 0 {
                 if let Some(end) = output_formatter.lint_command_info(&LintCommandInfo {
                     number_of_files: 0,
-                    number_of_rules: 0,
+                    number_of_rules: None,
                     threads_count: rayon::current_num_threads(),
                     start_time: now.elapsed(),
                 }) {
@@ -175,12 +180,7 @@ impl Runner for LintRunner {
         let mut nested_oxlintrc = FxHashMap::<&Path, Oxlintrc>::default();
         let mut nested_configs = FxHashMap::<PathBuf, ConfigStore>::default();
 
-        let use_nested_config =
-            // If the `--config` option is explicitly passed, we should not search for nested config files
-            // as the passed config file takes absolute precedence.
-            basic_options.config.is_none();
-
-        if experimental_nested_config && use_nested_config {
+        if use_nested_config {
             // get all of the unique directories among the paths to use for search for
             // oxlint config files in those directories
             // e.g. `/some/file.js` and `/some/other/file.js` would both result in `/some`

apps/oxlint/src/output_formatter/default.rs

@@ -27,12 +27,17 @@ impl InternalFormatter for DefaultOutputFormatter {
         let time = Self::get_execution_time(&lint_command_info.start_time);
         let s = if lint_command_info.number_of_files == 1 { "" } else { "s" };
 
-        Some(format!(
-            "Finished in {time} on {} file{s} with {} rules using {} threads.\n",
-            lint_command_info.number_of_files,
-            lint_command_info.number_of_rules,
-            lint_command_info.threads_count
-        ))
+        if let Some(number_of_rules) = lint_command_info.number_of_rules {
+            Some(format!(
+                "Finished in {time} on {} file{s} with {} rules using {} threads.\n",
+                lint_command_info.number_of_files, number_of_rules, lint_command_info.threads_count
+            ))
+        } else {
+            Some(format!(
+                "Finished in {time} on {} file{s} using {} threads.\n",
+                lint_command_info.number_of_files, lint_command_info.threads_count
+            ))
+        }
     }
 
     #[cfg(not(test))]
@@ -169,7 +174,7 @@ mod test {
         let formatter = DefaultOutputFormatter;
         let result = formatter.lint_command_info(&LintCommandInfo {
             number_of_files: 5,
-            number_of_rules: 10,
+            number_of_rules: Some(10),
             threads_count: 12,
             start_time: Duration::new(1, 0),
         });
@@ -181,6 +186,20 @@ mod test {
         );
     }
 
+    #[test]
+    fn lint_command_info_unknown_rules() {
+        let formatter = DefaultOutputFormatter;
+        let result = formatter.lint_command_info(&LintCommandInfo {
+            number_of_files: 5,
+            number_of_rules: None,
+            threads_count: 12,
+            start_time: Duration::new(1, 0),
+        });
+
+        assert!(result.is_some());
+        assert_eq!(result.unwrap(), "Finished in 1.0s on 5 files using 12 threads.\n");
+    }
+
     #[test]
     fn reporter_finish_no_results() {
         let mut reporter = GraphicalReporter::default();

apps/oxlint/src/output_formatter/mod.rs

@@ -55,8 +55,9 @@ impl FromStr for OutputFormat {
 pub struct LintCommandInfo {
     /// The number of files that were linted.
     pub number_of_files: usize,
-    /// The number of lint rules that were run.
-    pub number_of_rules: usize,
+    /// The number of lint rules that were run. If the number varies and can't be clearly
+    /// computed, then this defaults to None.
+    pub number_of_rules: Option<usize>,
     /// The used CPU threads count
     pub threads_count: usize,
     /// Some reporters want to output the duration it took to finished the task

apps/oxlint/src/snapshots/_--ignore-path fixtures__issue_7566__.oxlintignore fixtures__issue_7566__tests__main.js fixtures__issue_7566__tests__function__main.js@oxlint.snap

@@ -5,7 +5,7 @@ source: apps/oxlint/src/tester.rs
 arguments: --ignore-path fixtures/issue_7566/.oxlintignore fixtures/issue_7566/tests/main.js fixtures/issue_7566/tests/function/main.js
 working directory: 
 ----------
-Finished in <variable>ms on 0 files with 0 rules using 1 threads.
+Finished in <variable>ms on 0 files using 1 threads.
 ----------
 CLI result: LintNoFilesFound
 ----------

apps/oxlint/src/snapshots/_--ignore-path fixtures__linter__.customignore fixtures__linter__nan.js@oxlint.snap

@@ -5,7 +5,7 @@ source: apps/oxlint/src/tester.rs
 arguments: --ignore-path fixtures/linter/.customignore fixtures/linter/nan.js
 working directory: 
 ----------
-Finished in <variable>ms on 0 files with 0 rules using 1 threads.
+Finished in <variable>ms on 0 files using 1 threads.
 ----------
 CLI result: LintNoFilesFound
 ----------

apps/oxlint/src/snapshots/_-c fixtures__config_ignore_patterns__ignore_extension__eslintrc.json fixtures__config_ignore_patterns__ignore_extension__main.js@oxlint.snap

@@ -5,7 +5,7 @@ source: apps/oxlint/src/tester.rs
 arguments: -c fixtures/config_ignore_patterns/ignore_extension/eslintrc.json fixtures/config_ignore_patterns/ignore_extension/main.js
 working directory: 
 ----------
-Finished in <variable>ms on 0 files with 0 rules using 1 threads.
+Finished in <variable>ms on 0 files using 1 threads.
 ----------
 CLI result: LintNoFilesFound
 ----------

apps/oxlint/src/snapshots/fixtures__extends_config_--experimental-nested-config overrides@oxlint.snap

@@ -30,7 +30,7 @@ working directory: fixtures/extends_config
    `----
 
 Found 0 warnings and 3 errors.
-Finished in <variable>ms on 2 files with 100 rules using 1 threads.
+Finished in <variable>ms on 2 files using 1 threads.
 ----------
 CLI result: LintFoundErrors
 ----------

apps/oxlint/src/snapshots/fixtures__extends_config_--experimental-nested-config overrides_same_directory@oxlint.snap

@@ -14,7 +14,7 @@ working directory: fixtures/extends_config
   help: Delete this code.
 
 Found 1 warning and 0 errors.
-Finished in <variable>ms on 1 file with 100 rules using 1 threads.
+Finished in <variable>ms on 1 file using 1 threads.
 ----------
 CLI result: LintSucceeded
 ----------

apps/oxlint/src/snapshots/fixtures__nested_config_--experimental-nested-config -A no-console@oxlint.snap

@@ -28,7 +28,7 @@ working directory: fixtures/nested_config
   help: Delete this code.
 
 Found 1 warning and 2 errors.
-Finished in <variable>ms on 7 files with 100 rules using 1 threads.
+Finished in <variable>ms on 7 files using 1 threads.
 ----------
 CLI result: LintFoundErrors
 ----------

apps/oxlint/src/snapshots/fixtures__nested_config_--experimental-nested-config@oxlint.snap

@@ -51,7 +51,7 @@ working directory: fixtures/nested_config
   help: Delete this console statement.
 
 Found 1 warning and 5 errors.
-Finished in <variable>ms on 7 files with 101 rules using 1 threads.
+Finished in <variable>ms on 7 files using 1 threads.
 ----------
 CLI result: LintFoundErrors
 ----------

crates/oxc_linter/src/lib.rs

@@ -106,8 +106,11 @@ impl Linter {
         &self.options
     }
 
-    pub fn number_of_rules(&self) -> usize {
-        self.config.number_of_rules()
+    /// Returns the number of rules that will are being used, unless there
+    /// nested configurations in use, in which case it returns `None` since the
+    /// number of rules depends on which file is being linted.
+    pub fn number_of_rules(&self) -> Option<usize> {
+        self.nested_configs.is_empty().then_some(self.config.number_of_rules())
     }
 
     pub fn run<'a>(