supabase / supabase-flutter
85%

DEFAULT BRANCH: main
Repo Added
Token z3GUkNtD1PDnCOWpuPvCMvjUB924F4R9N regen
Build 732 Last
Files 80
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH main
branch: SELECT
CHANGE BRANCH
x
Sync Branches

coverage: 85.187% (-0.002%) from 85.189%
27986095786

push

github

web-flow 
chore: enable avoid-passing-async-when-sync-expected lint (#1455)

> Stacked on top of #1454. Review/merge that one first; the base will
retarget to `main` automatically once it lands.

## What

Enables the DCM rule
[`avoid-passing-async-when-sync-expected`](https://dcm.dev/docs/rules/common/avoid-passing-async-when-sync-expected/)
(removing it from the ratchet list in `supabase_lints`) and resolves the
13 resulting violations.

## Why

When an `async` callback is passed where a synchronous one is expected,
the returned `Future` is discarded, so if it throws the error is
silently lost. This is the same class of problem as the
`unawaited_futures`/`discarded_futures` lints in #1454, just at call
sites that take a callback. Enabling it closes the remaining
async-safety gap.

## How

Each of the 13 sites was made synchronous, and the inner async work was
either wrapped in `unawaited(...)` (genuine fire-and-forget) or
extracted into a dedicated method, preserving existing behavior and
error handling. Highlights:

- `realtime_channel.dart`: extracted the `joinPush.receive('ok', ...)`
body into `_handleJoinOk(...)` (the `InvalidJWTToken` setAuth handling
is preserved verbatim).
- `realtime_client.dart`: extracted reconnect into `_reconnect()`,
heartbeat timer callback made sync with `unawaited(sendHeartbeat())`.
- `supabase_client.dart`: `onAuthStateChangeSync` listener made sync
with `unawaited(_handleTokenChanged(...))`.
- `supabase_stream_builder.dart`: `onDone: controller.close` → `onDone:
() => unawaited(controller.close())` in the asyncMap/asyncExpand
reimplementations.
- Remaining sites are in test mock servers and a widget-test stub.

No public API signatures were changed.

## Verification

- `dcm analyze`: 0 `avoid-passing-async-when-sync-expected` across all
packages.
- `dart analyze`: clean, no new `unawaited_futures`/`discarded_futures`.
- Tests: realtime_client (channel/socket/mock), supabase
(mock/utilities), and supabase_flutter suites all pass.

40 of 44 new or added lines in 5 files covered. (90.91%)

13 existing lines in 1 file now uncovered.

4106 of 4820 relevant lines covered (85.19%)

3.62 hits per line

Relevant lines Covered
4820 RELEVANT LINES 4106 COVERED LINES
3.62 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
27986095786 main chore: enable avoid-passing-async-when-sync-expected lint (#1455) > Stacked on top of #1454. Review/merge that one first; the base will retarget to `main` automatically once it lands. ## What Enables the DCM rule [`avoid-passing-async-when-sync... push web-flow github
85.19
27985660550 refactor/refresh-flow Merge 9d8886820 into d9eb03332 Pull #1457 web-flow github
82.74
27985059140 refactor/refresh-flow Merge 6ea6fb8cb into d9eb03332 Pull #1457 web-flow github
82.74
27984271083 feat/signed-out-reason Merge 5d6c8e12e into d9eb03332 Pull #1453 web-flow github
82.7
27984307788 fix/recover-session-duplicate-auth-error Merge d9eb03332 into db754cf06 Pull #1450 web-flow github
85.23
27984173732 feat/signed-out-reason Merge 946f20038 into 934b46ce4 Pull #1453 web-flow github
82.38
27971004383 chore/avoid-passing-async-when-sync-expected Merge 11159716a into db754cf06 Pull #1455 web-flow github
85.19
27970805559 main fix(realtime_client): detect dropped connections on iOS via WebSocket ping (#1451) ## What Enable `pingInterval` on the native (`dart:io`) WebSocket so dropped connections are detected consistently across platforms. ## Why Fixes #1071. When th... push web-flow github
85.19
27970373556 chore/avoid-passing-async-when-sync-expected Merge c5f817808 into efdfeb7ad Pull #1455 web-flow github
85.18
27970191844 main fix(supabase_flutter): clear auth params from web URL after exchange (#1456) ## What On Flutter web, after a successful OAuth/PKCE sign-in the auth `code` (or implicit-flow tokens) stays in the browser URL. On the next page refresh, `supabase_fl... push web-flow github
85.26
See All Builds (731)

Badge your Repo: supabase-flutter

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh