feat: use xcbeautify for xcodebuild output if installed (react-native-community#1392)

* feat: use xcbeautify for xcodebuild output if installed

The run-ios command already detects and uses xcpretty (https://github.com/xcpretty/xcpretty) if it's available.

This commit adds support for another xcodebuild output formatter, xcbeautify (https://github.com/thii/xcbeautify).

The logic for what is used is as follows:
1. If xcbeautify is available use it
2. If xcbeautify isn't available but xcpretty is use xcpretty
3. If neither are available fallback to raw xcodebuild output

* chore: styling

Co-authored-by: Mike Grabowski <[email protected]>

Author: liamjones

docs/commands.md

@@ -443,7 +443,7 @@ Do not launch packager while building.
 
 #### `--verbose`
 
-Do not use `xcpretty` even if installed.
+Do not use `xcbeautify` or `xcpretty` even if installed.
 
 #### `--port <number>`
 

packages/platform-ios/src/commands/runIOS/index.ts

@@ -313,13 +313,17 @@ function buildProject(
         `(using "xcodebuild ${xcodebuildArgs.join(' ')}")`,
       )}`,
     );
-    let xcpretty: ChildProcess | any;
+    let xcodebuildOutputFormatter: ChildProcess | any;
     if (!args.verbose) {
-      xcpretty =
-        xcprettyAvailable() &&
-        child_process.spawn('xcpretty', [], {
+      if (xcbeautifyAvailable()) {
+        xcodebuildOutputFormatter = child_process.spawn('xcbeautify', [], {
           stdio: ['pipe', process.stdout, process.stderr],
         });
+      } else if (xcprettyAvailable()) {
+        xcodebuildOutputFormatter = child_process.spawn('xcpretty', [], {
+          stdio: ['pipe', process.stdout, process.stderr],
+        });
+      }
     }
     const buildProcess = child_process.spawn(
       'xcodebuild',
@@ -331,8 +335,8 @@ function buildProject(
     buildProcess.stdout.on('data', (data: Buffer) => {
       const stringData = data.toString();
       buildOutput += stringData;
-      if (xcpretty) {
-        xcpretty.stdin.write(data);
+      if (xcodebuildOutputFormatter) {
+        xcodebuildOutputFormatter.stdin.write(data);
       } else {
         if (logger.isVerbose()) {
           logger.debug(stringData);
@@ -347,8 +351,8 @@ function buildProject(
       errorOutput += data;
     });
     buildProcess.on('close', (code: number) => {
-      if (xcpretty) {
-        xcpretty.stdin.end();
+      if (xcodebuildOutputFormatter) {
+        xcodebuildOutputFormatter.stdin.end();
       } else {
         loader.stop();
       }
@@ -454,6 +458,17 @@ function getPlatformName(buildOutput: string) {
   return platformNameMatch[1];
 }
 
+function xcbeautifyAvailable() {
+  try {
+    child_process.execSync('xcbeautify --version', {
+      stdio: [0, 'pipe', 'ignore'],
+    });
+  } catch (error) {
+    return false;
+  }
+  return true;
+}
+
 function xcprettyAvailable() {
   try {
     child_process.execSync('xcpretty --version', {
@@ -600,7 +615,7 @@ export default {
     },
     {
       name: '--verbose',
-      description: 'Do not use xcpretty even if installed',
+      description: 'Do not use xcbeautify or xcpretty even if installed',
     },
     {
       name: '--port <number>',