Skip to content

Fix MCPEXP002 constant mismatch causing macOS Debug build failure#1743

Closed
jeffhandley with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-failing-build-macos
Closed

Fix MCPEXP002 constant mismatch causing macOS Debug build failure#1743
jeffhandley with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-failing-build-macos

Conversation

Copilot AI commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

The build (macos-latest, Debug) workflow failed with CS0117 due to mixed usage of Subclassing_* and Extensibility_* MCPEXP002 constants after the rename. This PR reconciles the constant surface and updates remaining stale call sites.

  • Compatibility constants for MCPEXP002

    • Added Extensibility_DiagnosticId and Extensibility_Url as aliases in Experimentals to keep both naming paths valid during transition.
    • Added Extensibility_Message for the renamed extensibility terminology.
  • Updated stale attribute call sites

    • Switched [Experimental(...)] usage from Subclassing_* to Extensibility_* in:
      • McpServerRequestHandler.RoutingNameParameter
      • JsonRpcMessageContext.RoutingName
  • Resulting pattern

    [Experimental(Experimentals.Extensibility_DiagnosticId, UrlFormat = Experimentals.Extensibility_Url)]
    public string? RoutingName { get; set; }

This keeps MCPEXP002 behavior unchanged while removing the source of compile-time symbol mismatches in CI.

Co-authored-by: jeffhandley <1031940+jeffhandley@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job build (macos-latest, Debug) Fix MCPEXP002 constant mismatch causing macOS Debug build failure Jul 26, 2026
Copilot AI requested a review from jeffhandley July 26, 2026 05:52
…Url throughout

Co-authored-by: jeffhandley <1031940+jeffhandley@users.noreply.github.com>
@jeffhandley

Copy link
Copy Markdown
Contributor

I intended to iterate on #1739 rather than creating a fresh PR.

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.

2 participants