zeno-config 1.0.3 → 2.1.0

package/README.md

@@ -22,17 +22,14 @@ Install the package and its peer dependencies:
 
 ```bash
 # Using pnpm (recommended)
-
-# Base dependencies
-pnpm add -D zeno-config eslint prettier @eslint/js @stylistic/eslint-plugin eslint-config-prettier eslint-plugin-import-x eslint-plugin-n eslint-plugin-prettier eslint-plugin-unicorn
-
-# For React projects, also install:
-pnpm add -D eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-jsx-a11y eslint-plugin-react-you-might-not-need-an-effect
+pnpm add -D zeno-config eslint prettier
 
 # For TypeScript projects, also install:
-pnpm add -D typescript typescript-eslint
+pnpm add -D typescript
 ```
 
+All ESLint plugins are bundled with zeno-config, so you don't need to install them separately.
+
 ## Usage
 
 ### ESLint Configuration
@@ -44,23 +41,12 @@ Create an `eslint.config.js` file in your project root:
 ```javascript
 import { defineZenoConfig } from 'zeno-config/eslint';
 
-// defineZenoConfig returns a Promise - ESLint 9+ flat config supports this natively
 export default defineZenoConfig({
 	ts: true, // Enable TypeScript support (default: false)
 	react: true, // Enable React support (default: false)
 });
 ```
 
