refactor(cli): use native lint and fmt config discovery

Author: fengmk2

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/custom-fmt.json

@@ -0,0 +1,4 @@
+{
+  "singleQuote": false,
+  "semi": true
+}

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/custom-lint.json

@@ -0,0 +1,5 @@
+{
+  "rules": {
+    "no-console": "off"
+  }
+}

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

@@ -0,0 +1,5 @@
+{
+  "name": "oxc-config-auto-discovery",
+  "private": true,
+  "type": "module"
+}

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

@@ -0,0 +1,37 @@
+[[case]]
+name = "discover_root_config"
+vp = ["local", "global"]
+steps = [
+  { argv = ["vp", "lint", "src"], comment = "The root lint config enables no-console.", continue-on-failure = true },
+  { argv = ["vp", "fmt", "src"] },
+  { argv = ["vpt", "print-file", "src/index.js"], comment = "The root fmt config selects single quotes and no semicolons." },
+  { argv = ["vp", "check", "src"], comment = "The composite command uses the same discovered settings.", continue-on-failure = true },
+]
+
+[[case]]
+name = "explicit_config"
+vp = ["local", "global"]
+steps = [
+  { argv = ["vp", "lint", "-c", "custom-lint.json", "src"], comment = "An explicit lint config overrides the root config." },
+  { argv = ["vp", "fmt", "--config", "custom-fmt.json", "src"] },
+  { argv = ["vpt", "print-file", "src/index.js"], comment = "The explicit fmt config keeps double quotes and semicolons." },
+  { argv = ["vp", "lint", "--config=custom-lint.json", "src"] },
+  { argv = ["vp", "fmt", "-c", "custom-fmt.json", "--check", "src"] },
+]
+
+[[case]]
+name = "discover_config_from_subdirectory"
+vp = "local"
+steps = [
+  { argv = ["vpt", "write-file", "src/vite.config.ts", "export default {};\n"] },
+  { argv = ["vp", "lint", "index.js"], cwd = "src", comment = "A config without lint settings is skipped in favor of the parent config.", continue-on-failure = true },
+  { argv = ["vp", "fmt", "index.js"], cwd = "src" },
+  { argv = ["vpt", "print-file", "src/index.js"], comment = "The parent fmt settings are discovered too." },
+  { argv = ["vpt", "write-file", "src/vite.config.ts", "export default { lint: { rules: { 'no-console': 'off' } }, fmt: { singleQuote: false, semi: true } };\n"] },
+  { argv = ["vp", "lint", "index.js"], cwd = "src", comment = "The config in the working directory takes precedence when it has lint settings." },
+  { argv = ["vp", "fmt", "index.js"], cwd = "src" },
+  { argv = ["vpt", "print-file", "src/index.js"], comment = "The working directory's fmt settings take precedence too." },
+  { argv = ["vp", "lint", "src/index.js"], comment = "Running from the root keeps per-file nested lint configs disabled.", continue-on-failure = true },
+  { argv = ["vp", "fmt", "src/index.js"] },
+  { argv = ["vpt", "print-file", "src/index.js"], comment = "Running from the root keeps per-file nested fmt configs disabled." },
+]

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/snapshots/discover_config_from_subdirectory.md

@@ -0,0 +1,102 @@
+# discover_config_from_subdirectory
+
+## `vpt write-file src/vite.config.ts 'export default {};
+'`
+
+```
+```
+
+## `cd src && vp lint index.js`
+
+A config without lint settings is skipped in favor of the parent config.
+
+**Exit code:** 1
+
+```
+
+  × eslint(no-console): Unexpected console statement.
+   ╭─[index.js:1:1]
+ 1 │ console.log("hello");
+   · ───────────
+   ╰────
+  help: Delete this console statement.
+
+Found 0 warnings and 1 error.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `cd src && vp fmt index.js`
+
+```
+Finished in <duration> on 1 files using <n> threads.
+```
+
+## `vpt print-file src/index.js`
+
+The parent fmt settings are discovered too.
+
+```
+console.log('hello')
+```
+
+## `vpt write-file src/vite.config.ts 'export default { lint: { rules: { '\''no-console'\'': '\''off'\'' } }, fmt: { singleQuote: false, semi: true } };
+'`
+
+```
+```
+
+## `cd src && vp lint index.js`
+
+The config in the working directory takes precedence when it has lint settings.
+
+```
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `cd src && vp fmt index.js`
+
+```
+Finished in <duration> on 1 files using <n> threads.
+```
+
+## `vpt print-file src/index.js`
+
+The working directory's fmt settings take precedence too.
+
+```
+console.log("hello");
+```
+
+## `vp lint src/index.js`
+
+Running from the root keeps per-file nested lint configs disabled.
+
+**Exit code:** 1
+
+```
+
+  × eslint(no-console): Unexpected console statement.
+   ╭─[src/index.js:1:1]
+ 1 │ console.log("hello");
+   · ───────────
+   ╰────
+  help: Delete this console statement.
+
+Found 0 warnings and 1 error.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp fmt src/index.js`
+
+```
+Finished in <duration> on 1 files using <n> threads.
+```
+
+## `vpt print-file src/index.js`
+
+Running from the root keeps per-file nested fmt configs disabled.
+
+```
+console.log('hello')
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/snapshots/discover_root_config.global.md

