Fix parseInt usage and add error handling for sendMessage calls

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

Author: Copilot

src/content-script/index.jsx

@@ -290,8 +290,13 @@ async function prepareForRightClickMenu() {
         prompt = await toolsConfig[data.itemId].genPrompt(data.selectionText)
       } else if (data.itemId.startsWith('custom_')) {
         // Handle custom selection tools from context menu
-        const customIndex = parseInt(data.itemId.replace('custom_', ''))
-        if (customIndex >= 0 && customIndex < userConfig.customSelectionTools.length) {
+        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)

src/popup/sections/SelectionTools.jsx

@@ -56,9 +56,11 @@ export function SelectionTools({ config, updateConfig }) {
               customSelectionTools[editingIndex] = editingTool
               await updateConfig({ customSelectionTools })
             }
-            Browser.runtime.sendMessage({
-              type: 'REFRESH_MENU',
-            })
+            Browser.runtime
+              .sendMessage({
+                type: 'REFRESH_MENU',
+              })
+              .catch(console.error)
             setEditing(false)
           }}
         >
@@ -111,9 +113,11 @@ export function SelectionTools({ config, updateConfig }) {
               const activeSelectionTools = config.activeSelectionTools.filter((i) => i !== key)
               if (checked) activeSelectionTools.push(key)
               await updateConfig({ activeSelectionTools })
-              Browser.runtime.sendMessage({
-                type: 'REFRESH_MENU',
-              })
+              Browser.runtime
+                .sendMessage({
+                  type: 'REFRESH_MENU',
+                })
+                .catch(console.error)
             }}
           />
           {t(toolsConfig[key].label)}
@@ -133,9 +137,11 @@ export function SelectionTools({ config, updateConfig }) {
                   const customSelectionTools = [...config.customSelectionTools]
                   customSelectionTools[index] = { ...tool, active: e.target.checked }
                   await updateConfig({ customSelectionTools })
-                  Browser.runtime.sendMessage({
-                    type: 'REFRESH_MENU',
-                  })
+                  Browser.runtime
+                    .sendMessage({
+                      type: 'REFRESH_MENU',
+                    })
+                    .catch(console.error)
                 }}
               />
               {tool.name}
@@ -160,9 +166,11 @@ export function SelectionTools({ config, updateConfig }) {
                     const customSelectionTools = [...config.customSelectionTools]
                     customSelectionTools.splice(index, 1)
                     await updateConfig({ customSelectionTools })
-                    Browser.runtime.sendMessage({
-                      type: 'REFRESH_MENU',
-                    })
+                    Browser.runtime
+                      .sendMessage({
+                        type: 'REFRESH_MENU',
+                      })
+                      .catch(console.error)
                   }}
                 >
                   <TrashIcon />