Skip to content

[0.86] Guards around nativeProps usage to prevent race conditions #1345

Closed

Description

@byCedric

Target Branch

0.86

Link to commit or PR to be picked

react/react-native#52646

Description

As mentioned by https://x.com/alireza_hadjar/status/2077397528220471331

This has been a cause for crashes due to race conditions for a while, causing up to 23% of the crashes for this user.

It has been fixed a year ago, but is only now hitting >=0.87.0-rc.0. Because Expo SDK 57 uses RN 0.86, we'd like to cherry pick this too due to the impact

Activity

  1. added
    Type Pick RequestPick requests to include commits inside a React Native release
    on Jul 16, 2026
  2. github-actions commented on Jul 16, 2026

    @github-actions

    Thank you for opening a new React Native Pick Request.

    These are the criteria we follow when accepting or rejecting a code change into a React Native release branch (source).

    If your Pick Request does not satisfy the criteria below, please close it as it will not be considered.

    ✅ Which Pick Requests we accept
    1. ✅ Fixes for regressions to core APIs. Examples:
      • A core component is behaving differently between versions 0.X and 0.X-1
      • The TurboModule.getEnforcing function is throwing an unexpected assertion error.
    2. ✅ Fixes to bugs in core React Native. Examples:
      • Breakpoints in React Native DevTools not working correctly in the debugger
      • Fast Refresh not working as expected
      • Metro bundler not starting properly or ignoring the configuration file
      • Buttons that are not clickable
    3. ✅ Fixes to APIs used by third-party libraries and out-of-tree platforms. Examples:
      • An API used by react-native-macos is not behaving as expected or is regressing
    4. ✅ Bump of patch version of dependencies. Example:
      • Bump Gradle from 8.11.0 to 8.11.1
    5. ✅ Fixes for low/high severity security vulnerabilities. Example:
      • Bump Gradle from 8.11.0 to 8.12.0 if there is a security vulnerability in 8.11 and no 8.11.x version contains the same fix.
    6. ✅ Fixes and reverts of accidental breaking changes. Example:
      • Reverting or fix-forwarding a Kotlin migration of a file (e.g. ReactRootView.java) if it is causing a breaking change for the React Native ecosystem.
    7. ✅ Performance improvements. Examples:
      • Fixing unnecessary re-renders in FlatList.
    8. ✅ Any Pick Request if the release is still in RC0 (after RC1, the previously listed criteria apply)
    ❌ Which Pick Requests we don't accept
    1. ❌ Any Pick Request that introduces a breaking change after RC1. Example:
      • A bugfix that contains a refactoring, resulting in changes to the public API of a class
    2. ❌ Any Pick Request that introduces a new feature after RC1. Example:
      • A commit that introduces a new API or a specific feature for a component.
    3. ❌ Bump of major or minor version of dependencies. Examples:
      • Bump Gradle from 8.11.0 to 8.12.0
      • Bump Gradle from 8.11.0 to 9.0.0
    4. ❌ Bump of a dependency to a pre-release version. Example:
      • Bump Gradle from 8.11.0 to 8.11.1-beta.1
    5. ❌ Any Pick Request that introduces changes to the React Native testing infrastructure after RC1. Examples:
      • Changes to GitHub Actions workflows to optimize them, even if they are on main.
      • Changes to the logic used to publish packages to NPM or Maven Central
    6. ❌ Pick Request composed of multiple commits that have several merge conflicts.
      • If your Pick Request is too complex to apply to the release branch, it will be rejected.
    7. ❌ Any non-critical improvement that landed on main.
      • These will generally ship in the following version unless they fit one of the criteria above.
    8. ❌ Any Pick Request considered a "nice to have".
      • Even one-line changes carry a risk of destabilizing and further delaying the release.
    ℹ️ What makes for a good Pick Request

    In order for your Pick Request to be considered, please do the following:

    • ℹ️ Limit your Pick Request to one commit from main, or one PR against the release branch.
    • ℹ️ Include a clear explanation of why your Pick Request should be considered, ideally referencing one of the criteria above.
    • ℹ️ Target only one minor version of React Native (e.g. [0.76]). To have your Pick Request applied to more versions, please open separate Pick Requests.
  3. moved this from Inbox to Needs Pick in React Native 0.86 Releaseson Jul 23, 2026
  4. gabrieldonadel commented on Jul 23, 2026

    @gabrieldonadel

    @react-native-bot merge 923e7ddcaf4d2de45ca7dedafbb84c6b87dbffc4 0.86-stable

  5. react-native-bot commented on Jul 23, 2026

    @react-native-bot
    Collaborator

    ❌ I failed to process your request with this error:

    Failed to cherry-pick 923e7ddcaf4d2de45ca7dedafbb84c6b87dbffc4 on branch 0.86-stable with: 1 with 'Command failed: cd react-native; git cherry-pick 923e7ddcaf4d2de45ca7dedafbb84c6b87dbffc4'
    

    console output:

    On branch 0.86-stable
    You are currently cherry-picking commit 923e7ddcaf.
      (fix conflicts and run "git cherry-pick --continue")
      (use "git cherry-pick --skip" to skip this patch)
      (use "git cherry-pick --abort" to cancel the cherry-pick operation)
    
    Changes to be committed:
    	modified:   packages/react-native/ReactCommon/react/renderer/core/ShadowNode.cpp
    	modified:   packages/react-native/ReactCommon/react/renderer/core/ShadowNodeFamily.h
    	modified:   packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.cpp
    	modified:   packages/react-native/ReactCommon/react/renderer/uimanager/tests/FabricUIManagerTest.cpp
    	modified:   scripts/cxx-api/api-snapshots/ReactAndroidDebugCxx.api
    	modified:   scripts/cxx-api/api-snapshots/ReactAndroidReleaseCxx.api
    	modified:   scripts/cxx-api/api-snapshots/ReactAppleDebugCxx.api
    	modified:   scripts/cxx-api/api-snapshots/ReactAppleReleaseCxx.api
    	modified:   scripts/cxx-api/api-snapshots/ReactCommonDebugCxx.api
    	modified:   scripts/cxx-api/api-snapshots/ReactCommonReleaseCxx.api
    
    Unmerged paths:
      (use "git add/rm <file>..." as appropriate to mark resolution)
    	deleted by us:   scripts/cxx-api/api-snapshots/ReactAndroidNewarchCxx.api
    	deleted by us:   scripts/cxx-api/api-snapshots/ReactAppleNewarchCxx.api
    	deleted by us:   scripts/cxx-api/api-snapshots/ReactCommonNewarchCxx.api
    
    
    
  6. gabrieldonadel commented on Jul 23, 2026

    @gabrieldonadel

    Picked in 0b941bc51a740670470c05e08bca5898d1a6b78c

  7. moved this from Needs Pick to Done / Picked in React Native 0.86 Releaseson Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type Pick RequestPick requests to include commits inside a React Native release

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions