Skip to content

Conversation

@isaacs
Copy link
Contributor

@isaacs isaacs commented Feb 2, 2026

Update the logic to infer the type properly when a Deno file is part of a non-Deno monorepo project, enabling using the denols and ts_ls (or tsgo) LSPs side by side in the same monorepo without conflict.

Fix: #4300

@isaacs isaacs force-pushed the isaacs/deno-ts_ls-inference branch 7 times, most recently from 06b89f2 to 9ef0da1 Compare February 2, 2026 18:44
@isaacs
Copy link
Contributor Author

isaacs commented Feb 2, 2026

Removed the virtualTextDocument handler, because I had missed that it was added to the lsp config on the master branch already 😅

@isaacs isaacs force-pushed the isaacs/deno-ts_ls-inference branch 2 times, most recently from b55ce19 to 4902571 Compare February 2, 2026 18:46
@isaacs isaacs changed the title fix(deno, ts_ls): Deno inference and virtual files fix(deno, ts_ls): denols/ts_ls inference Feb 2, 2026
@KingJordan152
Copy link

Would it be possible to also add this logic to vtsls?

@isaacs
Copy link
Contributor Author

isaacs commented Feb 5, 2026

Would it be possible to also add this logic to vtsls?

Sure, I'll push another commit in just a moment.

Update the logic to infer the type properly when a Deno file is part of
a non-Deno monorepo project, enabling using the denols and ts_ls (or
tsgo) LSPs side by side in the same monorepo without conflict.

Fix: neovim#4300
@isaacs isaacs force-pushed the isaacs/deno-ts_ls-inference branch from 4902571 to 7701661 Compare February 5, 2026 00:37
@justinmk
Copy link
Member

Very nice. Thank you for looking into this!

@justinmk justinmk merged commit 05d01a4 into neovim:master Feb 12, 2026
5 checks passed
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.

denols and ts_ls conflict, neither activate in many typical scenarios

3 participants