supabase / supabase-flutter / 24099181632

80%

Build Type

push

github

Committed by web-flow

Commit Message

feat(postgrest): add automatic retry for transient failures (#1338)

* feat(postgrest): add automatic retry for transient failures

Implements retry logic for the Flutter PostgREST client, mirroring the
Swift SDK (SDK-771) and supabase-js reference implementations.

Key behavior:
- Only retries idempotent methods: GET and HEAD
- Retry conditions: HTTP 520 or network/connection error
- Up to 3 retries with exponential backoff: 1s → 2s → 4s (capped at 30s)
- Adds X-Retry-Count: <n> header on each retry attempt
- Enabled by default; disable globally via PostgrestClient(retryEnabled: false)
- Per-request override via .retry(enabled: false/true)

Acceptance Criteria:
- [x] Retry logic only applies to GET and HEAD
- [x] HTTP 520 and network errors trigger retries; other status codes do not
- [x] Exponential backoff: 1s, 2s, 4s (capped at 30s)
- [x] X-Retry-Count header present on retried requests
- [x] retryEnabled: false on PostgrestClient disables globally
- [x] .retry(enabled: false/true) overrides per request
- [x] All existing tests pass
- [x] 11 new tests cover all retry scenarios

Linear: SDK-785

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(postgrest): resolve analyzer warnings in retry implementation

- Use initializing formal (this.retryEnabled) to fix prefer_initializing_formals
- Remove @visibleForTesting from retryDelay in PostgrestQueryBuilder and
  PostgrestRpcBuilder since they are called from production code in
  postgrest.dart; keep annotation only on PostgrestClient and PostgrestBuilder

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(postgrest): avoid mutating shared _headers map during execution

_execute() was writing Prefer, Accept-Profile, Content-Profile,
Content-Type, and X-Retry-Count directly into _headers. Because
_copyWith passes the same map reference when headers are not
overridden, sibling builders share the map, and awaiting a builder
more than once accumulates mutations.

Switch to a pe... (continued)

Coverage Stats

110 of 158 new or added lines in 9 files covered. (69.62%)

2 existing lines in 2 files now uncovered.

3489 of 4301 relevant lines covered (81.12%)

2.9 hits per line