12 files changed, 365 insertions, 299 deletions

diff --git a/src/stores/AppStore.js b/src/stores/AppStore.js
index 469e7519e..85f74a91e 100644
--- a/src/stores/AppStore.js
+++ b/src/stores/AppStore.js
@@ -251,16 +251,14 @@ export default class AppStore extends Store {
     // macOS catalina notifications hack
     // notifications got stuck after upgrade but forcing a notification
     // via `new Notification` triggered the permission request
-    if (isMac) {
-      if (!localStorage.getItem(CATALINA_NOTIFICATION_HACK_KEY)) {
-        debug('Triggering macOS Catalina notification permission trigger');
-        // eslint-disable-next-line no-new
-        new window.Notification('Welcome to Ferdi 5', {
-          body: 'Have a wonderful day & happy messaging.',
-        });
+    if (isMac && !localStorage.getItem(CATALINA_NOTIFICATION_HACK_KEY)) {
+      debug('Triggering macOS Catalina notification permission trigger');
+      // eslint-disable-next-line no-new
+      new window.Notification('Welcome to Ferdi 5', {
+        body: 'Have a wonderful day & happy messaging.',
+      });
 
-        localStorage.setItem(CATALINA_NOTIFICATION_HACK_KEY, true);
-      }
+      localStorage.setItem(CATALINA_NOTIFICATION_HACK_KEY, true);
     }
   }
 
@@ -325,7 +323,7 @@ export default class AppStore extends Store {
 
     debug('New notification', title, options);
 
-    notification.onclick = () => {
+    notification.addEventListener('click', () => {
       if (serviceId) {
         this.actions.service.sendIPCMessage({
           channel: `notification-onclick:${notificationId}`,
@@ -346,7 +344,7 @@ export default class AppStore extends Store {
 
         debug('Notification click handler');
       }
-    };
+    });
   }
 
   @action _setBadge({ unreadDirectMessageCount, unreadIndirectMessageCount }) {
@@ -360,7 +358,7 @@ export default class AppStore extends Store {
     ) {
       indicator = 0;
     } else {
-      indicator = parseInt(indicator, 10);
+      indicator = Number.parseInt(indicator, 10);
     }
 
     ipcRenderer.send('updateAppIndicator', {
@@ -379,8 +377,8 @@ export default class AppStore extends Store {
         debug('disabling launch on startup');
         autoLauncher.disable();
       }
-    } catch (err) {
-      console.warn(err);
+    } catch (error) {
+      console.warn(error);
     }
   }
 
@@ -438,7 +436,7 @@ export default class AppStore extends Store {
     const allServiceIds = await getServiceIdsFromPartitions();
     const allOrphanedServiceIds = allServiceIds.filter(
       id =>
-        !this.stores.services.all.find(
+        !this.stores.services.all.some(
           s => id.replace('service-', '') === s.id,
         ),
     );
@@ -447,8 +445,8 @@ export default class AppStore extends Store {
       await Promise.all(
         allOrphanedServiceIds.map(id => removeServicePartitionDirectory(id)),
       );
-    } catch (ex) {
-      console.log('Error while deleting service partition directory - ', ex);
+    } catch (error) {
+      console.log('Error while deleting service partition directory -', error);
     }
     await Promise.all(
       this.stores.services.all.map(s =>
diff --git a/src/stores/FeaturesStore.js b/src/stores/FeaturesStore.js
index 1d50dd714..8e0134d7f 100644
--- a/src/stores/FeaturesStore.js
+++ b/src/stores/FeaturesStore.js
@@ -51,7 +51,9 @@ export default class FeaturesStore extends Store {
       let requestResult = {};
       try {
         requestResult = this.featuresRequest.execute().result;
-      } catch (e) {} // eslint-disable-line no-empty
+      } catch (error) {
+        console.error(error);
+      }
       Object.assign(features, requestResult);
     }
     runInAction('FeaturesStore::_updateFeatures', () => {
@@ -69,15 +71,15 @@ export default class FeaturesStore extends Store {
   }
 
   _setupFeatures() {
-    serviceProxy(this.stores, this.actions);
-    basicAuth(this.stores, this.actions);
+    serviceProxy(this.stores);
+    basicAuth();
     workspaces(this.stores, this.actions);
-    quickSwitch(this.stores, this.actions);
-    nightlyBuilds(this.stores, this.actions);
-    publishDebugInfo(this.stores, this.actions);
+    quickSwitch();
+    nightlyBuilds();
+    publishDebugInfo();
     settingsWS(this.stores, this.actions);
     communityRecipes(this.stores, this.actions);
     todos(this.stores, this.actions);
-    appearance(this.stores, this.actions);
+    appearance(this.stores);
   }
 }
diff --git a/src/stores/GlobalErrorStore.js b/src/stores/GlobalErrorStore.js
index aacaa247f..7cbfdc608 100644
--- a/src/stores/GlobalErrorStore.js
+++ b/src/stores/GlobalErrorStore.js
@@ -12,9 +12,9 @@ export default class GlobalErrorStore extends Store {
   constructor(...args) {
     super(...args);
 
-    window.onerror = (...errorArgs) => {
+    window.addEventListener('error', (...errorArgs) => {
       this._handleConsoleError.call(this, ['error', ...errorArgs]);
-    };
+    });
 
     const origConsoleError = console.error;
     window.console.error = (...errorArgs) => {
@@ -38,7 +38,7 @@ export default class GlobalErrorStore extends Store {
   }
 
   _handleConsoleError(type, error, url, line) {
-    if (typeof type === 'object' && type.length && type.length >= 1) {
+    if (typeof type === 'object' && type.length > 0) {
       this.messages.push({
         type: type[0],
         info: type,
@@ -53,14 +53,14 @@ export default class GlobalErrorStore extends Store {
     }
   }
 
-  _handleRequests = action(async (request) => {
+  _handleRequests = action(async request => {
     if (request.isError) {
       this.error = request.error;
 
       if (request.error.json) {
         try {
           this.response = await request.error.json();
-        } catch (error) {
+        } catch {
           this.response = {};
         }
         if (this.error.status === 401) {
diff --git a/src/stores/RecipesStore.js b/src/stores/RecipesStore.js
index d2acebb75..bfbdc57a8 100644
--- a/src/stores/RecipesStore.js
+++ b/src/stores/RecipesStore.js
@@ -25,9 +25,7 @@ export default class RecipesStore extends Store {
     this.actions.recipe.update.listen(this._update.bind(this));
 
     // Reactions
-    this.registerReactions([
-      this._checkIfRecipeIsInstalled.bind(this),
-    ]);
+    this.registerReactions([this._checkIfRecipeIsInstalled.bind(this)]);
   }
 
   setup() {
@@ -39,7 +37,10 @@ export default class RecipesStore extends Store {
   }
 
   @computed get active() {
-    const match = matchRoute('/settings/services/add/:id', this.stores.router.location.pathname);
+    const match = matchRoute(
+      '/settings/services/add/:id',
+      this.stores.router.location.pathname,
+    );
     if (match) {
       const activeRecipe = this.one(match.id);
       if (activeRecipe) {
@@ -53,11 +54,11 @@ export default class RecipesStore extends Store {
   }
 
   @computed get recipeIdForServices() {
-    return this.stores.services.all.map((s) => s.recipe.id);
+    return this.stores.services.all.map(s => s.recipe.id);
   }
 
   one(id) {
-    return this.all.find((recipe) => recipe.id === id);
+    return this.all.find(recipe => recipe.id === id);
   }
 
   isInstalled(id) {
@@ -77,41 +78,43 @@ export default class RecipesStore extends Store {
     const recipes = {};
 
     // Hackfix, reference this.all to fetch services
-    debug(`Check Recipe updates for ${this.all.map((recipe) => recipe.id)}`);
+    debug(`Check Recipe updates for ${this.all.map(recipe => recipe.id)}`);
 
-    recipeIds.forEach((r) => {
+    for (const r of recipeIds) {
       const recipe = this.one(r);
       recipes[r] = recipe.version;
-    });
+    }
 
     if (Object.keys(recipes).length === 0) return;
 
-    const remoteUpdates = await this.getRecipeUpdatesRequest.execute(recipes)._promise;
+    const remoteUpdates = await this.getRecipeUpdatesRequest.execute(recipes)
+      ._promise;
 
     // Check for local updates
     const allJsonFile = asarRecipesPath('all.json');
     const allJson = readJSONSync(allJsonFile);
     const localUpdates = [];
 
-    Object.keys(recipes).forEach((recipe) => {
+    for (const recipe of Object.keys(recipes)) {
       const version = recipes[recipe];
 
       // Find recipe in local recipe repository
-      const localRecipe = allJson.find((r) => r.id === recipe);
+      const localRecipe = allJson.find(r => r.id === recipe);
 
       if (localRecipe && semver.lt(version, localRecipe.version)) {
         localUpdates.push(recipe);
       }
-    });
+    }
 
-    const updates = [
-      ...remoteUpdates,
-      ...localUpdates,
-    ];
-    debug('Got update information (local, remote):', localUpdates, remoteUpdates);
+    const updates = [...remoteUpdates, ...localUpdates];
+    debug(
+      'Got update information (local, remote):',
+      localUpdates,
+      remoteUpdates,
+    );
 
     const length = updates.length - 1;
-    const syncUpdate = async (i) => {
+    const syncUpdate = async i => {
       const update = updates[i];
 
       this.actions.recipe.install({ recipeId: update });
@@ -134,7 +137,9 @@ export default class RecipesStore extends Store {
   async _checkIfRecipeIsInstalled() {
     const { router } = this.stores;
 
-    const match = router.location && matchRoute('/settings/services/add/:id', router.location.pathname);
+    const match =
+      router.location &&
+      matchRoute('/settings/services/add/:id', router.location.pathname);
     if (match) {
       const recipeId = match.id;
 
@@ -142,9 +147,11 @@ export default class RecipesStore extends Store {
         router.push('/settings/recipes');
         debug(`Recipe ${recipeId} is not installed, trying to install it`);
 
-        const recipe = await this.installRecipeRequest.execute(recipeId)._promise;
+        const recipe = await this.installRecipeRequest.execute(recipeId)
+          ._promise;
         if (recipe) {
-          await this.allRecipesRequest.invalidate({ immediately: true })._promise;
+          await this.allRecipesRequest.invalidate({ immediately: true })
+            ._promise;
           router.push(`/settings/services/add/${recipeId}`);
         } else {
           router.push('/settings/recipes');
diff --git a/src/stores/RequestStore.js b/src/stores/RequestStore.js
index a92f4c685..6d2f2ef91 100644
--- a/src/stores/RequestStore.js
+++ b/src/stores/RequestStore.js
@@ -13,7 +13,7 @@ export default class RequestStore extends Store {
 
   @observable showRequiredRequestsError = false;
 
-  @observable localServerPort = 45569;
+  @observable localServerPort = 45_569;
 
   retries = 0;
 
@@ -22,11 +22,11 @@ export default class RequestStore extends Store {
   constructor(...args) {
     super(...args);
 
-    this.actions.requests.retryRequiredRequests.listen(this._retryRequiredRequests.bind(this));
+    this.actions.requests.retryRequiredRequests.listen(
+      this._retryRequiredRequests.bind(this),
+    );
 
-    this.registerReactions([
-      this._autoRetry.bind(this),
-    ]);
+    this.registerReactions([this._autoRetry.bind(this)]);
   }
 
   setup() {
@@ -41,13 +41,11 @@ export default class RequestStore extends Store {
   }
 
   @computed get areRequiredRequestsSuccessful() {
-    return !this.userInfoRequest.isError
-      && !this.servicesRequest.isError;
+    return !this.userInfoRequest.isError && !this.servicesRequest.isError;
   }
 
   @computed get areRequiredRequestsLoading() {
-    return this.userInfoRequest.isExecuting
-      || this.servicesRequest.isExecuting;
+    return this.userInfoRequest.isExecuting || this.servicesRequest.isExecuting;
   }
 
   @action _retryRequiredRequests() {
@@ -67,7 +65,7 @@ export default class RequestStore extends Store {
         }
 
         this._autoRetry();
-        debug(`Retry required requests delayed in ${(delay) / 1000}s`);
+        debug(`Retry required requests delayed in ${delay / 1000}s`);
       }, delay);
     }
   }
diff --git a/src/stores/ServicesStore.js b/src/stores/ServicesStore.js
index 75bc71388..67fd4103f 100644
--- a/src/stores/ServicesStore.js
+++ b/src/stores/ServicesStore.js
@@ -10,7 +10,10 @@ import Request from './lib/Request';
 import CachedRequest from './lib/CachedRequest';
 import { matchRoute } from '../helpers/routing-helpers';
 import { isInTimeframe } from '../helpers/schedule-helpers';
-import { getRecipeDirectory, getDevRecipeDirectory } from '../helpers/recipe-helpers';
+import {
+  getRecipeDirectory,
+  getDevRecipeDirectory,
+} from '../helpers/recipe-helpers';
 import { workspaceStore } from '../features/workspaces';
 import { KEEP_WS_LOADED_USID } from '../config';
 import { SPELLCHECKER_LOCALES } from '../i18n/languages';
@@ -125,63 +128,49 @@ export default class ServicesStore extends Store {
   setup() {
     // Single key reactions for the sake of your CPU
     reaction(
-      () => (
-        this.stores.settings.app.enableSpellchecking
-      ),
+      () => this.stores.settings.app.enableSpellchecking,
       () => {
         this._shareSettingsWithServiceProcess();
       },
     );
 
     reaction(
-      () => (
-        this.stores.settings.app.spellcheckerLanguage
-      ),
+      () => this.stores.settings.app.spellcheckerLanguage,
       () => {
         this._shareSettingsWithServiceProcess();
       },
     );
 
     reaction(
-      () => (
-        this.stores.settings.app.darkMode
-      ),
+      () => this.stores.settings.app.darkMode,
       () => {
         this._shareSettingsWithServiceProcess();
       },
     );
 
     reaction(
-      () => (
-        this.stores.settings.app.adaptableDarkMode
-      ),
+      () => this.stores.settings.app.adaptableDarkMode,
       () => {
         this._shareSettingsWithServiceProcess();
       },
     );
 
     reaction(
-      () => (
-        this.stores.settings.app.universalDarkMode
-      ),
+      () => this.stores.settings.app.universalDarkMode,
       () => {
         this._shareSettingsWithServiceProcess();
       },
     );
 
     reaction(
-      () => (
-        this.stores.settings.app.searchEngine
-      ),
+      () => this.stores.settings.app.searchEngine,
       () => {
         this._shareSettingsWithServiceProcess();
       },
     );
 
     reaction(
-      () => (
-        this.stores.settings.app.clipboardNotifications
-      ),
+      () => this.stores.settings.app.clipboardNotifications,
       () => {
         this._shareSettingsWithServiceProcess();
       },
@@ -215,12 +204,12 @@ export default class ServicesStore extends Store {
    * Run various maintenance tasks on services
    */
   _serviceMaintenance() {
-    this.enabled.forEach(service => {
+    for (const service of this.enabled) {
       // Defines which services should be hibernated or woken up
       if (!service.isActive) {
         if (
           !service.lastHibernated &&
-          (Date.now() - service.lastUsed) >
+          Date.now() - service.lastUsed >
             ms(`${this.stores.settings.all.app.hibernationStrategy}s`)
         ) {
           // If service is stale, hibernate it.
@@ -230,8 +219,8 @@ export default class ServicesStore extends Store {
         if (
           service.lastHibernated &&
           Number(this.stores.settings.all.app.wakeUpStrategy) > 0 &&
-          (Date.now() - service.lastHibernated) >
-          ms(`${this.stores.settings.all.app.wakeUpStrategy}s`)
+          Date.now() - service.lastHibernated >
+            ms(`${this.stores.settings.all.app.wakeUpStrategy}s`)
         ) {
           // If service is in hibernation and the wakeup time has elapsed, wake it.
           this._awake({ serviceId: service.id });
@@ -240,7 +229,7 @@ export default class ServicesStore extends Store {
 
       if (
         service.lastPoll &&
-        (service.lastPoll - service.lastPollAnswer) > ms('1m')
+        service.lastPoll - service.lastPollAnswer > ms('1m')
       ) {
         // If service did not reply for more than 1m try to reload.
         if (!service.isActive) {
@@ -261,7 +250,7 @@ export default class ServicesStore extends Store {
         service.lostRecipeConnection = false;
         service.lostRecipeReloadAttempt = 0;
       }
-    });
+    }
   }
 
   // Computed props
@@ -270,8 +259,7 @@ export default class ServicesStore extends Store {
       const services = this.allServicesRequest.execute().result;
       if (services) {
         return observable(
-          services
-            .slice()
+          [...services]
             .slice()
             .sort((a, b) => a.order - b.order)
             .map((s, index) => {
@@ -318,11 +306,11 @@ export default class ServicesStore extends Store {
         // Check if workspace needs to be kept loaded
         if (workspace.services.includes(KEEP_WS_LOADED_USID)) {
           // Get services for workspace
-          const serviceIDs = workspace.services.filter(
-            i => i !== KEEP_WS_LOADED_USID,
+          const serviceIDs = new Set(
+            workspace.services.filter(i => i !== KEEP_WS_LOADED_USID),
           );
           const wsServices = filteredServices.filter(service =>
-            serviceIDs.includes(service.id),
+            serviceIDs.has(service.id),
           );
 
           displayedServices = [...displayedServices, ...wsServices];
@@ -410,12 +398,14 @@ export default class ServicesStore extends Store {
       customIcon: false,
       isDarkModeEnabled: false,
       spellcheckerLanguage:
-          SPELLCHECKER_LOCALES[this.stores.settings.app.spellcheckerLanguage],
+        SPELLCHECKER_LOCALES[this.stores.settings.app.spellcheckerLanguage],
       userAgentPref: '',
       ...serviceData,
     };
 
-    const data = skipCleanup ? serviceData : this._cleanUpTeamIdAndCustomUrl(recipeId, serviceData);
+    const data = skipCleanup
+      ? serviceData
+      : this._cleanUpTeamIdAndCustomUrl(recipeId, serviceData);
 
     const response = await this.createServiceRequest.execute(recipeId, data)
       ._promise;
@@ -562,7 +552,8 @@ export default class ServicesStore extends Store {
   // Write your scripts here
   console.log("Hello, World!", config);
 };
-`);
+`,
+        );
       }
     } else {
       ensureFileSync(filePath);
@@ -580,9 +571,9 @@ export default class ServicesStore extends Store {
     if (!keepActiveRoute) this.stores.router.push('/');
     const service = this.one(serviceId);
 
-    this.all.forEach(s => {
+    for (const s of this.all) {
       s.isActive = false;
-    });
+    }
     service.isActive = true;
     this._awake({ serviceId: service.id });
 
@@ -618,9 +609,9 @@ export default class ServicesStore extends Store {
       this.allDisplayed.length,
     );
 
-    this.all.forEach(s => {
+    for (const s of this.all) {
       s.isActive = false;
-    });
+    }
     this.allDisplayed[nextIndex].isActive = true;
   }
 
@@ -631,9 +622,9 @@ export default class ServicesStore extends Store {
       this.allDisplayed.length,
     );
 
-    this.all.forEach(s => {
+    for (const s of this.all) {
       s.isActive = false;
-    });
+    }
     this.allDisplayed[prevIndex].isActive = true;
   }
 
@@ -699,101 +690,128 @@ export default class ServicesStore extends Store {
   @action _handleIPCMessage({ serviceId, channel, args }) {
     const service = this.one(serviceId);
 
-    if (channel === 'hello') {
-      debug('Received hello event from', serviceId);
-
-      this._initRecipePolling(service.id);
-      this._initializeServiceRecipeInWebview(serviceId);
-      this._shareSettingsWithServiceProcess();
-    } else if (channel === 'alive') {
-      service.lastPollAnswer = Date.now();
-    } else if (channel === 'message-counts') {
-      debug(`Received unread message info from '${serviceId}'`, args[0]);
+    // eslint-disable-next-line default-case
+    switch (channel) {
+      case 'hello': {
+        debug('Received hello event from', serviceId);
 
-      this.actions.service.setUnreadMessageCount({
-        serviceId,
-        count: {
-          direct: args[0].direct,
-          indirect: args[0].indirect,
-        },
-      });
-    } else if (channel === 'notification') {
-      const { options } = args[0];
+        this._initRecipePolling(service.id);
+        this._initializeServiceRecipeInWebview(serviceId);
+        this._shareSettingsWithServiceProcess();
 
-      // Check if we are in scheduled Do-not-Disturb time
-      const { scheduledDNDEnabled, scheduledDNDStart, scheduledDNDEnd } =
-        this.stores.settings.all.app;
+        break;
+      }
+      case 'alive': {
+        service.lastPollAnswer = Date.now();
 
-      if (
-        scheduledDNDEnabled &&
-        isInTimeframe(scheduledDNDStart, scheduledDNDEnd)
-      ) {
-        return;
+        break;
       }
+      case 'message-counts': {
+        debug(`Received unread message info from '${serviceId}'`, args[0]);
 
-      if (
-        service.recipe.hasNotificationSound ||
-        service.isMuted ||
-        this.stores.settings.all.app.isAppMuted
-      ) {
-        Object.assign(options, {
-          silent: true,
+        this.actions.service.setUnreadMessageCount({
+          serviceId,
+          count: {
+            direct: args[0].direct,
+            indirect: args[0].indirect,
+          },
         });
+
+        break;
       }
+      case 'notification': {
+        const { options } = args[0];
 
-      if (service.isNotificationEnabled) {
-        let title = `Notification from ${service.name}`;
-        if (!this.stores.settings.all.app.privateNotifications) {
-          options.body = typeof options.body === 'string' ? options.body : '';
-          title =
-            typeof args[0].title === 'string' ? args[0].title : service.name;
-        } else {
-          // Remove message data from notification in private mode
-          options.body = '';
-          options.icon = '/assets/img/notification-badge.gif';
+        // Check if we are in scheduled Do-not-Disturb time
+        const { scheduledDNDEnabled, scheduledDNDStart, scheduledDNDEnd } =
+          this.stores.settings.all.app;
+
+        if (
+          scheduledDNDEnabled &&
+          isInTimeframe(scheduledDNDStart, scheduledDNDEnd)
+        ) {
+          return;
         }
 
-        console.log(title, options);
+        if (
+          service.recipe.hasNotificationSound ||
+          service.isMuted ||
+          this.stores.settings.all.app.isAppMuted
+        ) {
+          Object.assign(options, {
+            silent: true,
+          });
+        }
 
-        this.actions.app.notify({
-          notificationId: args[0].notificationId,
-          title,
-          options,
-          serviceId,
-        });
+        if (service.isNotificationEnabled) {
+          let title = `Notification from ${service.name}`;
+          if (!this.stores.settings.all.app.privateNotifications) {
+            options.body = typeof options.body === 'string' ? options.body : '';
+            title =
+              typeof args[0].title === 'string' ? args[0].title : service.name;
+          } else {
+            // Remove message data from notification in private mode
+            options.body = '';
+            options.icon = '/assets/img/notification-badge.gif';
+          }
+
+          console.log(title, options);
+
+          this.actions.app.notify({
+            notificationId: args[0].notificationId,
+            title,
+            options,
+            serviceId,
+          });
+        }
+
+        break;
       }
-    } else if (channel === 'avatar') {
-      const url = args[0];
-      if (service.iconUrl !== url && !service.hasCustomUploadedIcon) {
-        service.customIconUrl = url;
+      case 'avatar': {
+        const url = args[0];
+        if (service.iconUrl !== url && !service.hasCustomUploadedIcon) {
+          service.customIconUrl = url;
+
+          this.actions.service.updateService({
+            serviceId,
+            serviceData: {
+              customIconUrl: url,
+            },
+            redirect: false,
+          });
+        }
 
-        this.actions.service.updateService({
-          serviceId,
-          serviceData: {
-            customIconUrl: url,
-          },
-          redirect: false,
-        });
+        break;
       }
-    } else if (channel === 'new-window') {
-      const url = args[0];
+      case 'new-window': {
+        const url = args[0];
 
-      this.actions.app.openExternalUrl({ url });
-    } else if (channel === 'set-service-spellchecker-language') {
-      if (!args) {
-        console.warn('Did not receive locale');
-      } else {
-        this.actions.service.updateService({
-          serviceId,
-          serviceData: {
-            spellcheckerLanguage: args[0] === 'reset' ? '' : args[0],
-          },
-          redirect: false,
-        });
+        this.actions.app.openExternalUrl({ url });
+
+        break;
+      }
+      case 'set-service-spellchecker-language': {
+        if (!args) {
+          console.warn('Did not receive locale');
+        } else {
+          this.actions.service.updateService({
+            serviceId,
+            serviceData: {
+              spellcheckerLanguage: args[0] === 'reset' ? '' : args[0],
+            },
+            redirect: false,
+          });
+        }
+
+        break;
+      }
+      case 'feature:todos': {
+        Object.assign(args[0].data, { serviceId });
+        this.actions.todos.handleHostMessage(args[0]);
+
+        break;
       }
-    } else if (channel === 'feature:todos') {
-      Object.assign(args[0].data, { serviceId });
-      this.actions.todos.handleHostMessage(args[0]);
+      // No default
     }
   }
 
@@ -809,13 +827,13 @@ export default class ServicesStore extends Store {
   }
 
   @action _sendIPCMessageToAllServices({ channel, args }) {
-    this.all.forEach(s =>
+    for (const s of this.all) {
       this.actions.service.sendIPCMessage({
         serviceId: s.id,
         channel,
         args,
-      }),
-    );
+      });
+    }
   }
 
   @action _openWindow({ event }) {
@@ -863,11 +881,11 @@ export default class ServicesStore extends Store {
   }
 
   @action _reloadAll() {
-    this.enabled.forEach(s =>
+    for (const s of this.enabled) {
       this._reload({
         serviceId: s.id,
-      }),
-    );
+      });
+    }
   }
 
   @action _reloadUpdatedServices() {
@@ -901,17 +919,17 @@ export default class ServicesStore extends Store {
 
     const services = {};
     // TODO: simplify this
-    this.all.forEach((s, index) => {
+    for (const [index] of this.all.entries()) {
       services[this.all[index].id] = index;
-    });
+    }
 
     this.reorderServicesRequest.execute(services);
     this.allServicesRequest.patch(data => {
-      data.forEach(s => {
+      for (const s of data) {
         const service = s;
 
         service.order = services[s.id];
-      });
+      }
     });
   }
 
@@ -1001,13 +1019,14 @@ export default class ServicesStore extends Store {
       }`,
     );
 
+    // eslint-disable-next-line unicorn/consistent-function-scoping
     const resetTimer = service => {
       service.lastPollAnswer = Date.now();
       service.lastPoll = Date.now();
     };
 
     if (!serviceId) {
-      this.allDisplayed.forEach(service => resetTimer(service));
+      for (const service of this.allDisplayed) resetTimer(service);
     } else {
       const service = this.one(serviceId);
       if (service) {
@@ -1043,7 +1062,7 @@ export default class ServicesStore extends Store {
 
   _mapActiveServiceToServiceModelReaction() {
     const { activeService } = this.stores.settings.all.service;
-    if (this.allDisplayed.length) {
+    if (this.allDisplayed.length > 0) {
       this.allDisplayed.map(service =>
         Object.assign(service, {
           isActive: activeService
@@ -1102,14 +1121,14 @@ export default class ServicesStore extends Store {
     const { enabled } = this;
     const { isAppMuted } = this.stores.settings.app;
 
-    enabled.forEach(service => {
+    for (const service of enabled) {
       const { isAttached } = service;
       const isMuted = isAppMuted || service.isMuted;
 
       if (isAttached) {
         service.webview.audioMuted = isMuted;
       }
-    });
+    }
   }
 
   _shareSettingsWithServiceProcess() {
@@ -1151,7 +1170,7 @@ export default class ServicesStore extends Store {
 
     if (
       this.allDisplayed.findIndex(service => service.isActive) === -1 &&
-      this.allDisplayed.length !== 0
+      this.allDisplayed.length > 0
     ) {
       debug('No active service found, setting active service to index 0');
 
diff --git a/src/stores/SettingsStore.js b/src/stores/SettingsStore.js
index 9aade974c..690a18374 100644
--- a/src/stores/SettingsStore.js
+++ b/src/stores/SettingsStore.js
@@ -1,11 +1,11 @@
 import { ipcRenderer } from 'electron';
 import { getCurrentWindow } from '@electron/remote';
-import {
-  action, computed, observable, reaction,
-} from 'mobx';
+import { action, computed, observable, reaction } from 'mobx';
 import localStorage from 'mobx-localstorage';
 import {
-  FILE_SYSTEM_SETTINGS_TYPES, LOCAL_SERVER, SEARCH_ENGINE_DDG,
+  FILE_SYSTEM_SETTINGS_TYPES,
+  LOCAL_SERVER,
+  SEARCH_ENGINE_DDG,
 } from '../config';
 import { API, DEFAULT_APP_SETTINGS } from '../environment';
 import { getLocale } from '../helpers/i18n-helpers';
@@ -17,7 +17,10 @@ import Store from './lib/Store';
 const debug = require('debug')('Ferdi:SettingsStore');
 
 export default class SettingsStore extends Store {
-  @observable updateAppSettingsRequest = new Request(this.api.local, 'updateAppSettings');
+  @observable updateAppSettingsRequest = new Request(
+    this.api.local,
+    'updateAppSettings',
+  );
 
   startup = true;
 
@@ -40,9 +43,7 @@ export default class SettingsStore extends Store {
     await this._migrate();
 
     reaction(
-      () => (
-        this.all.app.autohideMenuBar
-      ),
+      () => this.all.app.autohideMenuBar,
       () => {
         const currentWindow = getCurrentWindow();
         currentWindow.setMenuBarVisibility(!this.all.app.autohideMenuBar);
@@ -51,10 +52,8 @@ export default class SettingsStore extends Store {
     );
 
     reaction(
-      () => (
-        this.all.app.server
-      ),
-      (server) => {
+      () => this.all.app.server,
+      server => {
         if (server === LOCAL_SERVER) {
           ipcRenderer.send('startLocalServer');
         }
@@ -65,7 +64,10 @@ export default class SettingsStore extends Store {
     // Inactivity lock timer
     let inactivityTimer;
     getCurrentWindow().on('blur', () => {
-      if (this.all.app.lockingFeatureEnabled && this.all.app.inactivityLock !== 0) {
+      if (
+        this.all.app.lockingFeatureEnabled &&
+        this.all.app.inactivityLock !== 0
+      ) {
         inactivityTimer = setTimeout(() => {
           this.actions.settings.update({
             type: 'app',
@@ -84,7 +86,11 @@ export default class SettingsStore extends Store {
 
     ipcRenderer.on('appSettings', (event, resp) => {
       // Lock on startup if enabled in settings
-      if (this.startup && resp.type === 'app' && resp.data.lockingFeatureEnabled) {
+      if (
+        this.startup &&
+        resp.type === 'app' &&
+        resp.data.lockingFeatureEnabled
+      ) {
         this.startup = false;
         process.nextTick(() => {
           if (!this.all.app.locked) {
@@ -97,9 +103,9 @@ export default class SettingsStore extends Store {
       ipcRenderer.send('initialAppSettings', resp);
     });
 
-    this.fileSystemSettingsTypes.forEach((type) => {
+    for (const type of this.fileSystemSettingsTypes) {
       ipcRenderer.send('getAppSettings', type);
-    });
+    }
   }
 
   @computed get app() {
@@ -111,15 +117,19 @@ export default class SettingsStore extends Store {
   }
 
   @computed get service() {
-    return localStorage.getItem('service') || {
-      activeService: '',
-    };
+    return (
+      localStorage.getItem('service') || {
+        activeService: '',
+      }
+    );
   }
 
   @computed get stats() {
-    return localStorage.getItem('stats') || {
-      activeService: '',
-    };
+    return (
+      localStorage.getItem('stats') || {
+        activeService: '',
+      }
+    );
   }
 
   @computed get migration() {
@@ -230,9 +240,7 @@ export default class SettingsStore extends Store {
     });
 
     this._ensureMigrationAndMarkDone('5.4.4-beta.2-settings', () => {
-      const {
-        showServiceNavigationBar,
-      } = this.all.app;
+      const { showServiceNavigationBar } = this.all.app;
 
       this.actions.settings.update({
         type: 'app',
@@ -248,7 +256,7 @@ export default class SettingsStore extends Store {
         data: {
           todoServer: 'isUsingCustomTodoService',
           customTodoServer: legacySettings.todoServer,
-          automaticUpdates: !(legacySettings.noUpdates),
+          automaticUpdates: !legacySettings.noUpdates,
         },
       });
 
diff --git a/src/stores/UserStore.js b/src/stores/UserStore.js
index 2e009893a..e3d57c662 100644
--- a/src/stores/UserStore.js
+++ b/src/stores/UserStore.js
@@ -46,7 +46,10 @@ export default class UserStore extends Store {
 
   @observable updateUserInfoRequest = new Request(this.api.user, 'updateInfo');
 
-  @observable getLegacyServicesRequest = new CachedRequest(this.api.user, 'getLegacyServices');
+  @observable getLegacyServicesRequest = new CachedRequest(
+    this.api.user,
+    'getLegacyServices',
+  );
 
   @observable deleteAccountRequest = new CachedRequest(this.api.user, 'delete');
 
@@ -81,13 +84,17 @@ export default class UserStore extends Store {
 
     // Register action handlers
     this.actions.user.login.listen(this._login.bind(this));
-    this.actions.user.retrievePassword.listen(this._retrievePassword.bind(this));
+    this.actions.user.retrievePassword.listen(
+      this._retrievePassword.bind(this),
+    );
     this.actions.user.logout.listen(this._logout.bind(this));
     this.actions.user.signup.listen(this._signup.bind(this));
     this.actions.user.invite.listen(this._invite.bind(this));
     this.actions.user.update.listen(this._update.bind(this));
     this.actions.user.resetStatus.listen(this._resetStatus.bind(this));
-    this.actions.user.importLegacyServices.listen(this._importLegacyServices.bind(this));
+    this.actions.user.importLegacyServices.listen(
+      this._importLegacyServices.bind(this),
+    );
     this.actions.user.delete.listen(this._delete.bind(this));
 
     // Reactions
@@ -176,7 +183,14 @@ export default class UserStore extends Store {
   }
 
   @action async _signup({
-    firstname, lastname, email, password, accountType, company, plan, currency,
+    firstname,
+    lastname,
+    email,
+    password,
+    accountType,
+    company,
+    plan,
+    currency,
   }) {
     const authToken = await this.signupRequest.execute({
       firstname,
@@ -205,7 +219,7 @@ export default class UserStore extends Store {
   }
 
   @action async _invite({ invites }) {
-    const data = invites.filter((invite) => invite.email !== '');
+    const data = invites.filter(invite => invite.email !== '');
 
     const response = await this.inviteRequest.execute(data)._promise;
 
@@ -220,7 +234,8 @@ export default class UserStore extends Store {
   @action async _update({ userData }) {
     if (!this.isLoggedIn) return;
 
-    const response = await this.updateUserInfoRequest.execute(userData)._promise;
+    const response = await this.updateUserInfoRequest.execute(userData)
+      ._promise;
 
     this.getUserInfoRequest.patch(() => response.data);
     this.actionStatus = response.status || [];
@@ -250,19 +265,27 @@ export default class UserStore extends Store {
     this.isImportLegacyServicesExecuting = true;
 
     // Reduces recipe duplicates
-    const recipes = services.filter((obj, pos, arr) => arr.map((mapObj) => mapObj.recipe.id).indexOf(obj.recipe.id) === pos).map((s) => s.recipe.id);
+    const recipes = services
+      .filter(
+        (obj, pos, arr) =>
+          arr.map(mapObj => mapObj.recipe.id).indexOf(obj.recipe.id) === pos,
+      )
+      .map(s => s.recipe.id);
 
     // Install recipes
-    for (const recipe of recipes) { // eslint-disable-line no-unused-vars
-      // eslint-disable-next-line
+    for (const recipe of recipes) {
+      // eslint-disable-line no-unused-vars
+      // eslint-disable-next-line no-await-in-loop
       await this.stores.recipes._install({ recipeId: recipe });
     }
 
-    for (const service of services) { // eslint-disable-line no-unused-vars
+    for (const service of services) {
+      // eslint-disable-line no-unused-vars
       this.actions.service.createFromLegacyService({
         data: service,
       });
-      await this.stores.services.createServiceRequest._promise; // eslint-disable-line
+      // eslint-disable-next-line no-await-in-loop
+      await this.stores.services.createServiceRequest._promise;
     }
 
     this.isImportLegacyServicesExecuting = false;
@@ -281,8 +304,7 @@ export default class UserStore extends Store {
 
     const { router } = this.stores;
     const currentRoute = window.location.hash;
-    if (!this.isLoggedIn
-      && currentRoute.includes('token=')) {
+    if (!this.isLoggedIn && currentRoute.includes('token=')) {
       router.push(this.WELCOME_ROUTE);
       const token = currentRoute.split('=')[1];
 
@@ -293,20 +315,18 @@ export default class UserStore extends Store {
           this._tokenLogin(token);
         }, 1000);
       }
-    } else if (!this.isLoggedIn
-      && !currentRoute.includes(this.BASE_ROUTE)) {
+    } else if (!this.isLoggedIn && !currentRoute.includes(this.BASE_ROUTE)) {
       router.push(this.WELCOME_ROUTE);
-    } else if (this.isLoggedIn
-      && currentRoute === this.LOGOUT_ROUTE) {
+    } else if (this.isLoggedIn && currentRoute === this.LOGOUT_ROUTE) {
       this.actions.user.logout();
       router.push(this.LOGIN_ROUTE);
-    } else if (this.isLoggedIn
-      && currentRoute.includes(this.BASE_ROUTE)
-      && (this.hasCompletedSignup
-        || this.hasCompletedSignup === null)) {
-      if (!isDevMode) {
-        this.stores.router.push('/');
-      }
+    } else if (
+      this.isLoggedIn &&
+      currentRoute.includes(this.BASE_ROUTE) &&
+      (this.hasCompletedSignup || this.hasCompletedSignup === null) &&
+      !isDevMode
+    ) {
+      this.stores.router.push('/');
     }
   };
 
@@ -316,7 +336,7 @@ export default class UserStore extends Store {
       let data;
       try {
         data = await this.getUserInfoRequest.execute()._promise;
-      } catch (e) {
+      } catch {
         return false;
       }
 
@@ -336,12 +356,12 @@ export default class UserStore extends Store {
     try {
       const decoded = jwt.decode(authToken);
 
-      return ({
+      return {
         id: decoded.userId,
         tokenExpiry: moment.unix(decoded.exp).toISOString(),
         authToken,
-      });
-    } catch (err) {
+      };
+    } catch {
       this._logout();
       return false;
     }
@@ -372,7 +392,7 @@ export default class UserStore extends Store {
   async _migrateUserLocale() {
     try {
       await this.getUserInfoRequest._promise;
-    } catch (e) {
+    } catch {
       return false;
     }
 
diff --git a/src/stores/index.ts b/src/stores/index.ts
index 4cd4e92ea..1760ddfa2 100644
--- a/src/stores/index.ts
+++ b/src/stores/index.ts
@@ -34,10 +34,10 @@ export default (api, actions, router) => {
   });
 
   // Initialize all stores
-  Object.keys(stores).forEach(name => {
+  for (const name of Object.keys(stores)) {
     if (stores[name] && stores[name].initialize) {
       stores[name].initialize();
     }
-  });
+  }
   return stores;
 };
diff --git a/src/stores/lib/CachedRequest.js b/src/stores/lib/CachedRequest.js
index 94f615144..a6dd47f7d 100644
--- a/src/stores/lib/CachedRequest.js
+++ b/src/stores/lib/CachedRequest.js
@@ -1,4 +1,3 @@
-// @flow
 import { action } from 'mobx';
 import { isEqual, remove } from 'lodash';
 import Request from './Request';
@@ -30,48 +29,60 @@ export default class CachedRequest extends Request {
 
     // This timeout is necessary to avoid warnings from mobx
     // regarding triggering actions as side-effect of getters
-    setTimeout(action(() => {
-      this.isExecuting = true;
-      // Apply the previous result from this call immediately (cached)
-      if (existingApiCall) {
-        this.result = existingApiCall.result;
-      }
-    }), 0);
+    setTimeout(
+      action(() => {
+        this.isExecuting = true;
+        // Apply the previous result from this call immediately (cached)
+        if (existingApiCall) {
+          this.result = existingApiCall.result;
+        }
+      }),
+      0,
+    );
 
     // Issue api call & save it as promise that is handled to update the results of the operation
-    this._promise = new Promise((resolve) => {
+    this._promise = new Promise(resolve => {
       this._api[this._method](...callArgs)
-        .then((result) => {
-          setTimeout(action(() => {
-            this.result = result;
-            if (this._currentApiCall) this._currentApiCall.result = result;
-            this.isExecuting = false;
-            this.isError = false;
-            this.wasExecuted = true;
-            this._isInvalidated = false;
-            this._isWaitingForResponse = false;
-            this._triggerHooks();
-            resolve(result);
-          }), 1);
+        .then(result => {
+          setTimeout(
+            action(() => {
+              this.result = result;
+              if (this._currentApiCall) this._currentApiCall.result = result;
+              this.isExecuting = false;
+              this.isError = false;
+              this.wasExecuted = true;
+              this._isInvalidated = false;
+              this._isWaitingForResponse = false;
+              this._triggerHooks();
+              resolve(result);
+            }),
+            1,
+          );
           return result;
         })
-        .catch(action((error) => {
-          setTimeout(action(() => {
-            this.error = error;
-            this.isExecuting = false;
-            this.isError = true;
-            this.wasExecuted = true;
-            this._isWaitingForResponse = false;
-            this._triggerHooks();
-            // reject(error);
-          }), 1);
-        }));
+        .catch(
+          action(error => {
+            setTimeout(
+              action(() => {
+                this.error = error;
+                this.isExecuting = false;
+                this.isError = true;
+                this.wasExecuted = true;
+                this._isWaitingForResponse = false;
+                this._triggerHooks();
+                // reject(error);
+              }),
+              1,
+            );
+          }),
+        );
     });
 
     this._isWaitingForResponse = true;
     return this;
   }
 
+  // eslint-disable-next-line unicorn/no-object-as-default-parameter
   invalidate(options = { immediately: false }) {
     this._isInvalidated = true;
     if (options.immediately && this._currentApiCall) {
@@ -81,18 +92,21 @@ export default class CachedRequest extends Request {
   }
 
   patch(modify) {
-    return new Promise((resolve) => {
-      setTimeout(action(() => {
-        const override = modify(this.result);
-        if (override !== undefined) this.result = override;
-        if (this._currentApiCall) this._currentApiCall.result = this.result;
-        resolve(this);
-      }), 0);
+    return new Promise(resolve => {
+      setTimeout(
+        action(() => {
+          const override = modify(this.result);
+          if (override !== undefined) this.result = override;
+          if (this._currentApiCall) this._currentApiCall.result = this.result;
+          resolve(this);
+        }),
+        0,
+      );
     });
   }
 
   removeCacheForCallWith(...args) {
-    remove(this._apiCalls, (c) => isEqual(c.args, args));
+    remove(this._apiCalls, c => isEqual(c.args, args));
   }
 
   _addApiCall(args) {
@@ -102,6 +116,6 @@ export default class CachedRequest extends Request {
   }
 
   _findApiCall(args) {
-    return this._apiCalls.find((c) => isEqual(c.args, args));
+    return this._apiCalls.find(c => isEqual(c.args, args));
   }
 }
diff --git a/src/stores/lib/Request.js b/src/stores/lib/Request.js
index 32ffe4367..39f32729a 100644
--- a/src/stores/lib/Request.js
+++ b/src/stores/lib/Request.js
@@ -107,7 +107,7 @@ export default class Request {
   }
 
   _triggerHooks() {
-    Request._hooks.forEach((hook) => hook(this));
+    for (const hook of Request._hooks) hook(this);
   }
 
   reset = () => {
diff --git a/src/stores/lib/Store.js b/src/stores/lib/Store.js
index b03a7e725..b39070ce8 100644
--- a/src/stores/lib/Store.js
+++ b/src/stores/lib/Store.js
@@ -28,18 +28,18 @@ export default class Store {
   }
 
   registerReactions(reactions) {
-    reactions.forEach((reaction) => this._reactions.push(new Reaction(reaction)));
+    for (const reaction of reactions) this._reactions.push(new Reaction(reaction));
   }
 
   setup() {}
 
   initialize() {
     this.setup();
-    this._reactions.forEach((reaction) => reaction.start());
+    for (const reaction of this._reactions) reaction.start();
   }
 
   teardown() {
-    this._reactions.forEach((reaction) => reaction.stop());
+    for (const reaction of this._reactions) reaction.stop();
   }
 
   resetStatus() {