RenameSymbolAsync doesn't rename the symbol in the implementing class

[Youssef1313]

If I rename a method parameter in an interface through RenameSymbolAsync, it doesn't get updated in the implementing class.

For example:

public interface I
{
    void MyMethod(string oldName); // Rename this to newName
}

public class C : I
{
    public void MyMethod(string oldName) { }
}

The result is:

public interface I
{
    void MyMethod(string newName); // Rename this to newName
}

public class C : I
{
    public void MyMethod(string oldName) { }
}

I was expecting it to be:

public interface I
{
    void MyMethod(string newName); // Rename this to newName
}

public class C : I
{
    public void MyMethod(string newName) { }
}

Found this while writing code fix in roslyn-analyzers repo. (dotnet/roslyn-analyzers#3706)

[sharwell]

I'm fairly certain this is by design. Code fix implementations can manually located and rename related symbols, but are not required to.

[Youssef1313]

@sharwell Thanks for reply.

Was there any reason in designing that API not to rename the related symbols?
It would be much easier if that work is done inside the RenameSymbolAsync method.

[CyrusNajmabadi]

This was by design as cascading here was felt to be innapropriate when we originally tried it. i.e. people wanted the scope of change here to not include optional things like this. We considered an option for it, but there wasn't much demand. I would likely take a PR here that added a programattic option to the rename api to do this.

[Evangelink]

So what's the recommended way to rename the symbols? It seems we can do something using the workspace but as far as I understand this isn't available from the roslyn-analyzers code-fixes.