microsoft / microsoft/TypeScript
Cache `getExePath()` for performance reasons
オープン
まだ誰も着手していません。
Possible Improvement
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.3k
- 平均マージ
- 2日 4時間
- マージ済み PR(30日)
- 132
説明
Would it be possible to have `getExePath()` cache its result so we don't have to do all this lookup every time `tsgo` is invoked? The platform and arch rarely changes once installed on disk.
In one of the repositories I'm maintaining, we've inlined what `tsgo.js` does and added caching on top of it. We're not a very big monorepo, but this shaved off several seconds for us.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
tsgo.js で getExePath() を見つけ、tsgo が呼び出されたときにそれが何回実行されるかを追跡することから始めます。プラットフォームとアーキテクチャ全体で既存の検索動作を確認し、キャッシュによって選択された実行ファイルが保持され、検索の繰り返しが回避されることを確認します。繰り返し呼び出しても検索が再実行されず、既存の動作が変わらないことが完了の条件です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- cli, performance, tooling
- issue の種類
- リファクタリング
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 静か
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 55/100