fix: failed to check inside github action (#914)

Fixes voidzero-dev/setup-vp#7

I'm unsure whether we can produce good terminal output while retaining
the GitHub reporter. If my fix is off course, please feel free to close
this PR.

Co-authored-by: MK (fengmk2) <fengmk2@gmail.com>

Author: hyoban

packages/cli/binding/src/cli.rs

@@ -1109,6 +1109,12 @@ async fn execute_direct_subcommand(
                 if fix {
                     args.push("--fix".to_string());
                 }
+                // `vp check` parses oxlint's human-readable summary output to print
+                // unified pass/fail lines. When `GITHUB_ACTIONS=true`, oxlint auto-switches
+                // to the GitHub reporter, which omits that summary on success and makes the
+                // 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 has_paths {
                     args.extend(paths.iter().cloned());
                 }

packages/cli/snap-tests/check-pass-typecheck-github-actions/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "check-pass-typecheck-github-actions",
+  "version": "0.0.0",
+  "private": true
+}

packages/cli/snap-tests/check-pass-typecheck-github-actions/snap.txt

@@ -0,0 +1,3 @@
+> vp check
+pass: All 4 files are correctly formatted (<variable>ms, <variable> threads)
+pass: Found no warnings, lint errors, or type errors in 2 files (<variable>ms, <variable> threads)

packages/cli/snap-tests/check-pass-typecheck-github-actions/src/index.js

@@ -0,0 +1,5 @@
+function hello() {
+  return "hello";
+}
+
+export { hello };

packages/cli/snap-tests/check-pass-typecheck-github-actions/steps.json

@@ -0,0 +1,7 @@
+{
+  "env": {
+    "GITHUB_ACTIONS": "true",
+    "VITE_DISABLE_AUTO_INSTALL": "1"
+  },
+  "commands": ["vp check"]
+}

packages/cli/snap-tests/check-pass-typecheck-github-actions/vite.config.ts

@@ -0,0 +1,8 @@
+export default {
+  lint: {
+    options: {
+      typeAware: true,
+      typeCheck: true,
+    },
+  },
+};