Use 'manifest2' when 'manifest' does not exist

When using EAS Update, 'manifest' will be null and 'manifest2' is
defined instead

Author: imjoehaines

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ### TBD
 
+### Fixed
+
+- (expo): preliminary support for the Expo EAS Update. Please send us feedback on how we can improve our support! [kennethlynne](https://github.com/kennethlynne) [#1686](https://github.com/bugsnag/bugsnag-js/pull/1686) [#1695](https://github.com/bugsnag/bugsnag-js/pull/1695)
+
 ### Changed
 
 - (plugin-react-navigation): Allow React Navigation v6 as a peer dependency [#1691](https://github.com/bugsnag/bugsnag-js/pull/1691)

packages/expo/src/config.js

@@ -5,9 +5,14 @@ const Constants = require('expo-constants').default
 
 // If the developer property is not present in the manifest, it means the app is
 // not connected to a development tool and is either a published app running in
-// the Expo client, or a standalone app (we also assume production for a missing
-// manifest, but that should never happen)
-const IS_PRODUCTION = !Constants.manifest || !Constants.manifest.developer
+// the Expo client, or a standalone app
+let IS_PRODUCTION = true
+
+if (Constants.manifest) {
+  IS_PRODUCTION = !Constants.manifest.developer
+} else if (Constants.manifest2) {
+  IS_PRODUCTION = !Constants.manifest2?.extra?.expoGo?.developer
+}
 
 // The app can still run in production "mode" in development environments, in which
 // cases the global boolean __DEV__ will be set to true

packages/expo/src/notifier.js

@@ -43,17 +43,22 @@ const Bugsnag = {
     if (typeof opts === 'string') opts = { apiKey: opts }
     if (!opts) opts = {}
 
-    // attempt to fetch apiKey from app.json if we didn't get one explicitly passed
-    if (!opts.apiKey &&
-      Constants.manifest &&
-      Constants.manifest.extra &&
-      Constants.manifest.extra.bugsnag &&
-      Constants.manifest.extra.bugsnag.apiKey) {
-      opts.apiKey = Constants.manifest.extra.bugsnag.apiKey
+    // read the api key from app.json if one is not explicitly passed
+    if (!opts.apiKey) {
+      if (Constants.manifest?.extra?.bugsnag?.apiKey) {
+        opts.apiKey = Constants.manifest.extra.bugsnag.apiKey
+      } else if (Constants.manifest2?.extra?.expoClient?.extra?.bugsnag?.apiKey) {
+        opts.apiKey = Constants.manifest2.extra.expoClient.extra.bugsnag.apiKey
+      }
     }
 
-    if (!opts.appVersion && Constants.manifest && Constants.manifest.version) {
-      opts.appVersion = Constants.manifest.version
+    // read the version from app.json if one is not explicitly passed
+    if (!opts.appVersion) {
+      if (Constants.manifest?.version) {
+        opts.appVersion = Constants.manifest.version
+      } else if (Constants.manifest2?.extra?.expoClient?.version) {
+        opts.appVersion = Constants.manifest2.extra.expoClient.version
+      }
     }
 
     const bugsnag = new Client(opts, schema, internalPlugins, { name, version, url })

packages/plugin-expo-app/app.js

@@ -29,7 +29,7 @@ module.exports = {
     }
 
     client.addOnSession(session => {
-      if (Constants.manifest.revisionId) {
+      if (Constants.manifest?.revisionId) {
         session.app.codeBundleId = Constants.manifest.revisionId
       } else if (Constants.manifest2?.extra?.expoClient?.revisionId) {
         session.app.codeBundleId = Constants.manifest2.extra.expoClient.revisionId
@@ -49,8 +49,10 @@ module.exports = {
 
       event.addMetadata('app', { nativeBundleVersion, nativeVersionCode })
 
-      if (Constants.manifest.revisionId) {
+      if (Constants.manifest?.revisionId) {
         event.app.codeBundleId = Constants.manifest.revisionId
+      } else if (Constants.manifest2?.extra?.expoClient?.revisionId) {
+        event.app.codeBundleId = Constants.manifest2.extra.expoClient.revisionId
       }
     }, true)
   }