Skip to content

Conversation

@devcrocod
Copy link
Contributor

  • add the warning message to ExperimentalMcpApi annotation
  • change level from error to warning

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

@devcrocod devcrocod requested review from Copilot and kpavlov November 13, 2025 14:37
Copilot finished reviewing on behalf of devcrocod November 13, 2025 14:39
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 enhances the ExperimentalMcpApi annotation by adding an informative warning message and reducing the severity from error to warning level.

  • Added a descriptive message explaining the experimental nature of the API
  • Changed the opt-in requirement level from ERROR to WARNING to make the API more accessible while still alerting users

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kpavlov kpavlov merged commit 4aeed92 into main Nov 13, 2025
10 checks passed
@kpavlov kpavlov deleted the devcrocod/change-level-for-expr branch November 13, 2025 20:05
kpavlov pushed a commit that referenced this pull request Nov 14, 2025
…#398)

- add the warning message to `ExperimentalMcpApi` annotation
- change level from error to warning

## 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)
- [x] Documentation update

## Checklist
- [x] I have read the [MCP
Documentation](https://modelcontextprotocol.io)
- [x] My code follows the repository's style guidelines
- [x] New and existing tests pass locally
- [ ] I have added appropriate error handling
- [x] I have added or updated documentation as needed
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