fix: trap hook error

jdx · jdx · commit a0f770ce3863 · 2018-01-28T12:06:22.000-08:00
diff --git a/src/command.ts b/src/command.ts
@@ -103,7 +103,11 @@ export default abstract class Command {
     } catch (err) {
       if (err.code === 'EEXIT') throw err
       if (this.config && this.config.engine) {
-        await this.config.engine.runHook('error', err)
+        try {
+          await this.config.engine.runHook('error', err)
+        } catch (err) {
+          cli.warn(err, {context: {description: 'error running hook'}})
+        }
       }
       cli.error(err)
     }

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,11 @@ export default abstract class Command {`
`103`	`103`	`} catch (err) {`
`104`	`104`	`if (err.code === 'EEXIT') throw err`
`105`	`105`	`if (this.config && this.config.engine) {`
`106`		`- await this.config.engine.runHook('error', err)`
	`106`	`+ try {`
	`107`	`+ await this.config.engine.runHook('error', err)`
	`108`	`+ } catch (err) {`
	`109`	`+ cli.warn(err, {context: {description: 'error running hook'}})`
	`110`	`+ }`
`107`	`111`	`}`
`108`	`112`	`cli.error(err)`
`109`	`113`	`}`