build: refactor linting

.lintstagedrc.json

@@ -1,6 +1,6 @@
 {
 	"$schema": "https://json.schemastore.org/lintstagedrc.schema.json",
 	"*": "prettier --ignore-unknown --write",
-	"{src/**,__tests__/**}.{mjs,js,cjs,ts,tsx}": "eslint --ext .mjs,.js,.cjs,.ts,.tsx --fix",
+	"{src/**,__tests__/**}.{mjs,js,cjs,ts,tsx}": "eslint --fix",
 	"src/**.ts": "vitest related --run --config ../../vitest.config.ts"
 }

.vscode/settings.json

@@ -1,6 +1,6 @@
 {
-	"eslint.workingDirectories": [{ "pattern": "./apps/*" }, { "pattern": "./packages/*" }],
 	"eslint.validate": ["javascript", "javascriptreact", "typescript", "typescriptreact"],
+	"eslint.experimental.useFlatConfig": true,
 	"editor.defaultFormatter": "esbenp.prettier-vscode",
 	"editor.formatOnSave": true,
 	"editor.codeActionsOnSave": {

apps/guide/package.json

@@ -17,8 +17,8 @@
 		"dev:next": "next dev",
 		"dev:css": "yarn generate:css --watch",
 		"generate:css": "unocss 'src/**/*.tsx' 'contentlayer.config.ts' '../../packages/ui/src/lib/components/**/*.tsx' --out-file ./src/styles/unocss.css --config ../../unocss.config.ts",
-		"lint": "prettier --check . && cross-env TIMING=1 eslint src --ext .mjs,.js,.cjs,.ts,.tsx --format=pretty",
-		"format": "prettier --write . && cross-env TIMING=1 eslint src --ext .mjs,.js,.cjs,.ts,.tsx --fix --format=pretty",
+		"lint": "prettier --check . && cross-env TIMING=1 eslint --format=pretty src",
+		"format": "prettier --write . && cross-env TIMING=1 eslint --fix --format=pretty src",
 		"fmt": "yarn format"
 	},
 	"type": "commonjs",
@@ -74,14 +74,14 @@
 		"@types/react": "^18.2.20",
 		"@types/react-dom": "^18.2.7",
 		"@unocss/cli": "^0.55.2",
-		"@unocss/eslint-config": "^0.55.2",
+		"@unocss/eslint-plugin": "^0.55.2",
 		"@unocss/reset": "^0.55.2",
 		"@vitejs/plugin-react": "^4.0.4",
 		"@vitest/coverage-v8": "^0.34.2",
-		"concurrently": "^8.2.0",
+		"concurrently": "^8.2.1",
 		"cross-env": "^7.0.3",
 		"eslint": "^8.47.0",
-		"eslint-config-neon": "^0.1.47",
+		"eslint-config-neon": "^0.1.54",
 		"eslint-formatter-pretty": "^5.0.0",
 		"happy-dom": "^10.10.4",
 		"hast-util-to-string": "^2.0.0",

apps/website/package.json

@@ -19,8 +19,8 @@
 		"dev:next": "next dev",
 		"dev:css": "yarn generate:css --watch",
 		"generate:css": "unocss 'src/**/*.tsx' '../../packages/ui/src/lib/components/**/*.tsx' --out-file ./src/styles/unocss.css --config ../../unocss.config.ts",
-		"lint": "prettier --check . && cross-env TIMING=1 eslint src --ext .mjs,.js,.cjs,.ts,.tsx --format=pretty",
-		"format": "prettier --write . && cross-env TIMING=1 eslint src --ext .mjs,.js,.cjs,.ts,.tsx --fix --format=pretty",
+		"lint": "prettier --check . && cross-env TIMING=1 eslint --format=pretty src",
+		"format": "prettier --write . && cross-env TIMING=1 eslint --fix --format=pretty src",
 		"fmt": "yarn format"
 	},
 	"type": "module",
@@ -83,15 +83,15 @@
 		"@types/react": "^18.2.20",
 		"@types/react-dom": "^18.2.7",
 		"@unocss/cli": "^0.55.2",
