Changed electron fs-job to invoke, for simplefying storage

(And it helps for the modloader)

electron/index.js

@@ -1,9 +1,8 @@
 /* eslint-disable quotes,no-undef */
 
-const { app, BrowserWindow, Menu, MenuItem, session } = require("electron");
+const { app, BrowserWindow, Menu, MenuItem, ipcMain, shell } = require("electron");
 const path = require("path");
 const url = require("url");
-const { ipcMain, shell } = require("electron");
 const fs = require("fs");
 const steam = require("./steam");
 const asyncLock = require("async-lock");
@@ -287,10 +286,14 @@ async function performFsJob(job) {
     }
 }
 
-ipcMain.on("fs-job", async (event, arg) => {
+ipcMain.handle(
+    "fs-job",
+    (event, arg) =>
+        new Promise(async (resolve, reject) => {
             const result = await performFsJob(arg);
-    event.reply("fs-response", { id: arg.id, result });
+            resolve(result);
-});
+        })
+);
 
 steam.init(isDev);
 steam.listen();

electron/package.json

@@ -10,12 +10,10 @@
         "start": "electron --disable-direct-composition --in-process-gpu ."
     },
     "devDependencies": {
-        "electron": "10.4.0"
+        "electron": "10.4.0",
+        "async-lock": "^1.2.8"
     },
     "optionalDependencies": {
         "shapez.io-private-artifacts": "github:tobspr/shapez.io-private-artifacts#abi-v85"
-    },
-    "dependencies": {
-        "async-lock": "^1.2.8"
     }
 }

electron/yarn.lock

@@ -503,10 +503,6 @@ serialize-error@^7.0.1:
   dependencies:
     type-fest "^0.13.1"
 
-"shapez.io-private-artifacts@github:tobspr/shapez.io-private-artifacts#abi-v85":
-  version "0.1.0"
-  resolved "git+ssh://git@github.com/tobspr/shapez.io-private-artifacts.git#63adf7e0ea4b90c2a29053ce1f0ec9d573b3ac0a"
-
 sprintf-js@^1.1.2:
   version "1.1.2"
   resolved "https://registry.yarnpkg.com/sprintf-js/-/sprintf-js-1.1.2.tgz#da1765262bf8c0f571749f2ad6c26300207ae673"

src/js/platform/electron/storage.js

@@ -7,24 +7,6 @@ const logger = createLogger("electron-storage");
 export class StorageImplElectron extends StorageInterface {
     constructor(app) {
         super(app);
-
-        /** @type {Object.<number, {resolve:Function, reject: Function}>} */
-        this.jobs = {};
-        this.jobId = 0;
-
-        getIPCRenderer().on("fs-response", (event, arg) => {
-            const id = arg.id;
-            if (!this.jobs[id]) {
-                logger.warn("Got unhandled FS response, job not known:", id);
-                return;
-            }
-            const { resolve, reject } = this.jobs[id];
-            if (arg.result.success) {
-                resolve(arg.result.data);
-            } else {
-                reject(arg.result.error);
-            }
-        });
     }
 
     initialize() {
@@ -33,42 +15,52 @@ export class StorageImplElectron extends StorageInterface {
 
     writeFileAsync(filename, contents) {
         return new Promise((resolve, reject) => {
-            // ipcMain
+            getIPCRenderer()
-            const jobId = ++this.jobId;
+                .invoke("fs-job", {
-            this.jobs[jobId] = { resolve, reject };
-
-            getIPCRenderer().send("fs-job", {
                     type: "write",
                     filename,
                     contents,
-                id: jobId,
+                })
+                .then(result => {
+                    if (result.success) {
+                        resolve(result.data);
+                    } else {
+                        reject(result.error);
+                    }
                 });
         });
     }
 
     readFileAsync(filename) {
         return new Promise((resolve, reject) => {
-            // ipcMain
+            getIPCRenderer()
-            const jobId = ++this.jobId;
+                .invoke("fs-job", {
-            this.jobs[jobId] = { resolve, reject };
-
-            getIPCRenderer().send("fs-job", {
                     type: "read",
                     filename,
-                id: jobId,
+                })
+                .then(result => {
+                    if (result.success) {
+                        resolve(result.data);
+                    } else {
+                        reject(result.error);
+                    }
                 });
         });
     }
 
     deleteFileAsync(filename) {
         return new Promise((resolve, reject) => {
-            // ipcMain
+            getIPCRenderer()
-            const jobId = ++this.jobId;
+                .invoke("fs-job", {
-            this.jobs[jobId] = { resolve, reject };
-            getIPCRenderer().send("fs-job", {
                     type: "delete",
                     filename,
-                id: jobId,
+                })
+                .then(result => {
+                    if (result.success) {
+                        resolve(result.data);
+                    } else {
+                        reject(result.error);
+                    }
                 });
         });
     }