Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[nightly/3.5.0] ReferenceError: Property '_IS_FABRIC' doesn't exist, js engine: hermes #5061

Closed
efstathiosntonas opened this issue Sep 11, 2023 · 11 comments · Fixed by #5065
Closed
Labels
Missing repro This issue need minimum repro scenario Needs review Issue is ready to be reviewed by a maintainer Platform: Android This issue is specific to Android Platform: iOS This issue is specific to iOS

Comments

@efstathiosntonas
Copy link
Contributor

efstathiosntonas commented Sep 11, 2023

Description

Hey folks, I know you're fine tuning Fabric just before releasing 3.5.0, just wanted to bring this to your attention.

I've deleted node_modules, cleared derived data, deintegrated pods, --reset-cache on metro but the issue persists.

Please note that I use "expo-modules-core": "1.7.0" and "expo": "^49.0.10" on a bare Paper react-native project.

Let me know if you need me to test anything.

Steps to reproduce

  1. Update to 3.5.0-nightly-20230910-073b64568
  2. This was happening on previous nightly versions too.

See attached screenshot:
simulator_screenshot_EA9424C7-ED53-4238-A1E9-79FA31DC0A93

Snack or a link to a repository

none

Reanimated version

3.5.0-nightly-20230910-073b64568

React Native version

0.72.4

Platforms

Android, iOS

JavaScript runtime

Hermes

Workflow

React Native (without Expo)

Architecture

Paper (Old Architecture)

Build type

Debug mode

Device

iOS simulator

Device model

No response

Acknowledgements

Yes

@efstathiosntonas efstathiosntonas added the Needs review Issue is ready to be reviewed by a maintainer label Sep 11, 2023
@github-actions github-actions bot added the Missing repro This issue need minimum repro scenario label Sep 11, 2023
@github-actions
Copy link

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?

@github-actions github-actions bot added Platform: Android This issue is specific to Android Platform: iOS This issue is specific to iOS labels Sep 11, 2023
@thefranfran
Copy link

Same here

@radko93
Copy link
Contributor

radko93 commented Sep 11, 2023

Facing the same issue on 3.5.0. Not an expo app but using Expo. Not a metro cache issue since it also crashes on a CI build.

@piaskowyk
Copy link
Member

Ok, I need to check it 🫡

@efstathiosntonas
Copy link
Contributor Author

efstathiosntonas commented Sep 11, 2023

@piaskowyk thanks, iirc this goes back 4-5 days so probably around 200230905 the changes took place.

@efstathiosntonas
Copy link
Contributor Author

efstathiosntonas commented Sep 11, 2023

@piaskowyk I've just installed 3.5.0 and I get this error. The only release that won't throw error is 3.5.0-nightly-20230904-dbfef8ba4 all the releases afterwards have this issue.

@kartikk221
Copy link

kartikk221 commented Sep 11, 2023

@efstathiosntonas Is right. Just installed 3.5.0 and am receiving this error when trying to startup my app. Note! The app builds fine.

Slightly different error traces from OP:
IMG_5905
IMG_5906

package.json dependencies:

