doc: add MCP_GATEWAY_API_KEY and MCP_GATEWAY_CONTAINER_RUNTIME to AGENTS.md#10085
Conversation
There was a problem hiding this comment.
Pull request overview
Documents two supported environment variables in the primary agent reference, resolving #10084.
Changes:
- Documents the deprecated
MCP_GATEWAY_API_KEYalias. - Documents the Docker/Podman runtime override.
Show a summary per file
| File | Description |
|---|---|
AGENTS.md |
Adds both missing environment-variable entries. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Medium
🔒 mcpg Read-Only Stress — defaultSurface coverage: MCP tool calls + proxied CLI (REST) + GraphQL mutations
Notes:
Overall: PASS References: §30179062009
|
🔒 mcpg Read-Only Stress — gVisorSurface coverage: MCP tool calls + proxied CLI (REST) + GraphQL mutations
Overall: PASS Gateway enforcement detail (Part B): The gateway exposes exactly 22 read-only tools ( References: §30179061989
|
🔒 mcpg Read-Only Stress — docker-sbxSurface coverage: MCP tool calls + proxied CLI (REST) + GraphQL mutations
Overall: PASS Notes:
References: §30179061988
|
AGENTS.mdwas missing two environment variables that are documented inREADME.mdanddocs/ENVIRONMENT_VARIABLES.md, leaving AI agents usingAGENTS.mdas their primary reference unaware of these options.Changes
MCP_GATEWAY_API_KEY— added as a deprecated alias forMCP_GATEWAY_AGENT_ID(recognized ininternal/config/validation_env.goandrun_containerized.sh); still accepted with a deprecation warning at lower precedenceMCP_GATEWAY_CONTAINER_RUNTIME— added to document runtime override for JSON stdincontainerservers (dockerdefault,podmansupported; read ininternal/config/container_runtime.go)Both entries are placed immediately after
MCP_GATEWAY_AGENT_IDin the Environment Variables section.