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

App crashes randomly on Android 12 with JSApplicationIllegalArgumentException: Animated node with tag ### does not exists #33375

Closed
LiuIos opened this issue Mar 4, 2022 · 156 comments
Labels
API: Animated Needs: Author Feedback Needs: Repro This issue could be improved with a clear list of steps to reproduce the issue. Platform: Android Android applications. Resolution: Fixed A PR that fixes this issue has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Comments

@LiuIos
Copy link

LiuIos commented Mar 4, 2022

Description

com.facebook.react.bridge.JSApplicationIllegalArgumentException: Animated node with tag 201 does not exists
at com.facebook.react.animated.NativeAnimatedNodesManager.connectAnimatedNodes(NativeAnimatedNodesManager.java:317)
at com.facebook.react.animated.NativeAnimatedModule$15.execute(NativeAnimatedModule.java:681)
at com.facebook.react.animated.NativeAnimatedModule.executeAllOperations(NativeAnimatedModule.java:258)
at com.facebook.react.animated.NativeAnimatedModule.access$400(NativeAnimatedModule.java:85)
at com.facebook.react.animated.NativeAnimatedModule$3.execute(NativeAnimatedModule.java:287)
at com.facebook.react.uimanager.UIViewOperationQueue$UIBlockOperation.execute(UIViewOperationQueue.java:573)
at com.facebook.react.uimanager.UIViewOperationQueue$1.run(UIViewOperationQueue.java:908)
at com.facebook.react.uimanager.UIViewOperationQueue.flushPendingBatches(UIViewOperationQueue.java:1019)
at com.facebook.react.uimanager.UIViewOperationQueue.access$2600(UIViewOperationQueue.java:47)
at com.facebook.react.uimanager.UIViewOperationQueue$DispatchUIFrameCallback.doFrameGuarded(UIViewOperationQueue.java:1079)
at com.facebook.react.uimanager.GuardedFrameCallback.doFrame(GuardedFrameCallback.java:29)
at com.facebook.react.modules.core.ReactChoreographer$ReactChoreographerDispatcher.doFrame(ReactChoreographer.java:175)
at com.facebook.react.modules.core.ChoreographerCompat$FrameCallback$1.doFrame(ChoreographerCompat.java:85)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1140)
at android.view.Choreographer.doCallbacks(Choreographer.java:946)
at android.view.Choreographer.doFrame(Choreographer.java:870)
at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1127)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:210)
at android.os.Looper.loop(Looper.java:299)
at android.app.ActivityThread.main(ActivityThread.java:8085)
at java.lang.reflect.Method.invoke(Method.java:-2)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:556)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1045)

Version

0.64.2

Output of npx react-native info

System:
OS: macOS 11.4
CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
Memory: 65.65 MB / 8.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.10.0 - /usr/local/bin/node
Yarn: 1.22.10 - /usr/local/bin/yarn
npm: 7.24.0 - /usr/local/bin/npm
Watchman: 2021.09.13.00 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
Android SDK:
API Levels: 23, 25, 27, 28, 29, 30
Build Tools: 28.0.3, 29.0.2, 30.0.2
System Images: android-29 | Intel x86 Atom_64
Android NDK: 22.1.7171670
IDEs:
Android Studio: 4.2 AI-202.7660.26.42.7351085
Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_292 - /Library/Java/JavaVirtualMachines/adoptopenjdk-8.jdk/Contents/Home/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: 0.64.2 => 0.64.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found

Steps to reproduce

App crashes randomly in Android

Snack, code example, screenshot, or link to a repository

No response

@react-native-bot react-native-bot added the Platform: Android Android applications. label Mar 4, 2022
@jefersonvinicius
Copy link

I'am getting this error too!

I created a repo with a simple app as example: https://github.com/jefersonvinicius/react-native-crashing-example

Some extra information:

  • At least with me, the error just happens in release apk. The app works perfectly in debug mode.
  • The error is happening in an Android 12 device, a Galaxy S10 SM-G973F. I tried replicate the error in Android 9 but no error is threw.
  • I don't know if is related, but I realize the crash happens often when I'am scrolling.

@zhuanghongji
Copy link

Meeting the same crash randomly at version both 0.64.2 and 0.67.3

