feat(49834): omit self-referenced enum declaration in JsDoc

Author: a-tarasyuk

src/compiler/checker.ts

@@ -677,6 +677,7 @@ m2: ${(this.mapper2 as unknown as DebugTypeMapper).__debugToString().split("\n")
             resolveName(name, location, meaning, excludeGlobals) {
                 return resolveName(location, escapeLeadingUnderscores(name), meaning, /*nameNotFoundMessage*/ undefined, /*nameArg*/ undefined, /*isUse*/ false, excludeGlobals);
             },
+            resolveEntityName,
             getJsxNamespace: n => unescapeLeadingUnderscores(getJsxNamespace(n)),
             getJsxFragmentFactory: n => {
                 const jsxFragmentFactory = getJsxFragmentFactoryEntity(n);

src/compiler/types.ts

@@ -4582,6 +4582,7 @@ namespace ts {
          */
         /* @internal */ getAllPossiblePropertiesOfTypes(type: readonly Type[]): Symbol[];
         /* @internal */ resolveName(name: string, location: Node | undefined, meaning: SymbolFlags, excludeGlobals: boolean): Symbol | undefined;
+        /* @internal */ resolveEntityName(name: EntityNameOrEntityNameExpression, meaning: SymbolFlags, ignoreErrors?: boolean, dontResolveAlias?: boolean, location?: Node): Symbol | undefined;
         /* @internal */ getJsxNamespace(location?: Node): string;
         /* @internal */ getJsxFragmentFactory(location: Node): string | undefined;
 

src/services/goToDefinition.ts

@@ -294,6 +294,17 @@ namespace ts.GoToDefinition {
 
     function getSymbol(node: Node, checker: TypeChecker, stopAtAlias: boolean | undefined) {
         const symbol = checker.getSymbolAtLocation(node);
+        if (isEntityNameExpression(node) && isInJSDoc(node)) {
+            const location = getJSDocHost(node);
+            if (location && isEnumMember(location) && location === symbol?.valueDeclaration) {
+                const flags = SymbolFlags.Type | SymbolFlags.Namespace | SymbolFlags.Value;
+                const symbol = checker.resolveEntityName(node, flags, /*ignoreErrors*/ true, /*dontResolveAlias*/ true, getSourceFileOfNode(location));
+                if (symbol) {
+                    return { symbol, failedAliasResolution: false };
+                }
+            }
+        }
+
         // If this is an alias, and the request came at the declaration location
         // get the aliased symbol instead. This allows for goto def on an import e.g.
         //   import {A, B} from "mod";

tests/cases/fourslash/gotoDefinitionLinkTag2.ts

@@ -0,0 +1,13 @@
+/// <reference path="fourslash.ts" />
+
+// @Filename: a.ts
+////enum E {
+////    /** {@link /*1*/[|Foo|]} */
+////    Foo
+////}
+////interface [|/*2*/Foo|] {
+////    foo: E.Foo;
+////}
+
+goTo.marker("1");
+verify.goToDefinitionIs("2");

tests/cases/fourslash/gotoDefinitionLinkTag3.ts

@@ -0,0 +1,15 @@
+/// <reference path="fourslash.ts" />
+
+// @filename: a.ts
+////interface [|/*2*/Foo|] {
+////    foo: E.Foo;
+////}
+
+// @Filename: b.ts
+////enum E {
+////    /** {@link /*1*/[|Foo|]} */
+////    Foo
+////}
+
+goTo.marker("1");
+verify.goToDefinitionIs("2");

tests/cases/fourslash/gotoDefinitionLinkTag4.ts

@@ -0,0 +1,9 @@
+/// <reference path="fourslash.ts" />
+////enum E {
+////    /** {@link /*1*/[|B|]} */
+////    A,
+////    [|/*2*/B|]
+////}
+
+goTo.marker("1");
+verify.goToDefinitionIs("2");