@fedify/lint 2.3.9-dev.2048 → 2.3.10

package/deno.json

@@ -1,12 +1,13 @@
 {
   "name": "@fedify/lint",
-  "version": "2.3.9-dev.2048+c88fcc8c",
+  "version": "2.3.10",
   "license": "MIT",
   "exports": {
     ".": "./src/mod.ts",
     "./oxlint": "./src/oxlint.ts"
   },
   "imports": {
+    "@fedify/fixture": "jsr:@fedify/fixture@^2.2.14",
     "eslint": "npm:eslint@^9.0.0",
     "estree": "npm:@types/estree@^1.0.8",
     "oxlint": "npm:oxlint@^1.66.0"

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-C1Ho3dCQ.cjs");
+const require_outbox_listener_delivery_required = require("./outbox-listener-delivery-required-CwToC306.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 { C as RULE_IDS, S as eslint$19, T as version, _ as eslint$5, a as eslint$18, b as eslint$10, c as eslint$8, d as eslint$15, f as eslint$1, g as eslint$4, h as eslint$3, i as eslint$17, l as eslint$9, m as eslint$2, n as eslint$20, o as eslint$6, p as eslint, r as eslint$16, s as eslint$7, t as eslint$21, u as eslint$14, v as eslint$12, w as name, x as eslint$11, y as eslint$13 } from "./outbox-listener-delivery-required-CxugrN84.js";
+import { C as RULE_IDS, S as eslint$19, T as version, _ as eslint$5, a as eslint$18, b as eslint$10, c as eslint$8, d as eslint$15, f as eslint$1, g as eslint$4, h as eslint$3, i as eslint$17, l as eslint$9, m as eslint$2, n as eslint$20, o as eslint$6, p as eslint, r as eslint$16, s as eslint$7, t as eslint$21, u as eslint$14, v as eslint$12, w as name, x as eslint$11, y as eslint$13 } from "./outbox-listener-delivery-required-BNgI_uik.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-CxugrN84.js → outbox-listener-delivery-required-BNgI_uik.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.3.9-dev.2048+c88fcc8c";
+var version = "2.3.10";
 //#endregion
 //#region src/lib/const.ts
 /**
@@ -595,7 +595,7 @@ const DELIVERY_METHOD_NAMES = new Set(["sendActivity", "forwardActivity"]);
 const getMemberPropertyName = (expr) => {
 	if (expr.type !== "MemberExpression") return null;
 	const property = expr.property;
-	if (property.type === "Identifier") return property.name;
+	if (property.type === "Identifier" && !expr.computed) return property.name;
 	if (property.type === "Literal" && typeof property.value === "string") return property.value;
 	return null;
 };
@@ -731,35 +731,211 @@ function buildContextExpressionPattern(contextName) {
 	const boundedName = String.raw`(?<![\w$])${name}(?![\w$])`;
 	return String.raw`(?:${boundedName}|\(\s*${boundedName}(?:\s+as\s+[^)]+)?\s*\))`;
 }
-const resolveListenerReference = (expr, bindings, seen = /* @__PURE__ */ new Set()) => {
-	if (isFunction(expr)) return expr;
-	if (expr.type === "Identifier") {
-		if (seen.has(expr.name)) return null;
-		seen.add(expr.name);
-		const binding = bindings.get(expr.name);
-		if (binding == null || !isNode(binding)) return null;
-		if (isFunction(binding) || binding.type === "FunctionDeclaration") return binding;
-		if (binding.type === "Identifier") return resolveListenerReference(binding, bindings, seen);
-		return null;
+const getParent = (node) => node.parent ?? null;
+const isFunctionLike = (node) => node.type === "FunctionDeclaration" || isFunction(node);
+function bindPattern(node, bindings) {
+	switch (node.type) {
+		case "Identifier":
+			bindings.set(node.name, { value: null });
+			break;
+		case "AssignmentPattern":
+			bindPattern(node.left, bindings);
+			break;
+		case "RestElement":
+			bindPattern(node.argument, bindings);
+			break;
+		case "ObjectPattern":
+			for (const property of node.properties) bindPattern(property.type === "Property" ? property.value : property, bindings);
+			break;
+		case "ArrayPattern":
+			for (const element of node.elements) if (element != null) bindPattern(element, bindings);
+			break;
 	}
-	if (expr.type === "MemberExpression" && expr.object.type === "Identifier" && !expr.computed) {
-		const binding = bindings.get(expr.object.name);
-		if (binding == null || !isNode(binding) || binding.type !== "ObjectExpression") return null;
-		const propertyName = getMemberPropertyName(expr);
-		if (propertyName == null) return null;
-		for (const prop of binding.properties) {
-			if (!isNode(prop) || prop.type !== "Property") continue;
-			if ((prop.key.type === "Identifier" ? prop.key.name : prop.key.type === "Literal" && typeof prop.key.value === "string" ? prop.key.value : null) !== propertyName || !isNode(prop.value)) continue;
-			const value = prop.value;
-			if (isFunction(value) || value.type === "FunctionDeclaration") return value;
+}
+function createBindingIndex() {
+	const scopes = /* @__PURE__ */ new Map();
+	const scopeBindings = (scope) => {
+		let bindings = scopes.get(scope);
+		if (bindings == null) {
+			bindings = /* @__PURE__ */ new Map();
+			scopes.set(scope, bindings);
+			if (isFunctionLike(scope)) {
+				if (scope.type === "FunctionExpression" && scope.id != null) bindings.set(scope.id.name, { value: scope });
+				for (const param of scope.params) bindPattern(param, bindings);
+			}
+			if (scope.type === "ClassExpression" && scope.id != null) bindings.set(scope.id.name, { value: null });
+			if (scope.type === "CatchClause" && scope.param != null) bindPattern(scope.param, bindings);
+		}
+		return bindings;
+	};
+	const enclosingScope = (node, functionScope = false) => {
+		let scope = getParent(node);
+		while (scope.type !== "Program" && scope.type !== "StaticBlock" && scope.type !== "TSModuleBlock" && !isFunctionLike(scope) && (functionScope || ![
+			"BlockStatement",
+			"ForStatement",
+			"ForInStatement",
+			"ForOfStatement",
+			"SwitchStatement",
+			"CatchClause"
+		].includes(scope.type))) scope = getParent(scope);
+		return scope;
+	};
+	const lookup = (node, name) => {
+		for (let scope = node; scope != null; scope = getParent(scope)) {
+			const binding = scopeBindings(scope).get(name);
+			if (binding != null) return binding;
 		}
+		return null;
+	};
+	return {
+		lookup,
+		declare(node, pattern, value, hoisted = false) {
+			const bindings = scopeBindings(enclosingScope(node, hoisted));
+			if (pattern.type === "Identifier") {
+				if (hoisted && value == null && bindings.has(pattern.name)) return;
+				if (node.type === "FunctionDeclaration" && bindings.get(pattern.name)?.initializedVariable) return;
+				bindings.set(pattern.name, {
+					value,
+					initializedVariable: node.type === "VariableDeclarator" && value != null
+				});
+			} else bindPattern(pattern, bindings);
+		}
+	};
+}
+const position = (node) => node.range?.[0] ?? node.start ?? -1;
+const endPosition = (node) => node.range?.[1] ?? node.end ?? -1;
+const enclosingFunction = (node) => {
+	for (let parent = getParent(node); parent != null; parent = getParent(parent)) if (isFunctionLike(parent)) return parent;
+	return null;
+};
+const isInside = (node, ancestor) => {
+	for (let current = node; current != null; current = getParent(current)) if (current === ancestor) return true;
+	return false;
+};
+const isDirectStatement = (node, fn) => {
+	let current = node;
+	while (getParent(current) !== fn.body) {
+		const parent = getParent(current);
+		if (parent == null || ![
+			"AwaitExpression",
+			"ExpressionStatement",
+			"ReturnStatement",
+			"VariableDeclarator",
+			"VariableDeclaration",
+			"BlockStatement"
+		].includes(parent.type)) return false;
+		current = parent;
+	}
+	return [
+		"ExpressionStatement",
+		"ReturnStatement",
+		"VariableDeclaration",
+		"BlockStatement"
+	].includes(current.type);
+};
+const resolveBindingValue = (expr, bindings, seen = /* @__PURE__ */ new Set()) => {
+	if (expr.type !== "Identifier") return expr;
+	const binding = bindings.lookup(expr, expr.name);
+	if (binding == null || binding.value == null || seen.has(binding)) return null;
+	seen.add(binding);
+	return resolveBindingValue(binding.value, bindings, seen);
+};
+const resolveListenerReference = (expr, bindings) => {
+	const target = resolveBindingValue(expr, bindings);
+	if (target == null) return null;
+	if (isFunctionLike(target)) return target;
+	if (target.type !== "MemberExpression") return null;
+	const object = resolveBindingValue(target.object, bindings);
+	if (object?.type !== "ObjectExpression") return null;
+	const propertyName = getMemberPropertyName(target);
+	if (propertyName == null) return null;
+	for (const prop of object.properties) {
+		if (prop.type !== "Property") continue;
+		if ((prop.key.type === "Identifier" && !prop.computed ? prop.key.name : prop.key.type === "Literal" && typeof prop.key.value === "string" ? prop.key.value : null) !== propertyName) continue;
+		const value = resolveBindingValue(prop.value, bindings);
+		if (value != null && isFunctionLike(value)) return value;
 	}
 	return null;
 };
-const listenerCallsDeliveryMethod = (sourceCode, listener) => {
-	const code = stripCommentsAndStrings(sourceCode.getText(listener));
+function unwrapArgument(node) {
+	while (node.type === "TSAsExpression" || node.type === "TSTypeAssertion" || node.type === "TSNonNullExpression") node = node.expression;
+	return node;
+}
+function ignoredLocalSetupFunctions(listener, bindings, calls, assignments) {
+	const ignored = [];
+	const directCalls = calls.filter((call) => enclosingFunction(call) === listener && isDirectStatement(call, listener));
+	for (const assignment of assignments) {
+		if (assignment.type !== "AssignmentExpression" || assignment.operator !== "=" || assignment.left.type !== "MemberExpression" || !isFunction(assignment.right)) continue;
+		const setup = enclosingFunction(assignment);
+		if (setup == null || setup === listener || !isInside(setup, listener)) continue;
+		const setupParent = getParent(setup);
+		const namedLocalSetup = enclosingFunction(setup) === listener && (setup.type === "FunctionDeclaration" || setupParent?.type === "VariableDeclarator" && setupParent.init === setup && setupParent.id.type === "Identifier");
+		const passedToCall = calls.some((call) => call.arguments.some((argument) => argument.type !== "SpreadElement" && resolveListenerReference(argument, bindings) === setup));
+		if (!namedLocalSetup || passedToCall) continue;
+		const installed = assignment.right;
+		if (!isDirectStatement(assignment, setup) || assignment.left.object.type !== "Identifier") {
+			ignored.push(installed);
+			continue;
+		}
+		const property = getMemberPropertyName(assignment.left);
+		if (property == null) continue;
+		const setupCalls = directCalls.filter((call) => resolveListenerReference(call.callee, bindings) === setup);
+		if (setupCalls.length === 0) {
+			const called = calls.some((call) => resolveListenerReference(call.callee, bindings) === setup);
+			ignored.push(called ? assignment.right : setup);
+			continue;
+		}
+		const objectBinding = bindings.lookup(assignment.left.object, assignment.left.object.name);
+		const deliveryCalls = directCalls.filter((call) => {
+			const callee = call.callee;
+			return callee.type === "MemberExpression" && callee.object.type === "Identifier" && getMemberPropertyName(callee) === property && bindings.lookup(callee.object, callee.object.name) === objectBinding;
+		});
+		if (!(objectBinding != null && setupCalls.some((setupCall) => deliveryCalls.some((deliveryCall) => position(setupCall) >= 0 && position(setupCall) < position(deliveryCall))))) ignored.push(installed);
+	}
+	return ignored;
+}
+function functionCallsDelivery(sourceCode, listener, bindings, calls, assignments, contextIndex = 0, visited = /* @__PURE__ */ new Map()) {
+	let indices = visited.get(listener);
+	if (indices?.has(contextIndex)) return false;
+	if (indices == null) visited.set(listener, indices = /* @__PURE__ */ new Set());
+	indices.add(contextIndex);
+	if (listenerCallsDeliveryMethod(sourceCode, listener, contextIndex, ignoredLocalSetupFunctions(listener, bindings, calls, assignments))) return true;
+	const param = unwrapContextParam(listener.params[contextIndex]);
+	if (param?.type !== "Identifier") return false;
+	const contextBinding = bindings.lookup(listener, param.name);
+	for (const call of calls) {
+		let child = call;
+		let owner = getParent(child);
+		while (owner != null && !isFunctionLike(owner)) {
+			const field = owner;
+			if ((field.type === "PropertyDefinition" || field.type === "AccessorProperty") && !field.static && field.value === child) break;
+			child = owner;
+			owner = getParent(owner);
+		}
+		if (owner !== listener) continue;
+		const helper = resolveListenerReference(call.callee, bindings);
+		if (helper == null || "generator" in helper && helper.generator) continue;
+		for (const [index, argument] of call.arguments.entries()) {
+			if (argument.type === "SpreadElement") break;
+			const arg = unwrapArgument(argument);
+			if (arg.type !== "Identifier" || arg.name !== param.name || bindings.lookup(arg, arg.name) !== contextBinding || helper.params[index]?.type === "RestElement") continue;
+			if (functionCallsDelivery(sourceCode, helper, bindings, calls, assignments, index, visited)) return true;
+		}
+	}
+	return false;
+}
+const listenerCallsDeliveryMethod = (sourceCode, listener, contextIndex = 0, ignored = []) => {
+	let text = sourceCode.getText(listener);
+	const start = position(listener);
+	for (const fn of ignored) {
+		const from = position(fn) - start;
+		const to = endPosition(fn) - start;
+		if (from < 0 || to > text.length || to < from) continue;
+		text = text.slice(0, from) + " ".repeat(to - from) + text.slice(to);
+	}
+	const code = stripCommentsAndStrings(text);
 	const aliases = /* @__PURE__ */ new Set();
-	const contextParam = unwrapContextParam(listener.params[0]);
+	const contextParam = unwrapContextParam(listener.params[contextIndex]);
 	const contextName = contextParam?.type === "Identifier" ? contextParam.name : null;
 	if (contextParam?.type === "ObjectPattern") for (const prop of contextParam.properties) {
 		if (!isNode(prop) || prop.type !== "Property") continue;
@@ -788,8 +964,9 @@ const listenerCallsDeliveryMethod = (sourceCode, listener) => {
 function createRule(buildReport) {
 	return (context) => {
 		const federationTracker = trackFederationVariables();
-		const bindings = /* @__PURE__ */ new Map();
+		const bindings = createBindingIndex();
 		const pendingCalls = [];
+		const pendingAssignments = [];
 		const sourceCode = context.sourceCode;
 		const inspectCall = (node) => {
 			if (!hasMemberExpressionCallee(node) || !hasIdentifierProperty(node) || !hasMethodName("on")(node) || node.arguments.length < 2) return;
@@ -797,7 +974,7 @@ function createRule(buildReport) {
 			const listener = node.arguments[1];
 			const resolvedListener = isNode(listener) && isFunction(listener) ? listener : isNode(listener) ? resolveListenerReference(listener, bindings) : null;
 			if (resolvedListener == null) return;
-			if (listenerCallsDeliveryMethod(sourceCode, resolvedListener)) return;
+			if (functionCallsDelivery(sourceCode, resolvedListener, bindings, pendingCalls, pendingAssignments)) return;
 			context.report({
 				node: resolvedListener,
 				...buildReport
@@ -806,14 +983,28 @@ function createRule(buildReport) {
 		return {
 			VariableDeclarator(node) {
 				federationTracker.VariableDeclarator(node);
-				if (node.id.type === "Identifier" && node.init != null) bindings.set(node.id.name, node.init);
+				const declaration = getParent(node);
+				bindings.declare(node, node.id, node.init, declaration?.kind === "var");
 			},
 			FunctionDeclaration(node) {
-				if (node.id != null) bindings.set(node.id.name, node);
+				if (node.id == null) return;
+				const parent = getParent(node);
+				const owner = parent == null ? null : getParent(parent);
+				const inVarBody = parent?.type === "BlockStatement" && owner != null && (owner.type === "StaticBlock" || isFunctionLike(owner) && owner.body === parent);
+				bindings.declare(node, node.id, node, inVarBody);
+			},
+			ImportDeclaration(node) {
+				for (const specifier of node.specifiers) bindings.declare(node, specifier.local, null);
+			},
+			ClassDeclaration(node) {
+				if (node.id != null) bindings.declare(node, node.id, null);
 			},
 			CallExpression(node) {
 				pendingCalls.push(node);
 			},
+			AssignmentExpression(node) {
+				pendingAssignments.push(node);
+			},
 			"Program:exit"() {
 				for (const node of pendingCalls) inspectCall(node);
 			}

package/dist/{outbox-listener-delivery-required-C1Ho3dCQ.cjs → outbox-listener-delivery-required-CwToC306.cjs}

@@ -2,7 +2,7 @@ require("./index.cjs");
 let _fxts_core = require("@fxts/core");
 //#region deno.json
 var name = "@fedify/lint";
-var version = "2.3.9-dev.2048+c88fcc8c";
+var version = "2.3.10";
 //#endregion
 //#region src/lib/const.ts
 /**
@@ -596,7 +596,7 @@ const DELIVERY_METHOD_NAMES = new Set(["sendActivity", "forwardActivity"]);
 const getMemberPropertyName = (expr) => {
 	if (expr.type !== "MemberExpression") return null;
 	const property = expr.property;
-	if (property.type === "Identifier") return property.name;
+	if (property.type === "Identifier" && !expr.computed) return property.name;
 	if (property.type === "Literal" && typeof property.value === "string") return property.value;
 	return null;
 };
@@ -732,35 +732,211 @@ function buildContextExpressionPattern(contextName) {
 	const boundedName = String.raw`(?<![\w$])${name}(?![\w$])`;
 	return String.raw`(?:${boundedName}|\(\s*${boundedName}(?:\s+as\s+[^)]+)?\s*\))`;
 }
-const resolveListenerReference = (expr, bindings, seen = /* @__PURE__ */ new Set()) => {
-	if (isFunction(expr)) return expr;
-	if (expr.type === "Identifier") {
-		if (seen.has(expr.name)) return null;
-		seen.add(expr.name);
-		const binding = bindings.get(expr.name);
-		if (binding == null || !isNode(binding)) return null;
-		if (isFunction(binding) || binding.type === "FunctionDeclaration") return binding;
-		if (binding.type === "Identifier") return resolveListenerReference(binding, bindings, seen);
-		return null;
+const getParent = (node) => node.parent ?? null;
+const isFunctionLike = (node) => node.type === "FunctionDeclaration" || isFunction(node);
+function bindPattern(node, bindings) {
+	switch (node.type) {
+		case "Identifier":
+			bindings.set(node.name, { value: null });
+			break;
+		case "AssignmentPattern":
+			bindPattern(node.left, bindings);
+			break;
+		case "RestElement":
+			bindPattern(node.argument, bindings);
+			break;
+		case "ObjectPattern":
+			for (const property of node.properties) bindPattern(property.type === "Property" ? property.value : property, bindings);
+			break;
+		case "ArrayPattern":
+			for (const element of node.elements) if (element != null) bindPattern(element, bindings);
+			break;
 	}
-	if (expr.type === "MemberExpression" && expr.object.type === "Identifier" && !expr.computed) {
-		const binding = bindings.get(expr.object.name);
-		if (binding == null || !isNode(binding) || binding.type !== "ObjectExpression") return null;
-		const propertyName = getMemberPropertyName(expr);
-		if (propertyName == null) return null;
-		for (const prop of binding.properties) {
-			if (!isNode(prop) || prop.type !== "Property") continue;
-			if ((prop.key.type === "Identifier" ? prop.key.name : prop.key.type === "Literal" && typeof prop.key.value === "string" ? prop.key.value : null) !== propertyName || !isNode(prop.value)) continue;
-			const value = prop.value;
-			if (isFunction(value) || value.type === "FunctionDeclaration") return value;
+}
+function createBindingIndex() {
+	const scopes = /* @__PURE__ */ new Map();
+	const scopeBindings = (scope) => {
+		let bindings = scopes.get(scope);
+		if (bindings == null) {
+			bindings = /* @__PURE__ */ new Map();
+			scopes.set(scope, bindings);
+			if (isFunctionLike(scope)) {
+				if (scope.type === "FunctionExpression" && scope.id != null) bindings.set(scope.id.name, { value: scope });
+				for (const param of scope.params) bindPattern(param, bindings);
+			}
+			if (scope.type === "ClassExpression" && scope.id != null) bindings.set(scope.id.name, { value: null });
+			if (scope.type === "CatchClause" && scope.param != null) bindPattern(scope.param, bindings);
+		}
+		return bindings;
+	};
+	const enclosingScope = (node, functionScope = false) => {
+		let scope = getParent(node);
+		while (scope.type !== "Program" && scope.type !== "StaticBlock" && scope.type !== "TSModuleBlock" && !isFunctionLike(scope) && (functionScope || ![
+			"BlockStatement",
+			"ForStatement",
+			"ForInStatement",
+			"ForOfStatement",
+			"SwitchStatement",
+			"CatchClause"
+		].includes(scope.type))) scope = getParent(scope);
+		return scope;
+	};
+	const lookup = (node, name) => {
+		for (let scope = node; scope != null; scope = getParent(scope)) {
+			const binding = scopeBindings(scope).get(name);
+			if (binding != null) return binding;
 		}
+		return null;
+	};
+	return {
+		lookup,
+		declare(node, pattern, value, hoisted = false) {
+			const bindings = scopeBindings(enclosingScope(node, hoisted));
+			if (pattern.type === "Identifier") {
+				if (hoisted && value == null && bindings.has(pattern.name)) return;
+				if (node.type === "FunctionDeclaration" && bindings.get(pattern.name)?.initializedVariable) return;
+				bindings.set(pattern.name, {
+					value,
+					initializedVariable: node.type === "VariableDeclarator" && value != null
+				});
+			} else bindPattern(pattern, bindings);
+		}
+	};
+}
+const position = (node) => node.range?.[0] ?? node.start ?? -1;
+const endPosition = (node) => node.range?.[1] ?? node.end ?? -1;
+const enclosingFunction = (node) => {
+	for (let parent = getParent(node); parent != null; parent = getParent(parent)) if (isFunctionLike(parent)) return parent;
+	return null;
+};
+const isInside = (node, ancestor) => {
+	for (let current = node; current != null; current = getParent(current)) if (current === ancestor) return true;
+	return false;
+};
+const isDirectStatement = (node, fn) => {
+	let current = node;
+	while (getParent(current) !== fn.body) {
+		const parent = getParent(current);
+		if (parent == null || ![
+			"AwaitExpression",
+			"ExpressionStatement",
+			"ReturnStatement",
+			"VariableDeclarator",
+			"VariableDeclaration",
+			"BlockStatement"
+		].includes(parent.type)) return false;
+		current = parent;
+	}
+	return [
+		"ExpressionStatement",
+		"ReturnStatement",
+		"VariableDeclaration",
+		"BlockStatement"
+	].includes(current.type);
+};
+const resolveBindingValue = (expr, bindings, seen = /* @__PURE__ */ new Set()) => {
+	if (expr.type !== "Identifier") return expr;
+	const binding = bindings.lookup(expr, expr.name);
+	if (binding == null || binding.value == null || seen.has(binding)) return null;
+	seen.add(binding);
+	return resolveBindingValue(binding.value, bindings, seen);
+};
+const resolveListenerReference = (expr, bindings) => {
+	const target = resolveBindingValue(expr, bindings);
+	if (target == null) return null;
+	if (isFunctionLike(target)) return target;
+	if (target.type !== "MemberExpression") return null;
+	const object = resolveBindingValue(target.object, bindings);
+	if (object?.type !== "ObjectExpression") return null;
+	const propertyName = getMemberPropertyName(target);
+	if (propertyName == null) return null;
+	for (const prop of object.properties) {
+		if (prop.type !== "Property") continue;
+		if ((prop.key.type === "Identifier" && !prop.computed ? prop.key.name : prop.key.type === "Literal" && typeof prop.key.value === "string" ? prop.key.value : null) !== propertyName) continue;
+		const value = resolveBindingValue(prop.value, bindings);
+		if (value != null && isFunctionLike(value)) return value;
 	}
 	return null;
 };
-const listenerCallsDeliveryMethod = (sourceCode, listener) => {
-	const code = stripCommentsAndStrings(sourceCode.getText(listener));
+function unwrapArgument(node) {
+	while (node.type === "TSAsExpression" || node.type === "TSTypeAssertion" || node.type === "TSNonNullExpression") node = node.expression;
+	return node;
+}
+function ignoredLocalSetupFunctions(listener, bindings, calls, assignments) {
+	const ignored = [];
+	const directCalls = calls.filter((call) => enclosingFunction(call) === listener && isDirectStatement(call, listener));
+	for (const assignment of assignments) {
+		if (assignment.type !== "AssignmentExpression" || assignment.operator !== "=" || assignment.left.type !== "MemberExpression" || !isFunction(assignment.right)) continue;
+		const setup = enclosingFunction(assignment);
+		if (setup == null || setup === listener || !isInside(setup, listener)) continue;
+		const setupParent = getParent(setup);
+		const namedLocalSetup = enclosingFunction(setup) === listener && (setup.type === "FunctionDeclaration" || setupParent?.type === "VariableDeclarator" && setupParent.init === setup && setupParent.id.type === "Identifier");
+		const passedToCall = calls.some((call) => call.arguments.some((argument) => argument.type !== "SpreadElement" && resolveListenerReference(argument, bindings) === setup));
+		if (!namedLocalSetup || passedToCall) continue;
+		const installed = assignment.right;
+		if (!isDirectStatement(assignment, setup) || assignment.left.object.type !== "Identifier") {
+			ignored.push(installed);
+			continue;
+		}
+		const property = getMemberPropertyName(assignment.left);
+		if (property == null) continue;
+		const setupCalls = directCalls.filter((call) => resolveListenerReference(call.callee, bindings) === setup);
+		if (setupCalls.length === 0) {
+			const called = calls.some((call) => resolveListenerReference(call.callee, bindings) === setup);
+			ignored.push(called ? assignment.right : setup);
+			continue;
+		}
+		const objectBinding = bindings.lookup(assignment.left.object, assignment.left.object.name);
+		const deliveryCalls = directCalls.filter((call) => {
+			const callee = call.callee;
+			return callee.type === "MemberExpression" && callee.object.type === "Identifier" && getMemberPropertyName(callee) === property && bindings.lookup(callee.object, callee.object.name) === objectBinding;
+		});
+		if (!(objectBinding != null && setupCalls.some((setupCall) => deliveryCalls.some((deliveryCall) => position(setupCall) >= 0 && position(setupCall) < position(deliveryCall))))) ignored.push(installed);
+	}
+	return ignored;
+}
+function functionCallsDelivery(sourceCode, listener, bindings, calls, assignments, contextIndex = 0, visited = /* @__PURE__ */ new Map()) {
+	let indices = visited.get(listener);
+	if (indices?.has(contextIndex)) return false;
+	if (indices == null) visited.set(listener, indices = /* @__PURE__ */ new Set());
+	indices.add(contextIndex);
+	if (listenerCallsDeliveryMethod(sourceCode, listener, contextIndex, ignoredLocalSetupFunctions(listener, bindings, calls, assignments))) return true;
+	const param = unwrapContextParam(listener.params[contextIndex]);
+	if (param?.type !== "Identifier") return false;
+	const contextBinding = bindings.lookup(listener, param.name);
+	for (const call of calls) {
+		let child = call;
+		let owner = getParent(child);
+		while (owner != null && !isFunctionLike(owner)) {
+			const field = owner;
+			if ((field.type === "PropertyDefinition" || field.type === "AccessorProperty") && !field.static && field.value === child) break;
+			child = owner;
+			owner = getParent(owner);
+		}
+		if (owner !== listener) continue;
+		const helper = resolveListenerReference(call.callee, bindings);
+		if (helper == null || "generator" in helper && helper.generator) continue;
+		for (const [index, argument] of call.arguments.entries()) {
+			if (argument.type === "SpreadElement") break;
+			const arg = unwrapArgument(argument);
+			if (arg.type !== "Identifier" || arg.name !== param.name || bindings.lookup(arg, arg.name) !== contextBinding || helper.params[index]?.type === "RestElement") continue;
+			if (functionCallsDelivery(sourceCode, helper, bindings, calls, assignments, index, visited)) return true;
+		}
+	}
+	return false;
+}
+const listenerCallsDeliveryMethod = (sourceCode, listener, contextIndex = 0, ignored = []) => {
+	let text = sourceCode.getText(listener);
+	const start = position(listener);
+	for (const fn of ignored) {
+		const from = position(fn) - start;
+		const to = endPosition(fn) - start;
+		if (from < 0 || to > text.length || to < from) continue;
+		text = text.slice(0, from) + " ".repeat(to - from) + text.slice(to);
+	}
+	const code = stripCommentsAndStrings(text);
 	const aliases = /* @__PURE__ */ new Set();
-	const contextParam = unwrapContextParam(listener.params[0]);
+	const contextParam = unwrapContextParam(listener.params[contextIndex]);
 	const contextName = contextParam?.type === "Identifier" ? contextParam.name : null;
 	if (contextParam?.type === "ObjectPattern") for (const prop of contextParam.properties) {
 		if (!isNode(prop) || prop.type !== "Property") continue;
@@ -789,8 +965,9 @@ const listenerCallsDeliveryMethod = (sourceCode, listener) => {
 function createRule(buildReport) {
 	return (context) => {
 		const federationTracker = trackFederationVariables();
-		const bindings = /* @__PURE__ */ new Map();
+		const bindings = createBindingIndex();
 		const pendingCalls = [];
+		const pendingAssignments = [];
 		const sourceCode = context.sourceCode;
 		const inspectCall = (node) => {
 			if (!hasMemberExpressionCallee(node) || !hasIdentifierProperty(node) || !hasMethodName("on")(node) || node.arguments.length < 2) return;
@@ -798,7 +975,7 @@ function createRule(buildReport) {
 			const listener = node.arguments[1];
 			const resolvedListener = isNode(listener) && isFunction(listener) ? listener : isNode(listener) ? resolveListenerReference(listener, bindings) : null;
 			if (resolvedListener == null) return;
-			if (listenerCallsDeliveryMethod(sourceCode, resolvedListener)) return;
+			if (functionCallsDelivery(sourceCode, resolvedListener, bindings, pendingCalls, pendingAssignments)) return;
 			context.report({
 				node: resolvedListener,
 				...buildReport
@@ -807,14 +984,28 @@ function createRule(buildReport) {
 		return {
 			VariableDeclarator(node) {
 				federationTracker.VariableDeclarator(node);
-				if (node.id.type === "Identifier" && node.init != null) bindings.set(node.id.name, node.init);
+				const declaration = getParent(node);
+				bindings.declare(node, node.id, node.init, declaration?.kind === "var");
 			},
 			FunctionDeclaration(node) {
-				if (node.id != null) bindings.set(node.id.name, node);
+				if (node.id == null) return;
+				const parent = getParent(node);
+				const owner = parent == null ? null : getParent(parent);
+				const inVarBody = parent?.type === "BlockStatement" && owner != null && (owner.type === "StaticBlock" || isFunctionLike(owner) && owner.body === parent);
+				bindings.declare(node, node.id, node, inVarBody);
+			},
+			ImportDeclaration(node) {
+				for (const specifier of node.specifiers) bindings.declare(node, specifier.local, null);
+			},
+			ClassDeclaration(node) {
+				if (node.id != null) bindings.declare(node, node.id, null);
 			},
 			CallExpression(node) {
 				pendingCalls.push(node);
 			},
+			AssignmentExpression(node) {
+				pendingAssignments.push(node);
+			},
 			"Program:exit"() {
 				for (const node of pendingCalls) inspectCall(node);
 			}

package/dist/oxlint.cjs

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

package/dist/oxlint.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fedify/lint",
-  "version": "2.3.9-dev.2048+c88fcc8c",
+  "version": "2.3.10",
   "description": "Fedify linting rules and plugins",
   "keywords": [
     "Fedify",
@@ -54,7 +54,7 @@
   ],
   "peerDependencies": {
     "eslint": ">=9.0.0",
-    "@fedify/fedify": "^2.3.9-dev.2048+c88fcc8c"
+    "@fedify/fedify": "^2.3.10"
   },
   "peerDependenciesMeta": {
     "eslint": {