Skip to content

[instructions] Sync mcp-servers stdio example with MCP Gateway v0.1.5#30801

Merged
pelikhan merged 1 commit into
mainfrom
instructions/sync-mcp-gateway-v0-1-5-6a42d21536e95886
May 7, 2026
Merged

[instructions] Sync mcp-servers stdio example with MCP Gateway v0.1.5#30801
pelikhan merged 1 commit into
mainfrom
instructions/sync-mcp-gateway-v0-1-5-6a42d21536e95886

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 7, 2026

Instructions Update — MCP Gateway v0.1.5 breaking change

This PR updates .github/aw/syntax.md to reflect a breaking change in MCP Gateway v0.1.5 that was already enforced in the compiler but not yet reflected in the documentation.

Files Changed

  • syntax.md: Replace invalid command: "node" stdio example with correct Docker-based (container: + entrypoint:) and HTTP transport patterns.

Background

MCP Gateway v0.1.5 dropped support for non-Docker stdio MCP servers. The compiler (pkg/workflow/mcp_config_custom.go:71-80) now rejects any stdio server configured with a non-docker command (e.g. command: "node"). The previous example in syntax.md would fail compilation with an error like:

stdio MCP server uses command "node" which is not supported by MCP Gateway. Stdio servers must be containerized (use 'container' with 'entrypoint'), or switch to HTTP transport for servers that run directly on the runner.

The updated example shows both valid patterns:

  1. Container-based stdio (container: + entrypoint:) — for containerized tools
  2. HTTP transport (type: http + url:) — for Node/Python servers installed on the runner

Changesets Reviewed

  • minor-update-mcpg-v0-1-5.md — MCP Gateway v0.1.5 breaking rules (Docker-only stdio, no mounts on HTTP)
  • minor-rename-app-to-github-app.md — already documented ✓
  • minor-rename-safe-inputs-to-mcp-scripts.md — already documented ✓
  • minor-decouple-status-comment.md — already documented ✓
  • minor-disable-agent-sandbox-only.md — already documented ✓
  • minor-enable-reaction-status-comment-by-default.md — already documented ✓
  • minor-preserve-branch-name-in-safe-outputs.md — already documented ✓
  • minor-mcp-gateway-otlp-headers-string-only.md — affects mcp-gateway.opentelemetry.headers (undocumented advanced field, not in instruction files)
  • minor-add-cli-proxy-feature-flag.md — already documented ✓

Size Audit

File Before After Status
syntax.md 931 lines 940 lines ✓ (< 1000 limit)

Validation

  • Imperative tone, minimal example
  • Updated the correct sub-file (syntax.md)
  • No duplication introduced
  • File sizes within target limits
  • Verified against pkg/workflow/mcp_config_custom.go:71-80

Generated by Instructions Janitor · ● 635.3K ·

  • expires on May 9, 2026, 9:25 AM UTC

MCP Gateway v0.1.5 requires stdio servers to use Docker-based configuration
(container + entrypoint). The previous example using command: "node" is now
rejected by the compiler. Update the syntax.md example to show both the
Docker-based stdio pattern and the HTTP transport pattern for runner-installed
servers.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added automation documentation Improvements or additions to documentation instructions labels May 7, 2026
@pelikhan pelikhan merged commit d5e9466 into main May 7, 2026
@pelikhan pelikhan deleted the instructions/sync-mcp-gateway-v0-1-5-6a42d21536e95886 branch May 7, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation documentation Improvements or additions to documentation instructions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant