Revert "feat: Enable react-native-nitro-fetch (V3)"

[yuwenmemon]

Reverts #95518

Fixed Issues

$ #96917
$ #96933

[melvin-bot]

@inimaga Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1c13142b4a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[Julesssss]

Builds expected to fail due to mobile-expensify counterpart

[melvin-bot]

@Julesssss looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

[Julesssss]

@Julesssss looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

#97000 (comment)

[OSBotify]

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

[github-actions]

🚧 Julesssss has triggered a test Expensify/App build. You can view the workflow run here.

[OSBotify]

🚀 Cherry-picked to staging by https://github.com/yuwenmemon in version: 9.4.44-5 🚀

platform	result
🕸 web 🕸	success ✅
🤖 android 🤖	success ✅
🍎 iOS 🍎	success ✅

[github-actions]

🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
Built from App PR #97000.

Android 🤖	iOS 🍎
https://ad-hoc-expensify-cash.s3.us-east-1.amazonaws.com/rock-artifacts/ad-hoc/rock-android-Adhoc-055f6b063b5-4ecaa81-f9f4b5d0a6631acff79fe2ecede055cfec841094/index.html	https://ad-hoc-expensify-cash.s3.us-east-1.amazonaws.com/rock-artifacts/ad-hoc/rock-ios-device-AdHoc-055f6b0-4ecaa81-d9cae776916acb6ec042f742352d28e31cabb36c/index.html

Web 🕸️
https://97000.pr-testing.expensify.com

---

👀 View the workflow run that generated this build 👀

[MelvinBot]

🤖 No help site changes required.

I reviewed the changes in this PR. It's a straight revert of #95518 ("Enable react-native-nitro-fetch (V3)"), and every change is internal networking/infrastructure plumbing with no user-facing behavior:

• react-native-nitro-fetch polyfills, patches, and package/lockfile entries
• Certificate pinning (CertificatePinning.kt, pins.json, generateCertificatePins.sh)
• Prefetch queries (src/libs/Prefetch/*)
• Network/request internals (HttpUtils.ts, enhanceParameters.ts, Request.ts, Reauthentication.ts)
• Session auth-token handling (SessionUtils.ts, updateSessionAuthTokens.ts, actions/Session/*)
• Generated Nitro module bindings and associated tests

None of this maps to any customer-facing feature, setting, tab, or button documented under docs/articles, so no help-site article needs to be created or updated. No draft PR was opened.

@yuwenmemon, please review the linked help site PR and confirm it reflects the current behavior. Then mark the linked help site PR Ready for review

[OSBotify]

🚀 Deployed to production by https://github.com/marcaaron in version: 9.4.44-6 🚀

platform	result
🕸 web 🕸	success ✅
🤖 android 🤖	success ✅
🍎 iOS 🍎	success ✅

Bundle Size Analysis (Sentry):

• Android

[OSBotify]

🚀 Cherry-picked to staging by https://github.com/yuwenmemon in version: 9.4.45-0 🚀

platform	result
🕸 web 🕸	success ✅
🤖 android 🤖	success ✅
🍎 iOS 🍎	success ✅

[MelvinBot]

Help site review — no changes required

I reviewed the changes in this PR against the help site content under docs/articles.

Conclusion: No help site updates are required, and no draft PR was created.

This PR is a pure revert of #95518 ("Enable react-native-nitro-fetch (V3)"). Every change is internal networking/build plumbing with no user-facing surface:

• Removes the react-native-nitro-fetch library and its native (Cronet) integration, reverting to the shared OkHttp / fetch() networking path
• Certificate-pinning comments/config reverted to match (native only — pins.json, CertificatePinning.kt, MainApplication.kt)
• Removes the Prefetch/* request modules and related session/reauthentication token-refresh handling
• Reverts generated Nitro module code, Podfile.lock, package.json/package-lock.json, patches, and Jest/unit test setup

None of this changes any feature, tab, setting, button label, or documented workflow. There are no src/languages/* copy changes and nothing that any article under docs/articles describes. Accordingly, there is nothing to document and no naming/authoring conventions to apply.

If you believe a specific user-facing behavior did change as a result of this revert (e.g. something previously documented that no longer applies), reply with the detail and I'll create the docs PR.

---

@yuwenmemon, since no help site changes were required, there is no linked help site PR to review or mark Ready for review. Let me know if you'd like me to file one anyway for a specific behavior change.

[OSBotify]

🚀 Deployed to production by https://github.com/marcaaron in version: 9.4.45-14 🚀

platform	result
🕸 web 🕸	success ✅
🤖 android 🤖	success ✅
🍎 iOS 🍎	success ✅

Bundle Size Analysis (Sentry):

• Android

[OSBotify]

🚀 Cherry-picked to staging by https://github.com/yuwenmemon in version: 9.4.46-0 🚀

platform	result
🕸 web 🕸	success ✅
🤖 android 🤖	success ✅
🍎 iOS 🍎	success ✅

[MelvinBot]

No help site changes required

I reviewed the changes in this PR against the help site articles under App/docs/articles and no documentation updates are needed.

This PR reverts #95518 — re-enabling react-native-nitro-fetch (V3). Every change is internal / native-layer plumbing with no user-facing behavior change:

• Removes the react-native-nitro-fetch NitroFetch/Cronet networking path and its AutoPrefetcher boot entrypoint (index.js, MainApplication.kt)
• Reverts certificate-pinning code comments to reflect that fetch() again routes through the shared OkHttp client (CertificatePinning.kt, pins.json)
• Regenerated native module boilerplate (ExpensifyNitroUtils nitrogen output) and the NitroFetch/NitroModules pod entries in ios/Podfile.lock
• Removes the react-native-nitro-fetch Jest mock (jest/setup.ts)

The Expensify help site documents customer-facing product behavior (workspaces, billing, integrations, features), and none of that is affected by which native networking library backs fetch() under the hood. No draft PR was created since there is nothing to document.

@yuwenmemon, please confirm you agree no help site changes are required. If you believe something here does warrant a docs update, let me know what behavior changed and I'll open a draft PR.

[OSBotify]

🚀 Deployed to production by https://github.com/marcaaron in version: 9.4.46-10 🚀

platform	result
🕸 web 🕸	success ✅
🤖 android 🤖	success ✅
🍎 iOS 🍎	success ✅

Bundle Size Analysis (Sentry):

• Android