Require cycle warnings with React Native 0.57.0

[thedjinn]

Version

Tell us which versions you are using:

• react-native-router-flux v4.0.1
• react v16.5.0
• react-native v0.57.0

Actual behaviour

When importing react-native-router-flux in my React Native 0.57 project I get 4 warnings about require cycles. It's working fine but these log messages are spammy because they reappear during (hot) reloading.

2018-09-20 14:08:18.565 [warn][tid:com.facebook.react.JavaScript] Require cycle: node_modules/react-native-router-flux/src/navigationStore.js -> node_modules/react-native-router-flux/src/Reducer.js -> node_modules/react-native-router-flux/src/navigationStore.js
Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.

2018-09-20 14:08:18.564752+0200 RedactedAppName[52778:12799768] Require cycle: node_modules/react-native-router-flux/src/navigationStore.js -> node_modules/react-native-router-flux/src/Reducer.js -> node_modules/react-native-router-flux/src/navigationStore.js
Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.

2018-09-20 14:08:18.566 [warn][tid:com.facebook.react.JavaScript] Require cycle: node_modules/react-native-router-flux/src/navigationStore.js -> node_modules/react-native-router-flux/src/NavBar.js -> node_modules/react-native-router-flux/src/navigationStore.js
Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.

2018-09-20 14:08:18.566431+0200 RedactedAppName[52778:12799768] Require cycle: node_modules/react-native-router-flux/src/navigationStore.js -> node_modules/react-native-router-flux/src/NavBar.js -> node_modules/react-native-router-flux/src/navigationStore.js
Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.

Steps to reproduce

Use React Native 0.57 and import the react-native-router-flux package:

import {
    Router,
    Stack,
    Scene,
    Actions,
    Drawer
} from "react-native-router-flux";

[GunnarAK]

I am getting the same warnings on RN 57 with my own screen components.
Versions
"react": "16.5",
"react-native": "0.57.0",

Require cycle: js/components/index.js -> js/components/HomeScreen.js -> js/componentes/index.js

Require cycles are allowed, but can result in uninitialized values. Consider refactoring to remove the need for a cycle.

Stack:

metroRequire
<unknown>
loadModuleImplementation
guardedLoadModule
<unknown>
loadModuleImplementation
guardedLoadModule
metroRequire
<unknown>
loadModuleImplementation

[ramibaraka]

@aksonov Pls bump the React-navigation version. The latest version depends on a newer version of react-native-safe-area-view with this issue fixed

[aksonov]

@thedjinn I removed cyclic dependency for NavBar. Please check updated examples/react-native. If it works for you, you may use current master.

[aksonov]

Should be solved within master, will release shortly