First i think the point is useNativeDriver: true, so i try to set all useNativeDriver to false, but the crash still exits.

@LiuIos
Copy link
Author

LiuIos commented Mar 11, 2022

Is there a solution?

@coolguy001tv
Copy link

I'am getting this error too!

I created a repo with a simple app as example: https://github.com/jefersonvinicius/react-native-crashing-example

Some extra information:

  • At least with me, the error just happens in release apk. The app works perfectly in debug mode.
  • The error is happening in an Android 12 device, a Galaxy S10 SM-G973F. I tried replicate the error in Android 9 but no error is threw.
  • I don't know if is related, but I realize the crash happens often when I'am scrolling.

Nearly the same, but I got this error even in debug mode randomly.
I am facing this problem in Mi12(Xiaomi 2201123C) with Android 12.
It seems ok in other devices using android 10/9/8 or ios.

@coolguy001tv
Copy link

#33400
seems the same problem...

@DjamshidDjurayev
Copy link

same here

@guptayush
Copy link

guptayush commented Mar 15, 2022

I'am getting this error too!
I created a repo with a simple app as example: https://github.com/jefersonvinicius/react-native-crashing-example
Some extra information:

  • At least with me, the error just happens in release apk. The app works perfectly in debug mode.
  • The error is happening in an Android 12 device, a Galaxy S10 SM-G973F. I tried replicate the error in Android 9 but no error is threw.
  • I don't know if is related, but I realize the crash happens often when I'am scrolling.

Nearly the same, but I got this error even in debug mode randomly. I am facing this problem in Mi12(Xiaomi 2201123C) with Android 12. It seems ok in other devices using android 10/9/8 or ios.

@coolguy001tv can you try
setImmediate(()=> LayoutAnimation.configureNext(androidAnimation))

@coolguy001tv
Copy link

@guptayush thanks for the reply. But it doesn't help me a lot. I tried to comment the setImmediate rows (there's only one component using it in my project) , it still crashed.
Actually, my project has hundreds of pages, I don't know if it's some other animated things affect.

@coolguy001tv
Copy link

I'am getting this error too!

I created a repo with a simple app as example: https://github.com/jefersonvinicius/react-native-crashing-example

Some extra information:

  • At least with me, the error just happens in release apk. The app works perfectly in debug mode.
  • The error is happening in an Android 12 device, a Galaxy S10 SM-G973F. I tried replicate the error in Android 9 but no error is threw.
  • I don't know if is related, but I realize the crash happens often when I'am scrolling.

I tried your example in 0.67.1 with Mi12(Xiaomi 2201123C, android 12), which still crashed in debug mode and release apk. But I can only reproduce the crash in your Listing Page in the example, I can't reproduce in Animations Page.

@coolguy001tv
Copy link

Can somebody help? I get nearly mad...

@zhuanghongji
Copy link

Can somebody help? I get nearly mad...

Yesterday i got a fool-way but not try yet to avoid the crash for a short time:

  1. clone the match-react-native-version code
  2. goto com.facebook.react.animated.NativeAnimatedNodesManager.connectAnimatedNodes and add try...catch... to wrap all method body
  3. rebuild a .aar file to replace ./node_modules/react-native/android/.../...aar file
  4. maybe in 3th step, should replace all file in the dir

@mayankyadav-zepto
Copy link

mayankyadav-zepto commented Mar 18, 2022

Looked at Sentry logs of my RN app on production.

Here are the list of device models on which this crash is happening.

[ 'SM-G970F', 'SM-N970F', 'SM-F700F', 'SM-G990E', 'SM-G988B', 'SM-S908E', 'SM-G991B', 'SM-G780F', 'SM-A725F', 'SM-G996B', 'SM-G975F', 'SM-G980F', 'SM-G770F', 'SM-N770F', 'SM-G973F', 'SM-G985F', 'SM-N975F', 'SM-N980F', 'SM-A528B', 'SM-A525F', 'SM-F711B', 'SM-G998B', 'SM-G781B', 'SM-N986B' ]

Hope this helps. For now disabling animation on these devices.

All of them are on Android 12.

@coolguy001tv
Copy link

How can I disable all the animation? I have hundreds of pages and some other dependencies...

@coolguy001tv
Copy link