{
    "@fortawesome/react-native-fontawesome": "^0.3.0",
    "@gorhom/bottom-sheet": "git@github.com:kartikk221/react-native-bottom-sheet.git",
    "@react-native-async-storage/async-storage": "1.18.2",
    "@react-native-masked-view/masked-view": "0.2.9",
    "@react-navigation/bottom-tabs": "^6.5.8",
    "@react-navigation/native": "^6.1.7",
    "@react-navigation/native-stack": "^6.9.13",
    "@shopify/flash-list": "^1.5.0",
    "burnt": "^0.11.7",
    "cached-lookup": "^5.3.3",
    "events": "^3.3.0",
    "expo": "^49.0.10",
    "expo-apple-authentication": "~6.1.0",
    "expo-build-properties": "~0.8.3",
    "expo-clipboard": "^4.3.1",
    "expo-crypto": "~12.4.1",
    "expo-dev-client": "^2.4.8",
    "expo-font": "~11.4.0",
    "expo-haptics": "~12.4.0",
    "expo-image": "~1.3.2",
    "expo-linear-gradient": "~12.3.0",
    "expo-linking": "~5.0.2",
    "expo-notifications": "~0.20.1",
    "expo-secure-store": "~12.3.1",
    "expo-splash-screen": "^0.20.5",
    "expo-status-bar": "~1.6.0",
    "expo-store-review": "~6.4.0",
    "expo-system-ui": "~2.4.0",
    "expo-updates": "^0.18.12",
    "expo-web-browser": "~12.3.2",
    "p-queue": "^6.6.2",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "react-native": "0.72.3",
    "react-native-animateable-text": "^0.11.0",
    "react-native-collapsible-tab-view": "git@github.com:kartikk221/react-native-collapsible-tab-view.git",
    "react-native-fast-shadow": "^0.1.0",
    "react-native-gesture-handler": "^2.12.1",
    "react-native-hyperlink": "^0.0.22",
    "react-native-image-viewing": "^0.2.2",
    "react-native-mmkv": "^2.10.2",
    "react-native-pager-view": "6.2.0",
    "react-native-reanimated": "^3.5.0",
    "react-native-safe-area-context": "^4.7.2",
    "react-native-screens": "^3.25.0",
    "react-native-share": "^9.4.1",
    "react-native-svg": "^13.10.0",
    "react-native-svg-transformer": "^1.0.0",
    "react-native-tab-view": "^3.5.2",
    "react-native-url-polyfill": "^2.0.0",
    "react-native-web": "~0.19.6",
    "react-native-webview": "^13.2.3",
    "react-native-zoom-reanimated": "git@github.com:kartikk221/react-native-zoom-reanimated.git",
    "recyclerlistview": "^4.2.0"
  }

@efstathiosntonas
Copy link
Contributor Author

efstathiosntonas commented Sep 11, 2023

fixed:

react-native-reanimated+3.5.0.patch

diff --git a/node_modules/react-native-reanimated/src/reanimated2/NativeMethods.ts b/node_modules/react-native-reanimated/src/reanimated2/NativeMethods.ts
index b29ac0f..9eb2ad4 100644
--- a/node_modules/react-native-reanimated/src/reanimated2/NativeMethods.ts
+++ b/node_modules/react-native-reanimated/src/reanimated2/NativeMethods.ts
@@ -62,7 +62,7 @@ if (isWeb()) {
       return null;
     }
 
-    const measured = _IS_FABRIC
+    const measured = global._IS_FABRIC
       ? // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
         _measureFabric!(viewTag as ShadowNodeWrapper)
       : // eslint-disable-next-line @typescript-eslint/no-non-null-assertion

tried to make a PR but it's late and I was like this when I was trying to to overcome my previous PR about the workflows:
spike-monkey-typing

@taylor-ortiz
Copy link

@efstathiosntonas has this been resolved yet? I am also facing the same issue

@efstathiosntonas
Copy link
Contributor Author

@taylor-ortiz I've just opened a PR about it, you can use patch-package to patch it with this until it's merged and released

@efstathiosntonas efstathiosntonas changed the title [nightly] ReferenceError: Property '_IS_FABRIC' doesn't exist, js engine: hermes [nightly/3.5.0] ReferenceError: Property '_IS_FABRIC' doesn't exist, js engine: hermes Sep 11, 2023
@taylor-ortiz
Copy link

@efstathiosntonas appreciate it man. thank you.

github-merge-queue bot pushed a commit that referenced this issue Sep 12, 2023
fixes
#5061

Co-authored-by: Efstathios Ntonas <stathis@hikiapp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing repro This issue need minimum repro scenario Needs review Issue is ready to be reviewed by a maintainer Platform: Android This issue is specific to Android Platform: iOS This issue is specific to iOS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants