deps: @npmcli/[email protected]

This also removes any setting of `stdioString` since `true` is the
default now.

Author: lukekarrys

lib/dir.js

@@ -49,7 +49,6 @@ class DirFetcher extends Fetcher {
         pkg: mani,
         event: 'prepare',
         path: this.resolved,
-        stdioString: true,
         stdio,
         banner,
         env: {

lib/util/npm.js

@@ -10,5 +10,5 @@ module.exports = (npmBin, npmCommand, cwd, env, extra) => {
   // in temp directories.  this lets us link previously-seen repos that
   // are also being prepared.
 
-  return spawn(cmd, args, { cwd, stdioString: true, env }, extra)
+  return spawn(cmd, args, { cwd, env }, extra)
 }

package.json

@@ -47,7 +47,7 @@
     "@npmcli/git": "^4.0.0",
     "@npmcli/installed-package-contents": "^2.0.1",
     "@npmcli/promise-spawn": "^6.0.1",
-    "@npmcli/run-script": "^5.0.0",
+    "@npmcli/run-script": "^6.0.0",
     "cacache": "^17.0.0",
     "fs-minipass": "^2.1.0",
     "minipass": "^3.1.6",

tap-snapshots/test/util/npm.js.test.cjs

@@ -16,7 +16,7 @@ Object {
   "message": "oopsie",
   "signal": undefined,
   "stderr": "",
-  "stdout": "[\\"{NODE}\\",[\\"/path/to/npm/bin/npm-cli.js\\",\\"flerb\\"],{\\"cwd\\":\\"/cwd\\",\\"stdioString\\":true,\\"env\\":{\\"environmental\\":\\"variables\\"}}]",
+  "stdout": "[\\"{NODE}\\",[\\"/path/to/npm/bin/npm-cli.js\\",\\"flerb\\"],{\\"cwd\\":\\"/cwd\\",\\"env\\":{\\"environmental\\":\\"variables\\"}}]",
 }
 `
 
@@ -30,6 +30,6 @@ Object {
   "message": "oopsie",
   "signal": undefined,
   "stderr": "",
-  "stdout": "[\\"/path/to/npm\\",[\\"flerb\\"],{\\"cwd\\":\\"/cwd\\",\\"stdioString\\":true,\\"env\\":{\\"environmental\\":\\"variables\\"}}]",
+  "stdout": "[\\"/path/to/npm\\",[\\"flerb\\"],{\\"cwd\\":\\"/cwd\\",\\"env\\":{\\"environmental\\":\\"variables\\"}}]",
 }
 `