fix(ResponsiveAction): resolve warning about accessing reserved 'key' prop (#892)

Fixes #890

The ResponsiveAction component was incorrectly defining 'key' as a prop
in its interface, which caused React to warn when developers used the
standard key prop pattern with arrays of ResponsiveAction components.

Changes:
- Removed 'key' from ResponsiveActionProps interface since 'key' is a
  reserved React prop that cannot be accessed within components
- Updated ResponsiveActions to use child.key instead of child.props.key
  to access React's internal key value without triggering warnings

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: rhamilto

packages/module/src/ResponsiveAction/ResponsiveAction.tsx

@@ -7,8 +7,6 @@ export interface ResponsiveActionProps extends ButtonProps {
   isPinned?: boolean;
   /** Determines whether the action should always be displayed as pinned */
   isPersistent?: boolean;
-  /** Key for the action */
-  key?: string;
   /** Action label */
   children: React.ReactNode;
 };

packages/module/src/ResponsiveActions/ResponsiveActions.tsx

@@ -25,7 +25,8 @@ export const ResponsiveActions: FunctionComponent<ResponsiveActionsProps> = ({ o
 
   Children.forEach(children, (child, index) => {
     if (isValidElement<ResponsiveActionProps>(child)) {
-      const { isPersistent, isPinned, key = index, children, onClick, ...actionProps } = child.props;
+      const { isPersistent, isPinned, children, onClick, ...actionProps } = child.props;
+      const key = child.key ?? index;
 
       if (isPersistent || isPinned) {
         (isPersistent ? persistentActions : pinnedActions).push(