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

Allow specifying tsserver NODE_ENV --max-old-space-size via config variable #320

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

seankwalker
Copy link

Hey there, longtime tsuquyomi user here! Thank you for such a wonderful plugin 🙏 When working on larger projects, I noticed that using various commands (e.g. jump to definition, show hint) would make vim hang for a long time, with tsserver evidently struggling. With some hacking, I realized that bumping the --max-old-space-size of the node process running tsserver seemed to help with this.

This PR exposes config variable tsuquyomi_tsserver_max_old_space_size to allow the user to easily configure the heap size for the node process running tsserver.

There's no open issues for this or anything, but it's helped save me some time so figured I'd open a PR to see if there's any interest in getting this in mainline. Thanks!

** the Japanese in the docs was translated by GPT-4o so please forgive any mistakes

When working on larger projects, I noticed that using various commands
(e.g. jump to definition, show hint) would make vim hang for a long
time. With some hacking, I realized that bumping the
--max-old-space-size of the node process running tsserver seemed to help
with this.

We expose config variable `tsuquyomi_tsserver_max_old_space_size` to
allow the user to easily configure the heap size for the node process
running tsserver.
@seankwalker seankwalker changed the title Allow specifying NODE_ENV --max-old-space-size via config variable Allow specifying tsserver NODE_ENV --max-old-space-size via config variable Jun 6, 2024
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.

1 participant