Improve linting

src/main/lib/modules-manager.ts

@@ -1,6 +1,6 @@
 import Module from '../modules/module';
 
-export const init = async (...modules: Module[]) => {
+export const init = async (...modules: Module[]): Promise<void> => {
   await Promise.all(
     modules.map((module) =>
       module.init().catch((err) => {

src/main/modules/config.ts

@@ -21,8 +21,8 @@ class ConfigModule extends Module {
     this.workArea = electron.screen.getPrimaryDisplay().workArea;
   }
 
-  async load() {
-    const defaultConfig: Record<string, any> = this.getDefaultConfig();
+  async load(): Promise<void> {
+    const defaultConfig: Partial<Config> = this.getDefaultConfig();
     const pathUserData = app.getPath('userData');
 
     this.conf = new teeny(path.join(pathUserData, 'config.json'), defaultConfig);
@@ -78,15 +78,15 @@ class ConfigModule extends Module {
     return this.conf.get() as Config; // Maybe possible to type TeenyConf with Generics?
   }
 
-  get(key: keyof Config) {
+  get<T extends keyof Config>(key: T): Config[T] {
     if (!this.conf) {
       throw new Error('Config not loaded');
     }
 
     return this.conf.get(key);
   }
 
-  reload() {
+  reload(): void {
     if (!this.conf) {
       throw new Error('Config not loaded');
     }

src/main/modules/dock-menu-darwin.ts

@@ -23,7 +23,7 @@ class DockMenuDarwinModule extends ModuleWindow {
     this.pauseToggle = [];
   }
 
-  async load() {
+  async load(): Promise<void> {
     this.songDetails = [
       {
         label: 'Not playing',
@@ -84,13 +84,13 @@ class DockMenuDarwinModule extends ModuleWindow {
     this.setDockMenu(PlayerStatus.PAUSE);
   }
 
-  setDockMenu(state: PlayerStatus) {
+  setDockMenu(state: PlayerStatus): void {
     const playPauseItem = state === 'play' ? this.pauseToggle : this.playToggle;
     const menuTemplate = [...this.songDetails, ...playPauseItem, ...this.menu];
     app.dock.setMenu(Menu.buildFromTemplate(menuTemplate));
   }
 
-  updateTrayMetadata(metadata: TrackModel) {
+  updateTrayMetadata(metadata: TrackModel): void {
     this.songDetails = [
       {
         label: `${metadata.title}`,

src/main/modules/global-shortcuts.ts

@@ -6,7 +6,7 @@ import { globalShortcut } from 'electron';
 import ModuleWindow from './module-window';
 
 class GlobalShortcutsModule extends ModuleWindow {
-  async load() {
+  async load(): Promise<void> {
     globalShortcut.register('MediaPlayPause', () => {
       this.window.webContents.send('playback:playpause');
     });

src/main/modules/ipc.ts

@@ -23,7 +23,7 @@ class IpcModule extends ModuleWindow {
     this.forceQuit = false;
   }
 
-  async load() {
+  async load(): Promise<void> {
     ipcMain.on('app:restart', () => {
       app.relaunch({ args: ['process.argv.slice(1)', '--relaunch'] });
       app.exit(0);
@@ -49,7 +49,7 @@ class IpcModule extends ModuleWindow {
     });
   }
 
-  close(e: Event) {
+  close(e: Event): void {
     this.config.reload(); // HACKY
     const minimizeToTray = this.config.get('minimizeToTray');
 

src/main/modules/menu.ts

@@ -8,7 +8,7 @@ import { Menu, shell } from 'electron';
 import ModuleWindow from './module-window';
 
 class MenuModule extends ModuleWindow {
-  async load() {
+  async load(): Promise<void> {
     const template: Electron.MenuItemConstructorOptions[] = [
       { role: 'appMenu' },
       { role: 'fileMenu' },

src/main/modules/module-window.ts

@@ -12,7 +12,7 @@ class ModuleWindow extends Module {
     this.window = window;
   }
 
-  getWindow() {
+  getWindow(): Electron.BrowserWindow {
     return this.window;
   }
 }

src/main/modules/module.ts

@@ -14,7 +14,7 @@ class Module {
   }
 
   // To not be overriden
-  async init() {
+  async init(): Promise<void> {
     if (this.loaded) throw new TypeError(`Module ${this.constructor.name} is already loaded`);
 
     if (this.platforms.includes(os.platform())) {
@@ -28,7 +28,7 @@ class Module {
   }
 
   // Can (now) be an asynchronous method
-  async load() {
+  async load(): Promise<void> {
     throw new TypeError(`Module ${this.constructor.name} should have a load() method`);
     // Do whatever you want here :)
   }

src/main/modules/mpris.ts

@@ -21,7 +21,7 @@ class MprisModule extends ModuleWindow {
     this.platforms = ['linux'];
   }
 
-  async load() {
+  async load(): Promise<void> {
     this.player = mpris({
       name: 'museeks',
       identity: 'Museeks',

src/main/modules/power-monitor.ts

@@ -6,7 +6,7 @@ import * as electron from 'electron';
 import ModuleWindow from './module-window';
 
 class PowerMonitorModule extends ModuleWindow {
-  async load() {
+  async load(): Promise<void> {
     const { powerMonitor } = electron;
     const { window } = this;
 

src/main/modules/sleep-blocker.ts

@@ -18,24 +18,24 @@ class SleepBlocker extends ModuleWindow {
     this.platforms = ['win32', 'darwin', 'linux'];
   }
 
-  onStartPlayback = (_event: Event) => {
+  onStartPlayback = (_event: Event): void => {
     if (this.enabled && !powerSaveBlocker.isStarted(this.sleepBlockerId)) {
       // or 'prevent-display-sleep'
       this.sleepBlockerId = powerSaveBlocker.start('prevent-app-suspension');
     }
   };
 
-  onStopPlayback = (_event: Event) => {
+  onStopPlayback = (_event: Event): void => {
     if (powerSaveBlocker.isStarted(this.sleepBlockerId)) {
       powerSaveBlocker.stop(this.sleepBlockerId);
     }
   };
 
-  toggleSleepBlocker = (_event: Event, value: boolean) => {
+  toggleSleepBlocker = (_event: Event, value: boolean): void => {
     this.enabled = value;
   };
 
-  async load() {
+  async load(): Promise<void> {
     ipcMain.on('settings:toggleSleepBlocker', this.toggleSleepBlocker);
     ipcMain.on('playback:play', this.onStartPlayback);
     ipcMain.on('playback:pause', this.onStopPlayback);

src/main/modules/thumbar.ts

@@ -20,7 +20,7 @@ class ThumbarModule extends ModuleWindow {
     this.platforms = ['win32'];
   }
 
-  async load() {
+  async load(): Promise<void> {
     const { window } = this;
 
     const icons = {

src/main/modules/tray.ts

@@ -53,7 +53,7 @@ class TrayModule extends ModuleWindow {
     else if (os.platform() === 'darwin') this.trayIcon = trayIcons['tray-darwin-dark'];
   }
 
-  async load() {
+  async load(): Promise<void> {
     // Fix for gnome-shell and high-dpi
     if (os.platform() === 'linux') {
       ps.lookup(
@@ -165,7 +165,7 @@ class TrayModule extends ModuleWindow {
     });
   }
 
-  create() {
+  create(): void {
     this.tray = new Tray(this.trayIcon);
     this.tray.setToolTip('Museeks');
 
@@ -184,13 +184,13 @@ class TrayModule extends ModuleWindow {
     this.setContextMenu(this.status);
   }
 
-  destroy() {
+  destroy(): void {
     if (this.tray) {
       this.tray.destroy();
     }
   }
 
-  setContextMenu(state: PlayerStatus) {
+  setContextMenu(state: PlayerStatus): void {
     const playPauseItem = state === 'play' ? this.pauseToggle : this.playToggle;
     const menuTemplate = [...this.songDetails, ...playPauseItem, ...this.menu];
 
@@ -199,13 +199,13 @@ class TrayModule extends ModuleWindow {
     }
   }
 
-  refreshTrayIcon() {
+  refreshTrayIcon(): void {
     if (this.tray) {
       this.tray.setImage(this.trayIcon);
     }
   }
 
-  updateTrayMetadata(track: TrackModel) {
+  updateTrayMetadata(track: TrackModel): void {
     this.songDetails = [
       {
         label: `${track.title}`,