Rename Checkout Sheet Kit to Checkout Kit (React Native)#483
Open
Juanita-Dash wants to merge 1 commit intomainfrom
Open
Rename Checkout Sheet Kit to Checkout Kit (React Native)#483Juanita-Dash wants to merge 1 commit intomainfrom
Juanita-Dash wants to merge 1 commit intomainfrom
Conversation
3 tasks
Contributor
Author
fb2e013 to
b3d5d4d
Compare
This was referenced May 6, 2026
markmur
reviewed
May 6, 2026
| path: | | ||
| sample/ios/Pods | ||
| key: ${{ runner.os }}-cocoapods-${{ hashFiles('sample/ios/Podfile.lock', 'sample/Gemfile.lock', 'sample/Gemfile', 'package.json', 'sample/package.json', 'modules/@shopify/checkout-sheet-kit/package.json', 'pnpm-lock.yaml') }} | ||
| key: ${{ runner.os }}-cocoapods-${{ hashFiles('sample/ios/Podfile.lock', 'sample/Gemfile.lock', 'sample/Gemfile', 'package.json', 'sample/package.json', 'modules//checkout-kit/package.json', 'pnpm-lock.yaml') }} |
Contributor
There was a problem hiding this comment.
Suggested change
| key: ${{ runner.os }}-cocoapods-${{ hashFiles('sample/ios/Podfile.lock', 'sample/Gemfile.lock', 'sample/Gemfile', 'package.json', 'sample/package.json', 'modules//checkout-kit/package.json', 'pnpm-lock.yaml') }} | |
| key: ${{ runner.os }}-cocoapods-${{ hashFiles('sample/ios/Podfile.lock', 'sample/Gemfile.lock', 'sample/Gemfile', 'package.json', 'sample/package.json', 'modules/@shopify/checkout-kit/package.json', 'pnpm-lock.yaml') }} |
Contributor
Author
There was a problem hiding this comment.
Yea, saw this in a few places where the string replacement got messed up by claude and the /@shopify was dropped. Fixing as we speak
b3d5d4d to
3401bdf
Compare
c4032a1 to
98c974a
Compare
3401bdf to
0a48c04
Compare
0a48c04 to
abde1c1
Compare
98c974a to
d0cd665
Compare
abde1c1 to
912d71b
Compare
912d71b to
7fabb05
Compare
1215b82 to
b047936
Compare
b047936 to
7229059
Compare
Renames the React Native bridge surface (npm package, RN module name, TS API, source files, internal symbols) from "Checkout Sheet Kit" to "Checkout Kit". Upstream native dependencies (`ShopifyCheckoutSheetKit` iOS pod and `com.shopify:checkout-sheet-kit` Android Maven artifact) are intentionally left on their current names since those artifacts have not been republished under the new name yet — that's tracked as a follow-up. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
7229059 to
83e43ab
Compare
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
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.

