@gusnips/react 0.9.16 → 0.9.18

package/dist/query.d.ts

@@ -66,8 +66,8 @@ export interface QueryDefaultsOptions {
  */
 export declare function shouldRetry(error: unknown, durableLimitCodes?: readonly string[], maxRetryWaitSecs?: number): boolean;
 /**
- * How long before the next attempt: the server's stated wait when there is one, react-query's
- * exponential backoff otherwise, and never less than the backoff — a `Retry-After: 0` is a
+ * How long before the next attempt: the server's stated wait when there is one, an exponential
+ * backoff with jitter otherwise, and never less than the backoff — a `Retry-After: 0` is a
  * server saying "immediately", which for a client that just got refused is still too soon.
  */
 export declare function retryDelayMs(attemptIndex: number, error: unknown): number;

package/dist/query.js

@@ -15,8 +15,8 @@ export function shouldRetry(error, durableLimitCodes = [], maxRetryWaitSecs) {
     });
 }
 /**
- * How long before the next attempt: the server's stated wait when there is one, react-query's
- * exponential backoff otherwise, and never less than the backoff — a `Retry-After: 0` is a
+ * How long before the next attempt: the server's stated wait when there is one, an exponential
+ * backoff with jitter otherwise, and never less than the backoff — a `Retry-After: 0` is a
  * server saying "immediately", which for a client that just got refused is still too soon.
  */
 export function retryDelayMs(attemptIndex, error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gusnips/react",
-  "version": "0.9.16",
+  "version": "0.9.18",
   "description": "The headless runtime under a Vite + React SPA: typed fetch client with single-flight refresh, auth store, query defaults, route guards, error boundary, stale-tab recovery, i18n bootstrap.",
   "type": "module",
   "license": "MIT",
@@ -80,7 +80,7 @@
     "release:major": "bun pm version major && bun publish --access public"
   },
   "dependencies": {
-    "@gusnips/http": "0.1.3",
+    "@gusnips/http": "0.1.5",
     "clsx": "^2.1.1",
     "tailwind-merge": "^3.6.0"
   },

package/src/query.test.ts

@@ -1,5 +1,5 @@
 import { QueryClient, QueryObserver, type QueryObserverResult } from "@tanstack/react-query";
-import { describe, expect, it } from "vitest";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { ApiError } from "./api-error.ts";
 import { queryView, retryDelayMs, shouldRetry } from "./query.ts";
 
@@ -32,9 +32,10 @@ describe("shouldRetry", () => {
   it("reads only its own ApiError as an answer", () => {
     const vendor = Object.assign(new Error("Invalid login credentials"), { status: 400 });
     expect(shouldRetry(vendor)).toBe(true);
-    expect(retryDelayMs(0, Object.assign(new Error("x"), { status: 429, retryAfterSecs: 5 }))).toBe(
-      1000,
-    );
+    // The first backoff is at most 1.5 s, so 5 s would be the vendor's wait read as the server's.
+    expect(
+      retryDelayMs(0, Object.assign(new Error("x"), { status: 429, retryAfterSecs: 5 })),
+    ).toBeLessThan(5000);
   });
 
   it("retries 5xx", () => {
@@ -135,6 +136,14 @@ describe("shouldRetry", () => {
 });
 
 describe("retryDelayMs", () => {
+  // The middle of the jitter, so the numbers below are the backoff itself.
+  beforeEach(() => {
+    vi.spyOn(Math, "random").mockReturnValue(0.5);
+  });
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
   it("backs off exponentially when nothing said how long", () => {
     expect(retryDelayMs(0, refusal(500))).toBe(1000);
     expect(retryDelayMs(1, refusal(500))).toBe(2000);

package/src/query.ts

@@ -83,8 +83,8 @@ export function shouldRetry(
 }
 
 /**
- * How long before the next attempt: the server's stated wait when there is one, react-query's
- * exponential backoff otherwise, and never less than the backoff — a `Retry-After: 0` is a
+ * How long before the next attempt: the server's stated wait when there is one, an exponential
+ * backoff with jitter otherwise, and never less than the backoff — a `Retry-After: 0` is a
  * server saying "immediately", which for a client that just got refused is still too soon.
  */
 export function retryDelayMs(attemptIndex: number, error: unknown): number {