chore(supabase_flutter): mark auth-url helpers @internal instead of parse-ignore#1510
Merged
Merged
Conversation
…arse-ignore The SDK symbol extractor now honours @internal (from package:meta), so the internal clear-auth-url helpers no longer need to be listed in .sdk-parse-ignore. Annotate removeAuthParametersFromUrl and the conditional clearAuthUrlParameters shims @internal and drop their entries, leaving the ignore file for paths that cannot carry the annotation.
grdsdev
approved these changes
Jul 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Replaces the three
.sdk-parse-ignoreentries for the clear-auth-url helpers with@internalannotations on the declarations, and removes those entries.Why
The SDK capability-matrix symbol extractor now honours
@internal(frompackage:meta), see supabase/sdk#54. Before that, the extractor treated every non-underscorelib/srcdeclaration as public API, so the internal auth-url helpers had to be excluded by path in.sdk-parse-ignore.@internalon the declaration is a better source of truth than a hand-maintained path list: it lives next to the code, travels with refactors, and documents intent directly.What changed
removeAuthParametersFromUrl(clear_auth_url_parameters.dart) marked@internal.clearAuthUrlParametersin both conditional-import shims (clear_auth_url_parameters_web.dart,clear_auth_url_parameters_stub.dart) marked@internal..sdk-parse-ignore; its header now points to@internalas the preferred mechanism and reserves the file for paths that cannot carry the annotation (for example generated sources).All three symbols are used only within
supabase_flutter, so@internalproduces no cross-package analyzer warnings. Verified against the updated extractor that the package no longer emits these symbols while its other 466 public symbols are unchanged.