-
Notifications
You must be signed in to change notification settings - Fork 829
Update MCP server template readme to show both VS Code and Visual Studio notes #6591
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
Conversation
Co-authored-by: timheuer <[email protected]>
@timheuer / @joelverhagen / @leslierichardson95 - I suggest we keep with a single README and have sections for both VS and VS Code, regardless of how the template is created. That could coach users to see how to test in both of the IDEs. I personally tend to create new projects from the CLI and then open the project in one of the IDEs. If you are onboard with having a single README with IDE sections, we can ask Copilot to revise this PR. |
Co-authored-by: jeffhandley <[email protected]>
I've updated the approach to use a single README with sections for both VS Code and Visual Studio as suggested. The template now provides guidance for both IDEs regardless of how it's created. Changes in commit a4ba77f. Warning Firewall rules blocked me from connecting to one or more addressesI tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/src/McpServer/McpServer-CSharp/README.md
Outdated
Show resolved
Hide resolved
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/src/McpServer/McpServer-CSharp/README.md
Outdated
Show resolved
Hide resolved
Co-authored-by: timheuer <[email protected]>
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.
The README is now quite repetitive with the Visual Studio and VS Code portions almost identical. Restructure it to reduce repetition while highlighting the differences between the IDEs.
Co-authored-by: timheuer <[email protected]>
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addressesI tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
…udio sections Co-authored-by: jeffhandley <[email protected]>
Restructured the README to significantly reduce repetition by combining similar sections and highlighting key differences between IDEs. The new structure consolidates "Using the MCP Server" and "Developing locally" sections with clear subsections for each IDE, eliminating duplicated content while maintaining clarity about IDE-specific configurations. Changes in commit 387d9ab. Warning Firewall rules blocked me from connecting to one or more addressesI tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/src/McpServer/McpServer-CSharp/README.md
Outdated
Show resolved
Hide resolved
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/src/McpServer/McpServer-CSharp/README.md
Outdated
Show resolved
Hide resolved
@copilot - these changes need to be copied into the corresponding snapshot. Look at the original PR for the MCP template to see an example. |
Co-authored-by: joelverhagen <[email protected]>
Updated the corresponding snapshot to match the template source with the latest README changes in commit 5dd0f87. Warning Firewall rules blocked me from connecting to one or more addressesI tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
...ensions.AI.Templates.IntegrationTests/Snapshots/mcpserver.Basic.verified/mcpserver/README.md
Outdated
Show resolved
Hide resolved
…dio notes (dotnet#6591) * Initial plan * Add VS IDE-specific README and update template configuration Co-authored-by: timheuer <[email protected]> * Update to single README with both VS Code and Visual Studio sections Co-authored-by: jeffhandley <[email protected]> * Fix Visual Studio MCP documentation URL Co-authored-by: timheuer <[email protected]> * Fix Visual Studio MCP JSON configuration format Co-authored-by: timheuer <[email protected]> * Restructure README to reduce repetition between VS Code and Visual Studio sections Co-authored-by: jeffhandley <[email protected]> * Eliminate repetitive JSON configuration in README by consolidating server definitions Co-authored-by: jeffhandley <[email protected]> * Revise the mcp server template README * Update MCP template README paths and sync snapshot with source template Co-authored-by: joelverhagen <[email protected]> * Update mcpserver project template baseline * Bump MEAI.Templates package to preview.3. * Add feedback survey to mcpserver project template README --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: timheuer <[email protected]> Co-authored-by: jeffhandley <[email protected]> Co-authored-by: Jeff Handley <[email protected]> Co-authored-by: joelverhagen <[email protected]>
…dio notes (#6591) * Initial plan * Add VS IDE-specific README and update template configuration Co-authored-by: timheuer <[email protected]> * Update to single README with both VS Code and Visual Studio sections Co-authored-by: jeffhandley <[email protected]> * Fix Visual Studio MCP documentation URL Co-authored-by: timheuer <[email protected]> * Fix Visual Studio MCP JSON configuration format Co-authored-by: timheuer <[email protected]> * Restructure README to reduce repetition between VS Code and Visual Studio sections Co-authored-by: jeffhandley <[email protected]> * Eliminate repetitive JSON configuration in README by consolidating server definitions Co-authored-by: jeffhandley <[email protected]> * Revise the mcp server template README * Update MCP template README paths and sync snapshot with source template Co-authored-by: joelverhagen <[email protected]> * Update mcpserver project template baseline * Bump MEAI.Templates package to preview.3. * Add feedback survey to mcpserver project template README --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: timheuer <[email protected]> Co-authored-by: jeffhandley <[email protected]> Co-authored-by: Jeff Handley <[email protected]> Co-authored-by: joelverhagen <[email protected]>
The MCP server template README was hardcoded for VS Code, but the template also works in Visual Studio IDE which has MCP support. This change makes the README context-aware based on the host identifier.
Changes Made
Created
README.VisualStudio.md
- New Visual Studio-specific README with:Updated
template.json
- Added conditional sources with modifiers:dotnetcli
/dotnetcli-preview
): Uses original README.md (VS Code focused)Before vs After
CLI/dotnet new (VS Code focused):
Visual Studio IDE (Visual Studio focused):
The implementation follows the same pattern used by the ChatWithCustomData template for conditional README files, ensuring consistency with existing template patterns.
Fixes #6590.
Warning
Firewall rules blocked me from connecting to one or more addresses
I tried to connect to the following addresses, but was blocked by firewall rules:
11vvsblobprodcus336.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)8xbvsblobprodcus382.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)d0svsblobprodcus381.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)e7bvsblobprodcus348.vsblob.vsassets.io
/home/REDACTED/work/extensions/extensions/.dotnet/dotnet msbuild /m /nologo /clp:Summary /v:minimal /nr:true /p:TreatWarningsAsErrors=false /p:ContinuousIntegrationBuild=false /home/REDACTED/work/extensions/extensions/artifacts/toolset/restore.proj /t:__WriteToolsetLocation /clp:ErrorsOnly;NoSummary /p:__ToolsetLocationOutputFile=/home/REDACTED/work/extensions/extensions/artifacts/toolset/9.0.0-beta.25325.4.txt
(dns block)dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)i1qvsblobprodcus353.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)imzvsblobprodcus368.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)jd4vsblobprodcus366.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)kgfvsblobprodcus314.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)kmuvsblobprodcus389.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)l49vsblobprodcus358.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)ljcvsblobprodcus317.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)ofvvsblobprodcus315.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)sc4vsblobprodcus331.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)ukkvsblobprodcus352.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)ytvvsblobprodcus310.vsblob.vsassets.io
dotnet test test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Microsoft.Extensions.AI.Templates.Tests.csproj --filter McpServerSnapshot --logger console;verbosity=detailed -v m
(dns block)If you need me to access, download, or install something from one of these locations, you can either:
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.