feat(deps): upgrade upstream dependencies (#1375)

Brooooooklyn · github-actions[bot] · fengmk2 · web-flow · commit 48e49ca0c1c0 · 2026-04-14T11:45:51.000+08:00
Automated daily upgrade of upstream dependencies: - rolldown (latest tag) - vite (latest tag) - vitest (latest npm version) - tsdown (latest npm version) Build status: success  --- > [!NOTE] > **Medium Risk** > Mostly dependency bumps and snapshot updates, but upgrading `oxlint`/`oxfmt` can subtly change CLI behavior and output (including lint exit conditions), which may affect CI or user workflows. > > **Overview** > Upgrades upstream tooling versions (notably `oxfmt` `0.45.0`, `oxlint` `1.60.0`, and `tsdown` `0.21.8`, plus related lockfile updates) and aligns `packages/core` peer/bundled versions accordingly. > > Updates CLI snapshot tests to match new outputs/help text (including the new `--no-error-on-unmatched-pattern` option and updated lint/no-files messaging) and applies small lint-suppression comments for generic JSON/YAML/package helpers to satisfy stricter rules. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit 8249925. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup>  --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: MK <fengmk2@gmail.com>
diff --git a/packages/cli/snap-tests-global/migration-prettier-eslint-combo/snap.txt b/packages/cli/snap-tests-global/migration-prettier-eslint-combo/snap.txt
@@ -46,7 +46,7 @@ peerDependencyRules:
 > test ! -f eslint.config.mjs # check eslint config is removed
 > test ! -f .prettierrc.json # check prettier config is removed
 > cat vite.config.ts # check oxlint and oxfmt config merged into vite.config.ts
-import { defineConfig } from 'vite-plus';
+import { defineConfig } from "vite-plus";
 
 export default defineConfig({
   staged: {
diff --git a/packages/cli/snap-tests-global/migration-prettier-pkg-json/snap.txt b/packages/cli/snap-tests-global/migration-prettier-pkg-json/snap.txt
@@ -40,7 +40,7 @@ peerDependencyRules:
     vitest: '*'
 
 > cat vite.config.ts # check oxfmt config merged into vite.config.ts with semi/singleQuote settings
-import { defineConfig } from 'vite-plus';
+import { defineConfig } from "vite-plus";
 
 export default defineConfig({
   staged: {
diff --git a/packages/cli/snap-tests-global/migration-prettier-rerun/snap.txt b/packages/cli/snap-tests-global/migration-prettier-rerun/snap.txt
@@ -26,7 +26,7 @@ Prettier config migrated to .oxfmtrc.json
 
 > test ! -f .prettierrc.json # check prettier config is removed
 > cat vite.config.ts # check oxfmt config merged into vite.config.ts
-import { defineConfig } from 'vite-plus';
+import { defineConfig } from "vite-plus";
 
 export default defineConfig({
   fmt: {
diff --git a/packages/cli/snap-tests-global/migration-prettier/snap.txt b/packages/cli/snap-tests-global/migration-prettier/snap.txt
@@ -43,7 +43,7 @@ peerDependencyRules:
 
 > test ! -f .prettierrc.json # check prettier config is removed
 > cat vite.config.ts # check oxfmt config merged into vite.config.ts
-import { defineConfig } from 'vite-plus';
+import { defineConfig } from "vite-plus";
 
 export default defineConfig({
   staged: {
diff --git a/packages/cli/snap-tests-global/runtime-with-incompatible-env-node/steps.json b/packages/cli/snap-tests-global/runtime-with-incompatible-env-node/steps.json
@@ -1,4 +1,5 @@
 {
+  "serial": true,
   "ignoredPlatforms": ["win32"],
   "commands": ["vp env use 20.0.0", "vp exec node --version"],
   "after": ["vp env use --unset"]
diff --git a/packages/cli/snap-tests-global/runtime-with-incompatible-global-node/steps.json b/packages/cli/snap-tests-global/runtime-with-incompatible-global-node/steps.json
@@ -1,4 +1,5 @@
 {
+  "serial": true,
   "env": {},
   "commands": ["vp env default 20.0.0", "vp migrate --yes"],
   "after": ["vp env default lts"]
diff --git a/packages/cli/snap-tests/bin-oxlint-wrapper/snap.txt b/packages/cli/snap-tests/bin-oxlint-wrapper/snap.txt
@@ -82,6 +82,8 @@ Output
 
 Miscellaneous
         --silent              Do not display any diagnostics
+        --no-error-on-unmatched-pattern  Do not exit with an error when no files are selected for
+                              linting (for example, after applying ignore patterns)
         --threads=INT         Number of threads to use. Set to 1 for using only 1 CPU core.
         --print-config        This option outputs the configuration to be used. When present, no
                               linting is performed and only config-related options are valid.
diff --git a/packages/cli/snap-tests/check-oxlint-env/src/index.ts b/packages/cli/snap-tests/check-oxlint-env/src/index.ts
@@ -0,0 +1,3 @@
+export function hello(): string {
+  return 'hello';
+}
diff --git a/packages/cli/snap-tests/check-oxlint-env/tsconfig.json b/packages/cli/snap-tests/check-oxlint-env/tsconfig.json
@@ -0,0 +1,10 @@
+{
+  "compilerOptions": {
+    "target": "ES2022",
+    "module": "ESNext",
+    "moduleResolution": "bundler",
+    "strict": true,
+    "skipLibCheck": true
+  },
+  "include": ["src"]
+}
diff --git a/packages/cli/snap-tests/check-oxlint-env/vite.config.ts b/packages/cli/snap-tests/check-oxlint-env/vite.config.ts
@@ -0,0 +1,8 @@
+export default {
+  lint: {
+    options: {
+      typeAware: true,
+      typeCheck: true,
+    },
+  },
+};
diff --git a/packages/cli/snap-tests/command-helper/snap.txt b/packages/cli/snap-tests/command-helper/snap.txt
@@ -193,6 +193,8 @@ Output
 
 Miscellaneous
         --silent              Do not display any diagnostics
+        --no-error-on-unmatched-pattern  Do not exit with an error when no files are selected for
+                              linting (for example, after applying ignore patterns)
         --threads=INT         Number of threads to use. Set to 1 for using only 1 CPU core.
         --print-config        This option outputs the configuration to be used. When present, no
                               linting is performed and only config-related options are valid.
diff --git a/packages/cli/src/create/discovery.ts b/packages/cli/src/create/discovery.ts
@@ -83,11 +83,11 @@ export function discoverTemplate(
   if (localPackage) {
     const localPackagePath = path.join(workspaceInfo.rootDir, localPackage.path);
     const packageJsonPath = path.join(localPackagePath, 'package.json');
-    const pkg = readJsonFile<{
+    const pkg = readJsonFile(packageJsonPath) as {
       dependencies?: Record<string, string>;
       keywords?: string[];
       bin?: Record<string, string> | string;
-    }>(packageJsonPath);
+    };
     let binPath = '';
     if (pkg.bin) {
       if (typeof pkg.bin === 'string') {
diff --git a/packages/cli/src/migration/bin.ts b/packages/cli/src/migration/bin.ts
@@ -846,7 +846,9 @@ async function main() {
 
   // Early return if already using Vite+ (only ESLint/hooks migration may be needed)
   // In force-override mode (file: tgz overrides), skip this check and run full migration
-  const rootPkg = readNearestPackageJson<PackageDependencies>(workspaceInfoOptional.rootDir);
+  const rootPkg = readNearestPackageJson(
+    workspaceInfoOptional.rootDir,
+  ) as PackageDependencies | null;
   if (hasVitePlusDependency(rootPkg) && !isForceOverrideMode()) {
     let didMigrate = false;
     let installDurationMs = 0;
diff --git a/packages/cli/src/migration/migrator.ts b/packages/cli/src/migration/migrator.ts
@@ -143,10 +143,10 @@ export function detectEslintProject(
   if (!fs.existsSync(packageJsonPath)) {
     return { hasDependency: false };
   }
-  const pkg = readJsonFile<{
+  const pkg = readJsonFile(packageJsonPath) as {
     devDependencies?: Record<string, string>;
     dependencies?: Record<string, string>;
-  }>(packageJsonPath);
+  };
   let hasDependency = !!(pkg.devDependencies?.eslint || pkg.dependencies?.eslint);
   const configs = detectConfigs(projectPath);
   let configFile = configs.eslintConfig;
@@ -159,10 +159,10 @@ export function detectEslintProject(
       if (!fs.existsSync(pkgJsonPath)) {
         continue;
       }
-      const wpPkg = readJsonFile<{
+      const wpPkg = readJsonFile(pkgJsonPath) as {
         devDependencies?: Record<string, string>;
         dependencies?: Record<string, string>;
-      }>(pkgJsonPath);
+      };
       if (wpPkg.devDependencies?.eslint || wpPkg.dependencies?.eslint) {
         hasDependency = true;
         break;
@@ -404,10 +404,10 @@ export function detectPrettierProject(
   if (!fs.existsSync(packageJsonPath)) {
     return { hasDependency: false };
   }
-  const pkg = readJsonFile<{
+  const pkg = readJsonFile(packageJsonPath) as {
     devDependencies?: Record<string, string>;
     dependencies?: Record<string, string>;
-  }>(packageJsonPath);
+  };
   let hasDependency = !!(pkg.devDependencies?.prettier || pkg.dependencies?.prettier);
   const configs = detectConfigs(projectPath);
   const configFile = configs.prettierConfig;
@@ -419,10 +419,10 @@ export function detectPrettierProject(
       if (!fs.existsSync(pkgJsonPath)) {
         continue;
       }
-      const wpPkg = readJsonFile<{
+      const wpPkg = readJsonFile(pkgJsonPath) as {
         devDependencies?: Record<string, string>;
         dependencies?: Record<string, string>;
-      }>(pkgJsonPath);
+      };
       if (wpPkg.devDependencies?.prettier || wpPkg.dependencies?.prettier) {
         hasDependency = true;
         break;
@@ -498,7 +498,7 @@ export async function migratePrettierToOxfmt(
     // so that `vp fmt --migrate=prettier` can read it
     if (prettierConfigFile === PRETTIER_PACKAGE_JSON_CONFIG) {
       const packageJsonPath = path.join(projectPath, 'package.json');
-      const pkg = readJsonFile<{ prettier?: unknown }>(packageJsonPath);
+      const pkg = readJsonFile(packageJsonPath) as { prettier?: unknown };
       if (pkg.prettier) {
         tempPrettierConfig = path.join(projectPath, '.prettierrc.json');
         fs.writeFileSync(tempPrettierConfig, JSON.stringify(pkg.prettier, null, 2));
@@ -1582,7 +1582,7 @@ export function mergeViteConfigFiles(
   if (configs.oxlintConfig) {
     // Inject options.typeAware and options.typeCheck defaults before merging
     const fullOxlintPath = path.join(projectPath, configs.oxlintConfig);
-    const oxlintJson = readJsonFile<{ options?: Record<string, unknown> }>(fullOxlintPath, true);
+    const oxlintJson = readJsonFile(fullOxlintPath, true) as { options?: Record<string, unknown> };
     if (!oxlintJson.options) {
       oxlintJson.options = {};
     }
@@ -1919,7 +1919,7 @@ export function getOldHooksDir(rootDir: string): string | undefined {
   if (!fs.existsSync(packageJsonPath)) {
     return undefined;
   }
-  const pkg = readJsonFile<{ scripts?: { prepare?: string } }>(packageJsonPath);
+  const pkg = readJsonFile(packageJsonPath) as { scripts?: { prepare?: string } };
   if (!pkg.scripts?.prepare) {
     return undefined;
   }
@@ -2023,9 +2023,9 @@ export function setupGitHooks(
   const hasStandaloneConfig = hasStandaloneLintStagedConfig(projectPath);
   if (!stagedMerged && !hasStandaloneConfig) {
     // Use lint-staged config from package.json if available, otherwise use default
-    const pkgData = readJsonFile<{ 'lint-staged'?: Record<string, string | string[]> }>(
-      packageJsonPath,
-    );
+    const pkgData = readJsonFile(packageJsonPath) as {
+      'lint-staged'?: Record<string, string | string[]>;
+    };
     const stagedConfig = pkgData?.['lint-staged'] ?? DEFAULT_STAGED_CONFIG;
     const updated = rewriteScripts(JSON.stringify(stagedConfig), readRulesYaml());
     const finalConfig: Record<string, string | string[]> = updated
diff --git a/packages/cli/src/utils/json.ts b/packages/cli/src/utils/json.ts
@@ -4,16 +4,13 @@ import detectIndent from 'detect-indent';
 import { detectNewline } from 'detect-newline';
 import { parse as parseJsonc } from 'jsonc-parser';
 
-export function readJsonFile<T = Record<string, unknown>>(
-  file: string,
-  allowComments?: boolean,
-): T {
+export function readJsonFile(file: string, allowComments?: boolean): Record<string, unknown> {
   const content = fs.readFileSync(file, 'utf-8');
   const parseFunction = allowComments ? parseJsonc : JSON.parse;
-  return parseFunction(content) as T;
+  return parseFunction(content);
 }
 
-export function writeJsonFile<T = Record<string, unknown>>(file: string, data: T) {
+export function writeJsonFile(file: string, data: Record<string, unknown>) {
   let newline = '\n';
   let indent = '  ';
   if (fs.existsSync(file)) {
@@ -25,11 +22,11 @@ export function writeJsonFile<T = Record<string, unknown>>(file: string, data: T
   fs.writeFileSync(file, JSON.stringify(data, null, indent) + newline, 'utf-8');
 }
 
-export function editJsonFile<T = Record<string, unknown>>(
+export function editJsonFile<T extends Record<string, unknown> = Record<string, unknown>>(
   file: string,
   callback: (content: T) => T | undefined,
 ) {
-  const json = readJsonFile<T>(file);
+  const json = readJsonFile(file) as T;
   const newJson = callback(json);
   if (newJson) {
     writeJsonFile(file, newJson);
diff --git a/packages/cli/src/utils/package.ts b/packages/cli/src/utils/package.ts
@@ -44,11 +44,11 @@ export function detectPackageMetadata(
  * @param currentDir - The current directory to start searching from.
  * @returns The package.json content as a JSON object, or null if no package.json is found.
  */
-export function readNearestPackageJson<T = Record<string, unknown>>(currentDir: string): T | null {
+export function readNearestPackageJson(currentDir: string): Record<string, unknown> | null {
   do {
     const packageJsonPath = path.join(currentDir, 'package.json');
     if (fs.existsSync(packageJsonPath)) {
-      return readJsonFile<T>(packageJsonPath);
+      return readJsonFile(packageJsonPath);
     }
     currentDir = path.dirname(currentDir);
   } while (currentDir !== path.dirname(currentDir));
diff --git a/packages/cli/src/utils/workspace.ts b/packages/cli/src/utils/workspace.ts
@@ -62,13 +62,13 @@ export async function detectWorkspace(rootDir: string): Promise<WorkspaceInfoOpt
     const pnpmWorkspaceFile = path.join(result.rootDir, 'pnpm-workspace.yaml');
     const packageJsonFile = path.join(result.rootDir, 'package.json');
     if (fs.existsSync(pnpmWorkspaceFile)) {
-      const workspaceConfig = readYamlFile<{ packages?: string[] }>(pnpmWorkspaceFile);
+      const workspaceConfig = readYamlFile(pnpmWorkspaceFile) as { packages?: string[] };
       if (Array.isArray(workspaceConfig.packages)) {
         result.workspacePatterns = workspaceConfig.packages;
       }
     } else if (fs.existsSync(packageJsonFile)) {
       // Check for npm/yarn/bun workspace (array or object form)
-      const pkg = readJsonFile<{ workspaces?: NpmWorkspaces }>(packageJsonFile);
+      const pkg = readJsonFile(packageJsonFile) as { workspaces?: NpmWorkspaces };
       if (Array.isArray(pkg.workspaces)) {
         result.workspacePatterns = pkg.workspaces;
       } else if (pkg.workspaces && Array.isArray(pkg.workspaces.packages)) {
@@ -92,7 +92,7 @@ export async function detectWorkspace(rootDir: string): Promise<WorkspaceInfoOpt
     result.parentDirs = Array.from(dirs).sort();
 
     // Extract the scope from the package.json
-    const pkg = readJsonFile<{ name?: string }>(packageJsonFile);
+    const pkg = readJsonFile(packageJsonFile) as { name?: string };
     if (pkg.name) {
       result.monorepoScope = getScopeFromPackageName(pkg.name);
     }
@@ -124,13 +124,13 @@ export function discoverWorkspacePackages(
   );
   for (const packageJsonRelativePath of packageJsonRelativePaths) {
     const packageJsonPath = path.join(rootDir, packageJsonRelativePath);
-    const pkg = readJsonFile<{
+    const pkg = readJsonFile(packageJsonPath) as {
       name?: string;
       description?: string;
       version?: string;
       dependencies?: Record<string, string>;
       keywords?: string[];
-    }>(packageJsonPath);
+    };
     if (!pkg.name) {
       continue;
     }
diff --git a/packages/cli/src/utils/yaml.ts b/packages/cli/src/utils/yaml.ts
@@ -2,9 +2,9 @@ import fs from 'node:fs';
 
 import { type Document, parseDocument, parse as parseYaml, Scalar } from 'yaml';
 
-export function readYamlFile<T = Record<string, unknown>>(file: string): T {
+export function readYamlFile(file: string): Record<string, unknown> {
   const content = fs.readFileSync(file, 'utf-8');
-  return parseYaml(content) as T;
+  return parseYaml(content);
 }
 
 export type YamlDocument = Document.Parsed;
diff --git a/packages/core/package.json b/packages/core/package.json
@@ -136,8 +136,8 @@
   },
   "peerDependencies": {
     "@arethetypeswrong/core": "^0.18.1",
-    "@tsdown/css": "0.21.7",
-    "@tsdown/exe": "0.21.7",
+    "@tsdown/css": "0.21.8",
+    "@tsdown/exe": "0.21.8",
     "@types/node": "^20.19.0 || >=22.12.0",
     "@vitejs/devtools": "^0.1.0",
     "esbuild": "^0.27.0 || ^0.28.0",
@@ -219,6 +219,6 @@
   "bundledVersions": {
     "vite": "8.0.8",
     "rolldown": "1.0.0-rc.15",
-    "tsdown": "0.21.7"
+    "tsdown": "0.21.8"
   }
 }
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/pnpm-workspace.yaml b/pnpm-workspace.yaml

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`{`
	`2`	`+ "serial": true,`
`2`	`3`	`"ignoredPlatforms": ["win32"],`
`3`	`4`	`"commands": ["vp env use 20.0.0", "vp exec node --version"],`
`4`	`5`	`"after": ["vp env use --unset"]`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`{`
	`2`	`+ "serial": true,`
`2`	`3`	`"env": {},`
`3`	`4`	`"commands": ["vp env default 20.0.0", "vp migrate --yes"],`
`4`	`5`	`"after": ["vp env default lts"]`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+export function hello(): string {`
	`2`	`+ return 'hello';`
	`3`	`+}`