@fedify/lint 2.3.9 → 2.3.11-dev.2280

package/deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@fedify/lint",
-  "version": "2.3.9",
+  "version": "2.3.11-dev.2280+db3f75fc",
   "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-CiBWUvMt.cjs");
+const require_outbox_listener_delivery_required = require("./outbox-listener-delivery-required-Dv8KorpQ.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-DBgDh8Og.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-VJhYGF3C.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-CiBWUvMt.cjs → outbox-listener-delivery-required-Dv8KorpQ.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";
+var version = "2.3.11-dev.2280+db3f75fc";
 //#endregion
 //#region src/lib/const.ts
 /**
@@ -803,6 +803,37 @@ function createBindingIndex() {
 		}
 	};
 }
+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);
@@ -831,12 +862,45 @@ function unwrapArgument(node) {
 	while (node.type === "TSAsExpression" || node.type === "TSTypeAssertion" || node.type === "TSNonNullExpression") node = node.expression;
 	return node;
 }
-function functionCallsDelivery(sourceCode, listener, bindings, calls, contextIndex = 0, visited = /* @__PURE__ */ new Map()) {
+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)) return true;
+	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);
@@ -856,13 +920,21 @@ function functionCallsDelivery(sourceCode, listener, bindings, calls, contextInd
 			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, index, visited)) return true;
+			if (functionCallsDelivery(sourceCode, helper, bindings, calls, assignments, index, visited)) return true;
 		}
 	}
 	return false;
 }
-const listenerCallsDeliveryMethod = (sourceCode, listener, contextIndex = 0) => {
-	const code = stripCommentsAndStrings(sourceCode.getText(listener));
+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[contextIndex]);
 	const contextName = contextParam?.type === "Identifier" ? contextParam.name : null;
@@ -895,6 +967,7 @@ function createRule(buildReport) {
 		const federationTracker = trackFederationVariables();
 		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;
@@ -902,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 (functionCallsDelivery(sourceCode, resolvedListener, bindings, pendingCalls)) return;
+			if (functionCallsDelivery(sourceCode, resolvedListener, bindings, pendingCalls, pendingAssignments)) return;
 			context.report({
 				node: resolvedListener,
 				...buildReport
@@ -930,6 +1003,9 @@ function createRule(buildReport) {
 			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-DBgDh8Og.js → outbox-listener-delivery-required-VJhYGF3C.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";
+var version = "2.3.11-dev.2280+db3f75fc";
 //#endregion
 //#region src/lib/const.ts
 /**
@@ -802,6 +802,37 @@ function createBindingIndex() {
 		}
 	};
 }
+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);
@@ -830,12 +861,45 @@ function unwrapArgument(node) {
 	while (node.type === "TSAsExpression" || node.type === "TSTypeAssertion" || node.type === "TSNonNullExpression") node = node.expression;
 	return node;
 }
-function functionCallsDelivery(sourceCode, listener, bindings, calls, contextIndex = 0, visited = /* @__PURE__ */ new Map()) {
+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)) return true;
+	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);
@@ -855,13 +919,21 @@ function functionCallsDelivery(sourceCode, listener, bindings, calls, contextInd
 			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, index, visited)) return true;
+			if (functionCallsDelivery(sourceCode, helper, bindings, calls, assignments, index, visited)) return true;
 		}
 	}
 	return false;
 }
-const listenerCallsDeliveryMethod = (sourceCode, listener, contextIndex = 0) => {
-	const code = stripCommentsAndStrings(sourceCode.getText(listener));
+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[contextIndex]);
 	const contextName = contextParam?.type === "Identifier" ? contextParam.name : null;
@@ -894,6 +966,7 @@ function createRule(buildReport) {
 		const federationTracker = trackFederationVariables();
 		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;
@@ -901,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 (functionCallsDelivery(sourceCode, resolvedListener, bindings, pendingCalls)) return;
+			if (functionCallsDelivery(sourceCode, resolvedListener, bindings, pendingCalls, pendingAssignments)) return;
 			context.report({
 				node: resolvedListener,
 				...buildReport
@@ -929,6 +1002,9 @@ function createRule(buildReport) {
 			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-CiBWUvMt.cjs");
+const require_outbox_listener_delivery_required = require("./outbox-listener-delivery-required-Dv8KorpQ.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-DBgDh8Og.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-VJhYGF3C.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",
+  "version": "2.3.11-dev.2280+db3f75fc",
   "description": "Fedify linting rules and plugins",
   "keywords": [
     "Fedify",
@@ -54,7 +54,7 @@
   ],
   "peerDependencies": {
     "eslint": ">=9.0.0",
-    "@fedify/fedify": "^2.3.9"
+    "@fedify/fedify": "^2.3.11-dev.2280+db3f75fc"
   },
   "peerDependenciesMeta": {
     "eslint": {