Merge pull request #90 from yarastqt/yarastqt.extends-error

Throw custom YamlParseError while yaml parse

package.json

@@ -32,6 +32,7 @@
     "version": "conventional-changelog -p angular -i CHANGELOG.md -s && git add CHANGELOG.md"
   },
   "dependencies": {
+    "@babel/code-frame": "7.10.4",
     "@oclif/command": "1.5.19",
     "chalk": "4.1.0",
     "chokidar": "3.4.0",
@@ -48,6 +49,7 @@
     "@commitlint/cli": "9.1.2",
     "@commitlint/config-conventional": "9.1.2",
     "@oclif/dev-cli": "1.22.2",
+    "@types/babel__code-frame": "7.0.2",
     "@types/fs-extra": "8.1.0",
     "@types/jest": "25.2.1",
     "@typescript-eslint/eslint-plugin": "2.31.0",

src/cli/build.ts

@@ -86,9 +86,14 @@ export default class Build extends Command {
   }
 
   private async build(config: any) {
-    console.log(`----------------- ${chalk.yellow('Build started')} -----------------`)
-    await build(config)
-    console.log(`\n---------------- ${chalk.green('Build completed')} ----------------`)
+    console.log(`>---------------- ${chalk.yellow('Build started')} ----------------<`)
+    try {
+      await build(config)
+      console.log(`\n>--------------- ${chalk.green('Build completed')} ---------------<`)
+    } catch (error) {
+      console.log(error)
+      console.log(`\n>---------------- ${chalk.red('Build failed')} -----------------<`)
+    }
   }
 
   private emitWatching() {

src/core/yaml-interop.ts

@@ -1,14 +1,39 @@
 import Module from 'module'
 import { readFileSync } from 'fs'
 import YAML from 'yaml'
+import { codeFrameColumns } from '@babel/code-frame'
 
+class YamlParseError extends Error {
+  public location: any
+
+  constructor(fileName: string, source: string, message: string, location: any) {
+    super('')
+    // Override native stack with custom message.
+    this.stack = new Error(
+      [
+        // Take only message without code-frame.
+        `${message.split(/\n/)[0]} (${fileName}:${location.start.line}:${location.start.col})`,
+        codeFrameColumns(source, {
+          start: { line: location.start.line, column: location.start.col },
+        }),
+      ].join('\n'),
+    ).stack
+    this.location = location
+  }
+}
+
+/**
+ * Patched native `require` for importing module with `yaml` extensions.
+ */
 Module.prototype.require = new Proxy(Module.prototype.require, {
   apply(target, thisArg, args) {
     if (/\.ya?ml$/.test(args[0])) {
       const file = readFileSync(args[0], 'utf8')
       try {
-        return YAML.parse(file)
-      } catch (error) {}
+        return YAML.parse(file, { prettyErrors: true })
+      } catch (error) {
+        throw new YamlParseError(args[0], file, error.message, error.linePos)
+      }
     }
     return Reflect.apply(target, thisArg, args)
   },