Requires cycle warning about mobx-remotedev HOT 10 OPEN

Related to kolodny/jsan#25. Looks like that limitation was added somewhere in React Native, it doesn't have any issues, but better to handle on jsan part.

from mobx-remotedev.

This is from React Native, right? If I'd have a repro would look into it, but if someone can do a PR on jsan part would be much appreciated. Or waiting till Metro decides to make it opt in as per facebook/metro#287.

from mobx-remotedev.

@zalmoxisus when i open try to debug react native using remote devtool it shows unknown call relay:check .and it shows following thing on devtools.

In screenshot you can see the message too.

I have added you in repo so you can investigate.

Currently i am working it as private which i plan to release to open source later.
Thanks again.

from mobx-remotedev.

Thanks for adding me. Unfortunately I'm not using RN atm and don't have an emulator on this machine to check now. Could you try to replace ./node-modules/jasn/lib/utils.js and restore-special.js inside your project form here. Most likely not.

I see it's warning not only when requiring the lib so should be quite annoying, but it would need to restructure the whole lib into one file which isn't quite sustainable just for a linting warning as discussed in kolodny/jsan#25.

from mobx-remotedev.

@zalmoxisus i replaced it but it seems everything is same nothing changed. I needed to create new file for restore-special.js file.

let me know if i can do anything.

from mobx-remotedev.

Yes, I tried to move it to new restore-special.js file, thought it would ignore the cycle in run time this way, but seems no.

from mobx-remotedev.

@zalmoxisus is it due to package issue or which one? like many package etc are starting to showing up. If i don't use devtool it doesn't show any errors.

from mobx-remotedev.

Yes, we have that in jsan, see kolodny/jsan#25 for more details. But that's not quite an issue with the package itself, it works as expected, but Metro from React Native is warning on that. As many suggested in facebook/metro#287, that should be be opt-in.

from mobx-remotedev.

hmm should we ignore it or ? . According to metro they are trying to protect us from code with cycle that can create problem in future.

from mobx-remotedev.

I think yes, nobody had any problems in several years with the way the cycle is setup there. Just the warning flood is annoying.

The solution is to patch Metro as indicated in facebook/metro#287 (comment).

Let's keep the issue open, so others can find.

from mobx-remotedev.