-#### Required: Empty `.eslintrc` File
-
-You must create an empty `.eslintrc` file in your project root for the `import-x/no-unused-modules` rule to work correctly with flat config:
-
-```bash
-touch .eslintrc
-```
-
-This is a workaround for a [known limitation](https://github.com/import-js/eslint-plugin-import/issues/3079) where certain internal ESLint APIs are excluded when using flat config. The file can be empty or contain only `ignorePatterns` if you need to ignore specific files.
-
 #### `engines` field in `package.json`
 
 Add an `engines` field to your `package.json` with your supported Node.js versions. Some rules for Node.js use this:
@@ -83,6 +69,9 @@ export default defineZenoConfig(
 		react: true,
 		ts: true,
 
+		// Additional directories to ignore (added to defaults: node_modules, dist, build, coverage)
+		ignoreDirs: ['out', '.next'],
+
 		// If using .js extensions for React files, specify React directories
 		reactDirs: ['src/client', 'src/components'],
 
@@ -92,6 +81,12 @@ export default defineZenoConfig(
 		// Patterns to ignore for import/no-unresolved rule
 		ignoreExports: ['^@/'],
 
+		// Additional file patterns to allow dev dependencies in (for no-extraneous-dependencies rule)
+		additionalDevDependencies: [
+			'**/scripts/**/*.js',
+			'**/playwright.config.ts',
+		],
+
 		// Extensions to ignore for import/extensions rule
 		extensionsIgnorePattern: {
 			svg: 'always',
@@ -199,15 +194,18 @@ import {
 
 ### `defineZenoConfig(options, additionalESLintConfig)`
 
-| Option                    | Type       | Default     | Description                                                                         |
-| ------------------------- | ---------- | ----------- | ----------------------------------------------------------------------------------- |
-| `react`                   | `boolean`  | `false`     | Enable React-specific rules                                                         |
-| `ts`                      | `boolean`  | `false`     | Enable TypeScript-specific rules                                                    |
-| `reactDirs`               | `string[]` | `[]`        | Directories containing React files (for projects using .js for both React and Node) |
-| `nodeIgnoreDirs`          | `string[]` | `[]`        | Directories to ignore for Node-specific rules (defaults to `reactDirs` if not set)  |
-| `ignoreExports`           | `string[]` | `[]`        | Export patterns to ignore for import/no-unresolved rule                             |
-| `extensionsIgnorePattern` | `object`   | `{}`        | Extension patterns to ignore for import/extensions rule                             |
-| `webpackConfig`           | `string`   | `undefined` | Path to webpack config for import resolver                                          |
+| Option                      | Type       | Default     | Description                                                                                   |
+| --------------------------- | ---------- | ----------- | --------------------------------------------------------------------------------------------- |
+| `react`                     | `boolean`  | `false`     | Enable React-specific rules                                                                   |
+| `ts`                        | `boolean`  | `false`     | Enable TypeScript-specific rules                                                              |
+| `performanceMode`           | `boolean`  | `false`     | Disables expensive rules for better performance                                               |
+| `ignoreDirs`                | `string[]` | `[]`        | Additional directories to ignore (added to defaults: node_modules, dist, build, coverage)     |
+| `reactDirs`                 | `string[]` | `[]`        | Directories containing React files (for projects using .js for both React and Node)           |
+| `nodeIgnoreDirs`            | `string[]` | `[]`        | Directories to ignore for Node-specific rules (defaults to `reactDirs` if not set)            |
+| `ignoreExports`             | `string[]` | `[]`        | Export patterns to ignore for import/no-unresolved rule                                       |
+| `additionalDevDependencies` | `string[]` | `[]`        | Additional file patterns to allow dev dependencies in (for import/no-extraneous-dependencies) |
+| `extensionsIgnorePattern`   | `object`   | `{}`        | Extension patterns to ignore for import/extensions rule                                       |
+| `webpackConfig`             | `string`   | `undefined` | Path to webpack config for import resolver                                                    |
 
 ## Advanced Usage
 
@@ -220,14 +218,13 @@ import zenoInternals from 'zeno-config/eslint';
 
 const { configs, rules, extensions } = zenoInternals;
 
-// Note: getReact() and getTypescript() are async (return Promises)
-export default (async () => [
+export default [
 	...configs.getBase(),
 	...configs.getNode(),
-	...(await configs.getReact()),
-	...(await configs.getTypescript()),
+	...configs.getReact(),
+	...configs.getTypescript(),
 	// Your custom configs
-])();
+];
 ```
 
 ### Accessing Individual Rules

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zeno-config",
-  "version": "1.0.3",
+  "version": "2.1.0",
   "description": "Preconfigured and opinionated ESLint, Prettier, and TypeScript setup",
   "author": "Rick Brenn <brenn.rick@gmail.com>",
   "license": "MIT",
@@ -46,63 +46,38 @@
     "src/**"
   ],
   "peerDependencies": {
-    "@eslint/js": "^9.39.2",
-    "@stylistic/eslint-plugin": "^5.6.1",
     "eslint": "^9.39.2",
-    "eslint-config-prettier": "^10.1.8",
-    "eslint-plugin-import-x": "^4.16.1",
-    "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-n": "^17.23.1",
-    "eslint-plugin-prettier": "^5.5.4",
-    "eslint-plugin-react": "^7.37.5",
-    "eslint-plugin-react-hooks": "^7.0.1",
-    "eslint-plugin-react-you-might-not-need-an-effect": "^0.8.5",
-    "eslint-plugin-unicorn": "^62.0.0",
-    "prettier": "^3.7.4",
-    "typescript": "^5.9.3",
-    "typescript-eslint": "^8.50.1"
+    "prettier": "^3.8.0",
+    "typescript": "^5.9.3"
   },
   "peerDependenciesMeta": {
-    "eslint-plugin-jsx-a11y": {
-      "optional": true
-    },
-    "eslint-plugin-react": {
-      "optional": true
-    },
-    "eslint-plugin-react-hooks": {
-      "optional": true
-    },
-    "eslint-plugin-react-you-might-not-need-an-effect": {
-      "optional": true
-    },
     "typescript": {
       "optional": true
-    },
-    "typescript-eslint": {
-      "optional": true
     }
   },
   "devDependencies": {
-    "@eslint/js": "9.39.2",
-    "@stylistic/eslint-plugin": "5.6.1",
     "eslint": "9.39.2",
+    "prettier": "3.8.0",
+    "typescript": "5.9.3"
+  },
+  "dependencies": {
+    "@eslint/js": "9.39.2",
+    "@stylistic/eslint-plugin": "5.7.0",
     "eslint-config-prettier": "10.1.8",
+    "eslint-import-resolver-webpack": "^0.13.10",
     "eslint-plugin-import-x": "4.16.1",
     "eslint-plugin-jsx-a11y": "6.10.2",
-    "eslint-plugin-n": "17.23.1",
-    "eslint-plugin-prettier": "5.5.4",
+    "eslint-plugin-n": "17.23.2",
+    "eslint-plugin-prettier": "5.5.5",
     "eslint-plugin-react": "7.37.5",
     "eslint-plugin-react-hooks": "7.0.1",
     "eslint-plugin-react-you-might-not-need-an-effect": "0.8.5",
     "eslint-plugin-unicorn": "62.0.0",
-    "prettier": "3.7.4",
-    "typescript": "5.9.3",
-    "typescript-eslint": "8.50.1"
-  },
-  "dependencies": {
-    "globals": "16.5.0"
+    "globals": "17.0.0",
+    "typescript-eslint": "8.53.1"
   },
   "scripts": {
-    "inspect": "pnpm dlx eslint --inspect-config"
+    "inspect": "eslint --inspect-config",
+    "lint": "eslint \"**/*.{js,cjs,mjs}\""
   }
 }

package/src/eslint/index.d.ts

@@ -3,10 +3,14 @@ import { type Linter } from 'eslint';
 interface BaseConfigOptions {
 	/** Export patterns to ignore for import rules */
 	ignoreExports?: string[];
+	/** Additional file patterns to allow dev dependencies in (for no-extraneous-dependencies rule) */
+	additionalDevDependencies?: string[];
 	/** Extension patterns to ignore for import rules */
 	extensionsIgnorePattern?: Record<string, string>;
 	/** Path to webpack config for import resolver */
 	webpackConfig?: string;
+	/** Disables expensive rules for performance */
+	performanceMode?: boolean;
 }
 
 interface NodeConfigOptions {
@@ -21,17 +25,28 @@ interface ReactConfigOptions {
 	extensionsIgnorePattern?: Record<string, string>;
 }
 
+interface TypescriptConfigOptions {
+	/** Enable React-specific rules */
+	react?: boolean;
+}
+
 interface DefineZenoConfigOptions {
 	/** Enable React-specific rules */
 	react?: boolean;
 	/** Enable TypeScript-specific rules */
 	ts?: boolean;
+	/** Disables expensive rules for performance */
+	performanceMode?: boolean;
+	/** Additional directories to ignore (added to defaults: dist, build) */
+	ignoreDirs?: string[];
 	/** Directories containing React files (for projects using .js for both React and Node) */
 	reactDirs?: string[];
-	/** Directories to ignore for Node-specific rules */
+	/** Directories to ignore for Node-specific rules only */
 	nodeIgnoreDirs?: string[];
 	/** Export patterns to ignore for import rules */
 	ignoreExports?: string[];
+	/** Additional file patterns to allow dev dependencies in (for no-extraneous-dependencies rule) */
+	additionalDevDependencies?: string[];
 	/** Extension patterns to ignore for import rules */
 	extensionsIgnorePattern?: Record<string, string>;
 	/** Path to webpack config for import resolver */
@@ -57,15 +72,16 @@ interface Extensions {
 
 interface Configs {
 	getBase: (options?: BaseConfigOptions) => Linter.Config[];
-	getReact: (options?: ReactConfigOptions) => Promise<Linter.Config[]>;
+	getReact: (options?: ReactConfigOptions) => Linter.Config[];
 	getNode: (options?: NodeConfigOptions) => Linter.Config[];
-	getTypescript: () => Promise<Linter.Config[]>;
+	getTypescript: (options?: TypescriptConfigOptions) => Linter.Config[];
 }
 
 interface Rules {
 	getBaseRules: () => Linter.RulesRecord;
 	getImportPluginRules: (options?: {
 		ignoreExports?: string[];
+		additionalDevDependencies?: string[];
 		extensionsIgnorePattern?: Record<string, string>;
 	}) => Linter.RulesRecord;
 	getStylisticPluginRules: () => Linter.RulesRecord;
@@ -89,12 +105,9 @@ declare const internals: Internals;
 /**
  * Defines a Zeno ESLint configuration.
  *
- * This function is async because React and TypeScript dependencies are loaded
- * dynamically (only when enabled) to avoid errors when optional deps aren't installed.
- *
  * @param arg1 - Options object or additional config array. If an array, treated as additional config.
  * @param arg2 - Additional ESLint config objects to merge (only used if arg1 is options object).
- * @returns Promise that resolves to ESLint flat config array.
+ * @returns ESLint flat config array.
  *
  * @example
  * // With options object (using default export)
@@ -111,7 +124,7 @@ declare const internals: Internals;
 declare function defineZenoConfig(
 	arg1?: DefineZenoConfigOptions | Linter.Config[],
 	arg2?: Linter.Config[]
-): Promise<Linter.Config[]>;
+): Linter.Config[];
 
 export {
 	type BaseConfigOptions,
@@ -122,6 +135,7 @@ export {
 	type NodeConfigOptions,
 	type ReactConfigOptions,
 	type Rules,
+	type TypescriptConfigOptions,
 	defineZenoConfig,
 };
 export default internals;

package/src/eslint/index.js

@@ -6,6 +6,11 @@ import prettierPlugin from 'eslint-plugin-prettier/recommended';
 import stylisticPlugin from '@stylistic/eslint-plugin';
 import nodePlugin from 'eslint-plugin-n';
 import unicornPlugin from 'eslint-plugin-unicorn';
+import reactPlugin from 'eslint-plugin-react';
+import reactHooksPlugin from 'eslint-plugin-react-hooks';
+import jsxA11yPlugin from 'eslint-plugin-jsx-a11y';
+import reactYouMightNotNeedAnEffectPlugin from 'eslint-plugin-react-you-might-not-need-an-effect';
+import typescriptEslint from 'typescript-eslint';
 import getBaseRules from './rules/baseRules.js';
 import getImportPluginRules from './rules/importPluginRules.js';
 import getReactPluginRules from './rules/reactPluginRules.js';
@@ -33,7 +38,7 @@ import {
 	typescriptExtensionsString,
 } from '../extensions.js';
 
-const ignoreDirs = [
+const defaultIgnoreDirs = [
 	'**/node_modules/*',
 	'**/dist/*',
 	'**/build/*',
@@ -45,8 +50,10 @@ const ignoreDirs = [
  *
  * @param {Object} [options={}] - Configuration options.
  * @param {string[]} [options.ignoreExports] - Export patterns to ignore for import rules.
+ * @param {string[]} [options.additionalDevDependencies] - Additional file patterns to allow dev dependencies in (for no-extraneous-dependencies rule).
  * @param {Object} [options.extensionsIgnorePattern] - Extension patterns to ignore for import rules.
  * @param {string} [options.webpackConfig] - Path to webpack config for import resolver.
+ * @param {boolean} [options.performanceMode=false] - Disables expensive rules for performance.
  * @returns {Array} ESLint flat config array.
  */
 const baseConfig = (options = {}) => {
@@ -54,7 +61,6 @@ const baseConfig = (options = {}) => {
 		{
 			name: 'zeno/base',
 			files: [`**/*{${allExtensionsString}}`],
-			ignores: [...ignoreDirs],
 			languageOptions: {
 				ecmaVersion: 'latest',
 				sourceType: 'module',
@@ -79,7 +85,10 @@ const baseConfig = (options = {}) => {
 				...getBaseRules(),
 				...getImportPluginRules({
 					ignoreExports: options.ignoreExports,
+					additionalDevDependencies:
+						options.additionalDevDependencies,
 					extensionsIgnorePattern: options.extensionsIgnorePattern,
+					performanceMode: options.performanceMode,
 				}),
 				...getStylisticPluginRules(),
 				...getUnicornPluginRules(),
@@ -100,7 +109,7 @@ const baseConfig = (options = {}) => {
  * @returns {Array} ESLint flat config array.
  */
 const nodeConfig = (options = {}) => {
-	let ignores = [...ignoreDirs];
+	let ignores = [...defaultIgnoreDirs];
 	if (options.ignoreDirs && options.ignoreDirs.length > 0) {
 		const optionsIgnoreDirs = options.ignoreDirs.map((dir) => {
 			return `${dir}/**/*{${nodeExtensionsString}}`;
@@ -133,22 +142,9 @@ const nodeConfig = (options = {}) => {
  * @param {Object} [options={}] - Configuration options.
  * @param {string[]} [options.reactDirs] - Directories containing React files (for projects using .js for both React and Node).
  * @param {Object} [options.extensionsIgnorePattern] - Extension patterns to ignore for import rules.
- * @returns {Promise<Array>} ESLint flat config array.
+ * @returns {Array} ESLint flat config array.
  */
-const reactConfig = async (options = {}) => {
-	// Dynamically import optional React dependencies
-	const [
-		{ default: reactPlugin },
-		{ default: reactHooksPlugin },
-		{ default: jsxA11yPlugin },
-		{ default: reactYouMightNotNeedAnEffectPlugin },
-	] = await Promise.all([
-		import('eslint-plugin-react'),
-		import('eslint-plugin-react-hooks'),
-		import('eslint-plugin-jsx-a11y'),
-		import('eslint-plugin-react-you-might-not-need-an-effect'),
-	]);
-
+const reactConfig = (options = {}) => {
 	let files = [`**/*{${reactExtensionsString}}`];
 	let extensions = reactExtensions;
 
@@ -167,7 +163,6 @@ const reactConfig = async (options = {}) => {
 		{
 			name: 'zeno/react',
 			files,
-			ignores: [...ignoreDirs],
 			languageOptions: {
 				globals: {
 					...globals.browser,
@@ -215,13 +210,11 @@ const reactConfig = async (options = {}) => {
 
 /**
  * Creates the TypeScript-specific ESLint configuration.
- *
- * @returns {Promise<Array>} ESLint flat config array.
+ * @param {Object} [options={}] - Configuration options.
+ * @param {boolean} [options.react=false] - Enable React-specific rules.
+ * @returns {Array} ESLint flat config array.
  */
-const typescriptConfig = async () => {
-	// Dynamically import optional TypeScript dependencies
-	const { default: typescriptEslint } = await import('typescript-eslint');
-
+const typescriptConfig = (options = {}) => {
 	return [
 		{
 			name: 'zeno/typescript',
@@ -236,10 +229,36 @@ const typescriptConfig = async () => {
 				'@typescript-eslint': typescriptEslint.plugin,
 			},
 			rules: {
-				...getTypescriptPluginRules(),
+				...getTypescriptPluginRules({ react: options.react }),
+
+				// Rules handled by the TypeScript compiler
+				'constructor-super': 'off',
+				'getter-return': 'off',
+				'no-class-assign': 'off',
+				'no-const-assign': 'off',
+				'no-dupe-args': 'off',
+				'no-dupe-keys': 'off',
+				'no-func-assign': 'off',
+				'no-import-assign': 'off',
+				'no-new-native-nonconstructor': 'off',
+				'no-obj-calls': 'off',
+				'no-setter-return': 'off',
+				'no-this-before-super': 'off',
+				'no-undef': 'off',
+				'no-unreachable': 'off',
+				'no-unsafe-negation': 'off',
+				'valid-typeof': 'off',
+
+				'import-x/named': 'off',
 
-				// other rules the ts compiler handles
-				'react/prop-types': 'off',
+				...(options.react
+					? {
+							'react/default-props-match-prop-types': 'off',
+							'react/prop-types': 'off',
+							'react/forbid-foreign-prop-types': 'off',
+							'react/forbid-prop-types': 'off',
+						}
+					: {}),
 			},
 		},
 	];
@@ -282,21 +301,22 @@ const internals = {
 	},
 };
 
-// TODO: make sure to add a suggestion to set the engines fields in package.json
-
 /**
  * Defines a Zeno ESLint configuration.
  *
  * @param {Object|Array} arg1 - Options object or additional config array. If an array, treated as additional config.
  * @param {boolean} [arg1.react=false] - Enable React-specific rules.
  * @param {boolean} [arg1.ts=true] - Enable TypeScript-specific rules.
+ * @param {boolean} [arg1.performanceMode=false] - Disables expensive rules for performance.
+ * @param {string[]} [arg1.ignoreDirs=[]] - Additional directories to ignore (added to defaults: dist, build).
  * @param {string[]} [arg1.reactDirs=[]] - Directories containing React files (for projects using .js for both React and Node).
- * @param {string[]} [arg1.nodeIgnoreDirs=[]] - Directories to ignore for Node-specific rules.
+ * @param {string[]} [arg1.nodeIgnoreDirs=[]] - Directories to ignore for Node-specific rules only.
  * @param {string[]} [arg1.ignoreExports=[]] - Export patterns to ignore for import rules.
+ * @param {string[]} [arg1.additionalDevDependencies=[]] - Additional file patterns to allow dev dependencies in (for no-extraneous-dependencies rule).
  * @param {Object} [arg1.extensionsIgnorePattern={}] - Extension patterns to ignore for import rules.
  * @param {string} [arg1.webpackConfig] - Path to webpack config for import resolver.
  * @param {Array} [arg2] - Additional ESLint config objects to merge (only used if arg1 is options object).
- * @returns {Promise<Array>} ESLint flat config array.
+ * @returns {Array} ESLint flat config array.
  *
  * @example
  * // With options object
@@ -310,16 +330,21 @@ const internals = {
  * // With config array only
  * defineZenoConfig([customConfig])
  */
-const defineZenoConfig = async (arg1, arg2) => {
+const defineZenoConfig = (arg1, arg2) => {
 	let options = {
 		react: false,
 		ts: false,
+		performanceMode: false,
+
+		// additional directories to ignore (added to defaults: dist, build)
+		ignoreDirs: [],
 
 		// if a project uses .js file extension for both react and node files this will help separate the rules for each
 		reactDirs: [],
 		nodeIgnoreDirs: [],
 
 		ignoreExports: [],
+		additionalDevDependencies: [],
 		extensionsIgnorePattern: {},
 		webpackConfig: undefined,
 	};
@@ -333,6 +358,9 @@ const defineZenoConfig = async (arg1, arg2) => {
 	}
 
 	// Ensure array options are arrays
+	if (!Array.isArray(options.ignoreDirs)) {
+		options.ignoreDirs = [];
+	}
 	if (!Array.isArray(options.reactDirs)) {
 		options.reactDirs = [];
 	}
@@ -342,6 +370,9 @@ const defineZenoConfig = async (arg1, arg2) => {
 	if (!Array.isArray(options.ignoreExports)) {
 		options.ignoreExports = [];
 	}
+	if (!Array.isArray(options.additionalDevDependencies)) {
+		options.additionalDevDependencies = [];
+	}
 	if (
 		typeof options.extensionsIgnorePattern !== 'object' ||
 		options.extensionsIgnorePattern === null
@@ -358,24 +389,29 @@ const defineZenoConfig = async (arg1, arg2) => {
 		options.nodeIgnoreDirs = [...options.reactDirs];
 	}
 
-	// Load optional configs in parallel if needed
-	const [reactConfigResult, tsConfigResult] = await Promise.all([
-		options.react
-			? configs.getReact({
-					reactDirs: options.reactDirs,
-					extensionsIgnorePattern: options.extensionsIgnorePattern,
-				})
-			: Promise.resolve([]),
-		options.ts ? configs.getTypescript() : Promise.resolve([]),
-	]);
+	// Load optional configs if needed
+	const reactConfigResult = options.react
+		? configs.getReact({
+				reactDirs: options.reactDirs,
+				extensionsIgnorePattern: options.extensionsIgnorePattern,
+			})
+		: [];
+	const tsConfigResult = options.ts
+		? configs.getTypescript({ react: options.react })
+		: [];
 
 	return defineConfig([
+		{ ignores: [...defaultIgnoreDirs, ...options.ignoreDirs] },
 		...configs.getBase({
 			ignoreExports: options.ignoreExports,
+			additionalDevDependencies: options.additionalDevDependencies,
 			webpackConfig: options.webpackConfig,
 			extensionsIgnorePattern: options.extensionsIgnorePattern,
+			performanceMode: options.performanceMode,
+		}),
+		...configs.getNode({
+			ignoreDirs: options.nodeIgnoreDirs,
 		}),
-		...configs.getNode({ ignoreDirs: options.nodeIgnoreDirs }),
 		...reactConfigResult,
 		...tsConfigResult,
 

package/src/eslint/rules/baseRules.js

@@ -3,14 +3,12 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/array-callback-return
 		'array-callback-return': ['error', { allowImplicit: true }],
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/constructor-super
 		'constructor-super': 'error',
 
 		// https://eslint.org/docs/latest/rules/for-direction
 		'for-direction': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/getter-return
 		'getter-return': ['error', { allowImplicit: true }],
 
@@ -20,7 +18,6 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-await-in-loop
 		'no-await-in-loop': 'warn',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-class-assign
 		'no-class-assign': 'error',
 
@@ -30,7 +27,6 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-cond-assign
 		'no-cond-assign': ['error', 'always'],
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-const-assign
 		'no-const-assign': 'error',
 
@@ -49,18 +45,15 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-debugger
 		'no-debugger': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-dupe-args
 		'no-dupe-args': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-dupe-class-members
 		'no-dupe-class-members': 'error',
 
 		// https://eslint.org/docs/latest/rules/no-dupe-else-if
 		'no-dupe-else-if': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-dupe-keys
 		'no-dupe-keys': 'error',
 
@@ -82,12 +75,9 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-fallthrough
 		'no-fallthrough': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-func-assign
 		'no-func-assign': 'error',
 
-		// may still need this for TS since there is a scenario where you can assign an import to a variable
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-import-assign
 		'no-import-assign': 'error',
 
@@ -106,11 +96,9 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-misleading-character-class
 		'no-misleading-character-class': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-new-native-nonconstructor
 		'no-new-native-nonconstructor': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-obj-calls
 		'no-obj-calls': 'error',
 
@@ -126,7 +114,6 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-self-compare
 		'no-self-compare': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-setter-return
 		'no-setter-return': 'error',
 
@@ -136,14 +123,12 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-template-curly-in-string
 		'no-template-curly-in-string': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-this-before-super
 		'no-this-before-super': 'error',
 
 		// https://eslint.org/docs/latest/rules/no-unassigned-vars
 		'no-unassigned-vars': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-undef
 		'no-undef': ['error', { typeof: true }],
 
@@ -153,7 +138,6 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-unmodified-loop-condition
 		'no-unmodified-loop-condition': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS (only if allowUnreachableCode: false in config)
 		// https://eslint.org/docs/latest/rules/no-unreachable
 		'no-unreachable': 'error',
 
@@ -163,7 +147,6 @@ const getBaseRules = () => {
 		// https://eslint.org/docs/latest/rules/no-unsafe-finally
 		'no-unsafe-finally': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://eslint.org/docs/latest/rules/no-unsafe-negation
 		'no-unsafe-negation': ['error', { enforceForOrderingRelations: true }],
 
@@ -232,7 +215,7 @@ const getBaseRules = () => {
 
 		// https://eslint.org/docs/latest/rules/camelcase
 		camelcase: [
-			'error',
+			'warn',
 			{
 				properties: 'always',
 				ignoreDestructuring: false,
@@ -351,7 +334,7 @@ const getBaseRules = () => {
 		'no-array-constructor': 'error',
 
 		// https://eslint.org/docs/latest/rules/no-bitwise
-		'no-bitwise': 'error',
+		'no-bitwise': ['error', { allow: ['~'] }],
 
 		// https://eslint.org/docs/latest/rules/no-caller
 		'no-caller': 'error',
@@ -651,11 +634,11 @@ const getBaseRules = () => {
 			'error',
 			{
 				VariableDeclarator: {
-					array: true,
+					array: false,
 					object: true,
 				},
 				AssignmentExpression: {
-					array: true,
+					array: false,
 					object: true,
 				},
 			},

package/src/eslint/rules/importPluginRules.js

@@ -24,6 +24,12 @@ const getImportPluginRules = (options = {}) => {
 					`**/*.stories{${allExtensionsString}}`, // story files
 					`**/.prettierrc{${nodeExtensionsString}}`, // prettier config
 					`**/prettier.config{${nodeExtensionsString}}`, // prettier config
+					`**/.babelrc{${nodeExtensionsString}}`, // babel config
+					`**/babel.config{${nodeExtensionsString}}`, // babel config
+					`**/vite.config{${nodeExtensionsString}}`, // vite config
+					`**/vitest.config{${nodeExtensionsString}}`, // vitest config
+					`**/vitest.workspace{${nodeExtensionsString}}`, // vitest workspace config
+					...(options.additionalDevDependencies || []),
 				],
 				optionalDependencies: false,
 				peerDependencies: true,
@@ -43,18 +49,20 @@ const getImportPluginRules = (options = {}) => {
 		'import-x/no-named-as-default-member': 'off',
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-unused-modules.md
-		'import-x/no-unused-modules': [
-			'warn',
-			{
-				missingExports: false,
-				unusedExports: true,
-				ignoreUnusedTypeExports: false,
-				ignoreExports: [
-					`**/eslint.config{${nodeExtensionsString}}`,
-					...(options.ignoreExports || []),
-				],
-			},
-		],
+		// turning this off until I can iron out all of the false positives
+		'import-x/no-unused-modules': 'off',
+		// 'import-x/no-unused-modules': [
+		// 	'warn',
+		// 	{
+		// 		missingExports: false,
+		// 		unusedExports: true,
+		// 		ignoreUnusedTypeExports: false,
+		// 		ignoreExports: [
+		// 			`**/eslint.config{${nodeExtensionsString}}`,
+		// 			...(options.ignoreExports || []),
+		// 		],
+		// 	},
+		// ],
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-amd.md
 		'import-x/no-amd': 'error',
@@ -74,7 +82,6 @@ const getImportPluginRules = (options = {}) => {
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/default.md
 		'import-x/default': 'error',
 
-		// TODO: test this, may only be needed in JS and disabled in TS
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/named.md
 		'import-x/named': 'error',
 
@@ -85,7 +92,7 @@ const getImportPluginRules = (options = {}) => {
 		'import-x/no-absolute-path': 'error',
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-cycle.md
-		'import-x/no-cycle': 'error',
+		'import-x/no-cycle': options.performanceMode ? 'off' : 'error',
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-dynamic-require.md
 		'import-x/no-dynamic-require': 'error',

package/src/eslint/rules/reactPluginRules.js

@@ -4,13 +4,13 @@ const getReactPluginRules = (options = {}) => {
 	return {
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/boolean-prop-naming.md
 		'react/boolean-prop-naming': [
-			'error',
+			'off',
 			{
 				propTypeNames: ['bool', 'mutuallyExclusiveTrueProps'],
-				rule: '^(is|has|show|can)[A-Z]([A-Za-z0-9]?)+',
+				rule: '^(is|has|show|can|allow|should)[A-Z]([A-Za-z0-9]?)+',
 				validateNested: true,
 				message:
-					'Boolean prop ({{ propName }}) should start with "is", "has", "show", or "can"',
+					'Boolean prop ({{ propName }}) should start with "is", "has", "show", "can", "allow", or "should"',
 			},
 		],
 
@@ -33,7 +33,6 @@ const getReactPluginRules = (options = {}) => {
 			},
 		],
 
-		// TODO: can be disabled in TS
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/default-props-match-prop-types.md
 		'react/default-props-match-prop-types': [
 			'error',
@@ -59,11 +58,9 @@ const getReactPluginRules = (options = {}) => {
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/forbid-elements.md
 		'react/forbid-elements': 'off',
 
-		// TODO: can be disabled in TS
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/forbid-foreign-prop-types.md
 		'react/forbid-foreign-prop-types': 'error',
 
-		// TODO: can be disabled in TS
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/forbid-prop-types.md
 		'react/forbid-prop-types': [
 			'error',
@@ -213,7 +210,7 @@ const getReactPluginRules = (options = {}) => {
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-leaked-render.md
 		'react/jsx-no-leaked-render': [
-			'error',
+			'warn',
 			{ validStrategies: ['ternary', 'coerce'] },
 		],
 
@@ -441,7 +438,7 @@ const getReactPluginRules = (options = {}) => {
 		'react/self-closing-comp': 'error',
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/sort-comp.md
-		'react/sort-comp': 'error',
+		'react/sort-comp': 'off',
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/sort-default-props.md
 		'react/sort-default-props': 'off',

package/src/eslint/rules/stylisticPluginRules.js

@@ -117,9 +117,6 @@ const getStylisticPluginRules = () => {
 		// https://eslint.style/rules/jsx-self-closing-comp
 		'@stylistic/jsx-self-closing-comp': 'off', // conflicts with react/self-closing-comp
 
-		// https://eslint.style/rules/jsx-sort-props
-		'@stylistic/jsx-sort-props': 'off', // conflicts with react/jsx-sort-props
-
 		// https://eslint.style/rules/jsx-tag-spacing
 		'@stylistic/jsx-tag-spacing': 'off', // conflicts with prettier
 

package/src/eslint/rules/typescriptPluginRules.js

@@ -49,7 +49,6 @@ const getTypescriptPluginRules = () => {
 		// requires type information
 		'@typescript-eslint/consistent-type-exports': 'off',
 
-		// TODO: test this, might conflict with import plugin
 		// https://typescript-eslint.io/rules/consistent-type-imports
 		'@typescript-eslint/consistent-type-imports': 'off',
 
@@ -79,7 +78,6 @@ const getTypescriptPluginRules = () => {
 		// 'max-params': 'off',
 		'@typescript-eslint/max-params': 'off',
 
-		// TODO: test
 		// https://typescript-eslint.io/rules/member-ordering
 		'@typescript-eslint/member-ordering': 'off',
 

package/src/eslint/rules/unicornPluginRules.js

@@ -251,25 +251,25 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-array-some': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-at.md
-		'unicorn/prefer-at': 'error',
+		'unicorn/prefer-at': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-bigint-literals.md
-		'unicorn/prefer-bigint-literals': 'error',
+		'unicorn/prefer-bigint-literals': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-blob-reading-methods.md
-		'unicorn/prefer-blob-reading-methods': 'error',
+		'unicorn/prefer-blob-reading-methods': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-class-fields.md
-		'unicorn/prefer-class-fields': 'error',
+		'unicorn/prefer-class-fields': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-classlist-toggle.md
-		'unicorn/prefer-classlist-toggle': 'error',
+		'unicorn/prefer-classlist-toggle': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-code-point.md
-		'unicorn/prefer-code-point': 'error',
+		'unicorn/prefer-code-point': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-date-now.md
-		'unicorn/prefer-date-now': 'error',
+		'unicorn/prefer-date-now': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-default-parameters.md
 		'unicorn/prefer-default-parameters': 'error',
@@ -305,13 +305,13 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-json-parse-buffer': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-keyboard-event-key.md
-		'unicorn/prefer-keyboard-event-key': 'error',
+		'unicorn/prefer-keyboard-event-key': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-logical-operator-over-ternary.md
 		'unicorn/prefer-logical-operator-over-ternary': 'error',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-math-min-max.md
-		'unicorn/prefer-math-min-max': 'error',
+		'unicorn/prefer-math-min-max': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-math-trunc.md
 		'unicorn/prefer-math-trunc': 'off',
@@ -320,13 +320,13 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-modern-dom-apis': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-modern-math-apis.md
-		'unicorn/prefer-modern-math-apis': 'error',
+		'unicorn/prefer-modern-math-apis': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-module.md
 		'unicorn/prefer-module': 'error',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-native-coercion-functions.md
-		'unicorn/prefer-native-coercion-functions': 'error',
+		'unicorn/prefer-native-coercion-functions': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-negative-index.md
 		'unicorn/prefer-negative-index': 'off',
@@ -335,10 +335,10 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-node-protocol': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-number-properties.md
-		'unicorn/prefer-number-properties': 'error',
+		'unicorn/prefer-number-properties': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-object-from-entries.md
-		'unicorn/prefer-object-from-entries': 'error',
+		'unicorn/prefer-object-from-entries': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-optional-catch-binding.md
 		'unicorn/prefer-optional-catch-binding': 'off',
@@ -350,10 +350,10 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-query-selector': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-reflect-apply.md
-		'unicorn/prefer-reflect-apply': 'error',
+		'unicorn/prefer-reflect-apply': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-regexp-test.md
-		'unicorn/prefer-regexp-test': 'error',
+		'unicorn/prefer-regexp-test': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-response-static-json.md
 		'unicorn/prefer-response-static-json': 'error',
@@ -368,7 +368,7 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-single-call': 'error',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-spread.md
-		'unicorn/prefer-spread': 'error',
+		'unicorn/prefer-spread': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-string-raw.md
 		'unicorn/prefer-string-raw': 'off',
@@ -377,13 +377,13 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-string-replace-all': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-string-slice.md
-		'unicorn/prefer-string-slice': 'error',
+		'unicorn/prefer-string-slice': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-string-starts-ends-with.md
 		'unicorn/prefer-string-starts-ends-with': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-string-trim-start-end.md
-		'unicorn/prefer-string-trim-start-end': 'error',
+		'unicorn/prefer-string-trim-start-end': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-structured-clone.md
 		'unicorn/prefer-structured-clone': 'error',
@@ -392,13 +392,13 @@ const getUnicornPluginRules = () => {
 		'unicorn/prefer-switch': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-ternary.md
-		'unicorn/prefer-ternary': 'error',
+		'unicorn/prefer-ternary': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-top-level-await.md
 		'unicorn/prefer-top-level-await': 'error',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-type-error.md
-		'unicorn/prefer-type-error': 'error',
+		'unicorn/prefer-type-error': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prevent-abbreviations.md
 		'unicorn/prevent-abbreviations': 'off',