@storm-software/eslint 0.35.0 → 0.36.0

package/README.md

@@ -21,7 +21,7 @@ This package is part of the <b>⚡Storm-Ops</b> monorepo. The Storm-Ops packages
 
 <h3 align="center">💻 Visit <a href="https://stormsoftware.com" target="_blank">stormsoftware.com</a> to stay up to date with this developer</h3><br />
 
-[![Version](https://img.shields.io/badge/version-0.32.0-1fb2a6.svg?style=for-the-badge&color=1fb2a6)](https://prettier.io/)&nbsp;[![Nx](https://img.shields.io/badge/Nx-17.0.2-lightgrey?style=for-the-badge&logo=nx&logoWidth=20&&color=1fb2a6)](http://nx.dev/)&nbsp;[![NextJs](https://img.shields.io/badge/Next.js-14.0.2-lightgrey?style=for-the-badge&logo=nextdotjs&logoWidth=20&color=1fb2a6)](https://nextjs.org/)&nbsp;[![Commitizen friendly](https://img.shields.io/badge/commitizen-friendly-brightgreen.svg?style=for-the-badge&logo=commitlint&color=1fb2a6)](http://commitizen.github.io/cz-cli/)&nbsp;![Semantic-Release](https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079.svg?style=for-the-badge&color=1fb2a6)&nbsp;[![documented with Fumadocs](https://img.shields.io/badge/documented_with-fumadocs-success.svg?style=for-the-badge&logo=readthedocs&color=1fb2a6)](https://fumadocs.vercel.app/)&nbsp;![GitHub Workflow Status (with event)](https://img.shields.io/github/actions/workflow/status/storm-software/storm-ops/cr.yml?style=for-the-badge&logo=github-actions&color=1fb2a6)
+[![Version](https://img.shields.io/badge/version-0.36.0-1fb2a6.svg?style=for-the-badge&color=1fb2a6)](https://prettier.io/)&nbsp;[![Nx](https://img.shields.io/badge/Nx-17.0.2-lightgrey?style=for-the-badge&logo=nx&logoWidth=20&&color=1fb2a6)](http://nx.dev/)&nbsp;[![NextJs](https://img.shields.io/badge/Next.js-14.0.2-lightgrey?style=for-the-badge&logo=nextdotjs&logoWidth=20&color=1fb2a6)](https://nextjs.org/)&nbsp;[![Commitizen friendly](https://img.shields.io/badge/commitizen-friendly-brightgreen.svg?style=for-the-badge&logo=commitlint&color=1fb2a6)](http://commitizen.github.io/cz-cli/)&nbsp;![Semantic-Release](https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079.svg?style=for-the-badge&color=1fb2a6)&nbsp;[![documented with Fumadocs](https://img.shields.io/badge/documented_with-fumadocs-success.svg?style=for-the-badge&logo=readthedocs&color=1fb2a6)](https://fumadocs.vercel.app/)&nbsp;![GitHub Workflow Status (with event)](https://img.shields.io/github/actions/workflow/status/storm-software/storm-ops/cr.yml?style=for-the-badge&logo=github-actions&color=1fb2a6)
 
 <!-- prettier-ignore-start -->
 <!-- markdownlint-disable -->

package/dist/preset.mjs

@@ -337,7 +337,7 @@ var src = {
 
 const eslint = /*@__PURE__*/getDefaultExportFromCjs(src);
 
-const config$3 = {
+const config$4 = {
   // This rule has been deprecated, but not yet removed.
   "jsx-a11y/no-onchange": "off",
   // ensure emoji are accessible
@@ -612,7 +612,7 @@ const config$3 = {
   "jsx-a11y/prefer-tag-over-role": "off"
 };
 
-const config$2 = {
+const config$3 = {
   // We recommend using TypeScript over `prop-types`, as `prop-types` can add
   // to a project's build size.
   "react/prop-types": "off",
@@ -1177,7 +1177,7 @@ const config$2 = {
   ]
 };
 
-const config$1 = {
+const config$2 = {
   // Enforce Rules of Hooks
   // https://github.com/facebook/react/blob/c11015ff4f610ac2924d1fc6d569a17657a404fd/packages/eslint-plugin-react-hooks/src/RulesOfHooks.js
   "react-hooks/rules-of-hooks": "error",
@@ -1186,6 +1186,300 @@ const config$1 = {
   "react-hooks/exhaustive-deps": "warn"
 };
 
+const config$1 = {
+  /**
+   * Require return statements in array methods callbacks.
+   *
+   * 🚫 Not fixable -https://eslint.org/docs/rules/array-callback-return
+   */
+  "array-callback-return": ["error", { allowImplicit: true }],
+  /**
+   * Treat `var` statements as if they were block scoped.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/block-scoped-var
+   */
+  "block-scoped-var": "error",
+  /**
+   * Require curly braces for multiline blocks.
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/curly
+   */
+  curly: ["warn", "multi-line"],
+  /**
+   * Require default clauses in switch statements to be last (if used).
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/default-case-last
+   */
+  "default-case-last": "error",
+  /**
+   * Require triple equals (`===` and `!==`).
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/eqeqeq
+   */
+  eqeqeq: "error",
+  /**
+   * Require grouped accessor pairs in object literals and classes.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/grouped-accessor-pairs
+   */
+  "grouped-accessor-pairs": "error",
+  /**
+   * Disallow use of `alert()`.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-alert
+   */
+  "no-alert": "error",
+  /**
+   * Disallow use of `caller`/`callee`.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-caller
+   */
+  "no-caller": "error",
+  /**
+   * Disallow returning value in constructor.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-constructor-return
+   */
+  "no-constructor-return": "error",
+  /**
+   * Disallow using an `else` if the `if` block contains a return.
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/no-else-return
+   */
+  "no-else-return": "warn",
+  /**
+   * Disallow `eval()`.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-eval
+   */
+  "no-eval": "error",
+  /**
+   * Disallow extending native objects.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-extend-native
+   */
+  "no-extend-native": "error",
+  /**
+   * Disallow unnecessary function binding.
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/no-extra-bind
+   */
+  "no-extra-bind": "error",
+  /**
+   * Disallow unnecessary labels.
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/no-extra-label
+   */
+  "no-extra-label": "error",
+  /**
+   * Disallow floating decimals.
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/no-floating-decimal
+   */
+  "no-floating-decimal": "error",
+  /**
+   * Make people convert types explicitly e.g. `Boolean(foo)` instead of `!!foo`.
+   *
+   * 🔧 Partially Fixable - https://eslint.org/docs/rules/no-implicit-coercion
+   */
+  "no-implicit-coercion": "error",
+  /**
+   * Disallow use of `eval()`-like methods.
+   *
+   * https://eslint.org/docs/rules/no-implied-eval
+   */
+  "no-implied-eval": "error",
+  /**
+   * Disallow usage of `__iterator__` property.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-iterator
+   */
+  "no-iterator": "error",
+  /**
+   * Disallow use of labels for anything other than loops and switches.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-labels
+   */
+  "no-labels": ["error"],
+  /**
+   * Disallow unnecessary nested blocks.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-lone-blocks
+   */
+  "no-lone-blocks": "error",
+  /**
+   * Disallow `new` for side effects.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-new
+   */
+  "no-new": "error",
+  /**
+   * Disallow function constructors.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-new-func
+   */
+  "no-new-func": "error",
+  /**
+   * Disallow base types wrapper instances, such as `new String('foo')`.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-new-wrappers
+   */
+  "no-new-wrappers": "error",
+  /**
+   * Disallow use of octal escape sequences in string literals.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-octal-escape
+   */
+  "no-octal-escape": "error",
+  /**
+   * Disallow reassignment of function parameters.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-param-reassign
+   */
+  "no-param-reassign": "error",
+  /**
+   * Disallow usage of the deprecated `__proto__` property.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-proto
+   */
+  "no-proto": "error",
+  /**
+   * Disallow assignment in `return` statement.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-return-assign
+   */
+  "no-return-assign": "error",
+  /**
+   * Disallows unnecessary `return await`.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-return-await
+   */
+  "no-return-await": "error",
+  /**
+   * Disallow use of `javascript:` urls.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-script-url
+   */
+  "no-script-url": "error",
+  /**
+   * Disallow comparisons where both sides are exactly the same.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-self-compare
+   */
+  "no-self-compare": "error",
+  /**
+   * Disallow use of comma operator.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-sequences
+   */
+  "no-sequences": "error",
+  /**
+   * Disallow unnecessary `.call()` and `.apply()`.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-useless-call
+   */
+  "no-useless-call": "error",
+  /**
+   * Disallow unnecessary concatenation of strings.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/no-useless-concat
+   */
+  "no-useless-concat": "error",
+  /**
+   * Disallow redundant return statements.
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/no-useless-return
+   */
+  "no-useless-return": "warn",
+  /**
+   * Require using named capture groups in regular expressions.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/prefer-named-capture-group
+   */
+  "prefer-named-capture-group": "error",
+  /**
+   * Require using Error objects as Promise rejection reasons.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/prefer-promise-reject-errors
+   */
+  "prefer-promise-reject-errors": ["error", { allowEmptyReject: true }],
+  /**
+   * Disallow use of the RegExp constructor in favor of regular expression
+   * literals.
+   *
+   * 🚫 Not fixable - https://eslint.org/docs/rules/prefer-regex-literals
+   */
+  "prefer-regex-literals": "error",
+  /**
+   * Disallow "Yoda conditions", ensuring the comparison.
+   *
+   * 🔧 Fixable - https://eslint.org/docs/rules/yoda
+   */
+  yoda: "warn",
+  "unicorn/number-literal-case": "off",
+  "unicorn/template-indent": "off",
+  "unicorn/prevent-abbreviations": "off",
+  "unicorn/no-await-expression-member": "off",
+  "unicorn/no-useless-undefined": "off",
+  "unicorn/no-array-push-push": "off",
+  "unicorn/no-array-reduce": "off",
+  "unicorn/no-useless-switch-case": "off",
+  "@typescript-eslint/no-explicit-any": "off",
+  "@typescript-eslint/no-empty-function": "off",
+  "@typescript-eslint/no-var-requires": "off",
+  "@typescript-eslint/ban-ts-comment": "off",
+  "@typescript-eslint/no-empty-interface": "off",
+  "@typescript-eslint/explicit-module-boundary-types": "off",
+  "@typescript-eslint/explicit-function-return-type": "off",
+  "@typescript-eslint/no-unused-vars": [
+    "warn",
+    { varsIgnorePattern: "^_", argsIgnorePattern: "^_" }
+  ],
+  "unicorn/prefer-string-replace-all": "off",
+  "unicorn/no-abusive-eslint-disable": "off",
+  "unicorn/import-style": "off",
+  "unicorn/prefer-module": "off",
+  "unicorn/consistent-function-scoping": "off",
+  "class-methods-use-this": "off",
+  "@typescript-eslint/no-restricted-imports": [
+    "error",
+    {
+      "patterns": [
+        {
+          "group": ["nx/src/plugins/js*"],
+          "message": "Imports from 'nx/src/plugins/js' are not allowed. Use '@nx/js' instead"
+        },
+        {
+          "group": ["**/native-bindings", "**/native-bindings.js"],
+          "message": "Direct imports from native-bindings.js are not allowed. Import from index.js instead."
+        },
+        {
+          "group": ["create-storm-workspace"],
+          "message": "Direct imports from `create-storm-workspace` are not allowed. Instead install this package globally (example: 'npm i create-storm-workspace -g')."
+        },
+        {
+          "group": ["create-nx-workspace"],
+          "message": "Direct imports from `create-nx-workspace` are not allowed. Instead install this package globally (example: 'npm i create-nx-workspace -g')."
+        }
+      ]
+    }
+  ],
+  "@nx/enforce-module-boundaries": [
+    "error",
+    {
+      enforceBuildableLibDependency: true,
+      checkDynamicDependenciesExceptions: [".*"],
+      allow: [],
+      depConstraints: [
+        {
+          sourceTag: "*",
+          onlyDependOnLibsWithTags: ["*"]
+        }
+      ]
+    }
+  ]
+};
+
 const config = {
   /**
    * Require TSDoc comments conform to the TSDoc specification.
@@ -12629,67 +12923,11 @@ const formatConfig = (name, config = []) => {
   });
 };
 
-const DEFAULT_TYPESCRIPT_RULES = {
-  "unicorn/number-literal-case": 0,
-  "unicorn/template-indent": 0,
-  "unicorn/prevent-abbreviations": 0,
-  "unicorn/no-await-expression-member": 0,
-  "unicorn/no-useless-undefined": 0,
-  "unicorn/no-array-push-push": 0,
-  "unicorn/no-array-reduce": 0,
-  "unicorn/no-useless-switch-case": 0,
-  "@typescript-eslint/no-explicit-any": 0,
-  "@typescript-eslint/no-empty-function": 0,
-  "@typescript-eslint/no-var-requires": 0,
-  "@typescript-eslint/ban-ts-comment": 0,
-  "@typescript-eslint/no-empty-interface": 0,
-  "@typescript-eslint/explicit-module-boundary-types": 0,
-  "@typescript-eslint/explicit-function-return-type": 0,
-  "@typescript-eslint/no-unused-vars": [
-    "warn",
-    { varsIgnorePattern: "^_", argsIgnorePattern: "^_" }
-  ],
-  "unicorn/prefer-string-replace-all": 0,
-  "unicorn/no-abusive-eslint-disable": 0,
-  "unicorn/import-style": 0,
-  "unicorn/prefer-module": 0,
-  "unicorn/consistent-function-scoping": 0,
-  "class-methods-use-this": 0,
-  "@typescript-eslint/no-restricted-imports": [
-    "error",
-    {
-      "patterns": [
-        {
-          "group": ["nx/src/plugins/js*"],
-          "message": "Imports from 'nx/src/plugins/js' are not allowed. Use '@nx/js' instead"
-        },
-        {
-          "group": ["**/native-bindings", "**/native-bindings.js"],
-          "message": "Direct imports from native-bindings.js are not allowed. Import from index.js instead."
-        },
-        {
-          "group": ["create-storm-workspace"],
-          "message": "Direct imports from `create-storm-workspace` are not allowed. Instead install this package globally (example: 'npm i create-storm-workspace -g')."
-        },
-        {
-          "group": ["create-nx-workspace"],
-          "message": "Direct imports from `create-nx-workspace` are not allowed. Instead install this package globally (example: 'npm i create-nx-workspace -g')."
-        }
-      ]
-    }
-  ]
-};
 function stormPreset(options = {
-  moduleBoundaries: {
-    enforceBuildableLibDependency: true,
-    allow: [],
-    depConstraints: [
-      {
-        sourceTag: "*",
-        onlyDependOnLibsWithTags: ["*"]
-      }
-    ]
-  }
+  rules: {},
+  ignores: [],
+  markdown: {},
+  react: {}
 }, ...userConfigs) {
   const configs = [
     // https://eslint.org/docs/latest/rules/
@@ -12700,11 +12938,6 @@ function stormPreset(options = {
     eslintPluginUnicorn.configs["flat/recommended"],
     // Prettier
     prettierConfig,
-    // React
-    // https://www.npmjs.com/package/eslint-plugin-react
-    options.react !== false && {
-      plugins: { react, "react-hooks": reactHooks, "jsx-a11y": jsxA11y }
-    },
     // Import
     // https://www.npmjs.com/package/eslint-plugin-import
     // { plugins: { import: importEslint } },
@@ -12766,7 +12999,9 @@ function stormPreset(options = {
     ...userConfigs
   ].filter(Boolean);
   const typescriptConfig = {
+    // https://typescript-eslint.io/
     ...tsEslint$1.configs.stylisticTypeChecked,
+    // https://www.npmjs.com/package/eslint-plugin-unicorn
     ...eslintPluginUnicorn.configs["flat/recommended"],
     files: [CODE_FILE],
     languageOptions: {
@@ -12797,73 +13032,59 @@ function stormPreset(options = {
       ...eslintPluginUnicorn.configs["flat/recommended"].rules,
       // Banner
       ...plugin.configs["recommended"][1].rules,
-      // NX
-      "@nx/enforce-module-boundaries": [
-        "error",
-        options.moduleBoundaries ? options.moduleBoundaries : {
-          enforceBuildableLibDependency: true,
-          checkDynamicDependenciesExceptions: [".*"],
-          allow: [],
-          depConstraints: [
-            {
-              sourceTag: "*",
-              onlyDependOnLibsWithTags: ["*"]
-            }
-          ]
-        }
-      ],
-      ...DEFAULT_TYPESCRIPT_RULES,
-      ...options.typescript ?? {}
+      ...config$1,
+      ...options.rules ?? {}
     },
     ignores: ["dist", "coverage", "tmp", ...options.ignores || []]
   };
-  if (options.react !== false) {
+  if (options.react) {
+    typescriptConfig.plugins = {
+      ...typescriptConfig.plugins,
+      react,
+      "react-hooks": reactHooks,
+      "jsx-a11y": jsxA11y
+    };
     typescriptConfig.rules = {
       ...typescriptConfig.rules,
       // React
+      ...config$3,
       ...config$2,
-      ...config$1,
-      ...config$3
+      ...config$4,
+      ...options.react
     };
-    if (options.react?.rules) {
-      typescriptConfig.rules = {
-        ...typescriptConfig.rules,
-        ...options.react.rules
-      };
-    }
   }
   configs.push(typescriptConfig);
-  if (options.markdown !== false) {
+  if (options.markdown) {
     configs.push(...markdown.configs.recommended);
     configs.push({
       files: [CODE_BLOCK],
       processor: "markdown/markdown",
       rules: {
-        "unicorn/filename-case": 0,
-        "no-undef": 0,
-        "no-unused-expressions": 0,
-        "padded-blocks": 0,
-        "no-empty-pattern": 0,
-        "no-redeclare": 0,
-        "no-import-assign": 0,
-        ...options.markdown?.rules
+        "unicorn/filename-case": "off",
+        "no-undef": "off",
+        "no-unused-expressions": "off",
+        "padded-blocks": "off",
+        "no-empty-pattern": "off",
+        "no-redeclare": "off",
+        "no-import-assign": "off",
+        ...options.markdown
       }
     });
     configs.push({
       files: ["**/*.md/*.js", "**/*.md/*.ts"],
       rules: {
-        "unicorn/filename-case": 0,
-        "no-undef": 0,
-        "no-unused-expressions": 0,
-        "padded-blocks": 0,
-        "no-empty-pattern": 0,
-        "no-redeclare": 0,
-        "no-import-assign": 0,
-        ...options.markdown?.rules
+        "unicorn/filename-case": "off",
+        "no-undef": "off",
+        "no-unused-expressions": "off",
+        "padded-blocks": "off",
+        "no-empty-pattern": "off",
+        "no-redeclare": "off",
+        "no-import-assign": "off",
+        ...options.markdown
       }
     });
   }
   return formatConfig("Preset", configs);
 }
 
-export { DEFAULT_TYPESCRIPT_RULES, stormPreset as default };
+export { stormPreset as default };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@storm-software/eslint",
-  "version": "0.35.0",
+  "version": "0.36.0",
   "type": "module",
   "description": "⚡ A package containing the base ESLint configuration used by Storm Software across many projects.",
   "repository": {
@@ -31,6 +31,10 @@
     ".": {
       "types": "./dist/preset.d.mts",
       "import": "./dist/preset.mjs"
+    },
+    "./preset": {
+      "types": "./dist/preset.d.mts",
+      "import": "./dist/preset.mjs"
     }
   },
   "files": [