From 1d16db1916ca5763e7d8e6f1ffd7aa072b5977ef Mon Sep 17 00:00:00 2001
From: Robert Reinhard <robert.reinhard@bukwild.com>
Date: Mon, 21 Aug 2023 22:22:37 -0700
Subject: [PATCH] Catch errors when fetching image URL

---
 packages/sanity-next/src/lib/urlBuilding.ts | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/packages/sanity-next/src/lib/urlBuilding.ts b/packages/sanity-next/src/lib/urlBuilding.ts
index 1f3917e..ed4000c 100644
--- a/packages/sanity-next/src/lib/urlBuilding.ts
+++ b/packages/sanity-next/src/lib/urlBuilding.ts
@@ -25,7 +25,7 @@ export function makeImageUrl(
   options?: imageUrlBuildingOptions
 ): string | undefined {
   if (!source) return undefined
-  return makeImageBuilder(source, options).url()
+  return getBuilderUrl(makeImageBuilder(source, options))
 }
 
 // Add common conventions when building URLs to images
@@ -57,7 +57,19 @@ export function makeImageLoader(
   return ({ width, quality }: ImageLoaderProps): string => {
     let builder = makeImageBuilder(source, { width })
     if (quality) builder = builder.quality(quality)
-    return builder.url()
+    return getBuilderUrl(builder)
+  }
+}
+
+// Get the URL from an imageBuilder or gracefully fail.  This is to solve for
+// issues I experienced in Sanity preview preview mode when uploading images.
+// The `asset` property of the image source would be empty and this would
+// cause the `imageBuilder` to fatally error.
+function getBuilderUrl(builder: ImageUrlBuilder): string {
+  try { return builder.url() }
+  catch(e) {
+    console.error(e)
+    return ''
   }
 }