From 45217a0953d005cbe1645f110f8bab799197a707 Mon Sep 17 00:00:00 2001
From: Jason Kohles <jkohles@palantir.com>
Date: Thu, 13 Mar 2014 14:18:45 -0400
Subject: [PATCH] Make error messages from require() more useful

---
 src/main/js/lib/require.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/main/js/lib/require.js b/src/main/js/lib/require.js
index 90830dd..35d8212 100644
--- a/src/main/js/lib/require.js
+++ b/src/main/js/lib/require.js
@@ -244,7 +244,9 @@ When resolving module names to file paths, ScriptCraft uses the following rules.
     try {
       compiledWrapper = eval(code);
     } catch (e) {
-      throw 'Error:' + e + ' while evaluating module ' + canonizedFilename;
+      throw new Error( "Error evaluating module " + path
+        + " at " + canonizedFilename + " line #" + e.lineNumber
+        + ". Error was: " + e.message, canonizedFilename, e.lineNumber );
     }
     var __dirname = '' + file.parentFile.canonicalPath;
     var parameters = [
@@ -259,7 +261,9 @@ When resolving module names to file paths, ScriptCraft uses the following rules.
         .apply(moduleInfo.exports,  /* this */
                parameters);   
     } catch (e) {
-      throw 'Error:' + e + ' while executing module ' + canonizedFilename;
+      throw new Error( "Error executing module " + path
+        + " at " + canonizedFilename + " line #" + e.lineNumber
+        + ". Error was: " + e.message, canonizedFilename, e.lineNumber )
     }
     if ( hooks ) { 
       hooks.loaded( canonizedFilename );