bottleneckp - npm Package Compare versions

Comparing version 1.0.1 to 1.0.2

lib/Bottleneck.js

@@ -60,10 +60,24 @@ var PriorityQueue = function(size) {
 
-function Bottleneck(maxConcurrent, rateLimit, priorityRange) {
+function Bottleneck(maxConcurrent, rateLimit, priorityRange, defaultPriority) {
     if(isNaN(maxConcurrent) || isNaN(rateLimit)) {
         throw "maxConcurrent and rateLimit must be numbers";
     }
+
+    priorityRange = priorityRange || 1;
+    if(isNaN(priorityRange)) {
+        throw "priorityRange must be a number";
+    }
+    priorityRange = parseInt(priorityRange);
+    defaultPriority = defaultPriority ? defaultPriority : Math.floor(priorityRange/2);
+    if(isNaN(defaultPriority)) {
+        throw "defaultPriority must be a number";
+    }
+    defaultPriority = defaultPriority >= priorityRange ? priorityRange-1 : defaultPriority;
+    defaultPriority = parseInt(defaultPriority);
+
     this.rateLimit = parseInt(rateLimit);
     this.maxConcurrent = this.rateLimit ? 1 : parseInt(maxConcurrent);
-    this._waitingClients = new PriorityQueue(priorityRange || 1);
-    this._priorityRange = priorityRange || 1;
+    this._waitingClients = new PriorityQueue(priorityRange);
+    this._priorityRange = priorityRange;
+    this._defaultPriority = defaultPriority;
     this._nextRequest = Date.now();
@@ -81,3 +95,3 @@ this._tasksRunning = 0;
     }
-    priority = priority && + priority | 0 || 0;
+    priority = priority && + priority | 0 || self._defaultPriority;
     priority = priority > self._priorityRange-1 ? self._priorityRange-1 : priority;
@@ -84,0 +98,0 @@ this._waitingClients.enqueue(clientCallback, priority);

package.json

 {
   "name": "bottleneckp",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "asynchronous rate limiter with priority",
   "main": "./lib/Bottleneck.js",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "./node_modules/mocha/bin/mocha test/index.js"
   },
@@ -27,3 +27,6 @@ "repository": {
   },
-  "homepage": "https://github.com/darrenqc/BottleneckP#readme"
+  "homepage": "https://github.com/darrenqc/BottleneckP#readme",
+  "devDependencies": {
+    "mocha":"*"
+  }
 }

New alerts

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

Fixed alerts

No tests

Quality

Package does not have any tests. This is a strong signal of a poorly maintained or low quality package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

13356

increased by83.18%

Number of package files

8

increased by33.33%

Lines of code

329

increased by117.88%

Number of low quality alerts

0

decreased by-100%

Worsened metrics

Dev dependency count

1

increased byInfinity%

Number of low supply chain risk alerts

2

increased byInfinity%

No dependency changes