-
Notifications
You must be signed in to change notification settings - Fork 800
Description
Hopefully this is appropriate, prior, we were able to provide just a string to the includeSchema portion. Though I do like the approach of a type being passed now, I feel like the current JsonSchema object is too lax and doesnt really capture the needs. While working on my mcp server, I figured out that the best schema to provide was to just use the least common denominator approach, which is to use https://ai.google.dev/api/caching#Schema as the basis. Even though OpenAI's schema is better and does work for those llms that support it, gemini didnt conform to it well because it uses its own schema spec. So I built my own builder https://github.com/themixednuts/GhidraMCP/tree/main/src/main/java/com/themixednuts/utils/jsonschema
Is this something we can move towards?