fix: also wrap menus.mjs openSidePanel call in try/catch

contextMenus.onClicked.tab is optional per the Chrome API ('If the
click did not take place in a tab, this parameter will be missing').
The openSidePanel action dereferences tab.windowId/tab.id, so it can
throw synchronously, which the existing .catch() Promise handler
would not catch. Mirror the pattern already used in
src/background/commands.mjs by wrapping the call in try/catch.

Addresses devin-ai-integration[bot] review on src/background/menus.mjs:23.

Author: octo-patch

src/background/menus.mjs

@@ -15,7 +15,17 @@ const onClickMenu = (info, tab) => {
     // Keep the call synchronous to preserve the user-gesture requirement,
     // but observe the returned Promise so a rejected sidePanel.open() does
     // not become an unhandled rejection in the background script.
-    const result = menuConfig.openSidePanel.action(true, tab)
+    // Also wrap in try/catch because contextMenus.onClicked documents `tab`
+    // as optional ("If the click did not take place in a tab, this parameter
+    // will be missing"), so the openSidePanel action that dereferences
+    // tab.windowId/tab.id can throw synchronously.
+    let result
+    try {
+      result = menuConfig.openSidePanel.action(true, tab)
+    } catch (error) {
+      console.error('failed to open side panel', error)
+      return
+    }
     if (result && typeof result.catch === 'function') {
       result.catch((error) => {
         console.error('failed to open side panel', error)