Hoist named class/function expressions assigned to module.exports in JS declaration emit

[inq]

Bug

• module.exports = class Thing {...} was falling to the _default fallback in transformExportAssignment, emitting declare const _default: ...; export = _default;.

Fix

• Synthesize a top-level export = Thing; declare class Thing {...} by feeding a converted declaration through the existing transformClassDeclaration / transformFunctionDeclaration via SetOriginal. Reduces 4 submoduleTriaged baselines.

[inq]

@microsoft-github-policy-service agree

[Copilot]

Pull request overview

Fixes JS declaration emit for CommonJS module.exports = class/function Name { ... } by hoisting named class/function expressions into top-level declarations, avoiding the _default fallback and matching upstream TypeScript baselines more closely.

Changes:

• Teach transformExportAssignment to synthesize and transform a top-level class/function declaration when the exported expression is a named class/function expression.
• Update submodule reference baselines to reflect the corrected export = Name + hoisted declaration output.
• Reduce/adjust submoduleTriaged diffs by removing prior _default-based emit and associated DTS errors.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
internal/transformers/declarations/transform.go	Hoists named class/function expressions assigned via export assignment into synthesized declarations and exports the name.
testdata/baselines/reference/submoduleTriaged/conformance/jsDeclarationsExportAssignedVisibility(target=es2015).js.diff	Updates triaged diff to reflect new hoisted declaration shape (removing prior _default-based mismatch).
testdata/baselines/reference/submoduleTriaged/conformance/jsDeclarationsExportAssignedClassExpressionShadowing(target=es2015).js.diff	Updates triaged diff to reflect corrected export = Q behavior for named class expression export assignment.
testdata/baselines/reference/submoduleTriaged/conformance/jsDeclarationsExportAssignedClassExpression(target=es2015).js.diff	Updates triaged diff to reflect export = Thing + hoisted declare class Thing output.
testdata/baselines/reference/submoduleTriaged/compiler/jsDeclarationEmitExportAssignedFunctionWithExtraTypedefsMembers.js.diff	Updates triaged diff to reflect export = loader + hoisted declare function loader output.
testdata/baselines/reference/submodule/conformance/jsDeclarationsExportAssignedVisibility(target=es2015).js	Updates reference baseline output for visibility test to match new hoisted declaration behavior (and removes prior DTS errors section).
testdata/baselines/reference/submodule/conformance/jsDeclarationsExportAssignedClassExpressionShadowing(target=es2015).js	Updates reference baseline output for shadowing case to match new hoisted declaration behavior.
testdata/baselines/reference/submodule/conformance/jsDeclarationsExportAssignedClassExpression(target=es2015).js	Updates reference baseline output for class expression export assignment to export = Thing; declare class Thing ....
testdata/baselines/reference/submodule/compiler/jsDeclarationEmitExportAssignedFunctionWithExtraTypedefsMembers.js	Updates reference baseline output for named function expression export assignment to export = loader; declare function loader ....