Skip to content

Reconcile proxy-mode coverage docs with current implementation#7484

Merged
lpcox merged 2 commits into
mainfrom
copilot/update-proxy-mode-pattern-count
Jun 13, 2026
Merged

Reconcile proxy-mode coverage docs with current implementation#7484
lpcox merged 2 commits into
mainfrom
copilot/update-proxy-mode-pattern-count

Conversation

Copilot AI commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

The docs understated proxy-mode coverage and did not clearly connect the container entrypoint’s host/mode overrides to the public environment variable reference. This updates the affected docs to match the current implementation surface.

  • Proxy mode coverage

    • Updated the README proxy-mode summary to reflect current routing coverage:
      • ~50 REST URL patterns
      • ~15 GraphQL query patterns
    • Kept the README phrasing approximate while pointing readers to docs/PROXY_MODE.md for the authoritative details.
  • Containerized env var documentation

    • Clarified docs/ENVIRONMENT_VARIABLES.md so the general note explicitly calls out run_containerized.sh using:
      • MCP_GATEWAY_HOST
      • MCP_GATEWAY_MODE
    • This aligns the note text with the existing container-specific table and the shell entrypoint behavior.
  • Documentation delta illustrated

    - This maps ~25 REST URL patterns and GraphQL queries to guard tool names
    + This maps ~50 REST URL patterns and ~15 GraphQL query patterns to guard tool names

GitHub Advanced Security started work on behalf of lpcox June 13, 2026 14:32 View session
GitHub Advanced Security finished work on behalf of lpcox June 13, 2026 14:34
Copilot AI changed the title [WIP] Update proxy mode pattern count in README.md Reconcile proxy-mode coverage docs with current implementation Jun 13, 2026
Copilot finished work on behalf of lpcox June 13, 2026 14:36
Copilot AI requested a review from lpcox June 13, 2026 14:36
GitHub Advanced Security started work on behalf of lpcox June 13, 2026 14:39 View session
GitHub Advanced Security finished work on behalf of lpcox June 13, 2026 14:41
@lpcox lpcox marked this pull request as ready for review June 13, 2026 15:49
Copilot AI review requested due to automatic review settings June 13, 2026 15:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates user-facing documentation to accurately reflect the current proxy-mode routing coverage and to explicitly connect container entrypoint host/mode overrides to the public environment variable reference.

Changes:

  • Updated the README proxy-mode summary to reflect current routing coverage (~50 REST patterns, ~15 GraphQL patterns).
  • Clarified docs/ENVIRONMENT_VARIABLES.md to explicitly mention run_containerized.sh uses MCP_GATEWAY_HOST and MCP_GATEWAY_MODE for bind address and routing mode.
Show a summary per file
File Description
README.md Updates proxy-mode coverage summary and keeps docs/PROXY_MODE.md as the authoritative reference.
docs/ENVIRONMENT_VARIABLES.md Connects the general HOST/MODE/PORT note to the container entrypoint’s documented MCP_GATEWAY_HOST/MCP_GATEWAY_MODE overrides.

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: 0

@lpcox lpcox merged commit 31d098f into main Jun 13, 2026
40 checks passed
@lpcox lpcox deleted the copilot/update-proxy-mode-pattern-count branch June 13, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants