src/component/ResponsiveContainer.tsx (1)

257-304: ⚠️ Potential issue | 🟠 Major

HTML attributes are not forwarded in all code paths.

The ResponsiveContainer component has three return paths, but only one forwards HTML attributes:

1. Line 264 (nested container): Returns only props.children — attributes lost
2. Lines 294-298 (fixed dimensions): Returns only context provider — attributes lost
3. Line 304 (size observer): Uses SizeDetectorContainer — attributes forwarded ✓

When users provide fixed numeric width and height (or when nested), props like data-testid, aria-label, and role will be silently dropped.

🐛 Proposed fix to forward attributes in all paths

   if (isPositiveNumber(responsiveContainerContext.width) && isPositiveNumber(responsiveContainerContext.height)) {
-    return props.children;
+    return <SizeDetectorContainer {...props} ref={ref} />;
   }

And for the fixed dimensions path, consider wrapping children in a div with the HTML attributes, or always use SizeDetectorContainer:

   if (isNumber(calculatedWidth) && isNumber(calculatedHeight)) {
-    return (
-      <ResponsiveContainerContextProvider width={calculatedWidth} height={calculatedHeight}>
-        {props.children}
-      </ResponsiveContainerContextProvider>
-    );
+    return <SizeDetectorContainer {...props} width={width} height={height} ref={ref} />;
   }

Alternatively, if avoiding the extra div is important for these cases, document this limitation clearly.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/component/ResponsiveContainer.tsx` around lines 257 - 304,
ResponsiveContainer currently returns three different roots but only the
SizeDetectorContainer path forwards HTML attributes, so attributes like
data-testid/aria-label are dropped when returning props.children (nested case)
or ResponsiveContainerContextProvider (fixed-dimensions case). Fix by ensuring
HTML attributes from props are passed through on all return paths: spread the
leftover DOM props (e.g., using restProps or extracting known non-DOM props)
onto a wrapper element or onto ResponsiveContainerContextProvider when returning
fixed dimensions, and when returning props.children in the nested-container case
wrap children in a fragment that preserves attributes (e.g., a <div> or the
existing SizeDetectorContainer) so attributes are applied; update
ResponsiveContainer to forward ref and DOM props consistently and reuse
SizeDetectorContainer or ResponsiveContainerContextProvider wrappers to
centralize attribute forwarding (references: ResponsiveContainer,
SizeDetectorContainer, ResponsiveContainerContextProvider,
useResponsiveContainerContext, getDefaultWidthAndHeight,
calculateChartDimensions).