address review

Author: vonovak

packages/eas-cli/src/project/discourageExpoGoForProd.ts

@@ -1,43 +1,43 @@
 import { Workflow } from '@expo/eas-build-job';
 import chalk from 'chalk';
 import getenv from 'getenv';
-import resolveFrom from 'resolve-from';
 
 import { resolveWorkflowPerPlatformAsync } from './workflow';
+import { isExpoDevClientInstalled } from '../build/utils/devClient';
 import Log, { learnMore } from '../log';
 import type { ProfileData } from '../utils/profiles';
 import type { Client } from '../vcs/vcs';
 
 const suppressionEnvVarName = 'EAS_BUILD_NO_EXPO_GO_WARNING';
 
-export const discourageExpoGoForProd = (
+export async function discourageExpoGoForProdAsync(
   buildProfiles: ProfileData<'build'>[] | undefined,
   projectDir: string,
   vcsClient: Client
-): void => {
-  detectExpoGoProdBuildAsync(buildProfiles, projectDir, vcsClient)
-    .then(usesExpoGo => {
-      if (usesExpoGo) {
-        Log.newLine();
-        Log.warn(
-          `⚠️ It appears you're trying to build an app based on Expo Go for production. Expo Go is not a suitable environment for production apps.`
-        );
-        Log.warn(
-          learnMore('https://docs.expo.dev/develop/development-builds/expo-go-to-dev-build/', {
-            learnMoreMessage: 'Learn more about converting from Expo Go to a development build',
-            dim: false,
-          })
-        );
-        Log.warn(
-          chalk.dim(`To suppress this warning, set ${chalk.bold(`${suppressionEnvVarName}=true`)}.`)
-        );
-        Log.newLine();
-      }
-    })
-    .catch(err => {
-      Log.warn('Error detecting whether Expo Go is used:', err);
-    });
-};
+): Promise<void> {
+  try {
+    const usesExpoGo = await detectExpoGoProdBuildAsync(buildProfiles, projectDir, vcsClient);
+    if (!usesExpoGo) {
+      return;
+    }
+    Log.newLine();
+    Log.warn(
+      `⚠️ It appears you're trying to build an app based on Expo Go for production. Expo Go is not a suitable environment for production apps.`
+    );
+    Log.warn(
+      learnMore('https://docs.expo.dev/develop/development-builds/expo-go-to-dev-build/', {
+        learnMoreMessage: 'Learn more about converting from Expo Go to a development build',
+        dim: false,
+      })
+    );
+    Log.warn(
+      chalk.dim(`To suppress this warning, set ${chalk.bold(`${suppressionEnvVarName}=true`)}.`)
+    );
+    Log.newLine();
+  } catch (err) {
+    Log.warn('Error detecting whether Expo Go is used:', err);
+  }
+}
 
 export async function detectExpoGoProdBuildAsync(
   buildProfiles: ProfileData<'build'>[] | undefined,
@@ -51,7 +51,7 @@ export async function detectExpoGoProdBuildAsync(
     return false;
   }
 
-  const hasExpoDevClient = checkIfExpoDevClientInstalled(projectDir);
+  const hasExpoDevClient = isExpoDevClientInstalled(projectDir);
   if (hasExpoDevClient) {
     return false;
   }
@@ -67,7 +67,3 @@ async function checkIfManagedWorkflowAsync(
 
   return workflows.android === Workflow.MANAGED && workflows.ios === Workflow.MANAGED;
 }
-
-function checkIfExpoDevClientInstalled(projectDir: string): boolean {
-  return resolveFrom.silent(projectDir, 'expo-dev-client/package.json') !== undefined;
-}