feat: prefix build logs with target name

Author: satya164

Diff for: packages/react-native-builder-bob/src/index.ts

@@ -12,7 +12,7 @@ import buildModule from './targets/module';
 import buildTypescript from './targets/typescript';
 import buildCodegen from './targets/codegen';
 import customTarget from './targets/custom';
-import type { Options, Report, Target } from './types';
+import type { Options, Target } from './types';
 
 type ArgName = 'target';
 
@@ -58,9 +58,10 @@ yargs
     }
 
     if (!(await fs.pathExists(projectPackagePath))) {
-      logger.exit(
+      logger.error(
         `Couldn't find a 'package.json' file in '${root}'.\n  Are you in a project folder?`
       );
+      process.exit(1);
     }
 
     const pkg = JSON.parse(await fs.readFile(projectPackagePath, 'utf-8'));
@@ -98,10 +99,10 @@ yargs
     }
 
     if (!entryFile) {
-      logger.exit(
+      logger.error(
         `Couldn't find a 'index.js'. 'index.ts' or 'index.tsx' file under '${source}'.\n  Please re-run the CLI after creating it.`
       );
-      return;
+      process.exit(1);
     }
 
     pkg.devDependencies = Object.fromEntries(
@@ -463,71 +464,56 @@ yargs
     const result = await explorer.search();
 
     if (!result?.config) {
-      logger.exit(
+      logger.error(
         `No configuration found. Run '${argv.$0} init' to create one automatically.`
       );
+      process.exit(1);
     }
 
     const options: Options = result!.config;
 
     if (!options.targets?.length) {
-      logger.exit(
+      logger.error(
         `No targets found in the configuration in '${path.relative(
           root,
           result!.filepath
         )}'.`
       );
+      process.exit(1);
     }
 
     const source = options.source;
 
     if (!source) {
-      logger.exit(
+      logger.error(
         `No source option found in the configuration in '${path.relative(
           root,
           result!.filepath
         )}'.`
       );
+      process.exit(1);
     }
 
     const output = options.output;
 
     if (!output) {
-      logger.exit(
+      logger.error(
         `No source option found in the configuration in '${path.relative(
           root,
           result!.filepath
         )}'.`
       );
+      process.exit(1);
     }
 
     const exclude =
       options.exclude ?? '**/{__tests__,__fixtures__,__mocks__}/**';
 
-    const report = {
-      info: logger.info,
-      warn: logger.warn,
-      error: logger.error,
-      success: logger.success,
-    };
-
     if (argv.target != null) {
-      buildTarget(
-        argv.target,
-        report,
-        source as string,
-        output as string,
-        exclude
-      );
+      buildTarget(argv.target, source as string, output as string, exclude);
     } else {
       for (const target of options.targets!) {
-        buildTarget(
-          target,
-          report,
-          source as string,
-          output as string,
-          exclude
-        );
+        buildTarget(target, source as string, output as string, exclude);
       }
     }
   })
@@ -537,15 +523,14 @@ yargs
 
 async function buildTarget(
   target: Exclude<Options['targets'], undefined>[number],
-  report: Report,
   source: string,
   output: string,
   exclude: string
 ) {
   const targetName = Array.isArray(target) ? target[0] : target;
   const targetOptions = Array.isArray(target) ? target[1] : undefined;
 
-  report.info(`Building target ${kleur.blue(targetName)}`);
+  const report = logger.grouped(targetName);
 
   switch (targetName) {
     case 'commonjs':
@@ -594,6 +579,7 @@ async function buildTarget(
       });
       break;
     default:
-      logger.exit(`Invalid target ${kleur.blue(targetName)}.`);
+      logger.error(`Invalid target ${kleur.blue(targetName)}.`);
+      process.exit(1);
   }
 }

Diff for: packages/react-native-builder-bob/src/targets/codegen/patches/patchCodegenAndroidPackage.test.ts

@@ -19,7 +19,7 @@ const mockPackageJson = {
 const mockReport: Report = {
   info: console.log,
   warn: console.log,
-  error: console.log,
+  error: console.error,
   success: console.log,
 };
 

Diff for: packages/react-native-builder-bob/src/utils/logger.ts

@@ -4,9 +4,16 @@ const logger =
   (
     type: string,
     color: kleur.Color,
+    group?: string,
     stream: NodeJS.WriteStream = process.stdout
   ) =>
   (...messages: unknown[]) => {
+    if (group) {
+      messages.unshift(
+        `${kleur.gray('[')}${kleur.blue(group)}${kleur.gray(']')}`
+      );
+    }
+
     const message = `${color(kleur.bold(type))} ${messages
       .map((message) => {
         if (typeof message === 'string') {
@@ -22,10 +29,14 @@ const logger =
 
 export const info = logger('ℹ', kleur.blue);
 export const warn = logger('⚠', kleur.yellow);
-export const error = logger('✖', kleur.red, process.stderr);
+export const error = logger('✖', kleur.red, undefined, process.stderr);
 export const success = logger('✔', kleur.green);
 
-export const exit = (...messages: unknown[]) => {
-  error(...messages);
-  process.exit(1);
+export const grouped = (label: string) => {
+  return {
+    info: logger('ℹ', kleur.blue, label),
+    warn: logger('⚠', kleur.yellow, label),
+    error: logger('✖', kleur.red, label, process.stderr),
+    success: logger('✔', kleur.green, label),
+  };
 };