fix: fix regression of global mode report range in strict rule (#20462)

ntnyq · web-flow · commit 727451eff55b · 2026-01-27T15:04:13.000+01:00
* fix: fix regression of global mode report range in `strict`

* test: tweak test cases
diff --git a/lib/rules/strict.js b/lib/rules/strict.js
@@ -268,6 +268,7 @@ module.exports = {
 			}
 		}
 
+		/** @type {import('../types').Rule.RuleListener} */
 		const rule = {
 			Program(node) {
 				const useStrictDirectives = getUseStrictDirectives(node.body);
@@ -281,7 +282,16 @@ module.exports = {
 						node.body.length > 0 &&
 						useStrictDirectives.length === 0
 					) {
-						context.report({ node, messageId: "global" });
+						/*
+						 * Report the range as v9 does
+						 */
+						context.report({
+							loc: {
+								start: node.body[0].loc.start,
+								end: node.body.at(-1).loc.end,
+							},
+							messageId: "global",
+						});
 					}
 					reportAllExceptFirst(useStrictDirectives, "multiple", true);
 				} else {
diff --git a/tests/lib/rules/strict.js b/tests/lib/rules/strict.js
@@ -68,6 +68,10 @@ ruleTester.run("strict", rule, {
 		// "global" mode
 		{ code: "// Intentionally empty", options: ["global"] },
 		{ code: '"use strict"; foo();', options: ["global"] },
+		{
+			code: "/* license */\n/* eslint-disable rule-to-test/strict */\nfoo();",
+			options: ["global"],
+		},
 		{
 			code: "foo();",
 			options: ["global"],
@@ -397,6 +401,20 @@ ruleTester.run("strict", rule, {
 				},
 			],
 		},
+		{
+			code: "/* license */\nfunction foo() {}\nfunction bar() {}\n/* end */",
+			output: null,
+			options: ["global"],
+			errors: [
+				{
+					messageId: "global",
+					line: 2,
+					column: 1,
+					endLine: 3,
+					endColumn: 18,
+				},
+			],
+		},
 		{
 			code: "function foo() { 'use strict'; return; }",
 			output: null,
