iOS Warning: RCTBridge required dispatch_sync to load REAModule #3678

dimaischenko · 2022-10-15T08:35:15Z

Description

When the application starts, I get the following warning: RCTBridge required dispatch_sync to load REAModule. This may lead to deadlocks

Steps to reproduce

Every time the application is launched

Snack or a link to a repository

Sorry

Reanimated version

2.11.0

React Native version

0.70.3

Platforms

iOS

JavaScript runtime

No response

Workflow

React Native (without Expo)

Architecture

No response

Build type

No response

Device

No response

Device model

No response

Acknowledgements

Yes

The text was updated successfully, but these errors were encountered:

github-actions · 2022-10-15T08:35:29Z

Hey! 👋

The issue doesn't seem to contain a minimal reproduction.

Could you provide a snack or a link to a GitHub repository under your username that reproduces the problem?

dpyeates · 2022-10-15T11:00:47Z

I see am seeing this as well now.

godlikeAlex · 2022-10-15T12:04:08Z

+1

enahum · 2022-10-15T12:46:13Z

same

dimaischenko · 2022-10-15T15:57:17Z

Thank you, so the downgrade will help, but maybe someone will know what a patch or fixes for the current release

aymather · 2022-10-16T21:22:06Z

I can also confirm that moving from 2.10.0 -> 2.11.0 introduced this issue.

AmeyaDalvi · 2022-10-17T14:53:46Z

Facing the same issue, will a downgrade work? I'm assuming it will but it wont fix the issue

tomekzaw · 2022-10-17T19:27:17Z

Hey, we are investigating this issue and working on a fix.

Feel free to use 2.10.0 or call LogBox.ignoreLogs to ignore this warning.

Max-DVS · 2022-10-18T02:34:27Z

Hello,

I stumbled upon the same issue today using 2.11.0

Just start a fresh project and follow react navigation documentation to add a drawer...

The only missing piece on their docs is to add react-native-reanimated/plugin into babel...

did a pod install and the only thing I got was that error, a regular stack works while a drawer will throw that error.

The fix was to get back to 2.10.0

hadnet · 2022-10-19T03:36:58Z

Same here using Reanimated 3-RC3

alqamabinsadiq · 2022-10-19T18:46:46Z

Downgrading to 2.10.0 worked for me too.

tomekzaw · 2022-10-20T08:07:34Z

The offending PR is #3555. Fixed in #3698.

If you encounter warning "RCTBridge required dispatch_sync to load REAModule", you have four options:

Simply ignore the warning (it only appears in debug mode, so you won't see this in production build of your app).
Add the following code in index.js of your app:

import { LogBox } from 'react-native';

LogBox.ignoreLogs(['RCTBridge required dispatch_sync to load REAModule']);

Remove the following lines in node_modules/react-native-reanimated/ios/REAModule.mm (e.g. using patch-package):

- + (BOOL)requiresMainQueueSetup
- {
-   return YES;
- }

Downgrade to react-native-reanimated@2.10.0 and upgrade to 2.12.0 once this issue is fixed.

dimaischenko · 2022-10-20T09:31:31Z

@tomekzaw Thank you for such a quick response, work and clarification!

…3698) ## Description Fixes #3678. Reverts part of #3555 for Paper. On Paper, the warning "RCTBridge required dispatch_sync to load REAModule" was gone when I removed `requiresMainQueueSetup` method introduced in #3678. However, the PR description explicitly states that this method was added in order to eliminate the following warning: > This is because the `init` method was not present before and after adding it, it caused this warning: > > `Module RCTImagePickerManager requires main queue setup since it overrides init but doesn't implement 'requiresMainQueueSetup'. In a future release React Native will default to initializing all native modules on a background thread unless explicitly opted-out of.` However, after the removal of `requiresMainQueueSetup` no warning appears, so I assume that it is in fact not necessary to initialize REAModule on main queue, thus it's safe to remove `requiresMainQueueSetup`. On Fabric, removing `requiresMainQueueSetup` it crashes the app on reload, so I've left it as it is for now. We may address this inconsistency in a separate PR.

skam22 · 2022-10-21T18:56:27Z

not to nitpick but the method is LogBox.ignoreLogs, not LogBox.ignore:

import { LogBox } from 'react-native';

LogBox.ignoreLogs(['RCTBridge required dispatch_sync to load REAModule']);

