Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 11, 2025

The is_exclusive field in MCP server configuration is superseded by compliance level tagging and is no longer used.

Changes

Backend

  • Removed is_exclusive from MCPServerConfig Pydantic model
  • Removed is_server_exclusive() method from MCPToolManager
  • Removed field from API responses and CLI output

Configuration

  • Cleaned all JSON config files (8 files in config/, mocks/, scripts/)

Frontend

  • Removed from demo fallback data and public config

Documentation

  • Updated admin guide to remove field description and examples
  • Updated copilot instructions and mock README

The compliance_level field provides access control without the exclusivity constraint.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>phase out is_exclusive for mcp specification</issue_title>
<issue_description>The compliance tagging supersedes this. is_exclusive is no longer needed.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits November 11, 2025 13:20
- Removed is_exclusive field from MCPServerConfig Pydantic model
- Removed is_server_exclusive() method from MCPToolManager
- Removed is_exclusive from API responses in config_routes.py
- Removed is_exclusive check from CLI in config_manager.py
- Removed is_exclusive from all config JSON files
- Updated test files to remove is_exclusive references
- Updated documentation (admin guide and copilot instructions)

Co-authored-by: garland3 <[email protected]>
- Removed is_exclusive from fallback demo data in useChatConfig.js
- Removed is_exclusive from frontend public config.json
- Removed is_exclusive from mcp-http-mock README example

Co-authored-by: garland3 <[email protected]>
Copilot AI changed the title [WIP] Remove is_exclusive from mcp specification Remove deprecated is_exclusive field from MCP specification Nov 11, 2025
Copilot AI requested a review from garland3 November 11, 2025 13:27
Copilot finished work on behalf of garland3 November 11, 2025 13:27
@garland3 garland3 marked this pull request as ready for review November 14, 2025 10:59
Copy link
Collaborator

@garland3 garland3 left a comment

Choose a reason for hiding this comment

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

tested and it works.

@github-actions
Copy link

🔒 Security Scan Results

Security Scan Summary

Scan Results

Python SAST (Bandit)

⚠️ Security issues found in Python code

Recommendations

  • Review all SARIF files uploaded to GitHub Security tab
  • Address high and critical severity vulnerabilities immediately
  • Run npm audit fix and pip-audit locally to fix dependencies
  • Consider implementing additional security controls

1 similar comment
@github-actions
Copy link

🔒 Security Scan Results

Security Scan Summary

Scan Results

Python SAST (Bandit)

⚠️ Security issues found in Python code

Recommendations

  • Review all SARIF files uploaded to GitHub Security tab
  • Address high and critical severity vulnerabilities immediately
  • Run npm audit fix and pip-audit locally to fix dependencies
  • Consider implementing additional security controls

@garland3 garland3 merged commit f666942 into main Nov 14, 2025
14 checks passed
@garland3 garland3 deleted the copilot/remove-is-exclusive-mcp-specification branch November 14, 2025 11:03
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.

phase out is_exclusive for mcp specification

2 participants