expo-manifests 0.7.1 → 0.8.1

package/CHANGELOG.md

@@ -10,7 +10,23 @@
 
 ### 💡 Others
 
-## 0.7.1 — 2023-06-30
+## 0.8.1 — 2023-08-02
+
+### 🛠 Breaking changes
+
+- Drop support for `logUrl` which sent console logs to the legacy `expo-cli`. ([#18596](https://github.com/expo/expo/pull/18596) by [@EvanBacon](https://github.com/EvanBacon))
+
+## 0.8.0 — 2023-07-28
+
+### 🎉 New features
+
+- [iOS] Expose getMetadata method to match android. ([#23445](https://github.com/expo/expo/pull/23445) by [@wschurman](https://github.com/wschurman))
+
+### 🐛 Bug fixes
+
+- [iOS] Fix error in handling nested array. ([#23562](https://github.com/expo/expo/pull/23562) by [@douglowder](https://github.com/douglowder))
+
+## 0.7.1 - 2023-06-30
 
 ### 🐛 Bug fixes
 

package/android/build.gradle

@@ -3,7 +3,7 @@ apply plugin: 'kotlin-android'
 apply plugin: 'maven-publish'
 
 group = 'host.exp.exponent'
-version = '0.7.1'
+version = '0.8.1'
 
 buildscript {
   def expoModulesCorePlugin = new File(project(":expo-modules-core").projectDir.absolutePath, "ExpoModulesCorePlugin.gradle")
@@ -67,7 +67,7 @@ android {
     minSdkVersion safeExtGet("minSdkVersion", 21)
     targetSdkVersion safeExtGet("targetSdkVersion", 33)
     versionCode 31
-    versionName '0.7.1'
+    versionName '0.8.1'
     testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
   }
   lintOptions {

package/android/src/main/java/expo/modules/manifests/core/Manifest.kt

@@ -105,7 +105,6 @@ abstract class Manifest(protected val json: JSONObject) {
 
   fun getDebuggerHost(): String = getExpoGoConfigRootObject()!!.require("debuggerHost")
   fun getMainModuleName(): String = getExpoGoConfigRootObject()!!.require("mainModuleName")
-  fun getLogUrl(): String? = getExpoGoConfigRootObject()?.getNullable("logUrl")
   fun getHostUri(): String? = getExpoClientConfigRootObject()?.getNullable("hostUri")
 
   fun isVerified(): Boolean = json.getNullable("isVerified") ?: false
@@ -270,12 +269,19 @@ internal sealed class PluginType {
     private fun fromRawValue(value: Any): PluginType? {
       return when (value) {
         is JSONArray -> {
-          if (value.length() != 2) {
+          if (value.length() == 0) {
             throw IllegalArgumentException("Value for (key = plugins) has incorrect type")
           }
           val name = value.get(0) as? String ?: return null
-          val props = value.get(1) as? JSONObject ?: return null
-          WithProps(name to props.toMap())
+          when (value.length()) {
+            2 -> {
+              val props = value.get(1) as? JSONObject ?: return null
+              WithProps(name to props.toMap())
+            }
+            else -> {
+              WithoutProps(name)
+            }
+          }
         }
         is String -> {
           WithoutProps(value)

package/android/src/test/java/expo/modules/manifests/core/ManifestTest.kt

@@ -9,28 +9,35 @@ import org.junit.Test
 class ManifestTest {
   @Test
   fun getPluginProperties_emptyManifest_returnsNull() {
-    val manifestJson = JSONObject(emptyMap<String, Any>())
+    val manifestJson = JSONObject("{}")
     val manifest = Manifest.fromManifestJson(manifestJson)
     Truth.assertThat(manifest.getPluginProperties("test")).isNull()
   }
 
   @Test
   fun getPluginProperties_emptyPlugins_returnsNull() {
-    val manifestJson = JSONObject(mapOf<String, Any>("plugins" to emptyArray<Any>()))
+    val manifestJson = JSONObject("{\"plugins\": []}")
     val manifest = Manifest.fromManifestJson(manifestJson)
     Truth.assertThat(manifest.getPluginProperties("test")).isNull()
   }
 
   @Test
   fun getPluginProperties_nonMatchedPlugins_returnsNull() {
-    val manifestJson = JSONObject(mapOf<String, Any>("plugins" to arrayOf("hello")))
+    val manifestJson = JSONObject("{\"plugins\": [\"hello\"]}")
     val manifest = Manifest.fromManifestJson(manifestJson)
     Truth.assertThat(manifest.getPluginProperties("test")).isNull()
   }
 
   @Test
   fun getPluginProperties_matchedPluginWithoutProps_returnsNull() {
-    val manifestJson = JSONObject(mapOf<String, Any>("plugins" to arrayOf("test")))
+    val manifestJson = JSONObject("{\"plugins\": [\"test\"]}")
+    val manifest = Manifest.fromManifestJson(manifestJson)
+    Truth.assertThat(manifest.getPluginProperties("test")).isNull()
+  }
+
+  @Test
+  fun getPluginProperties_matchedPluginWithoutPropsAsNestedArray_returnsNull() {
+    val manifestJson = JSONObject("{\"plugins\": [[\"test\"]]}")
     val manifest = Manifest.fromManifestJson(manifestJson)
     Truth.assertThat(manifest.getPluginProperties("test")).isNull()
   }
@@ -38,8 +45,7 @@ class ManifestTest {
   @Test
   fun getPluginProperties_matchedPluginWithProps_returnsProps() {
     val props = mapOf<String, Any>("foo" to "bar")
-    val pluginWithProp = arrayOf("test", props)
-    val manifestJson = JSONObject(mapOf<String, Any>("plugins" to arrayOf(pluginWithProp)))
+    val manifestJson = JSONObject("{\"plugins\": [ [\"test\", {\"foo\":\"bar\"}] ]}")
     val manifest = Manifest.fromManifestJson(manifestJson)
     val result = manifest.getPluginProperties("test")
     Truth.assertThat(result).isNotNull()
@@ -49,8 +55,7 @@ class ManifestTest {
   @Test
   fun getPluginProperties_matchedPluginWithNestedProps_returnsNestedProps() {
     val props = mapOf<String, Any>("nested" to mapOf<String, Any>("insideNested" to true))
-    val pluginWithProp = arrayOf("test", props)
-    val manifestJson = JSONObject(mapOf<String, Any>("plugins" to arrayOf(pluginWithProp)))
+    val manifestJson = JSONObject("{\"plugins\":[[\"test\",{\"nested\":{\"insideNested\":true}}]]}")
     val manifest = Manifest.fromManifestJson(manifestJson)
     val result = manifest.getPluginProperties("test")
     Truth.assertThat(result).isNotNull()

package/ios/EXManifests/BaseLegacyManifest.swift

@@ -5,11 +5,11 @@ import Foundation
 @objc(EXManifestBaseLegacyManifest)
 @objcMembers
 public class BaseLegacyManifest: Manifest {
-  override func expoClientConfigRootObject() -> [String: Any]? {
+  public override func expoClientConfigRootObject() -> [String: Any]? {
     return rawManifestJSON()
   }
 
-  override func expoGoConfigRootObject() -> [String: Any]? {
+  public override func expoGoConfigRootObject() -> [String: Any]? {
     return rawManifestJSON()
   }
 

package/ios/EXManifests/Manifest.swift

@@ -3,8 +3,6 @@
 // this uses abstract class patterns
 // swiftlint:disable unavailable_function
 
-// swiftlint:disable type_body_length file_length
-
 import Foundation
 import UIKit
 
@@ -121,11 +119,11 @@ public class Manifest: NSObject {
     preconditionFailure("Must override in concrete class")
   }
 
-  func expoGoConfigRootObject() -> [String: Any]? {
+  public func expoGoConfigRootObject() -> [String: Any]? {
     preconditionFailure("Must override in concrete class")
   }
 
-  func expoClientConfigRootObject() -> [String: Any]? {
+  public func expoClientConfigRootObject() -> [String: Any]? {
     preconditionFailure("Must override in concrete class")
   }
 
@@ -150,6 +148,10 @@ public class Manifest: NSObject {
     return expoClientConfigRootObject()?.optionalValue(forKey: "revisionId")
   }
 
+  public func getMetadata() -> [String: Any]? {
+    return rawManifestJSONInternal.optionalValue(forKey: "metadata")
+  }
+
   public func slug() -> String? {
     return expoClientConfigRootObject()?.optionalValue(forKey: "slug")
   }
@@ -198,10 +200,6 @@ public class Manifest: NSObject {
     return expoGoConfigRootObject()?.optionalValue(forKey: "developer")
   }
 
-  public func logUrl() -> String? {
-    return expoGoConfigRootObject()?.optionalValue(forKey: "logUrl")
-  }
-
   public func facebookAppId() -> String? {
     return expoClientConfigRootObject()?.optionalValue(forKey: "facebookAppId")
   }
@@ -315,9 +313,8 @@ public class Manifest: NSObject {
       let sdkMajorVersion = expoGoSDKMajorVersion()
       if sdkMajorVersion > 0 && sdkMajorVersion < 48 {
         return "jsc"
-      } else {
-        return "hermes"
       }
+      return "hermes"
     }
     return jsEngine
   }
@@ -337,8 +334,14 @@ public class Manifest: NSObject {
           return nil
         }
         if let valueArray = value as? [Any],
-          let name = valueArray[0] as? String, let props = valueArray[1] as? [String: Any] {
-          return .withProps((name, props))
+          let name = valueArray[0] as? String {
+          if valueArray.count > 1 {
+            guard let props = valueArray[1] as? [String: Any] else {
+              return .withoutProps((name))
+            }
+            return .withProps((name, props))
+          }
+          return .withoutProps((name))
         }
         if let value = value as? String {
           return .withoutProps(value)
@@ -411,3 +414,5 @@ public class Manifest: NSObject {
     return nil
   }
 }
+
+// swiftlint:enable unavailable_function

package/ios/EXManifests/NewManifest.swift

@@ -73,11 +73,11 @@ public class NewManifest: Manifest {
     return launchAsset().requiredValue(forKey: "url")
   }
 
-  override func expoClientConfigRootObject() -> [String: Any]? {
+  public override func expoClientConfigRootObject() -> [String: Any]? {
     return extra()?.optionalValue(forKey: "expoClient")
   }
 
-  override func expoGoConfigRootObject() -> [String: Any]? {
+  public override func expoGoConfigRootObject() -> [String: Any]? {
     return extra()?.optionalValue(forKey: "expoGo")
   }
 }

package/ios/Tests/BareManifestSpec.swift

@@ -48,7 +48,6 @@ class BareManifestSpec : ExpoSpec {
         expect(manifest.orientation()).to(beNil())
         expect(manifest.experiments()).to(beNil())
         expect(manifest.developer()).to(beNil())
-        expect(manifest.logUrl()).to(beNil())
         expect(manifest.facebookAppId()).to(beNil())
         expect(manifest.facebookApplicationName()).to(beNil())
         expect(manifest.facebookAutoInitEnabled()) == false

package/ios/Tests/LegacyManifestSpec.swift

@@ -48,7 +48,6 @@ class LegacyManifestSpec : ExpoSpec {
         expect(manifest.orientation()).to(beNil())
         expect(manifest.experiments()).to(beNil())
         expect(manifest.developer()).to(beNil())
-        expect(manifest.logUrl()).to(beNil())
         expect(manifest.facebookAppId()).to(beNil())
         expect(manifest.facebookApplicationName()).to(beNil())
         expect(manifest.facebookAutoInitEnabled()) == false

package/ios/Tests/ManifestSpec.swift

@@ -12,7 +12,7 @@ final class ManifestSpec: ExpoSpec {
         var manifest = ManifestFactory.manifest(forManifestJSON: manifestJson)
         expect(manifest.getPluginProperties(packageName: "test")).to(beNil())
 
-        manifestJson = ["plugins": []]
+        manifestJson = ["plugins": [] as [Any]]
         manifest = ManifestFactory.manifest(forManifestJSON: manifestJson)
         expect(manifest.getPluginProperties(packageName: "test")).to(beNil())
 
@@ -28,11 +28,17 @@ final class ManifestSpec: ExpoSpec {
       }
 
       it("should return matched plugin properties") {
-        let manifestJson = ["plugins": [["test", ["foo": "bar"]]]]
+        let manifestJson = ["plugins": [["test", ["foo": "bar"]] as [Any]]]
         let manifest = ManifestFactory.manifest(forManifestJSON: manifestJson)
         let props = manifest.getPluginProperties(packageName: "test")
         expect(props as? [String: String]) == ["foo": "bar"]
       }
+
+      it("should not crash with array with name and no props") {
+        let manifestJson = ["plugins": [["test"]]]
+        let manifest = ManifestFactory.manifest(forManifestJSON: manifestJson)
+        expect(manifest.getPluginProperties(packageName: "test")).to(beNil())
+      }
     }
   }
 }

package/ios/Tests/NewManifestSpec.swift

@@ -43,7 +43,6 @@ class NewManifestSpec : ExpoSpec {
         expect(manifest.orientation()).to(beNil())
         expect(manifest.experiments()).to(beNil())
         expect(manifest.developer()).to(beNil())
-        expect(manifest.logUrl()).to(beNil())
         expect(manifest.facebookAppId()).to(beNil())
         expect(manifest.facebookApplicationName()).to(beNil())
         expect(manifest.facebookAutoInitEnabled()) == false

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-manifests",
-  "version": "0.7.1",
+  "version": "0.8.1",
   "description": "Code to parse and use Expo and Expo Updates manifests.",
   "main": "index.js",
   "keywords": [
@@ -20,7 +20,7 @@
   "license": "MIT",
   "homepage": "https://docs.expo.dev/versions/latest/sdk/module-template",
   "dependencies": {
-    "expo-json-utils": "~0.7.0"
+    "expo-json-utils": "~0.8.0"
   },
-  "gitHead": "554e8b88a1a5d7f54c67cdb66e928a0adfb8731d"
+  "gitHead": "2240630a92eb79a4e4bf73e1439916c394876478"
 }