chore(postgrest): address copilot comments

Author: mandarini

packages/core/postgrest-js/src/PostgrestBuilder.ts

@@ -18,9 +18,24 @@ import { ContainsNull } from './select-query-parser/types'
 
 /**
  * Sleep for a given number of milliseconds.
+ * If an AbortSignal is provided, the sleep resolves early when the signal is aborted.
  */
-function sleep(ms: number): Promise<void> {
-  return new Promise((resolve) => setTimeout(resolve, ms))
+function sleep(ms: number, signal?: AbortSignal): Promise<void> {
+  return new Promise((resolve) => {
+    if (signal?.aborted) {
+      resolve()
+      return
+    }
+    const id = setTimeout(() => {
+      signal?.removeEventListener('abort', onAbort)
+      resolve()
+    }, ms)
+    function onAbort() {
+      clearTimeout(id)
+      resolve()
+    }
+    signal?.addEventListener('abort', onAbort)
+  })
 }
 
 /**
@@ -159,9 +174,9 @@ export default abstract class PostgrestBuilder<
    *
    * Configure retry behavior for this request.
    *
-   * By default, retries are enabled for GET/HEAD requests that fail with
-   * 520 status codes (Cloudflare timeout/connection errors). Retries use
-   * exponential backoff (1s, 2s, 4s) with a maximum of 3 attempts.
+   * By default, retries are enabled for idempotent requests (GET, HEAD, OPTIONS)
+   * that fail with network errors or specific HTTP status codes (503, 520).
+   * Retries use exponential backoff (1s, 2s, 4s) with a maximum of 3 attempts.
    *
    * @param enabled - Whether to enable retries for this request
    *
@@ -251,7 +266,7 @@ export default abstract class PostgrestBuilder<
           if (this.retryEnabled && attemptCount < DEFAULT_MAX_RETRIES) {
             const delay = getRetryDelay(attemptCount)
             attemptCount++
-            await sleep(delay)
+            await sleep(delay, this.signal)
             continue
           }
 
@@ -268,7 +283,7 @@ export default abstract class PostgrestBuilder<
               : getRetryDelay(attemptCount)
           await res.text()
           attemptCount++
-          await sleep(delay)
+          await sleep(delay, this.signal)
           continue
         }
 

packages/core/postgrest-js/src/PostgrestClient.ts

@@ -55,9 +55,9 @@ export default class PostgrestClient<
    * @param options.timeout - Optional timeout in milliseconds for all requests. When set, requests will automatically abort after this duration to prevent indefinite hangs.
    * @param options.urlLengthLimit - Maximum URL length in characters before warnings/errors are triggered. Defaults to 8000.
    * @param options.retry - Enable or disable automatic retries for transient errors.
-   *   When enabled, GET/HEAD requests that fail with 520 errors (Cloudflare timeouts)
-   *   will be automatically retried up to 3 times with exponential backoff.
-   *   Defaults to `true`.
+   *   When enabled, idempotent requests (GET, HEAD, OPTIONS) that fail with network
+   *   errors or HTTP 503/520 responses will be automatically retried up to 3 times
+   *   with exponential backoff (1s, 2s, 4s). Defaults to `true`.
    * @example
    * ```ts
    * import { PostgrestClient } from '@supabase/postgrest-js'

packages/core/postgrest-js/src/PostgrestQueryBuilder.ts

@@ -24,9 +24,9 @@ export default class PostgrestQueryBuilder<
 
   /**
    * Enable or disable automatic retries for transient errors.
-   * When enabled, idempotent requests (GET/HEAD/OPTIONS) that fail with 520 or 503 errors
-   * are automatically retried with exponential backoff (up to 3 attempts).
-   * Defaults to `true` when not specified.
+   * When enabled, idempotent requests (GET/HEAD/OPTIONS) that fail with network
+   * errors or HTTP 503/520 responses are automatically retried with exponential
+   * backoff (1s, 2s, 4s, up to 3 attempts). Defaults to `true` when not specified.
    */
   retry?: boolean