Prevent virtualized SelectPanel measurement warnings

.changeset/fix-virtualized-select-panel-measurement.md

@@ -0,0 +1,5 @@
+---
+'@primer/react': patch
+---
+
+SelectPanel: Prevent console warnings when measuring virtualized items

packages/react/src/ActionList/Item.test.tsx

@@ -387,6 +387,21 @@ describe('ActionList.Item', () => {
     expect(tabs).toHaveLength(3)
   })
 
+  it('should forward a list-semantic item ref only to the outer list item', () => {
+    const ref = vi.fn<(node: HTMLLIElement | null) => void>()
+    const {getByRole} = HTMLRender(
+      <ActionList role="listbox" aria-label="Projects">
+        <ActionList.Item ref={ref} role="option">
+          Primer React
+        </ActionList.Item>
+      </ActionList>,
+    )
+    const option = getByRole('option')
+
+    expect(ref.mock.calls.filter(([node]) => node !== null)).toEqual([[option]])
+    expect(option.tagName).toBe('LI')
+  })
+
   it('should preserve consumer ref when tooltip wraps trigger', async () => {
     const user = userEvent.setup()
 

packages/react/src/ActionList/Item.tsx

@@ -340,7 +340,7 @@ const UnwrappedItem = <As extends React.ElementType = 'li'>(
             className={classes.ActionListContent}
             data-size={size}
             // @ts-ignore: ItemWrapper is polymorphic and the ref type depends on the rendered element ('button' or 'li')
-            ref={forwardedRef}
+            ref={listSemantics ? null : forwardedRef}
           >
             {/* Reset TooltipContext so that child components don't detect
                 the ConditionalTooltip and suppress their own internal tooltips. */}

packages/react/src/FilteredActionList/FilteredActionList.test.tsx

@@ -100,6 +100,21 @@ describe('FilteredActionList', () => {
       ).toBeInTheDocument()
     })
   })
+
+  it('does not warn when measuring virtualized items', () => {
+    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
+
+    try {
+      const {container} = render(<FilteredActionList items={items} onFilterChange={vi.fn()} virtualized />)
+
+      expect(container.querySelector('[data-index]')).toBeInTheDocument()
+      expect(warnSpy.mock.calls.flat().join(' ')).not.toContain(
+        "Missing attribute name 'data-index={index}' on measured element.",
+      )
+    } finally {
+      warnSpy.mockRestore()
+    }
+  })
 })
 
 describe('FilteredActionListBodyLoader', () => {