@parcel/workers 2.8.1 → 2.8.3

package/lib/Worker.js

@@ -61,8 +61,9 @@ class Worker extends _events().default {
 
     for (let i = 0; i < filteredArgs.length; i++) {
       let arg = filteredArgs[i];
+      let isArgWithParam = (arg === '-r' || arg === '--require') && filteredArgs[i + 1] === '@parcel/register' || arg === '--title';
 
-      if ((arg === '-r' || arg === '--require') && filteredArgs[i + 1] === '@parcel/register') {
+      if (isArgWithParam) {
         filteredArgs.splice(i, 2);
         i--;
       }

package/lib/WorkerFarm.js

@@ -136,7 +136,16 @@ class WorkerFarm extends _events().default {
 
     this.localWorker = require(this.options.workerPath);
     this.localWorkerInit = this.localWorker.childInit != null ? this.localWorker.childInit() : null;
-    this.run = this.createHandle('run');
+    this.run = this.createHandle('run'); // Worker thread stdout is by default piped into the process stdout, if there are enough worker
+    // threads to exceed the default listener limit, then anything else piping into stdout will trigger
+    // the `MaxListenersExceededWarning`, so we should ensure the max listeners is at least equal to the
+    // number of workers + 1 for the main thread.
+    //
+    // Note this can't be fixed easily where other things pipe into stdout -  even after starting > 10 worker
+    // threads `process.stdout.getMaxListeners()` will still return 10, however adding another pipe into `stdout`
+    // will give the warning with `<worker count + 1>` as the number of listeners.
+
+    process.stdout.setMaxListeners(Math.max(process.stdout.getMaxListeners(), WorkerFarm.getNumWorkers() + 1));
     this.startMaxWorkers();
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@parcel/workers",
-  "version": "2.8.1",
+  "version": "2.8.3",
   "description": "Blazing fast, zero configuration web application bundler",
   "license": "MIT",
   "publishConfig": {
@@ -21,20 +21,20 @@
     "node": ">= 12.0.0"
   },
   "dependencies": {
-    "@parcel/diagnostic": "2.8.1",
-    "@parcel/logger": "2.8.1",
-    "@parcel/types": "2.8.1",
-    "@parcel/utils": "2.8.1",
+    "@parcel/diagnostic": "2.8.3",
+    "@parcel/logger": "2.8.3",
+    "@parcel/types": "2.8.3",
+    "@parcel/utils": "2.8.3",
     "chrome-trace-event": "^1.0.2",
     "nullthrows": "^1.1.1"
   },
   "peerDependencies": {
-    "@parcel/core": "^2.8.1"
+    "@parcel/core": "^2.8.3"
   },
   "browser": {
     "./src/cpuCount.js": false,
     "./src/process/ProcessWorker.js": false,
     "./src/threads/ThreadsWorker.js": false
   },
-  "gitHead": "f8d3fc30ca5b33d8f8674525f2a741d662c5986a"
+  "gitHead": "349a6caf40ec8abb6a49fcae0765f8f8deb2073d"
 }

package/src/Worker.js

@@ -53,10 +53,11 @@ export default class Worker extends EventEmitter {
 
     for (let i = 0; i < filteredArgs.length; i++) {
       let arg = filteredArgs[i];
-      if (
-        (arg === '-r' || arg === '--require') &&
-        filteredArgs[i + 1] === '@parcel/register'
-      ) {
+      let isArgWithParam =
+        ((arg === '-r' || arg === '--require') &&
+          filteredArgs[i + 1] === '@parcel/register') ||
+        arg === '--title';
+      if (isArgWithParam) {
         filteredArgs.splice(i, 2);
         i--;
       }

package/src/WorkerFarm.js

@@ -101,6 +101,21 @@ export default class WorkerFarm extends EventEmitter {
       this.localWorker.childInit != null ? this.localWorker.childInit() : null;
     this.run = this.createHandle('run');
 
+    // Worker thread stdout is by default piped into the process stdout, if there are enough worker
+    // threads to exceed the default listener limit, then anything else piping into stdout will trigger
+    // the `MaxListenersExceededWarning`, so we should ensure the max listeners is at least equal to the
+    // number of workers + 1 for the main thread.
+    //
+    // Note this can't be fixed easily where other things pipe into stdout -  even after starting > 10 worker
+    // threads `process.stdout.getMaxListeners()` will still return 10, however adding another pipe into `stdout`
+    // will give the warning with `<worker count + 1>` as the number of listeners.
+    process.stdout.setMaxListeners(
+      Math.max(
+        process.stdout.getMaxListeners(),
+        WorkerFarm.getNumWorkers() + 1,
+      ),
+    );
+
     this.startMaxWorkers();
   }