@nitida/asset-client 0.20.2 → 0.20.4

package/src/index.ts

@@ -648,7 +648,22 @@ type OriginalHints = {
  * ```
  */
 export function getAssetUrl(
-  asset: Pick<AssetDTO, "sha"> & OriginalHints & VisibilityHint,
+  /**
+   * ⭐ `presets` ESTÁ EN EL TIPO PORQUE LA FUNCIÓN LO LEE.
+   *
+   * `transformFallbackFor` decide con este campo si devolver la clave de
+   * variante o caer a `/t/…width=N/`. Sin él en la firma, el tipo PROHIBÍA el
+   * campo que la implementación consulta: pasarlo era un error de compilación,
+   * y no pasarlo daba `…-x.webp` → 404 en silencio.
+   *
+   * Medido 2026-08-25 sobre un asset con `presets: "lmoqs"`:
+   *   sin `presets` → /f/v/<sha>-x.webp                     404
+   *   con `presets` → /t/format=webp,width=3840/<sha>.webp  200
+   */
+  asset: Pick<AssetDTO, "sha"> &
+    OriginalHints &
+    VisibilityHint &
+    Partial<Pick<AssetDTO, "presets">>,
   preset: VariantPreset,
 ): string {
   // Refuses rather than returning a URL that 404s. See `assertPublic`.
@@ -863,11 +878,30 @@ export async function getPrivateTransformUrl(
  * ```
  */
 export function hasPreset(
-  asset: Pick<AssetDTO, "presets">,
+  asset: { presets?: string | null },
   preset: VariantPreset,
 ): boolean {
-  if (preset === "mp3") return asset.presets.includes("mp3");
-  return stripMultiCharTokens(asset.presets).includes(PRESET_SHORT[preset]);
+  // ⭐⭐ `presets` PUEDE FALTAR, Y ASUMIR QUE NO ERA UN CRASH EN PRODUCCIÓN
+  //
+  // El DTO que `assets.byHash(sha)` devuelve no siempre trae `presets` — igual
+  // que no traía `sha`. Y `upload()` le pasa ese DTO crudo a `bestPresetForAsset`
+  // → `hasPreset` → acá. Con `presets` ausente, `asset.presets.includes(...)`
+  // y `stripMultiCharTokens(asset.presets)` reventaban con
+  // `undefined is not an object (evaluating 'presets.replace')`.
+  //
+  // Medido en el canario de neo 2026-08-25: verde con 0.30.2, rojo con 0.32.2,
+  // en la rama ORDINARIA del dedup (`existing.status === "ready"`) — o sea toda
+  // re-subida de bytes que ya existen, no el caso raro de ayer.
+  //
+  // ⭐ La invariante que faltaba no era «el objeto lleva `sha`» sino «todo
+  // objeto que `upload()` le pasa a un helper satisface la forma que el helper
+  // asume». Se arregla ensanchando la forma del helper —`presets` opcional—, no
+  // parcheando cada llamador de a uno. Un `presets` ausente significa «no sé qué
+  // variantes tiene», que es `false` para cualquier preset: el llamador cae a su
+  // default, que es lo correcto.
+  const presets = asset.presets ?? "";
+  if (preset === "mp3") return presets.includes("mp3");
+  return stripMultiCharTokens(presets).includes(PRESET_SHORT[preset]);
 }
 
 /**
@@ -894,8 +928,8 @@ export function hasPreset(
  * keep sending them, and this is the check every guide points at as the
  * reliable one. Order matters — strip the longest tokens first.
  */
-function stripMultiCharTokens(presets: string): string {
-  return presets
+function stripMultiCharTokens(presets: string | null | undefined): string {
+  return (presets ?? "")
     .replace(/transform-[0-9a-f]*/g, "")
     .replace(/upscale_[a-z0-9_]*/g, "")
     .replace(/mp3/g, "")