Azure / Azure/data-api-builder
[Bug]: Mcp valid but wrong
- Vorherrschende Sprache
- C#
- Sterne
- 1.5k
- Forks
- 370
- Ø Merge
- 3 T. 22 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
## 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!
Beitragsleitfaden
Rechercherichtung
Untersuche die 2.0 JSON-Schemadefinition auf die mcp-Konfiguration auf Entitätsebene und ermittle, wo die boolesche Kurzform akzeptiert wird. Überprüfe, dass mcp-Werte auf Entitätsebene die Objektform verwenden müssen, während runtime.mcp.enabled weiterhin die Kurzform akzeptiert; aktualisiere die relevanten Schemavalidierungsprüfungen, falls vorhanden, oder führe sie aus.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- csharp
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 76/100