Fix rest api

[Maschga]

Fix #823

⚠️ fix validation errors
✔️ add check to validate openapi spec

\cc @andig @naltatis

[andig]

Nice! Hast Du rausgefunden, warum die operationIds anscheinend einen Unterschied machen?

[andig]

Wäre https://github.com/getkin/kin-openapi nicht der bessere Validator?

[Maschga]

See docs.

Ich denke, die wird die Library openapi-mcp brauchen.

[Maschga]

Ich habe die Validierung von openapi-mcp genommen, weil du du wahrscheinlich auf die Funktionalität dieser Library aufbauen wirst.

[andig]

Yep, aber erstmal gehts nur um Validierung, nicht um weiteres Tooling

operationId is an optional unique string used to identify an operation. If provided, these IDs must be unique among all operations described in your API.

Ich muss nochmal fragen: sind die Teil der Lösung oder optional?

[Maschga]

operationId ist Teil der Lösung. Wenn dieser Parameter nicht bei allen Endpoints gegeben ist, wirft openapi-mcp einen Fehler.

Wenn es dir nicht wichtig ist, dass auch validiert wird, ob dieser Parameter bei allen Endpoints vorhanden ist, kann ich gerne auf deinen Validator wechseln.

[andig]

Dann lass uns wechseln- openapi-mcp tut ohnehin nicht, was es soll und wird vstl. nicht Teil der Lösung sein. VIELEN DANK!

[Maschga]

Wenn openapi-mcp nicht das macht was es soll und ersetzt werden soll, ist die Frage, ob die operationIDs immer noch wichtig sind und hinzugefügt werden sollten.

[Maschga]

Bei #823 (comment) brauche ich von dir auch noch eine Antwort, bevor der PR gemergt wird.

[andig]

Sind mir aktuell nicht wichtig!

[Maschga]

Dann reverte ich den Commit mit den operationIDs. Wenn es noch wichtig wird, ist es sowieso mit dem Copilot innerhalb von Sekunden hinzugefügt. :D