From 8dc5409c431830ee059862e196609187bcddacc9 Mon Sep 17 00:00:00 2001
From: Cashew <tungbinem@gmail.com>
Date: Fri, 29 Dec 2023 09:17:28 +0700
Subject: [PATCH 1/2] comment updates

---
 ext/js/accessibility/google-docs-util.js |  6 ++++--
 ext/js/language/deinflector.js           |  2 +-
 ext/js/language/sandbox/japanese-util.js |  6 ++++--
 types/ext/translation-internal.d.ts      | 14 +++++++-------
 4 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/ext/js/accessibility/google-docs-util.js b/ext/js/accessibility/google-docs-util.js
index 151b634092..cc6cbd698d 100644
--- a/ext/js/accessibility/google-docs-util.js
+++ b/ext/js/accessibility/google-docs-util.js
@@ -49,11 +49,13 @@ export class GoogleDocsUtil {
     }
 
     /**
+     * Gets this <style> node, or creates one if it doesn't exist.
+     *
+     * A <style> node is necessary to force the SVG <rect> elements to have a fill,
+     * which allows them to be included in document.elementsFromPoint's return value.
      * @returns {HTMLStyleElement}
      */
     static _getStyleNode() {
-        // This <style> node is necessary to force the SVG <rect> elements to have a fill,
-        // which allows them to be included in document.elementsFromPoint's return value.
         if (typeof this._styleNode === 'undefined') {
             const style = document.createElement('style');
             style.textContent = [
diff --git a/ext/js/language/deinflector.js b/ext/js/language/deinflector.js
index f0e02f6f31..d2d92e53d6 100644
--- a/ext/js/language/deinflector.js
+++ b/ext/js/language/deinflector.js
@@ -50,7 +50,7 @@ export class Deinflector {
      * @example
      * const deinflector = new Deinflector(deinflectionReasons);
      * // [{ term: '食べた', rules: 0, reasons: [] }, { term: '食べる', rules: 1, reasons: ['past'] }, { term: '食ぶ', rules: 2, reasons: ['potential', 'past'] }]
-     * console.log(deinflector.deinflect('食べさせられる'));
+     * console.log(deinflector.deinflect('食べた'));
      */
     deinflect(source) {
         const results = [this._createDeinflection(source, 0, [])];
diff --git a/ext/js/language/sandbox/japanese-util.js b/ext/js/language/sandbox/japanese-util.js
index 7d9413a6b3..f5fc1c8669 100644
--- a/ext/js/language/sandbox/japanese-util.js
+++ b/ext/js/language/sandbox/japanese-util.js
@@ -66,8 +66,10 @@ const CJK_IDEOGRAPH_RANGES = [
     CJK_COMPATIBILITY_IDEOGRAPHS_SUPPLEMENT_RANGE
 ];
 
-// Japanese character ranges, roughly ordered in order of expected frequency
-/** @type {import('japanese-util').CodepointRange[]} */
+/**
+ * Japanese character ranges, roughly ordered in order of expected frequency.
+ * @type {import('japanese-util').CodepointRange[]}
+ */
 const JAPANESE_RANGES = [
     HIRAGANA_RANGE,
     KATAKANA_RANGE,
diff --git a/types/ext/translation-internal.d.ts b/types/ext/translation-internal.d.ts
index 784a597983..5845b45d44 100644
--- a/types/ext/translation-internal.d.ts
+++ b/types/ext/translation-internal.d.ts
@@ -40,13 +40,13 @@ export type TextDeinflectionOptionsArrays = [
 
 export enum DeinflectionRuleFlags {
     None = 0x0,
-    VerbIchidan = 0b00000001, // Verb ichidan
-    VerbGodan = 0b00000010, // Verb godan
-    VerbSuru = 0b00000100, // Verb suru
-    VerbKuru = 0b00001000, // Verb kuru
-    VerbZuru = 0b00010000, // Verb zuru
-    AdjectiveI = 0b00100000, // Adjective i
-    IruEndingIntermediate = 0b01000000, // Intermediate -iru endings for progressive or perfect tense
+    VerbIchidan = 0b00000001,
+    VerbGodan = 0b00000010,
+    VerbSuru = 0b00000100,
+    VerbKuru = 0b00001000,
+    VerbZuru = 0b00010000,
+    AdjectiveI = 0b00100000,
+    IruEndingIntermediate = 0b01000000,
 }
 
 export type Deinflection = {

From f076d89dd4b4264c1ee78b3bc9f6879294e4bcb6 Mon Sep 17 00:00:00 2001
From: Cashew <tungbinem@gmail.com>
Date: Fri, 29 Dec 2023 09:42:03 +0700
Subject: [PATCH 2/2] comment updates

---
 ext/js/app/frontend.js           |  8 +++++---
 ext/js/background/backend.js     |  2 +-
 ext/js/comm/clipboard-monitor.js | 10 ++++++----
 3 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/ext/js/app/frontend.js b/ext/js/app/frontend.js
index b093ec33fe..e386bf64d9 100644
--- a/ext/js/app/frontend.js
+++ b/ext/js/app/frontend.js
@@ -552,9 +552,11 @@ export class Frontend {
             }
         }
 
-        // The token below is used as a unique identifier to ensure that a new _updatePopup call
-        // hasn't been started during the await.
-        /** @type {?import('core').TokenObject} */
+        /**
+         * The token below is used as a unique identifier to ensure that a new _updatePopup call
+         * hasn't been started during the await.
+         * @type {?import('core').TokenObject}
+         */
         const token = {};
         this._updatePopupToken = token;
         const popup = await popupPromise;
diff --git a/ext/js/background/backend.js b/ext/js/background/backend.js
index ae78a97b5c..c0d3cabdb9 100644
--- a/ext/js/background/backend.js
+++ b/ext/js/background/backend.js
@@ -1664,6 +1664,7 @@ export class Backend {
     }
 
     /**
+     * This function works around the need to have the "tabs" permission to access tab.url.
      * @param {number} timeout
      * @param {boolean} multiple
      * @param {import('backend').FindTabsPredicate} predicate
@@ -1671,7 +1672,6 @@ export class Backend {
      * @returns {Promise<import('backend').TabInfo[]|(?import('backend').TabInfo)>}
      */
     async _findTabs(timeout, multiple, predicate, predicateIsAsync) {
-        // This function works around the need to have the "tabs" permission to access tab.url.
         const tabs = await this._getAllTabs();
 
         let done = false;
diff --git a/ext/js/comm/clipboard-monitor.js b/ext/js/comm/clipboard-monitor.js
index 3b3a56a9d7..86d537fd44 100644
--- a/ext/js/comm/clipboard-monitor.js
+++ b/ext/js/comm/clipboard-monitor.js
@@ -47,11 +47,13 @@ export class ClipboardMonitor extends EventDispatcher {
     start() {
         this.stop();
 
-        // The token below is used as a unique identifier to ensure that a new clipboard monitor
-        // hasn't been started during the await call. The check below the await call
-        // will exit early if the reference has changed.
         let canChange = false;
-        /** @type {?import('core').TokenObject} */
+        /**
+         * This token is used as a unique identifier to ensure that a new clipboard monitor
+         * hasn't been started during the await call. The check below the await call
+         * will exit early if the reference has changed.
+         * @type {?import('core').TokenObject}
+         */
         const token = {};
         const intervalCallback = async () => {
             this._timerId = null;