npm-run-all2 - npm Package Compare versions

Comparing version 6.2.0 to 6.2.1

bin/common/bootstrap.js

@@ -11,3 +11,2 @@ /**
 // ------------------------------------------------------------------------------
-/* eslint-disable no-process-exit */
 
@@ -51,3 +50,1 @@ module.exports = function bootstrap (name) {
 }
-
-/* eslint-enable */

bin/common/parse-cli-args.js

@@ -8,4 +8,2 @@ /**
 
-/* eslint-disable no-process-env */
-
 // ------------------------------------------------------------------------------
@@ -247,3 +245,1 @@ // Helpers
 }
-
-/* eslint-enable */

bin/npm-run-all/main.js

@@ -55,3 +55,3 @@ /**
             npmPath: argv.npmPath,
-            aggregateOutput: group.parallel && argv.aggregateOutput
+            aggregateOutput: group.parallel && argv.aggregateOutput,
           }
@@ -65,3 +65,2 @@ ))
       promise.catch(err => {
-        // eslint-disable-next-line no-console
         console.error('ERROR:', err.message)
@@ -73,3 +72,2 @@ })
   } catch (err) {
-    // eslint-disable-next-line no-console
     console.error('ERROR:', err.message)
@@ -76,0 +74,0 @@

bin/run-p/main.js

@@ -55,3 +55,3 @@ /**
         npmPath: argv.npmPath,
-        aggregateOutput: argv.aggregateOutput
+        aggregateOutput: argv.aggregateOutput,
       }
@@ -62,3 +62,2 @@ )
       promise.catch(err => {
-        // eslint-disable-next-line no-console
         console.error('ERROR:', err.message)
@@ -70,3 +69,2 @@ })
   } catch (err) {
-    // eslint-disable-next-line no-console
     console.error('ERROR:', err.message)
@@ -73,0 +71,0 @@

bin/run-s/main.js

@@ -52,3 +52,3 @@ /**
         arguments: argv.rest,
-        npmPath: argv.npmPath
+        npmPath: argv.npmPath,
       }
@@ -59,3 +59,2 @@ )
       promise.catch(err => {
-        // eslint-disable-next-line no-console
         console.error('ERROR:', err.message)
@@ -67,3 +66,2 @@ })
   } catch (err) {
-    // eslint-disable-next-line no-console
     console.error('ERROR:', err.message)
@@ -70,0 +68,0 @@

CHANGELOG.md

@@ -10,6 +10,15 @@ # Changelog
 
-## [v6.2.0](https://github.com/bcomnes/npm-run-all2/compare/v6.1.2...v6.2.0)
+## [v6.2.1](https://github.com/bcomnes/npm-run-all2/compare/v6.2.0...v6.2.1)
 
 ### Merged
 
+- Compatibility: npm, yarn and pnpm run scripts [`#143`](https://github.com/bcomnes/npm-run-all2/pull/143)
+- Use neostandard + add more static code analysis  [`#142`](https://github.com/bcomnes/npm-run-all2/pull/142)
+- Upgrade: Bump c8 from 9.1.0 to 10.0.0 [`#141`](https://github.com/bcomnes/npm-run-all2/pull/141)
+- Upgrade: Bump p-queue from 7.4.1 to 8.0.1 [`#138`](https://github.com/bcomnes/npm-run-all2/pull/138)
+
+## [v6.2.0](https://github.com/bcomnes/npm-run-all2/compare/v6.1.2...v6.2.0) - 2024-05-17
+
+### Merged
+
 - Placeholder that unfolds into multiple tasks [`#134`](https://github.com/bcomnes/npm-run-all2/pull/134)
@@ -16,0 +25,0 @@ - 📝 add compatibility note for pnpm. [`#136`](https://github.com/bcomnes/npm-run-all2/pull/136)

lib/index.js

@@ -115,5 +115,26 @@ /**
 function parsePatterns (patternOrPatterns, args) {
-  const patterns = toArray(patternOrPatterns)
-  const hasPlaceholder = patterns.some(pattern => ARGS_PATTERN.test(pattern))
+  let patterns = toArray(patternOrPatterns)
 
+  const isNPM = process.env.npm_config_user_agent && process.env.npm_config_user_agent.startsWith('npm')
+
+  if (!isNPM) {
+    // yarn | pnpm
+    patterns = patterns.map((pattern) => {
+      const match = ARGS_PATTERN.exec(pattern)
+
+      if (!match) {
+        return pattern
+      }
+
+      const patternList = pattern.split(' ')
+      const doubleDashIndex = patternList.findIndex((item) => item === '--')
+      patternList.splice(doubleDashIndex, 1)
+      pattern = patternList.join(' ')
+
+      return pattern
+    })
+  }
+
+  const hasPlaceholder = patterns.some((pattern) => ARGS_PATTERN.test(pattern))
+
   return hasPlaceholder ? applyArguments(patterns, args) : patterns
@@ -232,3 +253,3 @@ }
  */
