X-LightYear opened a new pull request, #7341: URL: https://github.com/apache/rocketmq-dashboard/pull/7341
## Summary - preserve the optional `instanceId` when projecting alert rules for `rmq.alert.rule.list` - declare the additive field in the MCP output schema - add deterministic serialization and schema regression coverage ## Root cause `AlertRuleListItem.from(AlertRuleVO)` dropped `AlertRuleVO.instanceId`. The tool lists rules across the whole Studio deployment, so the serialized result could not identify the Instance to which a rule was bound. ## Fix Copy the optional instance binding into the MCP projection and document it in the catalog. Global rules continue to omit the field through the existing non-null serialization policy. ## Testing - pre-fix serialization regression failed because `instanceId` was absent - `AlertRuleListToolHandlerTest`: passed - `ToolOutputSchemaContractTest`: passed - Checkstyle: 0 violations - `git diff --check`: passed Fixes #7340 -- 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]
