fix: run_containerized.sh now accepts MCP_GATEWAY_AGENT_ID as primary deployment gate variable#7212
Merged
Merged
Conversation
…mary env var - Update check_required_env_vars() in run_containerized.sh to accept MCP_GATEWAY_AGENT_ID (preferred) or MCP_GATEWAY_API_KEY (deprecated fallback with a warning), mirroring the existing behaviour in internal/config/validation_env.go - Update error hint message to show MCP_GATEWAY_AGENT_ID - Update CONTRIBUTING.md Docker examples to use MCP_GATEWAY_AGENT_ID / agentId instead of the deprecated MCP_GATEWAY_API_KEY / apiKey Fixes: README Quick Start was already using MCP_GATEWAY_AGENT_ID; the container startup failure was caused by run_containerized.sh only checking MCP_GATEWAY_API_KEY. Resolves docs/ENVIRONMENT_VARIABLES.md line 13 (which already described the correct intended behaviour — now the script matches it).
Copilot
AI
changed the title
[WIP] Fix missing MCP_GATEWAY_API_KEY in Quick Start command
fix: run_containerized.sh now accepts MCP_GATEWAY_AGENT_ID as primary deployment gate variable
Jun 8, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Aligns container entrypoint env-var validation with the documented/implemented MCP_GATEWAY_AGENT_ID deployment gate, fixing container startup failures when following the Quick Start.
Changes:
- Update
run_containerized.shto acceptMCP_GATEWAY_AGENT_ID(preferred) orMCP_GATEWAY_API_KEY(deprecated) and emit a deprecation warning when using the legacy name. - Update Docker development examples in
CONTRIBUTING.mdto useMCP_GATEWAY_AGENT_ID/gateway.agentId.
Show a summary per file
| File | Description |
|---|---|
| run_containerized.sh | Updates required env-var checks to prefer MCP_GATEWAY_AGENT_ID and warn on legacy MCP_GATEWAY_API_KEY. |
| CONTRIBUTING.md | Updates Docker run/config examples to use MCP_GATEWAY_AGENT_ID and gateway.agentId. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 1
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.
run_containerized.shonly checkedMCP_GATEWAY_API_KEYwhile the README Quick Start (andvalidation_env.go) already usedMCP_GATEWAY_AGENT_ID, causing immediate container startup failure for anyone following the docs.Changes
run_containerized.sh—check_required_env_vars()now mirrorsvalidation_env.go: acceptsMCP_GATEWAY_AGENT_ID(preferred) or falls back toMCP_GATEWAY_API_KEYwith a deprecation warning. Error hint updated accordingly.CONTRIBUTING.md— Docker development examples updated fromMCP_GATEWAY_API_KEY/"apiKey"toMCP_GATEWAY_AGENT_ID/"agentId", consistent with README guidance.docs/ENVIRONMENT_VARIABLES.md— No change; line 13 already described the correct intended behaviour. The script now matches it.