[PM-3685] Remove ipcRenderer from electron-renderer-storage (#6481)

* [PM-3685] Remove ipcRenderer from renderer-storage

* Break out storage and keytar into separate functions

apps/desktop/src/platform/preload.ts

@@ -4,6 +4,27 @@ import { DeviceType, ThemeType } from "@bitwarden/common/enums";
 
 import { isDev, isWindowsStore } from "../utils";
 
+const storage = {
+  get: <T>(key: string): Promise<T> => ipcRenderer.invoke("storageService", { action: "get", key }),
+  has: (key: string): Promise<boolean> =>
+    ipcRenderer.invoke("storageService", { action: "has", key }),
+  save: (key: string, obj: any): Promise<void> =>
+    ipcRenderer.invoke("storageService", { action: "save", key, obj }),
+  remove: (key: string): Promise<void> =>
+    ipcRenderer.invoke("storageService", { action: "remove", key }),
+};
+
+const passwords = {
+  get: (key: string, keySuffix: string): Promise<string> =>
+    ipcRenderer.invoke("keytar", { action: "getPassword", key, keySuffix }),
+  has: (key: string, keySuffix: string): Promise<boolean> =>
+    ipcRenderer.invoke("keytar", { action: "hasPassword", key, keySuffix }),
+  set: (key: string, keySuffix: string, value: string): Promise<void> =>
+    ipcRenderer.invoke("keytar", { action: "setPassword", key, keySuffix, value }),
+  delete: (key: string, keySuffix: string): Promise<void> =>
+    ipcRenderer.invoke("keytar", { action: "deletePassword", key, keySuffix }),
+};
+
 export default {
   versions: {
     app: (): Promise<string> => ipcRenderer.invoke("appVersion"),
@@ -27,6 +48,9 @@ export default {
       }
     });
   },
+
+  storage,
+  passwords,
 };
 
 function deviceType(): DeviceType {

apps/desktop/src/platform/services/electron-renderer-secure-storage.service.ts

@@ -1,41 +1,22 @@
-import { ipcRenderer } from "electron";
-
 import { AbstractStorageService } from "@bitwarden/common/platform/abstractions/storage.service";
 import { StorageOptions } from "@bitwarden/common/platform/models/domain/storage-options";
 
 export class ElectronRendererSecureStorageService implements AbstractStorageService {
   async get<T>(key: string, options?: StorageOptions): Promise<T> {
-    const val = await ipcRenderer.invoke("keytar", {
+    const val = await ipc.platform.passwords.get(key, options?.keySuffix ?? "");
-      action: "getPassword",
-      key: key,
-      keySuffix: options?.keySuffix ?? "",
-    });
     return val != null ? (JSON.parse(val) as T) : null;
   }
 
   async has(key: string, options?: StorageOptions): Promise<boolean> {
-    const val = await ipcRenderer.invoke("keytar", {
+    const val = await ipc.platform.passwords.has(key, options?.keySuffix ?? "");
-      action: "hasPassword",
-      key: key,
-      keySuffix: options?.keySuffix ?? "",
-    });
     return !!val;
   }
 
   async save(key: string, obj: any, options?: StorageOptions): Promise<any> {
-    await ipcRenderer.invoke("keytar", {
+    await ipc.platform.passwords.set(key, options?.keySuffix ?? "", JSON.stringify(obj));
-      action: "setPassword",
-      key: key,
-      keySuffix: options?.keySuffix ?? "",
-      value: JSON.stringify(obj),
-    });
   }
 
   async remove(key: string, options?: StorageOptions): Promise<any> {
-    await ipcRenderer.invoke("keytar", {
+    await ipc.platform.passwords.delete(key, options?.keySuffix ?? "");
-      action: "deletePassword",
-      key: key,
-      keySuffix: options?.keySuffix ?? "",
-    });
   }
 }

apps/desktop/src/platform/services/electron-renderer-storage.service.ts

@@ -1,34 +1,19 @@
-import { ipcRenderer } from "electron";
-
 import { AbstractStorageService } from "@bitwarden/common/platform/abstractions/storage.service";
 
 export class ElectronRendererStorageService implements AbstractStorageService {
   get<T>(key: string): Promise<T> {
-    return ipcRenderer.invoke("storageService", {
+    return ipc.platform.storage.get(key);
-      action: "get",
-      key: key,
-    });
   }
 
   has(key: string): Promise<boolean> {
-    return ipcRenderer.invoke("storageService", {
+    return ipc.platform.storage.has(key);
-      action: "has",
-      key: key,
-    });
   }
 
   save(key: string, obj: any): Promise<any> {
-    return ipcRenderer.invoke("storageService", {
+    return ipc.platform.storage.save(key, obj);
-      action: "save",
-      key: key,
-      obj: obj,
-    });
   }
 
   remove(key: string): Promise<any> {
-    return ipcRenderer.invoke("storageService", {
+    return ipc.platform.storage.remove(key);
-      action: "remove",
-      key: key,
-    });
   }
 }