feat(check): pass --no-error-on-unmatched-pattern to oxlint natively

Now that oxlint supports --no-error-on-unmatched-pattern, pass it
directly instead of working around it at the vp check level. Simplify
the lint None branch to match the fmt pattern.

Add snap tests for `vp check --fix package.json` (non-lintable file,
should pass) and `vp lint package.json` (should exit non-zero).

Author: fengmk2

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

@@ -140,6 +140,9 @@ pub(crate) async fn execute_check(
         // parser think linting never started. Force the default reporter here so the
         // captured output is stable across local and CI environments.
         args.push("--format=default".to_string());
+        if suppress_unmatched {
+            args.push("--no-error-on-unmatched-pattern".to_string());
+        }
         if has_paths {
             args.extend(paths.iter().cloned());
         }
@@ -190,12 +193,11 @@ pub(crate) async fn execute_check(
                 ));
             }
             None => {
-                // Only suppress when the output is empty (no files to lint).
-                // If oxlint produced error output (config error, crash, etc.),
-                // surface it even when suppress_unmatched is active.
-                if suppress_unmatched && combined_output.trim().is_empty() {
-                    status = ExitStatus::SUCCESS;
-                } else {
+                // oxlint handles --no-error-on-unmatched-pattern natively and
+                // exits 0 when no files match, so we only need to guard
+                // against the edge case where output is unparsable but the
+                // process still succeeded.
+                if !(suppress_unmatched && status == ExitStatus::SUCCESS) {
                     output::error("Linting could not start");
                     if !combined_output.trim().is_empty() {
                         print_stdout_block(&combined_output);

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

@@ -1,16 +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 package.json # non-lintable file, should pass in --fix mode
+pass: Formatting completed for checked files (<variable>ms)
 
+> 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
 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
+Checking formatting...
+Expected at least one target file. All matched files may have been excluded by ignore rules.
 
 Formatting failed before analysis started

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

@@ -1,6 +1,7 @@
 {
   "commands": [
     "vp check --fix src/ignored/index.js # all files excluded by ignorePatterns, should pass in --fix mode",
+    "vp check --fix package.json # non-lintable file, 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/lint-unmatched-pattern/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "lint-unmatched-pattern",
+  "version": "0.0.0",
+  "private": true
+}

packages/cli/snap-tests/lint-unmatched-pattern/snap.txt

@@ -0,0 +1,3 @@
+[1]> vp lint package.json # non-lintable file, should exit non-zero
+No files found to lint. Please check your paths and ignore patterns.
+Finished in <variable>ms on 0 files with <variable> rules using <variable> threads.

packages/cli/snap-tests/lint-unmatched-pattern/steps.json

@@ -0,0 +1,3 @@
+{
+  "commands": ["vp lint package.json # non-lintable file, should exit non-zero"]
+}

rfcs/check-command.md

@@ -76,7 +76,7 @@ When file paths are provided:
 
 - Paths are appended to both `fmt` and `lint` sub-commands
 - In `--fix` mode, `--no-error-on-unmatched-pattern` is implicitly enabled for both `fmt` and `lint`, preventing errors when all provided paths are excluded by ignorePatterns. This is the common lint-staged use case where staged files may not match tool-specific patterns.
-- Without `--fix`, unmatched patterns are reported as errors unless `--no-error-on-unmatched-pattern` is explicitly passed. Note that oxfmt supports this flag natively, while oxlint does not — `vp check` handles the lint side by treating unparsable lint output as a pass when the flag is active.
+- Without `--fix`, unmatched patterns are reported as errors unless `--no-error-on-unmatched-pattern` is explicitly passed. Both oxfmt and oxlint support this flag natively.
 
 This enables lint-staged integration: