fix: fix spawn shell args

Author: zdm

lib/commands/package/link.js

@@ -4,7 +4,7 @@ import path from "node:path";
 import { readConfig } from "#core/config";
 import { exists } from "#core/fs";
 import { glob } from "#core/glob";
-import { shellEscape } from "#core/utils";
+import { shellQuote } from "#core/utils";
 import Command from "#lib/command";
 import Dependencies from "#lib/package/dependencies";
 
@@ -68,7 +68,7 @@ export default class extends Command {
                     console.log( "Installing dependencies ..." );
                 }
 
-                const res = childProcess.spawnSync( shellEscape( [ "npm", command ] ), {
+                const res = childProcess.spawnSync( shellQuote( [ "npm", command ] ), {
                     "stdio": "inherit",
                     "shell": true,
                 } );

lib/package.js

@@ -17,7 +17,7 @@ import stream from "#core/stream";
 import tar from "#core/tar";
 import Table from "#core/text/table";
 import { TmpDir, TmpFile } from "#core/tmp";
-import { confirm, objectIsEmpty, repeatAction, shellEscape } from "#core/utils";
+import { confirm, objectIsEmpty, repeatAction, shellQuote } from "#core/utils";
 import * as yaml from "#core/yaml";
 import Git from "#lib/git";
 import lintFile from "#lib/lint/file";
@@ -372,7 +372,7 @@ export default class Package {
         // pack package
         res = await repeatAction( async () => {
             const tmpDir = new TmpDir(),
-                command = shellEscape( [ "npm", "pack", "--json", "--pack-destination", tmpDir.path, this.root ] );
+                command = shellQuote( [ "npm", "pack", "--json", "--pack-destination", tmpDir.path, this.root ] );
 
             console.log( "Packing:", command );
 
@@ -435,7 +435,7 @@ export default class Package {
 
         args.push( pack.path );
 
-        const command = shellEscape( args );
+        const command = shellQuote( args );
 
         res = await repeatAction( async () => {
             console.log( "Publishing:", command );
@@ -461,7 +461,7 @@ export default class Package {
         // add additional tags
         if ( addTags.length ) {
             for ( const tag of addTags ) {
-                const command = shellEscape( [ "npm", "dist-tag", "add", `${ this.name }@${ this.version }`, tag ] );
+                const command = shellQuote( [ "npm", "dist-tag", "add", `${ this.name }@${ this.version }`, tag ] );
 
                 res = await repeatAction( async () => {
                     console.log( "Adding npm tag:", command );
@@ -628,7 +628,7 @@ export default class Package {
     }
 
     runCommand ( command, ...args ) {
-        const res = childProcess.spawnSync( shellEscape( [ command, ...args ] ), {
+        const res = childProcess.spawnSync( shellQuote( [ command, ...args ] ), {
             "cwd": this.root,
             "stdio": "inherit",
             "shell": true,
@@ -652,7 +652,7 @@ export default class Package {
             argv = [];
         }
 
-        const res = childProcess.spawnSync( shellEscape( [ "npm", "run", script, ...argv ] ), {
+        const res = childProcess.spawnSync( shellQuote( [ "npm", "run", script, ...argv ] ), {
             "cwd": this.root,
             "stdio": "inherit",
             "shell": true,

lib/package/release.js

@@ -8,7 +8,7 @@ import File from "#core/file";
 import { exists } from "#core/fs";
 import SemanticVersion from "#core/semantic-version";
 import { TmpFile } from "#core/tmp";
-import { confirm, repeatAction, shellEscape } from "#core/utils";
+import { confirm, repeatAction, shellQuote } from "#core/utils";
 import { lintFile } from "#lib/lint";
 
 export default class Publish {
@@ -437,7 +437,7 @@ ${ this.#changelog.linkifyMarkdown( changelogMarkdown ) }
 
         fs.writeFileSync( tmp.path, changelogMarkdown );
 
-        const res = childProcess.spawnSync( shellEscape( [ process.env.EDITOR, tmp.path ] ), {
+        const res = childProcess.spawnSync( shellQuote( [ process.env.EDITOR, tmp.path ] ), {
             "shell": true,
             "stdio": "inherit",
         } );