feat(registry): register the AGV MCP server - #9
Open
NagyVikt wants to merge 2 commits into
Open
Conversation
added 2 commits
August 13, 2026 14:55
Constraint: Resolve HOME through bash because MCP argument expansion is not portable. Tested: python3 -m json.tool on Claude and Codex registries.
Describe the prepare/execute token protocol, the expanded read-only tool surface, and the AGV_MCP_* state directories. Restate the hard boundary: motor arming, /cmd_vel, navigation goals, and arbitrary mutating services stay unexposed even with a valid token.
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
1 participant
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.
Registers the AGV MCP adapter in the Claude and Codex snapshots, alongside the
ted-eu configs that landed in #3.
What
agv-mcptoconfigs/claude.sanitized.jsonandconfigs/codex.sanitized.json.mcps/agv-mcp/skills.md: the prepare/execute tokenprotocol, the read-only tool surface, and the
AGV_MCP_*state directories.Notes
The launcher goes through
bash -lcbecause MCP clients do not consistentlyexpand
${HOME}in command arguments. No robot address, credential, or secretis stored in the registry; target selectors (
AGVC_ROBOT,AGVC_HOST,AGVC_IP,AGVC_CONTAINER) are inherited from the client environment.Motor arming,
/cmd_vel, navigation goals, arbitrary mutating services, andarbitrary parameter writes stay unexposed even with a valid token.
Reconciliation
mainhad diverged locally. The AGV commit was rebased ontoa6704ac; theconfigs/claude.sanitized.jsonconflict was resolved as a union, soted-eu,apify-ted-eu, andagv-mcpare all present. Validated withjq.🤖 Generated with Claude Code