Skip to content

feat: get_service_detail#1475

Open
nagisa-kunhah wants to merge 1 commit into
apache:aifrom
nagisa-kunhah:feat/mcp-detail-tools
Open

feat: get_service_detail#1475
nagisa-kunhah wants to merge 1 commit into
apache:aifrom
nagisa-kunhah:feat/mcp-detail-tools

Conversation

@nagisa-kunhah
Copy link
Copy Markdown

Please provide a description of this PR:

Changes

  • Add get_service_detail tool in DetailRegistrar to return service language and methods
  • Rename get_service_detail in ServiceRegistrar to get_service_distribution to better reflect its purpose (querying provider/consumer app distribution)
  • Remove invalid providerAppName field from search results
  • Add unit tests for detail_tools_test.go

Bug fixes

  • Fix SSE connection leak when request context is cancelled
  • Fix goroutine leak from unclosed pipe in stdio integration test
  • Fix handler signature and NewTextResult call in http_test.go
  • Code formatting (import ordering, map literal alignment)

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

  • Docs
  • Installation
  • User Experience
  • Dubboctl
  • Console
  • Core Component

Please check any characteristics that apply to this pull request.

@sonarqubecloud
Copy link
Copy Markdown

@nagisa-kunhah nagisa-kunhah marked this pull request as ready for review May 21, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant