fix(adapter-cloudflare-workers): throws status 404 if KVError instanceof

Author: vicentematus

.changeset/large-guests-fix.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/adapter-cloudflare-workers': minor
+---
+
+fix: return 404 for non-existent asset rather than throwing KVError

packages/adapter-cloudflare-workers/files/entry.js

@@ -2,6 +2,7 @@ import { Server } from 'SERVER';
 import { manifest, prerendered } from 'MANIFEST';
 import { getAssetFromKV, mapRequestToAsset } from '@cloudflare/kv-asset-handler';
 import static_asset_manifest_json from '__STATIC_CONTENT_MANIFEST';
+import { KVError } from '@cloudflare/kv-asset-handler/dist/types';
 const static_asset_manifest = JSON.parse(static_asset_manifest_json);
 
 const server = new Server(manifest);
@@ -86,19 +87,25 @@ export default {
  * @param {any} context
  */
 async function get_asset_from_kv(req, env, context, map = mapRequestToAsset) {
-	return await getAssetFromKV(
-		{
-			request: req,
-			waitUntil(promise) {
-				return context.waitUntil(promise);
+	try {
+		return await getAssetFromKV(
+			{
+				request: req,
+				waitUntil(promise) {
+					return context.waitUntil(promise);
+				}
+			},
+			{
+				ASSET_NAMESPACE: env.__STATIC_CONTENT,
+				ASSET_MANIFEST: static_asset_manifest,
+				mapRequestToAsset: map
 			}
-		},
-		{
-			ASSET_NAMESPACE: env.__STATIC_CONTENT,
-			ASSET_MANIFEST: static_asset_manifest,
-			mapRequestToAsset: map
-		}
-	);
+		);
+	} catch (err) {
+		if (err instanceof KVError) err.status = 404;
+
+		throw err;
+	}
 }
 
 /**