From 1b5798017df87cacfb659b5e975cb37cef7b19b5 Mon Sep 17 00:00:00 2001
From: Oleksandr T <oleksandr.tarasiuk@outlook.com>
Date: Wed, 20 Oct 2021 12:45:26 +0300
Subject: [PATCH] fix(46345): omit temp variable for computed property name in
 ambient context

---
 src/compiler/transformers/ts.ts               |  2 +-
 .../reference/decoratorInAmbientContext.js    | 26 +++++++++++++++++++
 .../decoratorInAmbientContext.symbols         | 23 ++++++++++++++++
 .../reference/decoratorInAmbientContext.types | 25 ++++++++++++++++++
 .../decorators/decoratorInAmbientContext.ts   | 10 +++++++
 5 files changed, 85 insertions(+), 1 deletion(-)
 create mode 100644 tests/baselines/reference/decoratorInAmbientContext.js
 create mode 100644 tests/baselines/reference/decoratorInAmbientContext.symbols
 create mode 100644 tests/baselines/reference/decoratorInAmbientContext.types
 create mode 100644 tests/cases/conformance/decorators/decoratorInAmbientContext.ts

diff --git a/src/compiler/transformers/ts.ts b/src/compiler/transformers/ts.ts
index 8561db1c633ef..2724e7a75cb00 100644
--- a/src/compiler/transformers/ts.ts
+++ b/src/compiler/transformers/ts.ts
@@ -1189,7 +1189,7 @@ namespace ts {
             //
 
             const prefix = getClassMemberPrefix(node, member);
-            const memberName = getExpressionForPropertyName(member, /*generateNameForComputedPropertyName*/ true);
+            const memberName = getExpressionForPropertyName(member, /*generateNameForComputedPropertyName*/ !hasSyntacticModifier(member, ModifierFlags.Ambient));
             const descriptor = languageVersion > ScriptTarget.ES3
                 ? member.kind === SyntaxKind.PropertyDeclaration
                     // We emit `void 0` here to indicate to `__decorate` that it can invoke `Object.defineProperty` directly, but that it
diff --git a/tests/baselines/reference/decoratorInAmbientContext.js b/tests/baselines/reference/decoratorInAmbientContext.js
new file mode 100644
index 0000000000000..3ae6267dbc276
--- /dev/null
+++ b/tests/baselines/reference/decoratorInAmbientContext.js
@@ -0,0 +1,26 @@
+//// [decoratorInAmbientContext.ts]
+declare function decorator(target: any, key: any): any;
+
+const b = Symbol('b');
+class Foo {
+    @decorator declare a: number;
+    @decorator declare [b]: number;
+}
+
+
+//// [decoratorInAmbientContext.js]
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+const b = Symbol('b');
+class Foo {
+}
+__decorate([
+    decorator
+], Foo.prototype, "a", void 0);
+__decorate([
+    decorator
+], Foo.prototype, b, void 0);
diff --git a/tests/baselines/reference/decoratorInAmbientContext.symbols b/tests/baselines/reference/decoratorInAmbientContext.symbols
new file mode 100644
index 0000000000000..9ba2380150d78
--- /dev/null
+++ b/tests/baselines/reference/decoratorInAmbientContext.symbols
@@ -0,0 +1,23 @@
+=== tests/cases/conformance/decorators/decoratorInAmbientContext.ts ===
+declare function decorator(target: any, key: any): any;
+>decorator : Symbol(decorator, Decl(decoratorInAmbientContext.ts, 0, 0))
+>target : Symbol(target, Decl(decoratorInAmbientContext.ts, 0, 27))
+>key : Symbol(key, Decl(decoratorInAmbientContext.ts, 0, 39))
+
+const b = Symbol('b');
+>b : Symbol(b, Decl(decoratorInAmbientContext.ts, 2, 5))
+>Symbol : Symbol(Symbol, Decl(lib.es5.d.ts, --, --), Decl(lib.es2015.symbol.d.ts, --, --), Decl(lib.es2015.symbol.wellknown.d.ts, --, --), Decl(lib.es2019.symbol.d.ts, --, --))
+
+class Foo {
+>Foo : Symbol(Foo, Decl(decoratorInAmbientContext.ts, 2, 22))
+
+    @decorator declare a: number;
+>decorator : Symbol(decorator, Decl(decoratorInAmbientContext.ts, 0, 0))
+>a : Symbol(Foo.a, Decl(decoratorInAmbientContext.ts, 3, 11))
+
+    @decorator declare [b]: number;
+>decorator : Symbol(decorator, Decl(decoratorInAmbientContext.ts, 0, 0))
+>[b] : Symbol(Foo[b], Decl(decoratorInAmbientContext.ts, 4, 33))
+>b : Symbol(b, Decl(decoratorInAmbientContext.ts, 2, 5))
+}
+
diff --git a/tests/baselines/reference/decoratorInAmbientContext.types b/tests/baselines/reference/decoratorInAmbientContext.types
new file mode 100644
index 0000000000000..b1aaf7e07638a
--- /dev/null
+++ b/tests/baselines/reference/decoratorInAmbientContext.types
@@ -0,0 +1,25 @@
+=== tests/cases/conformance/decorators/decoratorInAmbientContext.ts ===
+declare function decorator(target: any, key: any): any;
+>decorator : (target: any, key: any) => any
+>target : any
+>key : any
+
+const b = Symbol('b');
+>b : unique symbol
+>Symbol('b') : unique symbol
+>Symbol : SymbolConstructor
+>'b' : "b"
+
+class Foo {
+>Foo : Foo
+
+    @decorator declare a: number;
+>decorator : (target: any, key: any) => any
+>a : number
+
+    @decorator declare [b]: number;
+>decorator : (target: any, key: any) => any
+>[b] : number
+>b : unique symbol
+}
+
diff --git a/tests/cases/conformance/decorators/decoratorInAmbientContext.ts b/tests/cases/conformance/decorators/decoratorInAmbientContext.ts
new file mode 100644
index 0000000000000..19fce20a5b42b
--- /dev/null
+++ b/tests/cases/conformance/decorators/decoratorInAmbientContext.ts
@@ -0,0 +1,10 @@
+// @target: esnext
+// @experimentalDecorators: true
+
+declare function decorator(target: any, key: any): any;
+
+const b = Symbol('b');
+class Foo {
+    @decorator declare a: number;
+    @decorator declare [b]: number;
+}