@hot-updater/react-native 0.27.1 → 0.29.0

package/android/src/oldarch/HotUpdaterModule.kt

@@ -7,6 +7,7 @@ import com.facebook.react.bridge.Promise
 import com.facebook.react.bridge.ReactApplicationContext
 import com.facebook.react.bridge.ReactMethod
 import com.facebook.react.bridge.ReadableMap
+import com.facebook.react.bridge.WritableNativeArray
 import com.facebook.react.bridge.WritableNativeMap
 import com.facebook.react.modules.core.DeviceEventManagerModule
 import kotlinx.coroutines.CoroutineScope
@@ -14,13 +15,12 @@ import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.SupervisorJob
 import kotlinx.coroutines.cancel
 import kotlinx.coroutines.launch
-import org.json.JSONArray
-import org.json.JSONObject
 
 class HotUpdaterModule internal constructor(
     context: ReactApplicationContext,
 ) : HotUpdaterSpec(context) {
     private val mReactApplicationContext: ReactApplicationContext = context
+    private val cohortService = CohortService(context)
 
     // Managed coroutine scope for the module lifecycle
     private val moduleScope = CoroutineScope(SupervisorJob() + Dispatchers.Main)
@@ -153,32 +153,10 @@ class HotUpdaterModule internal constructor(
     }
 
     @ReactMethod(isBlockingSynchronousMethod = true)
-    override fun notifyAppReady(params: ReadableMap): String {
-        val bundleId = params.getString("bundleId")
-        val result = JSONObject()
-
-        if (bundleId == null) {
-            result.put("status", "STABLE")
-            return result.toString()
-        }
-
-        val impl = getInstance()
-        val statusMap = impl.notifyAppReady(bundleId)
-
-        result.put("status", statusMap["status"] as? String ?: "STABLE")
-        statusMap["crashedBundleId"]?.let {
-            result.put("crashedBundleId", it as String)
-        }
-
-        return result.toString()
-    }
+    override fun notifyAppReady(): WritableNativeMap = getInstance().notifyAppReady().toWritableNativeMap()
 
     @ReactMethod(isBlockingSynchronousMethod = true)
-    override fun getCrashHistory(): String {
-        val impl = getInstance()
-        val crashHistory = impl.getCrashHistory()
-        return JSONArray(crashHistory).toString()
-    }
+    override fun getCrashHistory(): WritableNativeArray = getInstance().getCrashHistory().toWritableNativeArray()
 
     @ReactMethod(isBlockingSynchronousMethod = true)
     override fun clearCrashHistory(): Boolean {
@@ -192,7 +170,23 @@ class HotUpdaterModule internal constructor(
         return impl.getBaseURL()
     }
 
+    @ReactMethod(isBlockingSynchronousMethod = true)
+    override fun getBundleId(): String? {
+        val impl = getInstance()
+        return impl.getBundleId()
+    }
+
+    @ReactMethod(isBlockingSynchronousMethod = true)
+    override fun getManifest(): WritableNativeMap = getInstance().getManifest().toWritableNativeMap()
+
     @ReactMethod
+    override fun setCohort(cohort: String) {
+        cohortService.setCohort(cohort)
+    }
+
+    @ReactMethod(isBlockingSynchronousMethod = true)
+    override fun getCohort(): String = cohortService.getCohort()
+
     override fun resetChannel(promise: Promise) {
         moduleScope.launch {
             try {

package/android/src/oldarch/HotUpdaterSpec.kt

@@ -4,6 +4,8 @@ import com.facebook.react.bridge.Promise
 import com.facebook.react.bridge.ReactApplicationContext
 import com.facebook.react.bridge.ReactContextBaseJavaModule
 import com.facebook.react.bridge.ReadableMap
+import com.facebook.react.bridge.WritableArray
+import com.facebook.react.bridge.WritableMap
 
 abstract class HotUpdaterSpec internal constructor(
     context: ReactApplicationContext,
@@ -17,13 +19,21 @@ abstract class HotUpdaterSpec internal constructor(
 
     abstract fun reloadProcess(promise: Promise)
 
-    abstract fun notifyAppReady(params: ReadableMap): String
+    abstract fun notifyAppReady(): WritableMap
 
-    abstract fun getCrashHistory(): String
+    abstract fun getCrashHistory(): WritableArray
 
     abstract fun clearCrashHistory(): Boolean
 
     abstract fun getBaseURL(): String
 
+    abstract fun setCohort(customId: String)
+
+    abstract fun getCohort(): String
+
+    abstract fun getBundleId(): String?
+
+    abstract fun getManifest(): WritableMap
+
     abstract fun resetChannel(promise: Promise)
 }