Azure / Azure/data-api-builder

[Bug]: Mcp valid but wrong

Abierto Apto para principiantes
#3,412 0 comentarios 0 reacciones 0 asignados Ver en GitHub
2.x mcp-server
Lenguaje dominante
C#
Estrellas
1.5k
Forks
370
Merge medio
3 d 22 h
PR fusionados (30 d)
9

Descripción

## 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!

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Inspecciona la definición del esquema JSON 2.0 para la configuración de mcp a nivel de entidad e identifica dónde se acepta la forma abreviada booleana. Verifica que los valores de mcp de entidad deban usar la forma de objeto, mientras que runtime.mcp.enabled siga aceptando su forma abreviada; actualiza o ejecuta las comprobaciones de validación del esquema relevantes, si existen.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
csharp
Área
api
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Tranquilo
Claridad
Bien especificado
Aptitud para principiantes
76/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.