feat: if throw true data is returned as it's

Author: Bekacru

src/fetch.ts

@@ -127,9 +127,7 @@ export const betterFetch = async <
 		}
 
 		if (options?.throw) {
-			return {
-				data: successContext.data,
-			} as any;
+			return successContext.data as any;
 		}
 
 		return {

src/test/fetch.test.ts

@@ -224,3 +224,27 @@ describe("fetch-error", () => {
 		await expect(f("/ok")).rejects.toThrowError(BetterFetchError);
 	});
 });
+
+describe("fetch-error-throw", () => {
+	const f = createFetch({
+		baseURL: "http://localhost:4001",
+		customFetchImpl: async (req, init) => {
+			const url = new URL(req.toString());
+			if (url.pathname.startsWith("/ok")) {
+				return new Response(JSON.stringify({ message: "ok" }));
+			}
+			return new Response(null, {
+				status: 500,
+			});
+		},
+		throw: true,
+	});
+	it("should throw if the response is not ok", async () => {
+		await expect(f("/not-ok")).rejects.toThrowError(BetterFetchError);
+	});
+
+	it("should return data without error object", async () => {
+		const res = await f<{ message: "ok" }>("/ok");
+		expect(res).toEqual({ message: "ok" });
+	});
+});

src/types.ts

@@ -149,8 +149,4 @@ export type BetterFetchResponse<
 	T,
 	E extends Record<string, unknown> | unknown = unknown,
 	Throw extends boolean = false,
-> = Throw extends true
-	? {
-			data: T;
-		}
-	: Data<T> | Error<E>;
+> = Throw extends true ? T : Data<T> | Error<E>;