Problem
The feedback tool definition in internal/mcp/tools.go defines the parameter as rating, but handleFeedback in server.go reads the field quality. This causes a confusing round-trip:
- Agent calls
feedback({rating: "helpful"}) per the schema
- Handler returns error: "quality parameter is required"
- Agent has to guess the correct param name
Fix
Either rename the schema param from rating to quality, or update the handler to accept rating. Schema should match handler.
Problem
The
feedbacktool definition ininternal/mcp/tools.godefines the parameter asrating, buthandleFeedbackinserver.goreads the fieldquality. This causes a confusing round-trip:feedback({rating: "helpful"})per the schemaFix
Either rename the schema param from
ratingtoquality, or update the handler to acceptrating. Schema should match handler.