@blamejs/core 0.15.40 → 0.15.42

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/CHANGELOG.md CHANGED
@@ -8,6 +8,10 @@ upgrading across more than a few patches at a time.
8
8
 
9
9
  ## v0.15.x
10
10
 
11
+ - v0.15.42 (2026-06-28) — **The agent orchestrator and tenant registries serialize registration per name, so two concurrent register() calls for the same name can no longer both create a row (duplicate-create / lost registration), and a new b.safeAsync.keyedSerializer exposes that per-key serialization.** b.agent.orchestrator and b.agent.tenant registered a name with a check-then-create: read the backend row for the name, throw a duplicate error if it exists, otherwise write the new row. Because the read and the write are separated by an await, two concurrent register() calls for the same name both observed absence and both wrote — a duplicate-create where the second silently clobbered the first and both callers saw success, violating the one-row-per-name invariant. register() (and unregister()) now run through a per-name in-process serializer, so concurrent calls for the same name apply one at a time and the second is correctly refused as a duplicate; distinct names still run concurrently. The serializer is exposed as b.safeAsync.keyedSerializer() for any read-modify-write or check-then-create that must not interleave per key. It is in-process only: a registry backend shared across processes still needs its own atomic create or unique constraint to refuse a cross-process duplicate. **Added:** *b.safeAsync.keyedSerializer — serialize async work per key* — b.safeAsync.keyedSerializer() returns a { run(key, fn) } that queues fn behind any in-flight or queued work for the same key and runs it once they settle, so a read-modify-write or a check-then-create on a shared store cannot interleave with another call for the same key in the same process. Distinct keys run concurrently, and the per-key chain is dropped once it drains. It is the serialization the agent registries now use, and the same primitive backs the lockout and bot-challenge per-key failure counters. **Fixed:** *Agent orchestrator + tenant registries serialize registration per name (no duplicate-create race)* — b.agent.orchestrator.register and b.agent.tenant.register did a check-then-create (await backend.get -> throw-if-exists -> await backend.set) with an await between the read and the write, so two concurrent registrations of the same name both passed the duplicate check and both wrote — the second silently clobbering the first while both callers saw success. Registration now serializes per name in-process, so concurrent calls for one name apply sequentially and the second is refused with the duplicate error; distinct names are unaffected. A backend shared across processes still needs its own uniqueness constraint to refuse a cross-process duplicate. **Detectors:** *Build guard: a registry check-then-create must serialize per key* — A codebase guard now fails the build if a primitive does an async check-then-create on a pluggable backend (await backend.get -> throw a /duplicate error -> await backend.set) without serializing per key, so the duplicate-create race fixed here cannot reappear at a new registry.
12
+
13
+ - v0.15.41 (2026-06-28) — **Counters kept on a shared cache — byte quotas and the static server's bandwidth/concurrency caps — now accumulate with an atomic compare-and-set, so concurrent requests can no longer lose each other's charges and slip under the limit.** Several caps maintained a counter on a cache with a non-atomic read-modify-write: read the current value, add to it in memory, write it back. On a cache shared across nodes, two concurrent requests both read the same value, each added only its own contribution, and one write clobbered the other — a lost update that under-counted the cap and let a peer slip under it. b.network.byteQuota (and the b.middleware.dailyByteQuota that composes it) and the static server's per-actor/global bandwidth and per-actor concurrency caps all did this. They now accumulate through the cache's atomic update() (compare-and-set, with retry on the cluster backend under contention), so every concurrent charge is counted. A cache backing these caps must support atomic update(); b.cache provides it, and both primitives refuse a get/set-only cache at construction. The single-node in-memory paths were already safe (they accumulate synchronously). **Fixed:** *Byte quota on a shared cache counts concurrent requests atomically (no lost updates)* — b.network.byteQuota's cache backend accumulated bytes with a get → add → set that is not atomic, so concurrent requests from one peer on a multi-node deployment lost each other's byte charges and the rolling daily byte budget was under-enforced. Accounting now uses the cache's atomic compare-and-set update(), counting every concurrent request, and retries the cluster CAS under a contention burst rather than dropping a charge. A cache wired to a byte quota must support update() — b.cache does; a plain get/set cache is refused at create() (byte-quota/cache-no-atomic-update), and a backend that can't actually commit an atomic update surfaces loud on first use instead of silently disabling the quota. b.middleware.dailyByteQuota, which composes byteQuota, inherits the fix. · *Static server bandwidth + concurrency caps count concurrent requests atomically* — b.staticServe's per-actor and global bandwidth caps and its per-actor concurrency cap kept their counters on the cache with a non-atomic get → add → set, so concurrent downloads from one actor on a multi-replica deployment lost each other's charges and the caps were under-enforced (a bandwidth/concurrency limit bypass). The counters now accumulate through the cache's atomic update() with the same contention retry, and the quota cache is required to support update() at create(). **Detectors:** *Build guard: a cache-backed counter must use atomic update(), not get → set* — A codebase guard now fails the build if a primitive accumulates a counter on a cache with a get → mutate → set read-modify-write instead of the atomic update(), so the lost-update class fixed here cannot reappear at a new cap. Caches used for lookups or cache-aside (the value is replaced wholesale, not incremented), or whose writes are serialized in-process, are allowlisted with the reason they cannot lose an increment.
14
+
11
15
  - v0.15.40 (2026-06-27) — **The durable webhook dispatcher's retry poller now claims due deliveries with FOR UPDATE SKIP LOCKED on Postgres and MySQL, so two pollers running at once (multiple app nodes) can no longer both grab the same delivery and send the webhook twice in one cycle.** The webhook dispatcher's retry poller claimed due deliveries by flipping them pending->in-flight and then re-selecting the in-flight rows by id. On Postgres or MySQL under the default READ COMMITTED isolation, two pollers running concurrently could both re-select the same row: the loser's UPDATE matched zero rows (the winner had already flipped it), but its reselect-by-id still re-read the row the winner had just claimed, so both pollers attempted the same HTTP delivery in one cycle. The claim now row-locks the due rows with SELECT ... FOR UPDATE SKIP LOCKED on the row-locking backends, so concurrent pollers see disjoint sets and each delivery is claimed by exactly one poller; sqlite (a single writer) keeps the existing mark-then-reselect, which it serializes safely. This matches the claim used by the framework's transactional outbox and cluster queue. Receivers that already dedup on the X-Webhook-Delivery-Id header were protected from a duplicate POST; this closes the at-most-once-per-cycle gap at the dispatcher itself. **Fixed:** *Webhook retry pollers no longer double-deliver under concurrency on Postgres / MySQL* — b.webhook.dispatcher's processRetries() claimed due deliveries with a mark-then-reselect that had no row lock, so on Postgres / MySQL at READ COMMITTED two concurrent pollers (for example, the dispatcher running on more than one app node) could both hand back the same delivery and POST it twice in a single retry cycle. The claim now uses SELECT ... FOR UPDATE SKIP LOCKED on those backends so each due row is locked by exactly one poller and concurrent pollers claim disjoint sets; the rows a poller selects are exactly the rows it owns. sqlite keeps the mark-then-reselect path, which its single writer serializes. Operators running the dispatcher on a single node, or whose receivers dedup on X-Webhook-Delivery-Id, were not exposed to a duplicate delivery; no configuration change is required to pick up the fix. **Detectors:** *Build guard: a competing-consumer claim must use FOR UPDATE SKIP LOCKED* — A codebase guard now fails the build if a poller that claims due rows across workers — SELECT status='pending' inside a transaction, then flip the rows to in-flight — omits FOR UPDATE SKIP LOCKED on the row-locking backends. Without the row lock, two pollers under READ COMMITTED both claim the same row; the guard keeps any future poller from re-introducing the shape, with the transactional outbox and cluster queue as the reference claims.
