eslint-linter-browserify 10.2.0 → 10.2.1

package/linter.cjs

@@ -4216,7 +4216,7 @@ function requireEslintVisitorKeys$2 () {
 	return eslintVisitorKeys$2;
 }
 
-var version = "10.2.0";
+var version = "10.2.1";
 var require$$3$1 = {
 	version: version};
 
@@ -11523,11 +11523,10 @@ function requireCodePathState () {
 			this.position = "try";
 
 			/**
-			 * If the `try` statement has a `finally` block, this affects how a
-			 * `return` statement behaves in the `try` block. Without `finally`,
-			 * `return` behaves as usual and doesn't require a fork; with `finally`,
-			 * `return` forks into the `finally` block, so we need a fork context
-			 * to track it.
+			 * If the `try` statement has a `finally` block, this affects how return-like
+			 * leaving paths behave in the `try` block. Without `finally`, they behave as
+			 * usual and don't require a fork; with `finally`, they fork into the
+			 * `finally` block, so we need a fork context to track them.
 			 * @type {ForkContext|null}
 			 */
 			this.returnedForkContext = hasFinalizer
@@ -12709,6 +12708,23 @@ function requireCodePathState () {
 			this.forkContext.add(segments);
 		}
 
+		/**
+		 * Records abrupt resumption paths from a suspended `yield` expression,
+		 * then splits normal post-`yield` continuation into a fresh segment.
+		 * @returns {void}
+		 */
+		makeYield() {
+			const forkContext = this.forkContext;
+			const leavingSegments = forkContext.head;
+
+			if (forkContext.reachable) {
+				getReturnContext(this).returnedForkContext.add(leavingSegments);
+				getThrowContext(this).thrownForkContext.add(leavingSegments);
+
+				forkContext.replaceHead(forkContext.makeNext(-1, -1));
+			}
+		}
+
 		/**
 		 * Makes a code path segment from the first throwable node to the `catch`
 		 * block or the `finally` block.
@@ -13484,9 +13500,10 @@ function requireCodePath () {
 		 * Final code path segments that represent normal completion of the code path.
 		 * For functions, this means both explicit `return` statements and implicit returns,
 		 * such as the last reachable segment in a function that does not have an
-		 * explicit `return` as this implicitly returns `undefined`. For scripts,
-		 * modules, class field initializers, and class static blocks, this means
-		 * all lines of code have been executed.
+		 * explicit `return` as this implicitly returns `undefined`, as well as
+		 * return-like exits from suspended `yield` expressions. For scripts, modules,
+		 * class field initializers, and class static blocks, this means all lines of
+		 * code have been executed.
 		 * These segments are also present in `finalSegments`.
 		 * This is a passthrough to the underlying `CodePathState`.
 		 * @type {CodePathSegment[]}
@@ -13496,7 +13513,8 @@ function requireCodePath () {
 		}
 
 		/**
-		 * Final code path segments that represent `throw` statements.
+		 * Final code path segments that represent `throw` statements and throw-like
+		 * exits from suspended `yield` expressions.
 		 * This is a passthrough to the underlying `CodePathState`.
 		 * These segments are also present in `finalSegments`.
 		 * @type {CodePathSegment[]}
@@ -13755,7 +13773,7 @@ function requireCodePathAnalyzer () {
 
 	/**
 	 * Checks if a given node appears as the value of a PropertyDefinition node.
-	 * @param {ASTNode} node THe node to check.
+	 * @param {ASTNode} node The node to check.
 	 * @returns {boolean} `true` if the node is a PropertyDefinition value,
 	 *      false if not.
 	 */
@@ -14225,7 +14243,7 @@ function requireCodePathAnalyzer () {
 
 			case "SwitchCase":
 				/*
-				 * Fork if this node is after the 2st node in `cases`.
+				 * Fork if this node is after the 1st node in `cases`.
 				 * It's similar to `else` blocks.
 				 * The next `test` node is processed in this path.
 				 */
@@ -14344,10 +14362,13 @@ function requireCodePathAnalyzer () {
 			case "ImportExpression":
 			case "MemberExpression":
 			case "NewExpression":
-			case "YieldExpression":
 				state.makeFirstThrowablePathInTryBlock();
 				break;
 
+			case "YieldExpression":
+				state.makeYield();
+				break;
+
 			case "WhileStatement":
 			case "DoWhileStatement":
 			case "ForStatement":
@@ -19283,15 +19304,16 @@ function requireFlatConfigSchema () {
 	}
 
 	/**
-	 * Validates that a given string is the form pluginName/objectName.
+	 * Validates that a given string matches the "pluginName/memberPlaceholder" pattern.
 	 * @param {string} value The string to check.
+	 * @param {string} memberPlaceholder The placeholder for the member portion of the expected format in the error message.
 	 * @returns {void}
-	 * @throws {TypeError} If the string isn't in the correct format.
+	 * @throws {TypeError} If the string doesn't match the expected pattern.
 	 */
-	function assertIsPluginMemberName(value) {
+	function assertIsPluginMemberName(value, memberPlaceholder) {
 		if (!/[\w\-@$]+(?:\/[\w\-$]+)+$/iu.test(value)) {
 			throw new TypeError(
-				`Expected string in the form "pluginName/objectName" but found "${value}".`,
+				`Expected string in the form "pluginName/${memberPlaceholder}" but found "${value}".`,
 			);
 		}
 	}
@@ -19434,7 +19456,9 @@ function requireFlatConfigSchema () {
 	/** @type {ObjectPropertySchema} */
 	const languageSchema = {
 		merge: "replace",
-		validate: assertIsPluginMemberName,
+		validate(value) {
+			assertIsPluginMemberName(value, "languageName");
+		},
 	};
 
 	/** @type {ObjectPropertySchema} */
@@ -19489,7 +19513,7 @@ function requireFlatConfigSchema () {
 		merge: "replace",
 		validate(value) {
 			if (typeof value === "string") {
-				assertIsPluginMemberName(value);
+				assertIsPluginMemberName(value, "processorName");
 			} else if (value && typeof value === "object") {
 				if (
 					typeof value.preprocess !== "function" ||
@@ -73973,12 +73997,18 @@ function requireNoAsyncPromiseExecutor () {
 		},
 
 		create(context) {
+			const sourceCode = context.sourceCode;
+
 			return {
 				"NewExpression[callee.name='Promise'][arguments.0.async=true]"(
 					node,
 				) {
+					if (!sourceCode.isGlobalReference(node.callee)) {
+						return;
+					}
+
 					context.report({
-						node: context.sourceCode.getFirstToken(
+						node: sourceCode.getFirstToken(
 							node.arguments[0],
 							token => token.value === "async",
 						),
@@ -98974,7 +99004,7 @@ function requireNoShadow () {
 
 			/**
 			 * Checks the current context for shadowed variables.
-			 * @param {Scope} scope Fixme
+			 * @param {Scope} scope The scope to check for shadowed variables.
 			 * @returns {void}
 			 */
 			function checkForShadows(scope) {

package/linter.js

@@ -4218,7 +4218,7 @@
 		return eslintVisitorKeys$2;
 	}
 
-	var version = "10.2.0";
+	var version = "10.2.1";
 	var require$$3$1 = {
 		version: version};
 
@@ -11525,11 +11525,10 @@
 				this.position = "try";
 
 				/**
-				 * If the `try` statement has a `finally` block, this affects how a
-				 * `return` statement behaves in the `try` block. Without `finally`,
-				 * `return` behaves as usual and doesn't require a fork; with `finally`,
-				 * `return` forks into the `finally` block, so we need a fork context
-				 * to track it.
+				 * If the `try` statement has a `finally` block, this affects how return-like
+				 * leaving paths behave in the `try` block. Without `finally`, they behave as
+				 * usual and don't require a fork; with `finally`, they fork into the
+				 * `finally` block, so we need a fork context to track them.
 				 * @type {ForkContext|null}
 				 */
 				this.returnedForkContext = hasFinalizer
@@ -12711,6 +12710,23 @@
 				this.forkContext.add(segments);
 			}
 
+			/**
+			 * Records abrupt resumption paths from a suspended `yield` expression,
+			 * then splits normal post-`yield` continuation into a fresh segment.
+			 * @returns {void}
+			 */
+			makeYield() {
+				const forkContext = this.forkContext;
+				const leavingSegments = forkContext.head;
+
+				if (forkContext.reachable) {
+					getReturnContext(this).returnedForkContext.add(leavingSegments);
+					getThrowContext(this).thrownForkContext.add(leavingSegments);
+
+					forkContext.replaceHead(forkContext.makeNext(-1, -1));
+				}
+			}
+
 			/**
 			 * Makes a code path segment from the first throwable node to the `catch`
 			 * block or the `finally` block.
@@ -13486,9 +13502,10 @@
 			 * Final code path segments that represent normal completion of the code path.
 			 * For functions, this means both explicit `return` statements and implicit returns,
 			 * such as the last reachable segment in a function that does not have an
-			 * explicit `return` as this implicitly returns `undefined`. For scripts,
-			 * modules, class field initializers, and class static blocks, this means
-			 * all lines of code have been executed.
+			 * explicit `return` as this implicitly returns `undefined`, as well as
+			 * return-like exits from suspended `yield` expressions. For scripts, modules,
+			 * class field initializers, and class static blocks, this means all lines of
+			 * code have been executed.
 			 * These segments are also present in `finalSegments`.
 			 * This is a passthrough to the underlying `CodePathState`.
 			 * @type {CodePathSegment[]}
@@ -13498,7 +13515,8 @@
 			}
 
 			/**
-			 * Final code path segments that represent `throw` statements.
+			 * Final code path segments that represent `throw` statements and throw-like
+			 * exits from suspended `yield` expressions.
 			 * This is a passthrough to the underlying `CodePathState`.
 			 * These segments are also present in `finalSegments`.
 			 * @type {CodePathSegment[]}
@@ -13757,7 +13775,7 @@
 
 		/**
 		 * Checks if a given node appears as the value of a PropertyDefinition node.
-		 * @param {ASTNode} node THe node to check.
+		 * @param {ASTNode} node The node to check.
 		 * @returns {boolean} `true` if the node is a PropertyDefinition value,
 		 *      false if not.
 		 */
@@ -14227,7 +14245,7 @@
 
 				case "SwitchCase":
 					/*
-					 * Fork if this node is after the 2st node in `cases`.
+					 * Fork if this node is after the 1st node in `cases`.
 					 * It's similar to `else` blocks.
 					 * The next `test` node is processed in this path.
 					 */
@@ -14346,10 +14364,13 @@
 				case "ImportExpression":
 				case "MemberExpression":
 				case "NewExpression":
-				case "YieldExpression":
 					state.makeFirstThrowablePathInTryBlock();
 					break;
 
+				case "YieldExpression":
+					state.makeYield();
+					break;
+
 				case "WhileStatement":
 				case "DoWhileStatement":
 				case "ForStatement":
@@ -19285,15 +19306,16 @@
 		}
 
 		/**
-		 * Validates that a given string is the form pluginName/objectName.
+		 * Validates that a given string matches the "pluginName/memberPlaceholder" pattern.
 		 * @param {string} value The string to check.
+		 * @param {string} memberPlaceholder The placeholder for the member portion of the expected format in the error message.
 		 * @returns {void}
-		 * @throws {TypeError} If the string isn't in the correct format.
+		 * @throws {TypeError} If the string doesn't match the expected pattern.
 		 */
-		function assertIsPluginMemberName(value) {
+		function assertIsPluginMemberName(value, memberPlaceholder) {
 			if (!/[\w\-@$]+(?:\/[\w\-$]+)+$/iu.test(value)) {
 				throw new TypeError(
-					`Expected string in the form "pluginName/objectName" but found "${value}".`,
+					`Expected string in the form "pluginName/${memberPlaceholder}" but found "${value}".`,
 				);
 			}
 		}
@@ -19436,7 +19458,9 @@
 		/** @type {ObjectPropertySchema} */
 		const languageSchema = {
 			merge: "replace",
-			validate: assertIsPluginMemberName,
+			validate(value) {
+				assertIsPluginMemberName(value, "languageName");
+			},
 		};
 
 		/** @type {ObjectPropertySchema} */
@@ -19491,7 +19515,7 @@
 			merge: "replace",
 			validate(value) {
 				if (typeof value === "string") {
-					assertIsPluginMemberName(value);
+					assertIsPluginMemberName(value, "processorName");
 				} else if (value && typeof value === "object") {
 					if (
 						typeof value.preprocess !== "function" ||
@@ -73975,12 +73999,18 @@
 			},
 
 			create(context) {
+				const sourceCode = context.sourceCode;
+
 				return {
 					"NewExpression[callee.name='Promise'][arguments.0.async=true]"(
 						node,
 					) {
+						if (!sourceCode.isGlobalReference(node.callee)) {
+							return;
+						}
+
 						context.report({
-							node: context.sourceCode.getFirstToken(
+							node: sourceCode.getFirstToken(
 								node.arguments[0],
 								token => token.value === "async",
 							),
@@ -98976,7 +99006,7 @@
 
 				/**
 				 * Checks the current context for shadowed variables.
-				 * @param {Scope} scope Fixme
+				 * @param {Scope} scope The scope to check for shadowed variables.
 				 * @returns {void}
 				 */
 				function checkForShadows(scope) {