@fedify/lint 2.4.0-dev.2122 → 2.4.0-dev.2127

package/deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@fedify/lint",
-  "version": "2.4.0-dev.2122+bdc0ce8d",
+  "version": "2.4.0-dev.2127+c41ce2eb",
   "license": "MIT",
   "exports": {
     ".": "./src/mod.ts",

package/dist/index.cjs

@@ -24,7 +24,7 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 	enumerable: true
 }) : target, mod));
 //#endregion
-const require_outbox_listener_delivery_required = require("./outbox-listener-delivery-required-DyJ1I2D0.cjs");
+const require_outbox_listener_delivery_required = require("./outbox-listener-delivery-required-ldftuG5P.cjs");
 let _fxts_core = require("@fxts/core");
 let _typescript_eslint_parser = require("@typescript-eslint/parser");
 _typescript_eslint_parser = __toESM(_typescript_eslint_parser, 1);

package/dist/index.js

@@ -1,4 +1,4 @@
-import { A as name, C as eslint$5, D as eslint$11, E as eslint$10, O as eslint$21, S as eslint$4, T as eslint$13, _ as eslint$15, a as eslint$23, b as eslint$2, c as eslint$16, d as eslint$22, f as eslint$6, g as eslint$14, h as eslint$9, i as eslint$27, j as version, k as RULE_IDS, l as eslint$17, m as eslint$8, n as eslint$25, o as eslint$18, p as eslint$7, r as eslint$26, s as eslint$19, t as eslint$24, u as eslint$20, v as eslint$1, w as eslint$12, x as eslint$3, y as eslint } from "./outbox-listener-delivery-required-6phcPuk2.js";
+import { A as name, C as eslint$5, D as eslint$11, E as eslint$10, O as eslint$21, S as eslint$4, T as eslint$13, _ as eslint$15, a as eslint$23, b as eslint$2, c as eslint$16, d as eslint$22, f as eslint$6, g as eslint$14, h as eslint$9, i as eslint$27, j as version, k as RULE_IDS, l as eslint$17, m as eslint$8, n as eslint$25, o as eslint$18, p as eslint$7, r as eslint$26, s as eslint$19, t as eslint$24, u as eslint$20, v as eslint$1, w as eslint$12, x as eslint$3, y as eslint } from "./outbox-listener-delivery-required-C_ixBt5W.js";
 import { fromEntries, keys, map, pipe } from "@fxts/core";
 import parser from "@typescript-eslint/parser";
 //#region src/index.ts

package/dist/{outbox-listener-delivery-required-6phcPuk2.js → outbox-listener-delivery-required-C_ixBt5W.js}

@@ -1,7 +1,7 @@
 import { always, every, head, isEmpty, isObject, negate, pipe, pipeLazy, prop, some, toArray, unless, when } from "@fxts/core";
 //#region deno.json
 var name = "@fedify/lint";
