refactor code

config/webpack.common.js

@@ -6,7 +6,6 @@ const MiniCssExtractPlugin = require("mini-css-extract-plugin");
 const { VueLoaderPlugin } = require("vue-loader");
 const PATHS = require("./paths");
 // var path = require("path");
-const ExtReloader = require("@reorx/webpack-ext-reloader");
 
 // To re-use webpack configuration across templates,
 // CLI maintains a common webpack configuration file - `webpack.common.js`.
@@ -91,18 +90,6 @@ const common = {
     ],
   },
   plugins: [
-    // extension auto reload
-    new ExtReloader({
-      port: 9090, // Which port use to create the server
-      reloadPage: true, // Force the reload of the page also
-      entries: {
-        // The entries used for the content/background scripts or extension pages
-        contentScript: "contentScript",
-        background: "background",
-        extensionPage: "popup",
-      },
-    }),
-
     new VueLoaderPlugin(),
     // Print file sizes
     new SizePlugin(),

config/webpack.config.js

@@ -6,6 +6,7 @@ const common = require("./webpack.common.js");
 const PATHS = require("./paths");
 var glob = require("glob");
 var path = require("path");
+const ExtReloader = require("@reorx/webpack-ext-reloader");
 
 // Merge webpack configuration files
 module.exports = (env, argv) => {
@@ -25,5 +26,16 @@ module.exports = (env, argv) => {
         stream: require.resolve("stream-browserify"),
       },
     },
+    plugins: [
+      argv.mode == "development"
+        ? new ExtReloader({
+            port: 9090, // Which port use to create the server
+            reloadPage: true, // Force the reload of the page also
+            entries: {
+              background: "background",
+            },
+          })
+        : false,
+    ],
   });
 };

doc/description.md

@@ -20,6 +20,12 @@ English, Russian, Japanese, Chinese and so on
 
 # Change Log
 
+- 0.1.78
+  - google reject by "Blue Argon"
+    - make ExtReloader works only on development
+    - remove any unused package
+    - refactor code
+    - double check any existence of eval()
 - 0.1.77
   - fix youtube dual subtitles time mismatch
 - 0.1.76

package.json

@@ -1,6 +1,6 @@
 {
   "name": "mouse_tooltip_translator",
-  "version": "0.1.77",
+  "version": "0.1.78",
   "description": "Mouse over to translate",
   "private": true,
   "scripts": {
@@ -43,7 +43,6 @@
     "he": "^1.2.0",
     "is-url": "^1.2.4",
     "jquery": "^3.5.1",
-    "load-script-once": "^2.0.1",
     "match-url-wildcard": "^0.0.5",
     "popper.js": "^1.16.1",
     "source-map-support": "^0.5.21",

public/manifest.json

@@ -1,6 +1,6 @@
 {
   "manifest_version": 3,
-  "version": "0.1.77",
+  "version": "0.1.78",
   "name": "__MSG_appName__",
   "description": "__MSG_appDesc__",
   "default_locale": "en",

src/background.js

@@ -239,10 +239,6 @@ function openIntroSite(reason) {
 }
 
 function addUninstallUrl() {
-  if (util.checkInDevMode()) {
-    return;
-  }
-
   var reviewPage =
     browser.name == "edge-chromium" ? edgeReviewPage : chromeReviewPage;
   chrome.runtime.setUninstallURL(reviewPage);

src/popup.vue

@@ -469,7 +469,6 @@ var translatorList = {
   google: "google",
   bing: "bing",
   "papago (Experimental)": "papago",
-  "deepl (Experimental)": "deepl"
 };
 
 var translateActionList = {

src/subtitle/youtube.js

@@ -1,5 +1,4 @@
-// https://terrillthompson.com/648
-// intercept youtube subtitle and concat it
+// intercept youtube subtitle and concat dual sub
 // restart playersubtitle for apply
 
 import $ from "jquery";
@@ -29,7 +28,7 @@ window.addEventListener(
   false
 );
 
-// check any subtitle request
+// check any subtitle request and concat dual sub
 interceptor.on("request", async ({ request, requestId }) => {
   try {
     if (request.url.includes("www.youtube.com/api/timedtext")) {
@@ -52,7 +51,7 @@ interceptor.on("request", async ({ request, requestId }) => {
   }
 });
 
-// check url change
+// check url change and turn on sub
 navigation.addEventListener("navigate", (e) => {
   var url = e.destination.url;
   activateCaption(url);
@@ -149,12 +148,6 @@ function concatWordSub(subtitle) {
       newEvents[newEvents.length - 1].segs[0].utf8 += oneLineSub
         ? ` ${oneLineSub}`
         : "";
-
-      // newEvents[newEvents.length - 1].dDurationMs =
-      //   event.tStartMs +
-      //   event.dDurationMs -
-      //   newEvents[newEvents.length - 1].tStartMs -
-      //   100;
     }
   }
 
@@ -205,21 +198,7 @@ async function getYoutubeMetaData(vParam) {
   return json;
 }
 
-async function getSubUrl(v, lang) {
-  var metaData = await getYoutubeMetaData(v);
-  var captionList =
-    metaData?.captions?.playerCaptionsTracklistRenderer?.captionTracks;
-  var langUrl = captionList.filter(
-    (caption) => !caption?.kind && caption.languageCode == lang
-  )?.[0]?.baseUrl;
-  return langUrl;
-}
-
 async function getTranslatedSubtitle(baseUrl, lang) {
-  // use user generated sub if exist
-  // var v = getVideoIdParam(baseUrl);
-  // var url = await getSubUrl(v, lang) + "&fmt=json3";
-
   var url = new URL(baseUrl);
   url.searchParams.set("tlang", lang);
   return await (await fetch(url.toString())).json();