-module.exports = function npmRunAll (patternOrPatterns, options) { // eslint-disable-line complexity
+module.exports = function npmRunAll (patternOrPatterns, options) {
   const stdin = (options && options.stdin) || null
@@ -295,3 +316,3 @@ const stdout = (options && options.stdout) || null
             lastPrefix: null,
-            lastIsLinebreak: true
+            lastIsLinebreak: true,
           },
@@ -303,3 +324,3 @@ printName,
           npmPath,
-          aggregateOutput
+          aggregateOutput,
         })
@@ -306,0 +327,0 @@ })

lib/read-package-json.js

@@ -30,4 +30,4 @@ /**
     taskList: Object.keys(body.scripts || {}),
-    packageInfo: { path, body }
+    packageInfo: { path, body },
   }))
 }

lib/run-task.js

@@ -170,5 +170,5 @@ /**
       if (basename.startsWith('npx')) {
-        newBasename = basename.replace('npx', 'npm') // eslint-disable-line no-process-env
+        newBasename = basename.replace('npx', 'npm')
       } else if (basename.startsWith('pnpx')) {
-        newBasename = basename.replace('pnpx', 'pnpm') // eslint-disable-line no-process-env
+        newBasename = basename.replace('pnpx', 'pnpm')
       }
@@ -178,3 +178,3 @@
         ? path.join(path.dirname(process.env.npm_execpath), newBasename)
-        : process.env.npm_execpath // eslint-disable-line no-process-env
+        : process.env.npm_execpath
     }
@@ -196,3 +196,3 @@
 
-    const isYarn = process.env.npm_config_user_agent && process.env.npm_config_user_agent.startsWith('yarn') // eslint-disable-line no-process-env
+    const isYarn = process.env.npm_config_user_agent && process.env.npm_config_user_agent.startsWith('yarn')
 
@@ -199,0 +199,0 @@ const spawnArgs = ['run']

lib/run-tasks.js

@@ -56,3 +56,3 @@ /**
   SIGUSR1: 30,
-  SIGUSR2: 31
+  SIGUSR2: 31,
 }
@@ -146,3 +146,3 @@
       const writer = new MemoryStream(null, {
-        readable: false
+        readable: false,
       })
@@ -149,0 +149,0 @@

lib/spawn.js

@@ -18,4 +18,2 @@ /**
  */
-module.exports = require(
-  process.platform === 'win32' ? './spawn-win32' : './spawn-posix'
-)
+module.exports = process.platform === 'win32' ? require('./spawn-win32') : require('./spawn-posix')

