supabase / supabase-flutter / 29318155135

87%

Build Type

push

github

Committed by web-flow

Commit Message

fix(supabase_common): align platform stats casing with other SDKs (#1596)

## What

Normalizes the platform name reported in the `platform=` segment of the
`X-Client-Info` header so it uses the same casing across all Supabase
SDKs.

Previously the Flutter SDK sent the raw `Platform.operatingSystem`
value, which is always lowercase (`macos`, `ios`, `android`, `linux`,
`windows`). Swift reports properly cased names (`macOS`, `iOS`,
`Android`, `Linux`, `Windows`). This mismatch made it harder to
aggregate stats per platform independently of framework/language.

## Changes

- Added `normalizePlatformName` in `supabase_common` backed by a lookup
map that maps each `Platform.operatingSystem` value to the shared,
Swift-aligned casing. Unknown values pass through unchanged so any new
platform still shows up in stats.
- `conditionalPlatform` (the `dart:io` implementation) now runs
`Platform.operatingSystem` through `normalizePlatformName`. This is the
single source used by both the `supabase` (pure Dart) and
`supabase_flutter` packages, so both are covered.
- Updated tests accordingly.

Only the platforms `dart:io` can actually report are mapped. visionOS,
watchOS, and tvOS aren't Flutter targets, and the alignment discussion
concluded those three aren't valuable to track.

## Context

Follows the casing agreement in supabase-community/core-csharp#2
([discussion](https://github.com/supabase-community/core-csharp/pull/2#discussion_r3571537527)).

Coverage Stats

2 of 4 new or added lines in 2 files covered. (50.0%)

16 existing lines in 2 files now uncovered.

4775 of 5468 relevant lines covered (87.33%)

4.12 hits per line