DEV Community

Discussion on: A simple way to replace npm scripts in Deno

View post

Replies for: That's a great observation. Do you have any suggestions on how to handle this case?

 
vonheikemen profile image
Heiker •

I found this package: exec. It does the same thing as that function with more options. You could use that. It only handles double quotes but it's an improvement.

If anyone is curious how they do it, here it is.

function splitCommand(command) {
  var myRegexp = /[^\s"]+|"([^"]*)"/gi;
  var splits = [];

  do {
    //Each call to exec returns the next regex match as an array
    var match = myRegexp.exec(command);
    if (match != null) {
      //Index 1 in the array is the captured group if it exists
      //Index 0 is the matched text, which we use if no captured group exists
      splits.push(match[1] ? match[1] : match[0]);
    }
  } while (match != null);

  return splits;
}
Thread Thread
 
09wattry profile image
Ryan • • Edited

What about double quotes enclosed within single quotes, echo 'And he said: "My name is!" ${name}'. It's a contrived example but you get the gist.

Thread Thread
 
vonheikemen profile image
Heiker •

For that you would get.

[ "echo", "'And", "he", "said:", "My name is!", "aa'" ]

See, the double quotes stayed, but everything else is broken. Having a bulletproof regex is a real pain. If you want to improve it, you have to modify this var myRegexp = /[^\s"]+|"([^"]*)"/gi;.