Don't filter away private completions if in the same context

Fixes #34405, which was introduced in PR #16953.

Author: elibarzilay

src/services/completions.ts

@@ -1826,8 +1826,14 @@ namespace ts.Completions {
                 if (canGetType) {
                     const typeForObject = typeChecker.getTypeAtLocation(objectLikeContainer);
                     if (!typeForObject) return GlobalsSearch.Fail;
-                    // In a binding pattern, get only known properties. Everywhere else we will get all possible properties.
-                    typeMembers = typeChecker.getPropertiesOfType(typeForObject).filter((symbol) => !(getDeclarationModifierFlagsFromSymbol(symbol) & ModifierFlags.NonPublicAccessibilityModifier));
+                    // In a binding pattern, get only known properties (unless in the same scope).
+                    // Everywhere else we will get all possible properties.
+                    const containerClass = getContainingClass(objectLikeContainer);
+                    typeMembers = typeChecker.getPropertiesOfType(typeForObject).filter(symbol =>
+                        // either public
+                        !(getDeclarationModifierFlagsFromSymbol(symbol) & ModifierFlags.NonPublicAccessibilityModifier)
+                        // or we're in it
+                        || containerClass && typeForObject.symbol.declarations.indexOf(containerClass) >= 0);
                     existingMembers = objectLikeContainer.elements;
                 }
             }

tests/cases/fourslash/completionListInObjectBindingPattern15.ts

@@ -0,0 +1,19 @@
+/// <reference path="fourslash.ts"/>
+
+////class Foo {
+////    private   xxx1 = 1;
+////    protected xxx2 = 2;
+////    public    xxx3 = 3;
+////    foo() {
+////        const { /*1*/ } = this;
+////    }
+////}
+////
+////const { /*2*/ } = new class {
+////    private   xxx1 = 1;
+////    protected xxx2 = 2;
+////    public    xxx3 = 3;
+////}
+
+verify.completions({ marker: "1", exact: ["xxx1", "xxx2", "xxx3", "foo"] });
+verify.completions({ marker: "2", exact: ["xxx3"] });