Vue LSP not reporting diagnostics

[drevantonder]

TL;DR Vue LSP is not reporting diagnostics. Fix is in #2097

---

I created #1952 just as @vue/language-tools v3 was released. This meant that Vue worked briefly (if at all) until it automatically updated to v3.

I fixed this by upgrading to v3 and correctly integrating with the typescript LSP in #2097. It hasn't been merged yet, but it has worked locally since.

Created this issue to bump the PR and just give a heads up that the Vue LSP does not work atm.

Love the work you guys are doing! opencode has been my daily driver for a while now! Thanks!

[github-actions[bot]]

This issue might be a duplicate of existing issues. Please check:

• #1952: Original Vue LSP issue created when @vue/language-tools v3 was released, causing the same diagnostic reporting problems

Feel free to ignore if none of these address your specific case.

[rekram1-node]

ty ill check out pr

[daleal]

This PR merged would be great! I love the LSP feature of opencode and I am missing it so much in Vue :( ❤️

[rekram1-node]

sorry this fell under the radar, ill check it out today

[daleal]

I know this week has been crazy af with the release of 1.x.x, so I am giving this a bump just so it doesn't get lost. Thank you! ❤️

[rekram1-node]

@daleal definitely acknowledged, just hard I need to resolve some merge conflicts for it and verify it works so once I find some time i can take more of a look, it just has had conflicts

[drevantonder]

@daleal definitely acknowledged, just hard I need to resolve some merge conflicts for it and verify it works so once I find some time i can take more of a look, it just has had conflicts

I'll see if I can get a chance to update the PR. Busy season/week for me atm with getting married, but would absolutely love to see this merged. 😆

[rekram1-node]

@drevantonder congrats!!!!

[drevantonder]

@daleal definitely acknowledged, just hard I need to resolve some merge conflicts for it and verify it works so once I find some time i can take more of a look, it just has had conflicts

I'll see if I can get a chance to update the PR. Busy season/week for me atm with getting married, but would absolutely love to see this merged. 😆

Sadly not looking like I'll have the time. I'll return December and update then if still pending.

[daleal]

Congrats @drevantonder!!! I opened this PR that is basically the exact same code you uploaded (I tested locally that the fix works too) but without the conflicts!