-var version = "2.4.0-dev.2122+bdc0ce8d";
+var version = "2.4.0-dev.2127+c41ce2eb";
 //#endregion
 //#region src/lib/const.ts
 /**
@@ -1135,11 +1135,46 @@ function alwaysExits(node) {
 		default: return false;
 	}
 }
-function collectReachableStatements(node, out) {
+function collectBindingExpressions(node, out, bindings, declares = false) {
+	switch (node.type) {
+		case "VariableDeclaration":
+			for (const decl of node.declarations) {
+				const names = [];
+				collectBoundNames(decl.id, names);
+				if (node.kind === "var") for (const name of names) bindings?.declaredHere.add(name);
+				collectBindingExpressions(decl.id, out, bindings, true);
+			}
+			break;
+		case "ObjectPattern":
+			for (const prop of node.properties) if (prop.type === "Property") {
+				if (prop.computed) out.push(prop.key);
+				collectBindingExpressions(prop.value, out, bindings, declares);
+			} else if (prop.type === "RestElement") collectBindingExpressions(prop.argument, out, bindings, declares);
+			break;
+		case "ArrayPattern":
+			for (const elem of node.elements) if (elem != null) collectBindingExpressions(elem, out, bindings, declares);
+			break;
+		case "AssignmentPattern":
+			out.push(node);
+			if (declares) bindings?.declarationPatterns.add(node);
+			collectBindingExpressions(node.left, out, bindings, declares);
+			break;
+		case "RestElement":
+			collectBindingExpressions(node.argument, out, bindings, declares);
+			break;
+		case "MemberExpression": {
+			const object = node.object;
+			if (object.type !== "Identifier") out.push(object);
+			if (node.computed) out.push(node.property);
+			break;
+		}
+	}
+}
+function collectReachableStatements(node, out, bindings) {
 	switch (node.type) {
 		case "BlockStatement":
 			for (const [index, statement] of node.body.entries()) {
-				collectReachableStatements(statement, out);
+				collectReachableStatements(statement, out, bindings);
 				if (alwaysExits(statement)) {
 					for (const rest of node.body.slice(index + 1)) if (rest.type === "FunctionDeclaration") out.push(rest);
 					return;
@@ -1149,49 +1184,53 @@ function collectReachableStatements(node, out) {
 		case "IfStatement": {
 			const test = node.test;
 			out.push(test);
-			if (!isStaticallyFalsy(test)) collectReachableStatements(node.consequent, out);
-			if (node.alternate != null && !isStaticallyTruthy(test)) collectReachableStatements(node.alternate, out);
+			if (!isStaticallyFalsy(test)) collectReachableStatements(node.consequent, out, bindings);
+			if (node.alternate != null && !isStaticallyTruthy(test)) collectReachableStatements(node.alternate, out, bindings);
 			return;
 		}
 		case "TryStatement":
-			collectReachableStatements(node.block, out);
-			if (node.handler != null) collectReachableStatements(node.handler.body, out);
-			if (node.finalizer != null) collectReachableStatements(node.finalizer, out);
+			collectReachableStatements(node.block, out, bindings);
+			if (node.handler != null) collectReachableStatements(node.handler.body, out, bindings);
+			if (node.finalizer != null) collectReachableStatements(node.finalizer, out, bindings);
 			return;
 		case "SwitchStatement":
 			out.push(node.discriminant);
 			for (const switchCase of node.cases) {
 				if (switchCase.test != null) out.push(switchCase.test);
 				for (const statement of switchCase.consequent) {
-					collectReachableStatements(statement, out);
+					collectReachableStatements(statement, out, bindings);
 					if (alwaysExits(statement)) break;
 				}
 			}
 			return;
 		case "WhileStatement":
+			out.push(node.test);
+			if (!isStaticallyFalsy(node.test)) collectReachableStatements(node.body, out, bindings);
+			return;
 		case "DoWhileStatement":
 			out.push(node.test);
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		case "ForStatement":
-			for (const head of [
-				node.init,
-				node.test,
-				node.update
-			]) if (head != null) out.push(head);
-			collectReachableStatements(node.body, out);
+			if (node.init != null) out.push(node.init);
+			if (node.test != null) out.push(node.test);
+			if (node.test == null || !isStaticallyFalsy(node.test)) {
+				if (node.update != null) out.push(node.update);
+				collectReachableStatements(node.body, out, bindings);
+			}
 			return;
 		case "ForInStatement":
 		case "ForOfStatement":
+			collectBindingExpressions(node.left, out, bindings);
 			out.push(node.right);
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		case "LabeledStatement":
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		case "WithStatement":
 			out.push(node.object);
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		default:
 			out.push(node);
@@ -1243,6 +1282,22 @@ function collectReferencedNames(node, out, crossFunctions = false) {
 	if (n.type === "AssignmentExpression" && n.operator === "=") {
 		if (getAssignmentTargetName(n.left) != null) {
 			collectReferencedNames(n.right, out, crossFunctions);
+			let current = n.left;
+			while (current.type === "MemberExpression") {
+				if (current.computed) collectReferencedNames(current.property, out, crossFunctions);
+				current = current.object;
+			}
+			return;
+		}
+	}
+	if (n.type === "AssignmentPattern") {
+		if (getAssignmentTargetName(n.left) != null) {
+			collectReferencedNames(n.right, out, crossFunctions);
+			let current = n.left;
+			while (current.type === "MemberExpression") {
+				if (current.computed) collectReferencedNames(current.property, out, crossFunctions);
+				current = current.object;
+			}
 			return;
 		}
 	}
@@ -1325,19 +1380,19 @@ function collectHeldFunctions(node, out) {
 * reachable, so it can be layered on top of (and correctly shadow) the outer
 * scope's names.
 */
