supabase / supabase-swift / 24670072301

81%

push

github

fix(storage): make signedURL optional in createSignedURLs response (#958)

When one of the requested paths does not exist, the Storage API returns
`signedURL: null` for that item. The Swift client was crashing with a
decoding error because the response struct had a non-optional `signedURL`.

The backend (`supabase/storage`) guarantees that exactly one of `signedURL`
or `error` is non-null per item — never both, never neither. This makes a
Swift enum the natural representation.

Changes:
- Added `SignedURLResult` enum with `.success(path:signedURL:)` and
  `.failure(path:error:)` cases, exposing only the relevant value per item
- `createSignedURLs` now returns `[SignedURLResult]` (new preferred overload)
- Old `createSignedURLs -> [URL]` overloads kept in Deprecated.swift with
  `@_disfavoredOverload` and a deprecation message; they delegate to the new
  method and `compactMap` out failures so existing callers continue to work
- Private response structs renamed to avoid shadowing public types
- Updated all tests to pattern-match on the enum cases
- Fixed SupabaseStorageTests mock to include the `path` field the API returns

`SignedURL` struct is unchanged — no breaking API changes.

Fixes: SDK-851

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

13 of 27 new or added lines in 2 files covered. (48.15%)

6416 of 7990 relevant lines covered (80.3%)

28.57 hits per line