Merge pull request facebook#1825 from zpao/jsx-sourcemap

Add support for inline sourcemaps in jsx executable

Author: zpao

bin/jsx

@@ -2,8 +2,7 @@
 // -*- mode: js -*-
 "use strict";
 
-var visitors = require('../vendor/fbtransform/visitors');
-var transform = require('jstransform').transform;
+var transform = require('../main').transform;
 
 require('commoner').version(
   require('../package.json').version
@@ -12,13 +11,14 @@ require('commoner').version(
 }).option(
   '--harmony',
   'Turns on JS transformations such as ES6 Classes etc.'
+).option(
+  '--source-map-inline',
+  'Embed inline sourcemap in transformed source'
 ).process(function(id, source) {
   // This is where JSX, ES6, etc. desugaring happens.
-  var visitorList;
-  if (this.options.harmony) {
-    visitorList = visitors.getAllVisitors();
-  } else {
-    visitorList = visitors.transformVisitors.react;
-  }
-  return transform(visitorList, source).code;
+  var options = {
+    harmony: this.options.harmony,
+    sourceMap: this.options.sourceMapInline
+  };
+  return transform(source, options);
 });

npm-react-tools/README.md

@@ -26,6 +26,7 @@ This package installs a `jsx` executable that can be used to transform JSX into
       --source-charset <utf8 | win1252 | ...>  Charset of source (default: utf8)
       --output-charset <utf8 | win1252 | ...>  Charset of output (default: utf8)
       --harmony                                Turns on JS transformations such as ES6 Classes etc.
+      --source-map-inline                      Embed inline sourcemap in transformed source
 
 ## API