-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Add NuGet MCP server #4601
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
base: main
Are you sure you want to change the base?
Add NuGet MCP server #4601
Conversation
|
We require contributors to sign our Contributor License Agreement, and we don't have @sandrohanea on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'. |
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this! Could you please briefly elaborate on the issues you ran into whilst attempting to directly invoke DLX? Can you try whether dotnet dnx uses more reliable results? I'd like to avoid the need for the wrapper script if possible.
Also, please follow the steps outlined as in https://zed.dev/docs/extensions/developing-extensions#publishing-your-extension as you are missing the entry in the extensions.toml here. Thanks!
|
Hello @MrSubidubi, Thanks for the follow-up! Here’s a concise summary of what I ran into:
From what I can see in Zed, the command returned by the extension is resolved relative to the extension directory, then executed via a non‑interactive shell with the environment inherited from Zed. There’s no extra PATH lookup in the extension API for Relevant paths:
Given that, I’d prefer not to hardcode paths or add platform‑specific PATH discovery in the extension. Would it be possible to expose a I see that currently, most of the extensions (if not all of them) are using Also noted on the publishing steps: I’ll add the missing entry in Thank you again! |
Add Zed extension to launch NuGet MCP server via dnx.
Couldn't start the server reliably with dnx command directly, so I had to create some js adapter to run in with node.
It is based on: https://devblogs.microsoft.com/dotnet/nuget-mcp-server-preview/
Is there any plan to allow dnx command directly from:
or equivalent?