@nitida/asset-client 0.20.1 → 0.20.3

package/src/access.ts

@@ -241,9 +241,24 @@ export function assertPublic(
  */
 export function assertSha(asset: { sha?: unknown }, fn: string): void {
   const sha = asset?.sha;
-  if (typeof sha === "string" && /^[0-9a-f]{16,64}$/i.test(sha)) return;
-  const hint =
-    asset && typeof asset === "object" && "sha256" in asset
+  // ⭐ EXACTAMENTE 16, no `{16,64}`.
+  //
+  // Las claves del CDN son de 16 hex — el SDK hace `.slice(0, 16)` en cada
+  // sitio donde arma una. El rango 17..64 no nombra ninguna clave real, así que
+  // aceptarlo era dejar pasar precisamente el error más probable: poner el
+  // `sha256` COMPLETO en la propiedad correcta. Medido: eso producía
+  // `…/f/v/<64 hex>-m.webp`, una URL que 404ea «lejos de acá» — literalmente lo
+  // que el comentario de arriba dice que este guarda existe para impedir.
+  //
+  // Y lo inducía el propio mensaje: quien lee rápido «Use `{ sha:
+  // theSha256.slice(0, 16) }`» y escribe `{ sha: sha256 }` no recibía
+  // excepción, recibía la URL rota.
+  if (typeof sha === "string" && /^[0-9a-f]{16}$/i.test(sha)) return;
+  const looksLikeSha256 =
+    typeof sha === "string" && /^[0-9a-f]{64}$/i.test(sha);
+  const hint = looksLikeSha256
+    ? " ⚠️ You passed a FULL sha256 (64 hex) as `sha`. CDN keys use the first 16: `{ ...asset, sha: asset.sha256.slice(0, 16) }`. Interpolating all 64 builds a URL that 404s."
+    : asset && typeof asset === "object" && "sha256" in asset
       ? " The value you passed has `sha256` but not `sha`. ⚠️ Since `@nitida/sdk@0.30.0` `upload()` returns BOTH, so a fresh upload result works as-is — if you are seeing this, the value came from somewhere else (an older SDK, a hand-built object, or a raw API response). Use `{ sha: theSha256.slice(0, 16) }`, or fetch the DTO with `assets.get(id)`."
       : ` Got ${JSON.stringify(sha)}.`;
   throw new Error(

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`.