fix(migrate): preserve Oxlint inheritance with shared TS configs

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

@@ -0,0 +1 @@
+**/node_modules

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

@@ -0,0 +1,6 @@
+{
+  "name": "create-oxlint-workspace",
+  "private": true,
+  "type": "module",
+  "packageManager": "pnpm@10.0.0"
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/create_monorepo_oxlint_extends/packages/starter-template/bin/index.mjs

@@ -0,0 +1,15 @@
+import { mkdirSync, writeFileSync } from 'node:fs';
+import path from 'node:path';
+
+const directoryFlag = process.argv.indexOf('--directory');
+const directory = directoryFlag === -1 ? 'app' : process.argv[directoryFlag + 1];
+mkdirSync(directory, { recursive: true });
+
+function write(filename, content) {
+  writeFileSync(path.join(directory, filename), content);
+}
+
+write('package.json', JSON.stringify({ name: path.basename(directory), type: 'module' }));
+write('vite.config.mjs', 'export default {};\n');
+write('.oxlintrc.json', JSON.stringify({ extends: ['../../shared/lint.json'] }));
+write('index.js', 'debugger;\n');

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/create_monorepo_oxlint_extends/packages/starter-template/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "starter-template",
+  "version": "0.0.0",
+  "private": true,
+  "type": "module",
+  "bin": "./bin/index.mjs"
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/create_monorepo_oxlint_extends/pnpm-workspace.yaml

@@ -0,0 +1,2 @@
+packages:
+  - packages/*

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/create_monorepo_oxlint_extends/shared/lint.json

@@ -0,0 +1,4 @@
+{
+  "rules": { "no-debugger": "error" },
+  "options": { "typeAware": false, "typeCheck": false }
+}

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

@@ -0,0 +1,17 @@
+[[case]]
+name = "create_monorepo_oxlint_extends"
+vp = "local"
+local-registry = true
+unset-env = ["VP_SKIP_INSTALL"]
+steps = [
+  { argv = ["node", "packages/starter-template/bin/index.mjs", "--directory", "packages/before"], snapshot = false },
+  { argv = ["vp", "lint", "--threads=1", "-c", ".oxlintrc.json", "index.js"], cwd = "packages/before", comment = "Original template rules inherit from the workspace", continue-on-failure = true },
+  { argv = ["vpt", "rm", "-r", "packages/before"], snapshot = false },
+  { argv = ["vp", "create", "starter", "--no-interactive", "--no-agent", "--no-editor", "--no-hooks", "--", "--directory", "app"] },
+  { argv = ["vpt", "print-file", "packages/app/vite.config.mjs"] },
+  { argv = ["vpt", "print-file", "shared/lint.ts"] },
+  { argv = ["vpt", "stat-file", "packages/app/.oxlintrc.json", "--assert", "missing"] },
+  { argv = ["vp", "lint", "--threads=1", "index.js"], cwd = "packages/app", comment = "The created package keeps the original diagnostic without a config flag", continue-on-failure = true },
+  { argv = ["vpt", "replace-file-content", "shared/lint.ts", '"error"', '"warn"'] },
+  { argv = ["vp", "lint", "--threads=1", "index.js"], cwd = "packages/app", comment = "The created package follows shared module edits" },
+]

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

@@ -0,0 +1,123 @@
+# create_monorepo_oxlint_extends
+
+## `node packages/starter-template/bin/index.mjs --directory packages/before`
+
+
+## `cd packages/before && vp lint --threads=1 -c .oxlintrc.json index.js`
+
+Original template rules inherit from the workspace
+
+**Exit code:** 1
+
+```
+
+  × eslint(no-debugger): `debugger` statement is not allowed
+   ╭─[index.js:1:1]
+ 1 │ debugger;
+   · ─────────
+   ╰────
+  help: Remove the debugger statement
+
+Found 0 warnings and 1 error.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vpt rm -r packages/before`
+
+
+## `vp create starter --no-interactive --no-agent --no-editor --no-hooks -- --directory app`
+
+```
+◇ Generating project…
+● Running: node <workspace>/packages/starter-template/bin/index.mjs --directory app
+◇ Monorepo integration...
+◆ ✔ Merged packages/app/.oxlintrc.json into packages/app/vite.config.mjs
+● Installing dependencies...
+● Dependencies installed
+● Formatting code...
+● Code formatted
+◇ Scaffolded packages/app
+• Node <version>  pnpm <version>
+✓ Dependencies installed in <duration>
+→ Next: cd packages/app && vp run
+```
+
+## `vpt print-file packages/app/vite.config.mjs`
+
+```
+import oxlintConfig1 from "../../shared/lint.js";
+export default {
+  lint: {
+    extends: [oxlintConfig1],
+    options: {},
+    jsPlugins: [{ name: "vite-plus", specifier: "vite-plus/oxlint-plugin" }],
+    rules: {
+      "vite-plus/prefer-vite-plus-imports": "error",
+    },
+  },
+};
+```
+
+## `vpt print-file shared/lint.ts`
+
+```
+// Generated by vp migrate from lint.json.
+import type { OxlintConfig } from "vite-plus/lint";
+
+export default {
+  "rules": {
+    "no-debugger": "error"
+  },
+  "options": {
+    "typeAware": false,
+    "typeCheck": false
+  }
+} satisfies OxlintConfig;
+```
+
+## `vpt stat-file packages/app/.oxlintrc.json --assert missing`
+
+```
+packages/app/.oxlintrc.json: missing
+```
+
+## `cd packages/app && vp lint --threads=1 index.js`
+
+The created package keeps the original diagnostic without a config flag
+
+**Exit code:** 1
+
+```
+
+  × eslint(no-debugger): `debugger` statement is not allowed
+   ╭─[index.js:1:1]
+ 1 │ debugger;
+   · ─────────
+   ╰────
+  help: Remove the debugger statement
+
+Found 0 warnings and 1 error.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vpt replace-file-content shared/lint.ts '"error"' '"warn"'`
+
+```
+```
+
+## `cd packages/app && vp lint --threads=1 index.js`
+
+The created package follows shared module edits
+
+```
+
+  ⚠ eslint(no-debugger): `debugger` statement is not allowed
+   ╭─[index.js:1:1]
+ 1 │ debugger;
+   · ─────────
+   ╰────
+  help: Remove the debugger statement
+
+Found 1 warning and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/create_monorepo_oxlint_extends/vite.config.mjs

@@ -0,0 +1,11 @@
+export default {
+  create: {
+    templates: [
+      {
+        name: 'starter',
+        description: 'A package that inherits workspace lint rules.',
+        template: './packages/starter-template',
+      },
+    ],
+  },
+};

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

@@ -19,6 +19,7 @@ VITE+ - The Unified Toolchain for the Web
 check vite.config.ts
 
 ```
+// vp migrate: merged .oxlintrc.json (aba19784a1d2c46ab69279abb5cab90f).
 import react from '@vitejs/plugin-react';
 import { defineConfig, lazyPlugins } from 'vite-plus';
 

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

@@ -20,6 +20,7 @@ VITE+ - The Unified Toolchain for the Web
 check vite.config.ts
 
 ```
+// vp migrate: merged .oxlintrc.json (aba19784a1d2c46ab69279abb5cab90f).
 import react from '@vitejs/plugin-react';
 import { defineConfig, lazyPlugins } from 'vite-plus';
 
@@ -225,6 +226,7 @@ check only-oxlint package.json
 check only-oxlint vite.config.ts
 
 ```
+// vp migrate: merged packages%2Fonly-oxlint%2F.oxlintrc.json (b648b436b7a8c2627c4008844cff0e45).
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({

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

@@ -27,6 +27,7 @@ VITE+ - The Unified Toolchain for the Web
 check vite.config.ts
 
 ```
+// vp migrate: merged .oxlintrc.json (aba19784a1d2c46ab69279abb5cab90f).
 import react from '@vitejs/plugin-react';
 import { defineConfig, lazyPlugins } from 'vite-plus';
 

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

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

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/.oxlintrc.json

@@ -0,0 +1,14 @@
+{
+  "extends": [
+    "./shared/nested/first.json",
+    "./shared/second.jsonc"
+  ],
+  "rules": {
+    "no-alert": "off",
+    "local/no-foo": "error"
+  },
+  "ignorePatterns": [
+    "shared/**",
+    "vite.config.ts"
+  ]
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/create-directory-link.mjs

@@ -0,0 +1,3 @@
+import { symlinkSync } from 'node:fs';
+
+symlinkSync('./configs/nested', 'aliases', process.platform === 'win32' ? 'junction' : 'dir');

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/example.test.js

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

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/index.js

@@ -0,0 +1,5 @@
+debugger;
+console.log("test");
+alert("allowed");
+export const foo = 1;
+export const bar = 1;

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

@@ -0,0 +1,10 @@
+{
+  "name": "migration-oxlint-extends",
+  "private": true,
+  "type": "module",
+  "devDependencies": {
+    "oxlint": "1.88.0",
+    "typescript": "6.0.3",
+    "@types/node": "24.10.3"
+  }
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/shared/base.json

@@ -0,0 +1,11 @@
+{
+  "options": {
+    "typeAware": false,
+    "typeCheck": false
+  },
+  "rules": {
+    "no-debugger": "error",
+    "no-console": "error",
+    "no-alert": "error"
+  }
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/shared/nested/first.json

@@ -0,0 +1,22 @@
+{
+  "extends": [
+    "../base.json"
+  ],
+  "overrides": [
+    {
+      "files": [
+        "**/*.test.js"
+      ],
+      "rules": {
+        "no-debugger": "warn",
+        "override/no-bar": "error"
+      },
+      "jsPlugins": [
+        {
+          "name": "override",
+          "specifier": "../override-plugin.mjs"
+        }
+      ]
+    }
+  ]
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/shared/override-plugin.mjs

@@ -0,0 +1,17 @@
+export default {
+  meta: { name: 'override' },
+  rules: {
+    'no-bar': {
+      meta: { schema: [] },
+      create(context) {
+        return {
+          Identifier(node) {
+            if (node.name === 'bar') {
+              context.report({ node, message: 'Do not use bar.' });
+            }
+          },
+        };
+      },
+    },
+  },
+};

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/shared/plugin.mjs

@@ -0,0 +1,17 @@
+export default {
+  meta: { name: 'local' },
+  rules: {
+    'no-foo': {
+      meta: { schema: [] },
+      create(context) {
+        return {
+          Identifier(node) {
+            if (node.name === 'foo') {
+              context.report({ node, message: 'Do not use foo.' });
+            }
+          },
+        };
+      },
+    },
+  },
+};

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/migration_oxlint_extends/shared/second.jsonc

@@ -0,0 +1,5 @@
+{
+  // Later entries take priority.
+  "rules": { "no-console": "warn" },
+  "jsPlugins": ["./plugin.mjs"]
+}