supabase / supabase-flutter / 27216654140

85%

push

github

fix(postgrest): include caller frames in PostgrestException stack traces (#1387)

## Summary

- Captures `StackTrace.current` synchronously at `PostgrestBuilder`
construction time (inside the constructor initializer list), which
always runs in user code context when chaining filter methods like
`.select()`, `.eq()`, etc.
- Rewrites `then()` to be non-`async`, using an intermediate native
`Future.then()` to enable Dart's Zone causal-chain machinery.
- Appends the construction-time trace to every error via
`StackTrace.fromString('$internalStack\n$_callerTrace')` and re-throws
with `Error.throwWithStackTrace`, so both `await builder` (plain usage)
and explicit `.then(onError:)` usage show user frames.

Fixes #899.

## Test plan

- [x] `dart test test/stack_trace_test.dart` passes both cases: plain
`await builder` and explicit `.then()` with `onError`
- [x] `dart test test/retry_test.dart` still passes (no regression in
retry behavior)
- [x] Integration tests pass against a live PostgREST instance

10 of 17 new or added lines in 1 file covered. (58.82%)

3568 of 4426 relevant lines covered (80.61%)

2.94 hits per line