feat: use typescript-eslint@v6's reworked configs

.eslintignore

@@ -1,4 +1,5 @@
 **/*.d.ts
+packages/**/*.min.js
 packages/**/dist/**/*
 packages/**/fixtures/**/*
 packages/webapi/**/*

.eslintrc.cjs

@@ -1,23 +1,55 @@
 module.exports = {
+  extends: [
+    'plugin:@typescript-eslint/recommended-type-checked',
+    'plugin:@typescript-eslint/stylistic-type-checked',
+    'prettier',
+  ],
   parser: '@typescript-eslint/parser',
-  extends: ['plugin:@typescript-eslint/recommended', 'prettier'],
+  parserOptions: {
+    project: ['./packages/*/tsconfig.json', './tsconfig.eslint.json'],
+    tsconfigRootDir: __dirname,
+  },
   plugins: ['@typescript-eslint', 'prettier', 'no-only-tests'],
   rules: {
+    // These off/configured-differently-by-default rules fit well for us
+    '@typescript-eslint/no-unused-vars': ['error', { ignoreRestSiblings: true }],
+    'no-only-tests/no-only-tests': 'error',
+    '@typescript-eslint/no-shadow': ['error'],
+    'no-console': 'warn',
+
+    /*
+    interface: 404 results in 198 files
+    type: 129 results in 73 files
+    */
+    '@typescript-eslint/consistent-type-definitions': 'off',
+
+    // Todo: do we want these?
+    '@typescript-eslint/await-thenable': 'off',
     '@typescript-eslint/ban-ts-comment': 'off',
-    '@typescript-eslint/camelcase': 'off',
-    '@typescript-eslint/explicit-module-boundary-types': 'off',
+    '@typescript-eslint/class-literal-property-style': 'off',
+    '@typescript-eslint/consistent-indexed-object-style': 'off',
+    '@typescript-eslint/dot-notation': 'off',
+    '@typescript-eslint/no-base-to-string': 'off',
+    '@typescript-eslint/no-confusing-void-expression': 'off',
     '@typescript-eslint/no-empty-function': 'off',
-    '@typescript-eslint/no-explicit-any': 'off',
-    '@typescript-eslint/no-non-null-assertion': 'off',
-    '@typescript-eslint/no-unused-vars': 'off',
-    '@typescript-eslint/no-use-before-define': 'off',
-    '@typescript-eslint/no-var-requires': 'off',
+    '@typescript-eslint/no-floating-promises': 'off',
+    '@typescript-eslint/no-misused-promises': 'off',
     '@typescript-eslint/no-this-alias': 'off',
-    'no-console': 'warn',
+    '@typescript-eslint/no-unsafe-argument': 'off',
+    '@typescript-eslint/no-unsafe-assignment': 'off',
+    '@typescript-eslint/no-unsafe-call': 'off',
+    '@typescript-eslint/no-unsafe-member-access': 'off',
+    '@typescript-eslint/no-unsafe-return': 'off',
+    '@typescript-eslint/prefer-nullish-coalescing': 'off',
+    '@typescript-eslint/prefer-string-starts-ends-with': 'off',
+    '@typescript-eslint/require-await': 'off',
+    '@typescript-eslint/restrict-plus-operands': 'off',
+    '@typescript-eslint/restrict-template-expressions': 'off',
+    '@typescript-eslint/sort-type-constituents': 'off',
+    '@typescript-eslint/unbound-method': 'off',
+
+    // These rules enabled by the preset configs don't work well for us
     'prefer-const': 'off',
-    'no-shadow': 'off',
-    '@typescript-eslint/no-shadow': ['error'],
-    'no-only-tests/no-only-tests': 'error',
   },
   overrides: [
     {
@@ -41,6 +73,7 @@ module.exports = {
     {
       files: ['benchmark/**/*.js'],
       rules: {
+        '@typescript-eslint/no-unused-vars': 'off',
         'no-console': 'off',
       },
     },

.github/workflows/ci.yml

@@ -49,6 +49,9 @@ jobs:
       - name: Install dependencies
         run: pnpm install
 
+      - name: Build (ignoring failures)
+        run: pnpm run build || true
+
       # Lint autofix cannot run on forks, so just skip those! See https://github.com/wearerequired/lint-action/issues/13
       - name: Lint (External)
         if: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.repo.owner.login != github.repository_owner }}

package.json

@@ -28,7 +28,7 @@
     "test:e2e": "cd packages/astro && pnpm playwright install && pnpm run test:e2e",
     "test:e2e:match": "cd packages/astro && pnpm playwright install && pnpm run test:e2e:match",
     "benchmark": "astro-benchmark",
-    "lint": "eslint --cache .",
+    "lint": "eslint . --report-unused-disable-directives",
     "version": "changeset version && node ./scripts/deps/update-example-versions.js && pnpm install --no-frozen-lockfile && pnpm run format",
     "preinstall": "npx only-allow pnpm"
   },
@@ -78,8 +78,8 @@
     "@changesets/changelog-github": "^0.4.8",
     "@changesets/cli": "^2.26.1",
     "@types/node": "^18.7.21",
-    "@typescript-eslint/eslint-plugin": "^5.58.0",
-    "@typescript-eslint/parser": "^5.58.0",
+    "@typescript-eslint/eslint-plugin": "6.0.0-alpha.158",
+    "@typescript-eslint/parser": "6.0.0-alpha.158",
     "esbuild": "^0.17.12",
     "eslint": "^8.38.0",
     "eslint-config-prettier": "^8.8.0",

packages/astro/astro.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-/* eslint-disable no-console */
 'use strict';
 
 // ISOMORPHIC FILE: NO TOP-LEVEL IMPORT/REQUIRE() ALLOWED

packages/astro/e2e/css-sourcemaps.test.js

@@ -1,5 +1,5 @@
 import { expect } from '@playwright/test';
-import { getColor, isWindows, testFactory } from './test-utils.js';
+import { isWindows, testFactory } from './test-utils.js';
 
 const test = testFactory({
 	root: './fixtures/css/',

packages/astro/e2e/hydration-race.test.js

@@ -19,7 +19,7 @@ test.describe('Hydration race', () => {
 	test('Islands inside of slots hydrate', async ({ page, astro }) => {
 		await page.goto(astro.resolveUrl('/slot'));
 
-		const html = await page.content();
+		await page.content();
 
 		const one = page.locator('#one');
 		await expect(one, 'updated text').toHaveText('Hello One in the client');

packages/astro/e2e/test-utils.js

@@ -18,8 +18,7 @@ for (let i = 0; i < testFiles.length; i++) {
 }
 
 export function loadFixture(inlineConfig) {
-	if (!inlineConfig || !inlineConfig.root)
-		throw new Error("Must provide { root: './fixtures/...' }");
+	if (!inlineConfig?.root) throw new Error("Must provide { root: './fixtures/...' }");
 
 	// resolve the relative root (i.e. "./fixtures/tailwindcss") to a full filepath
 	// without this, the main `loadFixture` helper will resolve relative to `packages/astro/test`

packages/astro/src/@types/astro.ts

@@ -1087,9 +1087,13 @@ export interface AstroUserConfig {
 	 * }
 	 * ```
 	 */
-	integrations?: Array<
-		AstroIntegration | (AstroIntegration | false | undefined | null)[] | false | undefined | null
-	>;
+	integrations?: (
+		| AstroIntegration
+		| (AstroIntegration | false | undefined | null)[]
+		| false
+		| undefined
+		| null
+	)[];
 
 	/**
 	 * @docs
@@ -1454,7 +1458,7 @@ export type GetStaticPaths = (
  * ```
  */
 export type InferGetStaticParamsType<T> = T extends () => infer R | Promise<infer R>
-	? R extends Array<infer U>
+	? R extends (infer U)[]
 		? U extends { params: infer P }
 			? P
 			: never
@@ -1485,7 +1489,7 @@ export type InferGetStaticParamsType<T> = T extends () => infer R | Promise<infe
  * ```
  */
 export type InferGetStaticPropsType<T> = T extends () => infer R | Promise<infer R>
-	? R extends Array<infer U>
+	? R extends (infer U)[]
 		? U extends { props: infer P }
 			? P
 			: never
@@ -1916,7 +1920,7 @@ export interface SSRResult {
 	links: Set<SSRElement>;
 	componentMetadata: Map<string, SSRComponentMetadata>;
 	propagators: Map<AstroComponentFactory, AstroComponentInstance>;
-	extraHead: Array<string>;
+	extraHead: string[];
 	cookies: AstroCookies | undefined;
 	createAstro(
 		Astro: AstroGlobalPartial,

packages/astro/src/assets/image-endpoint.ts

@@ -35,7 +35,7 @@ export const get: APIRoute = async ({ request }) => {
 		const url = new URL(request.url);
 		const transform = await imageService.parseURL(url, imageServiceConfig);
 
-		if (!transform || !transform.src) {
+		if (!transform?.src) {
 			throw new Error('Incorrect transform returned by `parseURL`');
 		}
 

packages/astro/src/assets/services/squoosh.ts

@@ -36,7 +36,7 @@ const service: LocalImageService = {
 	async transform(inputBuffer, transformOptions) {
 		const transform: BaseServiceTransform = transformOptions as BaseServiceTransform;
 
-		let format = transform.format!;
+		let format = transform.format;
 
 		const operations: Operation[] = [];
 

packages/astro/src/assets/services/vendor/squoosh/codecs.ts

@@ -291,7 +291,6 @@ export const codecs = {
   avif: {
     name: 'AVIF',
     extension: 'avif',
-    // eslint-disable-next-line no-control-regex
     detectors: [/^\x00\x00\x00 ftypavif\x00\x00\x00\x00/],
     dec: () =>
       instantiateEmscriptenWasm(avifDec as DecodeModuleFactory, avifDecWasm),
@@ -322,7 +321,6 @@ export const codecs = {
   oxipng: {
     name: 'OxiPNG',
     extension: 'png',
-    // eslint-disable-next-line no-control-regex
     detectors: [/^\x89PNG\x0D\x0A\x1A\x0A/],
     dec: async () => {
       await pngEncDecInit()

packages/astro/src/assets/services/vendor/squoosh/image.ts

@@ -29,14 +29,14 @@ export async function processBuffer(
 
 	switch (encoding) {
 		case 'avif':
-			return await impl.encodeAvif(imageData, { quality }) as Uint8Array;
+			return await impl.encodeAvif(imageData, { quality });
 		case 'jpeg':
 		case 'jpg':
-			return await impl.encodeJpeg(imageData, { quality }) as Uint8Array;
+			return await impl.encodeJpeg(imageData, { quality });
 		case 'png':
-			return await impl.encodePng(imageData) as Uint8Array;
+			return await impl.encodePng(imageData);
 		case 'webp':
-			return await impl.encodeWebp(imageData, { quality }) as Uint8Array;
+			return await impl.encodeWebp(imageData, { quality });
 		default:
 			throw Error(`Unsupported encoding format`)
 	}

packages/astro/src/assets/vendor/image-size/types/jp2.ts

@@ -49,7 +49,6 @@ export const JP2: IImage = {
     switch (nextBoxType) {
     case BoxTypes.rreq:
       // WHAT ARE THESE 4 BYTES?????
-      // eslint-disable-next-line no-case-declarations
       const MAGIC = 4
       offset = offset + 4 + MAGIC + calculateRREQLength(buffer.slice(offset + 4))
       return parseIHDR(buffer.slice(offset + 8, offset + 24))

packages/astro/src/assets/vendor/image-size/types/pnm.ts

@@ -19,7 +19,7 @@ const handlers: { [type: string]: Handler} = {
     let dimensions: string[] = []
 
     while (lines.length > 0) {
-      const line = lines.shift() as string
+      const line = lines.shift()!
       if (line[0] === '#') {
         continue
       }
@@ -39,7 +39,7 @@ const handlers: { [type: string]: Handler} = {
   pam: (lines) => {
     const size: { [key: string]: number } = {}
     while (lines.length > 0) {
-      const line = lines.shift() as string
+      const line = lines.shift()!
       if (line.length > 16 || line.charCodeAt(0) > 128) {
         continue
       }

packages/astro/src/assets/vendor/image-size/types/svg.ts

@@ -41,8 +41,8 @@ function parseLength(len: string) {
 function parseViewbox(viewbox: string): IAttributes {
   const bounds = viewbox.split(' ')
   return {
-    height: parseLength(bounds[3]) as number,
-    width: parseLength(bounds[2]) as number
+    height: parseLength(bounds[3])!,
+    width: parseLength(bounds[2])!
   }
 }
 
@@ -51,21 +51,21 @@ function parseAttributes(root: string): IAttributes {
   const height = root.match(extractorRegExps.height)
   const viewbox = root.match(extractorRegExps.viewbox)
   return {
-    height: height && parseLength(height[2]) as number,
-    viewbox: viewbox && parseViewbox(viewbox[2]) as IAttributes,
-    width: width && parseLength(width[2]) as number,
+    height: height && parseLength(height[2])!,
+    viewbox: viewbox && parseViewbox(viewbox[2])!,
+    width: width && parseLength(width[2])!,
   }
 }
 
 function calculateByDimensions(attrs: IAttributes): ISize {
   return {
-    height: attrs.height as number,
-    width: attrs.width as number,
+    height: attrs.height!,
+    width: attrs.width!,
   }
 }
 
 function calculateByViewbox(attrs: IAttributes, viewbox: IAttributes): ISize {
-  const ratio = (viewbox.width as number) / (viewbox.height as number)
+  const ratio = (viewbox.width!) / (viewbox.height!)
   if (attrs.width) {
     return {
       height: Math.floor(attrs.width / ratio),
@@ -79,8 +79,8 @@ function calculateByViewbox(attrs: IAttributes, viewbox: IAttributes): ISize {
     }
   }
   return {
-    height: viewbox.height as number,
-    width: viewbox.width as number,
+    height: viewbox.height!,
+    width: viewbox.width!,
   }
 }
 

packages/astro/src/assets/vendor/image-size/types/tiff.ts

@@ -44,7 +44,7 @@ function extractTags(buffer: Buffer, isBigEndian: boolean) {
   const tags: {[key: number]: number} = {}
 
   let temp: Buffer | undefined = buffer
-  while (temp && temp.length) {
+  while (temp?.length) {
     const code = readUInt(temp, 16, 0, isBigEndian)
     const type = readUInt(temp, 16, 2, isBigEndian)
     const length = readUInt(temp, 32, 4, isBigEndian)

packages/astro/src/cli/check/index.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import {
 	AstroCheck,
 	DiagnosticSeverity,

packages/astro/src/cli/index.ts

@@ -148,7 +148,7 @@ async function runCommand(cmd: string, flags: yargs.Arguments) {
 
 			telemetry.record(event.eventCliSession(cmd));
 			const packages = flags._.slice(3) as string[];
-			return await add(packages, { cwd: root, flags, logging, telemetry });
+			return await add(packages, { cwd: root, flags, logging });
 		}
 		case 'docs': {
 			telemetry.record(event.eventCliSession(cmd));

packages/astro/src/content/error-map.ts

@@ -14,7 +14,7 @@ export const errorMap: ZodErrorMap = (baseError, ctx) => {
 		// raise a single error when `key` does not match:
 		// > Did not match union.
 		// > key: Expected `'tutorial' | 'blog'`, received 'foo'
-		let typeOrLiteralErrByPath: Map<string, TypeOrLiteralErrByPathEntry> = new Map();
+		let typeOrLiteralErrByPath = new Map<string, TypeOrLiteralErrByPathEntry>();
 		for (const unionError of baseError.unionErrors.map((e) => e.errors).flat()) {
 			if (unionError.code === 'invalid_type' || unionError.code === 'invalid_literal') {
 				const flattenedErrorPath = flattenErrorPath(unionError.path);

packages/astro/src/content/utils.ts

@@ -177,7 +177,7 @@ export function getDataEntryExts(settings: Pick<AstroSettings, 'dataEntryTypes'>
 export function getEntryConfigByExtMap<TEntryType extends ContentEntryType | DataEntryType>(
 	entryTypes: TEntryType[]
 ): Map<string, TEntryType> {
-	const map: Map<string, TEntryType> = new Map();
+	const map = new Map<string, TEntryType>();
 	for (const entryType of entryTypes) {
 		for (const ext of entryType.extensions) {
 			map.set(ext, entryType);
@@ -310,7 +310,7 @@ function getYAMLErrorLine(rawData: string | undefined, objectKey: string) {
 	return numNewlinesBeforeKey;
 }
 
-export function parseFrontmatter(fileContents: string, filePath: string) {
+export function parseFrontmatter(fileContents: string) {
 	try {
 		// `matter` is empty string on cache results
 		// clear cache to prevent this

packages/astro/src/core/add/index.ts

@@ -1,4 +1,3 @@
-import type { AstroTelemetry } from '@astrojs/telemetry';
 import boxen from 'boxen';
 import { diffWords } from 'diff';
 import { execa } from 'execa';
@@ -30,7 +29,6 @@ import { wrapDefaultExport } from './wrapper.js';
 export interface AddOptions {
 	logging: LogOptions;
 	flags: yargs.Arguments;
-	telemetry: AstroTelemetry;
 	cwd?: string;
 }
 
@@ -83,7 +81,7 @@ async function getRegistry(): Promise<string> {
 	return stdout || 'https://registry.npmjs.org';
 }
 
-export default async function add(names: string[], { cwd, flags, logging, telemetry }: AddOptions) {
+export default async function add(names: string[], { cwd, flags, logging }: AddOptions) {
 	applyPolyfill();
 	if (flags.help || names.length === 0) {
 		printHelp({

packages/astro/src/core/app/index.ts

@@ -194,7 +194,7 @@ export class App {
 	): Promise<Response> {
 		const url = new URL(request.url);
 		const pathname = prependForwardSlash(this.removeBase(url.pathname));
-		const info = this.#routeDataToRouteInfo.get(routeData!)!;
+		const info = this.#routeDataToRouteInfo.get(routeData)!;
 		// may be used in the future for handling rel=modulepreload, rel=icon, rel=manifest etc.
 		const links = new Set<never>();
 		const styles = createStylesheetElementSet(info.styles);