chore: New linting setup. (#579)

cpojer · web-flow · commit e37a72327e6b · 2026-02-10T15:04:15.000+09:00
This PR enables a number of lint rules + fixes up the violations.
diff --git a/docs/.vitepress/config.mts b/docs/.vitepress/config.mts
@@ -718,7 +718,7 @@ export default extendConfig(
         provider: 'local',
       },
     },
-    transformHead({ page, pageData, assets }) {
+    transformHead({ page, pageData }) {
       // Remove .md suffix and replace index with empty string (to cover index.md)
       const url = 'https://viteplus.dev/' + page.replace(/\.md$/, '').replace(/index$/, '');
 
diff --git a/docs/.vitepress/env.d.ts b/docs/.vitepress/env.d.ts
@@ -1,7 +1,7 @@
 // Vue SFC module declaration
 declare module '*.vue' {
   import type { DefineComponent } from 'vue';
-  const component: DefineComponent<{}, {}, any>;
+  const component: DefineComponent<{}, {}, unknown>;
   export default component;
 }
 
diff --git a/docs/.vitepress/theme/components/home/FeatureToolbar.vue b/docs/.vitepress/theme/components/home/FeatureToolbar.vue
@@ -21,7 +21,9 @@ const scrollToSection = (e: Event, id: string) => {
   e.stopPropagation();
 
   const element = document.getElementById(id);
-  if (!element) return;
+  if (!element) {
+    return;
+  }
 
   // Get the toolbar height to offset the scroll
   const toolbar = (e.currentTarget as HTMLElement).closest('section');
@@ -38,7 +40,9 @@ const scrollToSection = (e: Event, id: string) => {
   let startTime: number | null = null;
 
   const animation = (currentTime: number) => {
-    if (startTime === null) startTime = currentTime;
+    if (startTime === null) {
+      startTime = currentTime;
+    }
     const timeElapsed = currentTime - startTime;
     const progress = Math.min(timeElapsed / duration, 1);
 
@@ -89,7 +93,9 @@ const determineActiveSection = () => {
   const sections = features
     .map((feature) => {
       const element = document.getElementById(feature.id);
-      if (!element) return null;
+      if (!element) {
+        return null;
+      }
 
       const rect = element.getBoundingClientRect();
       return {
@@ -107,7 +113,9 @@ const determineActiveSection = () => {
   let activeId = activeSection.value;
 
   for (const section of sections) {
-    if (!section) continue;
+    if (!section) {
+      continue;
+    }
 
     // If section top is near or above threshold and bottom is below threshold
     if (section.top <= threshold && section.bottom > threshold) {
diff --git a/docs/.vitepress/theme/components/home/Hero.vue b/docs/.vitepress/theme/components/home/Hero.vue
@@ -49,7 +49,9 @@ const smoothScrollTo = (e: Event, targetId: string) => {
   e.stopPropagation();
 
   const element = document.getElementById(targetId);
-  if (!element) return;
+  if (!element) {
+    return;
+  }
 
   const elementPosition = element.getBoundingClientRect().top + window.scrollY;
   const offsetPosition = elementPosition;
@@ -61,7 +63,9 @@ const smoothScrollTo = (e: Event, targetId: string) => {
   let startTime: number | null = null;
 
   const animation = (currentTime: number) => {
-    if (startTime === null) startTime = currentTime;
+    if (startTime === null) {
+      startTime = currentTime;
+    }
     const timeElapsed = currentTime - startTime;
     const progress = Math.min(timeElapsed / duration, 1);
 
diff --git a/docs/.vitepress/theme/components/home/Terminal.vue b/docs/.vitepress/theme/components/home/Terminal.vue
@@ -32,7 +32,9 @@ const goToNextTab = () => {
 
 // Handle animation completion
 const onAnimationComplete = () => {
-  if (!autoPlayEnabled.value) return;
+  if (!autoPlayEnabled.value) {
+    return;
+  }
 
   // Clear any existing timeout
   if (autoAdvanceTimeout) {
@@ -59,7 +61,9 @@ const onTabChange = () => {
 
 // Setup Intersection Observer
 onMounted(() => {
-  if (!sectionRef.value) return;
+  if (!sectionRef.value) {
+    return;
+  }
 
   observer = new IntersectionObserver(
     (entries) => {
diff --git a/packages/cli/build.ts b/packages/cli/build.ts
@@ -237,7 +237,9 @@ async function syncCorePackageExports() {
     const srcPath = join(coreClientDir, file);
     const shimPath = join(clientDir, file);
     // Skip directories
-    if (statSync(srcPath).isDirectory()) continue;
+    if (statSync(srcPath).isDirectory()) {
+      continue;
+    }
     if (file.endsWith('.js') || file.endsWith('.mjs') || file.endsWith('.cjs')) {
       await writeFile(shimPath, `export * from '${CORE_PACKAGE_NAME}/dist/client/${file}';\n`);
     } else if (file.endsWith('.d.ts') || file.endsWith('.d.mts') || file.endsWith('.d.cts')) {
@@ -288,7 +290,9 @@ async function syncTypesDir(srcDir: string, destDir: string, relativePath: strin
 
     if (statSync(srcPath).isDirectory()) {
       // Skip top-level internal directory - it's blocked by ./types/internal/* export
-      if (entry === 'internal' && relativePath === '') continue;
+      if (entry === 'internal' && relativePath === '') {
+        continue;
+      }
 
       await mkdir(destPath, { recursive: true });
       await syncTypesDir(srcPath, destPath, entryRelPath);
diff --git a/packages/cli/src/oxlint-config.ts b/packages/cli/src/oxlint-config.ts
@@ -572,7 +572,9 @@ function jsToJSONProps(typ: any): any {
 
 function transform(val: any, typ: any, getProps: any, key: any = '', parent: any = ''): any {
   function transformPrimitive(typ: string, val: any): any {
-    if (typeof typ === typeof val) return val;
+    if (typeof typ === typeof val) {
+      return val;
+    }
     return invalidValue(typ, val, key, parent);
   }
 
@@ -589,7 +591,9 @@ function transform(val: any, typ: any, getProps: any, key: any = '', parent: any
   }
 
   function transformEnum(cases: string[], val: any): any {
-    if (cases.indexOf(val) !== -1) return val;
+    if (cases.indexOf(val) !== -1) {
+      return val;
+    }
     return invalidValue(
       cases.map((a) => {
         return l(a);
@@ -602,7 +606,9 @@ function transform(val: any, typ: any, getProps: any, key: any = '', parent: any
 
   function transformArray(typ: any, val: any): any {
     // val must be an array with no invalid elements
-    if (!Array.isArray(val)) return invalidValue(l('array'), val, key, parent);
+    if (!Array.isArray(val)) {
+      return invalidValue(l('array'), val, key, parent);
+    }
     return val.map((el) => transform(el, typ, getProps));
   }
 
@@ -635,18 +641,26 @@ function transform(val: any, typ: any, getProps: any, key: any = '', parent: any
     return result;
   }
 
-  if (typ === 'any') return val;
+  if (typ === 'any') {
+    return val;
+  }
   if (typ === null) {
-    if (val === null) return val;
+    if (val === null) {
+      return val;
+    }
+    return invalidValue(typ, val, key, parent);
+  }
+  if (typ === false) {
     return invalidValue(typ, val, key, parent);
   }
-  if (typ === false) return invalidValue(typ, val, key, parent);
   let ref: any = undefined;
   while (typeof typ === 'object' && typ.ref !== undefined) {
     ref = typ.ref;
     typ = typeMap[typ.ref];
   }
-  if (Array.isArray(typ)) return transformEnum(typ, val);
+  if (Array.isArray(typ)) {
+    return transformEnum(typ, val);
+  }
   if (typeof typ === 'object') {
     return typ.hasOwnProperty('unionMembers')
       ? transformUnion(typ.unionMembers, val)
@@ -657,7 +671,9 @@ function transform(val: any, typ: any, getProps: any, key: any = '', parent: any
           : invalidValue(typ, val, key, parent);
   }
   // Numbers can be parsed by Date but shouldn't be.
-  if (typ === Date && typeof val !== 'number') return transformDate(val);
+  if (typ === Date && typeof val !== 'number') {
+    return transformDate(val);
+  }
   return transformPrimitive(typ, val);
 }
 
diff --git a/packages/cli/src/pack-bin.ts b/packages/cli/src/pack-bin.ts
@@ -75,7 +75,9 @@ cli
   .option('-F, --filter <pattern>', 'Filter configs (cwd or name), e.g. /pkg-name$/ or pkg-name')
   .option('--exports', 'Generate export-related metadata for package.json (experimental)')
   .action(async (input: string[], flags: InlineConfig) => {
-    if (input.length > 0) flags.entry = input;
+    if (input.length > 0) {
+      flags.entry = input;
+    }
     if (flags.envPrefix === undefined) {
       flags.envPrefix = DEFAULT_ENV_PREFIXES;
     }
@@ -84,7 +86,9 @@ cli
       const viteConfig = await resolveConfig({ root: process.cwd() }, 'build');
 
       const configFiles: string[] = [];
-      if (viteConfig.configFile) configFiles.push(viteConfig.configFile);
+      if (viteConfig.configFile) {
+        configFiles.push(viteConfig.configFile);
+      }
 
       const configs: ResolvedConfig[] = [];
       const packConfigs = Array.isArray(viteConfig.pack)
@@ -108,8 +112,8 @@ export async function runCLI(): Promise<void> {
 
   try {
     await cli.runMatchedCommand();
-  } catch (error: any) {
-    globalLogger.error(String(error.stack || error.message));
+  } catch (error) {
+    globalLogger.error(error instanceof Error ? error.stack || error.message : error);
     process.exit(1);
   }
 }
diff --git a/packages/core/build-support/rewrite-module-specifiers.ts b/packages/core/build-support/rewrite-module-specifiers.ts
@@ -58,8 +58,12 @@ function getQuoteChar(node: SgNode): string {
  * Matches exact, subpath (from/...), or file extension (from.xxx)
  */
 function matchesFrom(specifier: string, from: string): boolean {
-  if (specifier === from) return true;
-  if (!specifier.startsWith(from)) return false;
+  if (specifier === from) {
+    return true;
+  }
+  if (!specifier.startsWith(from)) {
+    return false;
+  }
   // Check the character after the prefix - must be '/', '.', or end of string
   const nextChar = specifier[from.length];
   return nextChar === '/' || nextChar === '.';
diff --git a/packages/core/build.ts b/packages/core/build.ts
@@ -339,7 +339,9 @@ async function bundleVitepress() {
 
   for (const file of vitepressDistFiles) {
     const stats = await stat(file);
-    if (!stats.isFile()) continue;
+    if (!stats.isFile()) {
+      continue;
+    }
 
     // Normalize paths to use forward slashes for consistent replacement on Windows
     const relativePath = toPosixPath(file).replace(
@@ -392,7 +394,9 @@ async function bundleVitepress() {
 
   for (const file of vitepressTypesFiles) {
     const stats = await stat(file);
-    if (!stats.isFile()) continue;
+    if (!stats.isFile()) {
+      continue;
+    }
 
     // Normalize paths to use forward slashes for consistent replacement on Windows
     const relativePath = toPosixPath(file).replace(toPosixPath(vitepressTypesDir), '');
diff --git a/packages/global/src/create/prompts.ts b/packages/global/src/create/prompts.ts
@@ -20,10 +20,14 @@ export async function promptPackageNameAndTargetDir(
       placeholder: defaultPackageName,
       defaultValue: defaultPackageName,
       validate: (value) => {
-        if (value.length === 0) return;
+        if (value.length === 0) {
+          return;
+        }
 
         const result = validateNpmPackageName(value);
-        if (result.validForNewPackages) return;
+        if (result.validForNewPackages) {
+          return;
+        }
         return result.errors?.[0] ?? result.warnings?.[0] ?? 'Invalid package name';
       },
     });
diff --git a/packages/global/src/utils/json.ts b/packages/global/src/utils/json.ts
@@ -3,12 +3,12 @@ import fs from 'node:fs';
 import detectIndent from 'detect-indent';
 import { detectNewline } from 'detect-newline';
 
-export function readJsonFile<T = Record<string, any>>(file: string): T {
+export function readJsonFile<T = Record<string, unknown>>(file: string): T {
   const content = fs.readFileSync(file, 'utf-8');
   return JSON.parse(content) as T;
 }
 
-export function writeJsonFile<T = Record<string, any>>(file: string, data: T) {
+export function writeJsonFile<T = Record<string, unknown>>(file: string, data: T) {
   let newline = '\n';
   let indent = '  ';
   if (fs.existsSync(file)) {
@@ -20,7 +20,7 @@ export function writeJsonFile<T = Record<string, any>>(file: string, data: T) {
   fs.writeFileSync(file, JSON.stringify(data, null, indent) + newline, 'utf-8');
 }
 
-export function editJsonFile<T = Record<string, any>>(
+export function editJsonFile<T = Record<string, unknown>>(
   file: string,
   callback: (content: T) => T | undefined,
 ) {
diff --git a/packages/global/src/utils/package.ts b/packages/global/src/utils/package.ts
@@ -43,7 +43,7 @@ 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, any>>(currentDir: string): T | null {
+export function readNearestPackageJson<T = Record<string, unknown>>(currentDir: string): T | null {
   do {
     const packageJsonPath = path.join(currentDir, 'package.json');
     if (fs.existsSync(packageJsonPath)) {
diff --git a/packages/global/src/utils/terminal.ts b/packages/global/src/utils/terminal.ts
@@ -41,15 +41,15 @@ function fadeToColor(count: number, endRgb: RGB) {
   return colors;
 }
 
+const lerp = (a: number, b: number, t: number) => a + (b - a) * t;
+
 function gradient(count: number, startRgb: RGB, endRgb: RGB) {
   const n = Math.max(count, 1);
   const denom = Math.max(n - 1, 1);
 
   const [sr, sg, sb] = startRgb;
   const [er, eg, eb] = endRgb;
 
-  const lerp = (a: number, b: number, t: number) => a + (b - a) * t;
-
   const colors: Array<RGB> = [];
   for (let i = 0; i < n; i++) {
     const t = i / denom;
@@ -80,6 +80,17 @@ function colorize(text: string, colors: Array<RGB>) {
   return out + RESET;
 }
 
+const to8bit = (hex: string) => {
+  if (hex.length === 2) {
+    return parseInt(hex, 16);
+  }
+  if (hex.length === 4) {
+    return Math.round((parseInt(hex, 16) / 0xffff) * 255);
+  }
+  const max = Math.pow(16, hex.length) - 1;
+  return Math.round((parseInt(hex, 16) / max) * 255);
+};
+
 async function getForegroundColor(): Promise<null | RGB> {
   const stdin = process.stdin;
   const stdout = process.stdout;
@@ -129,17 +140,6 @@ async function getForegroundColor(): Promise<null | RGB> {
       }, 50);
     };
 
-    const to8bit = (hex: string) => {
-      if (hex.length === 2) {
-        return parseInt(hex, 16);
-      }
-      if (hex.length === 4) {
-        return Math.round((parseInt(hex, 16) / 0xffff) * 255);
-      }
-      const max = Math.pow(16, hex.length) - 1;
-      return Math.round((parseInt(hex, 16) / max) * 255);
-    };
-
     const onData = (data: string) => {
       buffer += data;
       scheduleFlush();
diff --git a/packages/global/src/utils/workspace.ts b/packages/global/src/utils/workspace.ts
@@ -83,7 +83,7 @@ export async function detectWorkspace(rootDir: string): Promise<WorkspaceInfoOpt
         dirs.add(dir);
       }
     }
-    result.parentDirs = Array.from(dirs).sort();
+    result.parentDirs = Array.from(dirs).toSorted();
 
     // Extract the scope from the package.json
     const pkg = readJsonFile<{ name?: string }>(packageJsonFile);
diff --git a/packages/global/src/utils/yaml.ts b/packages/global/src/utils/yaml.ts
@@ -2,7 +2,7 @@ import fs from 'node:fs';
 
 import { type Document, type ParsedNode, parseDocument, parse as parseYaml, Scalar } from 'yaml';
 
-export function readYamlFile<T = Record<string, any>>(file: string): T {
+export function readYamlFile<T = Record<string, unknown>>(file: string): T {
   const content = fs.readFileSync(file, 'utf-8');
   return parseYaml(content) as T;
 }
diff --git a/packages/global/src/version.ts b/packages/global/src/version.ts
diff --git a/packages/test/build.ts b/packages/test/build.ts
diff --git a/packages/tools/src/index.ts b/packages/tools/src/index.ts
diff --git a/packages/tools/src/json-sort.ts b/packages/tools/src/json-sort.ts
diff --git a/packages/tools/src/merge-peer-deps.ts b/packages/tools/src/merge-peer-deps.ts
diff --git a/packages/tools/src/snap-test.ts b/packages/tools/src/snap-test.ts
diff --git a/packages/tools/src/sync-remote-deps.ts b/packages/tools/src/sync-remote-deps.ts
diff --git a/vite.config.ts b/vite.config.ts

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`// Vue SFC module declaration`
`2`	`2`	`declare module '*.vue' {`
`3`	`3`	`import type { DefineComponent } from 'vue';`
`4`		`- const component: DefineComponent<{}, {}, any>;`
	`4`	`+ const component: DefineComponent<{}, {}, unknown>;`
`5`	`5`	`export default component;`
`6`	`6`	`}`
`7`	`7`