feat(deps): upgrade oxlint to 1.54.0 and oxfmt to 0.39.0 (#795)

fengmk2 · web-flow · commit 6e99279cea8c · 2026-03-12T16:03:25.000+08:00
Remove forced `fmt: {}` injection that was required by older oxfmt versions.
The new oxfmt/oxlint releases no longer require `lint` and `fmt` fields in
vite.config.ts, so `injectFmtDefaultsIfMissing()` and `injectFmtDefaults()`
are removed along with empty placeholder fields in snap test fixtures.
diff --git a/packages/cli/snap-tests-global/command-staged-with-config/vite.config.ts b/packages/cli/snap-tests-global/command-staged-with-config/vite.config.ts
@@ -4,7 +4,6 @@ export default {
       'no-eval': 'error',
     },
   },
-  fmt: {},
   staged: {
     '*.ts': 'vp check --fix',
     '*.js': 'vp lint',
diff --git a/packages/cli/snap-tests-global/create-missing-typecheck/snap.txt b/packages/cli/snap-tests-global/create-missing-typecheck/snap.txt
@@ -6,7 +6,6 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix",
   },
-  fmt: {},
   lint: { options: { typeAware: true, typeCheck: true } },
 });
 
@@ -18,7 +17,6 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix",
   },
-  fmt: {},
   lint: { options: { typeAware: true, typeCheck: true } },
 });
 
