Skip to content

4233: Upgrade dependencies - #4300

Open
bahaaTuffaha wants to merge 6 commits into
mainfrom
4233-upgrade-dependencies
Open

4233: Upgrade dependencies#4300
bahaaTuffaha wants to merge 6 commits into
mainfrom
4233-upgrade-dependencies

Conversation

@bahaaTuffaha

@bahaaTuffaha bahaaTuffaha commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Short Description

Upgrade all dependencies. If a dependency upgrade is a lot of work, open a separate issue and/or PR for it to avoid too big PRs.

Proposed Changes

  • Upgraded react-native to 0.86.0 and switched the @react-native/* packages and @react-native-community/cli* to their matching releases.
  • Dropped the react-native patch.
  • Re-added @babel/plugin-transform-export-namespace-from to the devDependencies and to the plugins in babel.config.js.
  • Raised engines.node to >= 24.3.0 and switched the CircleCI jobs to cimg/node:24.18.1, matching the node = '24' already declared in mise.toml.

Native:

  • Upgraded @gorhom/bottom-sheet, react-native-paper, react-native-reanimated, react-native-worklets, @sentry/react-native, @dr.pogodin/react-native-fs, @dr.pogodin/react-native-static-server, lodash, react and react-test-renderer to their latest releases.
  • Upgraded @react-native-community/cli, @react-native-community/cli-platform-android and @react-native-community/cli-platform-ios to their latest releases.
  • Upgraded react-native-reanimated to ^4.5.3 and react-native-worklets to ^0.11.3.
  • Downgraded react-native-gesture-handler to the 2.x line (^2.32.0), which officially supports react-native 0.86. v3 repoints GestureDetector and PanGesture at the new hook-based API and renames the v2 equivalents to Legacy*, which @gorhom/bottom-sheet is not written against.
  • For the 4248 issue I wrapped LoadingErrorHandler in Regions and the Wrapper scroll view in LanguageSelection with a KeyboardAvoidingView to solve the keyboard issue.
  • Passed keyboardVerticalOffset={insets.top + insets.bottom} to all three Search, Regions and LanguageSelection.
  • Removed automaticallyAdjustKeyboardInsets which caused extra padding for iOS's flatLists and KeyboardAvoidingView already solves the issue for both android and iOS.
  • Added statusBarHeight={0} to the menu at HeaderMenu because its not sticking to the top anymore.

Web:

  • Upgraded @sentry/react, @tanstack/react-query, i18next, vite, react, react-dom, @types/react, @types/react-dom and @testing-library/jest-dom to their latest releases.
  • Held react-router on the 7.x line (^7.18.2) rather than 8.3.0, v8 is pure ESM with no CommonJS build, so jest cannot load it without running the web project in ESM mode, which breaks every jest.mock() call in the suite.
  • Replaced the mapbox-gl-rtl-text 0.2.3 asm.js build with the 0.4.0 WebAssembly build and updated rtlPluginUrl in shared/src/constants/map.ts.

Shared:

  • Upgraded @turf/distance, jest-environment-jsdom, @testing-library/jest-dom and react to their latest releases.
  • Pinned sanitize-html to exactly 2.17.5, since the 2.17.6 patch moves its htmlparser2 dependency to the ESM-only 12.x, which jest cannot parse and metro cannot resolve.

Tools:

  • Upgraded @sentry/cli to its latest release.

Side Effects

N/A

Checklist

Testing

Resolved Issues

Fixes: #4233 , #4248


@lunars97 lunars97 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your PR 💚 Looks good to me 👍 Tested with real android device 16 and emulator 11 as well as in Chrome, also tested Aschaffenburg and Malte. Did you already test e2e-tests? 🤔
One issue on iOS: when I run pod install, there is this error. I am seeing it for the first time and not sure whether I need to follow the instruction on it 👀

Image

Comment thread web/package.json Outdated
@bahaaTuffaha

Copy link
Copy Markdown
Contributor Author

Thank you for your PR 💚 Looks good to me 👍 Tested with real android device 16 and emulator 11 as well as in Chrome, also tested Aschaffenburg and Malte. Did you already test e2e-tests? 🤔 One issue on iOS: when I run pod install, there is this error. I am seeing it for the first time and not sure whether I need to follow the instruction on it 👀

Image

I didnt get that error on my end...
try:
cd ios
rm -rf Pods build
pod install

@lunars97

lunars97 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Thank you for your PR 💚 Looks good to me 👍 Tested with real android device 16 and emulator 11 as well as in Chrome, also tested Aschaffenburg and Malte. Did you already test e2e-tests? 🤔 One issue on iOS: when I run pod install, there is this error. I am seeing it for the first time and not sure whether I need to follow the instruction on it 👀
Image

I didnt get that error on my end... try: cd ios rm -rf Pods build pod install

It worked 👍

@lunars97 lunars97 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in iOS 26.4 simulator and Safari. Everything works as before, so no issues are found. LGTM! 👍 Also e2e-tests running successfully ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade dependencies

3 participants