eslint-plugin-flawless 1.4.1 → 1.4.2

package/dist/index.mjs

@@ -1,4 +1,4 @@
-import { n as plugin, t as PLUGIN_NAME } from "./plugin-p3_AxnS4.mjs";
+import { n as plugin, t as PLUGIN_NAME } from "./plugin-F1dJgCBi.mjs";
 //#region src/configs/index.ts
 const configs = { recommended: {
 	plugins: { [PLUGIN_NAME]: plugin },

package/dist/oxlint.mjs

@@ -1,4 +1,4 @@
-import { n as plugin, t as PLUGIN_NAME } from "./plugin-p3_AxnS4.mjs";
+import { n as plugin, t as PLUGIN_NAME } from "./plugin-F1dJgCBi.mjs";
 import { definePlugin } from "@oxlint/plugins";
 //#region src/oxlint.ts
 /**

package/dist/{plugin-p3_AxnS4.mjs → plugin-F1dJgCBi.mjs}

@@ -14,7 +14,7 @@ import * as core from "@eslint-react/core";
 import { getStaticTOMLValue } from "toml-eslint-parser";
 //#region package.json
 var name = "eslint-plugin-flawless";
-var version = "1.4.1";
+var version = "1.4.2";
 var repository = {
 	"url": "git+https://github.com/christopher-buss/eslint-plugin-flawless.git",
 	"type": "git"
@@ -3030,6 +3030,11 @@ const DEFAULTS$1 = {
 };
 /** Callee identifiers that name a vitest test block (`describe` is excluded). */
 const TEST_BLOCK_NAMES$1 = /* @__PURE__ */ new Set(["it", "test"]);
+/**
+* Root identifiers whose call arguments are assertion conditions. Matched by
+* name as well as by vitest resolution, so `assert` from `node:assert` counts.
+*/
+const ASSERTION_ROOT_NAMES = /* @__PURE__ */ new Set(["assert", "expect"]);
 const messages$14 = { [MESSAGE_ID$9]: "Avoid having conditionals in tests." };
 const schema$3 = [{
 	additionalProperties: false,
@@ -3113,12 +3118,43 @@ function resolveVitestName$1(sourceCode, identifier) {
 	return null;
 }
 /**
-* Collects the `?.` tokens along an optional chain's primary spine and the text
-* that replaces each with a non-null assertion. A non-computed member becomes
-* `!.` (`a?.b` -> `a!.b`); a computed member or optional call becomes `!`
-* (`a?.[x]` -> `a![x]`, `fn?.()` -> `fn!()`). Only the object/callee spine is
-* walked — optional chains inside computed keys or call arguments are their own
-* `ChainExpression` nodes and are visited (and fixed) separately.
+* Determines whether a node sits directly in the argument list of an assertion
+* call (`expect(...)`, `assert(...)`, `assert.ok(...)`, `expect(x).toBe(...)`).
+* The search walks up to the nearest enclosing call and stops there, so an
+* argument of some other function nested inside an assertion
+* (`expect(wrap(a && b))`) does not qualify; it also stops at a function
+* boundary, since a callback's body runs on the callback's terms rather than as
+* part of the assertion's condition.
+*
+* @param node - The node to locate.
+* @param sourceCode - Provides the scope used to resolve the callee.
+* @returns `true` when the node is an argument of an assertion call.
+*/
+function isInAssertionArgument(node, sourceCode) {
+	let current = node;
+	for (;;) {
+		const { parent } = current;
+		if (parent === void 0) return false;
+		if (parent.type === AST_NODE_TYPES.ArrowFunctionExpression || parent.type === AST_NODE_TYPES.FunctionDeclaration || parent.type === AST_NODE_TYPES.FunctionExpression) return false;
+		if (parent.type === AST_NODE_TYPES.CallExpression) {
+			if (current === parent.callee) return false;
+			const root = getRootIdentifier$1(parent.callee);
+			if (root === null) return false;
+			return ASSERTION_ROOT_NAMES.has(root.name) || ASSERTION_ROOT_NAMES.has(resolveVitestName$1(sourceCode, root) ?? "");
+		}
+		current = parent;
+	}
+}
+/**
+* Collects the `?.` tokens along an optional chain's primary spine and the parts
+* that convert each into a non-null assertion. The `!` is anchored to the token
+* preceding `?.` rather than written in place of it, because TypeScript forbids
+* a line break before `!` — a chain wrapped as `a\n?.b` has to become `a!\n.b`,
+* not `a\n!.b`. What replaces `?.` is therefore only the connector: `.` for a
+* plain member (`a?.b` -> `a!.b`), nothing for a computed member or optional
+* call (`a?.[x]` -> `a![x]`, `fn?.()` -> `fn!()`). Only the object/callee spine
+* is walked — optional chains inside computed keys or call arguments are their
+* own `ChainExpression` nodes and are visited (and fixed) separately.
 *
 * @param chain - The chain expression to convert.
 * @param sourceCode - Provides token lookups.
@@ -3131,8 +3167,10 @@ function collectOptionalTokenFixes(chain, sourceCode) {
 		if (node.type === AST_NODE_TYPES.MemberExpression) {
 			if (node.optional) {
 				const token = sourceCode.getTokenAfter(node.object, { filter: (candidate) => candidate.value === "?." });
-				if (token !== null) fixes.push({
-					text: node.computed ? "!" : "!.",
+				const anchor = token === null ? null : sourceCode.getTokenBefore(token);
+				if (token !== null && anchor !== null) fixes.push({
+					anchor,
+					text: node.computed ? "" : ".",
 					token
 				});
 			}
@@ -3142,8 +3180,10 @@ function collectOptionalTokenFixes(chain, sourceCode) {
 		if (node.type === AST_NODE_TYPES.CallExpression) {
 			if (node.optional) {
 				const token = sourceCode.getTokenAfter(node.callee, { filter: (candidate) => candidate.value === "?." });
-				if (token !== null) fixes.push({
-					text: "!",
+				const anchor = token === null ? null : sourceCode.getTokenBefore(token);
+				if (token !== null && anchor !== null) fixes.push({
+					anchor,
+					text: "",
 					token
 				});
 			}
@@ -3210,8 +3250,8 @@ function createOnce$10(context) {
 			if (!inTestCase || config.allowOptionalChaining) return;
 			context.report({
 				fix: (fixer) => {
-					return collectOptionalTokenFixes(node, sourceCode).map((optionalFix) => {
-						return fixer.replaceText(optionalFix.token, optionalFix.text);
+					return collectOptionalTokenFixes(node, sourceCode).flatMap((optionalFix) => {
+						return [fixer.insertTextAfter(optionalFix.anchor, "!"), fixer.replaceText(optionalFix.token, optionalFix.text)];
 					});
 				},
 				messageId: MESSAGE_ID$9,
@@ -3220,7 +3260,10 @@ function createOnce$10(context) {
 		},
 		"ConditionalExpression": maybeReportConditional,
 		"IfStatement": maybeReportConditional,
-		"LogicalExpression": maybeReportConditional,
+		"LogicalExpression": function(node) {
+			if (node.operator === "&&" && isInAssertionArgument(node, sourceCode)) return;
+			maybeReportConditional(node);
+		},
 		"SwitchStatement": maybeReportConditional
 	};
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-flawless",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "description": "Your ESLint plugin description",
   "keywords": [
     "eslint",