Hide property access completions of @ignored properties

Author: Andarist

src/compiler/checker.ts

@@ -303,6 +303,7 @@ import {
     getExternalModuleRequireArgument,
     getFirstConstructorWithBody,
     getFirstIdentifier,
+    getFirstJSDocTag,
     getFunctionFlags,
     getHostSignatureFromJSDoc,
     getIdentifierGeneratedImportReference,
@@ -805,6 +806,7 @@ import {
     JSDocPublicTag,
     JSDocSatisfiesTag,
     JSDocSignature,
+    JSDocTag,
     JSDocTemplateTag,
     JSDocThisTag,
     JSDocTypeAssertion,
@@ -34489,7 +34491,8 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
      * @param property the accessed property's symbol.
      */
     function isValidPropertyAccessForCompletions(node: PropertyAccessExpression | ImportTypeNode | QualifiedName, type: Type, property: Symbol): boolean {
-        return isPropertyAccessible(node, node.kind === SyntaxKind.PropertyAccessExpression && node.expression.kind === SyntaxKind.SuperKeyword, /*isWrite*/ false, type, property);
+        return isPropertyAccessible(node, node.kind === SyntaxKind.PropertyAccessExpression && node.expression.kind === SyntaxKind.SuperKeyword, /*isWrite*/ false, type, property)
+            && (!property.valueDeclaration || !getFirstJSDocTag(property.valueDeclaration, (t): t is JSDocTag => t.kind === SyntaxKind.JSDocTag && t.tagName.escapedText === "ignore"));
         // Previously we validated the 'this' type of methods but this adversely affected performance. See #31377 for more context.
     }
 

src/compiler/utilitiesPublic.ts

@@ -1260,8 +1260,11 @@ export function getJSDocTags(node: Node): readonly JSDocTag[] {
     return getJSDocTagsWorker(node, /*noCache*/ false);
 }
 
-/** Get the first JSDoc tag of a specified kind, or undefined if not present. */
-function getFirstJSDocTag<T extends JSDocTag>(node: Node, predicate: (tag: JSDocTag) => tag is T, noCache?: boolean): T | undefined {
+/**
+ * @internal
+ * Get the first JSDoc tag of a specified kind, or undefined if not present.
+ */
+export function getFirstJSDocTag<T extends JSDocTag>(node: Node, predicate: (tag: JSDocTag) => tag is T, noCache?: boolean): T | undefined {
     return find(getJSDocTagsWorker(node, noCache), predicate);
 }
 

tests/cases/fourslash/completionsIgnoreTagOnProperty1.ts

@@ -0,0 +1,16 @@
+/// <reference path="fourslash.ts" />
+
+//// type Foo = {
+////   /** @ignore */
+////   a: string;
+////   b: number;
+//// };
+////
+//// declare const foo: Foo;
+//// foo./**/
+
+verify.completions({
+    marker: "",
+    exact: "b",
+    isNewIdentifierLocation: false,
+});