tju-yxq opened a new pull request, #3052:
URL: https://github.com/apache/rocketmq-dashboard/pull/3052

   ## What is the purpose of the change
   
   The AI/MCP/CLI inventory tools returned complete inventories in one 
response. `rmq.topic.list` and `rmq.group.list` bypassed the provider-aware 
paged service methods, and `rmq.alert.rule.list` loaded every rule before 
filtering in memory. Large instances could therefore return thousands of 
projected rows to an AI client in one call.
   
   ## What changed
   
   - add optional `page` and `pageSize` inputs to the topic, consumer-group, 
and alert-rule tools
   - default to page 1 and 20 rows, with the catalog enforcing `pageSize <= 100`
   - route topic and consumer-group tools through 
`MetadataService.listTopicsPage` and `listConsumerGroupsPage`
   - route the alert-rule tool through the existing paged 
`AlertService.listRules(search, enabled, page, pageSize)`
   - change these three tool outputs from bare arrays to `{items, total, page, 
size}` objects while retaining the existing item projections
   - update gateway tests to assert the paged service calls, bounded 
projections, and page metadata
   
   ## Verification
   
   - focused tests: `ToolCatalogTest,ToolGatewayServiceTest` (38 tests) passed
   - Checkstyle: 0 violations
   - full server suite: `mvn -DskipTests=false test` (1,960 tests, 0 failures, 
0 errors)
   - production/config diff: 281 additions and 136 deletions across three 
handlers and the tool catalog
   
   Closes #3051


-- 
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]

Reply via email to