valtech-components 4.0.990 → 4.0.992

package/fesm2022/valtech-components.mjs

@@ -1,7 +1,7 @@
 import * as i0 from '@angular/core';
 import { InjectionToken, makeEnvironmentProviders, inject, APP_INITIALIZER, ErrorHandler, Injectable, DestroyRef, Inject, signal, computed, PLATFORM_ID, isDevMode, Optional, runInInjectionContext, effect, Pipe, Component, Input, ChangeDetectionStrategy, input, output, Injector, TemplateRef, ViewContainerRef, Directive, ElementRef, Renderer2, HostListener, EventEmitter, Output, HostBinding, SecurityContext, NgZone, ViewChild, ChangeDetectorRef, ContentChild, model, ViewEncapsulation, untracked, ViewChildren, viewChild, isSignal } from '@angular/core';
-import { BehaviorSubject, throwError, Subject, map, distinctUntilChanged, filter as filter$1, take as take$1, firstValueFrom, timeout as timeout$1, catchError as catchError$1, of, from, EMPTY, Observable, interval, defer, timer, merge, concat, debounceTime, switchMap as switchMap$1, takeUntil, isObservable, forkJoin, race, shareReplay as shareReplay$1, tap as tap$1 } from 'rxjs';
-import { catchError, switchMap, finalize, filter, take, timeout, map as map$1, tap, shareReplay, distinctUntilChanged as distinctUntilChanged$1, retry, debounceTime as debounceTime$1, takeUntil as takeUntil$1 } from 'rxjs/operators';
+import { BehaviorSubject, throwError, timer, Subject, map, distinctUntilChanged, filter as filter$1, take as take$1, firstValueFrom, timeout as timeout$1, catchError as catchError$1, of, from, EMPTY, Observable, interval, defer, merge, concat, debounceTime, switchMap as switchMap$1, takeUntil, isObservable, forkJoin, race, shareReplay as shareReplay$1, tap as tap$1 } from 'rxjs';
+import { catchError, switchMap, finalize, filter, take, timeout, retry, map as map$1, tap, shareReplay, distinctUntilChanged as distinctUntilChanged$1, debounceTime as debounceTime$1, takeUntil as takeUntil$1 } from 'rxjs/operators';
 import * as i1$3 from '@angular/common/http';
 import { provideHttpClient, withInterceptors, HttpContextToken, HttpClient, HttpErrorResponse, HttpContext, HttpParams } from '@angular/common/http';
 import * as i1$1 from '@angular/fire/firestore';
@@ -70,7 +70,7 @@ import fixWebmDuration from 'fix-webm-duration';
  * Current version of valtech-components.
  * This is automatically updated during the publish process.
  */
-const VERSION = '4.0.990';
+const VERSION = '4.0.992';
 
 function evaluateValtechAccess(rule, context, features = {}, visitedFeatures = new Set()) {
     if (rule == null)
@@ -1500,6 +1500,14 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "18.2.14", ngImpo
                     args: [PLATFORM_ID]
                 }] }] });
 
