feat(check): support --no-error-on-unmatched-pattern flag

In `--fix` mode with file paths, implicitly suppress "no matching
files" errors for both fmt and lint. This fixes the lint-staged use
case where staged files may be excluded by ignorePatterns (e.g.
package-lock.json excluded by fmt ignorePatterns).

Also expose `--no-error-on-unmatched-pattern` as an explicit CLI
flag for non-fix use cases. oxfmt supports this flag natively; for
oxlint (which does not), vp check handles it by treating unparseable
lint output as a pass when the flag is active.

Closes #1210

Author: fengmk2

crates/vite_global_cli/src/help.rs

@@ -759,6 +759,10 @@ fn delegated_help_doc(command: &str) -> Option<HelpDoc> {
                         row("--fix", "Auto-fix format and lint issues"),
                         row("--no-fmt", "Skip format check"),
                         row("--no-lint", "Skip lint check"),
+                        row(
+                            "--no-error-on-unmatched-pattern",
+                            "Do not exit with error when pattern is unmatched",
+                        ),
                         row("-h, --help", "Print help"),
                     ],
                 ),

packages/cli/binding/src/check/mod.rs

@@ -22,6 +22,7 @@ pub(crate) async fn execute_check(
     fix: bool,
     no_fmt: bool,
     no_lint: bool,
+    no_error_on_unmatched_pattern: bool,
     paths: Vec<String>,
     envs: &Arc<FxHashMap<Arc<OsStr>, Arc<OsStr>>>,
     cwd: &AbsolutePathBuf,
@@ -37,14 +38,20 @@ pub(crate) async fn execute_check(
 
     let mut status = ExitStatus::SUCCESS;
     let has_paths = !paths.is_empty();
+    // In --fix mode with file paths (the lint-staged use case), implicitly suppress
+    // "no matching files" errors. This is also available as an explicit flag for
+    // non-fix use cases.
+    let suppress_unmatched = no_error_on_unmatched_pattern || (fix && has_paths);
     let mut fmt_fix_started: Option<Instant> = None;
     let mut deferred_lint_pass: Option<(String, String)> = None;
     let resolved_vite_config = resolver.resolve_universal_vite_config().await?;
 
     if !no_fmt {
         let mut args = if fix { vec![] } else { vec!["--check".to_string()] };
-        if has_paths {
+        if suppress_unmatched {
             args.push("--no-error-on-unmatched-pattern".to_string());
+        }
+        if has_paths {
             args.extend(paths.iter().cloned());
         }
         let fmt_start = Instant::now();
@@ -87,11 +94,17 @@ pub(crate) async fn execute_check(
                     ));
                 }
                 None => {
-                    print_error_block(
-                        "Formatting could not start",
-                        &combined_output,
-                        "Formatting failed before analysis started",
-                    );
+                    if suppress_unmatched && status == ExitStatus::SUCCESS {
+                        // No files matched fmt patterns — treat as pass when
+                        // --no-error-on-unmatched-pattern is active (explicit or
+                        // implicit via --fix with paths).
+                    } else {
+                        print_error_block(
+                            "Formatting could not start",
+                            &combined_output,
+                            "Formatting failed before analysis started",
+                        );
+                    }
                 }
             }
         }
