eslint-plugin-reliability 3.1.4 → 3.1.5

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 ## [3.1.3] - 2026-05-03
 
+## 3.1.5
+
+### Patch Changes
+
+- [#252](https://github.com/ofri-peretz/eslint/pull/252) [`d67e395`](https://github.com/ofri-peretz/eslint/commit/d67e3953c2748ad36e6aebe0f24b1d04e518b4d0) Thanks [@ofri-peretz](https://github.com/ofri-peretz)! - Fix Codecov badge showing "unknown" — switch from flag to component URL format
+
 ## 3.1.4
 
 ### Patch Changes

package/README.md

@@ -10,7 +10,7 @@
   <a href="https://www.npmjs.com/package/eslint-plugin-reliability" target="_blank"><img src="https://img.shields.io/npm/v/eslint-plugin-reliability.svg" alt="NPM Version" /></a>
   <a href="https://www.npmjs.com/package/eslint-plugin-reliability" target="_blank"><img src="https://img.shields.io/npm/dm/eslint-plugin-reliability.svg" alt="NPM Downloads" /></a>
   <a href="https://opensource.org/licenses/MIT" target="_blank"><img src="https://img.shields.io/badge/License-MIT-yellow.svg" alt="Package License" /></a>
-  <a href="https://app.codecov.io/gh/ofri-peretz/eslint/components?components%5B0%5D=reliability" target="_blank"><img src="https://codecov.io/gh/ofri-peretz/eslint/graph/badge.svg?component=reliability" alt="Codecov" /></a>
+  <a href="https://app.codecov.io/gh/ofri-peretz/eslint/components?components%5B0%5D=eslint-plugin-reliability" target="_blank"><img src="https://codecov.io/gh/ofri-peretz/eslint/graph/badge.svg?component=eslint-plugin-reliability" alt="Codecov" /></a>
   <a href="https://github.com/ofri-peretz/eslint" target="_blank"><img src="https://img.shields.io/badge/Since-Dec_2025-blue?logo=rocket&logoColor=white" alt="Since Dec 2025" /></a>
 </p>
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-reliability",
-  "version": "3.1.4",
+  "version": "3.1.5",
   "description": "ESLint rules for runtime stability, fault tolerance, and type safety.",
   "type": "commonjs",
   "main": "./src/index.js",

package/src/index.js

@@ -41,7 +41,7 @@ exports.rules = {
 exports.plugin = {
     meta: {
         name: 'eslint-plugin-reliability',
-        version: '3.1.4',
+        version: '3.1.5',
     },
     rules: exports.rules,
 };

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

@@ -13,7 +13,6 @@ const eslint_devkit_2 = require("@interlace/eslint-devkit");
  */
 function isEmptyCatchBlock(catchClause) {
     const body = catchClause.body;
-    /* v8 ignore next 3 -- defensive: catch clause body is always BlockStatement in valid JS */
     if (!body || body.type !== 'BlockStatement') {
         return true;
     }
@@ -33,7 +32,6 @@ function hasExplanatoryComment(catchClause, sourceCode) {
     // Check for comments before the catch clause
     const comments = sourceCode.getAllComments();
     const catchStart = catchClause.loc?.start;
-    /* v8 ignore next 3 -- defensive: catch clause always has location, and no comments = no match */
     if (!catchStart || !comments.length) {
         return false;
     }

package/src/rules/error-handling/no-unhandled-promise.d.ts

@@ -11,7 +11,7 @@
  * @see https://cwe.mitre.org/data/definitions/1024.html
  * @see https://rules.sonarsource.com/javascript/RSPEC-4635/
  */
-import type { TSESLint } from '@interlace/eslint-devkit';
+import type { TSESLint, TSESTree } from '@interlace/eslint-devkit';
 type MessageIds = 'unhandledPromise' | 'addCatch' | 'useTryCatch' | 'useAwait';
 export interface Options {
     /** Ignore promises in test files. Default: true */
@@ -20,6 +20,29 @@ export interface Options {
     ignoreVoidExpressions?: boolean;
 }
 type RuleOptions = [Options?];
+/**
+ * Returns true if the call MIGHT return a Promise (default — we want to
+ * preserve detection for unknown calls). Returns false only when the
+ * callee is a known synchronous built-in (`setTimeout`, `console.log`,
+ * `Math.floor`, etc.) — those structurally never return promises and
+ * firing on them produces FPs. Keeping the default as "could be a
+ * promise" preserves recall on user-defined async functions.
+ *
+ * Exported for direct Layer-2 unit testing: the non-CallExpression early
+ * return is only reachable when called with a non-call node (kept for the
+ * future `checkIdentifier` listener), which the current CallExpression-only
+ * listener never produces.
+ */
+export declare function isLikelyPromiseExpression(node: TSESTree.Node): boolean;
+/**
+ * Check if promise is handled (has .catch, .then, or is in try/catch)
+ *
+ * Exported for direct Layer-2 unit testing: the Identifier branch is only
+ * reachable when called with an Identifier node (kept for the future
+ * `checkIdentifier` listener), which the current CallExpression-only
+ * listener never produces.
+ */
+export declare function isPromiseHandled(node: TSESTree.Node): boolean;
 export declare const noUnhandledPromise: TSESLint.RuleModule<MessageIds, RuleOptions, unknown, TSESLint.RuleListener> & {
     name: string;
 };

package/src/rules/error-handling/no-unhandled-promise.js

@@ -6,6 +6,8 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noUnhandledPromise = void 0;
+exports.isLikelyPromiseExpression = isLikelyPromiseExpression;
+exports.isPromiseHandled = isPromiseHandled;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
 /**
@@ -68,6 +70,11 @@ const SYNC_NAMESPACE_OBJECTS = new Set([
  * `Math.floor`, etc.) — those structurally never return promises and
  * firing on them produces FPs. Keeping the default as "could be a
  * promise" preserves recall on user-defined async functions.
+ *
+ * Exported for direct Layer-2 unit testing: the non-CallExpression early
+ * return is only reachable when called with a non-call node (kept for the
+ * future `checkIdentifier` listener), which the current CallExpression-only
+ * listener never produces.
  */
 function isLikelyPromiseExpression(node) {
     if (node.type !== 'CallExpression')
@@ -159,6 +166,11 @@ function isInsidePromiseCallback(node) {
 }
 /**
  * Check if promise is handled (has .catch, .then, or is in try/catch)
+ *
+ * Exported for direct Layer-2 unit testing: the Identifier branch is only
+ * reachable when called with an Identifier node (kept for the future
+ * `checkIdentifier` listener), which the current CallExpression-only
+ * listener never produces.
  */
 function isPromiseHandled(node) {
     // For identifiers, check if they're used in a promise chain

package/src/rules/reliability/no-jsdoc-terminator-in-example.d.ts

@@ -19,6 +19,16 @@ type MessageIds = 'jsdocTerminatorInExample' | 'wrapInQuotes';
 export interface Options {
 }
 type RuleOptions = [Options?];
+/**
+ * Check whether a comment line is inside an @example block.
+ * We track this by scanning line-by-line for @example / next-tag boundaries.
+ *
+ * Exported for direct Layer-2 unit testing: a real parser terminates the
+ * enclosing block comment at the first star-slash sequence, so a comment
+ * value containing one can never reach this function through a
+ * normally-parsed file.
+ */
+export declare function findTerminatorsInExamples(commentText: string): number[];
 export declare const noJsdocTerminatorInExample: TSESLint.RuleModule<MessageIds, RuleOptions, unknown, TSESLint.RuleListener> & {
     name: string;
 };

package/src/rules/reliability/no-jsdoc-terminator-in-example.js

@@ -6,10 +6,16 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noJsdocTerminatorInExample = void 0;
+exports.findTerminatorsInExamples = findTerminatorsInExamples;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 /**
  * Check whether a comment line is inside an @example block.
  * We track this by scanning line-by-line for @example / next-tag boundaries.
+ *
+ * Exported for direct Layer-2 unit testing: a real parser terminates the
+ * enclosing block comment at the first star-slash sequence, so a comment
+ * value containing one can never reach this function through a
+ * normally-parsed file.
  */
 function findTerminatorsInExamples(commentText) {
     const lines = commentText.split('\n');

package/src/rules/reliability/no-missing-null-checks.d.ts

@@ -11,7 +11,7 @@
  * @see https://cwe.mitre.org/data/definitions/476.html
  * @see https://rules.sonarsource.com/javascript/RSPEC-2259/
  */
-import type { TSESLint } from '@interlace/eslint-devkit';
+import type { TSESLint, TSESTree } from '@interlace/eslint-devkit';
 type MessageIds = 'missingNullCheck' | 'useOptionalChaining' | 'useNullishCoalescing' | 'addExplicitCheck';
 export interface Options {
     /** Ignore in test files. Default: true */
@@ -20,6 +20,15 @@ export interface Options {
     requireExplicitChecks?: boolean;
 }
 type RuleOptions = [Options?];
+/**
+ * Check if property access has null/undefined check
+ *
+ * Exported for direct Layer-2 unit testing: the `node.optional` and
+ * ChainExpression early returns are defensive duplicates of checks both
+ * callers perform before invoking this helper, so they are unreachable
+ * through the rule's listeners.
+ */
+export declare function hasNullCheck(node: TSESTree.MemberExpression, sourceCode: TSESLint.SourceCode): boolean;
 export declare const noMissingNullChecks: TSESLint.RuleModule<MessageIds, RuleOptions, unknown, TSESLint.RuleListener> & {
     name: string;
 };

package/src/rules/reliability/no-missing-null-checks.js

@@ -6,6 +6,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noMissingNullChecks = void 0;
+exports.hasNullCheck = hasNullCheck;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
 /**
@@ -128,6 +129,11 @@ function isProvablyNonNullableIdentifier(ident, scope) {
 }
 /**
  * Check if property access has null/undefined check
+ *
+ * Exported for direct Layer-2 unit testing: the `node.optional` and
+ * ChainExpression early returns are defensive duplicates of checks both
+ * callers perform before invoking this helper, so they are unreachable
+ * through the rule's listeners.
  */
 function hasNullCheck(node, sourceCode) {
     // Check if node itself uses optional chaining
@@ -445,7 +451,6 @@ exports.noMissingNullChecks = (0, eslint_devkit_2.createRule)({
                             },
                         ],
                     });
-                    /* c8 ignore next 4 -- defensive error handling, hard to trigger in tests */
                 }
                 catch {
                     // Silently skip if there's an error
@@ -458,7 +463,6 @@ exports.noMissingNullChecks = (0, eslint_devkit_2.createRule)({
          * Only check if it's an actual method call, not just a property access
          */
         function checkCallExpression(node) {
-            /* c8 ignore next 4 -- defensive type check, always true when called by ESLint visitor */
             // Ensure this is actually a CallExpression (not just a MemberExpression)
             if (node.type !== 'CallExpression') {
                 return;
@@ -526,7 +530,6 @@ exports.noMissingNullChecks = (0, eslint_devkit_2.createRule)({
                                 },
                             ],
                         });
-                        /* c8 ignore next 4 -- defensive error handling, hard to trigger in tests */
                     }
                     catch {
                         // Silently skip if there's an error