use babel sourcemaps in bundle server

Reviewed By: davidaurelio

Differential Revision: D3841557

fbshipit-source-id: a6d40cf224ba7c2fd0a8eb0f0e2f7cc4bf222bcb

react-packager/src/Bundler/Bundle.js

@@ -21,7 +21,6 @@ class Bundle extends BundleBase {
     super();
     this._sourceMap = false;
     this._sourceMapUrl = sourceMapUrl;
-    this._shouldCombineSourceMaps = false;
     this._numRequireCalls = 0;
     this._dev = dev;
     this._minify = minify;
@@ -41,12 +40,6 @@ class Bundle extends BundleBase {
       minify: this._minify,
       dev: this._dev,
     }).then(({code, map}) => {
-      // If we get a map from the transformer we'll switch to a mode
-      // were we're combining the source maps as opposed to
-      if (!this._shouldCombineSourceMaps && map != null) {
-        this._shouldCombineSourceMaps = true;
-      }
-
       this.replaceModuleAt(
         index, new ModuleTransport({...moduleTransport, code, map}));
     });
@@ -137,8 +130,8 @@ class Bundle extends BundleBase {
     this.getModules().forEach(module => {
       let map = module.map;
 
-      if (module.virtual) {
-        map = generateSourceMapForVirtualModule(module);
+      if (module.virtual || !map) {
+        map = generateMissingSourceMapForModule(module);
       }
 
       if (options.excludeSource) {
@@ -159,25 +152,9 @@ class Bundle extends BundleBase {
 
   getSourceMap(options) {
     super.assertFinalized();
-
-    if (this._shouldCombineSourceMaps) {
     return this._getCombinedSourceMaps(options);
   }
 
-    const mappings = this._getMappings();
-    const modules = this.getModules();
-    const map = {
-      file: this._getSourceMapFile(),
-      sources: modules.map(module => module.sourcePath),
-      version: 3,
-      names: [],
-      mappings: mappings,
-      sourcesContent: options.excludeSource
-        ? [] : modules.map(module => module.sourceCode)
-    };
-    return map;
-  }
-
   getEtag() {
     var eTag = crypto.createHash('md5').update(this.getSource()).digest('hex');
     return eTag;
@@ -189,53 +166,6 @@ class Bundle extends BundleBase {
       : 'bundle.js';
   }
 
-  _getMappings() {
-    const modules = super.getModules();
-
-    // The first line mapping in our package is basically the base64vlq code for
-    // zeros (A).
-    const firstLine = 'AAAA';
-
-    // Most other lines in our mappings are all zeros (for module, column etc)
-    // except for the lineno mappinp: curLineno - prevLineno = 1; Which is C.
-    const line = 'AACA';
-
-    const moduleLines = Object.create(null);
-    let mappings = '';
-    for (let i = 0; i < modules.length; i++) {
-      const module = modules[i];
-      const code = module.code;
-      let lastCharNewLine  = false;
-      moduleLines[module.sourcePath] = 0;
-      for (let t = 0; t < code.length; t++) {
-        if (t === 0 && i === 0) {
-          mappings += firstLine;
-        } else if (t === 0) {
-          mappings += 'AC';
-
-          // This is the only place were we actually don't know the mapping ahead
-          // of time. When it's a new module (and not the first) the lineno
-          // mapping is 0 (current) - number of lines in prev module.
-          mappings += base64VLQ.encode(
-            0 - moduleLines[modules[i - 1].sourcePath]
-          );
-          mappings += 'A';
-        } else if (lastCharNewLine) {
-          moduleLines[module.sourcePath]++;
-          mappings += line;
-        }
-        lastCharNewLine = code[t] === '\n';
-        if (lastCharNewLine) {
-          mappings += ';';
-        }
-      }
-      if (i !== modules.length - 1) {
-        mappings += ';';
-      }
-    }
-    return mappings;
-  }
-
   getJSModulePaths() {
     return this.getModules()
       // Filter out non-js files. Like images etc.
@@ -272,7 +202,6 @@ class Bundle extends BundleBase {
       ...super.toJSON(),
       sourceMapUrl: this._sourceMapUrl,
       numRequireCalls: this._numRequireCalls,
-      shouldCombineSourceMaps: this._shouldCombineSourceMaps,
     };
   }
 
@@ -281,7 +210,6 @@ class Bundle extends BundleBase {
 
     bundle._sourceMapUrl = json.sourceMapUrl;
     bundle._numRequireCalls = json.numRequireCalls;
-    bundle._shouldCombineSourceMaps = json.shouldCombineSourceMaps;
 
     BundleBase.fromJSON(bundle, json);
 
@@ -289,7 +217,7 @@ class Bundle extends BundleBase {
   }
 }
 
-function generateSourceMapForVirtualModule(module) {
+function generateMissingSourceMapForModule(module) {
   // All lines map 1-to-1
   let mappings = 'AAAA;';
 

react-packager/src/Bundler/__tests__/Bundle-test.js

@@ -138,50 +138,6 @@ describe('Bundle', () => {
   });
 
   describe('sourcemap bundle', () => {
-    pit('should create sourcemap', () => {
-      const otherBundle = new Bundle({sourceMapUrl: 'test_url'});
-
-      return Promise.resolve().then(() => {
-        return addModule({
-          bundle: otherBundle,
-          code: [
-            'transformed foo',
-            'transformed foo',
-            'transformed foo',
-          ].join('\n'),
-          sourceCode: [
-            'source foo',
-            'source foo',
-            'source foo',
-          ].join('\n'),
-          sourcePath: 'foo path',
-        });
-      }).then(() => {
-        return addModule({
-          bundle: otherBundle,
-          code: [
-            'transformed bar',
-            'transformed bar',
-            'transformed bar',
-          ].join('\n'),
-          sourceCode: [
-            'source bar',
-            'source bar',
-            'source bar',
-          ].join('\n'),
-          sourcePath: 'bar path',
-        });
-      }).then(() => {
-        otherBundle.setMainModuleId('foo');
-        otherBundle.finalize({
-          runBeforeMainModule: [],
-          runMainModule: true,
-        });
-        const sourceMap = otherBundle.getSourceMap({dev: true});
-        expect(sourceMap).toEqual(genSourceMap(otherBundle.getModules()));
-      });
-    });
-
     pit('should combine sourcemaps', () => {
       const otherBundle = new Bundle({sourceMapUrl: 'test_url'});
 

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

@@ -220,7 +220,7 @@ describe('Bundler', function() {
           transform: {
             dev: true,
             hot: false,
-            generateSourceMaps: false,
+            generateSourceMaps: true,
             projectRoots,
           }
         },

react-packager/src/Bundler/index.js

@@ -385,7 +385,6 @@ class Bundler {
         onProgress,
         minify,
         isolateModuleIDs,
-        generateSourceMaps: unbundle,
       });
     }
 
@@ -444,7 +443,7 @@ class Bundler {
     dev = true,
     minify = !dev,
     hot = false,
-    generateSourceMaps = false,
+    generateSourceMaps = true,
   }) {
     return this.getTransformOptions(
       entryFile,
@@ -482,7 +481,7 @@ class Bundler {
     minify = !dev,
     hot = false,
     recursive = true,
-    generateSourceMaps = false,
+    generateSourceMaps = true,
     isolateModuleIDs = false,
     onProgress,
   }) {

transformer.js

@@ -74,6 +74,7 @@ function buildBabelConfig(filename, options) {
   const extraConfig = {
     filename,
     sourceFileName: filename,
+    sourceMaps: options.generateSourceMaps,
   };
 
   let config = Object.assign({}, babelRC, extraConfig);