feat: allow sort scripts without run-s

index.js

@@ -148,6 +148,33 @@ const hasDevDependency = (dependency, packageJson) => {
   )
 }
 
+const runSValues = [
+  /npm-run-all .* -s(?:$|\s|&|>|<)/,
+  /npm-run-all .* --sequential/,
+  /npm-run-all .* --serial/,
+  /run-s/,
+]
+
+const hasRunS = (packageJson) => {
+  if (!hasDevDependency('npm-run-all', packageJson)) {
+    return false
+  }
+
+  const scripts = packageJson.scripts
+  const betterScripts = packageJson.betterScripts
+  if (scripts) {
+    return Object.values(scripts).some((script) =>
+      runSValues.some((runSValue) => runSValue.test(script)),
+    )
+  }
+
+  if (betterScripts) {
+    return Object.values(betterScripts).some((script) =>
+      runSValues.some((runSValue) => runSValue.test(script)),
+    )
+  }
+}
+
 const sortScripts = onObject((scripts, packageJson) => {
   const names = Object.keys(scripts)
   const prefixable = new Set()
@@ -161,7 +188,7 @@ const sortScripts = onObject((scripts, packageJson) => {
     return name
   })
 
-  if (!hasDevDependency('npm-run-all', packageJson)) {
+  if (!hasRunS(packageJson)) {
     keys.sort()
   }
 

tests/scripts.js

@@ -18,6 +18,23 @@ const fixture = {
   'pre-fetch-info': 'foo',
 }
 
+const fixtureWithRunS = {
+  test: 'node test.js',
+  multiply: '2 * 3', // between p(ostinstall) and install
+  watch: 'watch things',
+  prewatch: 'echo "about to watch"',
+  postinstall: 'echo "Installed"',
+  preinstall: 'echo "Installing"',
+  start: 'node server.js',
+  posttest: 'run-s abc def',
+  pretest: 'xyz',
+  postprettier: 'echo "so pretty"',
+  preprettier: 'echo "not pretty"',
+  prettier: 'prettier -l "**/*.js"',
+  prepare: 'npm run build',
+  'pre-fetch-info': 'foo',
+}
+
 const expectAllSorted = {
   preinstall: 'echo "Installing"',
   postinstall: 'echo "Installed"',
@@ -38,7 +55,7 @@ const expectAllSorted = {
 const expectPreAndPostSorted = {
   pretest: 'xyz',
   test: 'node test.js',
-  posttest: 'abc',
+  posttest: 'run-s abc def',
   multiply: '2 * 3',
   prewatch: 'echo "about to watch"',
   watch: 'watch things',
@@ -53,18 +70,39 @@ const expectPreAndPostSorted = {
 }
 
 for (const field of ['scripts', 'betterScripts']) {
-  test(`${field} when npm-run-all is not a dev dependency`, macro.sortObject, {
+  test(`${field} when npm-run-all is NOT a dev dependency`, macro.sortObject, {
     value: { [field]: fixture },
     expect: { [field]: expectAllSorted },
   })
-  test(`${field} when npm-run-all is a dev dependency`, macro.sortObject, {
-    value: {
-      [field]: fixture,
-      devDependencies: { 'npm-run-all': '^1.0.0' },
-    },
-    expect: {
-      [field]: expectPreAndPostSorted,
-      devDependencies: { 'npm-run-all': '^1.0.0' },
+
+  for (const type of ['run-s']) {
+    test(
+      `${field} when npm-run-all IS a dev dependency, and IS used in scripts in form of ${type}`,
+      macro.sortObject,
+      {
+        value: {
+          [field]: fixtureWithRunS,
+          devDependencies: { 'npm-run-all': '^1.0.0' },
+        },
+        expect: {
+          [field]: expectPreAndPostSorted,
+          devDependencies: { 'npm-run-all': '^1.0.0' },
+        },
+      },
+    )
+  }
+  test(
+    `${field} when npm-run-all IS a dev dependency, but is NOT used in scripts`,
+    macro.sortObject,
+    {
+      value: {
+        [field]: fixture,
+        devDependencies: { 'npm-run-all': '^1.0.0' },
+      },
+      expect: {
+        [field]: expectAllSorted,
+        devDependencies: { 'npm-run-all': '^1.0.0' },
+      },
     },
-  })
+  )
 }