fix: apps lookup issue on windows (#33)

pd4d10 · Jul 8, 2024 · 481caf9 · 481caf9
1 parent 02b699a
commit 481caf9
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/src/main/platforms/index.ts b/src/main/platforms/index.ts
@@ -1,7 +1,7 @@
 export function importByPlatform() {
   switch (process.platform) {
-    // case "win32":
-    //   return import("../platforms/win");
+    case "win32":
+      return import("../platforms/win");
     case "darwin":
       return import("./macos");
     case "linux":

diff --git a/src/main/platforms/win.ts b/src/main/platforms/win.ts
@@ -1,4 +1,4 @@
-import type { AppInfo } from "../reducers/app";
+import type { AppInfo } from "../../reducer";
 import type { AppReader } from "./utils";
 import fs from "fs";
 import path from "path";
@@ -135,9 +135,18 @@ export const adapter: AppReader = {
           "Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall",
         ),
       ];
-      return Promise.all(
-        items.map((itemValues) => getAppInfoFromRegeditItemValues(itemValues)),
+
+      const results = await Promise.all(
+        items.map((itemValues) =>
+          Result.wrapAsync(async () =>
+            getAppInfoFromRegeditItemValues(itemValues),
+          ),
+        ),
       );
+
+      const apps = results.flatMap((app) => (app.ok ? [app.unwrap()] : []));
+
+      return apps;
     }),
 
   readByPath: (p: string) =>