fix(main): split process and it works

chore: edit comments

Signed-off-by: kash / Hytus <[email protected]>

src/assets/index.ts

@@ -1,5 +1,7 @@
 import "./style.css"
 
+// ? Handlers in index.heml is not working correctly (it might be an effect of webpack?)
+
 const launcherConfig = {
   launcherVersion: "1.0", // TODO: should come from package.json
   gameLaunchPossible: true, // TODO: unimplemented

src/installlocators.ts

@@ -3,7 +3,7 @@
 import { lstat, readFile, readdir } from "node:fs/promises"
 import path from "node:path"
 
-import drivelist from "drivelist"
+import { list } from "drivelist"
 import regedit from "winreg"
 
 // TODO: refactor
@@ -47,7 +47,7 @@ function locateSteamInstall() {
 }
 
 async function locateMsStoreInstall(): Promise<string> {
-  const drives = await drivelist.list()
+  const drives = await list()
   const mountPaths = drives
     .filter((drive) => drive.isSystem)
     .flatMap((drive) => drive.mountpoints)

src/main.ts

@@ -60,6 +60,34 @@ function createWindow() {
 
 // () => app.enableSandbox()
 
+function init() {
+  ipcMain.on("launch-game", handleLaunchGame)
+  ipcMain.on("install-mods", (event, packagePaths) =>
+    installAllPackages(packagePaths, false)
+  )
+  ipcMain.on(
+    "set-config-value",
+    async <T extends MMLConfigKeyType>(
+      _: IpcMainEvent,
+      key: T,
+      value: MMLConfigType[T]
+    ) => {
+      await config.setConfigValue(key, value)
+    }
+  )
+  ipcMain.handle(
+    "get-config-value",
+    <T extends MMLConfigKeyType>(_: IpcMainInvokeEvent, key: T) => {
+      return config.getConfigValue(key)
+    }
+  )
+  createWindow()
+}
+
+app.whenReady().then(() => {
+  init()
+})
+
 app.on("ready", () => {
   const installPackagesPaths = process.argv
     .slice(1)
@@ -79,29 +107,6 @@ app.on("ready", () => {
 
   if (installPackagesPaths.length > 0) {
     installAllPackages(installPackagesPaths, true)
-  } else {
-    ipcMain.on("launch-game", handleLaunchGame)
-    ipcMain.on("install-mods", (event, packagePaths) =>
-      installAllPackages(packagePaths, false)
-    )
-    ipcMain.on(
-      "set-config-value",
-      async <T extends MMLConfigKeyType>(
-        _: IpcMainEvent,
-        key: T,
-        value: MMLConfigType[T]
-      ) => {
-        await config.setConfigValue(key, value)
-      }
-    )
-    ipcMain.handle(
-      "get-config-value",
-      <T extends MMLConfigKeyType>(_: IpcMainInvokeEvent, key: T) => {
-        return config.getConfigValue(key)
-      }
-    )
-
-    createWindow()
   }
 })
 

src/preload.ts

@@ -5,7 +5,6 @@ import type { MMLConfigKeyType, MMLConfigType } from "./types/config"
 // import { ILP_getInstalledMods } from "./installedlistpopulator"
 
 // TODO: refactor
-// TODO: fix not exposing keys (use ipc instead)
 
 // contextBridge.exposeInMainWorld("installedpopulator", ILP_getInstalledMods)
 contextBridge.exposeInMainWorld("moolah", {

webpack.renderer.config.ts

@@ -6,7 +6,7 @@ import { rendererRules } from "./webpack.rules"
 const rendererConfig: Configuration = {
   // XXX: should add `optimization` key for production?
   // XXX: should add `cache` key for developing?
-  devtool: "inline-source-map",
+  // devtool: "inline-source-map",
   module: {
     rules: rendererRules,
   },