eslint-plugin-unicorn 65.0.0 → 65.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "eslint-plugin-unicorn",
-	"version": "65.0.0",
+	"version": "65.0.1",
 	"description": "More than 100 powerful ESLint rules",
 	"license": "MIT",
 	"repository": "sindresorhus/eslint-plugin-unicorn",

package/rules/no-unused-array-method-return.js

@@ -177,7 +177,10 @@ function resolveReceiver(node, context, visitedNodes = new Set()) {
 		return value === undefined ? node : resolveReceiver(value, context, visitedNodes);
 	}
 
-	if (node.type === 'ChainExpression') {
+	// Transparent wrappers that do not change the receiver's runtime value.
+	if (
+		['ChainExpression', 'TSNonNullExpression', 'TSSatisfiesExpression'].includes(node.type)
+	) {
 		return resolveReceiver(node.expression, context, visitedNodes);
 	}
 
@@ -185,6 +188,27 @@ function resolveReceiver(node, context, visitedNodes = new Set()) {
 		return uncertainValue;
 	}
 
+	if (node.type === 'TSAsExpression' || node.type === 'TSTypeAssertion') {
+		let {typeAnnotation} = node;
+
+		// Unwrap `readonly Foo[]` / `readonly [A, B]` to the underlying type.
+		if (typeAnnotation.type === 'TSTypeOperator' && typeAnnotation.operator === 'readonly') {
+			typeAnnotation = typeAnnotation.typeAnnotation;
+		}
+
+		// An array-type assertion (`Foo[]`, `Array<…>`, `ReadonlyArray<…>`) guarantees
+		// an array receiver, regardless of the asserted expression, so report it.
+		// Anything else (`Foo`, `any`, `unknown`, unions, tuples, …) stays unresolved.
+		const isArrayTypeAssertion = typeAnnotation.type === 'TSArrayType'
+			|| (
+				typeAnnotation.type === 'TSTypeReference'
+				&& typeAnnotation.typeName.type === 'Identifier'
+				&& (typeAnnotation.typeName.name === 'Array' || typeAnnotation.typeName.name === 'ReadonlyArray')
+			);
+
+		return isArrayTypeAssertion ? node : uncertainValue;
+	}
+
 	// Supported receiver inference boundary:
 	// - the receiver expression itself, if it is direct array syntax like `[]`
 	// - trivial identifier aliases to that same initializer, like `const alias = values`

package/rules/prefer-https.js

@@ -31,6 +31,14 @@ function shouldReport(authority) {
 	return Boolean(hostname && hasPublicTld(hostname));
 }
 
+function isXmlNamespaceValue(text, matchIndex) {
+	// 30 chars covers `xmlns:somePrefix="` with a prefix up to 22 chars long.
+	const preceding = text.slice(Math.max(0, matchIndex - 30), matchIndex);
+	// \b prevents matching words ending in "xmlns" (e.g. notxmlns).
+	// [\w.-]+ covers XML NCNames, which allow hyphens and dots (e.g. xmlns:xsl-fo).
+	return /\bxmlns(?::[\w.-]+)?\s*=\s*["']?$/i.test(preceding);
+}
+
 /** @param {import('eslint').Rule.RuleContext} context */
 const create = context => {
 	let checked = false;
@@ -46,11 +54,16 @@ const create = context => {
 		const {text} = sourceCode;
 
 		for (const match of text.matchAll(HTTP_URL)) {
+			const start = match.index;
+
+			if (isXmlNamespaceValue(text, start)) {
+				continue;
+			}
+
 			if (!shouldReport(match.groups.authority)) {
 				continue;
 			}
 
-			const start = match.index;
 			const end = start + match[0].length;
 
 			context.report({