Fix: Better npm_config_argv emulation

__tests__/fixtures/lifecycle-scripts/npm_config_argv_env_vars/log-command.js

@@ -1,7 +1 @@
-try {
-  const argv = JSON.parse(process.env['npm_config_argv']);
-
-  console.log(`##${argv.cooked[0]}##`);
-} catch (err) {
-  console.log(`##${err.toString()}##`);
-}
+console.log(process.env['npm_config_argv']);

__tests__/lifecycle-scripts.js

@@ -81,10 +81,10 @@ test.concurrent(
       execCommand('test', 'npm_config_argv_env_vars', env),
     ]);
 
-    expect(stdouts[0]).toContain('##install##');
-    expect(stdouts[1]).toContain('##install##');
-    expect(stdouts[2]).toContain('##test##');
-    expect(stdouts[3]).toContain('##test##');
+    expect(stdouts[0]).toContain('"install"');
+    expect(stdouts[1]).toContain('"install"');
+    expect(stdouts[2]).toContain('"run","test"');
+    expect(stdouts[3]).toContain('"run","test"');
   },
 );
 

src/cli/index.js

@@ -140,19 +140,17 @@ export function main({
     setHelpMode();
   }
 
-  let command;
   if (!commandName) {
     commandName = 'install';
     isKnownCommand = true;
   }
 
-  if (isKnownCommand) {
-    command = commands[commandName];
-  } else {
+  if (!isKnownCommand) {
     // if command is not recognized, then set default to `run`
     args.unshift(commandName);
-    command = commands.run;
+    commandName = 'run';
   }
+  const command = commands[commandName];
 
   let warnAboutRunDashDash = false;
   // we are using "yarn <script> -abc" or "yarn run <script> -abc", we want -abc to be script options, not yarn options
@@ -448,6 +446,7 @@ export function main({
   config
     .init({
       cwd,
+      commandName,
 
       binLinks: commander.binLinks,
       modulesFolder: commander.modulesFolder,
@@ -469,8 +468,6 @@ export function main({
       networkTimeout: commander.networkTimeout,
       nonInteractive: commander.nonInteractive,
       scriptsPrependNodePath: commander.scriptsPrependNodePath,
-
-      commandName: commandName === 'run' ? commander.args[0] : commandName,
     })
     .then(() => {
       // option "no-progress" stored in yarn config

src/util/execute-lifecycle-script.js

@@ -61,8 +61,8 @@ export async function makeEnv(
   // parser used by npm. Since we use other parser, we just roughly emulate it's output. (See: #684)
   env.npm_config_argv = JSON.stringify({
     remain: [],
-    cooked: [config.commandName],
-    original: [config.commandName],
+    cooked: config.commandName === 'run' ? [config.commandName, stage] : [config.commandName],
+    original: process.argv.slice(2),
   });
 
   const manifest = await config.maybeReadManifest(cwd);