@nitida/asset-client 0.20.3 → 0.20.4

package/src/index.ts

@@ -878,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]);
 }
 
 /**
@@ -909,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, "")