package.json

 {
   "name": "npm-run-all2",
-  "version": "6.2.0",
+  "version": "6.2.1",
   "description": "A CLI tool to run multiple npm-scripts in parallel or sequential. (Maintainence fork)",
@@ -13,13 +13,17 @@ "bin": {
   "engines": {
-    "node": "^14.18.0 || >=16.0.0",
+    "node": "^14.18.0 || ^16.13.0 || >=18.0.0",
     "npm": ">= 8"
   },
   "scripts": {
-    "_mocha": "mocha --timeout 120000",
     "clean": "rm -rf coverage jsdoc \"test-workspace/{build,test.txt}\"",
     "docs": "jsdoc -c jsdoc.json",
-    "lint": "standard",
-    "pretest": "node scripts/make-slink.js && npm run lint",
-    "test": "c8 npm run _mocha",
-    "watch": "npm run _mocha -- --watch --growl",
+    "check:installed-check": "installed-check --ignore-dev",
+    "check:eslint": "eslint",
+    "check:knip": "knip",
+    "precheck": "node scripts/make-slink.js",
+    "check": "node ./bin/run-p/index.js check:*",
+    "pretest-mocha": "node scripts/make-slink.js",
+    "test-mocha": "c8 mocha --timeout 120000",
+    "test": "node ./bin/run-s/index.js check test-mocha",
+    "watch": "mocha --timeout 120000 --watch --growl",
     "version": "auto-changelog -p --template keepachangelog auto-changelog --breaking-pattern 'BREAKING CHANGE:' && git add CHANGELOG.md",
@@ -40,9 +44,11 @@ "preversion": "npm test",
     "auto-changelog": "^2.2.0",
-    "c8": "^9.0.0",
-    "fs-extra": "^11.1.0",
+    "c8": "^10.0.0",
+    "eslint": "^9.6.0",
     "gh-release": "^7.0.0",
+    "installed-check": "^9.3.0",
     "jsdoc": "^4.0.0",
+    "knip": "^5.23.3",
     "mocha": "^10.0.0",
-    "p-queue": "^7.3.4",
-    "standard": "^17.1.0",
+    "neostandard": "^0.11.0",
+    "rimraf": "^5.0.7",
     "yarn": "^1.12.3"
@@ -49,0 +55,0 @@ },

README.md

@@ -10,2 +10,3 @@ | index | [npm-run-all] | [run-s] | [run-p] | [Node API] |
 [![Coverage Status](https://codecov.io/gh/bcomnes/npm-run-all2/branch/master/graph/badge.svg)](https://codecov.io/gh/bcomnes/npm-run-all2)
+[![neostandard javascript style](https://img.shields.io/badge/code_style-neostandard-brightgreen?style=flat)](https://github.com/neostandard/neostandard)
 
@@ -12,0 +13,0 @@ A CLI tool to run multiple npm-scripts in parallel or sequential.

test/aggregate-output.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -43,3 +42,3 @@ * @author Toru Nagashima
       createExpectedOutput('first'),
-      ''
+      '',
     ].join('\n')
@@ -111,3 +110,3 @@
           'test-task:delayed third 3000',
-          '--silent', '--aggregate-output'
+          '--silent', '--aggregate-output',
         ],
@@ -114,0 +113,0 @@ stdout

test/argument-placeholders.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -17,8 +16,3 @@ * @author Toru Nagashima
 const nodeApi = require('../lib')
-const util = require('./lib/util')
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runPar = util.runPar
-const runSeq = util.runSeq
+const { result, removeResult, runAll, runPar, runSeq } = require('./lib/util')
 
@@ -25,0 +19,0 @@ // ------------------------------------------------------------------------------

test/common.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -16,8 +15,3 @@ * @author Toru Nagashima
 const BufferStream = require('./lib/buffer-stream')
-const util = require('./lib/util')
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runPar = util.runPar
-const runSeq = util.runSeq
+const { result, removeResult, runAll, runPar, runSeq } = require('./lib/util')
 
@@ -24,0 +18,0 @@ // ------------------------------------------------------------------------------

test/config.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -15,8 +14,3 @@ * @author Toru Nagashima
 const nodeApi = require('../lib')
-const util = require('./lib/util')
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runPar = util.runPar
-const runSeq = util.runSeq
+const { result, removeResult, runAll, runPar, runSeq } = require('./lib/util')
 
@@ -23,0 +17,0 @@ // ------------------------------------------------------------------------------

test/fail.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -15,8 +14,3 @@ * @author Toru Nagashima
 const nodeApi = require('../lib')
-const util = require('./lib/util')
-const delay = util.delay
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runPar = util.runPar
-const runSeq = util.runSeq
+const { delay, removeResult, runAll, runPar, runSeq } = require('./lib/util')
 
@@ -23,0 +17,0 @@ // ------------------------------------------------------------------------------

test/mixed.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -14,6 +13,3 @@ * @author Toru Nagashima
 const assert = require('assert').strict
-const util = require('./lib/util')
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
+const { result, removeResult, runAll } = require('./lib/util')
 
@@ -34,3 +30,3 @@ // ------------------------------------------------------------------------------
       '-p', 'test-task:append b', 'test-task:append c',
-      '-s', 'test-task:append d', 'test-task:append e'
+      '-s', 'test-task:append d', 'test-task:append e',
     ])
@@ -48,3 +44,3 @@ assert(
       '-p', 'test-task:append b', 'test-task:append c',
-      '-s', 'test-task:append d', 'test-task:append e'
+      '-s', 'test-task:append d', 'test-task:append e',
     ])
@@ -64,4 +60,4 @@ assert(
       '-s', 'test-task:append d', 'test-task:append e',
-      '-r'
+      '-r',
     ]))
 })

