Enable type-aware linting with oxlint (#2175)

Author: Copilot

.changeset/loud-laws-approve.md

@@ -0,0 +1,5 @@
+---
+"@sanity/pkg-utils": patch
+---
+
+Fix issues uncovered by type-aware linting with oxlint

.oxlintrc.json

@@ -27,11 +27,30 @@
       }
     },
     {
-      "files": ["playground/**", "test/**", "packages/**/test/**"],
+      "files": [
+        "playground/**",
+        "test/**",
+        "packages/**/test/**",
+        "**/*.test.ts",
+        "**/*.test.js",
+        "**/*.test.tsx",
+        "**/*.test.jsx",
+        "**/test.cjs"
+      ],
       "rules": {
         "eslint/no-console": "off",
         "eslint/no-unused-vars": "off",
-        "triple-slash-reference": "off"
+        "triple-slash-reference": "off",
+        "await-thenable": "off",
+        "no-floating-promises": "off",
+        "no-unnecessary-boolean-literal-compare": "off",
+        "no-unnecessary-template-expression": "off",
+        "no-unnecessary-type-arguments": "off",
+        "no-unnecessary-type-assertion": "off",
+        "no-unsafe-enum-comparison": "off",
+        "no-unsafe-type-assertion": "off",
+        "restrict-template-expressions": "off",
+        "unbound-method": "off"
       }
     }
   ]

package.json

