Write a flow-typed wrapper around the native module. Update example and README.md to use this wrapper.

Author: mauritsd

.flowconfig

@@ -0,0 +1,55 @@
+[ignore]
+; We fork some components by platform
+.*/*[.]android.js
+.*/example/.*
+
+; Ignore "BUCK" generated dirs
+<PROJECT_ROOT>/\.buckd/
+
+; Ignore unexpected extra "@providesModule"
+.*/node_modules/.*/node_modules/fbjs/.*
+
+; Ignore duplicate module providers
+; For RN Apps installed via npm, "Libraries" folder is inside
+; "node_modules/react-native" but in the source repo it is in the root
+.*/Libraries/react-native/React.js
+
+; Ignore polyfills
+.*/Libraries/polyfills/.*
+
+; Ignore metro
+.*/node_modules/metro/.*
+
+[include]
+
+[libs]
+node_modules/react-native/Libraries/react-native/react-native-interface.js
+node_modules/react-native/flow/
+node_modules/react-native/flow-github/
+
+[options]
+emoji=true
+
+module.system=haste
+
+munge_underscores=true
+
+module.name_mapper='^[./a-zA-Z0-9$_-]+\.\(bmp\|gif\|jpg\|jpeg\|png\|psd\|svg\|webp\|m4v\|mov\|mp4\|mpeg\|mpg\|webm\|aac\|aiff\|caf\|m4a\|mp3\|wav\|html\|pdf\)$' -> 'RelativeImageStub'
+
+module.file_ext=.js
+module.file_ext=.jsx
+module.file_ext=.json
+module.file_ext=.native.js
+
+suppress_type=$FlowIssue
+suppress_type=$FlowFixMe
+suppress_type=$FlowFixMeProps
+suppress_type=$FlowFixMeState
+
+suppress_comment=\\(.\\|\n\\)*\\$FlowFixMe\\($\\|[^(]\\|(\\(<VERSION>\\)? *\\(site=[a-z,_]*react_native[a-z,_]*\\)?)\\)
+suppress_comment=\\(.\\|\n\\)*\\$FlowIssue\\((\\(<VERSION>\\)? *\\(site=[a-z,_]*react_native[a-z,_]*\\)?)\\)?:? #[0-9]+
+suppress_comment=\\(.\\|\n\\)*\\$FlowFixedInNextDeploy
+suppress_comment=\\(.\\|\n\\)*\\$FlowExpectedError
+
+[version]
+^0.68.0

README.md

@@ -22,9 +22,7 @@ or A/B testing logic.
 * Explanations of how to set it up on the native side. In the meanwhile have
   a look at AppDelegate.m for iOS and MainActivity.java / MainApplication.java
   for Android.
-* Don't export the NativeModule directly, but have a simple JS wrapper around
-  it, with Flow annotations and API docs.
-* Our reloading mechanism on Android does not seem to work...
+
 
 ## Getting started
 
@@ -67,29 +65,34 @@ or
 
 ## Usage
 ```javascript
-import RNDynamicBundle from 'react-native-dynamic-bundle';
+import {
+  setActiveBundle,
+  registerBundle,
+  unregisterBundle,
+  reloadBundle
+} from 'react-native-dynamic-bundle';
 
 /* Register a bundle in the documents directory of the app. This could be
  * pre-packaged in your app, downloaded over http, etc. Paths are relative
  * to your documents directory.
  */
-RNDynamicBundle.registerBundle('a_b_test', 'bundles/a_b_test.bundle');
+registerBundle('a_b_test', 'bundles/a_b_test.bundle');
 
 /* Set the active bundle to a_b_test. This means that on the next load
  * this bundle will be loaded instead of the default.
  */
-RNDynamicBundle.setActiveBundle('a_b_test');
+setActiveBundle('a_b_test');
 
 /* Unregister a bundle once you're done with it. Note that you will have to
  * remove the file yourself.
  */
-RNDynamicBundle.unregisterBundle('a_b_test');
+unregisterBundle('a_b_test');
 
 /* In some circumstances (e.g. the user consents to an update) we want to
  * force a bundle reload instead of waiting until the next app restart.
  * Note that this will have to result in the destruction of the current
  * RCTBridge and its recreation with the new bundle URL. It is therefore
  * recommended to sync data and let actions complete before calling this.
  */
-RNDynamicBundle.reloadBundle();
+reloadBundle();
 ```

example/App.js

@@ -1,6 +1,6 @@
 /**
- * Sample React Native App
- * https://github.com/facebook/react-native
+ * Sample RNDynamicBundle app
+ * https://github.com/mauritsd/react-native-dynamic-bundle
  * @flow
  */
 
@@ -12,7 +12,12 @@ import {
   Button,
   TextInput
 } from 'react-native';
-import RNDynamicBundle from 'react-native-dynamic-bundle';
+import {
+  setActiveBundle,
+  registerBundle,
+  unregisterBundle,
+  reloadBundle
+} from 'react-native-dynamic-bundle';
 import RNFS from 'react-native-fs';
 
 type Props = {};
@@ -25,10 +30,6 @@ export default class App extends Component<Props> {
     }
   }
 
