eslint-plugin-chai-friendly 1.1.1 → 1.2.0

package/lib/rules/no-unused-expressions.js

@@ -51,6 +51,9 @@ module.exports = {
                     enforceForJSX: {
                         type: "boolean",
                         default: false
+                    },
+                    ignoreDirectives: {
+                        type: "boolean"
                     }
                 },
                 additionalProperties: false
@@ -66,7 +69,8 @@ module.exports = {
             allowShortCircuit = config.allowShortCircuit || false,
             allowTernary = config.allowTernary || false,
             allowTaggedTemplates = config.allowTaggedTemplates || false,
-            enforceForJSX = config.enforceForJSX || false;
+            enforceForJSX = config.enforceForJSX || false,
+            ignoreDirectives = config.ignoreDirectives || false;
 
         /**
          * @param {ASTNode} node - any node
@@ -101,12 +105,19 @@ module.exports = {
 
         /**
          * @param {ASTNode} node - any node
-         * @param {ASTNode[]} ancestors - the given node's ancestors
+         * @returns {boolean} whether the parser marked this node as a directive
+         */
+        function isParserDirective(node) {
+            return typeof node.directive === "string";
+        }
+
+        /**
+         * @param {ASTNode} node - any node
          * @returns {boolean} whether the given node is considered a directive in its current position
          */
-        function isDirective(node, ancestors) {
-            var parent = ancestors[ancestors.length - 1],
-                grandparent = ancestors[ancestors.length - 2];
+        function isDirective(node) {
+            var parent = node.parent,
+                grandparent = node.parent.parent;
 
             /**
              * https://tc39.es/ecma262/#directive-prologue
@@ -167,6 +178,20 @@ module.exports = {
             ThisExpression: alwaysTrue,
             UnaryExpression(node) {
                 return node.operator !== "void" && node.operator !== "delete";
+            },
+
+            // TypeScript-specific node types
+            TSAsExpression(node) {
+                return Checker.isDisallowed(node.expression);
+            },
+            TSTypeAssertion(node) {
+                return Checker.isDisallowed(node.expression);
+            },
+            TSNonNullExpression(node) {
+                return Checker.isDisallowed(node.expression);
+            },
+            TSInstantiationExpression(node) {
+                return Checker.isDisallowed(node.expression);
             }
         });
 
@@ -255,18 +280,14 @@ module.exports = {
             return null;
         }
 
-        const sourceCode = context.sourceCode ?? context.getSourceCode();
         return {
             ExpressionStatement: function(node) {
-                var valid = !Checker.isDisallowed(node.expression)
-                    || isDirective(node,
-                        (sourceCode.getAncestors
-                            ? sourceCode.getAncestors(node)
-                            : context.getAncestors()
-                        ))
-                    || isChaiExpectCall(node)
-                    || isChaiShouldCall(node);
-                if (!valid) {
+                if (Checker.isDisallowed(node.expression)
+                    && !isParserDirective(node)
+                    && !(ignoreDirectives && isDirective(node))
+                    && !isChaiExpectCall(node)
+                    && !isChaiShouldCall(node)
+                ) {
                     context.report({node, messageId: "unusedExpression"});
                 }
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-chai-friendly",
-  "version": "1.1.1",
+  "version": "1.2.0",
   "description": "This plugin makes 'no-unused-expressions' rule friendly towards chai expect statements.",
   "keywords": [
     "eslint",
@@ -49,8 +49,9 @@
     "typescript": "^5.8.3"
   },
   "overrides": {
-    "flatted": "3.4.1",
-    "serialize-javascript": "7.0.4",
+    "flatted": "3.4.2",
+    "serialize-javascript": "7.0.5",
+    "brace-expansion": "5.0.5",
     "@istanbuljs/load-nyc-config": {
       "js-yaml": "3.14.2"
     },