-
Notifications
You must be signed in to change notification settings - Fork 295
Add server-level description field to MCP runtime configuration #3016
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?
Conversation
Co-authored-by: JerryNixon <1749983+JerryNixon@users.noreply.github.com>
Co-authored-by: JerryNixon <1749983+JerryNixon@users.noreply.github.com>
…sponse object creation Co-authored-by: JerryNixon <1749983+JerryNixon@users.noreply.github.com>
Co-authored-by: JerryNixon <1749983+JerryNixon@users.noreply.github.com>
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.
Pull request overview
This PR adds a server-level description field to the MCP runtime configuration that enables DAB to expose semantic guidance to MCP clients via the initialize response's instructions field.
- Adds optional
descriptionparameter to MCP runtime configuration model - Implements CLI support for configuring the description via
dab configure --runtime.mcp.description - Wires the description value into the MCP initialize response as the
instructionsfield
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
src/Config/ObjectModel/McpRuntimeOptions.cs |
Adds Description property to store the server-level description |
src/Config/Converters/McpRuntimeOptionsConverterFactory.cs |
Implements serialization/deserialization for the description field |
src/Cli/ConfigGenerator.cs |
Updates configuration generator to handle description parameter and adds it to the MCP configuration check |
src/Cli/Commands/ConfigureOptions.cs |
Adds CLI option --runtime.mcp.description for setting the description |
src/Cli.Tests/ConfigureOptionsTests.cs |
Adds unit tests for setting and updating MCP description via CLI |
src/Azure.DataApiBuilder.Mcp/Core/McpStdioServer.cs |
Retrieves description from runtime config and includes it as instructions in initialize response |
Comments suppressed due to low confidence (1)
src/Azure.DataApiBuilder.Mcp/Core/McpStdioServer.cs:209
- The serialization uses JsonSerializer.Serialize without specifying JsonSerializerOptions. By default, System.Text.Json includes null properties in the output, which means the instructions field will be present with a null value when the description is not configured. To omit null properties from the response, you should configure JsonSerializerOptions with DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull. This is important for MCP protocol compliance where optional fields should be omitted rather than included with null values.
string json = JsonSerializer.Serialize(response);
Console.Out.WriteLine(json);
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Write description if it's provided | ||
| if (value is not null && !string.IsNullOrWhiteSpace(value.Description)) | ||
| { | ||
| writer.WritePropertyName("description"); | ||
| JsonSerializer.Serialize(writer, value.Description, options); | ||
| } |
Copilot
AI
Dec 12, 2025
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.
While CLI configuration tests exist, there are no unit tests verifying the serialization and deserialization of the description field in McpRuntimeOptions. Consider adding tests to verify that the description is correctly serialized to JSON and deserialized back, especially edge cases like empty strings, very long strings, and strings with special characters.
| // Get the description from runtime config if available | ||
| string? instructions = null; | ||
| try | ||
| { | ||
| RuntimeConfigProvider? runtimeConfigProvider = _serviceProvider.GetService<RuntimeConfigProvider>(); | ||
| if (runtimeConfigProvider != null) | ||
| { | ||
| RuntimeConfig runtimeConfig = runtimeConfigProvider.GetConfig(); | ||
| instructions = runtimeConfig.Runtime?.Mcp?.Description; | ||
| } | ||
| } | ||
| catch (Exception ex) | ||
| { | ||
| // If we can't get the config, continue without instructions | ||
| // Log to stderr for diagnostics | ||
| Console.Error.WriteLine($"[MCP DEBUG] Failed to retrieve MCP description from config: {ex.Message}"); | ||
| } | ||
|
|
||
| // Create the initialize response | ||
| var result = new | ||
| { | ||
| protocolVersion = _protocolVersion, | ||
| capabilities = new | ||
| { | ||
| tools = new { listChanged = true }, | ||
| logging = new { } | ||
| }, | ||
| serverInfo = new | ||
| { | ||
| name = "Data API Builder", | ||
| version = "1.0.0" | ||
| }, | ||
| instructions = !string.IsNullOrWhiteSpace(instructions) ? instructions : null | ||
| }; |
Copilot
AI
Dec 12, 2025
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 HandleInitialize method now includes logic to retrieve and include the description in the response, but there are no unit tests verifying this behavior. Consider adding tests to ensure that: 1) when a description is configured, it appears in the instructions field of the initialize response, 2) when no description is configured or it's empty, the instructions field is omitted from the response, and 3) the error handling path works correctly when the RuntimeConfigProvider service is unavailable or throws an exception.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Why make this change?
Closes #3015
MCP clients and agents require high-level behavioral context for servers via the
initializeresponse'sinstructionsfield. DAB previously had no mechanism to surface this configurable semantic guidance.What is this change?
Added optional
descriptionfield to MCP runtime configuration that populates the MCP protocol'sinstructionsfield:Configuration model
McpRuntimeOptionsnow acceptsdescriptionparameterMcpRuntimeOptionsConverterhandles serialization/deserializationCLI integration
dab configure --runtime.mcp.description "text"command supportMCP server response
HandleInitialize()retrieves description fromRuntimeConfig.Runtime.Mcp.Descriptioninstructionsin initialize response when non-emptyExample configuration:
{ "runtime": { "mcp": { "enabled": true, "description": "This MCP provides access to the Products database..." } } }How was this tested?
Sample Request(s)
CLI usage:
dab configure --runtime.mcp.description "This MCP provides access to the Products database and should be used to answer product-related or inventory-related questions from the user."MCP initialize response (when description configured):
{ "jsonrpc": "2.0", "id": 1, "result": { "protocolVersion": "2024-11-05", "capabilities": { ... }, "serverInfo": { ... }, "instructions": "This MCP provides access to the Products database..." } }Original prompt
descriptionvalue #3015💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.