[eas-cli] Silence all non-command output in non-interactive mode of e…

…as env:exec
expo · Jan 6, 2025 · 09c685e · 09c685e
1 parent 59bbe19
commit 09c685e
Showing 1 changed file with 50 additions and 15 deletions.
diff --git a/packages/eas-cli/src/commands/env/exec.ts b/packages/eas-cli/src/commands/env/exec.ts
@@ -72,12 +72,20 @@ export default class EnvExec extends EasCommand {
       graphqlClient,
       projectId,
       environment,
+      nonInteractive: parsedFlags.nonInteractive,
     });
 
-    await this.runCommandWithEnvVarsAsync({
-      command: parsedFlags.command,
-      environmentVariables,
-    });
+    if (parsedFlags.nonInteractive) {
+      await this.runCommandNonInteractiveWithEnvVarsAsync({
+        command: parsedFlags.command,
+        environmentVariables,
+      });
+    } else {
+      await this.runCommandWithEnvVarsAsync({
+        command: parsedFlags.command,
+        environmentVariables,
+      });
+    }
   }
 
   private sanitizeFlagsAndArgs(
@@ -103,6 +111,29 @@ export default class EnvExec extends EasCommand {
     };
   }
 
+  private async runCommandNonInteractiveWithEnvVarsAsync({
+    command,
+    environmentVariables,
+  }: {
+    command: string;
+    environmentVariables: Record<string, string>;
+  }): Promise<void> {
+    try {
+      const { stdout } = await spawnAsync('bash', ['-c', command], {
+        env: {
+          ...process.env,
+          ...environmentVariables,
+        },
+      });
+      Log.log(stdout);
+    } catch (error: any) {
+      if (error.stderr) {
+        Log.error(error.stderr);
+      }
+      throw error;
+    }
+  }
+
   private async runCommandWithEnvVarsAsync({
     command,
     environmentVariables,
@@ -147,10 +178,12 @@ export default class EnvExec extends EasCommand {
     graphqlClient,
     projectId,
     environment,
+    nonInteractive,
   }: {
     graphqlClient: ExpoGraphqlClient;
     projectId: string;
     environment: EnvironmentVariableEnvironment;
+    nonInteractive: boolean;
   }): Promise<Record<string, string>> {
     const environmentVariablesQueryResult =
       await EnvironmentVariablesQuery.byAppIdWithSensitiveAsync(graphqlClient, {
@@ -162,18 +195,20 @@ export default class EnvExec extends EasCommand {
       ({ value }) => !!value
     );
 
-    if (nonSecretEnvironmentVariables.length > 0) {
-      Log.log(
-        `Environment variables with visibility "Plain text" and "Sensitive" loaded from the "${environment.toLowerCase()}" environment on EAS: ${nonSecretEnvironmentVariables
-          .map(e => e.name)
-          .join(', ')}.`
-      );
-    } else {
-      Log.log(
-        `No environment variables with visibility "Plain text" and "Sensitive" found for the "${environment.toLowerCase()}" environment on EAS.`
-      );
+    if (!nonInteractive) {
+      if (nonSecretEnvironmentVariables.length > 0) {
+        Log.log(
+          `Environment variables with visibility "Plain text" and "Sensitive" loaded from the "${environment.toLowerCase()}" environment on EAS: ${nonSecretEnvironmentVariables
+            .map(e => e.name)
+            .join(', ')}.`
+        );
+      } else {
+        Log.log(
+          `No environment variables with visibility "Plain text" and "Sensitive" found for the "${environment.toLowerCase()}" environment on EAS.`
+        );
+      }
+      Log.newLine();
     }
-    Log.newLine();
 
     const environmentVariables: Record<string, string> = {};
     for (const { name, value } of nonSecretEnvironmentVariables) {