Extra ? added to method in method signature completion for optional method

[mjbvz]

Bug Report

🔎 Search Terms

• Method signature completions
• suggest / suggestions
• completions

🕗 Version & Regression Information

4.5.0-dev.20211029

💻 Code

For the code:

interface IFoo {
    barOpt?(x: number): void;
}

class Foo implements IFoo {
    |
 }

Complete barOpt inside Foo

🙁 Actual behavior

barOpt gets added with a ?

interface IFoo {
    barOpt?(x: number): void;
}

class Foo implements IFoo {
    barOpt?(x: number): void {
        
    }
}

🙂 Expected behavior

Since we have implemented the method, I'd expect us the remove the ?

[DanielRosenwasser]

This is especially true under exactOptionalPropertyTypes, but also who would ever want a sub-subclass to override with undefined?

[DanielRosenwasser]

Thanks @a-tarasyuk!