8000 cache builtin commands for shells to decrease terminal completion latency by meganrogge · Pull Request #252232 · microsoft/vscode · GitHub

More Web Proxy on the site http://driver.im/

cache builtin commands for shells to decrease terminal completion latency #252232

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

meganrogge merged 4 commits into main from merogge/perf

Jun 23, 2025

Contributor

meganrogge commented

•

To isolate the improvement, I first requested completions in a shell other than the target shell to discard the extension activation timing and to obtain a baseline. I then requested completions in the shell of interest and recorded the latency. After reloading the window, I repeated the process - first requesting completions in a non-target shell, then in the shell of interest - tracking the latency again.

Reduces latency by 2432ms for bash (90%), 460ms for pwsh (64%), and 661ms on zsh (79%).

Before:
after requesting completions in pwsh, requesting completions in bash

after reloading the window and after requesting completions in pwsh, requesting completions in bash

After:
after requesting completions in pwsh, requesting completions in bash

after reloading the window and after requesti 8000 ng completions in pwsh, requesting completions in bash

Opposite, but for pwsh

Before:

Reload

After:

Reload

Same, but for zsh

Before:

Reload

After:

Reload

Part of #251994

meganrogge added 4 commits

June 23, 2025 15:42


          wip #251994

03affd2


          use api instead of file


          use promise.all

ecec4ab


          use globalState, evict after a week

64a7b1c

meganrogge self-assigned this

vs-code-engineering bot added this to the June 2025 milestone

meganrogge requested a review from Tyriar

June 23, 2025 21:07

roblourens approved these changes

View reviewed changes

meganrogge merged commit 2e763a8 into main

8 checks passed

meganrogge deleted the merogge/perf branch

June 23, 2025 22:18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

0