Add error reporter to path loaders to print syntax errors

packages/cli/loader.js

@@ -15,25 +15,32 @@ const {
   loadRawFromRoot,
   loadFromRoot,
   error,
+  exitWithError,
   load,
 } = require('@rescripts/utilities')
 
+const errorReporter = (error) => {
+  if (error.code !== "ENOENT" && error.code !== "MODULE_NOT_FOUND") {
+    console.log(error);
+  }
+}
+
 const rootRescript = (() => {
   const configKeyI = findIndex(
     either(equals('--config'), equals('-c')),
     process.argv,
   )
 
   const loaded =
-    configKeyI >= 0
-      ? loadRawFromRoot(process.argv[configKeyI + 1]) ||
-        loadFromRoot(process.argv[configKeyI + 1]) ||
-        load(process.argv[configKeyI + 1])
+    (configKeyI >= 0
+      ? loadRawFromRoot(process.argv[configKeyI + 1], errorReporter) ||
+        loadFromRoot(process.argv[configKeyI + 1], errorReporter) ||
+        load(process.argv[configKeyI + 1], errorReporter)
       : loadFromPackageField('rescripts') ||
-        loadRawFromRoot('.rescriptsrc') ||
-        loadFromRoot('.rescriptsrc') ||
-        error(
-          "You're likely seeing this bug because you haven't defined a root rescript or your root rescript contains a syntactical error. If you're certain of otherwise, please file an issue.",
+        loadRawFromRoot('.rescriptsrc', errorReporter) ||
+        loadFromRoot('.rescriptsrc', errorReporter)) ||
+        exitWithError(
+          "Rescripts ran into an error. Either your root rescript isn't defined or it contains syntactical errors. If you're certain of otherwise, please file an issue.",
         )
 
   switch (type(loaded)) {
@@ -99,7 +106,7 @@ const gatherPipes = (scope, rescript = rootRescript) =>
         }
 
         default: {
-          error(
+          exitWithError(
             'It seems that your Rescripts configuration is invalid. Please refer to the docs or post an issue if you believe this to be an internal bug. Thank you!',
           )
           return

packages/utilities/errors.js

@@ -1,8 +1,13 @@
 const error = message => {
   console.error(message)
+}
+
+const exitWithError = message => {
+  error(message)
   process.exit(1)
 }
 
 module.exports = {
   error,
+  exitWithError,
 }

packages/utilities/paths.js

@@ -66,10 +66,11 @@ const paths = {
   ),
 }
 
-const makeSafe = loader => p => {
+const makeSafe = loader => (p, r) => {
   try {
     return loader(p)
   } catch (e) {
+    r?.(e)
     return null
   }
 }
@@ -86,11 +87,7 @@ const resolve = makeSafe(p => {
   }
 })
 
-const createFromLoader = (loader, prefix) =>
-  pipe(
-    m => join(prefix, m),
-    loader,
-  )
+const createFromLoader = (loader, prefix) => (m, r) => loader(join(prefix, m), r)
 
 const {root, reactScriptsNodeModules} = paths
 const loadFromRoot = createFromLoader(load, root)