Azure / Azure/data-api-builder

Follow up tasks for update_record tool

Open
#2,913 1 comment 0 reactions 1 assignee Assigned to @Aniruddh25 View on GitHub
mcp-server mssql
Dominant language
C#
Stars
1.5k
Forks
370
Avg merge
3d 17h
Merged PRs (30d)
8

Description

1. We should use the SqlResponseHelper to create a response that MCP endpoint needs to avoid the duplicate serialization and deserialization. (I think this is addressed in your PR`
2. If the ClientRoleHeaderAuthenticationMiddleware isn't invoked, the roleheader here will be empty. This needs to be tested.
3. Add CLI dab configure --runtime.mcp.dml-tools.update_record.enabled true
4. Update dab validate (warn when update_record.enabled and not mcp.enabled)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.