Repository navigation
Hoisted function declarations should be assigned to exports before any require calls #39853
Description
TypeScript Version: 4.2.0-beta
Search Terms:
- hoisted function
- exports hoisted function
- export hoisted function
- export hoist function
Code
// @module: CommonJS
// @showEmit
// @showEmittedFile: b.js
// @filename: a.ts
import { b } from "./b.js";
export function a() {
return b();
}
// @filename: b.ts
import { a } from "./a.js";
// Crashes when transpiled by TypeScript:
a();
export function b() {
return 123;
}Expected behavior:
The exports.* assignment should be done immediately after __esModule, just like how Babel does it.
// a.js
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.a = a;
const b_1 = require("./b.js");
function a() {
return b_1.b();
}
// b.js
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.b = b;
const a_js_1 = require("./a.js");
// Crashes when transpiled by TypeScript:
a_js_1.a();
function b() {
return 123;
}Actual behavior:
The exports assignment is performed after the function declaration, which causes behaviour differences between native ES modules (where hoisted function declarations are initialised before any code begins executing) and transpiled CommonJS modules (where hoisted function declarations are added to the exports object after require(…) calls)
// a.js
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.a = void 0;
const b_1 = require("./b.js");
function a() {
return b_1.b();
}
exports.a = a;
// b.js
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.b = void 0;
const a_js_1 = require("./a.js");
// Crashes when transpiled by TypeScript:
a_js_1.a();
function b() {
return 123;
}
exports.b = b;Related Issues: #37093
Activity
typescript-bot commentedon Apr 13, 2022
👋 Hi, I'm the Repro bot. I can help narrow down and track compiler bugs across releases! This comment reflects the current state of the repro in the issue body running against the nightly TypeScript.
Issue body code block by ExE Boss (@ExE-Boss)
👍 Compiled
Emit:
Historical Information
👍 Compiled
Emit:
👍 Compiled
Emit: