eslint-plugin-reliability 3.1.6 → 3.1.8

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-reliability",
-  "version": "3.1.6",
-  "description": "ESLint rules for runtime stability, fault tolerance, and type safety.",
+  "version": "3.1.8",
+  "description": "ESLint plugin for runtime reliability — enforces error handling, network timeouts, null checks, and safe type narrowing.",
   "type": "commonjs",
   "main": "./src/index.js",
   "types": "./src/index.d.ts",
@@ -22,51 +22,51 @@
   },
   "files": [
     "src/",
-    "dist/",
     "README.md",
-    "LICENSE",
-    "CHANGELOG.md"
+    "LICENSE"
   ],
   "keywords": [
     "eslint",
+    "eslintplugin",
     "eslint-plugin",
+    "static-analysis",
+    "linting",
+    "code-quality",
     "interlace-quality",
     "reliability",
     "error-handling",
+    "exception-handling",
+    "fault-tolerance",
+    "resilience",
     "type-safety",
-    "llm-optimized"
+    "null-safety",
+    "timeout",
+    "async",
+    "llm-optimized",
+    "ai-assistant",
+    "typescript"
   ],
   "engines": {
     "node": ">=18.0.0"
   },
   "dependencies": {
     "tslib": "^2.3.0",
-    "@interlace/eslint-devkit": "*"
-  },
-  "devDependencies": {
-    "@typescript-eslint/parser": "^8.46.2",
-    "@typescript-eslint/rule-tester": "^8.46.2"
+    "@interlace/eslint-devkit": "^1.6.1"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/ofri-peretz/eslint",
+    "url": "git+https://github.com/ofri-peretz/eslint.git",
     "directory": "packages/eslint-plugin-reliability"
   },
   "homepage": "https://eslint.interlace.tools/docs/quality/plugin-reliability?utm_source=npm&utm_medium=referral&utm_campaign=eslint-plugin-reliability",
   "bugs": {
     "url": "https://github.com/ofri-peretz/eslint/issues"
   },
-  "scripts": {
-    "build": "tsx ../../scripts/build-package.ts",
-    "test": "vitest run",
-    "test:coverage": "vitest run --coverage",
-    "typecheck": "tsc -p tsconfig.lib.json --noEmit"
-  },
   "peerDependencies": {
     "eslint": "^8.0.0 || ^9.0.0 || ^10.0.0"
   },
   "funding": {
     "type": "github",
-    "url": "https://github.com/ofri-peretz/eslint"
+    "url": "git+https://github.com/ofri-peretz/eslint.git"
   }
 }

package/src/index.d.ts

