From dcd9d22056e1c8a953645b21ca81ff9edcc40c85 Mon Sep 17 00:00:00 2001
From: Maciej Stosio <maciekstosio@users.noreply.github.com>
Date: Sat, 29 Jul 2023 22:31:12 +0200
Subject: [PATCH 1/3] Fix crash due to the calling .developer on null

---
 packages/expo/src/config.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/expo/src/config.js b/packages/expo/src/config.js
index 71002730..c61dbec1 100644
--- a/packages/expo/src/config.js
+++ b/packages/expo/src/config.js
@@ -7,7 +7,7 @@ const stringWithLength = require('@bugsnag/core/lib/validators/string-with-lengt
 // If the developer property is not present 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
-const IS_PRODUCTION = !Constants.expoConfig.developer && !Constants.expoGoConfig.developer
+const IS_PRODUCTION = !Constants.expoConfig?.developer && !Constants.expoGoConfig?.developer
 
 // The app can still run in production "mode" in development environments, in which
 // cases the global boolean __DEV__ will be set to true

From 828d9684c790cbaf8c46552d7ad0a9e034267cbc Mon Sep 17 00:00:00 2001
From: Yousif Ahmed <yousif@bugsnag.com>
Date: Wed, 2 Aug 2023 14:13:37 +0100
Subject: [PATCH 2/3] Guard access to `Constants.expoConfig` and
 `Constants.expoGoConfig`

---
 packages/expo/src/notifier.js         |  6 +++---
 packages/expo/test/index.test.js      | 12 ++++++------
 packages/plugin-expo-device/device.js |  2 +-
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/packages/expo/src/notifier.js b/packages/expo/src/notifier.js
index 6a377a3e..016e5a0c 100644
--- a/packages/expo/src/notifier.js
+++ b/packages/expo/src/notifier.js
@@ -28,7 +28,7 @@ const internalPlugins = [
   require('@bugsnag/plugin-expo-device'),
   require('@bugsnag/plugin-expo-app'),
   require('@bugsnag/plugin-console-breadcrumbs'),
-  require('@bugsnag/plugin-network-breadcrumbs')([NET_INFO_REACHABILITY_URL, Constants.expoConfig.logUrl || Constants.expoGoConfig?.logUrl]),
+  require('@bugsnag/plugin-network-breadcrumbs')([NET_INFO_REACHABILITY_URL, Constants.expoConfig?.logUrl || Constants.expoGoConfig?.logUrl]),
   require('@bugsnag/plugin-expo-app-state-breadcrumbs'),
   require('@bugsnag/plugin-expo-connectivity-breadcrumbs'),
   require('@bugsnag/plugin-react-native-orientation-breadcrumbs'),
@@ -45,12 +45,12 @@ const Bugsnag = {
 
     // read the api key from app.json if one is not explicitly passed
     if (!opts.apiKey) {
-      opts.apiKey = Constants.expoConfig.extra?.bugsnag?.apiKey
+      opts.apiKey = Constants.expoConfig?.extra?.bugsnag?.apiKey
     }
 
     // read the version from app.json if one is not explicitly passed
     if (!opts.appVersion) {
-      opts.appVersion = Constants.expoConfig.version
+      opts.appVersion = Constants.expoConfig?.version
     }
 
     const bugsnag = new Client(opts, schema, internalPlugins, { name, version, url })
diff --git a/packages/expo/test/index.test.js b/packages/expo/test/index.test.js
index 80693aa1..3fdf43d2 100644
--- a/packages/expo/test/index.test.js
+++ b/packages/expo/test/index.test.js
@@ -4,7 +4,7 @@ jest.mock('expo-constants', () => ({
   default: {
     platform: {},
     expoConfig: {},
-    expoGoConfig: {}
+    expoGoConfig: null
   }
 }))
 
@@ -12,7 +12,7 @@ jest.mock('../../plugin-expo-device/node_modules/expo-constants', () => ({
   default: {
     platform: {},
     expoConfig: {},
-    expoGoConfig: {}
+    expoGoConfig: null
   }
 }))
 
@@ -22,7 +22,7 @@ jest.mock('../../plugin-expo-app/node_modules/expo-constants', () => ({
   default: {
     platform: {},
     expoConfig: {},
-    expoGoConfig: {}
+    expoGoConfig: null
   }
 }))
 
@@ -244,7 +244,7 @@ describe('expo notifier', () => {
         default: {
           platform: {},
           expoConfig: {},
-          expoGoConfig: {}
+          expoGoConfig: null
         }
       }))
 
@@ -256,7 +256,7 @@ describe('expo notifier', () => {
       jest.mock('expo-constants', () => ({
         default: {
           platform: {},
-          expoConfig: {},
+          expoConfig: null,
           expoGoConfig: {
             developer: {
               tool: 'expo-cli'
@@ -279,7 +279,7 @@ describe('expo notifier', () => {
               tool: 'expo-cli'
             }
           },
-          expoGoConfig: {}
+          expoGoConfig: null
         }
       }))
 
diff --git a/packages/plugin-expo-device/device.js b/packages/plugin-expo-device/device.js
index 9ea630db..86fac499 100644
--- a/packages/plugin-expo-device/device.js
+++ b/packages/plugin-expo-device/device.js
@@ -31,7 +31,7 @@ module.exports = {
       runtimeVersions: {
         reactNative: rnVersion,
         expoApp: Constants.expoVersion,
-        expoSdk: Constants.expoConfig.sdkVersion,
+        expoSdk: Constants.expoConfig?.sdkVersion,
         androidApiLevel: Constants.platform.android ? String(Platform.Version) : undefined
       },
       totalMemory: Device.totalMemory

From a7eb3da271681c762711c0924ee3565f36296b57 Mon Sep 17 00:00:00 2001
From: Yousif Ahmed <yousif@bugsnag.com>
Date: Wed, 2 Aug 2023 18:00:02 +0100
Subject: [PATCH 3/3] update CHANGELOG.md

---
 CHANGELOG.md | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3dab70ca..0cd14664 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## TBD
+
+### Fixed
+
+- Fix a crash in configuration when Constants.expoGoConfig is null [#145](https://github.com/bugsnag/bugsnag-expo/pull/145)
+
 ## v49.0.0 (2023-07-13)
 
 This release adds support for expo 49