Modernize code patterns: use forEach and optional chaining

Co-authored-by: PeterDaveHello <[email protected]>

Author: Copilot

src/background/menus.mjs

@@ -85,8 +85,7 @@ export function refreshMenu() {
     }
 
     // Add custom selection tools that are active
-    for (let i = 0; i < userConfig.customSelectionTools.length; i++) {
-      const tool = userConfig.customSelectionTools[i]
+    userConfig.customSelectionTools.forEach((tool, i) => {
       if (tool.active && tool.name) {
         Browser.contextMenus.create({
           id: menuId + 'custom_' + i,
@@ -95,7 +94,7 @@ export function refreshMenu() {
           contexts: ['selection'],
         })
       }
-    }
+    })
 
     Browser.contextMenus.onClicked.addListener(onClickMenu)
   })

src/content-script/index.jsx

@@ -291,16 +291,9 @@ async function prepareForRightClickMenu() {
       } else if (data.itemId.startsWith('custom_')) {
         // Handle custom selection tools from context menu
         const customIndex = parseInt(data.itemId.replace('custom_', ''), 10)
-        if (
-          !isNaN(customIndex) &&
-          customIndex >= 0 &&
-          userConfig.customSelectionTools &&
-          customIndex < userConfig.customSelectionTools.length
-        ) {
-          const customTool = userConfig.customSelectionTools[customIndex]
-          if (customTool.active && customTool.name) {
-            prompt = customTool.prompt.replace('{{selection}}', data.selectionText)
-          }
+        const customTool = userConfig.customSelectionTools?.[customIndex]
+        if (customTool?.active && customTool?.name) {
+          prompt = customTool.prompt.replace('{{selection}}', data.selectionText)
         }
       } else if (data.itemId in menuConfig) {
         const menuItem = menuConfig[data.itemId]