cerxes/babel
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

make it illegal to export a __esModule property - #673

Browse Source
This commit is contained in:
Sebastian McKenzie 2015-02-03 14:41:11 +11:00
parent 630f1717f0
commit 92d9b3ff5f
5 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/6to5/transformation/modules/_default.js
View File

@ -212,6 +212,12 @@ DefaultFormatter.prototype.getExternalReference = function (node, nodes) {
}
};
DefaultFormatter.prototype.checkExportIdentifier = function (node) {
if (t.isIdentifier(node, { name: "__esModule" })) {
throw this.file.errorWithNode(node, "Illegal export __esModule - this is used internally for CommonJS interop");
}
};
DefaultFormatter.prototype.exportSpecifier = function (specifier, node, nodes) {
var inherits = false;
if (node.specifiers.length === 1) inherits = node;
@ -251,6 +257,7 @@ DefaultFormatter.prototype.buildExportsWildcard = function (objectIdentifier) {
};
DefaultFormatter.prototype.buildExportsAssignment = function (id, init) {
this.checkExportIdentifier(id);
return util.template("exports-assign", {
VALUE: init,
KEY: id

1
test/fixtures/transformation/es6-modules-common/illegal-export-esmodule-2/actual.js vendored Normal file
View File

@ -0,0 +1 @@
export { __esModule };

3
test/fixtures/transformation/es6-modules-common/illegal-export-esmodule-2/options.json vendored Normal file
View File

@ -0,0 +1,3 @@
{
"throws": "Illegal export __esModule - this is used internally for CommonJS interop"
}

1
test/fixtures/transformation/es6-modules-common/illegal-export-esmodule/actual.js vendored Normal file
View File

@ -0,0 +1 @@
export var __esModule = false;

3
test/fixtures/transformation/es6-modules-common/illegal-export-esmodule/options.json vendored Normal file
View File

@ -0,0 +1,3 @@
{
"throws": "Illegal export __esModule - this is used internally for CommonJS interop"
}