Support RN 0.56

This commit also upgrades TypeScript to version 2.9.x

lib/android/app/build.gradle

@@ -54,6 +54,9 @@ android {
         reactNative55 {
             dimension "RNN.reactNativeVersion"
         }
+        reactNative56 {
+            dimension "RNN.reactNativeVersion"
+        }
     }
 }
 

lib/android/app/src/main/java/com/reactnativenavigation/react/BundleDownloadListenerProvider.java

@@ -1,7 +1,5 @@
 package com.reactnativenavigation.react;
 
-import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
-
 public interface BundleDownloadListenerProvider {
-    void setBundleLoaderListener(DevBundleDownloadListener listener);
+    void setBundleLoaderListener(NavigationDevBundleDownloadListener listener);
 }

lib/android/app/src/main/java/com/reactnativenavigation/react/JsDevReloadHandler.java

@@ -7,13 +7,10 @@ import android.content.Intent;
 import android.content.IntentFilter;
 import android.view.KeyEvent;
 
-import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
 import com.facebook.react.devsupport.interfaces.DevSupportManager;
 import com.reactnativenavigation.utils.UiUtils;
 
-import javax.annotation.Nullable;
-
-public class JsDevReloadHandler implements DevBundleDownloadListener {
+public class JsDevReloadHandler extends JsDevReloadHandlerFacade {
     private static final String RELOAD_BROADCAST = "com.reactnativenavigation.broadcast.RELOAD";
 
     public interface ReloadListener {
@@ -40,16 +37,6 @@ public class JsDevReloadHandler implements DevBundleDownloadListener {
         UiUtils.runOnMainThread(reloadListener::onReload);
     }
 
-    @Override
-    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
-
-    }
-
-    @Override
-    public void onFailure(Exception cause) {
-
-    }
-
     public void setReloadListener(ReloadListener listener) {
         reloadListener = listener;
     }

lib/android/app/src/main/java/com/reactnativenavigation/react/NavigationDevBundleDownloadListener.java

@@ -0,0 +1,5 @@
+package com.reactnativenavigation.react;
+
+public interface NavigationDevBundleDownloadListener {
+    void onSuccess();
+}

lib/android/app/src/main/java/com/reactnativenavigation/react/NavigationReactNativeHost.java

@@ -25,7 +25,7 @@ public class NavigationReactNativeHost extends ReactNativeHost implements Bundle
 
     private final boolean isDebug;
     private final List<ReactPackage> additionalReactPackages;
-    private @Nullable DevBundleDownloadListener bundleListener;
+    private @Nullable NavigationDevBundleDownloadListener bundleListener;
     private final DevBundleDownloadListener bundleListenerMediator = new DevBundleDownloadListenerAdapter() {
         @Override
         public void onSuccess() {
@@ -48,7 +48,7 @@ public class NavigationReactNativeHost extends ReactNativeHost implements Bundle
     }
 
     @Override
-    public void setBundleLoaderListener(DevBundleDownloadListener listener) {
+    public void setBundleLoaderListener(NavigationDevBundleDownloadListener listener) {
         bundleListener = listener;
     }
 

lib/android/app/src/main/java/com/reactnativenavigation/react/ReloadHandler.java

@@ -1,10 +1,6 @@
 package com.reactnativenavigation.react;
 
-import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
-
-import javax.annotation.Nullable;
-
-public class ReloadHandler implements JsDevReloadHandler.ReloadListener, DevBundleDownloadListener {
+public class ReloadHandler extends ReloadHandlerFacade implements JsDevReloadHandler.ReloadListener {
 
     private Runnable onReloadListener = () -> {};
 
@@ -12,38 +8,16 @@ public class ReloadHandler implements JsDevReloadHandler.ReloadListener, DevBund
         this.onReloadListener = onReload;
     }
 
-    /**
-     * Called on RR and adb reload events
-     */
     @Override
     public void onReload() {
         onReloadListener.run();
     }
 
-    /**
-     * Called when the bundle was successfully reloaded
-     */
     @Override
     public void onSuccess() {
         onReloadListener.run();
     }
 
-    /**
-     * Bundle progress updates
-     */
-    @Override
-    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
-
-    }
-
-    /**
-     * Bundle load failure
-     */
-    @Override
-    public void onFailure(Exception cause) {
-
-    }
-
     public void destroy() {
         onReloadListener = null;
     }

lib/android/app/src/reactNative51/java/com/reactnativenavigation/react/JsDevReloadHandlerFacade.java

@@ -0,0 +1,22 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public class JsDevReloadHandlerFacade implements DevBundleDownloadListener, NavigationDevBundleDownloadListener {
+    @Override
+    public void onSuccess() {
+
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+}

lib/android/app/src/reactNative51/java/com/reactnativenavigation/react/ReloadHandlerFacade.java

@@ -0,0 +1,22 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public abstract class ReloadHandlerFacade implements DevBundleDownloadListener {
+    @Override
+    public void onSuccess() {
+
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+}

lib/android/app/src/reactNative55/java/com/reactnativenavigation/react/DevBundleDownloadListenerAdapter.java

@@ -0,0 +1,22 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public class DevBundleDownloadListenerAdapter implements DevBundleDownloadListener {
+    @Override
+    public void onSuccess() {
+
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+}

lib/android/app/src/reactNative55/java/com/reactnativenavigation/react/JsDevReloadHandlerFacade.java

@@ -0,0 +1,22 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public class JsDevReloadHandlerFacade implements DevBundleDownloadListener, NavigationDevBundleDownloadListener {
+    @Override
+    public void onSuccess() {
+
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+}

lib/android/app/src/reactNative55/java/com/reactnativenavigation/react/ReloadHandlerFacade.java

@@ -0,0 +1,22 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public abstract class ReloadHandlerFacade implements DevBundleDownloadListener {
+    @Override
+    public void onSuccess() {
+
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+}

lib/android/app/src/reactNative56/java/com/reactnativenavigation/react/DevBundleDownloadListenerAdapter.java

@@ -0,0 +1,28 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.bridge.NativeDeltaClient;
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public class DevBundleDownloadListenerAdapter implements DevBundleDownloadListener, NavigationDevBundleDownloadListener {
+    @Override
+    public void onSuccess(@Nullable NativeDeltaClient nativeDeltaClient) {
+        onSuccess();
+    }
+
+    @Override
+    public void onSuccess() {
+
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+}

lib/android/app/src/reactNative56/java/com/reactnativenavigation/react/JsDevReloadHandlerFacade.java

@@ -0,0 +1,28 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.bridge.NativeDeltaClient;
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public class JsDevReloadHandlerFacade implements DevBundleDownloadListener, NavigationDevBundleDownloadListener {
+    @Override
+    public void onSuccess(@Nullable NativeDeltaClient nativeDeltaClient) {
+        onSuccess();
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+
+    @Override
+    public void onSuccess() {
+
+    }
+}

lib/android/app/src/reactNative56/java/com/reactnativenavigation/react/ReloadHandlerFacade.java

@@ -0,0 +1,25 @@
+package com.reactnativenavigation.react;
+
+import com.facebook.react.bridge.NativeDeltaClient;
+import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
+
+import javax.annotation.Nullable;
+
+public abstract class ReloadHandlerFacade implements DevBundleDownloadListener {
+    @Override
+    public void onSuccess(@Nullable NativeDeltaClient nativeDeltaClient) {
+
+    }
+
+    @Override
+    public void onProgress(@Nullable String status, @Nullable Integer done, @Nullable Integer total) {
+
+    }
+
+    @Override
+    public void onFailure(Exception cause) {
+
+    }
+
+    protected abstract void onSuccess();
+}

lib/android/app/src/reactNative56/java/com/reactnativenavigation/react/SyncUiImplementation.java

@@ -0,0 +1,83 @@
+package com.reactnativenavigation.react;
+
+import android.support.annotation.Nullable;
+
+import com.facebook.react.bridge.ReactApplicationContext;
+import com.facebook.react.bridge.ReadableArray;
+import com.facebook.react.bridge.ReadableMap;
+import com.facebook.react.uimanager.ThemedReactContext;
+import com.facebook.react.uimanager.UIImplementation;
+import com.facebook.react.uimanager.UIImplementationProvider;
+import com.facebook.react.uimanager.UIManagerModule;
+import com.facebook.react.uimanager.ViewManager;
+import com.facebook.react.uimanager.common.MeasureSpecProvider;
+import com.facebook.react.uimanager.common.SizeMonitoringFrameLayout;
+import com.facebook.react.uimanager.events.EventDispatcher;
+
+import java.util.List;
+
+@SuppressWarnings("WeakerAccess")
+public class SyncUiImplementation extends UIImplementation {
+    private static final Object lock = new Object();
+
+    public static class Provider extends UIImplementationProvider {
+        @Override
+        public UIImplementation createUIImplementation(ReactApplicationContext reactContext, List<ViewManager> viewManagerList, EventDispatcher eventDispatcher, int minTimeLeftInFrameForNonBatchedOperationMs) {
+            return new SyncUiImplementation(reactContext, viewManagerList, eventDispatcher, minTimeLeftInFrameForNonBatchedOperationMs);
+        }
+
+        @Override
+        public UIImplementation createUIImplementation(ReactApplicationContext reactContext, UIManagerModule.ViewManagerResolver viewManagerResolver, EventDispatcher eventDispatcher, int minTimeLeftInFrameForNonBatchedOperationMs) {
+            return new SyncUiImplementation(reactContext, viewManagerResolver, eventDispatcher, minTimeLeftInFrameForNonBatchedOperationMs);
+        }
+    }
+
+    public SyncUiImplementation(ReactApplicationContext reactContext, List<ViewManager> viewManagerList, EventDispatcher eventDispatcher, int minTimeLeftInFrameForNonBatchedOperationMs) {
+        super(reactContext, viewManagerList, eventDispatcher, minTimeLeftInFrameForNonBatchedOperationMs);
+    }
+
+    public SyncUiImplementation(ReactApplicationContext reactContext, UIManagerModule.ViewManagerResolver viewManagerResolver, EventDispatcher eventDispatcher, int minTimeLeftInFrameForNonBatchedOperationMs) {
+        super(reactContext, viewManagerResolver, eventDispatcher, minTimeLeftInFrameForNonBatchedOperationMs);
+    }
+
+    @Override
+    public void manageChildren(
+            int viewTag,
+            @Nullable ReadableArray moveFrom,
+            @Nullable ReadableArray moveTo,
+            @Nullable ReadableArray addChildTags,
+            @Nullable ReadableArray addAtIndices,
+            @Nullable ReadableArray removeFrom) {
+        synchronized (lock) {
+            super.manageChildren(viewTag, moveFrom, moveTo, addChildTags, addAtIndices, removeFrom);
+        }
+    }
+
+    @Override
+    public void setChildren(int viewTag, ReadableArray childrenTags) {
+        synchronized (lock) {
+            super.setChildren(viewTag, childrenTags);
+        }
+    }
+
+    @Override
+    public void createView(int tag, String className, int rootViewTag, ReadableMap props) {
+        synchronized (lock) {
+            super.createView(tag, className, rootViewTag, props);
+        }
+    }
+
+    @Override
+    public void removeRootShadowNode(int rootViewTag) {
+        synchronized (lock) {
+            super.removeRootShadowNode(rootViewTag);
+        }
+    }
+
+    @Override
+    public <T extends SizeMonitoringFrameLayout & MeasureSpecProvider> void registerRootView(T rootView, int tag, ThemedReactContext context) {
+        synchronized (lock) {
+            super.registerRootView(rootView, tag, context);
+        }
+    }
+}

package.json

@@ -58,7 +58,7 @@
     "react-lifecycles-compat": "2.0.0",
     "hoist-non-react-statics": "2.x.x",
     "@types/react": "16.x.x",
-    "@types/react-native": "0.51.1",
+    "@types/react-native": "0.51.x",
     "@types/lodash": "4.x.x",
     "@types/prop-types": "15.x.x",
     "@types/react-test-renderer": "16.x.x",
@@ -76,7 +76,7 @@
     "semver": "5.x.x",
     "shell-utils": "1.x.x",
     "tslint": "5.x.x",
-    "typescript": "2.7.x",
+    "typescript": "2.9.x",
     "typedoc": "0.x.x",
     "handlebars": "4.x.x",
     "ts-node": "5.x.x"

tsconfig.json

@@ -7,6 +7,7 @@
     "module": "commonjs",
     "jsx": "react-native",
     "declaration": true,
+    "skipLibCheck": true,
     "types": [
       "jest",
       "lodash",

tslint.json

@@ -3,6 +3,11 @@
   "extends": [
     "tslint:recommended"
   ],
+  "linterOptions": {
+    "exclude": [
+      "**/*.test.tsx"
+    ]
+  },
   "jsRules": {
     "no-console": [false],
     "quotemark": [