test/package-config.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -16,8 +15,3 @@ * @author Toru Nagashima
 const nodeApi = require('../lib')
-const util = require('./lib/util')
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runPar = util.runPar
-const runSeq = util.runSeq
+const { result, removeResult, runAll, runPar, runSeq } = require('./lib/util')
 
@@ -24,0 +18,0 @@ // ------------------------------------------------------------------------------

test/parallel.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -16,8 +15,3 @@ * @author Toru Nagashima
 const spawnWithKill = require('./lib/spawn-with-kill')
-const util = require('./lib/util')
-const delay = util.delay
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runPar = util.runPar
+const { delay, result, removeResult, runAll, runPar } = require('./lib/util')
 
@@ -24,0 +18,0 @@ // ------------------------------------------------------------------------------

test/pattern.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -16,8 +15,3 @@ * @author Toru Nagashima
 const BufferStream = require('./lib/buffer-stream')
-const util = require('./lib/util')
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runPar = util.runPar
-const runSeq = util.runSeq
+const { result, removeResult, runAll, runPar, runSeq } = require('./lib/util')
 
@@ -24,0 +18,0 @@ // ------------------------------------------------------------------------------

test/print-label.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -45,3 +44,3 @@ * @author Toru Nagashima
       '[test-task:echo abc] ',
-      '[test-task:echo abc] abc'
+      '[test-task:echo abc] abc',
     ].join('\n')
@@ -138,3 +137,3 @@
       '[test-task:echo ab  ] ',
-      '[test-task:echo ab  ] ab'
+      '[test-task:echo ab  ] ab',
     ].join('\n')
@@ -177,3 +176,3 @@
       '\n[test-task:echo abcd] ',
-      '\n[test-task:echo abcd] abcd'
+      '\n[test-task:echo abcd] abcd',
     ]
@@ -183,3 +182,3 @@ const UNEXPECTED_PATTERNS = [
       /ab(cd)?a\b/,
-      /\n\n/
+      /\n\n/,
     ]
@@ -186,0 +185,0 @@

test/print-name.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -3,0 +2,0 @@ * @author Toru Nagashima

test/sequential.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -16,8 +15,3 @@ * @author Toru Nagashima
 const spawnWithKill = require('./lib/spawn-with-kill')
-const util = require('./lib/util')
-const delay = util.delay
-const result = util.result
-const removeResult = util.removeResult
-const runAll = util.runAll
-const runSeq = util.runSeq
+const { delay, result, removeResult, runAll, runSeq } = require('./lib/util')
 
@@ -24,0 +18,0 @@ // ------------------------------------------------------------------------------

test/yarn.js

@@ -1,2 +0,1 @@
-/* eslint-env mocha */
 /**
@@ -16,5 +15,3 @@ * @author Toru Nagashima
 const BufferStream = require('./lib/buffer-stream')
-const util = require('./lib/util')
-const result = util.result
-const removeResult = util.removeResult
+const { result, removeResult } = require('./lib/util')
 
@@ -21,0 +18,0 @@ // ------------------------------------------------------------------------------

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Filesystem access

Supply chain risk

Accesses the file system, and could potentially read sensitive data.

Found 1 instance in 1 package

Improved metrics

Number of package files

75

increased by1.35%

Number of lines in readme file

95

increased by1.06%

Number of medium supply chain risk alerts

2

decreased by-33.33%

Worsened metrics

Total package byte prevSize

220597

decreased by-2.3%

Dev dependency count

11

increased by22.22%

Lines of code

3941

decreased by-4.78%

No dependency changes