fix: tweak the message styles

Author: satya164

package.json

@@ -34,7 +34,6 @@
     "inquirer": "^6.2.2",
     "is-git-dirty": "^1.0.0",
     "metro-react-native-babel-preset": "^0.53.1",
-    "ora": "^3.3.0",
     "yargs": "^13.2.2"
   },
   "devDependencies": {
@@ -49,7 +48,6 @@
     "@types/fs-extra": "^5.0.5",
     "@types/glob": "^7.1.1",
     "@types/inquirer": "^6.0.0",
-    "@types/ora": "^3.2.0",
     "@types/yargs": "^12.0.11",
     "commitlint": "^7.5.2",
     "conventional-changelog-cli": "^2.0.12",

src/cli.ts

@@ -1,5 +1,6 @@
 import path from 'path';
 import fs from 'fs-extra';
+import chalk from 'chalk';
 import yargs from 'yargs';
 import inquirer from 'inquirer';
 import cosmiconfig from 'cosmiconfig';
@@ -148,7 +149,11 @@ yargs
       pkg.scripts.prepare = prepare;
     }
 
-    if (pkg.files) {
+    if (
+      pkg.files &&
+      JSON.stringify(pkg.files.slice().sort()) !==
+        JSON.stringify(files.slice().sort())
+    ) {
       const { replace } = await inquirer.prompt({
         type: 'confirm',
         name: 'replace',
@@ -190,7 +195,7 @@ yargs
       }
     }
 
-    console.log('🎉 Your project is configured!');
+    logger.success('Your project is configured!');
   })
   .command('build', 'build files for publishing', {}, async argv => {
     const result = explorer.searchSync();
@@ -236,36 +241,48 @@ yargs
       );
     }
 
+    const report = {
+      info: logger.info,
+      warn: logger.warn,
+      error: logger.error,
+      success: logger.success,
+    };
+
     for (const target of options.targets!) {
       const targetName = Array.isArray(target) ? target[0] : target;
       const targetOptions = Array.isArray(target) ? target[1] : undefined;
 
+      report.info(`Building target ${chalk.blue(targetName)}`);
+
       switch (targetName) {
         case 'commonjs':
           await buildCommonJS({
             root,
-            source: source as string,
+            source: path.resolve(root, source as string),
             output: path.resolve(root, output as string, 'commonjs'),
             options: targetOptions,
+            report,
           });
           break;
         case 'module':
           await buildModule({
             root,
-            source: source as string,
+            source: path.resolve(root, source as string),
             output: path.resolve(root, output as string, 'module'),
             options: targetOptions,
+            report,
           });
           break;
         case 'typescript':
           await buildTypescript({
             root,
-            source: source as string,
+            source: path.resolve(root, source as string),
             output: path.resolve(root, output as string, 'typescript'),
+            report,
           });
           break;
         default:
-          logger.exit(`Invalid target '${target}'.`);
+          logger.exit(`Invalid target ${chalk.blue(targetName)}.`);
       }
     }
   })

src/targets/commonjs.ts

@@ -1,6 +1,5 @@
 import del from 'del';
 import compile from '../utils/compile';
