Just dont reassign valueDeclaration

Author: Andarist

src/compiler/checker.ts

@@ -13230,7 +13230,7 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
      * @param lateSymbols The late-bound symbols of the parent.
      * @param decl The member to bind.
      */
-    function lateBindMember(parent: Symbol, earlySymbols: SymbolTable | undefined, lateSymbols: Map<__String, TransientSymbol>, decl: LateBoundDeclaration | LateBoundBinaryExpressionDeclaration, isFromAssignmentDeclarationMember = false) {
+    function lateBindMember(parent: Symbol, earlySymbols: SymbolTable | undefined, lateSymbols: Map<__String, TransientSymbol>, decl: LateBoundDeclaration | LateBoundBinaryExpressionDeclaration) {
         Debug.assert(!!decl.symbol, "The member is expected to have a symbol.");
         const links = getNodeLinks(decl);
         if (!links.resolvedSymbol) {
@@ -13245,13 +13245,10 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
 
                 // Get or add a late-bound symbol for the member. This allows us to merge late-bound accessor declarations.
                 let lateSymbol = lateSymbols.get(memberName);
-                if (isFromAssignmentDeclarationMember && lateSymbol) {
-                    return links.resolvedSymbol = lateSymbol;
-                }
                 if (!lateSymbol) lateSymbols.set(memberName, lateSymbol = createSymbol(SymbolFlags.None, memberName, CheckFlags.Late));
 
                 // Report an error if there's a symbol declaration with the same name and conflicting flags.
-                const earlySymbol = earlySymbols?.get(memberName);
+                const earlySymbol = earlySymbols && earlySymbols.get(memberName);
                 // Duplicate property declarations of classes are checked in checkClassForDuplicateDeclarations.
                 if (!(parent.flags & SymbolFlags.Class) && lateSymbol.flags & getExcludedSymbolFlags(symbolFlags)) {
                     // If we have an existing early-bound member, combine its declarations so that we can
@@ -13343,7 +13340,7 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
                         || assignmentKind === AssignmentDeclarationKind.Prototype; // A straight `Prototype` assignment probably can never have a computed name
                     if (isStatic === !isInstanceMember) {
                         if (hasLateBindableName(member)) {
-                            lateBindMember(symbol, earlySymbols, lateSymbols, member, /*isFromAssignmentDeclarationMember*/ true);
+                            lateBindMember(symbol, earlySymbols, lateSymbols, member);
                         }
                     }
                 }

tests/baselines/reference/lateBoundMethodNameAssigmentJS.js

@@ -14,7 +14,7 @@ export class MyClass {
 
 //// [lateBoundMethodNameAssigmentJS.d.ts]
 export class MyClass {
-    [_symbol](): Promise<void>;
+    [_symbol]: () => Promise<void>;
 }
 declare const _symbol: unique symbol;
 export {};