stacklok / toolhive / 19370017296

51%

push

github

Fix backend capability name translation in vmcp (#2587)

When conflict resolution renames capabilities (e.g., prefix strategy
changes "fetch" to "fetch_fetch"), the backend client must translate
the client-facing name back to the original name when forwarding
requests to backends.

This fix centralizes name translation in the backend client by using
the `BackendTarget.GetBackendCapabilityName()` method for all capability
types (tools, resources, prompts).

Changes:
- Add name translation in CallTool(), ReadResource(), GetPrompt()
- Remove redundant translation from handler factory
- Update test expectations to match new behavior
- Add composite tools logging to config loader for visibility
- Document OriginalCapabilityName field with usage warnings
- Add comprehensive examples to GetBackendCapabilityName() method

The routing table already stores original capability names via
`OriginalCapabilityName` field, this fix ensures they're used correctly
when forwarding to backends.

Fixes composite tool execution where workflow steps reference prefixed
tool names but backends expect original names.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>

1 of 19 new or added lines in 3 files covered. (5.26%)

23246 of 48218 relevant lines covered (48.21%)

36.58 hits per line