chore(deps): replace npm-run-all2 with concurrently better exit handling (#2356)

Author: ghiscoding

demos/aurelia/package.json

@@ -56,7 +56,6 @@
     "cypress": "catalog:",
     "cypress-real-events": "catalog:",
     "dompurify": "catalog:",
-    "npm-run-all2": "catalog:",
     "sass": "catalog:",
     "tslib": "catalog:",
     "typescript": "catalog:",

package.json

@@ -15,11 +15,11 @@
     "docs:1": "// install the framework you want to use (ie: 'angular:install') or 'pnpm install' to install everything",
     "docs:2": "// Start Dev by choosing the appropriate framework below (ie: 'dev:angular'), the single word 'dev' is to start the Vanilla flavor (default)",
     "docs:3": "// Start Cypress tests by choosing your framework (ie: 'angular:cypress'), default is 'cypress' to test Vanilla flavor (default)",
-    "dev": "pnpm build:universal && run-p dev:watch vite:watch",
-    "dev:angular": "pnpm build:universal && run-p universal:watch angular:watch",
-    "dev:aurelia": "pnpm build:universal && pnpm aurelia:build:framework && run-p universal:watch aurelia:watch",
-    "dev:react": "pnpm build:universal && pnpm react:build:framework && run-p universal:watch react:watch",
-    "dev:vue": "pnpm build:universal && pnpm vue:build:framework && run-p universal:watch vue:watch",
+    "dev": "pnpm build:universal && concurrently --kill-others-on-fail --prefix none --raw \"pnpm dev:watch\" \"pnpm vite:watch\"",
+    "dev:angular": "pnpm build:universal && concurrently --kill-others-on-fail --prefix none --raw \"pnpm universal:watch\" \"pnpm angular:watch\"",
+    "dev:aurelia": "pnpm build:universal && pnpm aurelia:build:framework && concurrently --kill-others-on-fail --prefix none --raw \"pnpm universal:watch\" \"pnpm aurelia:watch\"",
+    "dev:react": "pnpm build:universal && pnpm react:build:framework && concurrently --kill-others-on-fail --prefix none --raw \"pnpm universal:watch\" \"pnpm react:watch\"",
+    "dev:vue": "pnpm build:universal && pnpm vue:build:framework && concurrently --kill-others-on-fail --prefix none --raw \"pnpm universal:watch\" \"pnpm vue:watch\"",
     "dev:watch": "lerna watch --no-bail --scope=@slickgrid-universal/* --glob=\"src/**/*.{ts,scss}\" --ignored=\"**/*.spec.ts\" -- cross-env-shell pnpm run -r --filter $LERNA_PACKAGE_NAME dev",
     "build": "pnpm clean && pnpm lint && pnpm build:universal && pnpm build:wrappers",
     "bundle:zip": "pnpm clean && pnpm lint && pnpm build:universal && pnpm -r --stream --filter=./packages/** run bundle:zip",
@@ -114,7 +114,7 @@
     "globals": "catalog:",
     "jsdom": "catalog:",
     "jsdom-global": "catalog:",
-    "npm-run-all2": "catalog:",
+    "concurrently": "catalog:",
     "oxlint": "catalog:",
     "prettier": "^3.8.1",
     "remove-glob": "catalog:",

packages/common/build-watch.mjs

@@ -1,6 +1,6 @@
-import { copyfiles } from 'native-copyfiles';
 import { exec } from 'node:child_process';
 import { basename } from 'node:path';
+import { copyfiles } from 'native-copyfiles';
 
 /**
  * Special script used by the Watch in Development which will compile TypeScript files with tsc incremental and/or SASS files when changes occurs.
@@ -31,7 +31,7 @@ async function run() {
     for (const changedFile of changedFiles) {
       const fileWithExtension = basename(changedFile);
       const relativeFile = `./src/styles/${fileWithExtension}`;
-      copyfiles([relativeFile, 'dist/styles/sass'], { flat: true }, (err) => {
+      copyfiles([relativeFile], 'dist/styles/sass', { flat: true }, (err) => {
         err ? console.error(err) : console.log(`Copied "${fileWithExtension}" to "dist/styles/sass"`);
       });
     }

packages/common/package.json

@@ -34,7 +34,7 @@
   "scripts": {
     "are-types-wrong": "pnpx @arethetypeswrong/cli --pack .",
     "build": "pnpm run clean && tsc && pnpm run sass:bundle",
-    "sass:bundle": "run-s sass:build sass:copy",
+    "sass:bundle": "pnpm run sass:build && pnpm run sass:copy",
     "build:incremental": "tsc --incremental --declaration",
     "clean": "remove dist tsconfig.tsbuildinfo",
     "dev": "node build-watch.mjs",
@@ -46,15 +46,15 @@
     "sass-build-task:scss-compile:material-lite": "sass src/styles/slickgrid-theme-material.lite.scss dist/styles/css/slickgrid-theme-material.lite.css --style=compressed --quiet-deps --no-source-map --load-path=node_modules",
     "sass-build-task:scss-compile:salesforce": "sass src/styles/slickgrid-theme-salesforce.scss dist/styles/css/slickgrid-theme-salesforce.css --style=compressed --quiet-deps --no-source-map --load-path=node_modules",
     "sass-build-task:scss-compile:salesforce-lite": "sass src/styles/slickgrid-theme-salesforce.lite.scss dist/styles/css/slickgrid-theme-salesforce.lite.css --style=compressed --quiet-deps --no-source-map --load-path=node_modules",
-    "sass:build": "run-p sass-build-task:scss-compile:* && pnpm run sass:build:postcss",
+    "sass:build": "concurrently --prefix none \"pnpm:sass-build-task:scss-compile:*\" && pnpm run sass:build:postcss",
     "sass:build:postcss": "postcss --no-map --use cssnano --use autoprefixer --dir dist/styles/css dist/styles/css --style=compressed --quiet-deps --no-source-map",
     "sass:copy": "copyfiles \"./src/styles/*.scss\" dist/styles/sass --flat --stat",
     "sass:watch:default": "npm run sass-build-task:scss-compile:default -- --watch",
     "sass:watch:bootstrap": "npm run sass-build-task:scss-compile:bootstrap -- --watch",
     "sass:watch:material": "npm run sass-build-task:scss-compile:material -- --watch",
     "sass:watch:salesforce": "npm run sass-build-task:scss-compile:salesforce -- --watch",
-    "sass:watch": "run-p sass:watch:*",
-    "sass:watch-all": "run-p \"sass-build-task:scss-compile:* --watch\" --"
+    "sass:watch": "concurrently --prefix none \"pnpm:sass:watch:*\"",
+    "sass:watch-all": "concurrently --prefix none \"pnpm:sass-build-task:scss-compile:* --watch\""
   },
   "browserslist": [
     "baseline widely available"
@@ -77,8 +77,8 @@
   "devDependencies": {
     "autoprefixer": "^10.4.23",
     "cssnano": "^7.1.2",
+    "concurrently": "catalog:",
     "native-copyfiles": "catalog:",
-    "npm-run-all2": "catalog:",
     "postcss": "^8.5.6",
     "postcss-cli": "^11.0.1",
     "sass": "catalog:"

pnpm-lock.yaml

@@ -18,6 +18,7 @@ catalog:
   '@vitest/coverage-v8': ^4.0.18
   '@vitest/ui': ^4.0.18
   bootstrap: ^5.3.8
+  concurrently: ^9.2.1
   cross-env: ^10.1.0
   cypress: ^15.9.0
   cypress-real-events: ^1.15.0
@@ -30,7 +31,6 @@ catalog:
   jsdom-global: ^3.0.2
   multiple-select-vanilla: ^4.4.1
   native-copyfiles: ^2.0.1
-  npm-run-all2: ^8.0.4
   oxlint: ^1.42.0
   remove-glob: ^1.2.0
   rxjs: ^7.8.2