Allow application/vnd.api+json; ext=bulk

Author: jedisct1

pkg/openapi2mcp/register.go

@@ -1096,12 +1096,12 @@ func RegisterOpenAPITools(server *mcpserver.MCPServer, ops []OpenAPIOperation, d
 			var body []byte
 			var requestContentType string
 			if opCopy.RequestBody != nil && opCopy.RequestBody.Value != nil {
-				// Check for application/json first, then application/vnd.api+json
-				mt := opCopy.RequestBody.Value.Content.Get("application/json")
+				// Check for application/json first, then application/vnd.api+json (including with parameters)
+				mt := getContentByType(opCopy.RequestBody.Value.Content, "application/json")
 				if mt != nil {
 					requestContentType = "application/json"
 				} else {
-					mt = opCopy.RequestBody.Value.Content.Get("application/vnd.api+json")
+					mt = getContentByType(opCopy.RequestBody.Value.Content, "application/vnd.api+json")
 					if mt != nil {
 						requestContentType = "application/vnd.api+json"
 					}

pkg/openapi2mcp/schema.go

@@ -182,14 +182,19 @@ func BuildInputSchema(params openapi3.Parameters, requestBody *openapi3.RequestB
 	// Request body (application/json and application/vnd.api+json)
 	if requestBody != nil && requestBody.Value != nil {
 		for mtName := range requestBody.Value.Content {
-			if mtName != "application/json" && mtName != "application/vnd.api+json" {
+			// Check base content type without parameters
+			baseMT := mtName
+			if idx := strings.IndexByte(mtName, ';'); idx > 0 {
+				baseMT = strings.TrimSpace(mtName[:idx])
+			}
+			if baseMT != "application/json" && baseMT != "application/vnd.api+json" {
 				fmt.Fprintf(os.Stderr, "[WARN] Request body uses media type '%s'. Only 'application/json' and 'application/vnd.api+json' are fully supported.\n", mtName)
 			}
 		}
-		// Try application/json first, then application/vnd.api+json
-		mt := requestBody.Value.Content.Get("application/json")
+		// Try application/json first, then application/vnd.api+json (including with parameters)
+		mt := getContentByType(requestBody.Value.Content, "application/json")
 		if mt == nil {
-			mt = requestBody.Value.Content.Get("application/vnd.api+json")
+			mt = getContentByType(requestBody.Value.Content, "application/vnd.api+json")
 		}
 		if mt != nil && mt.Schema != nil && mt.Schema.Value != nil {
 			bodyProp := extractProperty(mt.Schema)

pkg/openapi2mcp/types.go

@@ -3,6 +3,12 @@
 // and github.com/jedisct1/openapi-mcp/pkg/mcp/server
 package openapi2mcp
 
+import (
+	"strings"
+
+	"github.com/getkin/kin-openapi/openapi3"
+)
+
 // LintIssue represents a single linting issue found in an OpenAPI spec
 type LintIssue struct {
 	Type       string `json:"type"`                // "error" or "warning"
@@ -28,3 +34,24 @@ type LintResult struct {
 type HTTPLintRequest struct {
 	OpenAPISpec string `json:"openapi_spec"` // The OpenAPI spec as a YAML or JSON string
 }
+
+// getContentByType finds content in an OpenAPI Content map by base content type,
+// ignoring parameters like charset or extensions.
+// For example, it will match "application/vnd.api+json; ext=bulk" when looking for "application/vnd.api+json"
+func getContentByType(content openapi3.Content, baseType string) *openapi3.MediaType {
+	// First try exact match for performance
+	if mt := content.Get(baseType); mt != nil {
+		return mt
+	}
+
+	// Then check for content types that start with the base type followed by semicolon
+	for contentType, mediaType := range content {
+		// Split on semicolon to get base content type
+		parts := strings.Split(contentType, ";")
+		if len(parts) > 0 && strings.TrimSpace(parts[0]) == baseType {
+			return mediaType
+		}
+	}
+
+	return nil
+}