bunqueue 2.8.20 → 2.8.21

package/dist/application/monitoringChecks.js

@@ -14,11 +14,11 @@ export function createMonitoringState() {
     };
 }
 /** Config from env vars */
-const QUEUE_IDLE_THRESHOLD_MS = parseInt(process.env.QUEUE_IDLE_THRESHOLD_MS ?? '30000');
-const QUEUE_SIZE_THRESHOLD = parseInt(process.env.QUEUE_SIZE_THRESHOLD ?? '0'); // 0 = disabled
-const MEMORY_WARNING_MB = parseInt(process.env.MEMORY_WARNING_MB ?? '0'); // 0 = disabled
-const STORAGE_WARNING_MB = parseInt(process.env.STORAGE_WARNING_MB ?? '0'); // 0 = disabled
-const WORKER_OVERLOAD_THRESHOLD_MS = parseInt(process.env.WORKER_OVERLOAD_THRESHOLD_MS ?? '30000');
+const QUEUE_IDLE_THRESHOLD_MS = parseInt(process.env.QUEUE_IDLE_THRESHOLD_MS ?? '30000', 10);
+const QUEUE_SIZE_THRESHOLD = parseInt(process.env.QUEUE_SIZE_THRESHOLD ?? '0', 10); // 0 = disabled
+const MEMORY_WARNING_MB = parseInt(process.env.MEMORY_WARNING_MB ?? '0', 10); // 0 = disabled
+const STORAGE_WARNING_MB = parseInt(process.env.STORAGE_WARNING_MB ?? '0', 10); // 0 = disabled
+const WORKER_OVERLOAD_THRESHOLD_MS = parseInt(process.env.WORKER_OVERLOAD_THRESHOLD_MS ?? '30000', 10);
 /** Check all monitoring conditions — called from cleanup interval (10s) */
 export function runMonitoringChecks(ctx) {
     if (!ctx.dashboardEmit)

package/dist/cli/commands/server.js

@@ -11,7 +11,7 @@ import { bootServer } from '../../infrastructure/server/bootstrap';
 /** Validate port number */
 function validatePort(value, name, defaultPort) {
     const port = parseInt(value, 10);
-    if (isNaN(port) || port < 1 || port > 65535) {
+    if (Number.isNaN(port) || port < 1 || port > 65535) {
         console.warn(`Warning: Invalid ${name} "${value}". Using default ${defaultPort}.`);
         return defaultPort;
     }

package/dist/cli/index.js

@@ -22,7 +22,7 @@ function resolveEnvPort(currentPort) {
     if (!envPort)
         return currentPort;
     const parsed = parseInt(envPort, 10);
-    if (isNaN(parsed) || parsed < 1 || parsed > 65535) {
+    if (Number.isNaN(parsed) || parsed < 1 || parsed > 65535) {
         console.warn(`Warning: Invalid env port "${envPort}". Using ${currentPort}.`);
         return currentPort;
     }
@@ -129,7 +129,7 @@ function applyPortFlag(allArgs, i, state) {
         return i;
     }
     const parsed = parseInt(nextArg, 10);
-    if (isNaN(parsed) || parsed < 1 || parsed > 65535) {
+    if (Number.isNaN(parsed) || parsed < 1 || parsed > 65535) {
         console.warn(`Warning: Invalid port "${nextArg}". Using default port 6789.`);
         state.port = 6789;
     }
@@ -208,7 +208,7 @@ export function parseGlobalOptions() {
         else if (arg.startsWith('--port=')) {
             const raw = arg.slice(7);
             const parsed = parseInt(raw, 10);
-            if (isNaN(parsed) || parsed < 1 || parsed > 65535) {
+            if (Number.isNaN(parsed) || parsed < 1 || parsed > 65535) {
                 console.warn(`Warning: Invalid port "${raw}". Using default port 6789.`);
                 hp.port = 6789;
             }

package/dist/client/queue/operations/management.d.ts

@@ -8,7 +8,7 @@ interface ManagementContext {
     embedded: boolean;
     tcp: TcpConnectionPool | null;
 }
-/** Remove a job (sync) */
+/** Remove a job (sync, fire-and-forget; use removeAsync to await the removal) */
 export declare function remove(ctx: ManagementContext, id: string): void;
 /** Remove a job (async) */
 export declare function removeAsync(ctx: ManagementContext, id: string): Promise<void>;

package/dist/client/queue/operations/management.js

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-non-null-assertion, @typescript-eslint/no-floating-promises, @typescript-eslint/no-unnecessary-condition */
 /**
  * Queue Management Operations
  * remove, retry, clean, promote, updateProgress, logs
@@ -6,12 +5,13 @@
 import { getSharedManager } from '../../manager';
 import { jobId } from '../../../domain/types/job';
 // ============ Remove Operations ============
-/** Remove a job (sync) */
+/** Remove a job (sync, fire-and-forget; use removeAsync to await the removal) */
 export function remove(ctx, id) {
+    // `void`: the cancellation is intentionally not awaited here (sync API).
     if (ctx.embedded)
-        getSharedManager().cancel(jobId(id));
+        void getSharedManager().cancel(jobId(id));
     else
-        ctx.tcp.send({ cmd: 'Cancel', id });
+        void ctx.tcp.send({ cmd: 'Cancel', id });
 }
 /** Remove a job (async) */
 export async function removeAsync(ctx, id) {

package/dist/domain/queue/temporalManager.js

@@ -17,7 +17,17 @@ export class TemporalManager {
      * Ordered by createdAt for efficient cleanQueue range queries
      * Uses equality check on jobId to prevent duplicate entries for the same job
      */
-    temporalIndex = new SkipList((a, b) => a.createdAt - b.createdAt, 16, 0.5, (a, b) => a.jobId === b.jobId);
+    // Total-order comparator: createdAt first, then jobId as a tie-break. Without
+    // the tie-break, a batch of jobs sharing one createdAt (the addBulk case —
+    // `now` is captured once) makes every node compare-equal, which (a) turns
+    // SkipList.insert's duplicate-check scan into O(n) per insert => O(n²) for the
+    // batch, and (b) makes SkipList.delete remove the WRONG same-createdAt node
+    // (it stops at the first compare-equal node). A total order fixes both: the
+    // dedup scan and delete both resolve to the exact (createdAt, jobId) node in
+    // O(log n). jobId is a string (UUIDv7 by default, or any custom id), so its
+    // lexicographic comparison is a valid total order in every case. Equality is
+    // still by jobId (now reached only for a true duplicate).
+    temporalIndex = new SkipList((a, b) => a.createdAt - b.createdAt || (a.jobId < b.jobId ? -1 : a.jobId > b.jobId ? 1 : 0), 16, 0.5, (a, b) => a.jobId === b.jobId);
     /** Set of delayed job IDs for tracking when they become ready */
     delayedJobIds = new Set();
     /**

package/dist/infrastructure/cloud/cloudAgent.d.ts

@@ -22,7 +22,6 @@ export declare class CloudAgent {
     private statsUpdateTimer;
     private unsubscribeEvents;
     private sequenceId;
-    private snapshotCount;
     private stopped;
     private serverHandles?;
     /** Event buffer — flushed into each HTTP snapshot */

package/dist/infrastructure/cloud/cloudAgent.js

@@ -27,7 +27,6 @@ export class CloudAgent {
     statsUpdateTimer = null;
     unsubscribeEvents = null;
     sequenceId = 0;
-    snapshotCount = 0;
     stopped = false;
     serverHandles;
     /** Event buffer — flushed into each HTTP snapshot */
@@ -164,7 +163,6 @@ export class CloudAgent {
     /** Collect and send a snapshot via HTTP */
     async sendSnapshot(_forceHeavy = false) {
         try {
-            this.snapshotCount++;
             const snapshot = await collectSnapshot({
                 queueManager: this.queueManager,
                 instanceId: this.instanceId,

package/dist/infrastructure/server/http.js

@@ -253,8 +253,8 @@ async function routeRequest(req, path, ctx, corsOrigins) {
     }
     if (path === '/dashboard/queues' && method === 'GET') {
         const url = new URL(req.url);
-        const limit = Math.min(Math.max(parseInt(url.searchParams.get('limit') ?? '100') || 100, 1), 500);
-        const offset = Math.max(parseInt(url.searchParams.get('offset') ?? '0') || 0, 0);
+        const limit = Math.min(Math.max(parseInt(url.searchParams.get('limit') ?? '100', 10) || 100, 1), 500);
+        const offset = Math.max(parseInt(url.searchParams.get('offset') ?? '0', 10) || 0, 0);
         return dashboardQueuesEndpoint(ctx.queueManager, limit, offset, corsOrigins);
     }
     const dashQueueMatch = path.match(RE_DASHBOARD_QUEUE_DETAIL);

package/dist/infrastructure/server/httpRouteQueues.js

@@ -60,7 +60,7 @@ async function routeJobOps(req, path, method, ctx, cors) {
             return jsonResponse(r, r.ok ? 200 : 400, cors);
         }
         if (method === 'GET') {
-            const timeout = parseInt(new URL(req.url).searchParams.get('timeout') ?? '0');
+            const timeout = parseInt(new URL(req.url).searchParams.get('timeout') ?? '0', 10);
             const r = await handleCommand({ cmd: 'PULL', queue, timeout }, ctx);
             return jsonResponse(r, 200, cors);
         }
@@ -125,8 +125,8 @@ async function routeJobOps(req, path, method, ctx, cors) {
                 : stateValues;
         const limitParam = url.searchParams.get('limit');
         const offsetParam = url.searchParams.get('offset');
-        const limit = limitParam ? parseInt(limitParam) : undefined;
-        const offset = offsetParam ? parseInt(offsetParam) : undefined;
+        const limit = limitParam ? parseInt(limitParam, 10) : undefined;
+        const offset = offsetParam ? parseInt(offsetParam, 10) : undefined;
         const r = await handleCommand({
             cmd: 'GetJobs',
             queue,

package/dist/infrastructure/server/sseHandler.js

@@ -116,6 +116,13 @@ export class SseHandler {
     // ── Job event broadcasting ─────────────────────────────────
     /** Broadcast job event to matching clients (with typed SSE event field) */
     broadcast(event) {
+        // No clients => nothing to send, and nothing to buffer that anyone could
+        // replay. Mirrors wsHandler.broadcast. Without this, every job event still
+        // paid JSON.stringify + encode + ring buffer + an O(queue size) per-event
+        // getQueueJobCounts, turning a bulk push into O(N²) even with no dashboard
+        // attached (the common high-throughput case).
+        if (this.clients.size === 0)
+            return;
         const id = ++this.eventId;
         const eventName = EVENT_MAP[event.eventType] ?? `job:${event.eventType}`;
         const eventData = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bunqueue",
-  "version": "2.8.20",
+  "version": "2.8.21",
   "description": "High-performance job queue for Bun & AI agents. SQLite persistence, cron scheduling, priorities, retries, DLQ, webhooks, native MCP server. Zero external dependencies.",
   "type": "module",
   "main": "dist/main.js",