eslint-plugin-th-rules 2.5.1 → 2.6.0

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [2.6.0](https://github.com/tomerh2001/eslint-plugin-th-rules/compare/v2.5.1...v2.6.0) (2026-01-14)
+
+
+### Features
+
+* add prefer-explicit-nil-or-empty-check rule and update dependencies ([c8fb3da](https://github.com/tomerh2001/eslint-plugin-th-rules/commit/c8fb3da5be98caf0c31a50fa72f16b07e410eff6))
+
 ## [2.5.1](https://github.com/tomerh2001/eslint-plugin-th-rules/compare/v2.5.0...v2.5.1) (2026-01-14)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-th-rules",
-  "version": "2.5.1",
+  "version": "2.6.0",
   "description": "A List of custom ESLint rules created by Tomer Horowitz",
   "keywords": [
     "eslint",
@@ -51,6 +51,7 @@
     "@types/lodash": "^4",
     "@types/requireindex": "^1.2.4",
     "@types/xo": "^0.39.9",
+    "@typescript-eslint/rule-tester": "^8.53.0",
     "bun-types": "latest",
     "eslint": "^9.39.2",
     "eslint-doc-generator": "^3.0.2",

package/src/index.js

@@ -115,6 +115,7 @@ plugin.configs['recommended-typescript'] = flatConfigs(
 			},
 		},
 		rules: {
+			'th-rules/prefer-explicit-nil-or-empty-check': 'off',
 			'@typescript-eslint/consistent-type-definitions': 'off',
 			'@typescript-eslint/no-extraneous-class': 'off',
 			'@typescript-eslint/no-duplicate-type-constituents': 'off',

package/src/rules/prefer-explicit-nil-or-empty-check.js

@@ -0,0 +1,115 @@
+'use strict';
+
+const ts = require('typescript');
+
+const meta = {
+	type: 'problem',
+	docs: {
+		description:
+			'Disallow implicit truthy/falsy checks on non-boolean values; require _.isNil or _.isEmpty',
+		category: 'Best Practices',
+		recommended: true,
+		url: 'https://github.com/tomerh2001/eslint-plugin-th-rules/blob/main/docs/rules/prefer-explicit-nil-or-empty-check.md',
+	},
+	hasSuggestions: true,
+	schema: [],
+	requiresTypeChecking: true,
+};
+
+function create(context) {
+	const sourceCode = context.getSourceCode();
+	const {parserServices} = context;
+
+	// Safety: rule is disabled if type info is unavailable
+	if (!parserServices?.program || !parserServices.esTreeNodeToTSNodeMap) {
+		return {};
+	}
+
+	const checker = parserServices.program.getTypeChecker();
+
+	function isImplicitTruthyTarget(node) {
+		return (
+			node.type === 'Identifier'
+			|| node.type === 'MemberExpression'
+		);
+	}
+
+	function isBooleanType(node) {
+		const tsNode = parserServices.esTreeNodeToTSNodeMap.get(node);
+		if (!tsNode) {
+			return false;
+		}
+
+		const type = checker.getTypeAtLocation(tsNode);
+
+		return (
+			(type.flags & ts.TypeFlags.Boolean) !== 0
+			|| (type.flags & ts.TypeFlags.BooleanLiteral) !== 0
+		);
+	}
+
+	function report(node, testedNode, negated) {
+		const text = sourceCode.getText(testedNode);
+
+		const isNilExpr = negated
+			? `!_.isNil(${text})`
+			: `_.isNil(${text})`;
+
+		const isEmptyExpr = negated
+			? `!_.isEmpty(${text})`
+			: `_.isEmpty(${text})`;
+
+		context.report({
+			node,
+			message:
+				'Avoid implicit truthy/falsy checks on non-boolean values; use _.isNil or _.isEmpty instead',
+			suggest: [
+				{
+					desc: `Replace with ${isNilExpr}`,
+					fix(fixer) {
+						return fixer.replaceText(node, isNilExpr);
+					},
+				},
+				{
+					desc: `Replace with ${isEmptyExpr}`,
+					fix(fixer) {
+						return fixer.replaceText(node, isEmptyExpr);
+					},
+				},
+			],
+		});
+	}
+
+	return {
+		IfStatement(node) {
+			const {test} = node;
+
+			// If (!value)
+			if (test.type === 'UnaryExpression' && test.operator === '!') {
+				const arg = test.argument;
+
+				if (!isImplicitTruthyTarget(arg)) {
+					return;
+				}
+
+				if (isBooleanType(arg)) {
+					return;
+				}
+
+				report(test, arg, true);
+				return;
+			}
+
+			// If (value)
+			if (isImplicitTruthyTarget(test)) {
+				if (isBooleanType(test)) {
+					return;
+				}
+
+				report(test, test, false);
+			}
+		},
+	};
+}
+
+module.exports = {meta, create};

package/tests/no-boolean-coercion.test.ts

@@ -1,4 +1,4 @@
-const {RuleTester} = require('eslint');
+const {RuleTester} = require('@typescript-eslint/rule-tester');
 const rule = require('../src/rules/no-boolean-coercion');
 
 const ruleTester = new RuleTester({});

package/tests/prefer-is-empty.test.ts

@@ -1,4 +1,4 @@
-const {RuleTester} = require('eslint');
+const {RuleTester} = require('@typescript-eslint/rule-tester');
 const rule = require('../src/rules/prefer-is-empty');
 
 const ruleTester = new RuleTester({});