Skip to content

Commit e6bec9c

Browse files
cwdickFacebook Github Bot 6
authored andcommitted
Add option to bundle server to generate full sourcemaps from babel
Reviewed By: bestander Differential Revision: D3863894 fbshipit-source-id: a282758e022d403743841bc59277196e6741ed18
1 parent fa6191f commit e6bec9c

File tree

4 files changed

+15
-5
lines changed

4 files changed

+15
-5
lines changed

local-cli/server/middleware/heapCaptureMiddleware.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,11 @@ function getSourceMapForUrl(url, onFailure, onSuccess) {
3232
}
3333

3434
const parsedUrl = urlLib.parse(url);
35+
const mapPath = parsedUrl.pathname.replace(/\.bundle$/, '.map');
3536
const options = {
3637
host: 'localhost',
3738
port: parsedUrl.port,
38-
path: parsedUrl.pathname.replace(/\.bundle$/, '.map') + parsedUrl.search,
39+
path: mapPath + parsedUrl.search + '&babelSourcemap=true',
3940
};
4041

4142
http.get(options, (res) => {

local-cli/server/middleware/jscProfilerMiddleware.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,11 @@ class TreeTransformator {
7979
}
8080

8181
const parsedUrl = urlLib.parse(url);
82+
const mapPath = parsedUrl.pathname.replace(/\.bundle$/, '.map');
8283
const options = {
8384
host: 'localhost',
8485
port: parsedUrl.port,
85-
path: parsedUrl.pathname.replace(/\.bundle$/, '.map') + parsedUrl.search,
86+
path: mapPath + parsedUrl.search + '&babelSourcemap=true',
8687
};
8788

8889
http.get(options, (res) => {

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -257,6 +257,7 @@ class Bundler {
257257
entryModuleOnly,
258258
resolutionResponse,
259259
isolateModuleIDs,
260+
generateSourceMaps,
260261
}) {
261262
const onResolutionResponse = response => {
262263
bundle.setMainModuleId(response.getModuleId(getMainModule(response)));
@@ -301,6 +302,7 @@ class Bundler {
301302
onResolutionResponse,
302303
finalizeBundle,
303304
isolateModuleIDs,
305+
generateSourceMaps,
304306
});
305307
}
306308

@@ -352,6 +354,7 @@ class Bundler {
352354
unbundle,
353355
resolutionResponse,
354356
isolateModuleIDs,
357+
generateSourceMaps,
355358
onResolutionResponse = noop,
356359
onModuleTransformed = noop,
357360
finalizeBundle = noop,
@@ -385,7 +388,7 @@ class Bundler {
385388
onProgress,
386389
minify,
387390
isolateModuleIDs,
388-
generateSourceMaps: unbundle,
391+
generateSourceMaps: unbundle || generateSourceMaps,
389392
});
390393
}
391394

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

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,10 @@ const bundleOpts = declareOpts({
150150
resolutionResponse: {
151151
type: 'object',
152152
},
153+
generateSourceMaps: {
154+
type: 'boolean',
155+
required: false,
156+
}
153157
});
154158

155159
const dependencyOpts = declareOpts({
@@ -814,7 +818,7 @@ class Server {
814818
sourceMapUrl: url.format(sourceMapUrlObj),
815819
entryFile: entryFile,
816820
dev: this._getBoolOptionFromQuery(urlObj.query, 'dev', true),
817-
minify: this._getBoolOptionFromQuery(urlObj.query, 'minify'),
821+
minify: this._getBoolOptionFromQuery(urlObj.query, 'minify', false),
818822
hot: this._getBoolOptionFromQuery(urlObj.query, 'hot', false),
819823
runModule: this._getBoolOptionFromQuery(urlObj.query, 'runModule', true),
820824
inlineSourceMap: this._getBoolOptionFromQuery(
@@ -828,11 +832,12 @@ class Server {
828832
'entryModuleOnly',
829833
false,
830834
),
835+
generateSourceMaps: this._getBoolOptionFromQuery(urlObj.query, 'babelSourcemap'),
831836
};
832837
}
833838

834839
_getBoolOptionFromQuery(query, opt, defaultVal) {
835-
if (query[opt] == null && defaultVal != null) {
840+
if (query[opt] == null) {
836841
return defaultVal;
837842
}
838843

0 commit comments

Comments
 (0)