Skip to content

Remove MCP9002 obsolete APIs and document obsolete diagnostics#1366

Merged
jeffhandley merged 2 commits intomainfrom
copilot/remove-obsolete-apis-mcp9002
Feb 24, 2026
Merged

Remove MCP9002 obsolete APIs and document obsolete diagnostics#1366
jeffhandley merged 2 commits intomainfrom
copilot/remove-obsolete-apis-mcp9002

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

Removes the 13 AddXxxFilter extension methods on IMcpServerBuilder that were deprecated under diagnostic MCP9002 in favor of WithRequestFilters() and WithMessageFilters().

Changes

  • McpServerBuilderExtensions.cs — Deleted the MCP9002-marked methods: AddListResourceTemplatesFilter, AddListToolsFilter, AddCallToolFilter, AddListPromptsFilter, AddGetPromptFilter, AddListResourcesFilter, AddReadResourceFilter, AddCompleteFilter, AddSubscribeToResourcesFilter, AddUnsubscribeFromResourcesFilter, AddSetLoggingLevelFilter, AddIncomingMessageFilter, AddOutgoingMessageFilter

  • Obsoletions.cs — Replaced MCP9002 constants with a comment recording what the ID was used for, preventing accidental reuse:

    // MCP9002 was used for the AddXxxFilter extension methods on IMcpServerBuilder that were superseded by
    // WithMessageFilters() and WithRequestFilters(). The APIs were removed; do not reuse this diagnostic ID.
  • docs/list-of-diagnostics.md — Added an Obsolete APIs section documenting all MCP9### diagnostics issued to date, including whether each remains in place or has been removed:

    Diagnostic ID Status Description
    MCP9001 In place EnumSchema/LegacyTitledEnumSchema deprecated per spec 2025-11-25
    MCP9002 Removed AddXxxFilter on IMcpServerBuilder superseded by WithRequestFilters()/WithMessageFilters()
Original prompt

Remove the APIs marked as Obsolete with "MCP9002". Leave a comment in Obsoletions.cs indicating what the diagnostic was used for to ensure the id is not reused.

Make any compensating changes needed for removing the APIs.

Augment docs/list-of-diagnostics.md to document the obsolete API diagnostics that have been used thus far, introducing an Obsolete APIs section at the bottom of the document. Include the status of the obsolete APIs whether they remain in place or have been removed.


✨ 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 changed the title [WIP] Remove obsolete APIs marked with MCP9002 Remove MCP9002 obsolete APIs and document obsolete diagnostics Feb 24, 2026
Copilot AI requested a review from jeffhandley February 24, 2026 01:05
@jeffhandley jeffhandley added this to the 1.0.0 (GA) milestone Feb 24, 2026
@jeffhandley jeffhandley added the breaking-change This issue or PR introduces a breaking change label Feb 24, 2026
@jeffhandley jeffhandley marked this pull request as ready for review February 24, 2026 01:40
@jeffhandley jeffhandley merged commit 9b2aaed into main Feb 24, 2026
11 checks passed
@jeffhandley jeffhandley deleted the copilot/remove-obsolete-apis-mcp9002 branch February 24, 2026 02:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change This issue or PR introduces a breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants