eslint 9.25.0 → 9.25.1

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

@@ -29,6 +29,8 @@ function alwaysFalse() {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
+		dialects: ["javascript", "typescript"],
+		language: "javascript",
 		type: "suggestion",
 
 		docs: {
@@ -83,6 +85,61 @@ module.exports = {
 			},
 		] = context.options;
 
+		/**
+		 * Has AST suggesting a directive.
+		 * @param {ASTNode} node any node
+		 * @returns {boolean} whether the given node structurally represents a directive
+		 */
+		function looksLikeDirective(node) {
+			return (
+				node.type === "ExpressionStatement" &&
+				node.expression.type === "Literal" &&
+				typeof node.expression.value === "string"
+			);
+		}
+
+		/**
+		 * Gets the leading sequence of members in a list that pass the predicate.
+		 * @param {Function} predicate ([a] -> Boolean) the function used to make the determination
+		 * @param {a[]} list the input list
+		 * @returns {a[]} the leading sequence of members in the given list that pass the given predicate
+		 */
+		function takeWhile(predicate, list) {
+			for (let i = 0; i < list.length; ++i) {
+				if (!predicate(list[i])) {
+					return list.slice(0, i);
+				}
+			}
+			return list.slice();
+		}
+
+		/**
+		 * Gets leading directives nodes in a Node body.
+		 * @param {ASTNode} node a Program or BlockStatement node
+		 * @returns {ASTNode[]} the leading sequence of directive nodes in the given node's body
+		 */
+		function directives(node) {
+			return takeWhile(looksLikeDirective, node.body);
+		}
+
+		/**
+		 * Detect if a Node is a directive.
+		 * @param {ASTNode} node any node
+		 * @returns {boolean} whether the given node is considered a directive in its current position
+		 */
+		function isDirective(node) {
+			/**
+			 * https://tc39.es/ecma262/#directive-prologue
+			 *
+			 * Only `FunctionBody`, `ScriptBody` and `ModuleBody` can have directive prologue.
+			 * Class static blocks do not have directive prologue.
+			 */
+			return (
+				astUtils.isTopLevelExpressionStatement(node) &&
+				directives(node.parent).includes(node)
+			);
+		}
+
 		/**
 		 * The member functions return `true` if the type has no side-effects.
 		 * Unknown nodes are handled as `false`, then this rule ignores those.
@@ -154,7 +211,7 @@ module.exports = {
 			ExpressionStatement(node) {
 				if (
 					Checker.isDisallowed(node.expression) &&
-					!astUtils.isDirective(node)
+					!isDirective(node)
 				) {
 					context.report({ node, messageId: "unusedExpression" });
 				}

package/lib/rules/utils/ast-utils.js

@@ -1088,13 +1088,13 @@ function isConstant(scope, node, inBooleanPosition) {
 }
 
 /**
- * Checks whether a node is an ExpressionStatement at the top level of a file or function body.
+ * Checks whether a node is an ExpressionStatement at the top level of a file, function body, or TypeScript module block.
  * A top-level ExpressionStatement node is a directive if it contains a single unparenthesized
  * string literal and if it occurs either as the first sibling or immediately after another
  * directive.
  * @param {ASTNode} node The node to check.
  * @returns {boolean} Whether or not the node is an ExpressionStatement at the top level of a
- * file or function body.
+ * file, function body, or TypeScript module block.
  */
 function isTopLevelExpressionStatement(node) {
 	if (node.type !== "ExpressionStatement") {
@@ -1104,6 +1104,7 @@ function isTopLevelExpressionStatement(node) {
 
 	return (
 		parent.type === "Program" ||
+		parent.type === "TSModuleBlock" ||
 		(parent.type === "BlockStatement" && isFunction(parent.parent))
 	);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "9.25.0",
+  "version": "9.25.1",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -110,7 +110,7 @@
     "@eslint/config-helpers": "^0.2.1",
     "@eslint/core": "^0.13.0",
     "@eslint/eslintrc": "^3.3.1",
-    "@eslint/js": "9.25.0",
+    "@eslint/js": "9.25.1",
     "@eslint/plugin-kit": "^0.2.8",
     "@humanfs/node": "^0.16.6",
     "@humanwhocodes/module-importer": "^1.0.1",