Turn off usage of 'debug' npm package using with electron-16 (fixes #17)

8 files changed, 57 insertions, 49 deletions

diff --git a/src/webview/badge.ts b/src/webview/badge.ts
index 1b66effb6..0ff1ecaf6 100644
--- a/src/webview/badge.ts
+++ b/src/webview/badge.ts
@@ -1,6 +1,7 @@
 import { ipcRenderer } from 'electron';
 
-const debug = require('debug')('Ferdium:Plugin:BadgeHandler');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:Plugin:BadgeHandler');
 
 export class BadgeHandler {
   // TODO: Need to extract this into a utility class and reuse outside of the recipes
@@ -26,7 +27,7 @@ export class BadgeHandler {
       indirect: this.safeParseInt(indirect),
     };
 
-    debug('Sending badge count to host: %j', count);
+    console.log('Sending badge count to host: %j', count);
     ipcRenderer.sendToHost('message-counts', count);
   }
 }
diff --git a/src/webview/darkmode.ts b/src/webview/darkmode.ts
index fbb0edf04..99ee68757 100644
--- a/src/webview/darkmode.ts
+++ b/src/webview/darkmode.ts
@@ -1,7 +1,8 @@
 import { join } from 'path';
 import { pathExistsSync, readFileSync } from 'fs-extra';
 
-const debug = require('debug')('Ferdium:DarkMode');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:DarkMode');
 
 const chars = [...'abcdefghijklmnopqrstuvwxyz'];
 
@@ -26,7 +27,7 @@ export function injectDarkModeStyle(recipePath: string) {
 
     document.querySelector('head')?.appendChild(styles);
 
-    debug('Injected Dark Mode style with ID', ID);
+    console.log('Injected Dark Mode style with ID', ID);
   }
 }
 
@@ -36,7 +37,7 @@ export function removeDarkModeStyle() {
   if (style) {
     style.remove();
 
-    debug('Removed Dark Mode Style with ID', ID);
+    console.log('Removed Dark Mode Style with ID', ID);
   }
 }
 
diff --git a/src/webview/dialogTitle.ts b/src/webview/dialogTitle.ts
index 393c2499f..a5bcb4c41 100644
--- a/src/webview/dialogTitle.ts
+++ b/src/webview/dialogTitle.ts
@@ -1,6 +1,7 @@
 import { ipcRenderer } from 'electron';
 
