fix: add key for MenuActionList

[winjo]

Types

• 🐛 Bug Fixes

Background or solution

Optimize MenuActionList recursiveRender to avoid error of react list key

🤖 Generated by Copilot at 0467eec

• Fix a bug where two consecutive separators would be rendered in the menu (link)
• Simplify the code by removing unnecessary fragments and the hasSeparator logic (link)
• Remove the redundant key prop from the Menu.SubMenu component (link)

Changelog

🤖 Generated by Copilot at 0467eec

Fix menu rendering bug and simplify code in actions component. Remove duplicate separators and redundant fragments from menu items in packages/core-browser/src/components/actions/index.tsx.

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +4.98 🎉

Comparison is base (0e85c94) 52.69% compared to head (0467eec) 57.67%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2809      +/-   ##
==========================================
+ Coverage   52.69%   57.67%   +4.98%     
==========================================
  Files        1336     1336              
  Lines       83883    83883              
  Branches    17439    17439              
==========================================
+ Hits        44201    48382    +4181     
+ Misses      36031    32266    -3765     
+ Partials     3651     3235     -416     

Flag	Coverage Δ
jsdom	52.69% <ø> (+<0.01%)	⬆️
node	16.79% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 107 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.