-  componentWillMount = () => {
-
-  }
-
   render() {
     return (
       <View style={styles.container}>
@@ -52,9 +53,9 @@ export default class App extends Component<Props> {
     });
     const result = await promise;
 
-    RNDynamicBundle.registerBundle('test', 'test.bundle');
-    RNDynamicBundle.setActiveBundle('test');
-    RNDynamicBundle.reloadBundle();
+    registerBundle('test', 'test.bundle');
+    setActiveBundle('test');
+    reloadBundle();
   }
 }
 

index.js

@@ -1,6 +1,42 @@
+/**
+ * react-native-dynamic-bundle
+ * @flow
+ */
 
+// $FlowExpectedError: library code, so RN is not in node_modules
 import { NativeModules } from 'react-native';
 
 const { RNDynamicBundle } = NativeModules;
 
-export default RNDynamicBundle;
+/**
+ * Set the active Javascript bundle to the bundle with the given bundle ID in
+ * the registry. This will cause the given bundle to be loaded on the next app
+ * startup or invocation of reloadBundle().
+ */
+export function setActiveBundle(bundleId: string) {
+  RNDynamicBundle.setActiveBundle(bundleId);
+}
+
+/**
+ * Register a Javascript bundle in the bundle registry. The path to the bundle
+ * should be relative to the documents directory on iOS and to the internal app
+ * storage directory on Android, i.e. the directory returned by getFilesDir().
+ */
+export function registerBundle(bundleId: string, relativePath: string) {
+  RNDynamicBundle.registerBundle(bundleId, relativePath);
+}
+
+/**
+ * Unregister a bundle from the bundle registry.
+ */
+export function unregisterBundle(bundleId: string) {
+  RNDynamicBundle.unregisterBundle(bundleId);
+}
+
+/**
+ * Reload the bundle that is used by the app immediately. This can be used to
+ * apply a new bundle that was set by setActiveBundle() immediately.
+ */
+export function reloadBundle() {
+  RNDynamicBundle.reloadBundle();
+}

package.json

@@ -25,5 +25,8 @@
   "peerDependencies": {
     "react-native": "*"
   },
-  "nativePackage": true
+  "nativePackage": true,
+  "devDependencies": {
+    "flow-bin": "^0.68.0"
+  }
 }

yarn.lock

@@ -0,0 +1,7 @@
+# THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY.
+# yarn lockfile v1
+
+
+flow-bin@^0.68.0:
+  version "0.68.0"
+  resolved "https://registry.yarnpkg.com/flow-bin/-/flow-bin-0.68.0.tgz#86c2d14857d306eb2e85e274f2eebf543564f623"