Can somebody help? I get nearly mad...

Yesterday i got a fool-way but not try yet to avoid the crash for a short time:

  1. clone the match-react-native-version code
  2. goto com.facebook.react.animated.NativeAnimatedNodesManager.connectAnimatedNodes and add try...catch... to wrap all method body
  3. rebuild a .aar file to replace ./node_modules/react-native/android/.../...aar file
  4. maybe in 3th step, should replace all file in the dir

I am trying to do this too...
But it's so hard to rebuild react-native in windows, I failed again and again with error like missing xxx.o.
But my colleague who is using mac seems to succeed. I'll try this method. When I have any new info, I shall talk here.

@zhuanghongji
Copy link

Today i find out another IllegalArgumentException crash and it is also only happen on Android 12 level 31, detail crash log below:

# main(2)
java.lang.IllegalArgumentException
Invalid parent node provided

com.facebook.react.animated.InterpolationAnimatedNode.onDetachedFromNode(InterpolationAnimatedNode.java:205)
com.facebook.react.animated.AnimatedNode.removeChild(AnimatedNode.java:39)
com.facebook.react.animated.NativeAnimatedNodesManager.disconnectAnimatedNodes(NativeAnimatedNodesManager.java:340)
com.facebook.react.animated.NativeAnimatedModule$16.execute(NativeAnimatedModule.java:709)
com.facebook.react.animated.NativeAnimatedModule.executeAllOperations(NativeAnimatedModule.java:258)
com.facebook.react.animated.NativeAnimatedModule.access$400(NativeAnimatedModule.java:85)
com.facebook.react.animated.NativeAnimatedModule$3.execute(NativeAnimatedModule.java:287)
com.facebook.react.uimanager.UIViewOperationQueue$UIBlockOperation.execute(UIViewOperationQueue.java:573)
com.facebook.react.uimanager.UIViewOperationQueue$1.run(UIViewOperationQueue.java:908)
com.facebook.react.uimanager.UIViewOperationQueue.flushPendingBatches(UIViewOperationQueue.java:1019)
com.facebook.react.uimanager.UIViewOperationQueue.access$2600(UIViewOperationQueue.java:47)
com.facebook.react.uimanager.UIViewOperationQueue$DispatchUIFrameCallback.doFrameGuarded(UIViewOperationQueue.java:1079)
com.facebook.react.uimanager.GuardedFrameCallback.doFrame(GuardedFrameCallback.java:29)
com.facebook.react.modules.core.ReactChoreographer$ReactChoreographerDispatcher.doFrame(ReactChoreographer.java:175)
com.facebook.react.modules.core.ChoreographerCompat$FrameCallback$1.doFrame(ChoreographerCompat.java:85)
android.view.Choreographer$CallbackRecord.run(Choreographer.java:1156)
android.view.Choreographer.doCallbacks(Choreographer.java:948)
android.view.Choreographer.doFrame(Choreographer.java:872)
android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1140)
android.os.Handler.handleCallback(Handler.java:938)
android.os.Handler.dispatchMessage(Handler.java:99)
android.os.Looper.loopOnce(Looper.java:210)
android.os.Looper.loop(Looper.java:299)
android.app.ActivityThread.main(ActivityThread.java:8188)
java.lang.reflect.Method.invoke(Native Method)
com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:556)
com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1045)

Maybe between the two crash have some relationship.

react-native: 0.64.2

@DjamshidDjurayev
Copy link

any updates on this issue? it happens only on android 12 devices. This crash also happens java.lang.IllegalArgumentException
Invalid parent node provided

@coolguy001tv
Copy link

We are also meeting those exceptions like java.lang.IllegalArgumentException,Invalid parent node provided etc.
Actually, we replace throw new JSApplicationIllegalArgumentException(...) to return in many files to try to avoid such exception. Because those exception throws after condition like if (parentNode == null) or if (childNode == null), I think maybe those node are released somewhere, so I'd like just to return instead of throw exception.(Of course it's not right to do so, for doing this may lead some memory leaks, but for now I have no other better idea).

