From af6c54bac7e02e4aed333c8e0dc4c45e0e9f8a78 Mon Sep 17 00:00:00 2001
From: Elian Doran <contact@eliandoran.me>
Date: Mon, 26 Jan 2026 21:00:56 +0200
Subject: [PATCH] fix(web-clipper): context menu got broken on Firefox

---
 .../entrypoints/background/context_menu.ts            | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/apps/web-clipper/entrypoints/background/context_menu.ts b/apps/web-clipper/entrypoints/background/context_menu.ts
index 28786df7bf..967466a0ad 100644
--- a/apps/web-clipper/entrypoints/background/context_menu.ts
+++ b/apps/web-clipper/entrypoints/background/context_menu.ts
@@ -31,12 +31,11 @@ const CONTEXT_MENU_ITEMS: Browser.contextMenus.CreateProperties[] = [
     }
 ];
 
-export default function setupContextMenu() {
+export default async function setupContextMenu() {
     // Context menu items need to be registered only once.
     // https://stackoverflow.com/questions/64318529/cannot-create-item-with-duplicate-context-menu-id-in-extension
-    browser.runtime.onInstalled.addListener(() => {
-        for (const item of CONTEXT_MENU_ITEMS) {
-            browser.contextMenus.create(item);
-        }
-    });
+    await browser.contextMenus.removeAll();
+    for (const item of CONTEXT_MENU_ITEMS) {
+        browser.contextMenus.create(item);
+    }
 }