feat(lint): export Oxlint plugin APIs from vite-plus (#2328)

`vp migrate` can break custom Oxlint plugins when it removes the
dependency that provides their API.

This PR adds two entry points for the API that matches the bundled
linter:

- `vite-plus/lint/plugins` exports helpers such as `definePlugin` and
`defineRule`.
- `vite-plus/lint/plugins-dev` exports `RuleTester`.

Both entry points support ESM and CommonJS. Projects can use them
through their `vite-plus` dependency, including with `pnpm`'s strict
dependency layout.

The migrator updates supported imports from `@oxlint/plugins`,
`oxlint/plugins-dev`, and the legacy plugin API in `oxlint`. The
`prefer-vite-plus-imports` lint rule applies the same changes. The
rewrite leaves `oxlint` config imports and ambiguous imports unchanged.
It also leaves `require()` calls unchanged.

Packages that declare `oxlint` or `@oxlint/plugins` in `dependencies` or
`peerDependencies` keep their upstream plugin imports and dependencies.
The same exemption applies to `@oxlint/plugins` in
`optionalDependencies`. This protects published plugins whose consumers
do not use `vite-plus`.

The cleanup removes `@oxlint/plugins` from `devDependencies` only when
no references remain in the project. It checks source files, package
import aliases, and ignored build output such as `dist`, `build`, and
`out`.

Author: fengmk2

.github/workflows/e2e-test.yml

@@ -327,6 +327,13 @@ jobs:
               vp check --fix
               vp run check
               vp test run --project unit --shard=1/3
+          - name: videojs-v10
+            node-version: 24
+            command: |
+              node $GITHUB_WORKSPACE/ecosystem-ci/verify-videojs-v10.ts
+              # The upstream preset keeps existing diagnostics at warning level.
+              vp lint --quiet
+              vp test run tools/oxlint/anti-slop/rules/tests
           - name: reactive-resume
             node-version: 24
             command: |

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/lint/plugin.js

@@ -0,0 +1,21 @@
+// Authored against the API vite-plus re-exports, with no `@oxlint/plugins`
+// dependency of its own: the point of the test is that this resolves and loads.
+import { definePlugin, defineRule } from 'vite-plus/lint/plugins';
+
+const noFoo = defineRule({
+  meta: { messages: { noFoo: 'Do not name things "foo".' } },
+  create(context) {
+    return {
+      Identifier(node) {
+        if (node.name === 'foo') {
+          context.report({ node, messageId: 'noFoo' });
+        }
+      },
+    };
+  },
+});
+
+export default definePlugin({
+  meta: { name: 'local' },
+  rules: { 'no-foo': noFoo },
+});

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "lint-oxlint-plugin-api",
+  "version": "0.0.0",
+  "private": true
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/snapshots.toml

@@ -0,0 +1,37 @@
+[[case]]
+name = "lint_oxlint_plugin_api"
+vp = "local"
+skip-platforms = [{ os = "linux", libc = "musl" }]
+steps = [
+  { argv = [
+    "vp",
+    "lint",
+    "src/uses-foo.ts",
+  ], comment = "the local JS plugin imports its API from vite-plus/lint/plugins. It declares no @oxlint/plugins dependency. A reported diagnostic therefore proves the export resolved and loaded", continue-on-failure = true },
+  { argv = [
+    "vp",
+    "lint",
+    "src/legacy-imports.ts",
+  ], comment = "prefer-vite-plus-imports reports the three legacy authoring specifiers", continue-on-failure = true },
+  { argv = [
+    "vp",
+    "lint",
+    "src/config-surface.ts",
+  ], comment = "oxlint still owns defineConfig and OxlintOverride, so these are clean", continue-on-failure = true },
+  { argv = [
+    "vp",
+    "lint",
+    "--fix",
+    "src/legacy-imports.ts",
+  ], comment = "the autofix matches what vp migrate rewrites", continue-on-failure = true },
+  { argv = [
+    "vpt",
+    "print-file",
+    "src/legacy-imports.ts",
+  ], continue-on-failure = true },
+  { argv = [
+    "vp",
+    "lint",
+    "src/legacy-imports.ts",
+  ], comment = "confirm the rewritten file is clean", continue-on-failure = true },
+]

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/snapshots/lint_oxlint_plugin_api.md

@@ -0,0 +1,109 @@
+# lint_oxlint_plugin_api
+
+## `vp lint src/uses-foo.ts`
+
+the local JS plugin imports its API from vite-plus/lint/plugins. It declares no @oxlint/plugins dependency. A reported diagnostic therefore proves the export resolved and loaded
+
+**Exit code:** 1
+
+```
+
+  × local(no-foo): Do not name things "foo".
+   ╭─[src/uses-foo.ts:1:14]
+ 1 │ export const foo = 1;
+   ·              ───
+ 2 │ export const bar = 2;
+   ╰────
+
+Found 0 warnings and 1 error.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp lint src/legacy-imports.ts`
+
+prefer-vite-plus-imports reports the three legacy authoring specifiers
+
+**Exit code:** 1
+
+```
+
+  × vite-plus(prefer-vite-plus-imports): Use 'vite-plus/lint/plugins' instead of 'oxlint' in Vite+ projects.
+   ╭─[src/legacy-imports.ts:1:28]
+ 1 │ import { defineRule } from 'oxlint';
+   ·                            ────────
+ 2 │ import { definePlugin } from '@oxlint/plugins';
+   ╰────
+
+  × vite-plus(prefer-vite-plus-imports): Use 'vite-plus/lint/plugins' instead of '@oxlint/plugins' in Vite+ projects.
+   ╭─[src/legacy-imports.ts:2:30]
+ 1 │ import { defineRule } from 'oxlint';
+ 2 │ import { definePlugin } from '@oxlint/plugins';
+   ·                              ─────────────────
+ 3 │ import { RuleTester } from 'oxlint/plugins-dev';
+   ╰────
+
+  × vite-plus(prefer-vite-plus-imports): Use 'vite-plus/lint/plugins-dev' instead of 'oxlint/plugins-dev' in Vite+ projects.
+   ╭─[src/legacy-imports.ts:3:28]
+ 2 │ import { definePlugin } from '@oxlint/plugins';
+ 3 │ import { RuleTester } from 'oxlint/plugins-dev';
+   ·                            ────────────────────
+ 4 │
+   ╰────
+
+  × vite-plus(prefer-vite-plus-imports): Use 'vite-plus/lint/plugins' instead of 'oxlint' in Vite+ projects.
+   ╭─[src/legacy-imports.ts:6:38]
+ 5 │ export { defineRule, definePlugin, RuleTester };
+ 6 │ export { 'defineRule' as rule } from 'oxlint';
+   ·                                      ────────
+ 7 │ export type { 'Context' as RuleContext } from 'oxlint';
+   ╰────
+
+  × vite-plus(prefer-vite-plus-imports): Use 'vite-plus/lint/plugins' instead of 'oxlint' in Vite+ projects.
+   ╭─[src/legacy-imports.ts:7:47]
+ 6 │ export { 'defineRule' as rule } from 'oxlint';
+ 7 │ export type { 'Context' as RuleContext } from 'oxlint';
+   ·                                               ────────
+   ╰────
+
+Found 0 warnings and 5 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp lint src/config-surface.ts`
+
+oxlint still owns defineConfig and OxlintOverride, so these are clean
+
+```
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp lint --fix src/legacy-imports.ts`
+
+the autofix matches what vp migrate rewrites
+
+```
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vpt print-file src/legacy-imports.ts`
+
+```
+import { defineRule } from 'vite-plus/lint/plugins';
+import { definePlugin } from 'vite-plus/lint/plugins';
+import { RuleTester } from 'vite-plus/lint/plugins-dev';
+
+export { defineRule, definePlugin, RuleTester };
+export { 'defineRule' as rule } from 'vite-plus/lint/plugins';
+export type { 'Context' as RuleContext } from 'vite-plus/lint/plugins';
+```
+
+## `vp lint src/legacy-imports.ts`
+
+confirm the rewritten file is clean
+
+```
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/src/config-surface.ts

@@ -0,0 +1,8 @@
+import { defineConfig } from 'oxlint';
+import type { OxlintOverride } from 'oxlint';
+
+export const override: OxlintOverride = { files: ['**/*.ts'] };
+
+export default defineConfig({ overrides: [override] });
+export { 'defineConfig' as config } from 'oxlint';
+export type { 'OxlintOverride' as Override } from 'oxlint';

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/src/legacy-imports.ts

@@ -0,0 +1,7 @@
+import { defineRule } from 'oxlint';
+import { definePlugin } from '@oxlint/plugins';
+import { RuleTester } from 'oxlint/plugins-dev';
+
+export { defineRule, definePlugin, RuleTester };
+export { 'defineRule' as rule } from 'oxlint';
+export type { 'Context' as RuleContext } from 'oxlint';

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/src/uses-foo.ts

@@ -0,0 +1,2 @@
+export const foo = 1;
+export const bar = 2;

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/lint_oxlint_plugin_api/vite.config.ts

@@ -0,0 +1,14 @@
+import { defineConfig } from 'vite-plus';
+
+export default defineConfig({
+  lint: {
+    jsPlugins: [
+      './lint/plugin.js',
+      { name: 'vite-plus', specifier: 'vite-plus/oxlint-plugin' },
+    ],
+    rules: {
+      'local/no-foo': 'error',
+      'vite-plus/prefer-vite-plus-imports': 'error',
+    },
+  },
+});

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_built_plugin/.gitignore

@@ -0,0 +1,2 @@
+dist/
+node_modules/