-
Notifications
You must be signed in to change notification settings - Fork 194
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 issue after latest VS Code update #963
Comments
What version of the extension are you using? Is it actually v0.10.5 or are you by chance running a pre-release build? If it's v0.10.5 then this sounds like a regression in VSCode? |
Not the pre-release, I would assume this is a regression from a VS Code update, as this version had been running fine since I installed it, until said (recent) VS Code update. |
Weird — thanks. I'll look into it |
cool, let me know if there is any debugging or logs available on my end that could be helpful |
@verdverm Sorry for getting back to this so late. I'm testing in 1.89.1 and am not seeing anything out of the ordinary. No runaway CPU usage or anything. Could you possibly provide a reproduction if you're still seeing this in 1.89.1? |
Still seeing the issue, try opening https://github.com/hofstadter-io/hof in VS Code |
I have yet to find the root cause, but I noticed the same thing. Running M2 pro. Additionally, each time I run the
Also tested with the VS Code silicon build, same issue. Hovering over classes often doesn't work and is stuck at "Loading...". but I wonder if that has to do with my regex (using
Note: removing these settings does not change the CPU usage and also keeps the process running after I quit VS Code. |
This is definitely a problem with the regex. Removing the settings should fix it but you'll need to make sure that you quite VSCode and kill any errant processes. That regex has nested That said I'm working on offloading regex stuff to a separate worker thread so we can at least prevent intellisense from breaking. |
Can you try the regex from this? #893 (comment) Another user was using tailwind variants and replaced the regex with something that's less likely to run into backtracking problems. I'm prototyping a change that would offload regex matching to a separate worker thread to allow us to detect this situation. I think it'll pan out but it may be a while before that ships. |
I am not sure I am hitting the same issue here but with this extension enabled I have an $ ps aux |grep tailwind
jdoss 1101634 1188 0.0 98484 11532 ? Sl 13:14 33:08 /usr/share/code/resources/app/node_modules.asar.unpacked/@vscode/ripgrep/bin/rg --files --hidden --case-sensitive --no-require-git -g **/{tailwind,tailwind.config,tailwind.*.config,tailwind.config.*}.{js,cjs,ts,mjs} -g !**/.git -g !**/.svn -g !**/.hg -g !**/CVS -g !**/.DS_Store -g !**/Thumbs.db -g !/{**/.git,**/.svn,**/.hg,**/CVS,**/.DS_Store,**/Thumbs.db,**/.git/**,**/node_modules/**,**/.hg/**,**/.svn/**} --no-ignore --follow --no-config --no-ignore-global Disabling the tailwindcss-intellisense extension causes the process to go away. This happens on the release and pre-release versions. |
@jdoss This seems like a separate issue (and maybe one with either vscode or related to your filesystem). We don't use ripgrep but we do register globs to be watched and I'm guessing VSCode is using that — and for whatever reason — it's being slow. Can you provide?:
Also — would you mind putting that into a separate issue? |
Done. #986 Sorry for the noise in this issue. |
No worries — thanks for opening! |
What version of VS Code are you using?
For example: v1.89.0
What version of Tailwind CSS IntelliSense are you using?
For example: v0.10.5
Describe your issue
Since getting the latest update to VS Code, this plugin has been running at around 150% CPU on an M2. I don't have any css or tailwind files open and this is still the case
The text was updated successfully, but these errors were encountered: