zeno-config 2.1.0 → 3.0.0

package/README.md

@@ -70,7 +70,7 @@ export default defineZenoConfig(
 		ts: true,
 
 		// Additional directories to ignore (added to defaults: node_modules, dist, build, coverage)
-		ignoreDirs: ['out', '.next'],
+		ignores: ['out', '.next'],
 
 		// If using .js extensions for React files, specify React directories
 		reactDirs: ['src/client', 'src/components'],
@@ -199,7 +199,7 @@ import {
 | `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)     |
+| `ignores`                   | `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                                       |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zeno-config",
-  "version": "2.1.0",
+  "version": "3.0.0",
   "description": "Preconfigured and opinionated ESLint, Prettier, and TypeScript setup",
   "author": "Rick Brenn <brenn.rick@gmail.com>",
   "license": "MIT",

package/src/eslint/index.d.ts

@@ -38,7 +38,7 @@ interface DefineZenoConfigOptions {
 	/** Disables expensive rules for performance */
 	performanceMode?: boolean;
 	/** Additional directories to ignore (added to defaults: dist, build) */
-	ignoreDirs?: string[];
+	ignores?: string[];
 	/** Directories containing React files (for projects using .js for both React and Node) */
 	reactDirs?: string[];
 	/** Directories to ignore for Node-specific rules only */

package/src/eslint/index.js

@@ -308,7 +308,7 @@ const internals = {
  * @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.ignores=[]] - 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 only.
  * @param {string[]} [arg1.ignoreExports=[]] - Export patterns to ignore for import rules.
@@ -337,7 +337,7 @@ const defineZenoConfig = (arg1, arg2) => {
 		performanceMode: false,
 
 		// additional directories to ignore (added to defaults: dist, build)
-		ignoreDirs: [],
+		ignores: [],
 
 		// if a project uses .js file extension for both react and node files this will help separate the rules for each
 		reactDirs: [],
@@ -358,8 +358,8 @@ const defineZenoConfig = (arg1, arg2) => {
 	}
 
 	// Ensure array options are arrays
-	if (!Array.isArray(options.ignoreDirs)) {
-		options.ignoreDirs = [];
+	if (!Array.isArray(options.ignores)) {
+		options.ignores = [];
 	}
 	if (!Array.isArray(options.reactDirs)) {
 		options.reactDirs = [];
@@ -401,7 +401,7 @@ const defineZenoConfig = (arg1, arg2) => {
 		: [];
 
 	return defineConfig([
-		{ ignores: [...defaultIgnoreDirs, ...options.ignoreDirs] },
+		{ ignores: [...defaultIgnoreDirs, ...options.ignores] },
 		...configs.getBase({
 			ignoreExports: options.ignoreExports,
 			additionalDevDependencies: options.additionalDevDependencies,
@@ -419,6 +419,14 @@ const defineZenoConfig = (arg1, arg2) => {
 
 		// Turn off formatting rules that might conflict with Prettier
 		prettierPlugin,
+
+		{
+			files: [`**/*{${allExtensionsString}}`],
+			rules: {
+				// prettier disables this but we can safely enable it
+				curly: 'error',
+			},
+		},
 	]);
 };
 

package/src/eslint/rules/baseRules.js

@@ -16,7 +16,7 @@ const getBaseRules = () => {
 		'no-async-promise-executor': 'error',
 
 		// https://eslint.org/docs/latest/rules/no-await-in-loop
-		'no-await-in-loop': 'warn',
+		'no-await-in-loop': 'off',
 
 		// https://eslint.org/docs/latest/rules/no-class-assign
 		'no-class-assign': 'error',
@@ -196,7 +196,7 @@ const getBaseRules = () => {
 		'no-useless-backreference': 'error',
 
 		// https://eslint.org/docs/latest/rules/require-atomic-updates
-		'require-atomic-updates': ['error', { allowProperties: false }],
+		'require-atomic-updates': ['off', { allowProperties: false }], // noticed a feew weird cases with this one
 
 		// https://eslint.org/docs/latest/rules/use-isnan
 		'use-isnan': 'error',
@@ -215,7 +215,7 @@ const getBaseRules = () => {
 
 		// https://eslint.org/docs/latest/rules/camelcase
 		camelcase: [
-			'warn',
+			'off',
 			{
 				properties: 'always',
 				ignoreDestructuring: false,
@@ -250,7 +250,7 @@ const getBaseRules = () => {
 		'default-case-last': 'error',
 
 		// https://eslint.org/docs/latest/rules/default-param-last
-		'default-param-last': 'error',
+		'default-param-last': 'off',
 
 		// https://eslint.org/docs/latest/rules/dot-notation
 		'dot-notation': ['error', { allowKeywords: true }],
@@ -319,7 +319,7 @@ const getBaseRules = () => {
 
 		// https://eslint.org/docs/latest/rules/new-cap
 		'new-cap': [
-			'error',
+			'off',
 			{
 				newIsCap: true,
 				capIsNew: true,
@@ -401,7 +401,7 @@ const getBaseRules = () => {
 				number: true,
 				string: true,
 				disallowTemplateShorthand: true,
-				allow: ['!!', '~'],
+				allow: ['!!', '~', '+'],
 			},
 		],
 

package/src/eslint/rules/importPluginRules.js

@@ -135,7 +135,7 @@ const getImportPluginRules = (options = {}) => {
 		'import-x/dynamic-import-chunkname': 'off',
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/exports-last.md
-		'import-x/exports-last': 'error',
+		'import-x/exports-last': 'off',
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/extensions.md
 		'import-x/extensions': [
@@ -148,7 +148,7 @@ const getImportPluginRules = (options = {}) => {
 		'import-x/first': 'error',
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/group-exports.md
-		'import-x/group-exports': 'error',
+		'import-x/group-exports': 'off',
 
 		// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/max-dependencies.md
 		'import-x/max-dependencies': 'off',

package/src/eslint/rules/jsxA11yPluginRules.js

@@ -73,7 +73,7 @@ const getJsxA11yPluginRules = () => {
 		'jsx-a11y/no-aria-hidden-on-focusable': 'off',
 
 		// https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/main/docs/rules/no-autofocus.md
-		'jsx-a11y/no-autofocus': 'error',
+		'jsx-a11y/no-autofocus': 'off',
 
 		// https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/main/docs/rules/no-distracting-elements.md
 		'jsx-a11y/no-distracting-elements': 'error',

package/src/eslint/rules/reactPluginRules.js

@@ -59,11 +59,11 @@ const getReactPluginRules = (options = {}) => {
 		'react/forbid-elements': 'off',
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/forbid-foreign-prop-types.md
-		'react/forbid-foreign-prop-types': 'error',
+		'react/forbid-foreign-prop-types': 'warn',
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/forbid-prop-types.md
 		'react/forbid-prop-types': [
-			'error',
+			'warn',
 			{
 				forbid: ['any', 'array', 'object'],
 				checkContextTypes: true,
@@ -142,7 +142,7 @@ const getReactPluginRules = (options = {}) => {
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-handler-names.md
 		'react/jsx-handler-names': [
-			'warn',
+			'off',
 			{
 				eventHandlerPrefix: 'handle',
 				eventHandlerPropPrefix: 'on',
@@ -210,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': [
-			'warn',
+			'off',
 			{ validStrategies: ['ternary', 'coerce'] },
 		],
 
@@ -254,7 +254,7 @@ const getReactPluginRules = (options = {}) => {
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-props-no-spreading.md
 		'react/jsx-props-no-spreading': [
-			'error',
+			'warn',
 			{
 				html: 'enforce',
 				custom: 'ignore',
@@ -457,7 +457,7 @@ const getReactPluginRules = (options = {}) => {
 		],
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/state-in-constructor.md
-		'react/state-in-constructor': ['error', 'always'],
+		'react/state-in-constructor': ['off', 'always'],
 
 		// https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/static-property-placement.md
 		'react/static-property-placement': 'off',

package/src/eslint/rules/unicornPluginRules.js

@@ -22,7 +22,7 @@ const getUnicornPluginRules = () => {
 		'unicorn/consistent-existence-index-check': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/consistent-function-scoping.md
-		'unicorn/consistent-function-scoping': 'error',
+		'unicorn/consistent-function-scoping': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/custom-error-definition.md
 		'unicorn/custom-error-definition': 'error',
@@ -44,7 +44,7 @@ const getUnicornPluginRules = () => {
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/filename-case.md
 		'unicorn/filename-case': [
-			'error',
+			'off',
 			{
 				cases: {
 					camelCase: true,
@@ -91,7 +91,7 @@ const getUnicornPluginRules = () => {
 		'unicorn/no-array-sort': 'warn',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-await-expression-member.md
-		'unicorn/no-await-expression-member': 'error',
+		'unicorn/no-await-expression-member': 'off', // auto fix breaks logic
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-await-in-promise-methods.md
 		'unicorn/no-await-in-promise-methods': 'error',
@@ -160,7 +160,7 @@ const getUnicornPluginRules = () => {
 		'unicorn/no-process-exit': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-single-promise-in-promise-methods.md
-		'unicorn/no-single-promise-in-promise-methods': 'error',
+		'unicorn/no-single-promise-in-promise-methods': 'off', // false positives with new Class as elements in arrays
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-static-only-class.md
 		'unicorn/no-static-only-class': 'error',
@@ -217,7 +217,7 @@ const getUnicornPluginRules = () => {
 		'unicorn/no-useless-spread': 'error',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-useless-switch-case.md
-		'unicorn/no-useless-switch-case': 'error',
+		'unicorn/no-useless-switch-case': 'off',
 
 		// this conflicts with requiring default props in React
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-useless-undefined.md
@@ -386,7 +386,7 @@ const getUnicornPluginRules = () => {
 		'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',
+		'unicorn/prefer-structured-clone': 'off',
 
 		// https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-switch.md
 		'unicorn/prefer-switch': 'off',