fix: handle undefined

Throw if upm-config dir-path could not be resolved
openupm · Oct 24, 2023 · 9108970 · 9108970
1 parent dc1e931
commit 9108970
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/core.ts b/lib/core.ts
@@ -369,7 +369,7 @@ export const saveManifest = function (data: PkgManifest) {
 
 // Get .upmconfig.toml directory
 export const getUpmConfigDir = async function (): Promise<string> {
-  let dirPath = "";
+  let dirPath: string | undefined = "";
   const systemUserSubPath = "Unity/config/ServiceAccounts";
   if (env.wsl) {
     if (!isWsl) {
@@ -387,8 +387,6 @@ export const getUpmConfigDir = async function (): Promise<string> {
       });
     }
   } else {
-    // TODO: Handle undefined
-    // @ts-ignore
     dirPath = process.env.USERPROFILE
       ? process.env.USERPROFILE
       : process.env.HOME;
@@ -399,6 +397,8 @@ export const getUpmConfigDir = async function (): Promise<string> {
       dirPath = path.join(process.env.ALLUSERSPROFILE, systemUserSubPath);
     }
   }
+  if (dirPath === undefined)
+    throw new Error("Could not resolve upm-config dir-path");
   return dirPath;
 };