SqlServer RevEng: Scaffold Included index columns

[bricelam]

See #22148 (comment)

We need a new provider hook to handle annotations when we copy them from ITableIndex to IIndex.

[ajcvickers]

The workaround for this is to find and replace where the names are wrong after performing the scaffolding. Please up-vote this issue if you're hitting it.

[richardthombs]

I've just hit this in a large database where all the key fields are suffixed with "ID". The scaffolding produces properties with a camel cased "Id" suffix, but the IncludeProperties still references the original database column name with the uppercase "ID".

Incidentally, it would be lovely if the error message made mention of which index has failed validation.

[bricelam]

PR #23083 removes the code to scaffold them entirely. Re-purposing this issue to add them back as part of fixing the original problem.