@@ -1,198 +1,17 @@
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
-import type { TSESLint } from '@interlace/eslint-devkit';
-export declare const rules: {
-    'no-unhandled-promise': TSESLint.RuleModule<"unhandledPromise" | "addCatch" | "useTryCatch" | "useAwait", [(import("./rules/error-handling/no-unhandled-promise").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-silent-errors': TSESLint.RuleModule<"silentError" | "addErrorLogging" | "addErrorHandling" | "rethrowError", [(import("./rules/error-handling/no-silent-errors").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-missing-error-context': TSESLint.RuleModule<"missingErrorContext" | "addErrorMessage" | "addErrorStack" | "useErrorClass", [(import("./rules/error-handling/no-missing-error-context").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'error-message': TSESLint.RuleModule<"addErrorMessage" | "missingErrorMessage", [(import("./rules/error-handling/error-message").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-missing-null-checks': TSESLint.RuleModule<"missingNullCheck" | "useOptionalChaining" | "useNullishCoalescing" | "addExplicitCheck", [(import("./rules/reliability/no-missing-null-checks").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-unsafe-type-narrowing': TSESLint.RuleModule<"unsafeTypeNarrowing" | "useTypeGuard" | "useProperNarrowing" | "validateBeforeAssert", [(import("./rules/reliability/no-unsafe-type-narrowing").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'require-network-timeout': TSESLint.RuleModule<"violationDetected", [(import("./rules/reliability/require-network-timeout").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-await-in-loop': TSESLint.RuleModule<"awaitInLoop" | "suggestPromiseAll" | "suggestConcurrent" | "considerSequential" | "asyncLoopPattern", [(import("./rules/reliability/no-await-in-loop").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-jsdoc-terminator-in-example': TSESLint.RuleModule<"jsdocTerminatorInExample" | "wrapInQuotes", [(import("./rules/reliability/no-jsdoc-terminator-in-example").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'error-handling/no-unhandled-promise': TSESLint.RuleModule<"unhandledPromise" | "addCatch" | "useTryCatch" | "useAwait", [(import("./rules/error-handling/no-unhandled-promise").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'error-handling/no-silent-errors': TSESLint.RuleModule<"silentError" | "addErrorLogging" | "addErrorHandling" | "rethrowError", [(import("./rules/error-handling/no-silent-errors").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'error-handling/no-missing-error-context': TSESLint.RuleModule<"missingErrorContext" | "addErrorMessage" | "addErrorStack" | "useErrorClass", [(import("./rules/error-handling/no-missing-error-context").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'error-handling/error-message': TSESLint.RuleModule<"addErrorMessage" | "missingErrorMessage", [(import("./rules/error-handling/error-message").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'reliability/no-missing-null-checks': TSESLint.RuleModule<"missingNullCheck" | "useOptionalChaining" | "useNullishCoalescing" | "addExplicitCheck", [(import("./rules/reliability/no-missing-null-checks").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'reliability/no-unsafe-type-narrowing': TSESLint.RuleModule<"unsafeTypeNarrowing" | "useTypeGuard" | "useProperNarrowing" | "validateBeforeAssert", [(import("./rules/reliability/no-unsafe-type-narrowing").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'reliability/require-network-timeout': TSESLint.RuleModule<"violationDetected", [(import("./rules/reliability/require-network-timeout").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'reliability/no-await-in-loop': TSESLint.RuleModule<"awaitInLoop" | "suggestPromiseAll" | "suggestConcurrent" | "considerSequential" | "asyncLoopPattern", [(import("./rules/reliability/no-await-in-loop").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'reliability/no-jsdoc-terminator-in-example': TSESLint.RuleModule<"jsdocTerminatorInExample" | "wrapInQuotes", [(import("./rules/reliability/no-jsdoc-terminator-in-example").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-};
+// Minimal declaration written by scripts/build-package.ts.
+//
+// tsc's generated entry declaration inlined every inferred rule-option type
+// (up to 166 kB per plugin) for an API no consumer calls directly. This types
+// what a flat config actually touches. Per-rule option types remain available
+// from the "./types" subpath export where a plugin provides one.
+import type { Linter, Rule } from 'eslint';
+
+export declare const rules: Record<string, Rule.RuleModule>;
+export declare const configs: Record<string, Linter.Config>;
 export declare const plugin: {
-    meta: {
-        name: string;
-        version: string;
-    };
-    rules: {
-        'no-unhandled-promise': TSESLint.RuleModule<"unhandledPromise" | "addCatch" | "useTryCatch" | "useAwait", [(import("./rules/error-handling/no-unhandled-promise").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-silent-errors': TSESLint.RuleModule<"silentError" | "addErrorLogging" | "addErrorHandling" | "rethrowError", [(import("./rules/error-handling/no-silent-errors").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-missing-error-context': TSESLint.RuleModule<"missingErrorContext" | "addErrorMessage" | "addErrorStack" | "useErrorClass", [(import("./rules/error-handling/no-missing-error-context").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'error-message': TSESLint.RuleModule<"addErrorMessage" | "missingErrorMessage", [(import("./rules/error-handling/error-message").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-missing-null-checks': TSESLint.RuleModule<"missingNullCheck" | "useOptionalChaining" | "useNullishCoalescing" | "addExplicitCheck", [(import("./rules/reliability/no-missing-null-checks").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-unsafe-type-narrowing': TSESLint.RuleModule<"unsafeTypeNarrowing" | "useTypeGuard" | "useProperNarrowing" | "validateBeforeAssert", [(import("./rules/reliability/no-unsafe-type-narrowing").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'require-network-timeout': TSESLint.RuleModule<"violationDetected", [(import("./rules/reliability/require-network-timeout").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-await-in-loop': TSESLint.RuleModule<"awaitInLoop" | "suggestPromiseAll" | "suggestConcurrent" | "considerSequential" | "asyncLoopPattern", [(import("./rules/reliability/no-await-in-loop").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-jsdoc-terminator-in-example': TSESLint.RuleModule<"jsdocTerminatorInExample" | "wrapInQuotes", [(import("./rules/reliability/no-jsdoc-terminator-in-example").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'error-handling/no-unhandled-promise': TSESLint.RuleModule<"unhandledPromise" | "addCatch" | "useTryCatch" | "useAwait", [(import("./rules/error-handling/no-unhandled-promise").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'error-handling/no-silent-errors': TSESLint.RuleModule<"silentError" | "addErrorLogging" | "addErrorHandling" | "rethrowError", [(import("./rules/error-handling/no-silent-errors").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'error-handling/no-missing-error-context': TSESLint.RuleModule<"missingErrorContext" | "addErrorMessage" | "addErrorStack" | "useErrorClass", [(import("./rules/error-handling/no-missing-error-context").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'error-handling/error-message': TSESLint.RuleModule<"addErrorMessage" | "missingErrorMessage", [(import("./rules/error-handling/error-message").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'reliability/no-missing-null-checks': TSESLint.RuleModule<"missingNullCheck" | "useOptionalChaining" | "useNullishCoalescing" | "addExplicitCheck", [(import("./rules/reliability/no-missing-null-checks").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'reliability/no-unsafe-type-narrowing': TSESLint.RuleModule<"unsafeTypeNarrowing" | "useTypeGuard" | "useProperNarrowing" | "validateBeforeAssert", [(import("./rules/reliability/no-unsafe-type-narrowing").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'reliability/require-network-timeout': TSESLint.RuleModule<"violationDetected", [(import("./rules/reliability/require-network-timeout").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'reliability/no-await-in-loop': TSESLint.RuleModule<"awaitInLoop" | "suggestPromiseAll" | "suggestConcurrent" | "considerSequential" | "asyncLoopPattern", [(import("./rules/reliability/no-await-in-loop").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'reliability/no-jsdoc-terminator-in-example': TSESLint.RuleModule<"jsdocTerminatorInExample" | "wrapInQuotes", [(import("./rules/reliability/no-jsdoc-terminator-in-example").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-    };
-};
-export declare const configs: {
-    recommended: {
-        plugins: {
-            reliability: {
-                meta: {
-                    name: string;
-                    version: string;
-                };
-                rules: {
-                    'no-unhandled-promise': TSESLint.RuleModule<"unhandledPromise" | "addCatch" | "useTryCatch" | "useAwait", [(import("./rules/error-handling/no-unhandled-promise").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-silent-errors': TSESLint.RuleModule<"silentError" | "addErrorLogging" | "addErrorHandling" | "rethrowError", [(import("./rules/error-handling/no-silent-errors").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-missing-error-context': TSESLint.RuleModule<"missingErrorContext" | "addErrorMessage" | "addErrorStack" | "useErrorClass", [(import("./rules/error-handling/no-missing-error-context").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'error-message': TSESLint.RuleModule<"addErrorMessage" | "missingErrorMessage", [(import("./rules/error-handling/error-message").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-missing-null-checks': TSESLint.RuleModule<"missingNullCheck" | "useOptionalChaining" | "useNullishCoalescing" | "addExplicitCheck", [(import("./rules/reliability/no-missing-null-checks").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-unsafe-type-narrowing': TSESLint.RuleModule<"unsafeTypeNarrowing" | "useTypeGuard" | "useProperNarrowing" | "validateBeforeAssert", [(import("./rules/reliability/no-unsafe-type-narrowing").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'require-network-timeout': TSESLint.RuleModule<"violationDetected", [(import("./rules/reliability/require-network-timeout").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-await-in-loop': TSESLint.RuleModule<"awaitInLoop" | "suggestPromiseAll" | "suggestConcurrent" | "considerSequential" | "asyncLoopPattern", [(import("./rules/reliability/no-await-in-loop").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-jsdoc-terminator-in-example': TSESLint.RuleModule<"jsdocTerminatorInExample" | "wrapInQuotes", [(import("./rules/reliability/no-jsdoc-terminator-in-example").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'error-handling/no-unhandled-promise': TSESLint.RuleModule<"unhandledPromise" | "addCatch" | "useTryCatch" | "useAwait", [(import("./rules/error-handling/no-unhandled-promise").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'error-handling/no-silent-errors': TSESLint.RuleModule<"silentError" | "addErrorLogging" | "addErrorHandling" | "rethrowError", [(import("./rules/error-handling/no-silent-errors").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'error-handling/no-missing-error-context': TSESLint.RuleModule<"missingErrorContext" | "addErrorMessage" | "addErrorStack" | "useErrorClass", [(import("./rules/error-handling/no-missing-error-context").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'error-handling/error-message': TSESLint.RuleModule<"addErrorMessage" | "missingErrorMessage", [(import("./rules/error-handling/error-message").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'reliability/no-missing-null-checks': TSESLint.RuleModule<"missingNullCheck" | "useOptionalChaining" | "useNullishCoalescing" | "addExplicitCheck", [(import("./rules/reliability/no-missing-null-checks").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'reliability/no-unsafe-type-narrowing': TSESLint.RuleModule<"unsafeTypeNarrowing" | "useTypeGuard" | "useProperNarrowing" | "validateBeforeAssert", [(import("./rules/reliability/no-unsafe-type-narrowing").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'reliability/require-network-timeout': TSESLint.RuleModule<"violationDetected", [(import("./rules/reliability/require-network-timeout").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'reliability/no-await-in-loop': TSESLint.RuleModule<"awaitInLoop" | "suggestPromiseAll" | "suggestConcurrent" | "considerSequential" | "asyncLoopPattern", [(import("./rules/reliability/no-await-in-loop").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'reliability/no-jsdoc-terminator-in-example': TSESLint.RuleModule<"jsdocTerminatorInExample" | "wrapInQuotes", [(import("./rules/reliability/no-jsdoc-terminator-in-example").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                };
-            };
-        };
-        rules: {
-            'reliability/no-silent-errors': "error";
-            'reliability/no-missing-null-checks': "warn";
-            'reliability/require-network-timeout': "error";
-        };
-    };
+  meta: { name: string; version: string };
+  rules: Record<string, Rule.RuleModule>;
 };
-export default plugin;
+
+declare const _default: typeof plugin;
+export default _default;

package/src/index.js

@@ -1,17 +1,10 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.configs = exports.plugin = exports.rules = void 0;
-// Error handling rules
 const no_unhandled_promise_1 = require("./rules/error-handling/no-unhandled-promise");
 const no_silent_errors_1 = require("./rules/error-handling/no-silent-errors");
 const no_missing_error_context_1 = require("./rules/error-handling/no-missing-error-context");
 const error_message_1 = require("./rules/error-handling/error-message");
-// Reliability rules
 const no_missing_null_checks_1 = require("./rules/reliability/no-missing-null-checks");
 const no_unsafe_type_narrowing_1 = require("./rules/reliability/no-unsafe-type-narrowing");
 const require_network_timeout_1 = require("./rules/reliability/require-network-timeout");
@@ -27,7 +20,6 @@ exports.rules = {
     'require-network-timeout': require_network_timeout_1.requireNetworkTimeout,
     'no-await-in-loop': no_await_in_loop_1.noAwaitInLoop,
     'no-jsdoc-terminator-in-example': no_jsdoc_terminator_in_example_1.noJsdocTerminatorInExample,
-    // Categorized names
     'error-handling/no-unhandled-promise': no_unhandled_promise_1.noUnhandledPromise,
     'error-handling/no-silent-errors': no_silent_errors_1.noSilentErrors,
     'error-handling/no-missing-error-context': no_missing_error_context_1.noMissingErrorContext,
@@ -41,7 +33,7 @@ exports.rules = {
 exports.plugin = {
     meta: {
         name: 'eslint-plugin-reliability',
-        version: '3.1.6',
+        version: '3.1.8',
     },
     rules: exports.rules,
 };

package/src/oxlint.d.ts

@@ -1,20 +1,17 @@
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
-/**
- * oxlint sub-export.
- *
- * Consumers wire oxlint to this entry via:
- *   { "jsPlugins": ["eslint-plugin-reliability/oxlint"] }
- *
- * oxlint's JS plugin loader does `require('eslint-plugin-reliability/oxlint')` and reads
- * `.rules` off the result. The compiled output of `export = plugin` is
- * `module.exports = plugin`, which gives oxlint the plugin object directly
- * (`{ meta, rules }`) — no wrapper, no `.default` indirection.
- *
- * Generated by scripts/generate-oxlint-shims.mjs. Do not edit by hand.
- */
-import { plugin } from './index';
-export = plugin;
+// Minimal declaration written by scripts/build-package.ts.
+//
+// tsc's generated entry declaration inlined every inferred rule-option type
+// (up to 166 kB per plugin) for an API no consumer calls directly. This types
+// what a flat config actually touches. Per-rule option types remain available
+// from the "./types" subpath export where a plugin provides one.
+import type { Linter, Rule } from 'eslint';
+
+export declare const rules: Record<string, Rule.RuleModule>;
+export declare const configs: Record<string, Linter.Config>;
+export declare const plugin: {
+  meta: { name: string; version: string };
+  rules: Record<string, Rule.RuleModule>;
+};
+
+declare const _default: typeof plugin;
+export default _default;

package/src/oxlint.js

@@ -1,21 +1,3 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
-/**
- * oxlint sub-export.
- *
- * Consumers wire oxlint to this entry via:
- *   { "jsPlugins": ["eslint-plugin-reliability/oxlint"] }
- *
- * oxlint's JS plugin loader does `require('eslint-plugin-reliability/oxlint')` and reads
- * `.rules` off the result. The compiled output of `export = plugin` is
- * `module.exports = plugin`, which gives oxlint the plugin object directly
- * (`{ meta, rules }`) — no wrapper, no `.default` indirection.
- *
- * Generated by scripts/generate-oxlint-shims.mjs. Do not edit by hand.
- */
 const index_1 = require("./index");
 module.exports = index_1.plugin;

package/src/rules/error-handling/error-message.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.errorMessage = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -52,7 +47,6 @@ exports.errorMessage = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { allowEmptyCatch = false } = options || {};
-        // Built-in Error constructors that should have messages
         const errorConstructors = new Set([
             'Error',
             'TypeError',
@@ -65,19 +59,14 @@ exports.errorMessage = (0, eslint_devkit_1.createRule)({
         function isErrorConstructor(name) {
             return errorConstructors.has(name);
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function hasMessageArgument(node) {
-            // Check if there are arguments
             if (!node.arguments || node.arguments.length === 0) {
                 return false;
             }
-            // Check if first argument is a non-empty string or expression
             const firstArg = node.arguments[0];
             if (firstArg.type === 'Literal') {
-                // Allow non-empty strings
                 return (typeof firstArg.value === 'string' && firstArg.value.trim().length > 0);
             }
-            // Allow any expression (variable, function call, etc.) as it might be dynamic
             return true;
         }
         function isInCatchClause(node) {
@@ -93,13 +82,11 @@ exports.errorMessage = (0, eslint_devkit_1.createRule)({
         function checkErrorCreation(node) {
             let constructorName = null;
             if (node.type === 'NewExpression') {
-                // new Error(...)
                 if (node.callee.type === 'Identifier') {
                     constructorName = node.callee.name;
                 }
             }
             else if (node.type === 'CallExpression') {
-                // Error(...) - function call style
                 if (node.callee.type === 'Identifier') {
                     constructorName = node.callee.name;
                 }
@@ -107,11 +94,9 @@ exports.errorMessage = (0, eslint_devkit_1.createRule)({
             if (!constructorName || !isErrorConstructor(constructorName)) {
                 return;
             }
-            // Allow empty catch if option is enabled
             if (allowEmptyCatch && isInCatchClause(node)) {
                 return;
             }
-            // Check if message is provided
             if (!hasMessageArgument(node)) {
                 context.report({
                     node,
@@ -125,13 +110,11 @@ exports.errorMessage = (0, eslint_devkit_1.createRule)({
                             data: { constructor: constructorName },
                             fix(fixer) {
                                 if (node.arguments.length === 0) {
-                                    // No arguments: replace the empty parentheses with ("Error message")
                                     const parenStart = node.callee.range[1];
                                     const parenEnd = node.range[1];
                                     return fixer.replaceTextRange([parenStart, parenEnd], '("Error message")');
                                 }
                                 else {
-                                    // Has arguments: replace the first argument with "Error message"
                                     return fixer.replaceText(node.arguments[0], '"Error message"');
                                 }
                             },

package/src/rules/error-handling/no-missing-error-context.js

@@ -1,39 +1,22 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noMissingErrorContext = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
-/**
- * Check if error has a message
- */
 function hasErrorMessage(node) {
     if (!node.argument) {
         return false;
     }
-    // `throw error;` — re-throwing a caught/named identifier. The original
-    // error already carries its own message + stack; demanding context on the
-    // re-throw is an FP. Same for `throw err`, `throw cause`, etc. Excludes
-    // the literal-like `undefined` / `NaN` / `Infinity` identifiers, which
-    // are typed as global constants but carry no diagnostic value.
     if (node.argument.type === 'Identifier' &&
         node.argument.name !== 'undefined' &&
         node.argument.name !== 'NaN' &&
         node.argument.name !== 'Infinity') {
         return true;
     }
-    // Check if it's a new Error() with message (includes TypeError, ReferenceError, etc.)
     if (node.argument.type === 'NewExpression' &&
         node.argument.callee.type === 'Identifier' &&
         (node.argument.callee.name === 'Error' ||
             node.argument.callee.name.endsWith('Error'))) {
-        // Check if first argument is a string (message) OR ANY expression — a
-        // custom error class like `new UserNotFoundError(userId)` builds its
-        // own message internally; the constructor argument IS the context.
         if (node.argument.arguments.length > 0) {
             const firstArg = node.argument.arguments[0];
             if (firstArg.type === 'Literal' && typeof firstArg.value === 'string') {
@@ -42,47 +25,34 @@ function hasErrorMessage(node) {
             if (firstArg.type === 'TemplateLiteral') {
                 return true;
             }
-            // Non-string argument to a custom *Error class — accept as context.
-            // The rule's purpose is "throws should carry information"; passing an
-            // identifier/object to the error constructor carries information.
             if (node.argument.callee.name !== 'Error' &&
                 node.argument.callee.name.endsWith('Error')) {
                 return true;
             }
         }
     }
-    // Check if it's a string literal
     if (node.argument.type === 'Literal' &&
         typeof node.argument.value === 'string') {
         return node.argument.value.length > 0;
     }
-    // Check if it's a template literal
     if (node.argument.type === 'TemplateLiteral') {
         return true;
     }
     return false;
 }
-/**
- * Check if error is an Error instance (has stack trace)
- */
 function hasErrorStack(node) {
     if (!node.argument) {
         return false;
     }
-    // `throw error;` — caught/named identifier already has a stack from the
-    // place where it was first thrown. Re-throws preserve the stack. Excludes
-    // `undefined` / `NaN` / `Infinity` global-constant identifiers.
     if (node.argument.type === 'Identifier' &&
         node.argument.name !== 'undefined' &&
         node.argument.name !== 'NaN' &&
         node.argument.name !== 'Infinity') {
         return true;
     }
-    // Check if it's a new Error() instance
     if (node.argument.type === 'NewExpression' &&
         node.argument.callee.type === 'Identifier') {
         const calleeName = node.argument.callee.name;
-        // Error, TypeError, ReferenceError, etc. all have stack traces
         if (calleeName === 'Error' || calleeName.endsWith('Error')) {
             return true;
         }
@@ -170,9 +140,6 @@ exports.noMissingErrorContext = (0, eslint_devkit_2.createRule)({
         if (isTestFile) {
             return {};
         }
-        /**
-         * Check throw statements
-         */
         function checkThrowStatement(node) {
             const missing = [];
             if (requireMessage && !hasErrorMessage(node)) {
@@ -193,7 +160,7 @@ exports.noMissingErrorContext = (0, eslint_devkit_2.createRule)({
                 suggest: [
                     {
                         messageId: 'addErrorMessage',
-                        fix: () => null, // Cannot auto-fix without context
+                        fix: () => null,
                     },
                     {
                         messageId: 'addErrorStack',