Part of https://github.com/shop/issues-checkout-kit/issues/883
Important
Scope of this PR vs. follow-up work
This PR renames the React Native bridge (npm package, RN module, TS public API, source files, internal Java/Swift symbols). It intentionally does not rename the upstream native dependencies, because
com.shopify:checkout-kit(Android Maven) andShopifyCheckoutKit(iOS CocoaPod, 3.x) are not yet published — onlycom.shopify:checkout-sheet-kitandShopifyCheckoutSheetKitexist today, so renaming them here would break CI withCould not find/None of your spec sources contain a spec satisfying the dependencyerrors.A follow-up PR will rename:
ShopifyCheckoutSheetKit ~> 3.8.0→ShopifyCheckoutKitcom.shopify:checkout-sheet-kit→com.shopify:checkout-kitimport ShopifyCheckoutSheetKitand allShopifyCheckoutSheetKit.*framework calls (inmodules/@shopify/checkout-kit/ios/*.swiftandsample/ios/ReactNativeTests/*.swift)sample/ios/ReactNative.xcodeproj/project.pbxproj(libShopifyCheckoutSheetKit.a,${PODS_CONFIGURATION_BUILD_DIR}/ShopifyCheckoutSheetKit,$(SRCROOT)/Pods/ShopifyCheckoutSheetKit/**, and the correspondingSWIFT_INCLUDE_PATHSentries) — kept aligned with the upstream pod's actual product name. The bridge pod paths (RNShopifyCheckoutKit) and renamed test file (ShopifyCheckoutKitTests.swift) in the same pbxproj are renamed in this PR....once those upstream artifacts are published under their new names.
Important
GitHub URLs and deep-links
All
https://github.com/Shopify/checkout-kit/...references in this PR (package.jsonhomepage/repository, podspecs.source, README badges, CONTRIBUTING/PR template links, etc.) currently point at a repo that doesn't exist publicly yet — they will 404 until the cross-platform monorepo migration completes (interim host:shopify-playground/checkout-kit; final:Shopify/checkout-kitonce made public).The npm
repository.directoryfield inmodules/@shopify/checkout-kit/package.json(react-native/modules/@shopify/checkout-kit) is anticipatory in the same way — it assumes the future monorepo layout wherereact-native/is a sibling ofswift/andandroid/. npm's "View source" deep-link on the package page will resolve correctly once the monorepo lands.This is the same anticipatory pattern used by the already-merged Swift and Android rename PRs (swift, android) — keeping the three rename PRs consistent.
A separate follow-up PR (cross-platform, coordinated with the monorepo restructure into
swift/,android/,react-native/subdirs) will fix the deep-links (/blob/main/...,/actions/...) once the file paths in the new monorepo layout are settled.What changes are you making?
Renames the React Native bridge from Shopify Checkout Sheet Kit to Shopify Checkout Kit.
Package and bridge (renamed in this PR)
@shopify/checkout-sheet-kit→@shopify/checkout-kitRNShopifyCheckoutSheetKit→RNShopifyCheckoutKitcom.shopify.reactnative.checkoutsheetkit→com.shopify.reactnative.checkoutkitRNShopifyCheckoutSheetKitSpec→RNShopifyCheckoutKitSpecTypeScript public API (renamed in this PR — drops "Sheet" everywhere)
ShopifyCheckoutSheet→ShopifyCheckoutShopifyCheckoutSheetProvider→ShopifyCheckoutProvideruseShopifyCheckoutSheet→useShopifyCheckoutShopifyCheckoutSheetContext→ShopifyCheckoutContextSource files and directories (renamed in this PR)
modules/@shopify/checkout-sheet-kit/→modules/@shopify/checkout-kit/ShopifyCheckoutSheetKit*.swift→ShopifyCheckoutKit*.swiftShopifyCheckoutSheetKitModule.java/Package.java→ShopifyCheckoutKitModule.java/Package.javaNativeShopifyCheckoutSheetKit.ts→NativeShopifyCheckoutKit.tsUpstream native deps (deferred to a follow-up PR — see callout above)
ShopifyCheckoutSheetKit ~> 3.8.0→ unchanged (would-beShopifyCheckoutKit)com.shopify:checkout-sheet-kit→ unchanged (would-becom.shopify:checkout-kit)Not changed (cross-team web checkout contracts)
ShopifyCheckoutSDK/<version>user-agent — parsed by web checkoutwindow.MobileCheckoutSdkJS bridge global — defined by web checkoutcheckout-sdk.myshopify.comtest fixture URLs — not a branding referenceHow to test
Lints + tests (all passing locally):
pnpm module lint pnpm sample lint pnpm test ./scripts/lint_swiftiOS sample:
pnpm pod-install && pnpm iosAndroid sample:
pnpm androidConsumer migration check — apps importing the old library will need to:
@shopify/checkout-sheet-kit→@shopify/checkout-kitShopifyCheckoutSheet→ShopifyCheckout,useShopifyCheckoutSheet→useShopifyCheckout,ShopifyCheckoutSheetProvider→ShopifyCheckoutProvider