ChangeSignature: should trigger in positional records

[jcouv]

ChangeSignature is currently not offered.
Note: we first need to clarify whether those are parameter or property declarations (tracked by #44556).

Note that we offer ChangeSignature on other constructor parameters:

Relates to #40726 (test plan for "records")

[jinujoseph]

cc @dpoeschl

[jcouv]

From discussion with @CyrusNajmabadi, let's take a stab for 16.8, otherwise push to 16.9.

[jcouv]

Assigned to @allisonchou since the duplicate issue had been assigned to her.
This may require new semantic model APIs to be able to update the base list (change signature on Base should update record Derived(int i) : Base(...);. Using GetSymbolInfo on the base type syntax correctly returns the constructor symbol.
We should verify that callers are updated (new Person(....)), but we expect that to just work.

[allisonchou]

@jcouv For record declarations, is there a way to get their associated IParameterSymbols if they have any?

For example, with following code -

record Rec(string One, string Two)
{
}

Rec is an INamedTypeSymbol. However, I haven't been able to find a straightforward way to get parameter symbols from an INamedTypeSymbol, which we need for change signature.

[jcouv]

@allisonchou Tagging @CyrusNajmabadi who is working on symbol for associating parameters and properties in records.

[Youssef1313]

I'm working on this issue as part of a quick pass for IDE features that need to handle record doc comments.

[jcouv]

@allisonchou GetDeclaredSymbol on the parameter syntax will give you the corresponding parameter symbol.