-function collectFunctionsByName(node, out) {
+function collectFunctionsByName(node, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations) {
 	if (node == null || typeof node !== "object") return;
 	if (Array.isArray(node)) {
-		for (const item of node) collectFunctionsByName(item, out);
+		for (const item of node) collectFunctionsByName(item, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 		return;
 	}
 	if (!isNode(node)) return;
 	const n = node;
-	const bindTo = (names, from) => {
+	const bindTo = (names, from, bindings = out) => {
 		const functions = [];
 		collectHeldFunctions(from, functions);
 		if (functions.length < 1) return;
-		for (const name of names) out.set(name, [...out.get(name) ?? [], ...functions]);
+		for (const name of names) bindings.set(name, [...bindings.get(name) ?? [], ...functions]);
 	};
 	if (n.type === "FunctionDeclaration") {
 		if (n.id?.name != null) bindTo([n.id.name], n);
@@ -1348,26 +1403,49 @@ function collectFunctionsByName(node, out) {
 		return;
 	}
 	if (isFunctionLikeNode(n)) return;
+	if (n.type === "StaticBlock") {
+		collectFunctionsByName(n.body, out, assignmentDefaults, declarationPatterns, /* @__PURE__ */ new Set(), scopeDeclarations);
+		return;
+	}
+	if (n.type === "VariableDeclaration") {
+		if (n.kind === "var" || scopeDeclarations.has(n)) for (const decl of n.declarations) {
+			const names = [];
+			collectBoundNames(decl.id, names);
+			for (const name of names) declaredHere.add(name);
+		}
+		for (const decl of n.declarations) collectFunctionsByName(decl, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
+		return;
+	}
 	if (n.type === "VariableDeclarator") {
 		const decl = n;
+		const names = [];
+		collectBoundNames(decl.id, names);
 		if (decl.init != null) {
-			const names = [];
-			collectBoundNames(decl.id, names);
 			bindTo(names, decl.init);
-			collectFunctionsByName(decl.init, out);
+			collectFunctionsByName(decl.init, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 		}
 		return;
 	}
+	if (n.type === "AssignmentPattern") {
+		const names = [];
+		if (n.left.type === "MemberExpression") {
+			const name = getAssignmentTargetName(n.left);
+			if (name != null) names.push(name);
+		} else collectBoundNames(n.left, names);
+		bindTo(names, n.right, n.left.type === "MemberExpression" || !declarationPatterns.has(n) ? assignmentDefaults : out);
+		collectFunctionsByName(n.right, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
+		return;
+	}
 	if (n.type === "AssignmentExpression") {
 		const name = getAssignmentTargetName(n.left);
 		if (name != null) bindTo([name], n.right);
-		collectFunctionsByName(n.right, out);
+		collectFunctionsByName(n.right, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 		return;
 	}
 	const record = n;
 	for (const key in record) {
 		if (key === "parent") continue;
-		collectFunctionsByName(record[key], out);
+		collectFunctionsByName(record[key], out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 	}
 }
 /**
@@ -1410,11 +1488,27 @@ function walkUsedScopes(root, visit) {
 		if (visited.has(scopeRoot)) return;
 		visited.add(scopeRoot);
 		const statements = [];
-		collectReachableStatements(scopeRoot, statements);
+		const declarationPatterns = /* @__PURE__ */ new Set();
+		const declaredHere = /* @__PURE__ */ new Set();
+		collectReachableStatements(scopeRoot, statements, {
+			declarationPatterns,
+			declaredHere
+		});
 		const functionsHere = /* @__PURE__ */ new Map();
-		for (const statement of statements) collectFunctionsByName(statement, functionsHere);
+		const assignmentDefaultsHere = /* @__PURE__ */ new Map();
+		for (const param of scopeFn?.params ?? []) {
+			const names = [];
+			collectBoundNames(param, names);
+			for (const name of names) functionsHere.set(name, []);
+		}
+		const scopeDeclarations = new Set(scopeRoot.type === "BlockStatement" ? scopeRoot.body : []);
+		for (const statement of statements) collectFunctionsByName(statement, functionsHere, assignmentDefaultsHere, declarationPatterns, declaredHere, scopeDeclarations);
 		const functionsByName = new Map(outerFunctionsByName);
 		for (const [name, functions] of functionsHere) functionsByName.set(name, functions);
+		for (const [name, functions] of assignmentDefaultsHere) {
+			const base = declaredHere.has(name) ? functionsHere.get(name) ?? [] : functionsByName.get(name) ?? [];
+			functionsByName.set(name, [...base, ...functions]);
+		}
 		visit({
 			fn: scopeFn,
 			statements,
@@ -1428,7 +1522,7 @@ function walkUsedScopes(root, visit) {
 			for (const fn of functions) reached.add(fn);
 		}
 		const held = /* @__PURE__ */ new Set();
-		for (const functions of functionsHere.values()) for (const fn of functions) held.add(fn);
+		for (const bindings of [functionsHere, assignmentDefaultsHere]) for (const functions of bindings.values()) for (const fn of functions) held.add(fn);
 		for (const statement of statements) {
 			const nested = [];
 			collectNestedFunctions(statement, nested);
@@ -1670,7 +1764,11 @@ function fateOf(expression, analysis, shape = "promise") {
 			case "DoWhileStatement":
 			case "ForStatement": return DROPPED;
 			case "LogicalExpression":
+				if (get(parent, "operator") === "&&" && get(parent, "left") === current) return DROPPED;
+				current = parent;
+				continue;
 			case "Property":
+				if (get(parent, "computed") && get(parent, "key") === current) return DROPPED;
 				current = parent;
 				continue;
 			case "ObjectExpression":
@@ -1689,6 +1787,7 @@ function fateOf(expression, analysis, shape = "promise") {
 				continue;
 			}
 			case "MemberExpression": {
+				if (get(parent, "computed") && get(parent, "property") === current) return DROPPED;
 				if (get(parent, "object") !== current) return HANDLED;
 				const call = parentOf(parent);
 				const name = memberName(parent);
@@ -1722,6 +1821,12 @@ function fateOf(expression, analysis, shape = "promise") {
 				if (name == null) return HANDLED;
 				return analysis.mentioned.has(name) ? HANDLED : DROPPED;
 			}
+			case "AssignmentPattern": {
+				if (get(parent, "right") !== current) return HANDLED;
+				const name = getAssignmentTargetName(asNode(get(parent, "left")) ?? parent);
+				if (name == null) return HANDLED;
+				return analysis.mentioned.has(name) ? HANDLED : DROPPED;
+			}
 			default: return HANDLED;
 		}
 	}

package/dist/{outbox-listener-delivery-required-DyJ1I2D0.cjs → outbox-listener-delivery-required-ldftuG5P.cjs}

@@ -1,7 +1,7 @@
 let _fxts_core = require("@fxts/core");
 //#region deno.json
 var name = "@fedify/lint";
-var version = "2.4.0-dev.2122+bdc0ce8d";
+var version = "2.4.0-dev.2127+c41ce2eb";
 //#endregion
 //#region src/lib/const.ts
 /**
@@ -1135,11 +1135,46 @@ function alwaysExits(node) {
 		default: return false;
 	}
 }
-function collectReachableStatements(node, out) {
+function collectBindingExpressions(node, out, bindings, declares = false) {
+	switch (node.type) {
+		case "VariableDeclaration":
+			for (const decl of node.declarations) {
+				const names = [];
+				collectBoundNames(decl.id, names);
+				if (node.kind === "var") for (const name of names) bindings?.declaredHere.add(name);
+				collectBindingExpressions(decl.id, out, bindings, true);
+			}
+			break;
+		case "ObjectPattern":
+			for (const prop of node.properties) if (prop.type === "Property") {
+				if (prop.computed) out.push(prop.key);
+				collectBindingExpressions(prop.value, out, bindings, declares);
+			} else if (prop.type === "RestElement") collectBindingExpressions(prop.argument, out, bindings, declares);
+			break;
+		case "ArrayPattern":
+			for (const elem of node.elements) if (elem != null) collectBindingExpressions(elem, out, bindings, declares);
+			break;
+		case "AssignmentPattern":
+			out.push(node);
+			if (declares) bindings?.declarationPatterns.add(node);
+			collectBindingExpressions(node.left, out, bindings, declares);
+			break;
+		case "RestElement":
+			collectBindingExpressions(node.argument, out, bindings, declares);
+			break;
+		case "MemberExpression": {
+			const object = node.object;
+			if (object.type !== "Identifier") out.push(object);
+			if (node.computed) out.push(node.property);
+			break;
+		}
+	}
+}
+function collectReachableStatements(node, out, bindings) {
 	switch (node.type) {
 		case "BlockStatement":
 			for (const [index, statement] of node.body.entries()) {
-				collectReachableStatements(statement, out);
+				collectReachableStatements(statement, out, bindings);
 				if (alwaysExits(statement)) {
 					for (const rest of node.body.slice(index + 1)) if (rest.type === "FunctionDeclaration") out.push(rest);
 					return;
@@ -1149,49 +1184,53 @@ function collectReachableStatements(node, out) {
 		case "IfStatement": {
 			const test = node.test;
 			out.push(test);
-			if (!isStaticallyFalsy(test)) collectReachableStatements(node.consequent, out);
-			if (node.alternate != null && !isStaticallyTruthy(test)) collectReachableStatements(node.alternate, out);
+			if (!isStaticallyFalsy(test)) collectReachableStatements(node.consequent, out, bindings);
+			if (node.alternate != null && !isStaticallyTruthy(test)) collectReachableStatements(node.alternate, out, bindings);
 			return;
 		}
 		case "TryStatement":
-			collectReachableStatements(node.block, out);
-			if (node.handler != null) collectReachableStatements(node.handler.body, out);
-			if (node.finalizer != null) collectReachableStatements(node.finalizer, out);
+			collectReachableStatements(node.block, out, bindings);
+			if (node.handler != null) collectReachableStatements(node.handler.body, out, bindings);
+			if (node.finalizer != null) collectReachableStatements(node.finalizer, out, bindings);
 			return;
 		case "SwitchStatement":
 			out.push(node.discriminant);
 			for (const switchCase of node.cases) {
 				if (switchCase.test != null) out.push(switchCase.test);
 				for (const statement of switchCase.consequent) {
-					collectReachableStatements(statement, out);
+					collectReachableStatements(statement, out, bindings);
 					if (alwaysExits(statement)) break;
 				}
 			}
 			return;
 		case "WhileStatement":
+			out.push(node.test);
+			if (!isStaticallyFalsy(node.test)) collectReachableStatements(node.body, out, bindings);
+			return;
 		case "DoWhileStatement":
 			out.push(node.test);
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		case "ForStatement":
-			for (const head of [
-				node.init,
-				node.test,
-				node.update
-			]) if (head != null) out.push(head);
-			collectReachableStatements(node.body, out);
+			if (node.init != null) out.push(node.init);
+			if (node.test != null) out.push(node.test);
+			if (node.test == null || !isStaticallyFalsy(node.test)) {
+				if (node.update != null) out.push(node.update);
+				collectReachableStatements(node.body, out, bindings);
+			}
 			return;
 		case "ForInStatement":
 		case "ForOfStatement":
+			collectBindingExpressions(node.left, out, bindings);
 			out.push(node.right);
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		case "LabeledStatement":
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		case "WithStatement":
 			out.push(node.object);
-			collectReachableStatements(node.body, out);
+			collectReachableStatements(node.body, out, bindings);
 			return;
 		default:
 			out.push(node);
@@ -1243,6 +1282,22 @@ function collectReferencedNames(node, out, crossFunctions = false) {
 	if (n.type === "AssignmentExpression" && n.operator === "=") {
 		if (getAssignmentTargetName(n.left) != null) {
 			collectReferencedNames(n.right, out, crossFunctions);
+			let current = n.left;
+			while (current.type === "MemberExpression") {
+				if (current.computed) collectReferencedNames(current.property, out, crossFunctions);
+				current = current.object;
+			}
+			return;
+		}
+	}
+	if (n.type === "AssignmentPattern") {
+		if (getAssignmentTargetName(n.left) != null) {
+			collectReferencedNames(n.right, out, crossFunctions);
+			let current = n.left;
+			while (current.type === "MemberExpression") {
+				if (current.computed) collectReferencedNames(current.property, out, crossFunctions);
+				current = current.object;
+			}
 			return;
 		}
 	}
@@ -1325,19 +1380,19 @@ function collectHeldFunctions(node, out) {
 * reachable, so it can be layered on top of (and correctly shadow) the outer
 * scope's names.
 */
-function collectFunctionsByName(node, out) {
+function collectFunctionsByName(node, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations) {
 	if (node == null || typeof node !== "object") return;
 	if (Array.isArray(node)) {
-		for (const item of node) collectFunctionsByName(item, out);
+		for (const item of node) collectFunctionsByName(item, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 		return;
 	}
 	if (!isNode(node)) return;
 	const n = node;
-	const bindTo = (names, from) => {
+	const bindTo = (names, from, bindings = out) => {
 		const functions = [];
 		collectHeldFunctions(from, functions);
 		if (functions.length < 1) return;
-		for (const name of names) out.set(name, [...out.get(name) ?? [], ...functions]);
+		for (const name of names) bindings.set(name, [...bindings.get(name) ?? [], ...functions]);
 	};
 	if (n.type === "FunctionDeclaration") {
 		if (n.id?.name != null) bindTo([n.id.name], n);
@@ -1348,26 +1403,49 @@ function collectFunctionsByName(node, out) {
 		return;
 	}
 	if (isFunctionLikeNode(n)) return;
+	if (n.type === "StaticBlock") {
+		collectFunctionsByName(n.body, out, assignmentDefaults, declarationPatterns, /* @__PURE__ */ new Set(), scopeDeclarations);
+		return;
+	}
+	if (n.type === "VariableDeclaration") {
+		if (n.kind === "var" || scopeDeclarations.has(n)) for (const decl of n.declarations) {
+			const names = [];
+			collectBoundNames(decl.id, names);
+			for (const name of names) declaredHere.add(name);
+		}
+		for (const decl of n.declarations) collectFunctionsByName(decl, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
+		return;
+	}
 	if (n.type === "VariableDeclarator") {
 		const decl = n;
+		const names = [];
+		collectBoundNames(decl.id, names);
 		if (decl.init != null) {
-			const names = [];
-			collectBoundNames(decl.id, names);
 			bindTo(names, decl.init);
-			collectFunctionsByName(decl.init, out);
+			collectFunctionsByName(decl.init, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 		}
 		return;
 	}
+	if (n.type === "AssignmentPattern") {
+		const names = [];
+		if (n.left.type === "MemberExpression") {
+			const name = getAssignmentTargetName(n.left);
+			if (name != null) names.push(name);
+		} else collectBoundNames(n.left, names);
+		bindTo(names, n.right, n.left.type === "MemberExpression" || !declarationPatterns.has(n) ? assignmentDefaults : out);
+		collectFunctionsByName(n.right, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
+		return;
+	}
 	if (n.type === "AssignmentExpression") {
 		const name = getAssignmentTargetName(n.left);
 		if (name != null) bindTo([name], n.right);
-		collectFunctionsByName(n.right, out);
+		collectFunctionsByName(n.right, out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 		return;
 	}
 	const record = n;
 	for (const key in record) {
 		if (key === "parent") continue;
-		collectFunctionsByName(record[key], out);
+		collectFunctionsByName(record[key], out, assignmentDefaults, declarationPatterns, declaredHere, scopeDeclarations);
 	}
 }
 /**
@@ -1410,11 +1488,27 @@ function walkUsedScopes(root, visit) {
 		if (visited.has(scopeRoot)) return;
 		visited.add(scopeRoot);
 		const statements = [];
-		collectReachableStatements(scopeRoot, statements);
+		const declarationPatterns = /* @__PURE__ */ new Set();
+		const declaredHere = /* @__PURE__ */ new Set();
+		collectReachableStatements(scopeRoot, statements, {
+			declarationPatterns,
+			declaredHere
+		});
 		const functionsHere = /* @__PURE__ */ new Map();
-		for (const statement of statements) collectFunctionsByName(statement, functionsHere);
+		const assignmentDefaultsHere = /* @__PURE__ */ new Map();
+		for (const param of scopeFn?.params ?? []) {
+			const names = [];
+			collectBoundNames(param, names);
+			for (const name of names) functionsHere.set(name, []);
+		}
+		const scopeDeclarations = new Set(scopeRoot.type === "BlockStatement" ? scopeRoot.body : []);
+		for (const statement of statements) collectFunctionsByName(statement, functionsHere, assignmentDefaultsHere, declarationPatterns, declaredHere, scopeDeclarations);
 		const functionsByName = new Map(outerFunctionsByName);
 		for (const [name, functions] of functionsHere) functionsByName.set(name, functions);
+		for (const [name, functions] of assignmentDefaultsHere) {
+			const base = declaredHere.has(name) ? functionsHere.get(name) ?? [] : functionsByName.get(name) ?? [];
+			functionsByName.set(name, [...base, ...functions]);
+		}
 		visit({
 			fn: scopeFn,
 			statements,
@@ -1428,7 +1522,7 @@ function walkUsedScopes(root, visit) {
 			for (const fn of functions) reached.add(fn);
 		}
 		const held = /* @__PURE__ */ new Set();
-		for (const functions of functionsHere.values()) for (const fn of functions) held.add(fn);
+		for (const bindings of [functionsHere, assignmentDefaultsHere]) for (const functions of bindings.values()) for (const fn of functions) held.add(fn);
 		for (const statement of statements) {
 			const nested = [];
 			collectNestedFunctions(statement, nested);
@@ -1670,7 +1764,11 @@ function fateOf(expression, analysis, shape = "promise") {
 			case "DoWhileStatement":
 			case "ForStatement": return DROPPED;
 			case "LogicalExpression":
+				if (get(parent, "operator") === "&&" && get(parent, "left") === current) return DROPPED;
+				current = parent;
+				continue;
 			case "Property":
+				if (get(parent, "computed") && get(parent, "key") === current) return DROPPED;
 				current = parent;
 				continue;
 			case "ObjectExpression":
@@ -1689,6 +1787,7 @@ function fateOf(expression, analysis, shape = "promise") {
 				continue;
 			}
 			case "MemberExpression": {
+				if (get(parent, "computed") && get(parent, "property") === current) return DROPPED;
 				if (get(parent, "object") !== current) return HANDLED;
 				const call = parentOf(parent);
 				const name = memberName(parent);
@@ -1722,6 +1821,12 @@ function fateOf(expression, analysis, shape = "promise") {
 				if (name == null) return HANDLED;
 				return analysis.mentioned.has(name) ? HANDLED : DROPPED;
 			}
+			case "AssignmentPattern": {
+				if (get(parent, "right") !== current) return HANDLED;
+				const name = getAssignmentTargetName(asNode(get(parent, "left")) ?? parent);
+				if (name == null) return HANDLED;
+				return analysis.mentioned.has(name) ? HANDLED : DROPPED;
+			}
 			default: return HANDLED;
 		}
 	}

package/dist/oxlint.cjs

@@ -1,4 +1,4 @@
-const require_outbox_listener_delivery_required = require("./outbox-listener-delivery-required-DyJ1I2D0.cjs");
+const require_outbox_listener_delivery_required = require("./outbox-listener-delivery-required-ldftuG5P.cjs");
 //#region src/oxlint.ts
 const rules = {
 	[require_outbox_listener_delivery_required.RULE_IDS.actorIdMismatch]: require_outbox_listener_delivery_required.eslint$18,

package/dist/oxlint.js

@@ -1,4 +1,4 @@
-import { A as name, C as eslint$5, D as eslint$11, E as eslint$10, O as eslint$21, S as eslint$4, T as eslint$13, _ as eslint$15, a as eslint$23, b as eslint$2, c as eslint$16, d as eslint$22, f as eslint$6, g as eslint$14, h as eslint$9, i as eslint$27, j as version, k as RULE_IDS, l as eslint$17, m as eslint$8, n as eslint$25, o as eslint$18, p as eslint$7, r as eslint$26, s as eslint$19, t as eslint$24, u as eslint$20, v as eslint$1, w as eslint$12, x as eslint$3, y as eslint } from "./outbox-listener-delivery-required-6phcPuk2.js";
+import { A as name, C as eslint$5, D as eslint$11, E as eslint$10, O as eslint$21, S as eslint$4, T as eslint$13, _ as eslint$15, a as eslint$23, b as eslint$2, c as eslint$16, d as eslint$22, f as eslint$6, g as eslint$14, h as eslint$9, i as eslint$27, j as version, k as RULE_IDS, l as eslint$17, m as eslint$8, n as eslint$25, o as eslint$18, p as eslint$7, r as eslint$26, s as eslint$19, t as eslint$24, u as eslint$20, v as eslint$1, w as eslint$12, x as eslint$3, y as eslint } from "./outbox-listener-delivery-required-C_ixBt5W.js";
 //#region src/oxlint.ts
 const rules = {
 	[RULE_IDS.actorIdMismatch]: eslint,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fedify/lint",
-  "version": "2.4.0-dev.2122+bdc0ce8d",
+  "version": "2.4.0-dev.2127+c41ce2eb",
   "description": "Fedify linting rules and plugins",
   "keywords": [
     "Fedify",
@@ -51,7 +51,7 @@
   ],
   "peerDependencies": {
     "eslint": ">=9.0.0",
-    "@fedify/fedify": "^2.4.0-dev.2122+bdc0ce8d"
+    "@fedify/fedify": "^2.4.0-dev.2127+c41ce2eb"
   },
   "peerDependenciesMeta": {
     "eslint": {
@@ -71,7 +71,7 @@
     "tsdown": "^0.22.0",
     "typescript": "^6.0.0",
     "@fedify/fixture": "2.0.0",
-    "@fedify/vocab": "2.4.0-dev.2122+bdc0ce8d"
+    "@fedify/vocab": "2.4.0-dev.2127+c41ce2eb"
   },
   "scripts": {
     "build:self": "tsdown",