-import * as logger from '../utils/logger';
 import { Input } from '../types';
 
 type Options = Input & {
@@ -12,10 +11,12 @@ export default async function build({
   source,
   output,
   options,
+  report
 }: Options) {
-  logger.info('building files for commonjs target');
+  report.info('Cleaning up previous build');
 
   await del([output]);
+
   await compile({
     root,
     source,
@@ -24,5 +25,6 @@ export default async function build({
       presets: [[require.resolve('metro-react-native-babel-preset')]],
     },
     flow: options && options.flow ? true : false,
+    report
   });
 }

src/targets/module.ts

@@ -1,6 +1,5 @@
 import del from 'del';
 import compile from '../utils/compile';
-import * as logger from '../utils/logger';
 import { Input } from '../types';
 
 type Options = Input & {
@@ -12,10 +11,12 @@ export default async function build({
   source,
   output,
   options,
+  report
 }: Options) {
-  logger.info('building files for module target');
+  report.info('Cleaning up previous build');
 
   await del([output]);
+
   await compile({
     root,
     source,
@@ -29,5 +30,6 @@ export default async function build({
       ],
     },
     flow: options && options.flow ? true : false,
+    report
   });
 }

src/targets/typescript.ts

@@ -1,20 +1,24 @@
+import chalk from 'chalk';
 import path from 'path';
 import child_process from 'child_process';
 import del from 'del';
-import * as logger from '../utils/logger';
 import { Input } from '../types';
 
-export default async function build({ root, output }: Input) {
-  logger.info('building files for typscript target');
+export default async function build({ root, output, report }: Input) {
+  report.info('Cleaning up previous build');
 
   await del([output]);
 
+  report.info(`Generating type definitions with ${chalk.blue('tsc')}`);
+
   child_process.execFileSync(path.join(root, 'node_modules', '.bin', 'tsc'), [
     '--declaration',
     '--emitDeclarationOnly',
     '--outDir',
     output,
   ]);
 
-  logger.info(`wrote definition files to ${path.relative(root, output)}`);
+  report.success(
+    `Wrote definition files to ${chalk.blue(path.relative(root, output))}`
+  );
 }

src/types.ts

@@ -1,13 +1,21 @@
+export type Log = (message: string) => void;
+
 export type Input = {
-  root: string,
-  source: string,
-  output: string,
+  root: string;
+  source: string;
+  output: string;
+  report: {
+    info: Log;
+    warn: Log;
+    success: Log;
+    error: Log;
+  };
 };
 
 export type Target = 'commonjs' | 'module' | 'typescript';
 
 export type Options = {
-  source?: string,
-  output?: string,
-  targets?: Array<Target | [Target, object]>
-}
+  source?: string;
+  output?: string;
+  targets?: Array<Target | [Target, object]>;
+};

src/utils/compile.ts

@@ -1,21 +1,22 @@
 import path from 'path';
 import fs from 'fs-extra';
+import chalk from 'chalk';
 import * as babel from '@babel/core';
 import glob from 'glob';
 import { Input } from '../types';
-import * as logger from './logger';
 
 type Options = Input & {
   options: babel.TransformOptions;
-  flow: boolean,
+  flow: boolean;
 };
 
 export default async function compile({
   root,
   source,
   output,
   options,
-  flow
+  flow,
+  report,
 }: Options) {
   const files = glob.sync('**/*', {
     cwd: source,
@@ -24,6 +25,12 @@ export default async function compile({
     ignore: '**/__tests__/**,**/__fixtures__/**',
   });
 
+  report.info(
+    `Compiling ${chalk.blue(String(files.length))} files in ${chalk.blue(
+      path.relative(root, source)
+    )} with ${chalk.blue('babel')}`
+  );
+
   await Promise.all(
     files.map(async filepath => {
       const outputFilename = path
@@ -68,5 +75,5 @@ export default async function compile({
     })
   );
 
-  logger.info(`built ${files.length} files in ${path.relative(root, source)} to ${path.relative(root, output)}`);
+  report.success(`Wrote files to ${chalk.blue(path.relative(root, output))}`);
 }

src/utils/logger.ts

@@ -1,12 +1,13 @@
 import chalk from 'chalk';
 
 const logger = (type: string, color: Function) => (...messages: unknown[]) => {
-  console.log(color(`[${type}]`), ...messages);
+  console.log(color(chalk.bold(type)), ...messages);
 };
 
-export const info = logger('info', chalk.blue);
-export const warn = logger('warn', chalk.yellow);
-export const error = logger('error', chalk.red);
+export const info = logger('ℹ', chalk.blue);
+export const warn = logger('⚠', chalk.yellow);
+export const error = logger('✖', chalk.red);
+export const success = logger('✓', chalk.green);
 
 export const exit = (...messages: unknown[]) => {
   error(...messages);

yarn.lock

@@ -1074,13 +1074,6 @@
   resolved "https://registry.yarnpkg.com/@types/node/-/node-11.11.6.tgz#df929d1bb2eee5afdda598a41930fe50b43eaa6a"
   integrity sha512-Exw4yUWMBXM3X+8oqzJNRqZSwUAaS4+7NdvHqQuFi/d+synz++xmX3QIf+BFqneW8N31R8Ky+sikfZUXq07ggQ==
 
-"@types/ora@^3.2.0":
-  version "3.2.0"
-  resolved "https://registry.yarnpkg.com/@types/ora/-/ora-3.2.0.tgz#b2f65d1283a8f36d8b0f9ee767e0732a2f429362"
-  integrity sha512-jll99xUKpiFbIFZSQcxm4numfsLaOWBzWNaRk3PvTSE7BPqTzzOCFmS0mQ7m8qkTfmYhuYbehTGsxkvRLPC++w==
-  dependencies:
-    ora "*"
-
 "@types/through@*":
   version "0.0.29"
   resolved "https://registry.yarnpkg.com/@types/through/-/through-0.0.29.tgz#72943aac922e179339c651fa34a4428a4d722f93"
@@ -3808,18 +3801,6 @@ optimist@^0.6.1:
     minimist "~0.0.1"
     wordwrap "~0.0.2"
 
-ora@*, ora@^3.3.0:
-  version "3.3.0"
-  resolved "https://registry.yarnpkg.com/ora/-/ora-3.3.0.tgz#9b3b6b03ba2dac40c7c7ce144cca76a0f1abb351"
-  integrity sha512-DTFh3rat9CftF9ldG++sa7ci6KocASWvm6aqNzuXnl8uNCkDprM39f8oZwnXnWoNIcbLYeBuPrruQ4+0+Bf0PA==
-  dependencies:
-    chalk "^2.4.2"
-    cli-cursor "^2.1.0"
-    cli-spinners "^2.0.0"
-    log-symbols "^2.2.0"
-    strip-ansi "^5.0.0"
-    wcwidth "^1.0.1"
-
 [email protected]:
   version "3.2.0"
   resolved "https://registry.yarnpkg.com/ora/-/ora-3.2.0.tgz#67e98a7e11f7f0ac95deaaaf11bb04de3d09e481"