supabase / supabase-flutter / 29017180430

87%

push

github

feat(client): add session-URL-detection predicate and persistSession flag (#1558)

## Summary

Implements the two `partially_implemented` client initialization options
tracked by SDK-1272:

- **`FlutterAuthClientOptions.detectSessionInUriPredicate`** — an
optional `bool Function(Uri)` predicate that decides whether an incoming
deep link should be treated as an auth callback and exchanged for a
session. When null, the existing default heuristic (checks for
`access_token`, `code`, `error`, `error_code`, `error_description` in
the query or fragment) is used. This lets apps disambiguate redirects
when they use those same parameters for other purposes.
- **`FlutterAuthClientOptions.persistSession`** — a real flag (default
`true`). When `false` and no `localStorage` is supplied, sessions are
kept in memory only (`EmptyLocalStorage` is selected automatically), so
opting out of persistence no longer requires manually wiring up an
`EmptyLocalStorage` backend. Supplying a custom `localStorage` still
takes precedence.

Both are new optional initialization options, so existing
`Supabase.initialize` calls are unaffected.

## Outcome

implemented

## Reference

Derived from the compliance matrix (`sdk-compliance.yaml`); mirrors
supabase-js `detectSessionInUrl` / `persistSession` GoTrue client
options.

## Tests

Added to `packages/supabase_flutter/test/deep_link_test.dart`:
- predicate returning `false` suppresses detection of an otherwise valid
auth callback
- predicate governs detection based on the incoming URI (and receives
the URI)
- `persistSession: true` persists to the default storage
- `persistSession: false` keeps the session in memory only

All 63 `supabase_flutter` tests pass; `melos format` and `dart analyze`
clean.

## Compliance matrix

- `client.authentication_integration.session_url_detection` →
`implemented`
- `client.session_management.persist_session` → `implemented`

SDK-1272

10 of 10 new or added lines in 3 files covered. (100.0%)

4525 of 5227 relevant lines covered (86.57%)

4.1 hits per line