fix(no-node-access): prevent duplicate errors for chained property access (#1260)

Fixes #1256

Author: y-hsgw

src/rules/no-node-access.ts

@@ -65,6 +65,12 @@ export default createTestingLibraryRule<Options, MessageIds>({
 				return;
 			}
 
+			// Skip if this MemberExpression is the object of another MemberExpression
+			// to avoid duplicate reports for chained property access
+			if (isMemberExpression(node.parent) && node.parent.object === node) {
+				return;
+			}
+
 			const propertyName = ASTUtils.isIdentifier(node.property)
 				? node.property.name
 				: null;

tests/rules/no-node-access.test.ts

@@ -527,6 +527,33 @@ ruleTester.run(rule.name, rule, {
 		},
 		{
 			code: `
+        import { screen } from '${testingFramework}';
+
+        const rowHeader = screen.getByRole('rowheader');
+        const radioContainer =
+            (rowHeader.parentElement && rowHeader.parentElement.nextElementSibling) ||
+            rowHeader.parentElement;
+      `,
+			errors: [
+				{
+					line: 6,
+					column: 24,
+					messageId: 'noNodeAccess',
+				},
+				{
+					line: 6,
+					column: 65,
+					messageId: 'noNodeAccess',
+				},
+				{
+					line: 7,
+					column: 23,
+					messageId: 'noNodeAccess',
+				},
+			],
+		},
+		{
+			code: `
         import { render } from '${testingFramework}';
 
         const { getByText } = render(<Example />)