fix: prevent sorting when npm-run-all2 is used (#361)

Add support for npm-run-all2 to prevent script sorting when sequential wildcard execution is needed.

Closes #359

Author: ikemo3

index.js

@@ -223,13 +223,16 @@ const hasDevDependency = (dependency, packageJson) => {
 }
 
 const runSRegExp =
-  /(?<=^|[\s&;<>|(])(?:run-s|npm-run-all .*(?:--sequential|--serial|-s))(?=$|[\s&;<>|)])/
+  /(?<=^|[\s&;<>|(])(?:run-s|npm-run-all2? .*(?:--sequential|--serial|-s))(?=$|[\s&;<>|)])/
 
 const isSequentialScript = (command) =>
   command.includes('*') && runSRegExp.test(command)
 
 const hasSequentialScript = (packageJson) => {
-  if (!hasDevDependency('npm-run-all', packageJson)) {
+  if (
+    !hasDevDependency('npm-run-all', packageJson) &&
+    !hasDevDependency('npm-run-all2', packageJson)
+  ) {
     return false
   }
   const scripts = ['scripts', 'betterScripts'].flatMap((field) =>

tests/scripts.js

@@ -67,6 +67,15 @@ function sortScriptsWithNpmRunAll(script) {
 
   return Object.keys(sortPackageJson(packageJson).scripts)
 }
+function sortScriptsWithNpmRunAll2(script) {
+  const packageJson = {
+    scripts: { z: 'z', a: 'a', maybeRunS: script },
+    devDependencies: { 'npm-run-all2': '^1.0.0' },
+  }
+
+  return Object.keys(sortPackageJson(packageJson).scripts)
+}
+
 const sortedScripts = ['a', 'maybeRunS', 'z']
 const unsortedScripts = ['z', 'a', 'maybeRunS']
 for (const { script, expected } of [
@@ -108,6 +117,10 @@ for (const { script, expected } of [
   test(`command: '${script}'`, (t) => {
     t.deepEqual(sortScriptsWithNpmRunAll(script), expected)
   })
+
+  test(`command: '${script}' with npm-run-all2`, (t) => {
+    t.deepEqual(sortScriptsWithNpmRunAll2(script), expected)
+  })
 }
 
 for (const field of ['scripts', 'betterScripts']) {
@@ -127,6 +140,16 @@ for (const field of ['scripts', 'betterScripts']) {
   })
 }
 
+// npm-run-all2
+for (const { script, expected } of [
+  // Should NOT sort
+  { script: 'npm-run-all2 -s "lint:*"', expected: unsortedScripts },
+]) {
+  test(`command: '${script}' with npm-run-all2`, (t) => {
+    t.deepEqual(sortScriptsWithNpmRunAll2(script), expected)
+  })
+}
+
 for (const field of ['scripts', 'betterScripts']) {
   test(
     `${field} does not sort pre/post scripts with colon together`,