claude: consume body immediately

Author: flicksolutions

src/routes/einzelverssynopse/[thirties=thirties]/[verse]/+page.server.js

@@ -92,6 +92,9 @@ export async function load({ fetch, params }) {
 
 	const nextVerse = index < filteredVerses.length - 1 ? filteredVerses[index + 1] : null;
 
+	/** Helper: fetch and consume the body immediately so no Response body is left unread */
+	const fetchJson = (/** @type {string} */ url) => fetch(url).then((r) => (r.ok ? r.json() : null));
+
 	// Fetch the textzeugen
 	const hasSuffix = verseparts[1]; // check if there is a suffix in the URL
 	await Promise.all(
@@ -107,7 +110,9 @@ export async function load({ fetch, params }) {
 					);
 
 					publisherData[element.handle] = versesToFetch.map((verseObject) => {
-						return fetch(`/einzelverssynopse/data/${handlePath}/${thirties}/${verseObject.verse}`);
+						return fetchJson(
+							`/einzelverssynopse/data/${handlePath}/${thirties}/${verseObject.verse}`
+						);
 					});
 					hasAdditions = true;
 				} else {
@@ -125,7 +130,9 @@ export async function load({ fetch, params }) {
 					}
 
 					publisherData[element.handle] = versesToFetch.map((verseObject) => {
-						return fetch(`/einzelverssynopse/data/${handlePath}/${thirties}/${verseObject.verse}`);
+						return fetchJson(
+							`/einzelverssynopse/data/${handlePath}/${thirties}/${verseObject.verse}`
+						);
 					});
 				}
 			}
@@ -138,30 +145,25 @@ export async function load({ fetch, params }) {
 			(/** @type {{ handle: string | number; }} */ element) => {
 				const handlePath = encodeURIComponent(String(element.handle));
 				publisherData[element.handle] = [
-					fetch(`/einzelverssynopse/data/fassungen/${handlePath}/${thirties}/${verse ?? '01'}`)
+					fetchJson(`/einzelverssynopse/data/fassungen/${handlePath}/${thirties}/${verse ?? '01'}`)
 				];
 			}
 		);
 	}
 
 	/** @type {string[]} */
 	let loss = [];
-	// Wait for all promises to resolve and filter those with status 200
+	// Wait for all promises to resolve and filter those with no data
 	await metadata;
 	const resolvedPublisherData = await Promise.all(
 		Object.entries(publisherData).map(async ([key, promiseArray]) => {
-			let responses = await Promise.all(promiseArray);
-			let data = null;
-			if (responses.some((res) => res.status === 200)) {
-				data = await Promise.all(
-					responses.filter((res) => res.status === 200).map(async (res) => await res.clone().json())
-				);
-			}
-			if (data === null && !key.includes('fr')) {
+			const results = await Promise.all(promiseArray);
+			const data = results.filter((r) => r !== null);
+			if (data.length === 0 && !key.includes('fr')) {
 				loss.push(key);
 				return null;
 			}
-			return [key, data];
+			return [key, data.length > 0 ? data : null];
 		})
 	).then((results) => results.filter((result) => result !== null));
 

src/routes/textzeugen/data/[witnes]/[page]/+server.js

@@ -20,15 +20,19 @@ export async function GET({ params, fetch }) {
 export async function entries() {
 	const { codices, fragments } = await metadata;
 	const handles = [...codices, ...fragments].map((c) => c.handle);
-	const metadataPage = handles.map((h) =>
-		fetch(`${URL_STATIC_API}/json/metadata-ms-page/${h}.json`).then((r) => {
+	const metadataPage = handles.map(async (h) => {
+		try {
+			const r = await fetch(`${URL_STATIC_API}/json/metadata-ms-page/${h}.json`);
 			if (!r.ok) {
 				console.error(`Failed to fetch metadata-ms-page for ${h}: ${r.status}`);
 				return null;
 			}
 			return r.json();
-		})
-	);
+		} catch (e) {
+			console.error(`Error fetching metadata-ms-page for ${h}:`, e);
+			return null;
+		}
+	});
 
 	/**
 	 * @type {PromiseLike<import("./$types").RouteParams[]> | { witnes: string; page: any; }[]}