eslint-plugin-reliability 3.1.10 → 3.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-reliability",
-  "version": "3.1.10",
+  "version": "3.1.11",
   "description": "ESLint plugin for runtime reliability — enforces error handling, network timeouts, null checks, and safe type narrowing.",
   "type": "commonjs",
   "main": "./src/index.js",
@@ -50,7 +50,7 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "@interlace/eslint-devkit": "^1.8.0"
+    "@interlace/eslint-devkit": "^1.10.0"
   },
   "repository": {
     "type": "git",

package/src/index.js

@@ -1,44 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.configs = exports.plugin = exports.rules = void 0;
-exports.rules = {
-    get 'no-unhandled-promise'() { return require("./rules/error-handling/no-unhandled-promise").noUnhandledPromise; },
-    get 'no-silent-errors'() { return require("./rules/error-handling/no-silent-errors").noSilentErrors; },
-    get 'no-missing-error-context'() { return require("./rules/error-handling/no-missing-error-context").noMissingErrorContext; },
-    get 'error-message'() { return require("./rules/error-handling/error-message").errorMessage; },
-    get 'no-missing-null-checks'() { return require("./rules/reliability/no-missing-null-checks").noMissingNullChecks; },
-    get 'no-unsafe-type-narrowing'() { return require("./rules/reliability/no-unsafe-type-narrowing").noUnsafeTypeNarrowing; },
-    get 'require-network-timeout'() { return require("./rules/reliability/require-network-timeout").requireNetworkTimeout; },
-    get 'no-await-in-loop'() { return require("./rules/reliability/no-await-in-loop").noAwaitInLoop; },
-    get 'no-jsdoc-terminator-in-example'() { return require("./rules/reliability/no-jsdoc-terminator-in-example").noJsdocTerminatorInExample; },
-    get 'error-handling/no-unhandled-promise'() { return require("./rules/error-handling/no-unhandled-promise").noUnhandledPromise; },
-    get 'error-handling/no-silent-errors'() { return require("./rules/error-handling/no-silent-errors").noSilentErrors; },
-    get 'error-handling/no-missing-error-context'() { return require("./rules/error-handling/no-missing-error-context").noMissingErrorContext; },
-    get 'error-handling/error-message'() { return require("./rules/error-handling/error-message").errorMessage; },
-    get 'reliability/no-missing-null-checks'() { return require("./rules/reliability/no-missing-null-checks").noMissingNullChecks; },
-    get 'reliability/no-unsafe-type-narrowing'() { return require("./rules/reliability/no-unsafe-type-narrowing").noUnsafeTypeNarrowing; },
-    get 'reliability/require-network-timeout'() { return require("./rules/reliability/require-network-timeout").requireNetworkTimeout; },
-    get 'reliability/no-await-in-loop'() { return require("./rules/reliability/no-await-in-loop").noAwaitInLoop; },
-    get 'reliability/no-jsdoc-terminator-in-example'() { return require("./rules/reliability/no-jsdoc-terminator-in-example").noJsdocTerminatorInExample; },
-
-};
-exports.plugin = {
-    meta: {
-        name: 'eslint-plugin-reliability',
-        version: '3.1.10',
-    },
-    rules: exports.rules,
-};
-exports.configs = {
-    recommended: {
-        plugins: {
-            reliability: exports.plugin,
-        },
-        rules: {
-            'reliability/no-silent-errors': 'error',
-            'reliability/no-missing-null-checks': 'warn',
-            'reliability/require-network-timeout': 'error',
-        },
-    },
-};
-exports.default = exports.plugin;
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.configs=exports.plugin=exports.rules=void 0;exports.rules={get"no-unhandled-promise"(){return require("./rules/error-handling/no-unhandled-promise").noUnhandledPromise},get"no-silent-errors"(){return require("./rules/error-handling/no-silent-errors").noSilentErrors},get"no-missing-error-context"(){return require("./rules/error-handling/no-missing-error-context").noMissingErrorContext},get"error-message"(){return require("./rules/error-handling/error-message").errorMessage},get"no-missing-null-checks"(){return require("./rules/reliability/no-missing-null-checks").noMissingNullChecks},get"no-unsafe-type-narrowing"(){return require("./rules/reliability/no-unsafe-type-narrowing").noUnsafeTypeNarrowing},get"require-network-timeout"(){return require("./rules/reliability/require-network-timeout").requireNetworkTimeout},get"no-await-in-loop"(){return require("./rules/reliability/no-await-in-loop").noAwaitInLoop},get"no-jsdoc-terminator-in-example"(){return require("./rules/reliability/no-jsdoc-terminator-in-example").noJsdocTerminatorInExample},get"error-handling/no-unhandled-promise"(){return require("./rules/error-handling/no-unhandled-promise").noUnhandledPromise},get"error-handling/no-silent-errors"(){return require("./rules/error-handling/no-silent-errors").noSilentErrors},get"error-handling/no-missing-error-context"(){return require("./rules/error-handling/no-missing-error-context").noMissingErrorContext},get"error-handling/error-message"(){return require("./rules/error-handling/error-message").errorMessage},get"reliability/no-missing-null-checks"(){return require("./rules/reliability/no-missing-null-checks").noMissingNullChecks},get"reliability/no-unsafe-type-narrowing"(){return require("./rules/reliability/no-unsafe-type-narrowing").noUnsafeTypeNarrowing},get"reliability/require-network-timeout"(){return require("./rules/reliability/require-network-timeout").requireNetworkTimeout},get"reliability/no-await-in-loop"(){return require("./rules/reliability/no-await-in-loop").noAwaitInLoop},get"reliability/no-jsdoc-terminator-in-example"(){return require("./rules/reliability/no-jsdoc-terminator-in-example").noJsdocTerminatorInExample}};exports.plugin={meta:{name:"eslint-plugin-reliability",version:"3.1.11"},rules:exports.rules};exports.configs={recommended:{plugins:{reliability:exports.plugin},rules:{"reliability/no-silent-errors":"error","reliability/no-missing-null-checks":"warn","reliability/require-network-timeout":"error"}}};exports.default=exports.plugin;

package/src/lib/eslint-plugin-reliability.js

@@ -1,6 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.eslintPluginReliability = eslintPluginReliability;
-function eslintPluginReliability() {
-    return 'eslint-plugin-reliability';
-}
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.eslintPluginReliability=eslintPluginReliability;function eslintPluginReliability(){return"eslint-plugin-reliability"}

package/src/oxlint.js

@@ -1,3 +1 @@
-"use strict";
-const index_1 = require("./index");
-module.exports = index_1.plugin;
+"use strict";const index_1=require("./index");module.exports=index_1.plugin;

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

@@ -1,131 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.errorMessage = void 0;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-exports.errorMessage = (0, eslint_devkit_1.createRule)({
-    name: 'error-message',
-    meta: {
-        type: 'problem',
-        docs: {
-            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-reliability/docs/rules/error-message.md',
-            description: 'Enforce providing a message when creating built-in Error objects for better debugging',
-        },
-        hasSuggestions: true,
-        messages: {
-            missingErrorMessage: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.INFO,
-                issueName: 'Missing Error Message',
-                description: 'Error constructor called without message parameter',
-                severity: 'HIGH',
-                fix: 'Add descriptive error message: new Error("description")',
-                documentationLink: 'https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/error-message.md',
-            }),
-            addErrorMessage: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.WARNING,
-                issueName: 'Add Error Message',
-                description: 'Add descriptive error message to constructor',
-                severity: 'HIGH',
-                fix: 'Add descriptive error message: new Error("description")',
-                documentationLink: 'https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/error-message.md',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    allowEmptyCatch: {
-                        type: 'boolean',
-                        default: false,
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [{ allowEmptyCatch: false }],
-    create(context) {
-        const [options] = context.options;
-        const { allowEmptyCatch = false } = options || {};
-        const errorConstructors = new Set([
-            'Error',
-            'TypeError',
-            'ReferenceError',
-            'SyntaxError',
-            'RangeError',
-            'EvalError',
-            'URIError',
-        ]);
-        function isErrorConstructor(name) {
-            return errorConstructors.has(name);
-        }
-        function hasMessageArgument(node) {
-            if (!node.arguments || node.arguments.length === 0) {
-                return false;
-            }
-            const firstArg = node.arguments[0];
-            if (firstArg.type === 'Literal') {
-                return (typeof firstArg.value === 'string' && firstArg.value.trim().length > 0);
-            }
-            return true;
-        }
-        function isInCatchClause(node) {
-            let current = node;
-            while (current) {
-                if (current.type === 'CatchClause') {
-                    return true;
-                }
-                current = current.parent;
-            }
-            return false;
-        }
-        function checkErrorCreation(node) {
-            let constructorName = null;
-            if (node.type === 'NewExpression') {
-                if (node.callee.type === 'Identifier') {
-                    constructorName = node.callee.name;
-                }
-            }
-            else if (node.type === 'CallExpression') {
-                if (node.callee.type === 'Identifier') {
-                    constructorName = node.callee.name;
-                }
-            }
-            if (!constructorName || !isErrorConstructor(constructorName)) {
-                return;
-            }
-            if (allowEmptyCatch && isInCatchClause(node)) {
-                return;
-            }
-            if (!hasMessageArgument(node)) {
-                context.report({
-                    node,
-                    messageId: 'missingErrorMessage',
-                    data: {
-                        constructor: constructorName,
-                    },
-                    suggest: [
-                        {
-                            messageId: 'addErrorMessage',
-                            data: { constructor: constructorName },
-                            fix(fixer) {
-                                if (node.arguments.length === 0) {
-                                    const parenStart = node.callee.range[1];
-                                    const parenEnd = node.range[1];
-                                    return fixer.replaceTextRange([parenStart, parenEnd], '("Error message")');
-                                }
-                                else {
-                                    return fixer.replaceText(node.arguments[0], '"Error message"');
-                                }
-                            },
-                        },
-                    ],
-                });
-            }
-        }
-        return {
-            NewExpression: checkErrorCreation,
-            CallExpression: checkErrorCreation,
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.errorMessage=void 0;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");exports.errorMessage=(0,eslint_devkit_1.createRule)({name:"error-message",meta:{type:"problem",docs:{url:"https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-reliability/docs/rules/error-message.md",description:"Enforce providing a message when creating built-in Error objects for better debugging"},hasSuggestions:true,messages:{missingErrorMessage:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.INFO,issueName:"Missing Error Message",description:"Error constructor called without message parameter",severity:"HIGH",fix:'Add descriptive error message: new Error("description")',documentationLink:"https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/error-message.md"}),addErrorMessage:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.WARNING,issueName:"Add Error Message",description:"Add descriptive error message to constructor",severity:"HIGH",fix:'Add descriptive error message: new Error("description")',documentationLink:"https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/error-message.md"})},schema:[{type:"object",properties:{allowEmptyCatch:{type:"boolean",default:false}},additionalProperties:false}]},defaultOptions:[{allowEmptyCatch:false}],create(context){const[options]=context.options;const{allowEmptyCatch=false}=options||{};const errorConstructors=new Set(["Error","TypeError","ReferenceError","SyntaxError","RangeError","EvalError","URIError"]);function isErrorConstructor(name){return errorConstructors.has(name)}function hasMessageArgument(node){if(!node.arguments||node.arguments.length===0){return false}const firstArg=node.arguments[0];if(firstArg.type==="Literal"){return typeof firstArg.value==="string"&&firstArg.value.trim().length>0}return true}function isInCatchClause(node){let current=node;while(current){if(current.type==="CatchClause"){return true}current=current.parent}return false}function checkErrorCreation(node){let constructorName=null;if(node.type==="NewExpression"){if(node.callee.type==="Identifier"){constructorName=node.callee.name}}else if(node.type==="CallExpression"){if(node.callee.type==="Identifier"){constructorName=node.callee.name}}if(!constructorName||!isErrorConstructor(constructorName)){return}if(allowEmptyCatch&&isInCatchClause(node)){return}if(!hasMessageArgument(node)){context.report({node,messageId:"missingErrorMessage",data:{constructor:constructorName},suggest:[{messageId:"addErrorMessage",data:{constructor:constructorName},fix(fixer){if(node.arguments.length===0){const parenStart=node.callee.range[1];const parenEnd=node.range[1];return fixer.replaceTextRange([parenStart,parenEnd],'("Error message")')}else{return fixer.replaceText(node.arguments[0],'"Error message"')}}}]})}}return{NewExpression:checkErrorCreation,CallExpression:checkErrorCreation}}});

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

@@ -1,180 +1 @@
-"use strict";
-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");
-function hasErrorMessage(node) {
-    if (!node.argument) {
-        return false;
-    }
-    if (node.argument.type === 'Identifier' &&
-        node.argument.name !== 'undefined' &&
-        node.argument.name !== 'NaN' &&
-        node.argument.name !== 'Infinity') {
-        return true;
-    }
-    if (node.argument.type === 'NewExpression' &&
-        node.argument.callee.type === 'Identifier' &&
-        (node.argument.callee.name === 'Error' ||
-            node.argument.callee.name.endsWith('Error'))) {
-        if (node.argument.arguments.length > 0) {
-            const firstArg = node.argument.arguments[0];
-            if (firstArg.type === 'Literal' && typeof firstArg.value === 'string') {
-                return firstArg.value.length > 0;
-            }
-            if (firstArg.type === 'TemplateLiteral') {
-                return true;
-            }
-            if (node.argument.callee.name !== 'Error' &&
-                node.argument.callee.name.endsWith('Error')) {
-                return true;
-            }
-        }
-    }
-    if (node.argument.type === 'Literal' &&
-        typeof node.argument.value === 'string') {
-        return node.argument.value.length > 0;
-    }
-    if (node.argument.type === 'TemplateLiteral') {
-        return true;
-    }
-    return false;
-}
-function hasErrorStack(node) {
-    if (!node.argument) {
-        return false;
-    }
-    if (node.argument.type === 'Identifier' &&
-        node.argument.name !== 'undefined' &&
-        node.argument.name !== 'NaN' &&
-        node.argument.name !== 'Infinity') {
-        return true;
-    }
-    if (node.argument.type === 'NewExpression' &&
-        node.argument.callee.type === 'Identifier') {
-        const calleeName = node.argument.callee.name;
-        if (calleeName === 'Error' || calleeName.endsWith('Error')) {
-            return true;
-        }
-    }
-    return false;
-}
-exports.noMissingErrorContext = (0, eslint_devkit_2.createRule)({
-    name: 'no-missing-error-context',
-    meta: {
-        type: 'suggestion',
-        docs: {
-            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-reliability/docs/rules/no-missing-error-context.md',
-            description: 'Detects thrown errors without context',
-        },
-        hasSuggestions: true,
-        messages: {
-            missingErrorContext: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.WARNING,
-                issueName: 'Missing error context',
-                description: 'Thrown error missing {{missing}}',
-                severity: 'MEDIUM',
-                fix: 'Add error message and use Error class for stack trace',
-                documentationLink: 'https://rules.sonarsource.com/javascript/RSPEC-1128/',
-            }),
-            addErrorMessage: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Add Error Message',
-                description: 'Add descriptive error message',
-                severity: 'LOW',
-                fix: 'throw new Error("descriptive message")',
-                documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error',
-            }),
-            addErrorStack: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Use Error Class',
-                description: 'Use Error class for stack trace',
-                severity: 'LOW',
-                fix: 'throw new Error(message) instead of throw message',
-                documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error',
-            }),
-            useErrorClass: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Use Specific Error',
-                description: 'Use specific error class',
-                severity: 'LOW',
-                fix: 'throw new TypeError("message") or throw new RangeError("message")',
-                documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/TypeError',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    requireMessage: {
-                        type: 'boolean',
-                        default: true,
-                        description: 'Require error message',
-                    },
-                    requireStackTrace: {
-                        type: 'boolean',
-                        default: false,
-                        description: 'Require stack trace',
-                    },
-                    ignoreInTests: {
-                        type: 'boolean',
-                        default: true,
-                        description: 'Ignore in test files',
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [
-        {
-            requireMessage: true,
-            requireStackTrace: false,
-            ignoreInTests: true,
-        },
-    ],
-    create(context, [options = {}]) {
-        const { requireMessage = true, requireStackTrace = false, ignoreInTests = true, } = options || {};
-        const filename = context.filename;
-        const isTestFile = ignoreInTests && /\.(test|spec)\.(ts|tsx|js|jsx)$/.test(filename);
-        if (isTestFile) {
-            return {};
-        }
-        function checkThrowStatement(node) {
-            const missing = [];
-            if (requireMessage && !hasErrorMessage(node)) {
-                missing.push('message');
-            }
-            if (requireStackTrace && !hasErrorStack(node)) {
-                missing.push('stack trace');
-            }
-            if (missing.length === 0) {
-                return;
-            }
-            context.report({
-                node,
-                messageId: 'missingErrorContext',
-                data: {
-                    missing: missing.join(' and '),
-                },
-                suggest: [
-                    {
-                        messageId: 'addErrorMessage',
-                        fix: () => null,
-                    },
-                    {
-                        messageId: 'addErrorStack',
-                        fix: () => null,
-                    },
-                    {
-                        messageId: 'useErrorClass',
-                        fix: () => null,
-                    },
-                ],
-            });
-        }
-        return {
-            ThrowStatement: checkThrowStatement,
-        };
-    },
-});
+"use strict";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");function hasErrorMessage(node){if(!node.argument){return false}if(node.argument.type==="Identifier"&&node.argument.name!=="undefined"&&node.argument.name!=="NaN"&&node.argument.name!=="Infinity"){return true}if(node.argument.type==="NewExpression"&&node.argument.callee.type==="Identifier"&&(node.argument.callee.name==="Error"||node.argument.callee.name.endsWith("Error"))){if(node.argument.arguments.length>0){const firstArg=node.argument.arguments[0];if(firstArg.type==="Literal"&&typeof firstArg.value==="string"){return firstArg.value.length>0}if(firstArg.type==="TemplateLiteral"){return true}if(node.argument.callee.name!=="Error"&&node.argument.callee.name.endsWith("Error")){return true}}}if(node.argument.type==="Literal"&&typeof node.argument.value==="string"){return node.argument.value.length>0}if(node.argument.type==="TemplateLiteral"){return true}return false}function hasErrorStack(node){if(!node.argument){return false}if(node.argument.type==="Identifier"&&node.argument.name!=="undefined"&&node.argument.name!=="NaN"&&node.argument.name!=="Infinity"){return true}if(node.argument.type==="NewExpression"&&node.argument.callee.type==="Identifier"){const calleeName=node.argument.callee.name;if(calleeName==="Error"||calleeName.endsWith("Error")){return true}}return false}exports.noMissingErrorContext=(0,eslint_devkit_2.createRule)({name:"no-missing-error-context",meta:{type:"suggestion",docs:{url:"https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-reliability/docs/rules/no-missing-error-context.md",description:"Detects thrown errors without context"},hasSuggestions:true,messages:{missingErrorContext:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.WARNING,issueName:"Missing error context",description:"Thrown error missing {{missing}}",severity:"MEDIUM",fix:"Add error message and use Error class for stack trace",documentationLink:"https://rules.sonarsource.com/javascript/RSPEC-1128/"}),addErrorMessage:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Add Error Message",description:"Add descriptive error message",severity:"LOW",fix:'throw new Error("descriptive message")',documentationLink:"https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error"}),addErrorStack:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Use Error Class",description:"Use Error class for stack trace",severity:"LOW",fix:"throw new Error(message) instead of throw message",documentationLink:"https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error"}),useErrorClass:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Use Specific Error",description:"Use specific error class",severity:"LOW",fix:'throw new TypeError("message") or throw new RangeError("message")',documentationLink:"https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/TypeError"})},schema:[{type:"object",properties:{requireMessage:{type:"boolean",default:true,description:"Require error message"},requireStackTrace:{type:"boolean",default:false,description:"Require stack trace"},ignoreInTests:{type:"boolean",default:true,description:"Ignore in test files"}},additionalProperties:false}]},defaultOptions:[{requireMessage:true,requireStackTrace:false,ignoreInTests:true}],create(context,[options={}]){const{requireMessage=true,requireStackTrace=false,ignoreInTests=true}=options||{};const filename=context.filename;const isTestFile=ignoreInTests&&/\.(test|spec)\.(ts|tsx|js|jsx)$/.test(filename);if(isTestFile){return{}}function checkThrowStatement(node){const missing=[];if(requireMessage&&!hasErrorMessage(node)){missing.push("message")}if(requireStackTrace&&!hasErrorStack(node)){missing.push("stack trace")}if(missing.length===0){return}context.report({node,messageId:"missingErrorContext",data:{missing:missing.join(" and ")},suggest:[{messageId:"addErrorMessage",fix:()=>null},{messageId:"addErrorStack",fix:()=>null},{messageId:"useErrorClass",fix:()=>null}]})}return{ThrowStatement:checkThrowStatement}}});

package/src/rules/error-handling/no-silent-errors.js

@@ -1,157 +1 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.noSilentErrors = void 0;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-function isEmptyCatchBlock(catchClause) {
-    const body = catchClause.body;
-    if (!body || body.type !== 'BlockStatement') {
-        return true;
-    }
-    const statements = body.body;
-    if (statements.length === 0) {
-        return true;
-    }
-    const nonEmptyStatements = statements.filter((stmt) => stmt.type !== 'EmptyStatement');
-    return nonEmptyStatements.length === 0;
-}
-function hasExplanatoryComment(catchClause, sourceCode) {
-    const comments = sourceCode.getAllComments();
-    const catchStart = catchClause.loc?.start;
-    if (!catchStart || !comments.length) {
-        return false;
-    }
-    const explanatoryPatterns = [
-        /intentional/i,
-        /expected/i,
-        /ignore/i,
-        /silent/i,
-        /noop/i,
-        /no-op/i,
-        /by design/i,
-        /known issue/i,
-        /legacy/i,
-        /third.?party/i,
-        /framework/i,
-        /library/i,
-        /not implemented/i,
-        /todo/i,
-        /fixme/i,
-    ];
-    for (const comment of comments) {
-        if (comment.loc && catchStart.line - comment.loc.end.line <= 2) {
-            const commentText = comment.value.toLowerCase();
-            if (explanatoryPatterns.some((pattern) => pattern.test(commentText))) {
-                return true;
-            }
-        }
-    }
-    return false;
-}
-exports.noSilentErrors = (0, eslint_devkit_2.createRule)({
-    name: 'no-silent-errors',
-    meta: {
-        type: 'problem',
-        docs: {
-            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-reliability/docs/rules/no-silent-errors.md',
-            description: 'Detects empty catch blocks',
-        },
-        hasSuggestions: true,
-        messages: {
-            silentError: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.WARNING,
-                issueName: 'Silent error',
-                description: 'Empty catch block detected - errors are silently ignored',
-                severity: 'MEDIUM',
-                fix: 'Add error logging or handling in catch block',
-                documentationLink: 'https://rules.sonarsource.com/javascript/RSPEC-1186/',
-            }),
-            addErrorLogging: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Add Error Logging',
-                description: 'Add error logging to catch block',
-                severity: 'LOW',
-                fix: 'catch (error) { console.error(error); }',
-                documentationLink: 'https://rules.sonarsource.com/javascript/RSPEC-1186/',
-            }),
-            addErrorHandling: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Add Error Handling',
-                description: 'Add error handling to catch block',
-                severity: 'LOW',
-                fix: 'catch (error) { handleError(error); }',
-                documentationLink: 'https://rules.sonarsource.com/javascript/RSPEC-1186/',
-            }),
-            rethrowError: (0, eslint_devkit_1.formatLLMMessage)({
-                icon: eslint_devkit_1.MessageIcons.INFO,
-                issueName: 'Rethrow Error',
-                description: 'Rethrow error if needed',
-                severity: 'LOW',
-                fix: 'catch (error) { throw new CustomError(error); }',
-                documentationLink: 'https://rules.sonarsource.com/javascript/RSPEC-1186/',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    allowWithComment: {
-                        type: 'boolean',
-                        default: false,
-                        description: 'Allow empty catch if comment explains why',
-                    },
-                    ignoreInTests: {
-                        type: 'boolean',
-                        default: true,
-                        description: 'Ignore in test files',
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [
-        {
-            allowWithComment: false,
-            ignoreInTests: true,
-        },
-    ],
-    create(context, [options = {}]) {
-        const { allowWithComment = false, ignoreInTests = true } = options || {};
-        const filename = context.filename;
-        const isTestFile = ignoreInTests && /\.(test|spec)\.(ts|tsx|js|jsx)$/.test(filename);
-        if (isTestFile) {
-            return {};
-        }
-        const sourceCode = context.sourceCode;
-        function checkCatchClause(node) {
-            if (!isEmptyCatchBlock(node)) {
-                return;
-            }
-            if (allowWithComment && hasExplanatoryComment(node, sourceCode)) {
-                return;
-            }
-            context.report({
-                node,
-                messageId: 'silentError',
-                suggest: [
-                    {
-                        messageId: 'addErrorLogging',
-                        fix: () => null,
-                    },
-                    {
-                        messageId: 'addErrorHandling',
-                        fix: () => null,
-                    },
-                    {
-                        messageId: 'rethrowError',
-                        fix: () => null,
-                    },
-                ],
-            });
-        }
-        return {
-            CatchClause: checkCatchClause,
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.noSilentErrors=void 0;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");function isEmptyCatchBlock(catchClause){const body=catchClause.body;if(!body||body.type!=="BlockStatement"){return true}const statements=body.body;if(statements.length===0){return true}const nonEmptyStatements=statements.filter(stmt=>stmt.type!=="EmptyStatement");return nonEmptyStatements.length===0}function hasExplanatoryComment(catchClause,sourceCode){const comments=sourceCode.getAllComments();const catchStart=catchClause.loc?.start;if(!catchStart||!comments.length){return false}const explanatoryPatterns=[/intentional/i,/expected/i,/ignore/i,/silent/i,/noop/i,/no-op/i,/by design/i,/known issue/i,/legacy/i,/third.?party/i,/framework/i,/library/i,/not implemented/i,/todo/i,/fixme/i];for(const comment of comments){if(comment.loc&&catchStart.line-comment.loc.end.line<=2){const commentText=comment.value.toLowerCase();if(explanatoryPatterns.some(pattern=>pattern.test(commentText))){return true}}}return false}exports.noSilentErrors=(0,eslint_devkit_2.createRule)({name:"no-silent-errors",meta:{type:"problem",docs:{url:"https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-reliability/docs/rules/no-silent-errors.md",description:"Detects empty catch blocks"},hasSuggestions:true,messages:{silentError:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.WARNING,issueName:"Silent error",description:"Empty catch block detected - errors are silently ignored",severity:"MEDIUM",fix:"Add error logging or handling in catch block",documentationLink:"https://rules.sonarsource.com/javascript/RSPEC-1186/"}),addErrorLogging:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Add Error Logging",description:"Add error logging to catch block",severity:"LOW",fix:"catch (error) { console.error(error); }",documentationLink:"https://rules.sonarsource.com/javascript/RSPEC-1186/"}),addErrorHandling:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Add Error Handling",description:"Add error handling to catch block",severity:"LOW",fix:"catch (error) { handleError(error); }",documentationLink:"https://rules.sonarsource.com/javascript/RSPEC-1186/"}),rethrowError:(0,eslint_devkit_1.formatLLMMessage)({icon:eslint_devkit_1.MessageIcons.INFO,issueName:"Rethrow Error",description:"Rethrow error if needed",severity:"LOW",fix:"catch (error) { throw new CustomError(error); }",documentationLink:"https://rules.sonarsource.com/javascript/RSPEC-1186/"})},schema:[{type:"object",properties:{allowWithComment:{type:"boolean",default:false,description:"Allow empty catch if comment explains why"},ignoreInTests:{type:"boolean",default:true,description:"Ignore in test files"}},additionalProperties:false}]},defaultOptions:[{allowWithComment:false,ignoreInTests:true}],create(context,[options={}]){const{allowWithComment=false,ignoreInTests=true}=options||{};const filename=context.filename;const isTestFile=ignoreInTests&&/\.(test|spec)\.(ts|tsx|js|jsx)$/.test(filename);if(isTestFile){return{}}const sourceCode=context.sourceCode;function checkCatchClause(node){if(!isEmptyCatchBlock(node)){return}if(allowWithComment&&hasExplanatoryComment(node,sourceCode)){return}context.report({node,messageId:"silentError",suggest:[{messageId:"addErrorLogging",fix:()=>null},{messageId:"addErrorHandling",fix:()=>null},{messageId:"rethrowError",fix:()=>null}]})}return{CatchClause:checkCatchClause}}});