@africanies/angular-web-sdk 0.1.4 → 0.1.5

package/libs/africanies-core/src/lib/http/api-client.ts

@@ -11,11 +11,9 @@ import {
   map,
   Observable,
   of,
-  retry,
   tap,
   throwError,
   timeout,
-  timer,
 } from 'rxjs';
 
 import {
@@ -30,7 +28,6 @@ import { AFRICANIES_SDK_CONFIG } from '../config/africanies-sdk.config';
 import { ShippingModeService } from '../shipping/shipping-mode.service';
 import { formatApiErrorMessage } from './api-error-message';
 import { HttpResponseCache } from './http-cache';
-import { isRetryableGetError } from './is-retryable-get-error';
 import { normalize } from './normalize';
 import {
   resolveApiRequestContext,
@@ -102,7 +99,8 @@ type ResponseMode = NonNullable<ApiRequestOptions['responseMode']>;
  * - HTTP failures rethrown as `Error` whose `.message` is already
  *   user-facing ({@link formatApiErrorMessage}) — consumers do not need to
  *   parse `HttpErrorResponse` bodies
- * - Exponential-backoff retry (max 1) for **GET only**
+ * - **No automatic retry** — fail fast so UIs can show Select / table /
+ *   error-state Retry instead of a stuck loading spinner
  * - Optional per-GET TTL cache via `cacheTtlMs`
  *
  * @example
@@ -474,20 +472,8 @@ export class ApiClient {
         break;
     }
 
-    // Retry only idempotent GETs on transient failures — never replay 401/404 etc.
-    if (method === 'GET') {
-      req$ = req$.pipe(
-        retry({
-          count: 1,
-          delay: (err, retryCount) => {
-            if (!isRetryableGetError(err)) {
-              return throwError(() => err);
-            }
-            return timer(2 ** (retryCount - 1) * 1000);
-          },
-        }),
-      );
-    }
+    // No automatic GET retry — admin shells own Retry via UI
+    // (`africanies-select` showRetry, table error→refreshClick, error-state).
 
     if (this.config.timeout != null) {
       req$ = req$.pipe(

package/libs/africanies-core/src/lib/http/is-retryable-get-error.ts

@@ -1,11 +1,15 @@
 import { HttpErrorResponse } from '@angular/common/http';
 
 /**
- * Whether a failed GET should be retried by {@link ApiClient}.
+ * Whether a failed GET *would* be a candidate for automatic retry.
+ *
+ * {@link ApiClient} no longer auto-retries (fail fast for manual UI Retry).
+ * Kept for callers/tests that still want to classify transient vs permanent
+ * failures.
+ *
+ * Transient: network blips, timeouts, 408/429, 5xx.
+ * Auth and other 4xx fail fast — retrying 401 does not help.
  *
- * Retries transient failures only (network blips, timeouts, 408/429, 5xx).
- * Auth and other 4xx errors fail fast — retrying 401 does not help and causes
- * duplicate requests + toasts.
  * @param err
  */
 export function isRetryableGetError(err: unknown): boolean {

package/libs/africanies-core/src/lib/query/provide-africanies-query-defaults.ts

@@ -9,8 +9,8 @@
  * (no caret/tilde): TanStack marks the Angular adapter experimental and
  * breaking changes land without major bumps.
  *
- * Retry/backoff here mirrors {@link ApiClient}'s GET retry (max 3,
- * exponential) so HTTP-layer and query-layer behavior stay aligned.
+ * Query retries default to **0** to match {@link ApiClient} (fail fast;
+ * screens use manual Retry). Apps can override per `QueryClient` if needed.
  */
 
 /**
@@ -23,9 +23,9 @@ export interface AfricaniesQueryClientDefaults {
     staleTime: number;
     /** Unused-query garbage-collection time (ms). */
     gcTime: number;
-    /** Max failed-fetch retries for queries. */
+    /** Max failed-fetch retries for queries (default 0 — fail fast). */
     retry: number;
-    /** Exponential delay matching ApiClient GET backoff. */
+    /** Delay between retries when {@link AfricaniesQueryClientDefaults.queries.retry} > 0. */
     retryDelay: (attemptIndex: number) => number;
   };
   mutations: {
@@ -82,8 +82,7 @@ export function createAfricaniesQueryClientDefaults(): AfricaniesQueryClientDefa
     queries: {
       staleTime: 60_000,
       gcTime: 5 * 60_000,
-      retry: 3,
-      // attemptIndex is 0-based in TanStack; match 1s, 2s, 4s style backoff.
+      retry: 0,
       retryDelay: (attemptIndex: number) =>
         Math.min(1000 * 2 ** attemptIndex, 30_000),
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@africanies/angular-web-sdk",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "packageManager": "pnpm@11.24.0",
   "license": "MIT",
   "contributors": [