@@ -5,8 +5,8 @@
   "type": "module",
   "scripts": {
     "build": "pnpm --filter '@sanity/pkg-utils' build",
-    "format": "prettier --write --cache --ignore-unknown . && oxlint --fix --quiet",
-    "lint": "oxlint",
+    "format": "prettier --write --cache --ignore-unknown . && oxlint --type-aware --fix --quiet",
+    "lint": "oxlint --type-aware",
     "playground:build": "pnpm --filter './playground/**' build",
     "playground:clean": "pnpm --filter './playground/**' --silent clean",
     "playground:typecheck": "pnpm --filter './playground/**' typecheck",
@@ -19,7 +19,7 @@
   "lint-staged": {
     "*.{js,jsx,ts,tsx,mjs,cjs}": [
       "prettier --cache --write",
-      "oxlint --fix --quiet"
+      "oxlint --type-aware --fix --quiet"
     ],
     "*": [
       "prettier --write --cache --ignore-unknown"
@@ -36,6 +36,7 @@
     "lint-staged": "^16.2.6",
     "npm-run-all2": "^8.0.4",
     "oxlint": "^1.26.0",
+    "oxlint-tsgolint": "^0.5.0",
     "prettier": "^3.6.2",
     "rimraf": "^6.1.0",
     "typescript": "catalog:"

packages/@sanity/pkg-utils/src/node/build.ts

@@ -74,6 +74,7 @@ export async function build(options: {
   const buildTasks = resolveBuildTasks(ctx)
 
   for (const task of buildTasks) {
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- TypeScript can't infer the correct handler type from discriminated union
     const handler = buildTaskHandlers[task.type] as TaskHandler<BuildTask>
     const taskName = handler.name(ctx, task)
 
@@ -90,7 +91,7 @@ export async function build(options: {
       spinner.error()
 
       if (err instanceof Error) {
-        const RE_CWD = new RegExp(`${cwd}`, 'g')
+        const RE_CWD = new RegExp(cwd, 'g')
 
         ctx.logger.error(err.message.replace(RE_CWD, '.'))
         ctx.logger.log()

packages/@sanity/pkg-utils/src/node/check.ts

@@ -74,14 +74,17 @@ export async function check(options: {
 
       const consoleSpy = createConsoleSpy()
 
+      const checks = []
       if (exportPaths.import.length) {
-        checkExports(exportPaths.import, {cwd, external, format: 'esm', logger})
+        checks.push(checkExports(exportPaths.import, {cwd, external, format: 'esm', logger}))
       }
 
       if (exportPaths.require.length) {
-        checkExports(exportPaths.require, {cwd, external, format: 'cjs', logger})
+        checks.push(checkExports(exportPaths.require, {cwd, external, format: 'cjs', logger}))
       }
 
+      await Promise.all(checks)
+
       consoleSpy.restore()
     }
 
@@ -94,7 +97,7 @@ export async function check(options: {
     spinner.error()
 
     if (err instanceof Error) {
-      const RE_CWD = new RegExp(`${cwd}`, 'g')
+      const RE_CWD = new RegExp(cwd, 'g')
 
       logger.error((err.stack || err.message).replace(RE_CWD, '.'))
       logger.log()
@@ -166,7 +169,7 @@ async function checkExports(
 
       logger.log()
     } else {
-      logger.error(`${err}`)
+      logger.error(String(err))
 
       logger.log()
     }

packages/@sanity/pkg-utils/src/node/core/config/loadConfig.ts

@@ -15,7 +15,7 @@ export async function loadConfig(options: {cwd: string}): Promise<PkgConfigOptio
 
   const root = path.dirname(pkgPath)
 
-  const configFile = await findConfigFile(root)
+  const configFile = findConfigFile(root)
 
   if (!configFile) {
     return undefined

packages/@sanity/pkg-utils/src/node/core/config/resolveConfigProperty.ts

@@ -1,14 +1,20 @@
 import type {PkgConfigProperty, PkgConfigPropertyResolver} from './types.ts'
 
+function isPkgConfigPropertyResolver<T>(
+  prop: PkgConfigProperty<T>,
+): prop is PkgConfigPropertyResolver<T> {
+  return typeof prop === 'function'
+}
+
 /** @internal */
 export function resolveConfigProperty<T>(
   prop: PkgConfigProperty<T> | undefined,
   initialValue: T,
 ): T {
   if (!prop) return initialValue
 
-  if (typeof prop === 'function') {
-    return (prop as PkgConfigPropertyResolver<T>)(initialValue)
+  if (isPkgConfigPropertyResolver(prop)) {
+    return prop(initialValue)
   }
 
   return prop

packages/@sanity/pkg-utils/src/node/core/pkg/parseExports.ts

@@ -9,6 +9,11 @@ import {pkgExtMap} from './pkgExt.ts'
 import type {PackageJSON} from './types.ts'
 import {validateExports} from './validateExports.ts'
 
+// Type guard to filter out falsy values
+function isTruthy<T>(value: T | false | null | undefined | 0 | ''): value is T {
+  return Boolean(value)
+}
+
 /** @alpha */
 export function parseExports(options: {
   cwd: string
@@ -89,7 +94,7 @@ export function parseExports(options: {
       }
 
       errors.push(
-        ...([
+        ...[
           'package.json: `exports` are missing, it should be:',
           `"exports": {`,
           `  ".": {`,
@@ -102,7 +107,7 @@ export function parseExports(options: {
           `  },`,
           `  "./package.json": "./package.json"`,
           `}`,
-        ].filter(Boolean) as string[]),
+        ].filter(isTruthy),
       )
     }
   }

packages/@sanity/pkg-utils/src/node/core/pkg/validatePkg.ts

@@ -18,8 +18,10 @@ const pkgSchema = z.object({
   exports: z.optional(
     z.record(
       z.union([
-        z.custom<`./${string}.json`>((val) => /^\.\/.*\.json$/.test(val as string)),
-        z.custom<`./${string}.css`>((val) => /^\.\/.*\.css$/.test(val as string)),
+        z.custom<`./${string}.json`>(
+          (val) => typeof val === 'string' && /^\.\/.*\.json$/.test(val),
+        ),
+        z.custom<`./${string}.css`>((val) => typeof val === 'string' && /^\.\/.*\.css$/.test(val)),
         z.object({
           types: z.optional(z.string()),
           source: z.optional(z.string()),
@@ -65,6 +67,7 @@ for (const key of pkgSchema.keyof()._def.values) {
 export function validatePkg(input: unknown): PackageJSON {
   const pkg = pkgSchema.parse(input)
 
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- Need to check raw input for typos
   const invalidKey = Object.keys(input as PackageJSON).find((key) => {
     const needle = key.toUpperCase()
 

packages/@sanity/pkg-utils/src/node/core/ts/loadTSConfig.ts

@@ -7,13 +7,13 @@ export async function loadTSConfig(options: {
 }): Promise<ts.ParsedCommandLine | undefined> {
   const {cwd, tsconfigPath} = options
 
-  const configPath = ts.findConfigFile(cwd, ts.sys.fileExists, tsconfigPath)
+  const configPath = ts.findConfigFile(cwd, (fileName) => ts.sys.fileExists(fileName), tsconfigPath)
 
   if (!configPath) {
     return undefined
   }
 
-  const configFile = ts.readConfigFile(configPath, ts.sys.readFile)
+  const configFile = ts.readConfigFile(configPath, (path) => ts.sys.readFile(path))
 
   return ts.parseJsonConfigFileContent(configFile.config, ts.sys, cwd)
 }