From a7754a33133ffcb3cbb1b46b94f58b38d203a31c Mon Sep 17 00:00:00 2001
From: Davide Tantillo <iamdjj@gmail.com>
Date: Tue, 17 Mar 2026 10:25:24 +0100
Subject: [PATCH] [ts][pixi-v7][pixi-v8] Fix asset loaders not throwing when
 http response is not ok. See #3042.

---
 spine-ts/spine-pixi-v7/src/assets/atlasLoader.ts  |  6 +++---
 .../spine-pixi-v7/src/assets/skeletonLoader.ts    | 15 ++++++++-------
 spine-ts/spine-pixi-v8/src/assets/atlasLoader.ts  |  5 +++--
 .../spine-pixi-v8/src/assets/skeletonLoader.ts    |  5 +++--
 4 files changed, 17 insertions(+), 14 deletions(-)

diff --git a/spine-ts/spine-pixi-v7/src/assets/atlasLoader.ts b/spine-ts/spine-pixi-v7/src/assets/atlasLoader.ts
index 7ff61ba14..b87674ee2 100644
--- a/spine-ts/spine-pixi-v7/src/assets/atlasLoader.ts
+++ b/spine-ts/spine-pixi-v7/src/assets/atlasLoader.ts
@@ -69,9 +69,10 @@ const spineTextureAtlasLoader: AssetExtension<RawAtlas | TextureAtlas, ISpineAtl
 		async load (url: string): Promise<RawAtlas> {
 			const response = await settings.ADAPTER.fetch(url);
 
-			const txt = await response.text();
+			if (!response.ok)
+				throw new Error(`[${loaderName}] Failed to fetch ${url}: ${response.status} ${response.statusText}`);
 
-			return txt;
+			return await response.text();
 		},
 
 		testParse (asset: unknown, options: ResolvedAsset): Promise<boolean> {
@@ -107,7 +108,6 @@ const spineTextureAtlasLoader: AssetExtension<RawAtlas | TextureAtlas, ISpineAtl
 			// we will wait for all promises for the textures at the same time at the end.
 			const textureLoadingPromises = [];
 
-
 			// setting preferCreateImageBitmap to false for loadTextures loader to allow loading PMA images
 			let oldPreferCreateImageBitmap = true;
 			for (const parser of loader.parsers) {
diff --git a/spine-ts/spine-pixi-v7/src/assets/skeletonLoader.ts b/spine-ts/spine-pixi-v7/src/assets/skeletonLoader.ts
index 610ef120a..a3d943b46 100644
--- a/spine-ts/spine-pixi-v7/src/assets/skeletonLoader.ts
+++ b/spine-ts/spine-pixi-v7/src/assets/skeletonLoader.ts
@@ -36,11 +36,11 @@ type SkeletonBinaryAsset = Uint8Array;
 
 const loaderName = "spineSkeletonLoader";
 
-function isJson(resource: any): resource is SkeletonJsonAsset {
+function isJson (resource: any): resource is SkeletonJsonAsset {
 	return resource.hasOwnProperty("bones");
 }
 
-function isBuffer(resource: any): resource is SkeletonBinaryAsset {
+function isBuffer (resource: any): resource is SkeletonBinaryAsset {
 	return resource instanceof Uint8Array;
 }
 
@@ -55,18 +55,19 @@ const spineLoaderExtension: AssetExtension<SkeletonJsonAsset | SkeletonBinaryAss
 			name: loaderName,
 		},
 
-		test(url) {
+		test (url) {
 			return checkExtension(url, ".skel");
 		},
 
-		async load(url: string): Promise<SkeletonBinaryAsset> {
+		async load (url: string): Promise<SkeletonBinaryAsset> {
 			const response = await settings.ADAPTER.fetch(url);
 
-			const buffer = new Uint8Array(await response.arrayBuffer());
+			if (!response.ok)
+				throw new Error(`[${loaderName}] Failed to fetch ${url}: ${response.status} ${response.statusText}`);
 
-			return buffer;
+			return new Uint8Array(await response.arrayBuffer());
 		},
-		testParse(asset: unknown, options: ResolvedAsset): Promise<boolean> {
+		testParse (asset: unknown, options: ResolvedAsset): Promise<boolean> {
 			const isJsonSpineModel = checkExtension(options.src!, ".json") && isJson(asset);
 			const isBinarySpineModel = checkExtension(options.src!, ".skel") && isBuffer(asset);
 			const isExplicitLoadParserSet = options.loadParser === loaderName;
diff --git a/spine-ts/spine-pixi-v8/src/assets/atlasLoader.ts b/spine-ts/spine-pixi-v8/src/assets/atlasLoader.ts
index 148e8d281..552454150 100644
--- a/spine-ts/spine-pixi-v8/src/assets/atlasLoader.ts
+++ b/spine-ts/spine-pixi-v8/src/assets/atlasLoader.ts
@@ -83,9 +83,10 @@ const spineTextureAtlasLoader: AssetExtension<RawAtlas | TextureAtlas, ISpineAtl
 		async load (url: string): Promise<RawAtlas> {
 			const response = await DOMAdapter.get().fetch(url);
 
-			const txt = await response.text();
+			if (!response.ok)
+				throw new Error(`[${loaderName}] Failed to fetch ${url}: ${response.status} ${response.statusText}`);
 
-			return txt;
+			return await response.text();
 		},
 
 		testParse (asset: unknown, options: ResolvedAsset): Promise<boolean> {
diff --git a/spine-ts/spine-pixi-v8/src/assets/skeletonLoader.ts b/spine-ts/spine-pixi-v8/src/assets/skeletonLoader.ts
index b6a74cc44..c99ad4d84 100644
--- a/spine-ts/spine-pixi-v8/src/assets/skeletonLoader.ts
+++ b/spine-ts/spine-pixi-v8/src/assets/skeletonLoader.ts
@@ -69,9 +69,10 @@ const spineLoaderExtension: AssetExtension<SkeletonJsonAsset | SkeletonBinaryAss
 		async load (url: string): Promise<SkeletonBinaryAsset> {
 			const response = await DOMAdapter.get().fetch(url);
 
-			const buffer = new Uint8Array(await response.arrayBuffer());
+			if (!response.ok)
+				throw new Error(`[${loaderName}] Failed to fetch ${url}: ${response.status} ${response.statusText}`);
 
-			return buffer;
+			return new Uint8Array(await response.arrayBuffer());
 		},
 		testParse (asset: unknown, options: ResolvedAsset): Promise<boolean> {
 			const isJsonSpineModel = checkExtension(options.src!, '.json') && isJson(asset);