refactor: improve retry logic with a for loop

This commit refactors the retry logic to use a `for` loop, which is cleaner and more concise than the previous `while` loop implementation. The exit conditions are now combined into a single check, improving readability and maintainability.

This change also addresses the original issue of stack overflows and memory leaks caused by the recursive implementation.

Author: google-labs-jules[bot]

pages/index.tsx

@@ -147,22 +147,28 @@ const Index = () => {
       isCutBorders ? "cutBorders" : "",
       anilistFilter ? `anilistID=${anilistFilter}` : "",
     ].join("&");
-    const res = await fetch(`${NEXT_PUBLIC_API_ENDPOINT}/search?${queryString}`, {
-      method: "POST",
-      body: formData,
-    });
+    let res;
+    for (let retries = 5; retries > 0; retries--) {
+      res = await fetch(`${NEXT_PUBLIC_API_ENDPOINT}/search?${queryString}`, {
+        method: "POST",
+        body: formData,
+      });
+      if (res.status !== 503 || retries === 1) {
+        break;
+      }
+      for (let i = 5; i > 0; i--) {
+        setMessageText(`Server is busy, retrying in ${i}s`);
+        await new Promise((resolve) => setTimeout(resolve, 1000));
+      }
+    }
     setIsSearching(false);
 
     if (res.status === 429) {
       setMessageText("You searched too many times, please try again later.");
       return;
     }
     if (res.status === 503) {
-      for (let i = 5; i > 0; i--) {
-        setMessageText(`Server is busy, retrying in ${i}s`);
-        await new Promise((resolve) => setTimeout(resolve, 1000));
-      }
-      search(imageBlob);
+      setMessageText("Server is busy, please try again later.");
       return;
     }
     if (res.status >= 400) {