After doing this, we find the crash happened less than before.(The log shows it indeed gets into the code before the return statement we just replaced, and without the exception, app can continue). But sometimes we find the app may freeze. When freezed, sometimes it may recover by locking the screen and then unlocking the screen (sometimes this operation won't help).

We know the right thing should be look deeper in why parentNode/childNode is null, but we are not android developer, so we have to try to bypass those exceptions.

Still waiting for help...

@mouricef
Copy link

Facing the same issue. it happens only on android 12 devices.

Sentry log - https://sentry.io/share/issue/412e43bba0594b70a3e4327ecbb4199d/

Any solution or workaround?

@coolguy001tv
Copy link

@LiuIos Maybe the title can be modified to be android 12 to be more specific. It seems that it happens only on android 12 devices according to our discussion.

@LiuIos LiuIos changed the title App crashes randomly - Android App crashes randomly - only Android12 Mar 22, 2022
@LiuIos
Copy link
Author

LiuIos commented Mar 22, 2022

@coolguy001tv 👌

@coolguy001tv
Copy link

coolguy001tv commented Mar 23, 2022

@cortinico sorry to disturb, but can you help us to solve this bug in android 12? It bothers us a lot in such a long time and there's still no one can offer deeper help.

@CaioQuirinoMedeiros
Copy link

I'm having the same problem for a few weeks now. I also confirm that it happens just on devices running Android 12... It just crashes randomly.
I tried to remove all Animated uses from the project, even removed all dependencies that used it, but the problem persisted, which makes me think that it is indeed a ReactNative issue.

@yanxlg
Copy link

yanxlg commented Mar 25, 2022

Has anyone tried react-native@0.63.4? I found that some people said it would be OK to reduce from 64 to 63.

@yanxlg
Copy link

yanxlg commented Mar 25, 2022

I'am getting this error too!

I created a repo with a simple app as example: https://github.com/jefersonvinicius/react-native-crashing-example

Some extra information:

  • At least with me, the error just happens in release apk. The app works perfectly in debug mode.
  • The error is happening in an Android 12 device, a Galaxy S10 SM-G973F. I tried replicate the error in Android 9 but no error is threw.
  • I don't know if is related, but I realize the crash happens often when I'am scrolling.

I did not get this error with mi 10 pro device!

I tried this sample project,but did not crashed in any page.Maybe my testing device is ok with this issue.But I fond that some people said it would be OK to reduce react-native from 64 to 63,can you try it and offer the result?

@DjamshidDjurayev
Copy link

what about 0.66?

@coolguy001tv
Copy link

Has anyone tried react-native@0.63.4? I found that some people said it would be OK to reduce from 64 to 63.

Can u post more info(like url) on this discussion?
And what's your android version in mi 10 pro device?

@yanxlg
Copy link

yanxlg commented Mar 25, 2022

Has anyone tried react-native@0.63.4? I found that some people said it would be OK to reduce from 64 to 63.

Can u post more info(like url) on this discussion? And what's your android version in mi 10 pro device?

stackoverflow

my device's android version is 12.It is upgrated yesterday.

@oleksandr-dziuban
Copy link

Hey @HessiPard it's reproducible for us on reanimated v1 or v2

@Sabbaghh
Copy link

Sabbaghh commented Jul 12, 2022

I reproduced my app on an empty react-native template, took me a whole day but it's working now.
seems like it's a dependencies issue.

@oleksandr-dziuban
Copy link

oleksandr-dziuban commented Jul 12, 2022

@Sabbaghh Is it possible to upload this project to Github so @mikehardy could take a look?

@Sabbaghh
Copy link

@oleksandr-dziuban Sorry, it's a private repo for a client that my company have.

@mikehardy
Copy link
Contributor

Don't tag me haha, I don't suffer this crash and I'm travelling. I've only been trying to set appropriate expectations since it was tagged in to the release discussion. Those expectations should be that if you are affected you've got to produce a reproduction others can run and a PR is worth more than a reproduction. Standard expectations really but somehow not widely understood 🤷

@oleksandr-dziuban
Copy link

oleksandr-dziuban commented Jul 12, 2022

@Sabbaghh if you can reproduce it on empty RN template os please just upload this template project to public repo. Private repo is not needed for bug reproduction)

@AlenToma
Copy link

AlenToma commented Jul 19, 2022

This happen for my app to, the sad thing is that I could only see this issue on Google crashes report.