+/**
+ * Códigos de status que ameritan reintento — cold-start / infra tibia, no un
+ * error de negocio. Nunca 4xx: eso es la app pidiendo algo mal, reintentar no
+ * lo arregla.
+ */
+const RETRYABLE_STATUSES = new Set([502, 503, 504]);
+/** Backoff corto entre reintentos — no queremos que la UI perciba una espera larga. */
+const RETRY_DELAY_MS = 400;
 /**
  * Marca un request cuyo 404 es un ESTADO ESPERADO del dominio, no un error —
  * ej. `GET /v1/game-profile/me` cuando el usuario todavía no creó su perfil
@@ -1517,7 +1525,15 @@ const EXPECTED_NOT_FOUND = new HttpContextToken(() => false);
  * Interceptor HTTP de observabilidad (Capa 1 del estándar de manejo de errores).
  *
  * Para CADA respuesta HTTP con error:
- * 1. Normaliza el error con `interpretError`.
+ * 0. Si es un GET (idempotente) que falló con 502/503/504, reintenta hasta 2
+ *    veces con un backoff corto (400ms/800ms) ANTES de pasar al resto del
+ *    pipeline — el caso típico es un cold-start de Lambda en dev: 3-4
+ *    requests en paralelo al bootstrap (auth/profile + runs + org/{id})
+ *    pegan contra lambdas frías al mismo tiempo, todas 504, y ~2s después ya
+ *    están tibias y funcionan. Detectado en Merotz dev 2026-08-17. Nunca
+ *    reintenta POST/PUT/DELETE/PATCH (no son idempotentes) ni 4xx (error de
+ *    negocio, reintentar no lo arregla).
+ * 1. Si sigue fallando tras los reintentos: normaliza el error con `interpretError`.
  * 2. Loguea un evento estructurado a consola con el prefijo `[HTTP]`.
  * 3. Reporta el error a Firebase Analytics (`source: 'http'`), si hay analytics.
  *
@@ -1541,7 +1557,16 @@ const errorLoggingInterceptor = (request, next) => {
     // `AnalyticsService` puede no estar provisto (apps sin Firebase). El inject
     // funcional con `optional: true` devuelve `null` en ese caso.
     const analytics = inject(AnalyticsService, { optional: true });
-    return next(request).pipe(catchError((error) => {
+    return next(request).pipe(retry({
+        count: 2,
+        delay: (error, retryCount) => {
+            const status = error?.status;
+            if (request.method !== 'GET' || !RETRYABLE_STATUSES.has(status)) {
+                return throwError(() => error);
+            }
+            return timer(RETRY_DELAY_MS * retryCount);
+        },
+    }), catchError((error) => {
         const interpreted = interpretError(error);
         const status = interpreted.status ?? error?.status;
         // 401 → ruido del refresh de auth. Log informativo, sin Analytics.
@@ -1933,7 +1958,20 @@ class I18nService {
         return namespace in this._content();
     }
     /**
-     * Carga idioma guardado en localStorage o detecta del navegador
+     * Carga el idioma guardado en localStorage, o adivina desde el navegador.
+     *
+     * La detección de acá es PROVISIONAL y NO se persiste, a propósito. El
+     * constructor corre antes que el APP_INITIALIZER de `provideValtechI18n`,
+     * que es quien conoce la config real de la app. Cuando esto persistía el
+     * idioma detectado, el initializer leía esa misma clave un instante después,
+     * la tomaba por preferencia guardada del usuario y entraba por la rama
+     * `stored` — así que `detectBrowserLanguage: false` no tenía efecto en
+     * ninguna app del factory. Escribir acá fabricaba una preferencia que el
+     * usuario nunca expresó.
+     *
+     * Persistir es responsabilidad de `setLanguage()`, que se llama cuando la
+     * elección sí es del usuario (o cuando el initializer ya decidió con la
+     * config a la vista).
      */
     loadStoredLanguage() {
         // SSR: sin localStorage/navigator. Mantenemos el default; el browser
@@ -1946,11 +1984,11 @@ class I18nService {
             this._lang.set(stored);
             return;
         }
-        // Detectar idioma del navegador
+        // Adivinanza provisional mientras el initializer no haya corrido. Sin
+        // persistir: ver el comentario del método.
         const browserLang = navigator.language.split('-')[0];
         if (this.isValidLanguage(browserLang)) {
             this._lang.set(browserLang);
-            localStorage.setItem(LANG_STORAGE_KEY$1, browserLang);
         }
     }
     /**
@@ -40767,13 +40805,16 @@ class SurveyBuilderComponent {
         }
     }
     async ngOnInit() {
+        // Sembrado directo: la app ya tenía la encuesta, no hace falta leerla.
+        if (this.props.initial) {
+            this.hydrate(this.props.initial.label, this.props.initial.subtitle, this.props.initial.questions);
+            return;
+        }
         if (!this.props.typeId)
             return;
         try {
             const cfg = await firstValueFrom(this.requests.getRequestType(this.props.typeId));
-            this.titleControl.setValue(cfg.label);
-            this.subtitleControl.setValue(cfg.subtitle ?? '');
-            this.questions.set([...cfg.fieldSchema].sort((a, b) => a.order - b.order).map(f => this.rowFromField(f)));
+            this.hydrate(cfg.label, cfg.subtitle ?? '', cfg.fieldSchema);
         }
         catch (err) {
             this.errors.handle(err, {
@@ -40960,6 +41001,11 @@ class SurveyBuilderComponent {
             });
         }
     }
+    hydrate(label, subtitle, questions) {
+        this.titleControl.setValue(label ?? '');
+        this.subtitleControl.setValue(subtitle ?? '');
+        this.questions.set([...(questions ?? [])].sort((a, b) => a.order - b.order).map(f => this.rowFromField(f)));
+    }
     rowFromField(f) {
         return {
             id: `row-${rowUid++}`,