supabase / supabase-flutter / 29023496087

87%

push

github

refactor: drop retry dependency in favor of an in-house helper (#1571)

## What

Removes the `retry` package from both `gotrue` and `storage_client`,
replacing it with a small self-contained `RetryOptions`/`retry` helper
(`lib/src/retry.dart`) in each package.

- `gotrue` used the top-level `retry()` (token refresh).
- `storage_client` used `RetryOptions(...).retry(...)` (request
retries).

## Why

Both packages pulled in `retry` for the same narrow use: exponential
backoff gated by a `retryIf` predicate. The helper is ~40 lines and
mirrors the package's behavior exactly:

- 200 ms delay factor, `2^attempt` backoff
- 25% jitter (`randomizationFactor`)
- 30 s max delay
- 8 max attempts by default

`gotrue`'s time-budget `retryIf` depends on these exact defaults, so
they are preserved.

## Testing

- New `retry_test.dart` in `gotrue` covers success-after-retries,
`maxAttempts` cutoff, `retryIf` short-circuit, `RetryOptions.retry` (the
storage usage shape), and exponential/capped delay math — all pass.
- `dart analyze` clean for both packages.

Note: the helper is duplicated in each package because they are
published independently and there is no shared internal utility package.

33 of 38 new or added lines in 2 files covered. (86.84%)

4601 of 5312 relevant lines covered (86.62%)

4.08 hits per line