-const debug = require('debug')('Ferdium:Plugin:DialogTitleHandler');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:Plugin:DialogTitleHandler');
 
 export class DialogTitleHandler {
   titleCache: { title: string };
@@ -25,7 +26,7 @@ export class DialogTitleHandler {
       return;
     }
 
-    debug('Sending active dialog title to host %s', newTitle);
+    console.log('Sending active dialog title to host %s', newTitle);
     ipcRenderer.sendToHost('active-dialog-title', newTitle);
 
     this.titleCache.title = newTitle;
diff --git a/src/webview/lib/RecipeWebview.ts b/src/webview/lib/RecipeWebview.ts
index 568c31255..cf70164ef 100644
--- a/src/webview/lib/RecipeWebview.ts
+++ b/src/webview/lib/RecipeWebview.ts
@@ -2,7 +2,8 @@ import { ipcRenderer } from 'electron';
 import { BrowserWindow } from '@electron/remote';
 import { pathExistsSync, readFileSync, existsSync } from 'fs-extra';
 
-const debug = require('debug')('Ferdium:Plugin:RecipeWebview');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:Plugin:RecipeWebview');
 
 class RecipeWebview {
   badgeHandler: any;
@@ -27,7 +28,7 @@ class RecipeWebview {
     ipcRenderer.on('poll', () => {
       this.loopFunc();
 
-      debug('Poll event');
+      console.log('Poll event');
 
       // This event is for checking if the service recipe is still actively
       // communicating with the client
@@ -109,7 +110,7 @@ class RecipeWebview {
 
         if (head) {
           head.append(styles);
-          debug('Append styles', styles);
+          console.log('Append styles', styles);
         }
       }
     });
@@ -121,13 +122,13 @@ class RecipeWebview {
         if (existsSync(file)) {
           return readFileSync(file, 'utf8');
         }
-        debug('Script not found', file);
+        console.log('Script not found', file);
         return null;
       }),
     ).then(scripts => {
       const scriptsFound = scripts.filter(script => script !== null);
       if (scriptsFound.length > 0) {
-        debug('Inject scripts to main world', scriptsFound);
+        console.log('Inject scripts to main world', scriptsFound);
         ipcRenderer.sendToHost('inject-js-unsafe', ...scriptsFound);
       }
     });
diff --git a/src/webview/notifications.ts b/src/webview/notifications.ts
index 5ae99424c..ff9c844d0 100644
--- a/src/webview/notifications.ts
+++ b/src/webview/notifications.ts
@@ -2,7 +2,8 @@ import { ipcRenderer } from 'electron';
 
 import { v1 as uuidV1 } from 'uuid';
 
-const debug = require('debug')('Ferdium:Notifications');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:Notifications');
 
 export class NotificationsHandler {
   onNotify = (data: { title: string; options: any; notificationId: string }) =>
@@ -10,7 +11,7 @@ export class NotificationsHandler {
 
   displayNotification(title: string, options: any) {
     return new Promise(resolve => {
-      debug('New notification', title, options);
+      console.log('New notification', title, options);
 
       const notificationId = uuidV1();
 
diff --git a/src/webview/recipe.js b/src/webview/recipe.js
index 4fcf28363..1f3017ccd 100644
--- a/src/webview/recipe.js
+++ b/src/webview/recipe.js
@@ -50,7 +50,8 @@ import {
 
 import { DEFAULT_APP_SETTINGS } from '../config';
 
-const debug = require('debug')('Ferdium:Plugin');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:Plugin');
 
 const badgeHandler = new BadgeHandler();
 
@@ -64,7 +65,7 @@ const notificationsHandler = new NotificationsHandler();
 const originalWindowOpen = window.open;
 
 window.open = (url, frameName, features) => {
-  debug('window.open', url, frameName, features);
+  console.log('window.open', url, frameName, features);
   if (!url) {
     // The service hasn't yet supplied a URL (as used in Skype).
     // Return a new dummy window object and wait for the service to change the properties
@@ -170,12 +171,12 @@ class RecipeController {
   async initialize() {
     for (const channel of Object.keys(this.ipcEvents)) {
       ipcRenderer.on(channel, (...args) => {
-        debug('Received IPC event for channel', channel, 'with', ...args);
+        console.log('Received IPC event for channel', channel, 'with', ...args);
         this[this.ipcEvents[channel]](...args);
       });
     }
 
-    debug('Send "hello" to host');
+    console.log('Send "hello" to host');
     setTimeout(() => ipcRenderer.sendToHost('hello'), 100);
 
     this.spellcheckingProvider = null;
@@ -198,9 +199,9 @@ class RecipeController {
   }
 
   loadRecipeModule(event, config, recipe) {
-    debug('loadRecipeModule');
+    console.log('loadRecipeModule');
     const modulePath = join(recipe.path, 'webview.js');
-    debug('module path', modulePath);
+    console.log('module path', modulePath);
     // Delete module from cache
     delete require.cache[require.resolve(modulePath)];
     try {
@@ -213,7 +214,7 @@ class RecipeController {
       if (existsSync(modulePath)) {
         // eslint-disable-next-line import/no-dynamic-require
         require(modulePath)(this.recipe, { ...config, recipe });
-        debug('Initialize Recipe', config, recipe);
+        console.log('Initialize Recipe', config, recipe);
       }
 
       this.settings.service = Object.assign(config, { recipe });
@@ -265,18 +266,18 @@ class RecipeController {
   }
 
   update() {
-    debug('enableSpellchecking', this.settings.app.enableSpellchecking);
-    debug('isDarkModeEnabled', this.settings.service.isDarkModeEnabled);
-    debug(
+    console.log('enableSpellchecking', this.settings.app.enableSpellchecking);
+    console.log('isDarkModeEnabled', this.settings.service.isDarkModeEnabled);
+    console.log(
       'System spellcheckerLanguage',
       this.settings.app.spellcheckerLanguage,
     );
-    debug(
+    console.log(
       'Service spellcheckerLanguage',
       this.settings.service.spellcheckerLanguage,
     );
-    debug('darkReaderSettigs', this.settings.service.darkReaderSettings);
-    debug('searchEngine', this.settings.app.searchEngine);
+    console.log('darkReaderSettigs', this.settings.service.darkReaderSettings);
+    console.log('searchEngine', this.settings.app.searchEngine);
 
     if (this.userscript && this.userscript.internal_setSettings) {
       this.userscript.internal_setSettings(this.settings);
@@ -284,10 +285,10 @@ class RecipeController {
 
     if (this.settings.app.enableSpellchecking) {
       let { spellcheckerLanguage } = this;
-      debug(`Setting spellchecker language to ${spellcheckerLanguage}`);
+      console.log(`Setting spellchecker language to ${spellcheckerLanguage}`);
       if (spellcheckerLanguage.includes('automatic')) {
         this.automaticLanguageDetection();
-        debug(
+        console.log(
           'Found `automatic` locale, falling back to user locale until detected',
           this.settings.app.locale,
         );
@@ -295,14 +296,14 @@ class RecipeController {
       }
       switchDict(spellcheckerLanguage, this.settings.service.id);
     } else {
-      debug('Disable spellchecker');
+      console.log('Disable spellchecker');
     }
 
     if (!this.recipe) {
       this.hasUpdatedBeforeRecipeLoaded = true;
     }
 
-    debug(
+    console.log(
       'Darkmode enabled?',
       this.settings.service.isDarkModeEnabled,
       'Dark theme active?',
@@ -322,11 +323,11 @@ class RecipeController {
       this.settings.service.isDarkModeEnabled &&
       this.settings.app.isDarkThemeActive !== false
     ) {
-      debug('Enable dark mode');
+      console.log('Enable dark mode');
 
       // Check if recipe has a custom dark mode handler
       if (this.recipe && this.recipe.darkModeHandler) {
-        debug('Using custom dark mode handler');
+        console.log('Using custom dark mode handler');
 
         // Remove other dark mode styles if they were already loaded
         if (this.hasUpdatedBeforeRecipeLoaded) {
@@ -337,7 +338,7 @@ class RecipeController {
 
         this.recipe.darkModeHandler(true, handlerConfig);
       } else if (darkModeStyleExists(this.settings.service.recipe.path)) {
-        debug('Injecting darkmode from recipe');
+        console.log('Injecting darkmode from recipe');
         injectDarkModeStyle(this.settings.service.recipe.path);
 
         // Make sure universal dark mode is disabled
@@ -347,7 +348,7 @@ class RecipeController {
         this.settings.app.universalDarkMode &&
         !ignoreList.includes(window.location.host)
       ) {
-        debug('Injecting Dark Reader');
+        console.log('Injecting Dark Reader');
 
         // Use Dark Reader instead
         const { brightness, contrast, sepia } =
@@ -361,8 +362,8 @@ class RecipeController {
         this.universalDarkModeInjected = true;
       }
     } else {
-      debug('Remove dark mode');
-      debug('DarkMode disabled - removing remaining styles');
+      console.log('Remove dark mode');
+      console.log('DarkMode disabled - removing remaining styles');
 
       if (this.recipe && this.recipe.darkModeHandler) {
         // Remove other dark mode styles if they were already loaded
@@ -374,10 +375,10 @@ class RecipeController {
 
         this.recipe.darkModeHandler(false, handlerConfig);
       } else if (isDarkModeStyleInjected()) {
-        debug('Removing injected darkmode from recipe');
+        console.log('Removing injected darkmode from recipe');
         removeDarkModeStyle();
       } else {
-        debug('Removing Dark Reader');
+        console.log('Removing Dark Reader');
 
         disableDarkMode();
         this.universalDarkModeInjected = false;
@@ -405,7 +406,7 @@ class RecipeController {
   }
 
   serviceIdEcho(event) {
-    debug('Received a service echo ping');
+    console.log('Received a service echo ping');
     event.sender.send('service-id', this.settings.service.id);
   }
 
@@ -427,7 +428,7 @@ class RecipeController {
         // Force a minimum length to get better detection results
         if (value.length < 25) return;
 
-        debug('Detecting language for', value);
+        console.log('Detecting language for', value);
         const locale = await ipcRenderer.invoke('detect-language', {
           sample: value,
         });
@@ -437,7 +438,7 @@ class RecipeController {
 
         const spellcheckerLocale =
           getSpellcheckerLocaleByFuzzyIdentifier(locale);
-        debug(
+        console.log(
           'Language detected reliably, setting spellchecker language to',
           spellcheckerLocale,
         );
diff --git a/src/webview/sessionHandler.ts b/src/webview/sessionHandler.ts
index e38368a18..a2906113d 100644
--- a/src/webview/sessionHandler.ts
+++ b/src/webview/sessionHandler.ts
@@ -1,4 +1,5 @@
-const debug = require('debug')('Ferdium:Plugin:SessionHandler');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:Plugin:SessionHandler');
 
 export class SessionHandler {
   async releaseServiceWorkers() {
@@ -7,10 +8,10 @@ export class SessionHandler {
         await window.navigator.serviceWorker.getRegistrations();
       for (const registration of registrations) {
         registration.unregister();
-        debug('ServiceWorker unregistered');
+        console.log('ServiceWorker unregistered');
       }
     } catch (error) {
-      debug(error);
+      console.log(error);
     }
   }
 }
diff --git a/src/webview/spellchecker.ts b/src/webview/spellchecker.ts
index bad2bb4ff..8cf16a7ba 100644
--- a/src/webview/spellchecker.ts
+++ b/src/webview/spellchecker.ts
@@ -2,7 +2,8 @@ import { ipcRenderer } from 'electron';
 import { SPELLCHECKER_LOCALES } from '../i18n/languages';
 import { isMac } from '../environment';
 
-const debug = require('debug')('Ferdium:spellchecker');
+// TODO: Go back to 'debug' from 'console.log' when https://github.com/electron/electron/issues/31689 is fixed
+// const debug = require('debug')('Ferdium:spellchecker');
 
 export function getSpellcheckerLocaleByFuzzyIdentifier(identifier: string) {
   const locales = Object.keys(SPELLCHECKER_LOCALES).filter(
@@ -16,14 +17,14 @@ export function getSpellcheckerLocaleByFuzzyIdentifier(identifier: string) {
 
 export function switchDict(fuzzyLocale: string, serviceId: string) {
   if (isMac) {
-    debug('Ignoring dictionary changes on macOS');
+    console.log('Ignoring dictionary changes on macOS');
     return;
   }
 
-  debug(`Setting spellchecker locale from: ${fuzzyLocale}`);
+  console.log(`Setting spellchecker locale from: ${fuzzyLocale}`);
   const locale = getSpellcheckerLocaleByFuzzyIdentifier(fuzzyLocale);
   if (locale) {
-    debug(`Sending spellcheck locales to host: ${locale}`);
+    console.log(`Sending spellcheck locales to host: ${locale}`);
     ipcRenderer.send('set-spellchecker-locales', { locale, serviceId });
   }
 }