tomekzaw · 2022-10-28T06:49:36Z

This issue has been fixed in #3698 and released in 2.12.0 so upgrade and you will never see this warning again.

…oftware-mansion#3698) ## Description Fixes software-mansion#3678. Reverts part of software-mansion#3555 for Paper. On Paper, the warning "RCTBridge required dispatch_sync to load REAModule" was gone when I removed `requiresMainQueueSetup` method introduced in software-mansion#3678. However, the PR description explicitly states that this method was added in order to eliminate the following warning: > This is because the `init` method was not present before and after adding it, it caused this warning: > > `Module RCTImagePickerManager requires main queue setup since it overrides init but doesn't implement 'requiresMainQueueSetup'. In a future release React Native will default to initializing all native modules on a background thread unless explicitly opted-out of.` However, after the removal of `requiresMainQueueSetup` no warning appears, so I assume that it is in fact not necessary to initialize REAModule on main queue, thus it's safe to remove `requiresMainQueueSetup`. On Fabric, removing `requiresMainQueueSetup` it crashes the app on reload, so I've left it as it is for now. We may address this inconsistency in a separate PR.

derwaldgeist · 2024-01-27T19:19:32Z

I'm seeing this in a brand-new installation of RN (macOS Sonoma, iOS 17). Shouldn't this be fixed by now?

tomekzaw · 2024-01-27T19:30:13Z

@derwaldgeist Yes, this should be fixed. Which version of Reanimated exactly are you using?

derwaldgeist · 2024-01-27T22:23:05Z

@derwaldgeist Yes, this should be fixed. Which version of Reanimated exactly are you using?

Oh, sorry. I just realized that this repo is related to a certain library. I just came here via Google and thought it was on the react-native repo itself. I just tried out RN, to check how far I can get on a Vision Pro (but the issue also occurred on iOS).

tuantvk · 2024-01-30T02:32:24Z

@tomekzaw I got the same warning in the new version.

"react-native": "0.73.3",
"react-native-reanimated": "^3.6.2",

