batch-cluster - npm Package Compare versions

Comparing version 12.0.0 to 12.1.0

CHANGELOG.md

@@ -21,2 +21,9 @@ # Changelog
 
+## v12.1.0
+
+- 🐞 `pidExists` now handles `EPERM` properly (previous implementation would
+  mischaracterize pids as being dead due to insufficient permissions)
+
+- 📦 Updated development dependencies and rebuilt docs
+
 ## v12.0.0
@@ -23,0 +30,0 @@

dist/BatchProcess.js

@@ -39,6 +39,6 @@ "use strict";
     constructor(proc, opts, onIdle) {
+        _BatchProcess_instances.add(this);
         this.proc = proc;
         this.opts = opts;
         this.onIdle = onIdle;
-        _BatchProcess_instances.add(this);
         this.start = Date.now();
@@ -45,0 +45,0 @@ _BatchProcess_lastHealthCheck.set(this, Date.now());

dist/Pids.js

@@ -25,5 +25,10 @@ "use strict";
     catch (err) {
+        // We're expecting err.code to be either "EPERM" (if we don't have
+        // permission to send `pid` and message), or "ESRCH" if that pid doesn't
+        // exist. EPERM means it _does_ exist!
+        if (err.code === "EPERM")
+            return true;
         // failed to get priority--assume the pid is gone.
+        return false;
     }
-    return false;
 }
@@ -30,0 +35,0 @@ exports.pidExists = pidExists;

dist/Rate.js

@@ -33,5 +33,5 @@ "use strict";
     constructor(periodMs = BatchClusterOptions_1.minuteMs, warmupMs = BatchClusterOptions_1.secondMs) {
+        _Rate_instances.add(this);
         this.periodMs = periodMs;
         this.warmupMs = warmupMs;
-        _Rate_instances.add(this);
         _Rate_start.set(this, Date.now());
@@ -38,0 +38,0 @@ _Rate_priorEventTimestamps.set(this, []);

dist/Task.js

@@ -32,5 +32,5 @@ "use strict";
     constructor(command, parser) {
+        _Task_instances.add(this);
         this.command = command;
         this.parser = parser;
-        _Task_instances.add(this);
         this.taskId = _taskId++;
@@ -37,0 +37,0 @@ _Task_opts.set(this, void 0);

package.json

 {
   "name": "batch-cluster",
-  "version": "12.0.0",
+  "version": "12.1.0",
   "description": "Manage a cluster of child processes",
@@ -59,5 +59,5 @@ "main": "dist/BatchCluster.js",
     "@types/mocha": "^10.0.1",
-    "@types/node": "^18.11.18",
-    "@typescript-eslint/eslint-plugin": "^5.50.0",
-    "@typescript-eslint/parser": "^5.50.0",
+    "@types/node": "^18.15.11",
+    "@typescript-eslint/eslint-plugin": "^5.58.0",
+    "@typescript-eslint/parser": "^5.58.0",
     "chai": "^4.3.7",
@@ -68,16 +68,16 @@ "chai-as-promised": "^7.1.1",
     "chai-withintoleranceof": "^1.0.1",
-    "eslint": "^8.33.0",
+    "eslint": "^8.38.0",
     "eslint-plugin-import": "^2.27.5",
     "mocha": "^10.2.0",
     "organize-imports-cli": "^0.10.0",
-    "prettier": "^2.8.3",
-    "rimraf": "^4.1.2",
+    "prettier": "^2.8.7",
+    "rimraf": "^5.0.0",
     "seedrandom": "^3.0.5",
     "serve": "^14.2.0",
     "source-map-support": "^0.5.21",
-    "split2": "^4.1.0",
+    "split2": "^4.2.0",
     "timekeeper": "^2.2.0",
-    "typedoc": "^0.23.24",
-    "typescript": "~4.9.5"
+    "typedoc": "^0.24.1",
+    "typescript": "~5.0.4"
   }
 }

No alert changes

Improved metrics

Total package byte prevSize

214545

0.27%

Lines of code

2950

0.17%

No dependency changes