skip operations with a request body with no supported MIME type

g-linville · g-linville · commit aa0c094e8245 · 2024-04-01T16:18:23.000-04:00
Signed-off-by: Grant Linville &lt;grant@acorn.io&gt;
diff --git a/pkg/loader/openapi.go b/pkg/loader/openapi.go
@@ -51,6 +51,7 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {
 			}
 		}
 
+	operations:
 		for method, operation := range pathObj.Operations() {
 			// Handle operation-level server override, if one exists
 			operationServer := pathServer
@@ -147,6 +148,11 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {
 					tool.Parameters.Arguments.Properties["requestBodyContent"] = &openapi3.SchemaRef{Value: arg}
 					break
 				}
+
+				if bodyMIME == "" {
+					// No supported MIME types found, so just skip this operation and move on.
+					continue operations
+				}
 			}
 
 			// OpenAI will get upset if we have an object schema with no properties,

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,7 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {`
`51`	`51`	`}`
`52`	`52`	`}`
`53`	`53`
	`54`	`+ operations:`
`54`	`55`	`for method, operation := range pathObj.Operations() {`
`55`	`56`	`// Handle operation-level server override, if one exists`
`56`	`57`	`operationServer := pathServer`
`@@ -147,6 +148,11 @@ func getOpenAPITools(t *openapi3.T, defaultHost string) ([]types.Tool, error) {`
`147`	`148`	`tool.Parameters.Arguments.Properties["requestBodyContent"] = &openapi3.SchemaRef{Value: arg}`
`148`	`149`	`break`
`149`	`150`	`}`
	`151`	`+`
	`152`	`+ if bodyMIME == "" {`
	`153`	`+ // No supported MIME types found, so just skip this operation and move on.`
	`154`	`+ continue operations`
	`155`	`+ }`
`150`	`156`	`}`
`151`	`157`
`152`	`158`	`// OpenAI will get upset if we have an object schema with no properties,`