X-LightYear opened a new pull request, #5017:
URL: https://github.com/apache/rocketmq-dashboard/pull/5017

   ## Summary
   
   - expose the existing managed Studio instance directory through the MCP tool 
catalog as `rmq.instance.list`
   - let planners filter by access type or the existing instance search query 
before selecting a canonical `instanceId`
   - return resource counts and count availability without exposing endpoint or 
credential fields
   
   ## Why
   
   Studio already serves the managed instance directory through `GET 
/api/instances`, and the web console uses it to select canonical Studio 
instance names. MCP exposed `rmq.instance.capabilities`, but an agent had no 
read-only discovery tool to learn valid instance targets in a multi-instance 
deployment.
   
   ## Implementation
   
   - reuse `InstanceService.listInstances(type, search)` so existing repository 
filtering, vendor ordering, provider resource counts, and per-instance 
count-unavailable behavior remain unchanged
   - add a safe `InstanceListItem` projection containing canonical instance ID, 
vendor/access type, region, counts, and count availability
   - register the tool as a platform-level L1 read-only tool with no required 
instanceId or capability lookup
   - omit `credentialId`, `adminCredentialRef`, cloud credential identifiers, 
and endpoints from the MCP output
   
   ## Testing
   
   - focused MCP/catalog/registration tests: 26 passed
   - instance service and REST neighbor tests: 103 passed
   - checkstyle: passed
   - `dev` profile full server suite: 3195 tests; 7 failures and 9 errors, all 
existing CLI-agent environment tests requiring `sh`/`cat` on Windows; no 
instance/MCP failures
   - package compilation completed; binary-license gate could not run because 
this sandbox has no `python3` executable
   
   ## Adversarial review
   
   - confirmed platform-level binding convention and catalog schema validation
   - confirmed the handler ignores the ambient target and delegates only the 
explicit filters
   - confirmed the projection cannot serialize credential fields
   - checked related #4957 (frontend physical-cluster/instance identity) and 
#3051/#3052 (large topic/group/alert inventory pagination); neither provides 
managed-instance MCP discovery
   
   Closes #5016


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