API to tell editor to trigger rename after applying refactoring

[mjbvz]

Problem
Using the new extract method refactoring in TS 2.5, extracted methods currently are always named newFunction*. Renaming the extracted method requires a second step after the refactoring as been applied

A better flow would automatically trigger the rename on the new function after the refactoring has been applied.

Proposal
In the getEditsForRefactor response, return an additional field indicating that an editor should perform another action, such as rename, after the refactor has been applied.

[mhegazy]

renameLocation is not sufficient?

[mjbvz]

Thanks! I overlooked this but will give it a try

[mhegazy]

Looks like the property is there, but extract method is not setting it correctly. we should just set it be the span of the new function name.

[mhegazy]

@Andy-MS can you take a look for release-2.5

[mhegazy]

After looking at the implementation, this is gonna be harder than we originally thought, so we will try to get this done for 2.5.3 instead.