Skip to content

Add Server Instructions content#1334

Closed
olaservo wants to merge 41 commits intomodelcontextprotocol:mainfrom
olaservo:add-server-instructions
Closed

Add Server Instructions content#1334
olaservo wants to merge 41 commits intomodelcontextprotocol:mainfrom
olaservo:add-server-instructions

Conversation

@olaservo
Copy link
Copy Markdown
Member

@olaservo olaservo commented Aug 12, 2025

I was originally going to post a personal blog and make some tweaks to docs related to Server Instructions, but thought it could be better to add an official blog post as a followup from the Prompts post.

Some of this could go into other parts of the documentation instead. Sharing what I have so far here to get feedback and suggestions.

Motivation and Context

More clients are starting to add support for this feature, and it relates to a lot of use cases I've seen discussed. A few of them are here, but its not totally exhaustive.

How Has This Been Tested?

Here is the related GitHub server PR with more details on testing: github/github-mcp-server#1091

Breaking Changes

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

VSCode is going to release this feature soon, so @dend maybe something like this could be coordinated for when that release happens? (I believe its only in Insiders so far?)

@olaservo
Copy link
Copy Markdown
Member Author

cc @dsp-ant @ihrpr @dend - curious for your suggestions on how and where this content could be added and refined. Thanks!

docs/clients.mdx Outdated
| [TheiaAI/TheiaIDE][TheiaAI/TheiaIDE] | ❌ | ❌ | ✅ | ❓ | ❌ | ❌ | ❓ | ❌ |
| [Tome][Tome] | ❌ | ❌ | ✅ | ❓ | ❌ | ❌ | ❓ | ❌ |
| [TypingMind App][TypingMind App] | ❌ | ❌ | ✅ | ❓ | ❌ | ❌ | ❓ | ❌ |
| [VS Code GitHub Copilot][VS Code] | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ❌ |
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.

we support it :) microsoft/vscode-copilot-chat#491

Suggested change
| [VS Code GitHub Copilot][VS Code] |||||||| |
| [VS Code GitHub Copilot][VS Code] |||||||| |

Copy link
Copy Markdown
Contributor

@dend dend left a comment

Choose a reason for hiding this comment

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

Some edits.

@dend dend added the blog label Aug 22, 2025
olaservo and others added 17 commits August 24, 2025 15:05
@olaservo
Copy link
Copy Markdown
Member Author

Created a new PR here since I accidentally pushed a bunch of whitespace changes here that were a nightmare to revert. #1482

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants