supabase / supabase-flutter / 37939928541

92%

Build Type

push

github

Committed by web-flow

Commit Message

refactor!: remove SupabaseClientPlugin and the public PostgrestTableExecutor (#1939)

## What

#1857 was merged before the plugin seam was ready. This takes the plugin
system and the executor extension point back out of the public API.
#1940 brings them back for review.

The typed builders keep their internal structure: they still collect a
`PostgrestTableRequest` and run it through `PostgrestHttpTableExecutor`.
Several later PRs build on that (#1907, #1913, #1910, #1897 and others).
Only the public surface is removed:

- `SupabaseClientPlugin`, `SupabaseClient(plugins:)`,
`SupabaseClient.plugins`, `Supabase.initialize(plugins:)` and the plugin
resume handling in the Flutter lifecycle listener, and
`testSupabaseClient(plugins:)`.
- `PostgrestClient.table(executor:)`,
`SupabaseQuerySchema(tableExecutor:)` and
`PostgrestTypedBuilder.request`.
- `PostgrestTableExecutor`, `PostgrestHttpTableExecutor`,
`PostgrestTableRequest`, `PostgrestTableResult`,
`PostgrestTableOperation`, `PostgrestResultShape`,
`PostgrestEmbeddedPage`, `PostgrestExplainOptions` and
`PostgrestRequestOptions` are no longer exported.
- `PostgrestTypedQueryBuilder` and `SupabaseTypedQueryBuilder` take a
`client` instead of an `executor`.

The matching `sdk-compliance.yaml` entries and the plugin paragraph in
`MIGRATION.md` are removed, and `AGENTS.md` describes the executor as
internal.

## Tests

`table_executor_test.dart`, `client_plugin_test.dart` and the Flutter
`plugin_test.dart` are removed with the API they cover. They come back
in #1940. The update/delete test in `typed_query_test.dart` used to
assert on `request.filter`. It now checks the filter on the request that
goes out.


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Breaking Changes**
* Supabase client initialization no longer accepts plugins, and plugin
lifecycle and request-wrapping support have been removed.
* Typed table queries no longer allow supplying a cus... (continued)

Coverage Stats

29 of 40 new or added lines in 7 files covered. (72.5%)

56 existing lines in 5 files now uncovered.

9407 of 10275 relevant lines covered (91.55%)

5.16 hits per line