12
16
 
13
17
  - v0.15.39 (2026-06-27) — **Nine more places that matched an operator-supplied regex against request data — User-Agent, Origin, request path, form fields, SMTP HELO, release-asset names — now screen the pattern for catastrophic backtracking (ReDoS) before use, and a new b.guardRegex.assertSafe helper makes that screening one call.** The previous release screened feature-flag and MCP regex patterns for ReDoS but did not sweep every place the framework matches an operator-supplied regex against attacker-controlled input. Nine more were found and fixed: the bot guard (User-Agent), CORS (Origin), the HTTP span middleware and the shared request skip-matcher used by CSRF / fetch-metadata / rate-limit / access-lock / age-gate and the request logger (request path), static serving (hashed-asset path pattern), form field validation (submitted field value), SMTP HELO generic-rDNS patterns (HELO name), and the self-updater's asset/signature patterns (names from a remote release feed). Each accepted an operator RegExp with only a type check and ran it on every matching request, so an accidentally catastrophic pattern such as (a+)+$ or ((a)+)+$ could pin a CPU on a crafted input — a length cap does not bound backtracking. Every one now screens the pattern through b.guardRegex at configuration time. A new b.guardRegex.assertSafe(input, label?, ErrorClass?, code?) primitive performs that screen in one call (accepting a RegExp or a pattern string), which operators can also use on their own patterns. **Added:** *b.guardRegex.assertSafe — screen a RegExp or pattern string for ReDoS in one call* — b.guardRegex.assertSafe(input, label?, ErrorClass?, code?, opts?) screens an already-compiled RegExp (its source) or a raw pattern string for the catastrophic-backtracking classes — nested, alternation-with, and lookaround quantifiers — throwing the supplied framework-error class (or the underlying GuardRegexError) on a hostile pattern and returning the input on success. It allows large or open-ended bounded repeats (`{8,}`, `{n,m}`): a single counted repeat matches in linear time and legitimate patterns (including the framework's own defaults) use them. It is the config-time guard used by the request-lifecycle fixes above, and operators can apply it to their own patterns before matching them against untrusted input. **Security:** *Operator regex patterns matched against request data are screened for ReDoS framework-wide* — An operator-supplied RegExp matched against attacker-controllable input is a denial-of-service surface if it has a catastrophic-backtracking shape: the input triggers exponential work in the regex engine. Nine sites accepted such patterns with only an `instanceof RegExp` type check and executed them per request — bot-guard against the User-Agent, CORS against the Origin header, the HTTP span middleware and the shared skip-path matcher (CSRF / fetch-metadata / rate-limit / access-lock / age-gate / request-log) against the request path, static serving against the request path, form validation against the submitted field value, SMTP HELO checks against the HELO name, and the self-updater against asset names from a remote release feed. Each now routes the pattern through b.guardRegex at configuration time, so a catastrophic shape is refused up front instead of being weaponized by a crafted request. A length bound on the input is not a defense: a nested-quantifier pattern backtracks catastrophically at a few dozen characters. **Detectors:** *Build guard: an operator regex matched against request input must be ReDoS-screened* — A codebase guard now fails the build if a primitive accepts an operator-supplied RegExp and executes it against request input without screening the pattern through b.guardRegex.assertSafe — so the catastrophic-backtracking class fixed in this release cannot be reintroduced at a new site (the trusted-input cases — local filesystem paths, operator config keys, operator-owned schemas — are explicitly allowlisted). · *Build guard: process.moduleLoadList filters must match the 'NativeModule X' naming* — A guard now fails the build if a test filters process.moduleLoadList by the 'node:X' name only. Node 20+ records a loaded builtin as 'NativeModule X', so a 'node:'-only filter in an edge-runtime no-eager-load test would rot green and miss a reintroduced top-level networking require.
