Azure / Azure/data-api-builder

[Bug]: Mcp valid but wrong

Open
#3,412 0 comments 0 reactions 0 assignees View on GitHub
2.x mcp-server
Dominant language
C#
Stars
1.5k
Forks
370
Avg merge
3d 17h
Merged PRs (30d)
8

Description

## What?

There is a Boolean shorthand on entities currently supported in the 2.0 JSON schema:

```json
{
"entities": {
"Products": {
"mcp": true
},
"SensitiveData": {
"mcp": false
}
}
}
```

However, here's the problem:

```json
{
"entities": {
"Products": {
"mcp": {
"dml-tools": true,
"custom-tool": false
}
},
"SensitiveData": {
"mcp": {
"dml-tools": false,
"custom-tool": false
}
}
}
}
```

### This is confusing

It is unclear which sub-property the shorthand is actually toggling.

## Fix

Remove the shorthand on entities, I have already removed it from the docs.

### Not in runtime!

Do not remove the `runtime.mcp.enabled` shorthand, that is still great!

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.