-		"@unocss/eslint-config": "^0.55.2",
+		"@unocss/eslint-plugin": "^0.55.2",
 		"@unocss/reset": "^0.55.2",
 		"@vitejs/plugin-react": "^4.0.4",
 		"@vitest/coverage-v8": "^0.34.2",
-		"concurrently": "^8.2.0",
+		"concurrently": "^8.2.1",
 		"cpy-cli": "^5.0.0",
 		"cross-env": "^7.0.3",
 		"eslint": "^8.47.0",
-		"eslint-config-neon": "^0.1.47",
+		"eslint-config-neon": "^0.1.54",
 		"eslint-formatter-pretty": "^5.0.0",
 		"happy-dom": "^10.10.4",
 		"lighthouse": "^11.0.0",

apps/website/src/hooks/useSystemThemeFallback.ts

@@ -8,6 +8,5 @@ export function useSystemThemeFallback() {
 		if (theme === 'auto') {
 			localStorage.setItem('theme', 'system');
 		}
-		// eslint-disable-next-line react-hooks/exhaustive-deps
 	}, []);
 }

eslint.config.js

@@ -0,0 +1,97 @@
+import unocss from '@unocss/eslint-plugin';
+import common from 'eslint-config-neon/flat/common.js';
+import edge from 'eslint-config-neon/flat/edge.js';
+import next from 'eslint-config-neon/flat/next.js';
+import node from 'eslint-config-neon/flat/node.js';
+import prettier from 'eslint-config-neon/flat/prettier.js';
+import react from 'eslint-config-neon/flat/react.js';
+import typescript from 'eslint-config-neon/flat/typescript.js';
+import deepMerge from 'ts-deepmerge';
+
+const generalRuleset = Object.freeze(
+	deepMerge.withOptions({ mergeArrays: false }, ...common, ...node, ...typescript, {
+		files: [
+			'{apps,packages}/**/src/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'{apps,packages}/**/bin/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'{apps,packages}/**/__tests__/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+		],
+		languageOptions: {
+			parserOptions: {
+				project: ['./tsconfig.eslint.json', './apps/*/tsconfig.eslint.json', './packages/*/tsconfig.eslint.json'],
+			},
+		},
+		rules: {
+			'@typescript-eslint/consistent-type-definitions': [2, 'interface'],
+			'jsdoc/no-undefined-types': 1,
+		},
+	}),
+);
+
+const reactRuleset = Object.freeze(
+	deepMerge.withOptions({ mergeArrays: false }, generalRuleset, ...react, ...next, ...edge, {
+		files: [
+			'apps/**/src/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'apps/**/bin/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'apps/**/__tests__/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'packages/ui/src/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+		],
+		plugins: {
+			'@unocss': unocss,
+		},
+		rules: {
+			'@next/next/no-html-link-for-pages': 0,
+			'react/react-in-jsx-scope': 0,
+			'react/jsx-filename-extension': [1, { extensions: ['.tsx'] }],
+		},
+		settings: {
+			react: {
+				version: 'detect',
+			},
+		},
+	}),
+);
+
+const prettierRuleset = Object.freeze(
+	deepMerge.withOptions({ mergeArrays: false }, ...prettier, {
+		files: [
+			'{apps,packages}/**/src/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'{apps,packages}/**/bin/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'{apps,packages}/**/__tests__/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+		],
+	}),
+);
+
+/** @type {import('eslint').Linter.FlatConfig[]} */
+export default [
+	{
+		ignores: ['**/.next', '**/coverage', '**/dist', '**/node_modules', '**/.contentlayer', '**/template'],
+	},
+	generalRuleset,
+	reactRuleset,
+	{
+		files: [
+			'packages/rest/src/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'packages/rest/__tests__/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+		],
+		rules: {
+			'n/prefer-global/url': 0,
+			'n/prefer-global/url-search-params': 0,
+			'n/prefer-global/buffer': 0,
+			'n/prefer-global/process': 0,
+			'no-restricted-globals': 0,
+			'unicorn/prefer-node-protocol': 0,
+		},
+	},
+	{
+		files: [
+			'packages/voice/src/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'packages/voice/__tests__/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+			'packages/voice/__mocks__/**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}',
+		],
+		rules: {
+			'import/extensions': 0,
+			'no-restricted-globals': 0,
+		},
+	},
+	prettierRuleset,
+];

package.json

@@ -5,6 +5,7 @@
 	"description": "A powerful library for interacting with the Discord API",
 	"private": true,
 	"scripts": {
+		"lint:fast": "eslint .",
 		"build": "turbo run build",
 		"build:affected": "turbo run build --filter='...[origin/main]'",
 		"build:apps": "turbo run build:local --filter='...{apps/*}'",
@@ -23,6 +24,7 @@
 		"update": "yarn upgrade-interactive",
 		"create-package": "node packages/scripts/src/packageScript.mjs"
 	},
+	"type": "module",
 	"contributors": [
 		"Crawl <icrawltogo@gmail.com>",
 		"Amish Shah <amishshah.2k@gmail.com>",
@@ -51,10 +53,14 @@
 		"@commitlint/config-angular": "^17.7.0",
 		"@favware/cliff-jumper": "^2.1.1",
 		"@favware/npm-deprecate": "^1.0.7",
+		"@unocss/eslint-plugin": "^0.55.2",
 		"conventional-changelog-cli": "^3.0.0",
+		"eslint": "^8.47.0",
+		"eslint-config-neon": "^0.1.54",
 		"husky": "^8.0.3",
 		"is-ci": "^3.0.1",
 		"lint-staged": "^14.0.1",
+		"ts-deepmerge": "^6.2.0",
 		"tsup": "^7.2.0",
 		"turbo": "^1.10.12",
 		"typescript": "^5.1.6",

packages/actions/package.json

@@ -7,8 +7,8 @@
 	"scripts": {
 		"test": "vitest run",
 		"build": "tsup",
-		"lint": "prettier --check . && cross-env TIMING=1 eslint src __tests__ --ext .mjs,.js,.ts --format=pretty",
-		"format": "prettier --write . && cross-env TIMING=1 eslint src __tests__ --ext .mjs,.js,.ts --fix --format=pretty",
+		"lint": "prettier --check . && cross-env TIMING=1 eslint --format=pretty src __tests__",
+		"format": "prettier --write . && cross-env TIMING=1 eslint --fix --format=pretty src __tests__",
 		"fmt": "yarn format"
 	},
 	"type": "module",
@@ -51,7 +51,7 @@
 		"@vitest/coverage-v8": "^0.34.2",
 		"cross-env": "^7.0.3",
 		"eslint": "^8.47.0",
-		"eslint-config-neon": "^0.1.47",
+		"eslint-config-neon": "^0.1.54",
 		"eslint-formatter-pretty": "^5.0.0",
 		"prettier": "^3.0.2",
 		"tsup": "^7.2.0",

packages/actions/src/formatTag/formatTag.ts

@@ -1,5 +1,4 @@
 export function formatTag(tag: string) {
-	// eslint-disable-next-line unicorn/no-unsafe-regex
 	const parsed = /(?:^@.*\/(?<package>.*)@v?)?(?<semver>\d+.\d+.\d+)-?.*/.exec(tag);
 	const parsedPackage = /(?<package>.*)@v?-?.*/.exec(tag);
 

packages/api-extractor-utils/package.json

@@ -6,8 +6,8 @@
 	"private": true,
 	"scripts": {
 		"build": "tsup",
-		"lint": "prettier --check . && cross-env TIMING=1 eslint src --ext .mjs,.js,.ts --format=pretty",
-		"format": "prettier --write . && cross-env TIMING=1 eslint src --ext .mjs,.js,.ts --fix --format=pretty",
+		"lint": "prettier --check . && cross-env TIMING=1 eslint --format=pretty src",
+		"format": "prettier --write . && cross-env TIMING=1 eslint --fix --format=pretty src",
 		"fmt": "yarn format"
 	},
 	"exports": {
@@ -52,7 +52,7 @@
 		"@types/node": "16.18.41",
 		"cross-env": "^7.0.3",
 		"eslint": "^8.47.0",
-		"eslint-config-neon": "^0.1.47",
+		"eslint-config-neon": "^0.1.54",
 		"eslint-formatter-pretty": "^5.0.0",
 		"prettier": "^3.0.2",
 		"tsup": "^7.2.0",

packages/api-extractor-utils/src/parse.ts

@@ -54,7 +54,7 @@ export function generatePath(items: readonly ApiItem[], version: string) {
 		}
 	}
 
-	// eslint-disable-next-line prefer-named-capture-group, unicorn/no-unsafe-regex
+	// eslint-disable-next-line prefer-named-capture-group
 	return path.replace(/@discordjs\/(.*)\/(.*)?/, `$1/${version}/$2`);
 }
 

packages/brokers/__tests__/index.test.ts

@@ -15,7 +15,6 @@ const mockRedisClient = {
 } as unknown as Redis;
 
 test('pubsub with custom encoding', async () => {
-	// eslint-disable-next-line @typescript-eslint/no-unsafe-return
 	const encode = vi.fn((data) => data);
 
 	const broker = new PubSubRedisBroker({ redisClient: mockRedisClient, encode });

packages/brokers/package.json

@@ -7,8 +7,8 @@
 		"test": "vitest run",
 		"build": "tsup",
 		"build:docs": "tsc -p tsconfig.docs.json",
-		"lint": "prettier --check . && cross-env TIMING=1 eslint src __tests__ --ext .mjs,.js,.ts --format=pretty",
-		"format": "prettier --write . && cross-env TIMING=1 eslint src __tests__ --ext .mjs,.js,.ts --fix --format=pretty",
+		"lint": "prettier --check . && cross-env TIMING=1 eslint --format=pretty src __tests__",
+		"format": "prettier --write . && cross-env TIMING=1 eslint --fix --format=pretty src __tests__",
 		"fmt": "yarn format",
 		"docs": "yarn build:docs && api-extractor run --local && api-extractor run --local --config ./api-extractor-docs.json",
 		"prepack": "yarn lint && yarn test && yarn build",
@@ -77,7 +77,7 @@
 		"@vitest/coverage-v8": "^0.34.2",
 		"cross-env": "^7.0.3",
 		"eslint": "^8.47.0",
-		"eslint-config-neon": "^0.1.47",
+		"eslint-config-neon": "^0.1.54",
 		"eslint-formatter-pretty": "^5.0.0",
 		"prettier": "^3.0.2",
 		"tsup": "^7.2.0",

packages/brokers/src/brokers/redis/BaseRedis.ts

@@ -113,7 +113,7 @@ export abstract class BaseRedisBroker<TEvents extends Record<string, any>>
 		}
 
 		this.listening = true;
-		// eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+
 		while (true) {
 			try {
 				const data = await this.streamReadClient.xreadgroupBuffer(

packages/builders/__tests__/interactions/SlashCommands/Options.test.ts

@@ -102,7 +102,6 @@ describe('Application Command toJSON() results', () => {
 			min_value: -1,
 			autocomplete: true,
 			// TODO
-			// @ts-expect-error You *can* send an empty array with autocomplete: true, should correct that in types
 			choices: [],
 		});
 
@@ -147,7 +146,6 @@ describe('Application Command toJSON() results', () => {
 			min_value: -1.23,
 			autocomplete: true,
 			// TODO
-			// @ts-expect-error You *can* send an empty array with autocomplete: true, should correct that in types
 			choices: [],
 		});
 
@@ -190,7 +188,6 @@ describe('Application Command toJSON() results', () => {
 			required: true,
 			autocomplete: true,
 			// TODO
-			// @ts-expect-error you *can* send an empty array with autocomplete: true, should correct that in types
 			choices: [],
 		});