Let `spm deinit` take an app back to CocoaPods by chrfalch · Pull Request #59368 · react/react-native

chrfalch · 2026-10-22T16:26:57Z

Summary:

SwiftPM is planned to become the default iOS setup, with CocoaPods as the opt-out. An app that wants CocoaPods then runs npx react-native spm deinit. This PR does not change the templates or the CLI. It makes spm deinit able to take an app fully back to CocoaPods, which the plan needs.

Today an app that was converted to SwiftPM cannot fully go back to CocoaPods:

The conversion (spm add --deintegrate, also run by plain npx react-native spm on a new app) deletes each Podfile line that names a React Native call. A call that spans several lines, such as use_react_native!(…), loses only its first line. The Podfile is then invalid Ruby.
spm deinit removes SwiftPM from the Xcode project, but it does not touch the Podfile.

This PR keeps the Podfile unchanged, so npx react-native spm deinit and then pod install go back to CocoaPods:

The conversion keeps the Podfile. spm add --deintegrate runs pod deintegrate and does not change the Podfile. The line editing is removed.
Plain npx react-native spm no longer converts a CocoaPods app. It stops before it changes anything and points to spm add --deintegrate.
pod install stops on an app that uses SwiftPM. The React Native CocoaPods scripts check for the SwiftPM marker file (.spm-injected.json) in the .xcodeproj next to the Podfile. If it is there, pod install stops with an error before it writes any file. Without this check, pod install would add CocoaPods back to the Xcode project and rewrite the generated Package.swift with a wrong path, and Xcode would fail to resolve packages.
The uncommitted-changes prompt comes first. If the .xcodeproj has uncommitted changes, the conversion asks before pod deintegrate. If the answer is no, nothing changes.
The conversion warns about other pods. If the Podfile declares pods that are not part of React Native, it lists them and continues. Pods that come from the react-native package (:path or :podspec into node_modules/react-native) are not listed. The Podfile is kept, so spm deinit and then pod install bring those pods back.
spm deinit undoes the .gitignore lines the conversion wrote. The conversion writes .gitignore only after a successful injection and records the lines in the marker file, so deinit removes only them.
spm deinit removes the ReactNativeSPM folders that are empty after it deletes its files, and prints "To go back to CocoaPods, run pod install."

The three SwiftPM iOS workflows checked that the Podfile has no use_react_native! after the conversion. The Podfile is now kept, so they check that the .xcodeproj has no Pods- references instead.

The docs in scripts/spm/__docs__ describe the way back and the pod install error. They also say that the conversion removes the app target's own REACT_NATIVE_PATH, and deinit does not restore it. pod install sets it again through react_native_post_install. Only a Podfile without that call needs a manual check.

Changelog:

[IOS] [CHANGED] - react-native spm add --deintegrate keeps the Podfile, plain react-native spm no longer converts a CocoaPods app, and pod install stops with an error on an app that uses SwiftPM. react-native spm deinit and then pod install take an app back to CocoaPods.

Test Plan:

Unit tests:

yarn jest packages/react-native/scripts/spm/__tests__

25 suites, 1233 tests pass. ESLint and Prettier are clean on the changed files, and ruby -c passes on the three changed Ruby files.

The Ruby scripts have no unit tests in this repo. I checked the pod install error on a real app instead (step 3 below).

A new app outside the monorepo, on React Native 0.89.0-nightly-20261021-c7429b0e9 with @react-native-community/cli 21.0.0-alpha.1. The changed scripts were copied into node_modules/react-native/scripts.

npx react-native spm add --deintegrate --yes converts the app. The Podfile is unchanged.
The SwiftPM build succeeds.
pod install stops: "This app uses Swift Package Manager for React Native (SpmOptOut.xcodeproj has .spm-injected.json), so pod install would break the Xcode project." No Pods/ folder is created. Podfile.lock, Package.swift and the git status of ios/ are unchanged.
The SwiftPM build succeeds again.
npx react-native run-ios and npx react-native build-ios build and run the app. The CLI sees the marker file and does not run pod install.
npx react-native spm deinit removes the marker. The Podfile is unchanged.
cd ios && pod install succeeds, and the CocoaPods build succeeds.