I cannot reproduce this on debug.

@mikehardy
Copy link
Contributor

@AlenToma that does not move the issue forward

expectations should be that if you are affected you've got to produce a reproduction others can run and a PR is worth more than a reproduction.

Share the reproduction!

@AlenToma
Copy link

I am trying to reproduce the issue with no luck.

I only see the crash on google crash report and hade not been able to reproduce the issue on release or otherwise on android emulator with android 12.

Still trying Though, will let you know if I can reproduce the issue.

@AlenToma
Copy link

Tried on Samsung devices on Samsung websites with no luck reproducing's the issue.
This a very special case it seems.

@AlenToma
Copy link

AlenToma commented Jul 19, 2022

Here is what I get on google play console.

com.facebook.react.bridge.JSApplicationIllegalArgumentException: 
  at com.facebook.react.animated.NativeAnimatedNodesManager.connectAnimatedNodes (NativeAnimatedNodesManager.java:340)
  at com.facebook.react.animated.NativeAnimatedModule$15.execute (NativeAnimatedModule.java:681)
  at com.facebook.react.animated.NativeAnimatedModule.executeAllOperations (NativeAnimatedModule.java:258)
  at com.facebook.react.animated.NativeAnimatedModule.access$400 (NativeAnimatedModule.java:85)
  at com.facebook.react.animated.NativeAnimatedModule$3.execute (NativeAnimatedModule.java:287)
  at com.facebook.react.uimanager.UIViewOperationQueue$UIBlockOperation.execute (UIViewOperationQueue.java:579)
  at com.facebook.react.uimanager.UIViewOperationQueue$1.run (UIViewOperationQueue.java:915)
  at com.facebook.react.uimanager.UIViewOperationQueue.flushPendingBatches (UIViewOperationQueue.java:1026)
  at com.facebook.react.uimanager.UIViewOperationQueue.access$2600 (UIViewOperationQueue.java:47)
  at com.facebook.react.uimanager.UIViewOperationQueue$2.runGuarded (UIViewOperationQueue.java:984)
  at com.facebook.react.bridge.GuardedRunnable.run (GuardedRunnable.java:31)
  at android.os.Handler.handleCallback (Handler.java:938)
  at android.os.Handler.dispatchMessage (Handler.java:99)
  at android.os.Looper.loopOnce (Looper.java:226)
  at android.os.Looper.loop (Looper.java:313)
  at android.app.ActivityThread.main (ActivityThread.java:8680)
  at java.lang.reflect.Method.invoke (Native Method)
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:567)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1135)

another thing is that all the reports comes from Samsung Galaxy A13 and i cant find this device on samsung emulator websites

@ardasatata

This comment was marked as duplicate.

@MuhammadRafeh
Copy link

Any progress?

@mikehardy
Copy link
Contributor

@MuhammadRafeh this issue needs a clean reproduction. Do you have one to share? If not, there's no progress. If no one else posts one, there's no progress. No need to ask, you'll see someone post it here, then you can just see it

@scrapecoder
Copy link

facing the same issue, the app is crashing on android 12 randomaly.

@MuhammadRafeh
Copy link

MuhammadRafeh commented Aug 7, 2022

@mikehardy facing this issue in production