diff --git a/packages/cli/snap-tests-global/migration-baseurl-tsconfig/snap.txt b/packages/cli/snap-tests-global/migration-baseurl-tsconfig/snap.txt
@@ -15,7 +15,6 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
-  fmt: {},
   lint: {
     "rules": {
       "no-unused-vars": "error"
diff --git a/packages/cli/snap-tests-global/migration-chained-lint-staged-pre-commit/snap.txt b/packages/cli/snap-tests-global/migration-chained-lint-staged-pre-commit/snap.txt
@@ -30,7 +30,6 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"
diff --git a/packages/cli/snap-tests-global/migration-env-prefix-lint-staged/snap.txt b/packages/cli/snap-tests-global/migration-env-prefix-lint-staged/snap.txt
@@ -30,7 +30,6 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"
diff --git a/packages/cli/snap-tests-global/migration-eslint-lint-staged/snap.txt b/packages/cli/snap-tests-global/migration-eslint-lint-staged/snap.txt
@@ -30,7 +30,6 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {
     "plugins": [
       "oxc",
diff --git a/packages/cli/snap-tests-global/migration-eslint-lintstagedrc/snap.txt b/packages/cli/snap-tests-global/migration-eslint-lintstagedrc/snap.txt
@@ -33,7 +33,6 @@ cat: .lintstagedrc.json: No such file or directory
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {
     "plugins": [
       "oxc",
diff --git a/packages/cli/snap-tests-global/migration-eslint/snap.txt b/packages/cli/snap-tests-global/migration-eslint/snap.txt
@@ -39,7 +39,6 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
-  fmt: {},
   lint: {
     "plugins": [
       "oxc",
diff --git a/packages/cli/snap-tests-global/migration-existing-husky-lint-staged/snap.txt b/packages/cli/snap-tests-global/migration-existing-husky-lint-staged/snap.txt
@@ -30,7 +30,6 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"
diff --git a/packages/cli/snap-tests-global/migration-existing-lint-staged-config/snap.txt b/packages/cli/snap-tests-global/migration-existing-lint-staged-config/snap.txt
@@ -33,7 +33,6 @@ cat: .lintstagedrc.json: No such file or directory
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.ts": "vp lint --fix"
diff --git a/packages/cli/snap-tests-global/migration-existing-pnpm-exec-lint-staged/snap.txt b/packages/cli/snap-tests-global/migration-existing-pnpm-exec-lint-staged/snap.txt
@@ -30,7 +30,6 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"
diff --git a/packages/cli/snap-tests-global/migration-from-tsdown-json-config/snap.txt b/packages/cli/snap-tests-global/migration-from-tsdown-json-config/snap.txt
@@ -24,7 +24,6 @@ export default defineConfig({
       "cwd": "./src"
     }
   },
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   server: {
     port: 3000,
@@ -75,7 +74,6 @@ export default defineConfig({
       "cwd": "./src"
     }
   },
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   server: {
     port: 3000,
diff --git a/packages/cli/snap-tests-global/migration-from-tsdown/snap.txt b/packages/cli/snap-tests-global/migration-from-tsdown/snap.txt
@@ -27,7 +27,6 @@ export default defineConfig({
     "*": "vp check --fix"
   },
   pack: tsdownConfig,
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
 });
 
@@ -79,7 +78,6 @@ export default defineConfig({
     "*": "vp check --fix"
   },
   pack: tsdownConfig,
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
 });
 
diff --git a/packages/cli/snap-tests-global/migration-lint-staged-in-scripts/snap.txt b/packages/cli/snap-tests-global/migration-lint-staged-in-scripts/snap.txt
@@ -31,7 +31,6 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"
diff --git a/packages/cli/snap-tests-global/migration-lintstagedrc-json/snap.txt b/packages/cli/snap-tests-global/migration-lintstagedrc-json/snap.txt
@@ -104,7 +104,6 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.@(js|ts|tsx|yml|yaml|md|json|html|toml)": [
diff --git a/packages/cli/snap-tests-global/migration-lintstagedrc-staged-exists/snap.txt b/packages/cli/snap-tests-global/migration-lintstagedrc-staged-exists/snap.txt
@@ -34,7 +34,6 @@ lintstagedrc.json still exists
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     '*.js': 'vp check --fix',
diff --git a/packages/cli/snap-tests-global/migration-merge-vite-config-js/snap.txt b/packages/cli/snap-tests-global/migration-merge-vite-config-js/snap.txt
@@ -12,7 +12,6 @@ export default {
   staged: {
     "*": "vp check --fix"
   },
-  fmt: {},
   lint: {
     "rules": {
       "no-unused-vars": "error"
diff --git a/packages/cli/snap-tests-global/migration-monorepo-pnpm-overrides-dependency-selector/snap.txt b/packages/cli/snap-tests-global/migration-monorepo-pnpm-overrides-dependency-selector/snap.txt
@@ -13,7 +13,6 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   plugins: [react()],
 });
diff --git a/packages/cli/snap-tests-global/migration-monorepo-yarn4/snap.txt b/packages/cli/snap-tests-global/migration-monorepo-yarn4/snap.txt
@@ -15,7 +15,6 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
-  fmt: {},
   lint: {
     "rules": {
       "no-unused-vars": "error"
diff --git a/packages/cli/snap-tests-global/migration-subpath/snap.txt b/packages/cli/snap-tests-global/migration-subpath/snap.txt
@@ -36,8 +36,8 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
-  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
+  
 });
 
 > git config --local core.hooksPath || echo 'core.hooksPath is not set' # should NOT be set
diff --git a/packages/cli/snap-tests/check-lint-fail-no-typecheck/vite.config.ts b/packages/cli/snap-tests/check-lint-fail-no-typecheck/vite.config.ts
@@ -1,5 +1,4 @@
 export default {
-  fmt: {},
   lint: {
     options: {
       typeAware: true,
diff --git a/packages/cli/snap-tests/check-lint-fail-typecheck/vite.config.ts b/packages/cli/snap-tests/check-lint-fail-typecheck/vite.config.ts
@@ -1,5 +1,4 @@
 export default {
-  fmt: {},
   lint: {
     options: {
       typeAware: true,
diff --git a/packages/cli/snap-tests/check-pass-no-typecheck/vite.config.ts b/packages/cli/snap-tests/check-pass-no-typecheck/vite.config.ts
@@ -1,5 +1,4 @@
 export default {
-  fmt: {},
   lint: {
     options: {
       typeAware: true,
diff --git a/packages/cli/snap-tests/check-pass-typecheck/vite.config.ts b/packages/cli/snap-tests/check-pass-typecheck/vite.config.ts
@@ -1,5 +1,4 @@
 export default {
-  fmt: {},
   lint: {
     options: {
       typeAware: true,
diff --git a/packages/cli/snap-tests/command-init-inline-config/snap.txt b/packages/cli/snap-tests/command-init-inline-config/snap.txt
@@ -5,7 +5,6 @@ Added 'lint' to 'vite.config.ts'.
 import { defineConfig } from "vite-plus";
 
 export default defineConfig({
-  fmt: {},
   lint: { options: { typeAware: true, typeCheck: true } },
 });
 
diff --git a/packages/cli/snap-tests/ignore_dist/vite.config.ts b/packages/cli/snap-tests/ignore_dist/vite.config.ts
@@ -1,5 +1,4 @@
 export default {
-  lint: {},
   run: {
     tasks: {
       lint: {
diff --git a/packages/cli/snap-tests/oxlint-typeaware/vite.config.ts b/packages/cli/snap-tests/oxlint-typeaware/vite.config.ts
@@ -1,5 +1,4 @@
 export default {
-  lint: {},
   run: {
     tasks: {
       lint: {
diff --git a/packages/cli/snap-tests/plain-terminal-ui-nested/vite.config.ts b/packages/cli/snap-tests/plain-terminal-ui-nested/vite.config.ts
@@ -1,5 +1,4 @@
 export default {
-  lint: {},
   run: {
     cache: true,
   },
diff --git a/packages/cli/snap-tests/workspace-lint-subpackage/packages/app-a/vite.config.ts b/packages/cli/snap-tests/workspace-lint-subpackage/packages/app-a/vite.config.ts
@@ -4,5 +4,4 @@ export default {
       'no-console': 'off',
     },
   },
-  fmt: {},
 };
diff --git a/packages/cli/snap-tests/workspace-lint-subpackage/vite.config.ts b/packages/cli/snap-tests/workspace-lint-subpackage/vite.config.ts
@@ -4,5 +4,4 @@ export default {
       'no-console': 'warn',
     },
   },
-  fmt: {},
 };
diff --git a/packages/cli/src/init-config.ts b/packages/cli/src/init-config.ts
@@ -138,26 +138,6 @@ function hasConfigKey(viteConfigPath: string, configKey: string): boolean {
   return new RegExp(`\\b${configKey}\\s*:`).test(viteConfig);
 }
 
-/**
- * Ensure `fmt: {}` exists in vite.config.ts so oxfmt auto-discovery doesn't
- * error when the config file lacks a `fmt` field.
- */
-function injectFmtDefaultsIfMissing(viteConfigPath: string): void {
-  if (hasConfigKey(viteConfigPath, 'fmt')) {
-    return;
-  }
-  const tempConfigPath = `${viteConfigPath}.fmt-init.json`;
-  fs.writeFileSync(tempConfigPath, '{}');
-  try {
-    const result = mergeJsonConfig(viteConfigPath, tempConfigPath, 'fmt');
-    if (result.updated) {
-      fs.writeFileSync(viteConfigPath, result.content);
-    }
-  } finally {
-    fs.rmSync(tempConfigPath, { force: true });
-  }
-}
-
 async function vpFmt(cwd: string, filePath: string): Promise<void> {
   const { binPath, envs } = await resolveFmt();
   const result = await runCommandSilently({
@@ -269,7 +249,6 @@ export async function applyToolInitConfigToViteConfig(
     if (generatedConfigPath) {
       fs.rmSync(generatedConfigPath, { force: true });
     }
-    injectFmtDefaultsIfMissing(viteConfigPath);
     await vpFmt(projectPath, path.relative(projectPath, viteConfigPath));
     return {
       handled: true,
@@ -297,7 +276,6 @@ export async function applyToolInitConfigToViteConfig(
 
   fs.writeFileSync(viteConfigPath, mergeResult.content);
   fs.rmSync(generatedConfigPath, { force: true });
-  injectFmtDefaultsIfMissing(viteConfigPath);
   await vpFmt(projectPath, path.relative(projectPath, viteConfigPath));
   return {
     handled: true,
diff --git a/packages/cli/src/migration/migrator.ts b/packages/cli/src/migration/migrator.ts
@@ -725,7 +725,6 @@ export function rewriteStandaloneProject(
   }
   mergeViteConfigFiles(projectPath, silent, report);
   injectLintTypeCheckDefaults(projectPath, silent, report);
-  injectFmtDefaults(projectPath, silent, report);
   mergeTsdownConfigFile(projectPath, silent, report);
   // rewrite imports in all TypeScript/JavaScript files
   rewriteAllImports(projectPath, silent, report);
@@ -771,7 +770,6 @@ export function rewriteMonorepo(
   }
   mergeViteConfigFiles(workspaceInfo.rootDir, silent, report);
   injectLintTypeCheckDefaults(workspaceInfo.rootDir, silent, report);
-  injectFmtDefaults(workspaceInfo.rootDir, silent, report);
   mergeTsdownConfigFile(workspaceInfo.rootDir, silent, report);
   // rewrite imports in all TypeScript/JavaScript files
   rewriteAllImports(workspaceInfo.rootDir, silent, report);
@@ -1327,26 +1325,6 @@ export function injectLintTypeCheckDefaults(
   );
 }
 
-/**
- * Inject an empty `fmt: {}` field into vite.config.ts if it doesn't already exist.
- * This is needed because oxfmt auto-discovers vite.config.ts and errors if
- * the `fmt` field is missing.
- */
-export function injectFmtDefaults(
-  projectPath: string,
-  silent = false,
-  report?: MigrationReport,
-): void {
-  injectConfigDefaults(
-    projectPath,
-    'fmt',
-    '.vite-plus-fmt-init.oxfmtrc.json',
-    '{}',
-    silent,
-    report,
-  );
-}
-
 function injectConfigDefaults(
   projectPath: string,
   configKey: string,
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/pnpm-workspace.yaml b/pnpm-workspace.yaml
