You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Why is "speaking mcp protocol" tightly coupled with "manage json schemas for un/marshaling and validation" when there are better solutions for the latter?
#366
This lib basically forces you to use a limited builder pattern for schema definition and manual validation with poor type strictness. NewToolWithRawSchema does not work as expected; for whatever reason, the provided schema is not being picked up by the client (no description or args). Am I missing something or is this a design flaw of this library? The supported builder pattern is nice for throwaway tools, but this is not how gophers want to manage a typed protocol. I'd much rather bring my own json schema generator and validator so I define structs once and get JSON() and Parse() methods for free (well, the cost of filling out property tags) and not worry about introducing inconsistencies
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
This lib basically forces you to use a limited builder pattern for schema definition and manual validation with poor type strictness. NewToolWithRawSchema does not work as expected; for whatever reason, the provided schema is not being picked up by the client (no description or args). Am I missing something or is this a design flaw of this library? The supported builder pattern is nice for throwaway tools, but this is not how gophers want to manage a typed protocol. I'd much rather bring my own json schema generator and validator so I define structs once and get JSON() and Parse() methods for free (well, the cost of filling out property tags) and not worry about introducing inconsistencies
Beta Was this translation helpful? Give feedback.
All reactions