zeno-config 2.1.0 → 2.2.0

package/package.json

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

package/src/eslint/index.js

@@ -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