Filter out undefined options.

raineorshine · Sep 16, 2023 · aed727c · aed727c
1 parent 424937a
commit aed727c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -289,7 +289,10 @@ export async function run(
   if (!options.prefix) {
     const defaultPrefix =
       packageManagers[options.packageManager || '']?.defaultPrefix || packageManagers.npm.defaultPrefix!
-    options.prefix = await defaultPrefix(options)
+    const prefix = await defaultPrefix(options)
+    if (prefix) {
+      options.prefix = prefix
+    }
   }
 
   let timeout: NodeJS.Timeout | undefined

diff --git a/src/lib/initOptions.ts b/src/lib/initOptions.ts
@@ -9,6 +9,7 @@ import { Target } from '../types/Target'
 import cacher from './cache'
 import determinePackageManager from './determinePackageManager'
 import exists from './exists'
+import keyValueBy from './keyValueBy'
 import programError from './programError'
 
 function parseFilterExpression(filterExpression: string[] | undefined): string[] | undefined
@@ -191,13 +192,19 @@ async function initOptions(runOptions: RunOptions, { cli }: { cli?: boolean } =
   }
   resolvedOptions.cacher = await cacher(resolvedOptions)
 
+  // remove undefined values
+  const resolvedOptionsFiltered: Options = keyValueBy(
+    resolvedOptions as { [key: string]: Options[keyof Options] },
+    (key, value) => (value !== undefined ? { [key]: value } : null),
+  )
+
   // print 'Using yarn/pnpm/etc' when autodetected
   // use resolved options so that options.json is set
   if (!options.packageManager && packageManager !== 'npm') {
-    print(resolvedOptions, `Using ${packageManager}`)
+    print(resolvedOptionsFiltered, `Using ${packageManager}`)
   }
 
-  return resolvedOptions
+  return resolvedOptionsFiltered
 }
 
 export default initOptions