Not checked:

The changed workflow checks have not run in CI yet.
Flow. My local Flow binary is older than the version .flowconfig requires, so it did not type-check.
run-ios --force-pods on a converted app. In the CLI source, this flag runs pod install anyway, which then stops with the error from step 3.
CLI 20.
After the conversion, the old .xcworkspace stays (as before this PR). The CLI builds it, and the build succeeds.

github-actions · 2026-10-26T15:11:41Z

Visual regression

1 visual difference(s) at dda80a2cf86c. Visual differences need maintainer approval. Review the report, then apply visual-change-approved.

ios `view/z-index/reordered`: Changed, 15396 px (2.160%)

Expected and Captured

Expected	Captured

Converting an app to SwiftPM with `--deintegrate` now moves the Podfile to Podfile.bak instead of deleting React Native lines from it (that edit left invalid Ruby for multi-line calls). `spm deinit` restores the Podfile, undoes the .gitignore lines the conversion wrote, removes empty ReactNativeSPM folders, and says to run `pod install`. The conversion refuses, before changing anything, when the Podfile has pods that are not React Native or when Podfile.bak already exists.

The Podfile is moved to Podfile.bak, so no pod is lost and `spm deinit` restores it. An explicit `--deintegrate` now lists the other pods, says CocoaPods will not install them while the app uses SwiftPM unless SwiftPM provides them, and continues. RNTester needs this: its Podfile has local example pods that SwiftPM provides. The automatic conversion of a new app still skips such Podfiles.

cipolleschi

Thanks for this!
I'd changes some bits, to simplify the feature... See my comments!

