PowerShell / PowerShell/PowerShellEditorServices

Consider caching Alias & Command information

オープン
#992 コメント 4 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Area-CodeLens Area-Extension Terminal Issue-Enhancement Issue-Performance
主要言語
C#
スター
767
フォーク
266
平均マージ
3日 16時間
マージ済み PR(30日)
1

説明

On every textDocument/references request and every textDocument/codeLens request, we rely on the pipeline to get the current available.

This is time consuming and bottlenecked by the pipeline thread. As such, we should consider caching this result based on the state of the Integrated Console.

Coming off the the back of #980, one of the patterns I liked was the ability to tell if something was run in the PSIC using PSRL’s ENTER handler and F8’s handler.

Bringing this concept in, we can use it as a “Dirty” check where we cache aliases and command info results, and only refresh the cache when the PSIC is Dirty.

That way, for example, our textDocument/references request and textDocument/codeLens request only rely on the pipeline once the cache is Dirty and an otherwise run unblocked giving us a perf improvement.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

src/PowerShellEditorServices/Language/LanguageService.cs の327行目付近にあるリンクされた検索から始め、次に #980 で説明されている Integrated Console の動作を確認します。PSRL の ENTER ハンドラーと F8 のハンドラーも含めて確認してください。PSIC が dirty のときにエイリアス情報とコマンド情報がどのようにキャッシュされ、無効化されるかを定義します。完了条件は、キャッシュの更新が必要な場合にのみ参照と codeLens のリクエストがパイプラインを使用することです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
csharp, powershell
領域
performance
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。