AC Storefront: GraphQL values Sendable

[kieran-osgood-shopify]

What changes are you making?

Important

When reviewing, consider these changes alongside the ones in #274.
#88 highlighted an issue with ObservableObjects + Swift Concurrency that makes it awkward from a library perspective.
To head off future issues and to encourage consistency in the API, I've added #274 as a recommended but not "essential" step in the Swift 6 migration

This PR makes the Storefront/GraphQL checkout path safe to use from Swift concurrency contexts by tightening Sendable boundaries around the Storefront client, cache, request/response values, and test doubles.

Warning

This intentionally includes a breaking Swift API change: ShopifyAcceleratedCheckouts.Customer is now a Sendable value type instead of an ObservableObject class.

Customer is checkout identity data, not independently-owned UI state. Modeling it as an immutable struct matches normal Swift/SwiftUI patterns: app/configuration state owns and publishes values, while async/network code receives safe value snapshots.

Consumers should continue to observe the configuration object and assign a new customer value when inputs change:

@StateObject private var configuration = ShopifyAcceleratedCheckouts.Configuration(
    storefrontDomain: EnvironmentVariables.storefrontDomain,
    storefrontAccessToken: EnvironmentVariables.storefrontAccessToken
)

configuration.customer = ShopifyAcceleratedCheckouts.Customer(
    email: email.isEmpty ? nil : email,
    phoneNumber: phone.isEmpty ? nil : phone
)

The sample app was updated to use this pattern in:

platforms/swift/Samples/ShopifyAcceleratedCheckoutsApp/ShopifyAcceleratedCheckoutsApp/ShopifyAcceleratedCheckoutsApp.swift

This removes the need for an internal CustomerIdentity wrapper and lets Storefront cart creation accept Customer? directly.

How to test

Validated locally:

• dev swift test
• dev swift lint
• dev swift api check
• dev swift build samples

---

Before you merge

Important

• I've added tests to support my implementation
• I have read and agree with the Contribution Guidelines
• I have read and agree with the Code of Conduct
• I've updated the relevant platform README (platforms/swift/README.md and/or platforms/android/README.md)

---

Releasing a new Swift version?

• I have bumped the version in platforms/swift/ShopifyCheckoutKit.podspec
• I have bumped the version in platforms/swift/Sources/ShopifyCheckoutKit/ShopifyCheckoutKit.swift
• I have updated platforms/swift/CHANGELOG.md
• I have updated the SwiftPM/CocoaPods version snippets in platforms/swift/README.md (major version only)

Releasing a new Android version?

• I have bumped the versionName in platforms/android/lib/build.gradle
• I have updated platforms/android/CHANGELOG.md
• I have updated the Gradle/Maven version snippets in platforms/android/README.md

Tip

See the Contributing documentation for the full release process per platform.

[kieran-osgood-shopify]

• Fix Swift 6 test target isolation #91 : 2 dependent PRs (#98 , #99 )
• Align PassKit protocol abstractions with Swift 6 #90
• Isolate wallet controllers to the main actor #89
• #88 Continuation: Make AcceleratedCheckouts.Configuration Sendable #274
• AC Storefront: GraphQL values Sendable #88 👈 (View in Graphite)
• Make generated JSON coding key Sendable #87
• Logger: NSLock guaranteed #86
• ShopifyCheckoutKit.Configuration: NSLock guaranteed #85
• UI: @MainActor isolated #84
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

React Native — Coverage Report

Lines	Statements	Branches	Functions
	91.66% (319/348)	87.86% (181/206)	100% (82/82)

[kieran-osgood-shopify]

the script appears to have been running from the wrong dir - which means the .swiftlint paths were resolving the wrong paths - I had tons of file changes as a result on generated files

[kieran-osgood-shopify]

Merge activity

• Jun 16, 10:54 AM UTC: @kieran-osgood-shopify merged this pull request with Graphite.