From 856f24260c40537d06822a1c0ca057626edec431 Mon Sep 17 00:00:00 2001
From: Yucheng Chuang <ducksteven@gmail.com>
Date: Tue, 16 Apr 2024 16:46:44 +0800
Subject: [PATCH] Fix ESLint

---
 firefox/hacks.js  |  3 ++-
 src/background.js | 53 ++++++++++++++++++++++++-----------------------
 2 files changed, 29 insertions(+), 27 deletions(-)

diff --git a/firefox/hacks.js b/firefox/hacks.js
index 7a91a18..e9666cf 100644
--- a/firefox/hacks.js
+++ b/firefox/hacks.js
@@ -1 +1,2 @@
-globalThis["PERIDOCIALLY_REFRESH_MENU"] = true;
+/* eslint-disable no-undef */
+globalThis.PERIDOCIALLY_REFRESH_MENU = true;
diff --git a/src/background.js b/src/background.js
index 4739dda..7aba0fc 100644
--- a/src/background.js
+++ b/src/background.js
@@ -62,6 +62,29 @@ async function flashBadge(type) {
   chrome.alarms.create('clear', { when: Date.now() + FLASH_BADGE_TIMEOUT });
 }
 
+function createMenus() {
+  chrome.contextMenus.create({
+    id: 'current-page',
+    title: 'Copy [Page Title](URL)',
+    type: 'normal',
+    contexts: ['page'],
+  });
+
+  chrome.contextMenus.create({
+    id: 'link',
+    title: 'Copy [Link Content](URL)',
+    type: 'normal',
+    contexts: ['link'],
+  });
+
+  chrome.contextMenus.create({
+    id: 'image',
+    title: 'Copy ![](Image URL)', // TODO: how to fetch alt text?
+    type: 'normal',
+    contexts: ['image'],
+  });
+}
+
 chrome.alarms.onAlarm.addListener((alarm) => {
   const entrypoint = chrome.action /* MV3 */ || chrome.browserAction; /* Firefox MV2 */
 
@@ -176,36 +199,14 @@ async function mustGetCurrentTab() {
   return Promise.resolve(tabs[0]);
 }
 
-function createMenus() {
-  chrome.contextMenus.create({
-    id: 'current-page',
-    title: 'Copy [Page Title](URL)',
-    type: 'normal',
-    contexts: ['page'],
-  });
-
-  chrome.contextMenus.create({
-    id: 'link',
-    title: 'Copy [Link Content](URL)',
-    type: 'normal',
-    contexts: ['link'],
-  });
-
-  chrome.contextMenus.create({
-    id: 'image',
-    title: 'Copy ![](Image URL)', // TODO: how to fetch alt text?
-    type: 'normal',
-    contexts: ['image'],
-  });
-}
-
 chrome.runtime.onInstalled.addListener(createMenus);
 
-if (globalThis["PERIDOCIALLY_REFRESH_MENU"] === true) {
+// eslint-disable-next-line no-undef
+if (globalThis.PERIDOCIALLY_REFRESH_MENU === true) {
   // Hack for Firefox, in which Context Menu disappears after some time.
   // See https://discourse.mozilla.org/t/strange-mv3-behaviour-browser-runtime-oninstalled-event-and-menus-create/111208/7
-  console.info("Hack PERIDOCIALLY_REFRESH_MENU is enabled");
-  chrome.alarms.create("refreshMenu", { periodInMinutes: 0.5});
+  console.info('Hack PERIDOCIALLY_REFRESH_MENU is enabled');
+  chrome.alarms.create('refreshMenu', { periodInMinutes: 0.5 });
 }
 
 // NOTE: All listeners must be registered at top level scope.