cipolleschi · 2026-10-29T16:19:42Z

 ```

+On a fresh CocoaPods app, this converts the app to SwiftPM. It runs
+`pod deintegrate` and moves the `Podfile` to `Podfile.bak`. A Podfile with pods


Podfile can remain Podfile. Why do we need to change it?

Done. The Podfile now stays as it is.

cipolleschi · 2026-10-29T16:20:17Z

+`pod deintegrate` and moves the `Podfile` to `Podfile.bak`. A Podfile with pods
+that are not part of React Native is not converted automatically;
+`spm add --deintegrate` converts it and warns that those pods are not installed.
+Without a Podfile, the CLI needs `@react-native-community/cli` 21 or newer to


if we keep it, we can lift this requirement

Done. The CLI 21 requirement is gone, because the Podfile stays in place.

cipolleschi · 2026-10-29T16:20:52Z

+2. moves the `Podfile` to `Podfile.bak`, unchanged. `spm deinit` moves it back.
+   If `Podfile.bak` already exists, the command stops before it changes
+   anything.


I would not do this step.

Done. The step is removed.

cipolleschi · 2026-10-29T16:23:08Z

-Non-RN pods can stay side-by-side. Your Podfile still lists them. Remove the
-leftover React Native lines (see step 2 above), then re-integrate the pods with
-a normal install:
+`--deintegrate` converts an app whose Podfile has pods that are not part of


it might be confusing that we have two different behaviors for spm --deintegrate and spm.
I understand that we would like to remove a step (manually calling --deintegrate) but perhaps it is not a bad thing that the user has to call it themselves.

Done. Plain spm no longer runs --deintegrate. On a CocoaPods app it stops and points to spm add --deintegrate.

cipolleschi · 2026-10-29T16:25:12Z

+[local native module](#local-native-modules). The pods stay in `Podfile.bak`,
+and `spm deinit` restores the Podfile.
+
+To keep a CocoaPods-only pod next to SwiftPM, convert by hand:


are there Cocoapods-only pods? I believe that most Native iOS libraries nowadays supports Swift PM. For React Native packages, we should strive to auto-convert them or to fail. If the users have a pod that still needs to be converted to SwiftPM and that can't be converted automatically, I would probably fail and ask them to open a PR/Issue to convert the library.

I kept this a warning, as in my top-level comment (point 4). The Podfile now stays unchanged, so spm deinit and then pod install bring those pods back. An error would also block RNTester: its Podfile has local pods that SwiftPM provides through spmModules.

cipolleschi · 2026-10-29T16:31:05Z

+  return true;
+}
+
+function restorePodfile(appRoot /*: string */) /*: void */ {


The move isn't recorded in the marker (unlike the .gitignore lines), so deinit renames any Podfile.bak to Podfile, including one the user made themselves. This goes away if we keep the Podfile in place.

Gone. The Podfile.bak code is removed.

cipolleschi · 2026-10-29T16:31:07Z

    return false;
  }
-  if (podfileHasThirdPartyPods(appRoot)) {
+  if (podfileHasThirdPartyPods(appRoot) || hasPodfileBackup(appRoot)) {


When Podfile.bak exists, this skips without saying why. The fail-fast check then suggests spm add --deintegrate, which refuses for the same reason. Could we log the reason here?

If we get rid of the Podfile.bak, we can simplify the code significantly

Gone. The Podfile.bak code is removed.

cipolleschi · 2026-10-29T16:31:08Z

-function podfileHasThirdPartyPods(appRoot /*: string */) /*: boolean */ {
+// The names of the pods the Podfile declares explicitly (`pod '...'` or
+// `pod('...')`), deduplicated, in order. React Native itself comes in through
+// use_react_native!, so every name here is a pod that is not part of it.


Older or brownfield Podfiles can list RN pods explicitly (e.g. pod 'React-Core', :path => '../node_modules/react-native/'), so this would report them as non-RN. Pods from test targets also end up in the list.

Done. Pods with :path or :podspec into node_modules/react-native now count as React Native pods. Pods in test targets stay in the list: to find them, we would need to parse the Ruby target blocks.

cipolleschi · 2026-10-29T16:31:10Z

+    const podNames = podfilePodNames(appRoot);
+    if (podNames.length > 0) {
+      logError(
+        'Your Podfile declares pods that are not part of React Native: ' +


Nit: this is a warning, but it goes through logError and the run continues. If we decide to fail here instead (see my comment in the docs), this is the place to do it.

Done. It is now printed as a warning.

cipolleschi · 2026-10-29T16:31:11Z

 function runDeintegrate(appRoot /*: string */) /*: void */ {
+  const podfilePath = path.join(appRoot, 'Podfile');
+  const backupPath = path.join(appRoot, 'Podfile.bak');
+  if (refuseToOverwritePodfileBackup(appRoot)) {


Nit: main already runs this check before we get here, so this one never fires on that path. Is it only meant for direct callers?

Gone. This check was part of the Podfile.bak code.

chrfalch · 2026-10-29T17:01:18Z

Thanks for the review, Riccardo! I agree with most of it. My plan:

Keep the Podfile in place. --deintegrate runs pod deintegrate and leaves the Podfile unchanged. All the Podfile.bak code goes, and with it the bugs you found around it and the need for CLI 21. The way back is then spm deinit followed by pod install. To catch a pod install run by mistake, the next spm run or Xcode sync stops with a clear error when CocoaPods is back in the project.
Plain npx react-native spm no longer converts a CocoaPods app. The user, or the CLI, runs spm add --deintegrate explicitly.
Order of steps: the uncommitted-changes prompt moves before any change, and .gitignore is written only after a successful injection.
Other pods: I would like to keep this a warning, not an error. The Podfile is now kept unchanged, so nothing is lost. An error would also block RNTester: its Podfile has local pods that SwiftPM provides through spmModules. I will fix the two problems you found: React Native pods that are listed by name now count as React Native pods, and the message is printed as a warning.
deinit → remove: I would leave the name as it is in this PR. deinit already exists, and other tools call these scripts directly. Happy to do the rename in a separate PR if you want it.

I will push these as new commits.

Review feedback: `spm add --deintegrate` no longer moves the Podfile to Podfile.bak. It runs `pod deintegrate` and keeps the Podfile unchanged, so `spm deinit` and then `pod install` go back to CocoaPods. - Plain `spm` no longer converts a CocoaPods app; it points to `spm add --deintegrate`. - The React Native CocoaPods scripts stop `pod install` before any write when the .xcodeproj next to the Podfile has .spm-injected.json. Otherwise `pod install` rewrites the generated Package.swift with a wrong path and Xcode fails to resolve packages before any build phase runs. - The uncommitted-changes prompt comes before `pod deintegrate`. - .gitignore is written only after a successful injection. - Pods sourced from node_modules/react-native count as React Native pods, and the other-pods message is a warning. - The SwiftPM iOS workflows check for `Pods-` references in the pbxproj instead of the Podfile content.

chrfalch · 2026-10-30T08:05:18Z

Pushed the changes in 0640f08. One change from my earlier plan (point 1): a mistaken pod install is now stopped by pod install itself. The React Native CocoaPods scripts see the SwiftPM marker in the .xcodeproj and stop with an error before they write any file. A later check in spm or the Xcode sync phase was too late: pod install rewrites the generated Package.swift, and Xcode fails to resolve packages before any build phase runs. The test plan in the description has the details.

cipolleschi · 2026-10-30T14:19:29Z

+  for (const line of fs.readFileSync(podfilePath, 'utf8').split('\n')) {
+    // `#{` is Ruby string interpolation, not a comment. A line that ends in
+    // `,` continues on the next one.
+    const code = continued + line.split(/#(?!\{)/)[0].trimEnd();


Small edge case: continued ends with a space, so if a blank or comment-only line sits in the middle of a multi-line pod call, code.endsWith(',') is false and the statement ends before its :path. For example:

pod 'React-Core', # comment :path => '../node_modules/react-native/'

This gets reported as a non-RN pod. Trimming the whole code string should fix it.

Good catch, fixed in dae2c23. The whole joined line is trimmed now, so a blank or comment line inside a pod call no longer ends it. I added a test with both cases.

cipolleschi · 2026-10-30T14:19:29Z

+      );
+    }
+    if (fs.existsSync(path.join(appRoot, 'Podfile'))) {
+      log('To go back to CocoaPods, run `pod install`.');


Nit: this is printed even when deinit found nothing to remove. Also, when the command was redirected from the JS root, it might help to say where to run it (e.g. cd ios && pod install, or displayPath(appRoot)).

Fixed in dae2c23. The line is printed only when deinit removed the SwiftPM setup. When the command was run from the JS root, it says cd ios && pod install. Tests added for both.

- A multi-line `pod` call with a blank or comment line in the middle no longer ends early, so a pod sourced from node_modules/react-native is not reported as a non-React Native pod. - `spm deinit` says to run `pod install` only when it removed the SwiftPM setup, and says `cd ios && pod install` when run from the JS root.

meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 22, 2026

facebook-github-tools Bot added p: Expo Partner: Expo Partner labels Oct 22, 2026

chrfalch requested a review from cipolleschi October 22, 2026 16:31

facebook-github-tools Bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Oct 22, 2026

chrfalch added 2 commits October 28, 2026 15:06

chrfalch force-pushed the chrfalch/spm-cocoapods-opt-out branch from b75a2b8 to dda80a2 Compare October 28, 2026 14:06

cipolleschi reviewed Oct 29, 2026

View reviewed changes

cipolleschi requested changes Oct 29, 2026

View reviewed changes

chrfalch changed the title ~~Let spm deinit take an app back to CocoaPods~~ Keep the Podfile so spm deinit can take an app back to CocoaPods Oct 30, 2026

chrfalch changed the title ~~Keep the Podfile so spm deinit can take an app back to CocoaPods~~ Let spm deinit take an app back to CocoaPods Oct 30, 2026

cipolleschi reviewed Oct 30, 2026

View reviewed changes

Conversation

chrfalch commented Oct 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary:

Changelog:

Test Plan:

Uh oh!

github-actions Bot commented Oct 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Visual regression

ios view/z-index/reordered: Changed, 15396 px (2.160%)

Uh oh!

cipolleschi left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chrfalch commented Oct 29, 2026

Uh oh!

chrfalch commented Oct 30, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This branch has not been deployed

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

chrfalch commented Oct 22, 2026 •

edited

Loading

github-actions Bot commented Oct 26, 2026 •

edited

Loading

ios `view/z-index/reordered`: Changed, 15396 px (2.160%)