feat: use @oclif/errors for errors

BREAKING CHANGE: all the error logging functions have been moved to @oclif/errors

Author: jdx

examples/action.ts

@@ -53,3 +53,4 @@ async function main() {
   cli.error('oh no')
 }
 main()
+.catch(require('@oclif/errors/handle'))

examples/errors.ts

@@ -1,5 +1,4 @@
 import cli from '../src'
 
 cli.warn('uh oh!')
-cli.warn('uh oh!', 'myscope')
 cli.error('uh oh!')

package.json

@@ -21,7 +21,8 @@
     "supports-color": "^5.4.0"
   },
   "devDependencies": {
-    "@oclif/tslint": "^1.1.0",
+    "@oclif/errors": "^1.0.9",
+    "@oclif/tslint": "^1.1.1",
     "@types/ansi-styles": "^2.0.30",
     "@types/chai": "^4.1.3",
     "@types/clean-stack": "^1.3.0",

src/config.ts

@@ -25,7 +25,6 @@ const actionType = (
 const Action = actionType === 'spinner' ? require('./action/spinner').default : require('./action/simple').default
 
 export class Config {
-  logLevel: Levels = 'warn'
   outputLevel: Levels = 'info'
   action: ActionBase = new Action()
   errorsHandled = false
@@ -35,8 +34,6 @@ export class Config {
   set debug(v: boolean) { globals.debug = v }
   get context(): any { return globals.context || {} }
   set context(v: any) { globals.context = v }
-  get errlog(): string | undefined { return globals.errlog }
-  set errlog(v: string | undefined) { globals.errlog = v }
 }
 
 function fetch() {

src/errors.ts

@@ -1,33 +1,19 @@
-import * as EventEmitter from 'events'
+import * as Errors from '@oclif/errors'
+import * as util from 'util'
 
 import {ActionBase} from './action/base'
+import {config, Config} from './config'
 import deps from './deps'
-import Errors, {CLIError, Options as ErrorOptions} from './errors'
 import {ExitError} from './exit'
-import * as Logger from './logger'
-import Output from './output'
 import {IPromptOptions} from './prompt'
 import * as Table from './styled/table'
 
-import {config, Config} from './config'
-
-const e = new EventEmitter()
-const output = Output(e)
-const errors = Errors(e)
-const logger = Logger.default(e)
-
 export const cli = {
   config,
-  trace: output('trace'),
-  debug: output('debug'),
-  info: output('info'),
-  log: output('info'),
-
-  warn: errors('warn'),
-  error: errors('error'),
-  fatal: errors('fatal'),
 
-  exit(code = 1, error?: Error) { throw new ExitError(code, error) },
+  warn: Errors.warn,
+  error: Errors.error,
+  exit: Errors.exit,
 
   get prompt() { return deps.prompt.prompt },
   get confirm() { return deps.prompt.confirm },
@@ -39,19 +25,35 @@ export const cli = {
 
   async done() {
     config.action.stop()
-    await logger.flush()
     // await flushStdout()
-  }
+  },
+
+  trace(format: string, ...args: string[]) {
+    if (this.config.outputLevel === 'trace') {
+      process.stdout.write(util.format(format, ...args) + '\n')
+    }
+  },
+
+  debug(format: string, ...args: string[]) {
+    if (['trace', 'debug'].includes(this.config.outputLevel)) {
+      process.stdout.write(util.format(format, ...args) + '\n')
+    }
+  },
+
+  info(format: string, ...args: string[]) {
+    process.stdout.write(util.format(format, ...args) + '\n')
+  },
+
+  log(format: string, ...args: string[]) {
+    this.info(format, ...args)
+  },
 }
 export default cli
 
 export {
   config,
   ActionBase,
-  CLIError,
   Config,
-  ErrorOptions,
-  Errors,
   ExitError,
   IPromptOptions,
   Table,