fix(check): respect oxlint exit code so denyWarnings fails vp check

[fengmk2]

vp check was unconditionally resetting the status to success whenever
lint reported only warnings, masking the non-zero exit code produced by
oxlint when options.denyWarnings (or --deny-warnings) is enabled.
Drop the override and trust oxlint's exit code so warning-only runs fail
the check when warnings have been opted in as failures.

Closes #1424

---

Note

Medium Risk
Changes vp check exit-code behavior for warning-only lint results, which can cause previously-passing pipelines to fail when denyWarnings is enabled. Scope is small and localized to the check command plus a new regression snap-test.

Overview
vp check no longer forces a success exit status when oxlint reports only warnings; it now propagates oxlint’s actual exit code so lint.options.denyWarnings (or --deny-warnings) correctly fails the check.

Adds a new snap-test fixture check-lint-warn-deny-warnings that configures denyWarnings: true with a warning-level no-console rule and asserts the expected warning output and summary.

^{Reviewed by Cursor Bugbot for commit f6466c2. Configure here.}

[netlify]

✅ Deploy Preview for viteplus-preview canceled.

Name	Link
🔨 Latest commit	bcd1a84
🔍 Latest deploy log	https://app.netlify.com/projects/viteplus-preview/deploys/69e6d53bf841610008dfc329

[fengmk2]

• fix(check): respect oxlint exit code so denyWarnings fails vp check #1425 👈 (View in Graphite)
• main

---

How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[fengmk2]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit f6466c2. Configure here.}

[cpojer]

I didn't know this feature existed when I built the vp check wrapper for oxlint.

I don't know why this feature exits in oxlint tbh, wouldn't it be better to force people to just set their lint rules to "error"?

[fengmk2]

I don't know why this feature exits in oxlint tbh, wouldn't it be better to force people to just set their lint rules to "error"?

Many of the built-in rules are configured as warn by default. By setting denyWarnings, there's no need to find and change all these rules to error individually. I believe this is the intended purpose. 😃

[cpojer]

Yeah, I understand that, but imho the problem is that the concept of a warning exists at all in Oxlint. There should only be errors.

[fengmk2]

vinext ci fails fix: cloudflare/vinext#864