I think maybe it is related to the new version react-native.

  // This dispatch_async avoids a deadlock while configuring native modules
  dispatch_async(dispatch_get_global_queue(QOS_CLASS_USER_INTERACTIVE, 0), ^{
    id a11yManager = [self->_bridge moduleForName:@"AccessibilityManager"
                                             lazilyLoadIfNecessary:YES];
    [[NSNotificationCenter defaultCenter] addObserver:self
                                             selector:@selector(didReceiveNewContentSizeMultiplier)
                                                 name:@"RCTAccessibilityManagerDidUpdateMultiplierNotification"
                                               object:a11yManager];

packages/react-native/React/Modules/RCTUIManager.m

correafederico25 · 2024-01-30T07:42:27Z

@tuantvk I also have the same warning in version 0.73.3

A few days ago I built an app with version 0.73.2 and I didn't have any problems. 1 hour ago I built a new app with version 0.73.3 and started seeing this warning.

Do you think this warning could cause problems later? Or we could just ignore it.

{
  "name": "AwesomeProject2",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "android": "react-native run-android",
    "ios": "react-native run-ios",
    "lint": "eslint .",
    "start": "react-native start",
    "test": "jest"
  },
  "dependencies": {
    "react": "18.2.0",
    "react-native": "0.73.3"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@babel/preset-env": "^7.20.0",
    "@babel/runtime": "^7.20.0",
    "@react-native/babel-preset": "0.73.20",
    "@react-native/eslint-config": "0.73.2",
    "@react-native/metro-config": "0.73.4",
    "@react-native/typescript-config": "0.73.1",
    "@types/react": "^18.2.6",
    "@types/react-test-renderer": "^18.0.0",
    "babel-jest": "^29.6.3",
    "eslint": "^8.19.0",
    "jest": "^29.6.3",
    "prettier": "2.8.8",
    "react-test-renderer": "18.2.0",
    "typescript": "5.0.4"
  },
  "engines": {
    "node": ">=18"
  }
}

Milutin-P · 2024-02-01T12:23:08Z

@correafederico25 Are you sure that it is the same message? I got RCTBridge required dispatch_sync to load RCTAccessibilityManager. This may lead to deadlocks. Which is not related to RNA, I would say.

uen · 2024-02-01T14:17:59Z

Check facebook/react-native#42728 - it is fixed in 0.73.4, and not an issue with reanimated, but this is the first thing that comes up on google if you search 'RCTBridge required dispatch_sync' and leave out the RCTAccessibilityManager part

dimaischenko added the Needs review Issue is ready to be reviewed by a maintainer label Oct 15, 2022

github-actions bot added the Missing repro This issue need minimum repro scenario label Oct 15, 2022

github-actions bot added the Platform: iOS This issue is specific to iOS label Oct 15, 2022

This comment was marked as outdated.

Sign in to view

tomekzaw mentioned this issue Oct 20, 2022

Fix iOS Warning: RCTBridge required dispatch_sync to load REAModule #3698

Merged

tomekzaw pinned this issue Oct 20, 2022

piaskowyk closed this as completed in #3698 Oct 20, 2022

tomekzaw unpinned this issue Oct 28, 2022

eVoloshchak mentioned this issue Dec 5, 2022

[HOLD for payment 2022-12-05] [$500] Bug: DEV - [iOS] dev console error on app launch reported by @aimane-chnaif Expensify/App#12152

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

iOS Warning: RCTBridge required dispatch_sync to load REAModule #3678

iOS Warning: RCTBridge required dispatch_sync to load REAModule #3678

dimaischenko commented Oct 15, 2022

github-actions bot commented Oct 15, 2022

dpyeates commented Oct 15, 2022

godlikeAlex commented Oct 15, 2022

enahum commented Oct 15, 2022

This comment was marked as outdated.

dimaischenko commented Oct 15, 2022

aymather commented Oct 16, 2022

AmeyaDalvi commented Oct 17, 2022

tomekzaw commented Oct 17, 2022 •

edited

Loading

Max-DVS commented Oct 18, 2022

hadnet commented Oct 19, 2022

alqamabinsadiq commented Oct 19, 2022

tomekzaw commented Oct 20, 2022 •

edited

Loading

dimaischenko commented Oct 20, 2022

skam22 commented Oct 21, 2022 •

edited by tomekzaw

Loading

tomekzaw commented Oct 28, 2022

derwaldgeist commented Jan 27, 2024

tomekzaw commented Jan 27, 2024

derwaldgeist commented Jan 27, 2024

tuantvk commented Jan 30, 2024

correafederico25 commented Jan 30, 2024 •

edited

Loading

Milutin-P commented Feb 1, 2024

uen commented Feb 1, 2024 •

edited

Loading

iOS Warning: RCTBridge required dispatch_sync to load REAModule #3678

iOS Warning: RCTBridge required dispatch_sync to load REAModule #3678

Comments

dimaischenko commented Oct 15, 2022

Description

Steps to reproduce

Snack or a link to a repository

Reanimated version

React Native version

Platforms

JavaScript runtime

Workflow

Architecture

Build type

Device

Device model

Acknowledgements

github-actions bot commented Oct 15, 2022

dpyeates commented Oct 15, 2022

godlikeAlex commented Oct 15, 2022

enahum commented Oct 15, 2022

This comment was marked as outdated.

dimaischenko commented Oct 15, 2022

aymather commented Oct 16, 2022

AmeyaDalvi commented Oct 17, 2022

tomekzaw commented Oct 17, 2022 • edited Loading

Max-DVS commented Oct 18, 2022

hadnet commented Oct 19, 2022

alqamabinsadiq commented Oct 19, 2022

tomekzaw commented Oct 20, 2022 • edited Loading

dimaischenko commented Oct 20, 2022

skam22 commented Oct 21, 2022 • edited by tomekzaw Loading

tomekzaw commented Oct 28, 2022

derwaldgeist commented Jan 27, 2024

tomekzaw commented Jan 27, 2024

derwaldgeist commented Jan 27, 2024

tuantvk commented Jan 30, 2024

correafederico25 commented Jan 30, 2024 • edited Loading

Milutin-P commented Feb 1, 2024

uen commented Feb 1, 2024 • edited Loading

tomekzaw commented Oct 17, 2022 •

edited

Loading

tomekzaw commented Oct 20, 2022 •

edited

Loading

skam22 commented Oct 21, 2022 •

edited by tomekzaw

Loading

correafederico25 commented Jan 30, 2024 •

edited

Loading

uen commented Feb 1, 2024 •

edited

Loading