Skip to content
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.

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

perf(typescript-plugin): use named pipe servers more efficiently #5070

Merged
merged 17 commits into from
Dec 20, 2024

Conversation

johnsoncodehk
Copy link
Member

@johnsoncodehk johnsoncodehk commented Dec 19, 2024

  • Named pipe server search number dropped from 40 to 20
  • Instead of re-doing the server connect every time for every request, a valid server connect is retained and reused for all requests.
  • containsFile results are now cached.
  • Server status polling interval changed from 1 second to 2.5 seconds
  • Pushed by the named pipe server when component names/props change rather than pulled by the client everytime

Copy link

pkg-pr-new bot commented Dec 19, 2024

Open in Stackblitz

vue-component-meta

npm i https://pkg.pr.new/vuejs/language-tools/vue-component-meta@5070

vue-component-type-helpers

npm i https://pkg.pr.new/vuejs/language-tools/vue-component-type-helpers@5070

@vue/language-plugin-pug

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-plugin-pug@5070

@vue/language-core

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-core@5070

@vue/language-service

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-service@5070

@vue/language-server

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-server@5070

vue-tsc

npm i https://pkg.pr.new/vuejs/language-tools/vue-tsc@5070

@vue/typescript-plugin

npm i https://pkg.pr.new/vuejs/language-tools/@vue/typescript-plugin@5070

commit: 5d77adf

@johnsoncodehk johnsoncodehk changed the title refactor(typescript-plugin): use named pipe servers more efficiently perf(typescript-plugin): use named pipe servers more efficiently Dec 19, 2024
@johnsoncodehk johnsoncodehk marked this pull request as ready for review December 19, 2024 21:19
@johnsoncodehk johnsoncodehk merged commit 3862e76 into master Dec 20, 2024
9 checks passed
@johnsoncodehk johnsoncodehk deleted the redo-named-pipe branch December 20, 2024 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants