fix: discard NODE_ENV when installing project dependencies (#6224)

Avoid empty `node_modules` when the user has set `NODE_ENV` to
`production` in the shell environment.

In the long run we should have a more comprehensive preflight check
for execution environment though.

Author: haoqunjiang

packages/@vue/cli/lib/util/ProjectPackageManager.js

@@ -347,15 +347,25 @@ class PackageManager {
   }
 
   async runCommand (command, args) {
+    const prevNodeEnv = process.env.NODE_ENV
+    // In the use case of Vue CLI, when installing dependencies,
+    // the `NODE_ENV` environment variable does no good;
+    // it only confuses users by skipping dev deps (when set to `production`).
+    delete process.env.NODE_ENV
+
     await this.setRegistryEnvs()
-    return await executeCommand(
+    await executeCommand(
       this.bin,
       [
         ...PACKAGE_MANAGER_CONFIG[this.bin][command],
         ...(args || [])
       ],
       this.context
     )
+
+    if (prevNodeEnv) {
+      process.env.NODE_ENV = prevNodeEnv
+    }
   }
 
   async install () {