• Examine all public types in Protocol namespace that are not sealed
• Fix JsonRpcMessageContext - seal the class (has implicit public constructor)
• Fix RequestSchema (nested in ElicitRequestParams) - seal the class (has implicit public constructor)
• Fix LegacyTitledEnumSchema (nested in ElicitRequestParams) - seal the class
• Delete EnumSchema type and update all references to use LegacyTitledEnumSchema instead
• Fix Converter class in PrimitiveSchemaDefinition - seal the class
• Fix Converter class in ResourceContents - seal the class
• Fix Converter class in ContentBlock - seal the class
• Build and run tests to verify changes
• Run code review
• Run CodeQL security scan

Summary

This PR ensures that all public reference types in the Protocol namespace are either sealed or have only internal/private/private protected constructors, preventing external inheritance.

The abstract base classes (JsonRpcMessage, JsonRpcMessageWithId, RequestParams, PaginatedRequestParams, Result, PaginatedResult, NotificationParams, ContentBlock, ResourceContents, Reference, PrimitiveSchemaDefinition, TransportBase) already had private protected constructors.

The following public classes were sealed:

• JsonRpcMessageContext
• ElicitRequestParams.RequestSchema
• ElicitRequestParams.LegacyTitledEnumSchema
• ContentBlock.Converter
• ResourceContents.Converter
• ElicitRequestParams.PrimitiveSchemaDefinition.Converter

Note: The EnumSchema type was deleted and all references updated to use LegacyTitledEnumSchema instead.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.