Repository navigation
[Automated] Draft docs (kagent): feat: add token passthrough for remote MCP servers - #646
Open
github-actions[bot] wants to merge 1 commit into
Open
github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
…te MCP servers Signed-off-by: GitHub Action <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
kagent-dev/kagent#3257 — feat: add token passthrough for remote MCP servers
Docs versions:
1.x(first shipped in v1.0.0-alpha15)No code-path rule covers 7 touched file(s). The planner could route these only by identifier search, so the candidate pages may miss what they changed, including in the other mode:
docs/architecture/credential-injection.mdgo/api/gen/kagent/config/v1/adk.pb.gogo/harness/codex/README.mdproto/kagent/config/v1/adk.protopython/packages/kagent-adk/src/kagent/adk/types.pypython/packages/kagent-proto/src/kagent/config/v1/adk_pb2.pyWhat changes for users: RemoteMCPServer can set
spec.tokenPassthrough: trueto send the incoming A2A bearer token to the MCP server asAuthorization: Bearer <token>. Requests without a caller token fail, and the API server rejects a RemoteMCPServer that combinestokenPassthroughwith anAuthorizationentry inheadersFrom. ModelConfigapiKeyPassthroughis renamed totokenPassthrough.What the docs now say: Your first MCP tool now shows a complete RemoteMCPServer example for caller-token authentication, including the discovery label and the
headersFromconflict. Agent harness now states thatcodexandclaudereject RemoteMCPServertokenPassthrough.Not verified: No cluster; the configuration and commands are unrun.
How
kagent-3257was drafted, and what was not verifiedPlan, and what changed it
tokenPassthroughto the MCP tool documentation and note harness restrictions.apiKeyPassthroughtotokenPassthrough, from the target copies and the dossier grep.your-first-mcp-tool.mdand added thecodex/clauderestriction toagent-harness.md.Why a documentation change is necessary
The pull request adds
RemoteMCPServer.spec.tokenPassthroughand a validation rule that rejectsheadersFromentries namedAuthorizationwhen passthrough is enabled. The existing MCP task guide shows how to create a RemoteMCPServer, but it did not show this authentication path or the discovery label that the pull request body says servers that require a caller token should set. The existing harness page listed MCP restrictions forcodexandclaude, but not the new passthrough rejection.What changed on disk
docs-site/content/kagent/1.x/get-started/your-first-mcp-tool.md: Added a caller-token RemoteMCPServer example, field table, and harness caveat.docs-site/content/kagent/1.x/agents/agent-harness.md: Added RemoteMCPServertokenPassthroughto thecodexandclaudetool restrictions.How to verify this change
spec.tokenPassthrough: trueand thekagent.dev/discovery: disabledlabel.Authorizationentry inspec.headersFrom. The API server should reject it withheadersFrom cannot set Authorization when tokenPassthrough is enabled.Authorization: Bearer <caller token>on its request.Proposed release note
No
proposed-release-note.md:markers.notes_placeableis false for this repository. Suggested note: Remote MCP servers can now setspec.tokenPassthroughto forward the caller's A2A bearer token as the server'sAuthorizationheader. ModelConfigspec.apiKeyPassthroughis renamed tospec.tokenPassthrough.What was not verified
How the pages were chosen (triage report)
Feature clusters
Kept apart on purpose. Merging wrongly produces a guide that documents two things and explains neither, so these are reported for a person to merge rather than grouped automatically:
docs-site/content/kagent/1.x/observability/launch-ui.mdas a candidate, but for one of them it is one page among more than 3, so they are kept apart; merge them if both drafts do land thereDraft a documentation change
kagent-dev/kagent#3257 — feat: add token passthrough for remote MCP servers
kagent-dev/websiteeitan/remote-mcp-token-passthrough. It contains only the passthrough feature and the API rename.docs/architecture/credential-injection.mdgo/api/gen/kagent/config/v1/adk.pb.gogo/harness/codex/README.mdproto/kagent/config/v1/adk.protopython/packages/kagent-adk/src/kagent/adk/types.pypython/packages/kagent-proto/src/kagent/config/v1/adk_pb2.pyDraft branch:
pr-tracker-draft-kagent-3257