Change some more but I am less sure about this

Author: jakebailey

internal/transformers/moduletransforms/commonjsmodule.go

@@ -1080,7 +1080,14 @@ func (tx *CommonJSModuleTransformer) visitTopLevelVariableStatement(node *ast.Va
 					propertyAccess,
 					v.Name().Clone(tx.Factory()),
 				))
+			} else if ast.IsIdentifier(v.Name()) {
+				expression := tx.transformInitializedVariable(v)
+				if expression != nil {
+					pushExpression(tx.Visitor().VisitNode(expression))
+				}
 			} else {
+				// For binding patterns, we can't do exports.{pattern} = value
+				// Just emit the assignment and let appendExportsOfVariableStatement handle the exports
 				expression := transformers.ConvertVariableDeclarationToAssignmentExpression(tx.EmitContext(), v)
 				if expression != nil {
 					pushExpression(tx.Visitor().VisitNode(expression))
@@ -1096,6 +1103,21 @@ func (tx *CommonJSModuleTransformer) visitTopLevelVariableStatement(node *ast.Va
 	return tx.visitTopLevelNestedVariableStatement(node)
 }
 
+func (tx *CommonJSModuleTransformer) transformInitializedVariable(node *ast.VariableDeclaration) *ast.Expression {
+	if node.Initializer == nil {
+		return nil
+	}
+	name := node.Name()
+	propertyAccess := tx.Factory().NewPropertyAccessExpression(
+		tx.Factory().NewIdentifier("exports"),
+		nil, /*questionDotToken*/
+		name,
+		ast.NodeFlagsNone,
+	)
+	tx.EmitContext().AssignCommentAndSourceMapRanges(propertyAccess, name)
+	return tx.Factory().NewAssignmentExpression(propertyAccess, node.Initializer)
+}
+
 // Visits a top-level nested variable statement as it may contain `var` declarations that are hoisted and may still be
 // exported with `export {}`.
 func (tx *CommonJSModuleTransformer) visitTopLevelNestedVariableStatement(node *ast.VariableStatement) *ast.Node {

testdata/baselines/reference/compiler/exportDestructuring.js

@@ -0,0 +1,13 @@
+//// [tests/cases/compiler/exportDestructuring.ts] ////
+
+//// [exportDestructuring.ts]
+const arr = [1, 2];
+export const [a, b] = arr;
+
+
+//// [exportDestructuring.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.b = exports.a = void 0;
+const arr = [1, 2];
+[exports.a, exports.b] = arr;

testdata/baselines/reference/compiler/exportDestructuring.symbols

@@ -0,0 +1,11 @@
+//// [tests/cases/compiler/exportDestructuring.ts] ////
+
+=== exportDestructuring.ts ===
+const arr = [1, 2];
+>arr : Symbol(arr, Decl(exportDestructuring.ts, 0, 5))
+
+export const [a, b] = arr;
+>a : Symbol(a, Decl(exportDestructuring.ts, 1, 14))
+>b : Symbol(b, Decl(exportDestructuring.ts, 1, 16))
+>arr : Symbol(arr, Decl(exportDestructuring.ts, 0, 5))
+

testdata/baselines/reference/compiler/exportDestructuring.types

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/exportDestructuring.ts] ////
+
+=== exportDestructuring.ts ===
+const arr = [1, 2];
+>arr : number[]
+>[1, 2] : number[]
+>1 : 1
+>2 : 2
+
+export const [a, b] = arr;
+>a : number
+>b : number
+>arr : number[]
+

testdata/baselines/reference/submodule/compiler/importDeclWithClassModifiers.js

@@ -16,6 +16,6 @@ var b: a;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.c = exports.b = exports.a = void 0;
 exports.a = x.c;
-b = x.c;
+exports.b = x.c;
 exports.c = x.c;
 var b;

testdata/baselines/reference/submodule/compiler/importDeclWithClassModifiers.js.diff

@@ -1,39 +1,6 @@
 --- old.unicodeEscapesInNames01(target=es5).js
 +++ new.unicodeEscapesInNames01(target=es5).js
-@@= skipped -124, +124 lines =@@
- "use strict";
- Object.defineProperty(exports, "__esModule", { value: true });
- exports.x = void 0;
--exports.\u0078 = 10;
-+exports.x = 10;
- exports.x++;
- //# sourceMappingURL=identifierVariableWithEscape1.js.map
- //// [identifierVariableWithEscape2.js]
- "use strict";
- Object.defineProperty(exports, "__esModule", { value: true });
- exports.xx = void 0;
--exports.x\u0078 = 10;
-+exports.xx = 10;
- exports.xx++;
- //# sourceMappingURL=identifierVariableWithEscape2.js.map
- //// [identifierVariableWithExtendedEscape1.js]
- "use strict";
- Object.defineProperty(exports, "__esModule", { value: true });
- exports.x = void 0;
--exports.\u{78} = 10;
-+exports.x = 10;
- exports.x++;
- //# sourceMappingURL=identifierVariableWithExtendedEscape1.js.map
- //// [identifierVariableWithExtendedEscape2.js]
- "use strict";
- Object.defineProperty(exports, "__esModule", { value: true });
- exports.xx = void 0;
--exports.x\u{78} = 10;
-+exports.xx = 10;
- exports.xx++;
- //# sourceMappingURL=identifierVariableWithExtendedEscape2.js.map
- //// [IdentifierNameWithEscape1.js]
-@@= skipped -29, +29 lines =@@
+@@= skipped -153, +153 lines =@@
  Object.defineProperty(exports, "__esModule", { value: true });
  exports.IdentifierNameWithEscape1 = void 0;
  class IdentifierNameWithEscape1 {