Skip to content

Conversation

@tommaso-moro
Copy link
Contributor

@tommaso-moro tommaso-moro commented Nov 27, 2025

  • Adds a new md file in /docs with comprehensive guidance on server configuration, and examples.
  • Fixes the remote-server.md docs to include mention of the new tool X-MCP-Tools header

Rendered markdown

Copilot AI review requested due to automatic review settings November 27, 2025 16:40
@tommaso-moro tommaso-moro requested a review from a team as a code owner November 27, 2025 16:40
Copilot finished reviewing on behalf of tommaso-moro November 27, 2025 16:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds comprehensive server configuration documentation and updates existing docs to reference the new X-MCP-Tools header capability.

  • Adds a new comprehensive configuration guide with examples for all configuration modes
  • Updates remote-server.md to document the X-MCP-Tools header
  • Adds cross-references from README and remote-server docs to the new configuration guide

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
docs/server-configuration.md New comprehensive guide with configuration examples for toolsets, individual tools, read-only mode, dynamic discovery, and lockdown mode with side-by-side remote/local examples
docs/remote-server.md Documents the new X-MCP-Tools header and adds link to the new configuration guide
README.md Adds link to the new server configuration guide in the tool configuration section

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@tommaso-moro tommaso-moro changed the title Tommy/tool-specific-config-docs Tommy/server-config-docs Nov 27, 2025
Copy link

@xemii43251-jpg xemii43251-jpg left a comment

Choose a reason for hiding this comment

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link

@xemii43251-jpg xemii43251-jpg left a comment

Choose a reason for hiding this comment

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

privacysandbox.com

Choose a reason for hiding this comment

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

Build

tonytrg
tonytrg previously approved these changes Nov 28, 2025
Copy link
Contributor

@tonytrg tonytrg left a comment

Choose a reason for hiding this comment

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

💯

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
tommaso-moro and others added 5 commits November 28, 2025 09:52
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tommaso-moro tommaso-moro requested a review from tonytrg November 28, 2025 10:06
tonytrg
tonytrg previously approved these changes Nov 28, 2025
Co-authored-by: Tony Truong <tonytrg@github.com>
@tommaso-moro tommaso-moro requested a review from tonytrg November 28, 2025 10:15
@tommaso-moro tommaso-moro merged commit 9c73e88 into main Nov 28, 2025
17 checks passed
@tommaso-moro tommaso-moro deleted the tommy/docs-tools-update branch November 28, 2025 10:17
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.

4 participants