Vino1016 opened a new pull request, #583: URL: https://github.com/apache/flink-agents/pull/583
Linked issue: #578 ### Purpose of change Change `org.apache.flink.agents.integrations.mcp.MCPPrompt.PromptArgument.required` from primitive `boolean` to wrapper `Boolean`, and default `required = null` to `false`. This improves compatibility with MCP prompt argument payloads where `required` may be omitted or explicitly `null`, while keeping required-argument behavior stable. ### Tests Updated unit test in `org.apache.flink.agents.integrations.mcp.MCPPromptTest`: - `testPromptArgumentDefaultsRequiredToFalseWhenNull` verifies `required = null` is normalized to `false`. ### API Yes, this touches public Java API in `MCPPrompt.PromptArgument`: - Constructor parameter changed from `boolean` to `Boolean`. - `isRequired()` return type changed from `boolean` to `Boolean`. - Behavior update: `null` is accepted and treated as `false`. ### Documentation - [ ] `doc-needed` - [x] `doc-not-needed` - [ ] `doc-included` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
