fix: sendResponse not working in firefox

mozilla/webextension-polyfill#172

README.md

@@ -56,6 +56,7 @@ Then run the following:
 1. Go to the browser address bar and type [about:debugging#/runtime/this-firefox](about:debugging#/runtime/this-firefox) (not the [about:addons](about:addons))
 1. Click on the `Load Temporary Add-on` list item after click on setting button.
 1. Select your `dist-firefox` folder in the project root.
+1. [Debug popup in firefox](https://firefox-source-docs.mozilla.org/devtools-user/browser_toolbox/index.html)
 
 ### Other Commands
 

src/content/Content.tsx

@@ -9,7 +9,6 @@ export function Content() {
   const { t } = useTranslation();
   const [isSelecting, setIsSelecting] = useState(false);
   const { parseReadability } = useReadability();
-
   const [selectedElement, setSelectedElement] = useState<HTMLElement | null>(null);
   useMessaging({ setIsSelecting, parseReadability, selectedElement });
   const { updateSelector } = useSelectorGenerator();

src/content/hooks/useMessaging.ts

@@ -20,15 +20,18 @@ export function useMessaging(
           if (parameter.selectedElement === null) return;
           const text = parameter.selectedElement.textContent ?? '';
           const html = parameter.selectedElement.outerHTML;
-          sendResponse({ action: ITabActions.startClippingResponse, text, html } satisfies IStartClippingResponseMessage);
-          break;
+          const response = { action: ITabActions.startClippingResponse, text, html } satisfies IStartClippingResponseMessage;
+          sendResponse(response);
+          // return the response instead of `sendResponse`, otherwise response will be `undefined` in firefox. In Chrome, `sendResponse` works fine.
+          return response;
         }
         case ITabActions.getReadability: {
           const article = parameter.parseReadability();
           // Get the current webpage URL
           const url = window.location.href;
-          sendResponse({ action: ITabActions.getReadabilityResponse, article, url });
-          break;
+          const response = { action: ITabActions.getReadabilityResponse, article, url };
+          sendResponse(response);
+          return response;
         }
       }
     });