@@ -60,6 +60,7 @@ var bCrypto = require("./crypto");
60
60
  var agentAudit = require("./agent-audit");
61
61
  var vaultAad = require("./vault-aad");
62
62
  var validateOpts = require("./validate-opts");
63
+ var safeAsync = require("./safe-async");
63
64
 
64
65
  var audit = lazyRequire(function () { return require("./audit"); });
65
66
  var cluster = lazyRequire(function () { return require("./cluster"); });
@@ -165,6 +166,11 @@ function create(opts) {
165
166
 
166
167
  var ctx = {
167
168
  backend: backend,
169
+ // Serializes register/unregister per name so a concurrent pair for the same
170
+ // name can't interleave the check-then-create (await get -> throw-if-exists
171
+ // -> await set) and both write — a duplicate-create / lost-registration. In
172
+ // process only; a shared persistent backend also needs its own uniqueness.
173
+ registrySerializer: safeAsync.keyedSerializer(),
168
174
  cluster: clusterImpl,
169
175
  audit: auditImpl,
170
176
  permissions: permissions,
@@ -235,9 +241,9 @@ function create(opts) {
235
241
  }
236
242
 
237
243
  return {
238
- register: function (name, agent, regOpts) { return _register(ctx, name, agent, regOpts || {}); },
244
+ register: function (name, agent, regOpts) { return ctx.registrySerializer.run(name, function () { return _register(ctx, name, agent, regOpts || {}); }); },
239
245
  hydrate: function (name, agent) { return _hydrate(ctx, name, agent); },
240
- unregister: function (name, args) { return _unregister(ctx, name, args || {}); },
246
+ unregister: function (name, args) { return ctx.registrySerializer.run(name, function () { return _unregister(ctx, name, args || {}); }); },
241
247
  lookup: function (name, args) { return _lookup(ctx, name, args || {}); },
242
248
  list: function (args) { return _list(ctx, args || {}); },
243
249
  spawnConsumers: function (args) { return _spawnConsumers(ctx, args || {}); },
@@ -60,6 +60,7 @@ var agentAudit = require("./agent-audit");
60
60
  var safeJson = require("./safe-json");
61
61
  var vaultAad = require("./vault-aad");
62
62
  var validateOpts = require("./validate-opts");
63
+ var safeAsync = require("./safe-async");
63
64
 
64
65
  var audit = lazyRequire(function () { return require("./audit"); });
65
66
  var cryptoField = lazyRequire(function () { return require("./crypto-field"); });
@@ -159,13 +160,19 @@ function create(opts) {
159
160
  var permissions = opts.permissions || null;
160
161
  var ctx = {
161
162
  backend: backend, audit: auditImpl, permissions: permissions,
163
+ // Serializes register/unregister per tenantId so a concurrent pair for the
164
+ // same tenant can't interleave the check-then-create (await get ->
165
+ // throw-if-exists -> await set) and both write — a duplicate-create /
166
+ // lost-registration. In process only; a shared persistent backend also
167
+ // needs its own uniqueness constraint.
168
+ registrySerializer: safeAsync.keyedSerializer(),
162
169
  // Archived tenants — keys retained but no live config; restore
163
170
  // requires explicit operator opt-in.
164
171
  archive: new Map(),
165
172
  };
166
173
  return {
167
- register: function (tenantId, regOpts) { return _register(ctx, tenantId, regOpts || {}); },
168
- unregister: function (tenantId, args) { return _unregister(ctx, tenantId, args || {}); },
174
+ register: function (tenantId, regOpts) { return ctx.registrySerializer.run(tenantId, function () { return _register(ctx, tenantId, regOpts || {}); }); },
175
+ unregister: function (tenantId, args) { return ctx.registrySerializer.run(tenantId, function () { return _unregister(ctx, tenantId, args || {}); }); },
169
176
  lookup: function (tenantId, args) { return _lookup(ctx, tenantId, args || {}); },
170
177
  list: function (args) { return _list(ctx, args || {}); },
171
178
  check: function (actor, agentTenantId) { return _check(ctx, actor, agentTenantId); },
@@ -84,6 +84,7 @@ var numericBounds = require("../numeric-bounds");
84
84
  var lazyRequire = require("../lazy-require");
85
85
  var requestHelpers = require("../request-helpers");
86
86
  var validateOpts = require("../validate-opts");
87
+ var safeAsync = require("../safe-async");
87
88
  var { LockoutError } = require("../framework-error");
88
89
 
89
90
  var observability = lazyRequire(function () { return require("../observability"); });
@@ -264,17 +265,11 @@ function create(opts) {
264
265
 
265
266
  // Per-key serialization of the failure counter (read→increment→write on an
266
267
  // async store): concurrent recordFailure calls for the same key would lose
267
- // updates, letting parallel failures stay under the lockout threshold. A
268
- // per-key promise chain applies them sequentially in-process.
269
- var _recordChains = new Map();
268
+ // updates, letting parallel failures stay under the lockout threshold. The
269
+ // keyed serializer applies them sequentially in-process.
270
+ var _recordSerializer = safeAsync.keyedSerializer();
270
271
  function recordFailure(key, callOpts) {
271
- var prev = _recordChains.get(key) || Promise.resolve();
272
- var run = prev.then(function () { return _doRecordFailure(key, callOpts); },
273
- function () { return _doRecordFailure(key, callOpts); });
274
- var tail = run.then(function () {}, function () {});
275
- _recordChains.set(key, tail);
276
- tail.then(function () { if (_recordChains.get(key) === tail) _recordChains.delete(key); });
277
- return run;
272
+ return _recordSerializer.run(key, function () { return _doRecordFailure(key, callOpts); });
278
273
  }
279
274
 
280
275
  async function _doRecordFailure(key, callOpts) {
@@ -41,6 +41,7 @@ var lazyRequire = require("./lazy-require");
41
41
  var requestHelpers = require("./request-helpers");
42
42
  var validateOpts = require("./validate-opts");
43
43
  var numericBounds = require("./numeric-bounds");
44
+ var safeAsync = require("./safe-async");
44
45
  var { AuthBotChallengeError } = require("./framework-error");
45
46
 
46
47
  var observability = lazyRequire(function () { return require("./observability"); });
@@ -280,15 +281,9 @@ function create(opts) {
280
281
  // per-key promise chain applies advances for a key sequentially. (Cross-node
281
282
  // atomicity additionally needs an atomic store; this fixes the in-process
282
283
  // race the gate actually depends on.)
283
- var _advanceChains = new Map();
284
+ var _advanceSerializer = safeAsync.keyedSerializer();
284
285
  function _advanceFailure(key, req) {
285
- var prev = _advanceChains.get(key) || Promise.resolve();
286
- var run = prev.then(function () { return _doAdvanceFailure(key, req); },
287
- function () { return _doAdvanceFailure(key, req); });
288
- var tail = run.then(function () {}, function () {});
289
- _advanceChains.set(key, tail);
290
- tail.then(function () { if (_advanceChains.get(key) === tail) _advanceChains.delete(key); });
291
- return run;
286
+ return _advanceSerializer.run(key, function () { return _doAdvanceFailure(key, req); });
292
287
  }
293
288
 
294
289
  async function _doAdvanceFailure(key, req) {
@@ -51,6 +51,10 @@ var ByteQuotaError = defineClass("ByteQuotaError", { alwaysPermanent: true });
51
51
 
52
52
  var BINS_PER_DAY = 24; // 24 hours in a day
53
53
  var BIN_MS = C.TIME.hours(1);
54
+ // Outer retries when the cache's internal CAS exhausts under a concurrency
55
+ // burst on one key — absorbs the burst so a charge is not dropped (each retry
56
+ // re-reads the latest counter). Total CAS attempts ≈ this × the cache's own 5.
57
+ var ACCOUNT_MAX_RETRIES = 6;
54
58
 
55
59
  function _hourBin(nowMs) { return Math.floor(nowMs / BIN_MS); }
56
60
  function _newEntry() { return { bins: new Array(BINS_PER_DAY).fill(0), startHour: 0 }; }
@@ -108,22 +112,44 @@ function _memoryBackend() {
108
112
 
109
113
  function _cacheBackend(cache) {
110
114
  function _key(k) { return "byteQuota:" + k; }
111
- async function _read(key) {
112
- var raw = await cache.get(_key(key));
115
+ function _coerce(raw) {
113
116
  return raw && typeof raw === "object" && Array.isArray(raw.bins) ? raw : _newEntry();
114
117
  }
115
118
  return {
116
119
  async total(key, nowMs) {
117
- var entry = await _read(key);
118
- var slid = _slideAndSum(entry, _hourBin(nowMs));
119
- if (slid.moved) await cache.set(_key(key), slid.entry, { ttlMs: BIN_MS * BINS_PER_DAY });
120
+ // Read-only: slide + sum without writing back. A writeback here would
121
+ // race a concurrent account() — it could read the pre-increment entry
122
+ // and clobber the increment (lost update). The slide is recomputed on
123
+ // every read/account, so persisting it is a pure optimization not worth
124
+ // the clobber risk.
125
+ var slid = _slideAndSum(_coerce(await cache.get(_key(key))), _hourBin(nowMs));
120
126
  return slid.total;
121
127
  },
122
128
  async account(key, bytes, nowMs) {
123
- var entry = await _read(key);
124
- var slid = _slideAndSum(entry, _hourBin(nowMs));
125
- slid.entry.bins[BINS_PER_DAY - 1] += bytes;
126
- await cache.set(_key(key), slid.entry, { ttlMs: BIN_MS * BINS_PER_DAY });
129
+ // Atomic read-modify-write. A plain get → mutate → set loses concurrent
130
+ // charges on the shared cache (two callers read the same counter and one
131
+ // set clobbers the other). cache.update runs the slide + add under a
132
+ // compare-and-set (with retry on the cluster backend), so every
133
+ // concurrent record is counted toward the quota.
134
+ var nowHour = _hourBin(nowMs);
135
+ // The cluster CAS retries internally and throws UPDATE_CONTENTION once it
136
+ // gives up. A concurrency burst on one key must not drop a charge (that
137
+ // lets a peer undercount the quota), so retry the whole RMW; each attempt
138
+ // re-reads the latest value (a full transaction whose latency naturally
139
+ // spreads the contenders) and eventually wins the CAS.
140
+ for (var attempt = 0; ; attempt++) {
141
+ try {
142
+ await cache.update(_key(key), function (current) {
143
+ var slid = _slideAndSum(_coerce(current), nowHour);
144
+ slid.entry.bins[BINS_PER_DAY - 1] += bytes;
145
+ return { value: slid.entry };
146
+ }, { ttlMs: BIN_MS * BINS_PER_DAY });
147
+ return;
148
+ } catch (e) {
149
+ if (e && e.code === "UPDATE_CONTENTION" && attempt < ACCOUNT_MAX_RETRIES) continue;
150
+ throw e;
151
+ }
152
+ }
127
153
  },
128
154
  async reset(key) {
129
155
  if (typeof cache.delete === "function") await cache.delete(_key(key));
@@ -172,9 +198,27 @@ function create(opts) {
172
198
  _requirePositiveBytes("bytesPerDay", opts.bytesPerDay);
173
199
  var bytesPerDay = opts.bytesPerDay;
174
200
  var now = typeof opts.now === "function" ? opts.now : function () { return Date.now(); };
175
- var backend = opts.cache && typeof opts.cache.get === "function"
176
- ? _cacheBackend(opts.cache)
177
- : _memoryBackend();
201
+ var backend;
202
+ if (opts.cache && typeof opts.cache.get === "function") {
203
+ // A byte quota is a shared counter — the cache backend MUST support atomic
204
+ // read-modify-write. A plain get/set cache loses concurrent byte charges
205
+ // (lost update) on the multi-node path, silently under-counting the quota.
206
+ // This is an early-catch for a cache that lacks an update method entirely;
207
+ // a b.cache whose backend doesn't implement update (e.g. a get/set-only
208
+ // custom backend) exposes the method but throws UNSUPPORTED at call time —
209
+ // that case is surfaced loud on the first record() (see below).
210
+ if (typeof opts.cache.update !== "function") {
211
+ throw new ByteQuotaError(
212
+ "byte-quota/cache-no-atomic-update",
213
+ "network.byteQuota: a cache backing a byte quota must support atomic update() — " +
214
+ "a plain get/set cache loses concurrent byte charges on the shared path; " +
215
+ "use b.cache.create(...), which provides it"
216
+ );
217
+ }
218
+ backend = _cacheBackend(opts.cache);
219
+ } else {
220
+ backend = _memoryBackend();
221
+ }
178
222
 
179
223
  var _emitAudit = audit().namespaced("network.byte_quota", opts.audit);
180
224
 
@@ -235,9 +279,25 @@ function create(opts) {
235
279
  var nowMs = now();
236
280
  try { await backend.account(key, bytes, nowMs); }
237
281
  catch (e) {
282
+ // A cache backend that doesn't actually implement atomic update() throws
283
+ // UNSUPPORTED at call time (the public update method exists on every
284
+ // b.cache, so the get/set-only case can't be caught at construction).
285
+ // Under the drop-silent policy below this would silently disable the
286
+ // quota — surface it LOUD so the operator fixes the backend rather than
287
+ // running an unenforced quota.
288
+ if (e && e.code === "UNSUPPORTED") {
289
+ throw new ByteQuotaError(
290
+ "byte-quota/cache-no-atomic-update",
291
+ "network.byteQuota: the configured cache backend does not support atomic update() — " +
292
+ "a byte quota cannot enforce on a get/set-only backend; use a cache whose backend " +
293
+ "implements update (the memory or cluster backend)"
294
+ );
295
+ }
238
296
  _emitAudit("backend_error", "failure", { phase: "record", key: key, bytes: bytes, error: (e && e.message) || String(e) });
239
- // Drop-silent after audit the operation already succeeded; the
240
- // alternative throw would punish the handler that already accepted bytes.
297
+ // Drop-silent after audit for a transient/unreachable backend the
298
+ // operation already succeeded; a throw would punish the handler that
299
+ // already accepted bytes. A contention burst is retried in account()
300
+ // before it can reach here.
241
301
  return;
242
302
  }
243
303
  _emitMetric("recorded", bytes, {});
package/lib/safe-async.js CHANGED
@@ -1611,6 +1611,47 @@ function flushLoop(fn, intervalMs, opts) {
1611
1611
  // We re-export them here under safe-async-shaped names so call sites
1612
1612
  // reaching for async safety primitives find them in one place.
1613
1613
 
1614
+ /**
1615
+ * @primitive b.safeAsync.keyedSerializer
1616
+ * @signature b.safeAsync.keyedSerializer()
1617
+ * @since 0.15.42
1618
+ * @status stable
1619
+ * @related b.safeAsync.parallel
1620
+ *
1621
+ * Serializes async work per key: `run(key, fn)` queues `fn` behind any in-flight
1622
+ * or queued work for the same `key` and runs it once they settle, so a
1623
+ * read-modify-write or a check-then-create on a shared store cannot interleave
1624
+ * with another call for the same key in the same process. Different keys run
1625
+ * concurrently. The per-key chain is dropped once it drains, so the map does not
1626
+ * grow without bound.
1627
+ *
1628
+ * In-process only: it serializes calls within ONE process. A registry shared
1629
+ * across processes still needs its backend's own atomic create / unique
1630
+ * constraint to refuse a cross-process duplicate.
1631
+ *
1632
+ * @example
1633
+ * var reg = b.safeAsync.keyedSerializer();
1634
+ * // concurrent register("acme") calls apply one-at-a-time, so the second
1635
+ * // sees the first's row and is refused as a duplicate:
1636
+ * await reg.run("acme", function () { return register("acme", row); });
1637
+ */
1638
+ function keyedSerializer() {
1639
+ var chains = new Map();
1640
+ function run(key, fn) {
1641
+ var prev = chains.get(key) || Promise.resolve();
1642
+ // Run fn after prev settles (resolved OR rejected) so one failure does not
1643
+ // wedge the key's queue. Wrap so prev's settled value/reason is NOT leaked
1644
+ // into fn as an argument — fn runs as a thunk with its own closure, never
1645
+ // the previous task's result.
1646
+ var result = prev.then(function () { return fn(); }, function () { return fn(); });
1647
+ var tail = result.then(function () {}, function () {});
1648
+ chains.set(key, tail);
1649
+ tail.then(function () { if (chains.get(key) === tail) chains.delete(key); });
1650
+ return result;
1651
+ }
1652
+ return { run: run };
1653
+ }
1654
+
1614
1655
  var retryHelper = require("./retry");
1615
1656
 
1616
1657
  var asyncRetry = retryHelper.withRetry;
@@ -1632,6 +1673,7 @@ module.exports = {
1632
1673
  makeBatchDrain: makeBatchDrain,
1633
1674
  makeBatchingSink: makeBatchingSink,
1634
1675
  parallel: parallel,
1676
+ keyedSerializer: keyedSerializer,
1635
1677
  Mutex: Mutex,
1636
1678
  Semaphore: Semaphore,
1637
1679
  Once: Once,
package/lib/static.js CHANGED
@@ -655,13 +655,23 @@ function _validateCreateOpts(opts) {
655
655
  "'user-content'; got " + JSON.stringify(opts.mountType));
656
656
  }
657
657
  // Quotas require a cache for cluster-shared coordination.
658
- if ((opts.maxBytesPerActorPerWindowMs > 0 ||
659
- opts.maxBytesAllActorsPerWindowMs > 0 ||
660
- opts.maxConcurrentDownloadsPerActor > 0) &&
661
- !opts.cache) {
662
- throw _err("BAD_OPT",
663
- "staticServe.create: bandwidth / concurrency quotas require opts.cache " +
664
- "(pass cache: b.cache.create({ backend: 'cluster' }) so multi-replica deploys honor caps globally)");
658
+ if (opts.maxBytesPerActorPerWindowMs > 0 ||
659
+ opts.maxBytesAllActorsPerWindowMs > 0 ||
660
+ opts.maxConcurrentDownloadsPerActor > 0) {
661
+ if (!opts.cache) {
662
+ throw _err("BAD_OPT",
663
+ "staticServe.create: bandwidth / concurrency quotas require opts.cache " +
664
+ "(pass cache: b.cache.create({ backend: 'cluster' }) so multi-replica deploys honor caps globally)");
665
+ }
666
+ // The bandwidth / concurrency counters accumulate atomically via
667
+ // cache.update; a cache that lacks an update method entirely cannot
668
+ // coordinate the caps. (A b.cache whose backend can't do atomic RMW exposes
669
+ // update but throws UNSUPPORTED at call time — that surfaces on first use.)
670
+ if (typeof opts.cache.update !== "function") {
671
+ throw _err("BAD_OPT",
672
+ "staticServe.create: the quota cache must support atomic update() — a plain " +
673
+ "get/set cache loses concurrent bandwidth/concurrency charges; use b.cache.create(...)");
674
+ }
665
675
  }
666
676
  }
667
677
 
@@ -688,17 +698,36 @@ async function _checkBandwidthQuota(cache, actorKey, perActorCap, globalCap, win
688
698
  return { ok: true, windowStart: windowStart, now: now };
689
699
  }
690
700
 
701
+ // Atomic counter adjust on the cache. A plain cache.get → mutate → cache.set is
702
+ // a non-atomic read-modify-write: two concurrent requests read the same value
703
+ // and one set clobbers the other (lost update), under-counting on the shared
704
+ // (cluster) cache so the bandwidth / concurrency caps can be bypassed. cache.update
705
+ // runs the adjust under a compare-and-set; the cluster CAS retries internally and
706
+ // throws UPDATE_CONTENTION once it gives up, so retry the whole RMW under a burst
707
+ // (each attempt re-reads the latest value) rather than dropping the adjustment.
708
+ var STATIC_COUNTER_MAX_RETRIES = 6;
709
+ async function _atomicCounter(cache, key, mutate, ttlMs) {
710
+ for (var attempt = 0; ; attempt++) {
711
+ try {
712
+ await cache.update(key, function (current) {
713
+ var c = (typeof current === "number" && isFinite(current)) ? current : 0;
714
+ return { value: mutate(c) };
715
+ }, { ttlMs: ttlMs });
716
+ return;
717
+ } catch (e) {
718
+ if (e && e.code === "UPDATE_CONTENTION" && attempt < STATIC_COUNTER_MAX_RETRIES) continue;
719
+ throw e;
720
+ }
721
+ }
722
+ }
723
+
691
724
  async function _consumeBandwidth(cache, actorKey, perActorCap, globalCap, windowMs, bytes) {
692
725
  if (!cache) return;
693
726
  if (perActorCap > 0 && actorKey) {
694
- var aKey = "static:bw:actor:" + actorKey;
695
- var aUsed = (await cache.get(aKey)) || 0;
696
- await cache.set(aKey, aUsed + bytes, { ttlMs: windowMs });
727
+ await _atomicCounter(cache, "static:bw:actor:" + actorKey, function (c) { return c + bytes; }, windowMs);
697
728
  }
698
729
  if (globalCap > 0) {
699
- var gKey = "static:bw:global";
700
- var gUsed = (await cache.get(gKey)) || 0;
701
- await cache.set(gKey, gUsed + bytes, { ttlMs: windowMs });
730
+ await _atomicCounter(cache, "static:bw:global", function (c) { return c + bytes; }, windowMs);
702
731
  }
703
732
  }
704
733
 
@@ -712,17 +741,12 @@ async function _checkConcurrencyCap(cache, actorKey, cap) {
712
741
 
713
742
  async function _incConcurrency(cache, actorKey) {
714
743
  if (!cache || !actorKey) return;
715
- var key = "static:conc:" + actorKey;
716
- var current = (await cache.get(key)) || 0;
717
- await cache.set(key, current + 1, { ttlMs: C.TIME.minutes(10) });
744
+ await _atomicCounter(cache, "static:conc:" + actorKey, function (c) { return c + 1; }, C.TIME.minutes(10));
718
745
  }
719
746
 
720
747
  async function _decConcurrency(cache, actorKey) {
721
748
  if (!cache || !actorKey) return;
722
- var key = "static:conc:" + actorKey;
723
- var current = (await cache.get(key)) || 0;
724
- var next = current > 0 ? current - 1 : 0;
725
- await cache.set(key, next, { ttlMs: C.TIME.minutes(10) });
749
+ await _atomicCounter(cache, "static:conc:" + actorKey, function (c) { return c > 0 ? c - 1 : 0; }, C.TIME.minutes(10));
726
750
  }
727
751
 
728
752
  function _actorKeyFromContext(ctx) {
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@blamejs/core",
3
- "version": "0.15.40",
3
+ "version": "0.15.42",
4
4
  "description": "The Node framework that owns its stack.",
5
5
  "license": "Apache-2.0",
6
6
  "author": "blamejs contributors",
package/sbom.cdx.json CHANGED
@@ -2,10 +2,10 @@
2
2
  "$schema": "http://cyclonedx.org/schema/bom-1.5.schema.json",
3
3
  "bomFormat": "CycloneDX",
4
4
  "specVersion": "1.5",
5
- "serialNumber": "urn:uuid:4538b9a5-f7e6-4723-be61-b03f474f0c0e",
5
+ "serialNumber": "urn:uuid:25fff8f4-b758-413d-b63d-fc1b5b5a19ad",
6
6
  "version": 1,
7
7
  "metadata": {
8
- "timestamp": "2026-06-28T06:47:43.444Z",
8
+ "timestamp": "2026-06-28T09:41:21.905Z",
9
9
  "lifecycles": [
10
10
  {
11
11
  "phase": "build"
@@ -19,14 +19,14 @@
19
19
  }
20
20
  ],
21
21
  "component": {
22
- "bom-ref": "@blamejs/core@0.15.40",
22
+ "bom-ref": "@blamejs/core@0.15.42",
23
23
  "type": "application",
24
24
  "name": "blamejs",
25
- "version": "0.15.40",
25
+ "version": "0.15.42",
26
26
  "scope": "required",
27
27
  "author": "blamejs contributors",
28
28
  "description": "The Node framework that owns its stack.",
29
- "purl": "pkg:npm/%40blamejs/core@0.15.40",
29
+ "purl": "pkg:npm/%40blamejs/core@0.15.42",
30
30
  "properties": [],
31
31
  "externalReferences": [
32
32
  {
@@ -54,7 +54,7 @@
54
54
  "components": [],
55
55
  "dependencies": [
56
56
  {
57
- "ref": "@blamejs/core@0.15.40",
57
+ "ref": "@blamejs/core@0.15.42",
58
58
  "dependsOn": []
59
59
  }
60
60
  ]