Exception com.facebook.react.bridge.JSApplicationIllegalArgumentException:
  at com.facebook.react.animated.NativeAnimatedNodesManager.connectAnimatedNodes (NativeAnimatedNodesManager.java)
  at com.facebook.react.animated.NativeAnimatedModule$15.execute (NativeAnimatedModule.java)
  at com.facebook.react.animated.NativeAnimatedModule.executeAllOperations (NativeAnimatedModule.java)
  at com.facebook.react.animated.NativeAnimatedModule.access$400 (NativeAnimatedModule.java)
  at com.facebook.react.animated.NativeAnimatedModule$3.execute (NativeAnimatedModule.java)
  at com.facebook.react.uimanager.UIViewOperationQueue$UIBlockOperation.execute (UIViewOperationQueue.java)
  at com.facebook.react.uimanager.UIViewOperationQueue$1.run (UIViewOperationQueue.java)
  at com.facebook.react.uimanager.UIViewOperationQueue.flushPendingBatches (UIViewOperationQueue.java)
  at com.facebook.react.uimanager.UIViewOperationQueue.access$2600 (UIViewOperationQueue.java)
  at com.facebook.react.uimanager.UIViewOperationQueue$DispatchUIFrameCallback.doFrameGuarded (UIViewOperationQueue.java)
  at com.facebook.react.uimanager.GuardedFrameCallback.doFrame (GuardedFrameCallback.java)
  at com.facebook.react.modules.core.ReactChoreographer$ReactChoreographerDispatcher.doFrame (ReactChoreographer.java)
  at com.facebook.react.modules.core.ChoreographerCompat$FrameCallback$1.doFrame (ChoreographerCompat.java)
  at android.view.Choreographer$CallbackRecord.run (Choreographer.java:1106)
  at android.view.Choreographer.doCallbacks (Choreographer.java:866)
  at android.view.Choreographer.doFrame (Choreographer.java:792)
  at android.view.Choreographer$FrameDisplayEventReceiver.run (Choreographer.java:1092)
  at android.os.Handler.handleCallback (Handler.java:938)
  at android.os.Handler.dispatchMessage (Handler.java:99)
  at android.os.Looper.loopOnce (Looper.java:226)
  at android.os.Looper.loop (Looper.java:313)
  at android.app.ActivityThread.main (ActivityThread.java:8680)
  at java.lang.reflect.Method.invoke (Method.java)
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:567)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1135)

@mikehardy
Copy link
Contributor

People still tagging me like they haven't read the comments 😂. Please don't, thank you

@kelset kelset added the Needs: Repro This issue could be improved with a clear list of steps to reproduce the issue. label Aug 8, 2022
@github-actions
Copy link

github-actions bot commented Aug 8, 2022

⚠️ Missing Reproducible Example
ℹ️ It looks like your issue is missing a reproducible example. Please provide a Snack or a repository that demonstrates the issue you are reporting in a minimal, complete, and reproducible manner.

@kelset
Copy link
Contributor

kelset commented Aug 8, 2022

Unless someone can provide a repro with RN 0.69 of this issue we'll close this off.

@GaelCO
Copy link

GaelCO commented Aug 9, 2022

Hi @kelset ,

It's very hard to make a repro, because this issue happens only on some devices (samsung galaxy A13, Vivo V2118).
I understand that it is almost impossible to fix without repro, but that does not mean that the issue does not exist and that it should be closed.

Is it possible to add some checks on the node integrity before connecting it?

@kelset
Copy link
Contributor

kelset commented Aug 9, 2022

I understand that it is almost impossible to fix without repro, but that does not mean that the issue does not exist and that it should be closed.

closed means simply that there's nothing we can do about it. I hope you can understand that seeing constant notification on this issue by comments with really nothing to add is just adding noise on the maintainers. An alternative would be to lock it, and just tell people to submit PRs to try and improve the situation. Would that be better?

@guillaumekh
Copy link

guillaumekh commented Aug 9, 2022

An alternative would be to lock it, and just tell people to submit PRs to try and improve the situation. Would that be better?

As a disciplined, silent watcher of this issue, I support that 🙏

@Dannark
Copy link

Dannark commented Aug 9, 2022

I just recorded this error today from google Robo testLab. I was using Pixel 6a, API nível 32 with RN 0.68.1 I can't share the video but the steps was this:

{
...[More steps]...
  }, {
    "sequence": 301,
    "sourceScreenId": "73",
    "destinationScreenId": "80",
    "startTimeSeconds": 576.609728,
    "endTimeSeconds": 578.248523,
    "executionResult": "SUCCESS",
    "target": {
      "type": "SWIPE",
      "childSequence": "0.0.1.0.1.0.0.0.0.0.0.0.1.0.0.0.0.0.0.0.0.0.0.0.1.0.0.0.0.1.0.0.0.1",
      "targetDetails": {
        "className": "y7.d",
        "androidClassName": "android.widget.HorizontalScrollView"
      }
    }
  }, {
    "sequence": 302,
    "sourceScreenId": "80",
    "destinationScreenId": "80",
    "startTimeSeconds": 578.897558,
    "endTimeSeconds": 579.630209,
    "executionResult": "SUCCESS",
    "target": {
      "type": "SWIPE",
      "childSequence": "0.0.1.0.1.0.0.0.0.0.0.0.1.0.0.0.1.1.0.2",
      "targetDetails": {
        "className": "com.facebook.react.views.text.s",
        "androidClassName": "android.widget.TextView",
        "textLabel": "Início"
      }
    }
  }, {
    "sequence": 303,
    "sourceScreenId": "80",
    "destinationScreenId": "80",
    "startTimeSeconds": 580.006176,
    "endTimeSeconds": 581.236872,
    "executionResult": "TERMINATED",
    "target": {
      "type": "TAP",
      "childSequence": "0.0.1.0.1.0.0.0.0.0.0.0.1.0.0.0.1.1.0",
      "targetDetails": {
        "className": "com.facebook.react.views.view.g",
        "androidClassName": "android.view.ViewGroup"
      }
    }
  }]
}