@@ -177,11 +190,18 @@ pub(crate) async fn execute_check(
                 ));
             }
             None => {
-                output::error("Linting could not start");
-                if !combined_output.trim().is_empty() {
-                    print_stdout_block(&combined_output);
+                if suppress_unmatched {
+                    // oxlint does not support --no-error-on-unmatched-pattern natively,
+                    // so we handle it here: when all files are excluded by ignorePatterns
+                    // and the flag is active, treat it as a pass instead of an error.
+                    status = ExitStatus::SUCCESS;
+                } else {
+                    output::error("Linting could not start");
+                    if !combined_output.trim().is_empty() {
+                        print_stdout_block(&combined_output);
+                    }
+                    print_summary_line("Linting failed before analysis started");
                 }
-                print_summary_line("Linting failed before analysis started");
             }
         }
         if status != ExitStatus::SUCCESS {
@@ -193,8 +213,10 @@ pub(crate) async fn execute_check(
     // (e.g. the curly rule adding braces to if-statements)
     if fix && !no_fmt && !no_lint {
         let mut args = Vec::new();
-        if has_paths {
+        if suppress_unmatched {
             args.push("--no-error-on-unmatched-pattern".to_string());
+        }
+        if has_paths {
             args.extend(paths.into_iter());
         }
         let captured = resolve_and_capture_output(

packages/cli/binding/src/cli/mod.rs

@@ -63,9 +63,23 @@ async fn execute_direct_subcommand(
     let cwd_arc: Arc<AbsolutePath> = cwd.clone().into();
 
     let status = match subcommand {
-        SynthesizableSubcommand::Check { fix, no_fmt, no_lint, paths } => {
+        SynthesizableSubcommand::Check {
+            fix,
+            no_fmt,
+            no_lint,
+            no_error_on_unmatched_pattern,
+            paths,
+        } => {
             return crate::check::execute_check(
-                &resolver, fix, no_fmt, no_lint, paths, &envs, cwd, &cwd_arc,
+                &resolver,
+                fix,
+                no_fmt,
+                no_lint,
+                no_error_on_unmatched_pattern,
+                paths,
+                &envs,
+                cwd,
+                &cwd_arc,
             )
             .await;
         }

packages/cli/binding/src/cli/types.rs

@@ -93,6 +93,9 @@ pub enum SynthesizableSubcommand {
         /// Skip lint check
         #[arg(long = "no-lint")]
         no_lint: bool,
+        /// Do not exit with error when pattern is unmatched
+        #[arg(long = "no-error-on-unmatched-pattern")]
+        no_error_on_unmatched_pattern: bool,
         /// File paths to check (passed through to fmt and lint)
         #[arg(trailing_var_arg = true)]
         paths: Vec<String>,

packages/cli/snap-tests-global/command-check-help/snap.txt

@@ -6,10 +6,11 @@ Usage: vp check [OPTIONS] [PATHS]...
 Run format, lint, and type checks.
 
 Options:
-  --fix       Auto-fix format and lint issues
-  --no-fmt    Skip format check
-  --no-lint   Skip lint check
-  -h, --help  Print help
+  --fix                            Auto-fix format and lint issues
+  --no-fmt                         Skip format check
+  --no-lint                        Skip lint check
+  --no-error-on-unmatched-pattern  Do not exit with error when pattern is unmatched
+  -h, --help                       Print help
 
 Examples:
   vp check
@@ -27,10 +28,11 @@ Usage: vp check [OPTIONS] [PATHS]...
 Run format, lint, and type checks.
 
 Options:
-  --fix       Auto-fix format and lint issues
-  --no-fmt    Skip format check
-  --no-lint   Skip lint check
-  -h, --help  Print help
+  --fix                            Auto-fix format and lint issues
+  --no-fmt                         Skip format check
+  --no-lint                        Skip lint check
+  --no-error-on-unmatched-pattern  Do not exit with error when pattern is unmatched
+  -h, --help                       Print help
 
 Examples:
   vp check
@@ -48,10 +50,11 @@ Usage: vp check [OPTIONS] [PATHS]...
 Run format, lint, and type checks.
 
 Options:
-  --fix       Auto-fix format and lint issues
-  --no-fmt    Skip format check
-  --no-lint   Skip lint check
-  -h, --help  Print help
+  --fix                            Auto-fix format and lint issues
+  --no-fmt                         Skip format check
+  --no-lint                        Skip lint check
+  --no-error-on-unmatched-pattern  Do not exit with error when pattern is unmatched
+  -h, --help                       Print help
 
 Examples:
   vp check

packages/cli/snap-tests/check-fix-no-error-unmatched/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "check-fix-no-error-unmatched",
+  "version": "0.0.0",
+  "private": true
+}

packages/cli/snap-tests/check-fix-no-error-unmatched/snap.txt

@@ -0,0 +1,16 @@
+> vp check --fix src/ignored/index.js # all files excluded by ignorePatterns, should pass in --fix mode
+pass: Formatting completed for checked files (<variable>ms)
+pass: Found no warnings or lint errors in 0 files (<variable>ms, <variable> threads)
+
+> vp check --no-error-on-unmatched-pattern src/ignored/index.js # explicit flag without --fix, should also pass
+pass: Found no warnings or lint errors in 0 files (<variable>ms, <variable> threads)
+
+> vp check --fix --no-error-on-unmatched-pattern src/ignored/index.js # both flags set, should pass
+pass: Formatting completed for checked files (<variable>ms)
+pass: Found no warnings or lint errors in 0 files (<variable>ms, <variable> threads)
+
+[2]> vp check src/ignored/index.js # without --fix or explicit flag, should exit non-zero
+error: Formatting could not start
+Expected at least one target file
+
+Formatting failed before analysis started

packages/cli/snap-tests/check-fix-no-error-unmatched/src/ignored/index.js

@@ -0,0 +1,2 @@
+// This file is excluded by both fmt and lint ignorePatterns.
+export const hello = 'world';

packages/cli/snap-tests/check-fix-no-error-unmatched/steps.json

@@ -0,0 +1,8 @@
+{
+  "commands": [
+    "vp check --fix src/ignored/index.js # all files excluded by ignorePatterns, should pass in --fix mode",
+    "vp check --no-error-on-unmatched-pattern src/ignored/index.js # explicit flag without --fix, should also pass",
+    "vp check --fix --no-error-on-unmatched-pattern src/ignored/index.js # both flags set, should pass",
+    "vp check src/ignored/index.js # without --fix or explicit flag, should exit non-zero"
+  ]
+}

packages/cli/snap-tests/check-fix-no-error-unmatched/vite.config.ts

@@ -0,0 +1,8 @@
+export default {
+  fmt: {
+    ignorePatterns: ['src/ignored/**/*'],
+  },
+  lint: {
+    ignorePatterns: ['src/ignored/**/*'],
+  },
+};