@rulvar/testing 1.33.0 → 1.35.0

package/dist/index.d.ts

@@ -154,7 +154,10 @@ interface VcrRow {
   * seeds its counters past the numbers already on disk, so the
   * numbering continues across sequential sessions; a duplicate
   * number inside a fully numbered group refuses replay as ambiguous
-  * (v1.32.0 review P2).
+  * (v1.32.0 review P2). The numbering ends at
+  * `Number.MAX_SAFE_INTEGER`: a session refuses with a typed
+  * ConfigError to claim a number past it, before dispatching the
+  * provider and before touching the file (v1.33.0 review P3).
   */
   occurrence?: number;
   requestHash: string;
@@ -210,7 +213,14 @@ declare function requestHash(req: ChatRequest): string;
 * recorder session may be active on a cassette at a time: two
 * concurrently constructed recorders seed identically and claim
 * colliding numbers, which replay refuses as ambiguous instead of
-* silently serving either order. The wrapped adapters are drop-in:
+* silently serving either order. The numbering ends at
+* `Number.MAX_SAFE_INTEGER`: a group that already numbers it refuses
+* the appending session at construction, and a session whose counter
+* would pass it refuses that call before dispatching the provider,
+* both with a typed ConfigError and without touching the file,
+* because the next float increment would stall at 2 ** 53 and
+* silently duplicate one unsafe number on every following row
+* (v1.33.0 review P3). The wrapped adapters are drop-in:
 * same ids, providers, caps, and event streams.
 */
 declare function record(options: {

package/dist/index.js

@@ -389,7 +389,14 @@ function groupRows(rows, cassette) {
 * recorder session may be active on a cassette at a time: two
 * concurrently constructed recorders seed identically and claim
 * colliding numbers, which replay refuses as ambiguous instead of
-* silently serving either order. The wrapped adapters are drop-in:
+* silently serving either order. The numbering ends at
+* `Number.MAX_SAFE_INTEGER`: a group that already numbers it refuses
+* the appending session at construction, and a session whose counter
+* would pass it refuses that call before dispatching the provider,
+* both with a typed ConfigError and without touching the file,
+* because the next float increment would stall at 2 ** 53 and
+* silently duplicate one unsafe number on every following row
+* (v1.33.0 review P3). The wrapped adapters are drop-in:
 * same ids, providers, caps, and event streams.
 */
 function record(options) {
@@ -401,8 +408,10 @@ function record(options) {
 		for (const [adapterId, forAdapter] of groupRows(existing.rows, options.cassette)) {
 			const forSeeds = /* @__PURE__ */ new Map();
 			for (const [hash, rows] of forAdapter) {
-				const numbered = rows.map((row) => row.occurrence).filter((value) => value !== void 0);
-				forSeeds.set(hash, numbered.length === 0 ? 0 : Math.max(...numbered) + 1);
+				let highest = -1;
+				for (const row of rows) if (row.occurrence !== void 0 && row.occurrence > highest) highest = row.occurrence;
+				if (highest >= Number.MAX_SAFE_INTEGER) throw new ConfigError(`${options.cassette} already numbers occurrence ${String(Number.MAX_SAFE_INTEGER)} for adapter '${adapterId}' hash ${hash.slice(0, 12)}; the numbering has reached the safe integer ceiling, so no further exchange for this request can be appended; record a fresh cassette instead`);
+				forSeeds.set(hash, highest + 1);
 			}
 			seeds.set(adapterId, forSeeds);
 		}
@@ -417,6 +426,7 @@ function record(options) {
 			stream(req, signal) {
 				const hash = requestHash(req);
 				const occurrence = occurrences.get(hash) ?? 0;
+				if (!Number.isSafeInteger(occurrence)) throw new ConfigError(`${options.cassette} has no safe occurrence number left for adapter '${adapter.id}' hash ${hash.slice(0, 12)}; an earlier call in this session claimed ${String(Number.MAX_SAFE_INTEGER)}, so this exchange cannot be numbered; record a fresh cassette instead`);
 				occurrences.set(hash, occurrence + 1);
 				return (async function* () {
 					const events = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rulvar/testing",
-  "version": "1.33.0",
+  "version": "1.35.0",
   "description": "Rulvar test harness: createTestEngine, FakeAdapter, VCR cassettes, replay-strict runs, matchers.",
   "type": "module",
   "license": "Apache-2.0",
@@ -26,7 +26,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@rulvar/core": "1.33.0"
+    "@rulvar/core": "1.35.0"
   },
   "devDependencies": {
     "@types/node": "^22.20.0",