typescript-go icon indicating copy to clipboard operation
typescript-go copied to clipboard

Add option for overriding `GOMEMLIMIT`

Open maschwenk opened this issue 2 months ago • 2 comments

  • https://github.com/microsoft/typescript-go/pull/1807

But less silly

maschwenk avatar Oct 03 '25 19:10 maschwenk

@maschwenk I think I want to try to get this in, just so we can have people try this and see how it behaves.

The current code looks fine, though it requires a merge from main.

jakebailey avatar Dec 12 '25 20:12 jakebailey

@jakebailey on it!

maschwenk avatar Dec 12 '25 21:12 maschwenk

This seems to work, though it does not integrate with the restart command because it's resolved before the client is created and then .restart() provide no opportunity to update it.

It may be that we should instead move this code into a user preference in the server itself and then configure it there on the fly.

Unfortunately, in my own testing, I felt like the option probably hurt more than it helped. If I set it to something like 2GiB, then opening the main TS codebase would use 1.4G, when without the option it would use 600MB. Not sure what to make of that.

@jakebailey sort of similar to my experience, hence why I left this open for a while. Still, maybe over time this will be a useful dial to turn

maschwenk avatar Dec 15 '25 14:12 maschwenk