Revert "[eas-cli] Use expo-updates runtime version CLI to generate ru…

…ntime ve…" (#2264)

This reverts commit 561e3bb.

CHANGELOG.md

@@ -12,7 +12,6 @@ This is the log of notable changes to EAS CLI and related packages.
 
 - Fix expo-updates package version detection for canaries. ([#2243](https://github.com/expo/eas-cli/pull/2243) by [@wschurman](https://github.com/wschurman))
 - Add missing `config` property to `eas.json` schema. ([#2248](https://github.com/expo/eas-cli/pull/2248) by [@sjchmiela](https://github.com/sjchmiela))
-- Use expo-updates runtime version CLI to generate runtime versions. ([#2251](https://github.com/expo/eas-cli/pull/2251) by [@wschurman](https://github.com/wschurman))
 
 ### 🧹 Chores
 

packages/eas-cli/src/build/metadata.ts

@@ -1,3 +1,4 @@
+import { Updates } from '@expo/config-plugins';
 import { Metadata, Platform, sanitizeMetadata } from '@expo/eas-build-job';
 import { IosEnterpriseProvisioning } from '@expo/eas-json';
 import fs from 'fs-extra';
@@ -14,7 +15,6 @@ import {
   isClassicUpdatesSupportedAsync,
   isExpoUpdatesInstalled,
 } from '../project/projectUtils';
-import { resolveRuntimeVersionAsync } from '../project/resolveRuntimeVersionAsync';
 import {
   readChannelSafelyAsync as readAndroidChannelSafelyAsync,
   readReleaseChannelSafelyAsync as readAndroidReleaseChannelSafelyAsync,
@@ -37,7 +37,9 @@ export async function collectMetadataAsync<T extends Platform>(
     workflow: ctx.workflow,
     credentialsSource: ctx.buildProfile.credentialsSource,
     sdkVersion: ctx.exp.sdkVersion,
-    runtimeVersion: (await resolveRuntimeVersionAsync(ctx)) ?? undefined,
+    runtimeVersion:
+      (await Updates.getRuntimeVersionNullableAsync(ctx.projectDir, ctx.exp, ctx.platform)) ??
+      undefined,
     reactNativeVersion: await getReactNativeVersionAsync(ctx.projectDir),
     ...channelOrReleaseChannel,
     distribution,

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

@@ -31,11 +31,7 @@ import {
   shouldUseVersionedExpoCLI,
   shouldUseVersionedExpoCLIWithExplicitPlatforms,
 } from '../utils/expoCli';
-import {
-  ExpoUpdatesCLIInvalidCommandError,
-  ExpoUpdatesCLIModuleNotFoundError,
-  expoUpdatesCommandAsync,
-} from '../utils/expoUpdatesCli';
+import { expoUpdatesCommandAsync } from '../utils/expoUpdatesCli';
 import chunk from '../utils/expodash/chunk';
 import { truthy } from '../utils/expodash/filter';
 import uniqBy from '../utils/expodash/uniqBy';
@@ -721,31 +717,31 @@ async function getRuntimeVersionForPlatformAsync({
     return 'UNVERSIONED';
   }
 
-  try {
-    const resolvedRuntimeVersionJSONResult = await expoUpdatesCommandAsync(projectDir, [
-      'runtimeversion:resolve',
-      '--platform',
-      platform,
-    ]);
-    const runtimeVersionResult = JSON.parse(resolvedRuntimeVersionJSONResult);
-    if (runtimeVersionResult.fingerprintSources) {
-      Log.debug(`Resolved fingeprint runtime version for platform "${platform}". Sources:`);
-      Log.debug(runtimeVersionResult.fingerprintSources);
-    }
-    return nullthrows(runtimeVersionResult.runtimeVersion);
-  } catch (e: any) {
-    // if it's a known set of errors thrown by the CLI it means that we need to default back to the
-    // previous behavior, otherwise we throw the error since something is wrong
-    if (
-      !(e instanceof ExpoUpdatesCLIModuleNotFoundError) &&
-      !(e instanceof ExpoUpdatesCLIInvalidCommandError)
-    ) {
-      throw e;
-    }
-  }
-
   const runtimeVersion = exp[platform]?.runtimeVersion ?? exp.runtimeVersion;
   if (typeof runtimeVersion === 'object') {
+    const policy = runtimeVersion.policy;
+
+    if (policy === 'fingerprintExperimental') {
+      // log to inform the user that the fingerprint has been calculated
+      Log.warn(
+        `Calculating native fingerprint for platform ${platform} using current state of the "${platform}" directory. ` +
+          `If the fingerprint differs from the build's fingerint, ensure the state of your project is consistent ` +
+          `(repository is clean, ios and android native directories are in the same state as the build if applicable).`
+      );
+
+      const fingerprintRawString = await expoUpdatesCommandAsync(projectDir, [
+        'fingerprint:generate',
+        '--platform',
+        platform,
+      ]);
+      const fingerprintObject = JSON.parse(fingerprintRawString);
+      const hash = nullthrows(
+        fingerprintObject.hash,
+        'invalid response from expo-update CLI for fingerprint generation'
+      );
+      return hash;
+    }
+
     const workflow = await resolveWorkflowAsync(
       projectDir,
       platform as EASBuildJobPlatform,

packages/eas-cli/src/rollout/actions/CreateRollout.ts

@@ -1,3 +1,4 @@
+import { Updates } from '@expo/config-plugins';
 import assert from 'assert';
 
 import { SelectRuntime } from './SelectRuntime';
@@ -23,7 +24,6 @@ import {
 } from '../../graphql/queries/ChannelQuery';
 import { UpdateQuery } from '../../graphql/queries/UpdateQuery';
 import Log from '../../log';
-import { resolveRuntimeVersionAsync } from '../../project/resolveRuntimeVersionAsync';
 import { confirmAsync, promptAsync } from '../../prompts';
 import { truthy } from '../../utils/expodash/filter';
 import {
@@ -275,7 +275,7 @@ export class CreateRollout implements EASUpdateAction<UpdateChannelBasicInfoFrag
     const runtimes = (
       await Promise.all(
         platforms.map(platform =>
-          resolveRuntimeVersionAsync({ projectDir: ctx.app.projectDir, exp: ctx.app.exp, platform })
+          Updates.getRuntimeVersionAsync(ctx.app.projectDir, ctx.app.exp, platform)
         )
       )
     ).filter(truthy);

packages/eas-cli/src/utils/expoUpdatesCli.ts

@@ -1,10 +1,8 @@
 import spawnAsync from '@expo/spawn-async';
+import chalk from 'chalk';
 import resolveFrom, { silent as silentResolveFrom } from 'resolve-from';
 
-import { link } from '../log';
-
-export class ExpoUpdatesCLIModuleNotFoundError extends Error {}
-export class ExpoUpdatesCLIInvalidCommandError extends Error {}
+import Log, { link } from '../log';
 
 export async function expoUpdatesCommandAsync(projectDir: string, args: string[]): Promise<string> {
   let expoUpdatesCli;
@@ -14,7 +12,7 @@ export async function expoUpdatesCommandAsync(projectDir: string, args: string[]
       resolveFrom(projectDir, 'expo-updates/bin/cli.js');
   } catch (e: any) {
     if (e.code === 'MODULE_NOT_FOUND') {
-      throw new ExpoUpdatesCLIModuleNotFoundError(
+      throw new Error(
         `The \`expo-updates\` package was not found. Follow the installation directions at ${link(
           'https://docs.expo.dev/bare/installing-expo-modules/'
         )}`
@@ -23,16 +21,20 @@ export async function expoUpdatesCommandAsync(projectDir: string, args: string[]
     throw e;
   }
 
-  try {
-    return (await spawnAsync(expoUpdatesCli, args)).stdout;
-  } catch (e: any) {
-    if (e.stderr) {
-      if ((e.stderr as string).includes('Invalid command')) {
-        throw new ExpoUpdatesCLIInvalidCommandError(
-          `The command specified by ${args} was not valid in the \`expo-updates\` CLI.`
-        );
-      }
-    }
-    throw e;
+  const spawnPromise = spawnAsync(expoUpdatesCli, args, {
+    stdio: ['inherit', 'pipe', 'pipe'], // inherit stdin so user can install a missing expo-cli from inside this command
+  });
+  const {
+    child: { stderr },
+  } = spawnPromise;
+  if (!stderr) {
+    throw new Error('Failed to spawn expo-updates cli');
   }
+  stderr.on('data', data => {
+    for (const line of data.toString().trim().split('\n')) {
+      Log.warn(`${chalk.gray('[expo-cli]')} ${line}`);
+    }
+  });
+  const result = await spawnPromise;
+  return result.stdout;
 }