KCL extension for Zed - #13567
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Merging this PR will not alter performance
Comparing Footnotes
|
|
KCL lib will indeed read your API key from $ZOO_API_TOKEN, if it's set. Might want to let the extension take some sort of configuration to read the API key from somewhere else too. |
c1dd597 to
21ae5e1
Compare
|
Updated the main comment to note the folder reorganization effort and the test process for both vscode and zed clients. Includes a fix to keep the LSP server from reporting errors on both push and pull interactions. |
|
|
I had codex write a Zed extension wrapper around the KCL LSP. Jon pointed out we could better organize the directories associated with the LSP and clients, so that's been rolled into this PR as well.
Testing
Zed
I installed this extension in Zed as a "dev extension". At first it was reporting all errors twice. This has been fixed by preventing the lsp server from reporting errors on both push and pull interactions with the server. VSCode client had to be tested as well to validate this didn't break things over there.
modeling-app/rust/kcl-lsp-zed-clientfolderVSCode
Questions
ZOO_API_KEYenv variable automatically or is no key needed?After Merging
This extension needs to be added to the official extension library once, after which it seems hands-off on our end. The extension will pull the latest KCL LSP release automatically.
extensions.tomlpnpm sort-extensions