[react-packager] Better transform errors

jestSupport/scriptPreprocess.js

@@ -2,8 +2,8 @@
 
 var transformer = require('../packager/transformer.js');
 
-function transformSource(src) {
+function transformSource(src, filename) {
-  return transformer.transform(null, src).code;
+  return transformer.transform(src, filename).code;
 }
 
 module.exports = {
@@ -15,7 +15,7 @@ module.exports = {
     }
 
     try {
-      return transformSource(src);
+      return transformSource(src, fileName);
     } catch(e) {
       throw new Error('\nError transforming file:\n  js/' +
         (fileName.split('/js/')[1] || fileName) + ':' + e.lineNumber + ': \'' +

packager/react-packager/src/JSTransformer/__tests__/Transformer-test.js

@@ -35,7 +35,7 @@ describe('Transformer', function() {
       callback(null, 'content');
     });
 
-    return new Transformer(OPTIONS).loadFileAndTransform([], 'file', {})
+    return new Transformer(OPTIONS).loadFileAndTransform('file')
       .then(function(data) {
         expect(data).toEqual({
           code: 'transformed',
@@ -58,7 +58,7 @@ describe('Transformer', function() {
       callback(null, {error: esprimaError});
     });
 
-    return new Transformer(OPTIONS).loadFileAndTransform([], 'foo-file.js', {})
+    return new Transformer(OPTIONS).loadFileAndTransform('foo-file.js')
       .catch(function(error) {
         expect(error.type).toEqual('TransformError');
         expect(error.snippet).toEqual([

packager/react-packager/src/JSTransformer/index.js

@@ -78,11 +78,7 @@ Transformer.prototype.invalidateFile = function(filePath) {
   this._cache.invalidate(filePath);
 };
 
-Transformer.prototype.loadFileAndTransform = function(
+Transformer.prototype.loadFileAndTransform = function(filePath) {
-  transformSets,
-  filePath,
-  options
-) {
   if (this._failedToStart) {
     return this._failedToStart;
   }
@@ -92,15 +88,14 @@ Transformer.prototype.loadFileAndTransform = function(
     return readFile(filePath)
       .then(function(buffer) {
         var sourceCode = buffer.toString();
-        var opts = _.extend({}, options, {filename: filePath});
+
         return q.nfbind(workers)({
-          transformSets: transformSets,
           sourceCode: sourceCode,
-          options: opts,
+          filename: filePath,
         }).then(
           function(res) {
             if (res.error) {
-              throw formatEsprimaError(res.error, filePath, sourceCode);
+              throw formatError(res.error, filePath, sourceCode);
             }
 
             return {
@@ -117,11 +112,26 @@ Transformer.prototype.loadFileAndTransform = function(
 function TransformError() {}
 util.inherits(TransformError, SyntaxError);
 
-function formatEsprimaError(err, filename, source) {
+function formatError(err, filename, source) {
-  if (!(err.lineNumber && err.column)) {
+  if (err.lineNumber && err.column) {
-    return err;
+    return formatEsprimaError(err, filename, source);
+  } else {
+    return formatGenericError(err, filename, source);
   }
+}
 
+function formatGenericError(err, filename) {
+  var msg = 'TransformError: ' + filename + ': ' + err.message;
+  var error = new TransformError();
+  var stack = err.stack.split('\n').slice(0, -1);
+  stack.push(msg);
+  error.stack = stack.join('\n');
+  error.message = msg;
+  error.type = 'TransformError';
+  return error;
+}
+
+function formatEsprimaError(err, filename, source) {
   var stack = err.stack.split('\n');
   stack.shift();
 

packager/react-packager/src/Packager/__tests__/Packager-test.js

@@ -49,7 +49,7 @@ describe('Packager', function() {
     });
 
     require('../../JSTransformer').prototype.loadFileAndTransform
-      .mockImpl(function(tsets, path) {
+      .mockImpl(function(path) {
         return q({
           code: 'transformed ' + path,
           sourceCode: 'source ' + path,

packager/react-packager/src/Packager/index.js

@@ -126,9 +126,7 @@ Packager.prototype.getDependencies = function(main) {
 Packager.prototype._transformModule = function(module) {
   var resolver = this._resolver;
   return this._transformer.loadFileAndTransform(
-    ['es6'],
+    path.resolve(module.path)
-    path.resolve(module.path),
-    this._opts.transformer || {}
   ).then(function(transformed) {
     return _.extend(
       {},

packager/transformer.js

@@ -16,10 +16,11 @@ var staticTypeSyntax =
 var visitorList = reactVisitors;
 
 
-function transform(transformSets, srcTxt) {
+function transform(srcTxt, filename) {
   var options = {
     es3: true,
-    sourceType: 'nonStrictModule'
+    sourceType: 'nonStrictModule',
+    filename: filename,
   };
 
   // These tranforms mostly just erase type annotations and static typing
@@ -42,8 +43,8 @@ module.exports = function(data, callback) {
   var result;
   try {
     result = transform(
-      data.transformSets,
+      data.sourceCode,
-      data.sourceCode
+      data.filename
     );
   } catch (e) {
     return callback(null, {