@trackunit/eslint-plugin-trackunit 0.6.60 → 0.6.62

package/CHANGELOG.md

@@ -1,3 +1,24 @@
+## 0.6.62 (2026-07-13)
+
+### 🩹 Fixes
+
+- **eslint-plugin-trackunit:** stop require-list-item-virtualization-props false-positiving on nested <li>s in JSX attributes ([5f5191845f1](https://github.com/Trackunit/manager/commit/5f5191845f1))
+
+### 🧱 Updated Dependencies
+
+- Updated shared-utils to 1.15.62
+
+### ❤️ Thank You
+
+- Cursor @cursoragent
+- Michael Buss Rønne
+
+## 0.6.61 (2026-07-10)
+
+### 🧱 Updated Dependencies
+
+- Updated shared-utils to 1.15.61
+
 ## 0.6.60 (2026-07-09)
 
 ### 🧱 Updated Dependencies

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trackunit/eslint-plugin-trackunit",
-  "version": "0.6.60",
+  "version": "0.6.62",
   "license": "SEE LICENSE IN LICENSE.txt",
   "repository": "https://github.com/Trackunit/manager",
   "engines": {

package/src/lib/rules/require-list-item-virtualization-props/require-list-item-virtualization-props.js

@@ -60,6 +60,12 @@ const hasRequiredVirtualizationProps = (element, context) => {
     // Check if all required props are present
     return requiredProps.every(prop => propNames.has(prop));
 };
+/**
+ * Check if a JSXElement is opened with the given tag name
+ */
+const isJSXElementNamed = (element, name) => element.type === utils_1.AST_NODE_TYPES.JSXElement &&
+    element.openingElement.name.type === utils_1.AST_NODE_TYPES.JSXIdentifier &&
+    element.openingElement.name.name === name;
 /**
  * Check if we're inside a List component's render function
  */
@@ -67,11 +73,21 @@ const isInsideListRenderFunction = (node) => {
     let current = node.parent;
     while (current) {
         // Look for JSX element named "List"
-        if (current.type === utils_1.AST_NODE_TYPES.JSXElement &&
-            current.openingElement.name.type === utils_1.AST_NODE_TYPES.JSXIdentifier &&
-            current.openingElement.name.name === "List") {
+        if (isJSXElementNamed(current, "List")) {
             return true;
         }
+        // Once the walk enters a JSX attribute value, we've left the "rendered output" path and
+        // stepped into some element's prop (e.g. a Tooltip's `label` or a ListItem's `description`).
+        // Only keep walking if that attribute belongs to the <List> itself (e.g. its `children`
+        // render-prop is authored as an explicit attribute rather than JSX children) - otherwise any
+        // unrelated <li> nested arbitrarily deep inside a row's props (tooltip content, a nested
+        // list, ...) would be misidentified as the row's own virtualized list item.
+        if (current.type === utils_1.AST_NODE_TYPES.JSXAttribute) {
+            const owningElement = current.parent.parent;
+            if (!isJSXElementNamed(owningElement, "List")) {
+                return false;
+            }
+        }
         current = current.parent;
     }
     return false;