fix: only processing the first response success handler

Fixes an issue with response success handlers where only the first handler was processed.

src/retrieve.test.ts

@@ -955,6 +955,22 @@ describe('retrieve', () => {
 					} satisfies RetrieveConfig,
 					'test',
 				],
+				[
+					{
+						url: 'http://example.org',
+						responseSuccessHandlers: [
+							(retrieveResponse) => {
+								retrieveResponse.data = 'test'
+								return Promise.resolve(retrieveResponse)
+							},
+							(retrieveResponse) => {
+								retrieveResponse.data = 'overridden data'
+								return Promise.resolve(retrieveResponse)
+							},
+						],
+					} satisfies RetrieveConfig,
+					'overridden data',
+				],
 			])('onResponseSuccess handlers produce response', async (config: RetrieveConfig, expectedData: unknown) => {
 				vi.spyOn(global, 'fetch').mockImplementation(() => Promise.resolve(new Response('OK')))
 

src/retrieve.ts

@@ -313,10 +313,10 @@ export async function retrieve(config: RetrieveConfig): Promise<RetrieveResponse
 	let retrieveResponse = await createRetrieveResponse(response)
 
 	for (const responseSuccessHandler of config.responseSuccessHandlers ?? []) {
-		if (retrieveResponse.response.ok) {
-			retrieveResponse = await responseSuccessHandler(retrieveResponse)
-		}
+		retrieveResponse = await responseSuccessHandler(retrieveResponse)
+	}
 
+	if (retrieveResponse.response.ok) {
 		return retrieveResponse
 	}