@solarisdk/browser 0.1.6 → 0.1.7

package/dist/index.js

@@ -109,15 +109,19 @@ async function responseSaysRetryable(res) {
         return false;
     }
 }
-// A few 503s are answers, not outages: retrying cannot change them, and swallowing the body would hide the
-// code from the caller. `ProxyTierUnavailable` (explicitly requested proxy tier not servable) is one.
+// Some 5xx are answers, not outages: retrying cannot change them, and swallowing the body would hide the
+// code from the caller. The gateway says so itself with `retryable: false` (502 `PoolResponseUnparseable`:
+// the session may already exist, so a retry creates a second one; 503 `EgressBlockPeekUnsupportedOnFlavor`).
+// `ProxyTierUnavailable` (explicitly requested proxy tier not servable) is also terminal, whatever its flag.
 const TERMINAL_503_CODES = new Set(["ProxyTierUnavailable"]);
-async function responseIsTerminal503(res) {
-    if (res.status !== 503)
-        return false;
+async function responseIsTerminal(res) {
     try {
         const body = (await res.clone().json());
-        return typeof body?.code === "string" && TERMINAL_503_CODES.has(body.code);
+        if (body?.retryable === false)
+            return true;
+        return (res.status === 503 &&
+            typeof body?.code === "string" &&
+            TERMINAL_503_CODES.has(body.code));
     }
     catch {
         return false;
@@ -575,10 +579,13 @@ export class Solari {
                 clearTimeout(timer);
                 if (res.ok)
                     return res;
-                const retryThis = (isRetryableStatus(res.status) && !(await responseIsTerminal503(res))) ||
+                const retryThis = (isRetryableStatus(res.status) && !(await responseIsTerminal(res))) ||
                     (isSafeToReplay(method, opts?.idempotencyKey) &&
                         (await responseSaysRetryable(res)));
-                if (!retryThis)
+                // Out of attempts: hand the last response back so the caller maps its
+                // status and `code` like any other error, instead of a status-less
+                // "exhausted" error that hides why the gateway refused.
+                if (!retryThis || attempt === this.maxAttempts)
                     return res;
                 await res.text().catch(() => { });
                 lastErr = new SolariError(`Solari ${method} ${path}: ${res.status}`, res.status);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@solarisdk/browser",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "Official client SDK for Solari Browser.",
   "license": "Apache-2.0",
   "type": "module",