@@ -0,0 +1,58 @@
+# discover_root_config
+
+## `vp lint src`
+
+The root lint config enables no-console.
+
+**Exit code:** 1
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+  × eslint(no-console): Unexpected console statement.
+   ╭─[src/index.js:1:1]
+ 1 │ console.log("hello");
+   · ───────────
+   ╰────
+  help: Delete this console statement.
+
+Found 0 warnings and 1 error.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp fmt src`
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+Finished in <duration> on 1 files using <n> threads.
+```
+
+## `vpt print-file src/index.js`
+
+The root fmt config selects single quotes and no semicolons.
+
+```
+console.log('hello')
+```
+
+## `vp check src`
+
+The composite command uses the same discovered settings.
+
+**Exit code:** 1
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+pass: All 1 file are correctly formatted (<duration>, <n> threads)
+error: Lint issues found
+× eslint(no-console): Unexpected console statement.
+   ╭─[src/index.js:1:1]
+ 1 │ console.log('hello')
+   · ───────────
+   ╰────
+  help: Delete this console statement.
+
+Found 1 error and 0 warnings in 1 file (<duration>, <n> threads)
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/snapshots/discover_root_config.local.md

@@ -0,0 +1,53 @@
+# discover_root_config
+
+## `vp lint src`
+
+The root lint config enables no-console.
+
+**Exit code:** 1
+
+```
+
+  × eslint(no-console): Unexpected console statement.
+   ╭─[src/index.js:1:1]
+ 1 │ console.log("hello");
+   · ───────────
+   ╰────
+  help: Delete this console statement.
+
+Found 0 warnings and 1 error.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp fmt src`
+
+```
+Finished in <duration> on 1 files using <n> threads.
+```
+
+## `vpt print-file src/index.js`
+
+The root fmt config selects single quotes and no semicolons.
+
+```
+console.log('hello')
+```
+
+## `vp check src`
+
+The composite command uses the same discovered settings.
+
+**Exit code:** 1
+
+```
+pass: All 1 file are correctly formatted (<duration>, <n> threads)
+error: Lint issues found
+× eslint(no-console): Unexpected console statement.
+   ╭─[src/index.js:1:1]
+ 1 │ console.log('hello')
+   · ───────────
+   ╰────
+  help: Delete this console statement.
+
+Found 1 error and 0 warnings in 1 file (<duration>, <n> threads)
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/snapshots/explicit_config.global.md

@@ -0,0 +1,48 @@
+# explicit_config
+
+## `vp lint -c custom-lint.json src`
+
+An explicit lint config overrides the root config.
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp fmt --config custom-fmt.json src`
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+Finished in <duration> on 1 files using <n> threads.
+```
+
+## `vpt print-file src/index.js`
+
+The explicit fmt config keeps double quotes and semicolons.
+
+```
+console.log("hello");
+```
+
+## `vp lint --config=custom-lint.json src`
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp fmt -c custom-fmt.json --check src`
+
+```
+VITE+ - The Unified Toolchain for the Web
+
+Checking formatting...
+
+All matched files use the correct format.
+Finished in <duration> on 1 files using <n> threads.
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/snapshots/explicit_config.local.md

@@ -0,0 +1,40 @@
+# explicit_config
+
+## `vp lint -c custom-lint.json src`
+
+An explicit lint config overrides the root config.
+
+```
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp fmt --config custom-fmt.json src`
+
+```
+Finished in <duration> on 1 files using <n> threads.
+```
+
+## `vpt print-file src/index.js`
+
+The explicit fmt config keeps double quotes and semicolons.
+
+```
+console.log("hello");
+```
+
+## `vp lint --config=custom-lint.json src`
+
+```
+Found 0 warnings and 0 errors.
+Finished in <duration> on 1 file with <n> rules using <n> threads.
+```
+
+## `vp fmt -c custom-fmt.json --check src`
+
+```
+Checking formatting...
+
+All matched files use the correct format.
+Finished in <duration> on 1 files using <n> threads.
+```

crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/oxc_config_auto_discovery/src/index.js

@@ -0,0 +1 @@
+console.log("hello");