The step 301 was a single swipe on a HorizontalScroll list (with 12 items) until about 95% of it's end,
Step 302: The script waited about 1s before clicking on the home tab called Inicio
Step 303: The home screen was already loaded and the script seems to have tried to click in something but it did crashed so fast (less than a half sec) that it couldn't event show the TAP in the video it self.

@jhmobile
Copy link

the app is crashing on android 12 randomaly.

com.facebook.react.bridge.JSApplicationIllegalArgumentException: connectAnimatedNodes: Animated node with tag (child) [290] does not exist
at com.facebook.react.animated.NativeAnimatedNodesManager.connectAnimatedNodes(:5)
at com.facebook.react.animated.NativeAnimatedModule$15.execute(:1)
at com.facebook.react.animated.NativeAnimatedModule.executeAllOperations(:5)
at com.facebook.react.animated.NativeAnimatedModule.access$400(:1)
at com.facebook.react.animated.NativeAnimatedModule$3.execute(:1)
at com.facebook.react.uimanager.UIViewOperationQueue$UIBlockOperation.execute(:1)
at com.facebook.react.uimanager.UIViewOperationQueue$1.run(:19)
at com.facebook.react.uimanager.UIViewOperationQueue.flushPendingBatches(:10)
at com.facebook.react.uimanager.UIViewOperationQueue.access$2600(:1)
at com.facebook.react.uimanager.UIViewOperationQueue$DispatchUIFrameCallback.doFrameGuarded(:6)
at com.facebook.react.uimanager.GuardedFrameCallback.doFrame(:1)
at com.facebook.react.modules.core.ReactChoreographer$ReactChoreographerDispatcher.doFrame(:7)
at com.facebook.react.modules.core.ChoreographerCompat$FrameCallback$1.doFrame(:1)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1184)
at android.view.Choreographer.doCallbacks(Choreographer.java:986)
at android.view.Choreographer.doFrame(Choreographer.java:907)
at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1171)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:233)
at android.os.Looper.loop(Looper.java:334)
at android.app.ActivityThread.main(ActivityThread.java:8396)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:582)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1068)

@cortinico
Copy link
Contributor

closed means simply that there's nothing we can do about it

As @kelset suggested, we're going to close and lock this issue for the time being.

Sadly, the amount of users commenting with "same issue", "+1", or "any updates?" is increasing the maintainers burden for us all and is not bringing any value.

The current status is the following:

  • There seems to be a crash on Android 12 related to animations with the message Animated node with tag (child) [290] does not exist
  • As of today, there is no reproducer that can help us investigate.
  • The crash seems to be isolated to Samsung devices and is probably related to a OEM re-implementaation of part of the framework
  • There was an attempt to fix this crash by offering the ReactFeatureFlags.enableSynchronizationForAnimated flag. As of today, no one has confirmed tha this flag mitigates the crash.

We understand that this is creating issues for your apps & users, but in order to solve this we're going to accept only:

  • Pull requests which are effectively solving this issue
  • New Issues referencing this same problem with provided reproducers. New Issues without a reproducer will be closed.

@facebook facebook locked as spam and limited conversation to collaborators Aug 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
API: Animated Needs: Author Feedback Needs: Repro This issue could be improved with a clear list of steps to reproduce the issue. Platform: Android Android applications. Resolution: Fixed A PR that fixes this issue has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.
Projects
None yet
Development

No branches or pull requests