refactor: improve api error handling to avoid html parse error

Author: joaotonaco

.changeset/many-wombats-burn.md

@@ -0,0 +1,5 @@
+---
+"@squarecloud/api": patch
+---
+
+Improve API error handling to avoid HTML parse error.

src/services/api.ts

@@ -1,7 +1,5 @@
 import type { APIVersion } from "@squarecloud/api-types/v2";
 
-import { existsSync } from "node:fs";
-import { mkdir, writeFile } from "node:fs/promises";
 import { SquareCloudAPIError } from "@/structures";
 import type {
 	APIEndpoint,
@@ -27,9 +25,25 @@ export class APIService {
 		const response = await fetch(url, init).catch((err) => {
 			throw new SquareCloudAPIError(err.code, err.message);
 		});
-		const data = await response
-			.json()
-			.catch(() => this.handleParseError(response));
+
+		if (response.status === 413) {
+			throw new SquareCloudAPIError("PAYLOAD_TOO_LARGE", "Payload too large");
+		}
+
+		if (response.status === 429) {
+			throw new SquareCloudAPIError(
+				"RATE_LIMIT_EXCEEDED",
+				"Rate limit exceeded",
+			);
+		}
+
+		if (response.status === 502 || response.status === 504) {
+			throw new SquareCloudAPIError("SERVER_UNAVAILABLE", "Server unavailable");
+		}
+
+		const data = await response.json().catch(() => {
+			throw new SquareCloudAPIError("CANNOT_PARSE_RESPONSE", "Try again later");
+		});
 
 		if (!data || data.status === "error" || !response.ok) {
 			throw new SquareCloudAPIError(data?.code || "COMMON_ERROR");
@@ -69,20 +83,4 @@ export class APIService {
 
 		return { url, init };
 	}
-
-	private async handleParseError(response: Response) {
-		const text = await response.text();
-		const dir = ".squarecloud/logs/";
-		const path = `${dir}${this.userId}-${Date.now()}.log`;
-
-		if (!existsSync(dir)) {
-			await mkdir(dir);
-		}
-		await writeFile(path, text);
-
-		throw new SquareCloudAPIError(
-			"CANNOT_PARSE_JSON",
-			`Saved request text content to ${path}`,
-		);
-	}
 }