eslint-plugin-constructors 1.4.0 → 1.4.2

package/README.md

@@ -6,6 +6,8 @@
     <img alt="[Latest Build]" src="https://img.shields.io/npm/v/eslint-plugin-constructors?logo=icinga&style=for-the-badge&label=Latest%20Build&color=32fcee"></a>
 <a href="#license">
     <img alt="[License: MIT]" src="https://img.shields.io/badge/License-MIT-f1f20a.svg?logo=internetarchive&logoColor=white&labelColor=464646&style=for-the-badge"></a>
+<a>
+    <img src="https://img.shields.io/npm/dm/eslint-plugin-constructors?style=for-the-badge&logo=npm&color=black"></a>
 <a target="_blank" href="https://eslint.org/">
     <img alt="[Requires ESLint >=9.0.0]" src="https://img.shields.io/badge/Requires-ESLint_%3E%3D9.0.0-4B32C3?style=for-the-badge&logo=eslint"></a>
 <br><br>
@@ -50,7 +52,7 @@ export default [
 ]
 ```
 
-Available bundles: `recommended`, `problem`
+Available bundles: `recommended`, `problem` (deterministic bugs only)
 
 **Note:** Rule config is either a severity string (`'off'`, `'warn'`, `'error'`) or an array `[severity, options]` when the rule accepts options.
 
@@ -226,7 +228,7 @@ If you prefer calling built-ins w/o `new` (which the runtime allows for some of
 
 ---
 
-## <img height=30 align="top" src="https://api.iconify.design/boxicons/code-alt-filled.svg?color=%232b2b2a"> Shield embed
+## <img width=30 align="top" src="https://api.iconify.design/boxicons/code-alt-filled.svg?color=%232b2b2a"> Shield embed
 
 <a target="_blank" href="https://codeberg.org/adamlui/eslint-plugin-constructors/#readme">
     <img height=32 alt="[Linted by eslint-plugin-constructors]" src="https://img.shields.io/badge/Linted_by-eslint--plugin--constructors-black?logo=eslint&logoColor=white&labelColor=464646&style=for-the-badge"></a>
@@ -268,6 +270,20 @@ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLI
 
 ## <img height=30 width=29 align="top" src="https://api.iconify.design/bxs/package.svg?color=%232b2b2a"> Related
 
+<!-- eslint-plugin-chrome-extensions -->
+
+### <a target="_blank" href="https://codeberg.org/adamlui/eslint-plugin-chrome-extensions/#readme"><img height=15 width=auto src="https://cdn.staticdelivr.com/gl/adamlui/eslint-plugin-greasemonkey/e9128375d4/assets/images/icons/eslint/purple/icon16.png"> eslint-plugin-chrome-extensions</a>
+
+<details>
+
+<summary>ESLint rules for Chrome extensions.</summary><br>
+
+> <a target="_blank" href="https://codeberg.org/adamlui/eslint-plugin-chrome-extensions/#readme"><img width=555 src="https://cdn.staticdelivr.com/gl/adamlui/eslint-plugin-chrome-extensions/0b5e3e49/assets/images/screenshots/vs-code/squiggles.png"></a>
+
+> <a target="_blank" href="https://codeberg.org/adamlui/eslint-plugin-chrome-extensions/#installation"><b>Install</b></a> / <a target="_blank" href="https://codeberg.org/adamlui/eslint-plugin-chrome-extensions/#readme">Readme</a> / <a target="_blank" href="https://codeberg.org/adamlui/eslint-plugin-chrome-extensions/#supported-rules">Supported rules</a>
+
+</details>
+
 <!-- eslint-plugin-greasemonkey -->
 
 ### <a target="_blank" href="https://codeberg.org/adamlui/eslint-plugin-greasemonkey/#readme"><img height=15 width=auto src="https://cdn.staticdelivr.com/gl/adamlui/eslint-plugin-greasemonkey/e9128375d4/assets/images/icons/eslint/purple/icon16.png"> eslint-plugin-greasemonkey</a>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-constructors",
-  "version": "1.4.0",
+  "version": "1.4.2",
   "description": "ESLint rules for constructor usage",
   "type": "module",
   "engines": {
@@ -71,10 +71,10 @@
     "@eslint/json": "^2.1.0",
     "@eslint/markdown": "^8.0.3",
     "@stylistic/eslint-plugin": "^5.10.0",
-    "eslint": "^10.11.0",
+    "eslint": "^10.12.0",
     "eslint-plugin-eslint-plugin": "^7.6.2",
     "eslint-plugin-import-x": "^4.17.1",
-    "eslint-plugin-n": "^18.4.0",
+    "eslint-plugin-n": "^18.4.1",
     "eslint-plugin-no-secrets": "^2.3.3",
     "eslint-plugin-no-unsanitized": "^4.1.5",
     "eslint-plugin-promise": "^7.3.0",

package/src/index.js

@@ -4,7 +4,7 @@ import noExplicitReturn from './rules/no-explicit-return.js'
 import requireErrorSubclassName from './rules/require-error-subclass-name.js'
 import requireNew from './rules/require-new.js'
 
-import { initDeprecatedAlias } from './utils.js'
+import { bundleRules, initDeprecatedAlias } from './utils.js'
 
 const plugin = {
     rules: {
@@ -23,17 +23,11 @@ for (const entry of Object.entries(oldRules))
 plugin.configs = {
     recommended: {
         plugins: { constructors: plugin },
-        rules: {
-            'constructors/no-explicit-return': 'error',
-            'constructors/require-error-subclass-name': 'error',
-            'constructors/require-new': 'error'
-        }
+        rules: bundleRules(plugin.rules, rule => rule.meta.docs.recommended, 'constructors')
     },
     problem: {
         plugins: { constructors: plugin },
-        rules: Object.fromEntries(Object.entries(plugin.rules)
-            .filter(([, rule]) => rule.meta.type == 'problem')
-            .map(([id]) => [`constructors/${id}`, 'error']))
+        rules: bundleRules(plugin.rules, rule => rule.meta.type == 'problem', 'constructors')
     }
 }
 

package/src/rules/no-explicit-return.js

@@ -23,7 +23,7 @@ export default {
             recommended: true
         },
         schema: [],
-        messages: { unexpectedReturn: 'Unexpected return statement in constructor' }
+        messages: { unexpectedReturn: 'Unexpected return statement in constructor.' }
     },
 
     create(context) {

package/src/rules/require-error-subclass-name.js

@@ -7,7 +7,7 @@ export default {
             recommended: true
         },
         schema: [],
-        messages: { endWithError: 'Class extending Error must have a name ending in "Error"' }
+        messages: { endWithError: 'Class extending Error must have a name ending in "Error".' }
     },
 
     create(context) {

package/src/rules/require-new.js

@@ -31,7 +31,7 @@ export default {
             additionalProperties: false
         }],
         defaultOptions: [{ mode: 'throwing' }],
-        messages: { requireNew: 'Use `new {{name}}()` instead of `{{name}}()`' }
+        messages: { requireNew: 'Use `new {{name}}()` instead of `{{name}}()`.' }
     },
 
     create(context) {

package/src/utils.js

@@ -1,3 +1,9 @@
+export function bundleRules(rules, predicate, pluginName) {
+    return Object.fromEntries(Object.entries(rules)
+        .filter(([, rule]) => predicate(rule))
+        .map(([ruleId]) => [`${pluginName}/${ruleId}`, 'error']))
+}
+
 export function initDeprecatedAlias(plugin, [oldName, alias]) { /* eslint-disable security/detect-object-injection */
     const routes = typeof alias == 'string' ? { [alias]: null } // 'old-rule': 'new-rule'
         : Array.isArray(alias) ? Object.fromEntries(alias.map(entry =>