Расширяем ReSharper — Context Actions

Хорошая работа!
Интересно, все-таки, будет ли это работать на 5-ом решарпере?

Несколько замечаний по примеру:

1) В R# уже есть базовый класс дня написания Context Actions. Это класс BulbItemImpl. Его можно использовать примерно так:

class MyCA : BulbItemImpl, IContextAction
{
    private readonly ICSharpContextActionDataProvider myProvider;

    protected MyCa(ICSharpContextActionDataProvider provider)
    {
      myProvider = provider;
    }

    public bool IsAvailable(IUserDataHolder cache)
    {
      return ...;
    }

    public override string Text
    {
      get { return "My menu item text"; }
    }

    protected override Action<ITextControl> ExecuteTransaction(ISolution solution, IProgressIndicator progress)
    {
      //Do Something

      //here you can return 'null'
      return control =>
        {
          //Do something with caret, selection, etc.
          //Ex: control.Caret.MoveTo(myElement.GetDocumentRange().TextRange.StartOffset);
        };
    }  
}

Плюсы BulbItemImpl в том, что он будет корректно работать в случае изменений в нескольких файлах (например, вы меняете что-то в иерархии). Для долгих операций этот класс будет показывать окошко с прогресом. А так, конечно, разницы большой нет.

2) using (ReadLockCookie.Create()) {… } — делать в CA не надо

3) Использовать LowLevelModificationUtil нельзя. На то они и LowLevel. В этом просто примере это работет (хотя в R# 5 вообще-то должно упасть с исключением), но в более сложных случаях разрушит цивилизацию. Вместо этого надо использовать метод ICSharpExpression.ReplaceBy. И вообще, обычно все узлы синтаксического дерева снабнажены методами для изменения самих себя.

4) Лучше стараться не переходить к TreeNode (т.е. вызывать метод ToTreeNode), которые содержат в себе уже не только семантическую информацию, а еще и синтаксическую. Например, в давнном CA все вызовы ToTreeNode избыточны. Но это замечаение больше по стилю: интерфейс TreeNode менее стабилен и менее понятен (много синтаксического «мусора»).

советую набросать простенькую DSL на том же F#, например, потому что поиск по дереву в C# выглядит неопрятно, со всеми этими приведениями типов, проверками на null и так далее.

А почему бы Linq не использовать? В C# вместе с расширяющими методами получается очень серьёзный и гибкий довесок.