continue
continue copied to clipboard
Feat/worker thread
fixes #856
Deploy Preview for continuedev canceled.
| Name | Link |
|---|---|
| Latest commit | 2105cea7bde4e116ec3c568091dd8b6040dcb3f0 |
| Latest deploy log | https://app.netlify.com/sites/continuedev/deploys/65fb651b8523ae00086473e6 |
@Iamshankhadeep I finally got around to testing this locally. Everything looks perfect, including the packaging with esbuild (you solved this very cleanly!). But it seems that VS Code isn't doing well with worker threads. In debugging mode, the worker never responds (can verify this by placing a breakpoint in the on message function).
Then when I packaged the extension and installed it as a .vsix, everything seemed to work fine, until using the @ codebase context provider causes the extension host to crash.
Had you seen the same issue or have any ideas? https://github.com/microsoft/vscode/issues/88386 seems like the most closely related issue, though seemed to have been solved
@Iamshankhadeep I finally got around to testing this locally. Everything looks perfect, including the packaging with esbuild (you solved this very cleanly!). But it seems that VS Code isn't doing well with worker threads. In debugging mode, the worker never responds (can verify this by placing a breakpoint in the on message function).
Then when I packaged the extension and installed it as a .vsix, everything seemed to work fine, until using the @ codebase context provider causes the extension host to crash.
Had you seen the same issue or have any ideas? microsoft/vscode#88386 seems like the most closely related issue, though seemed to have been solved
I was having the same issue while testing on this worker_thread but terminating the worker thread worker.terminate(); seems to fix that.
until using the @ codebase context provider causes the extension host to crash.
I can think of one issue probably using @ codebase opening too many worker threads before terminating previous theard crasing, or probably adding some smart resourse limit
@Iamshankhadeep the worker limit is a good idea. When you had the same problem, was it in debug mode, or was this with a packaged version of the extension? I would be curious whether you end up seeing the same problem that I did after running cd extensions/vscode && node scripts/package.js and then installing build/continue-patch.vsix
@Iamshankhadeep the worker limit is a good idea. When you had the same problem, was it in debug mode, or was this with a packaged version of the extension? I would be curious whether you end up seeing the same problem that I did after running
cd extensions/vscode && node scripts/package.jsand then installingbuild/continue-patch.vsix
let me try that and I'll let you know 🔥