Commit 6398320

Anton Golub <antongolub@antongolub.com>
2025-08-07 09:05:13
fix(cli): filter `npm install` flags (#1308)
1 parent 3188694
Changed files (2)
build/deps.cjs
@@ -36,9 +36,14 @@ function installDeps(dependencies, prefix, registry, installerType = "npm") {
 }
 var installers = {
   npm: (_0) => __async(null, [_0], function* ({ packages, prefix, registry }) {
-    const prefixFlag = prefix ? `--prefix=${prefix}` : "";
-    const registryFlag = registry ? `--registry=${registry}` : "";
-    yield import_index.$`npm install --no-save --no-audit --no-fund ${registryFlag} ${prefixFlag} ${packages}`.nothrow();
+    const flags = [
+      "--no-save",
+      "--no-audit",
+      "--no-fund",
+      prefix && `--prefix=${prefix}`,
+      registry && `--registry=${registry}`
+    ].filter(Boolean);
+    yield import_index.$`npm install ${flags} ${packages}`.nothrow();
   })
 };
 var builtins = new Set(import_node_module.builtinModules);
src/deps.ts
@@ -53,9 +53,14 @@ type DepsInstaller = (opts: {
 
 const installers: Record<any, DepsInstaller> = {
   npm: async ({ packages, prefix, registry }) => {
-    const prefixFlag = prefix ? `--prefix=${prefix}` : ''
-    const registryFlag = registry ? `--registry=${registry}` : ''
-    await $`npm install --no-save --no-audit --no-fund ${registryFlag} ${prefixFlag} ${packages}`.nothrow()
+    const flags = [
+      '--no-save',
+      '--no-audit',
+      '--no-fund',
+      prefix && `--prefix=${prefix}`,
+      registry && `--registry=${registry}`,
+    ].filter(Boolean)
+    await $`npm install ${flags} ${packages}`.nothrow()
   },
 }