Skip to content

Normalize internal namespaces from McpServer to McpServerManager#2

Merged
sharpninja merged 1 commit intomainfrom
claude/magical-ritchie
Apr 1, 2026
Merged

Normalize internal namespaces from McpServer to McpServerManager#2
sharpninja merged 1 commit intomainfrom
claude/magical-ritchie

Conversation

@sharpninja
Copy link
Copy Markdown
Owner

Overview

This PR standardizes internal namespace naming across the codebase by converting McpServer prefixes to McpServerManager for consistency with the project structure.

Changes

  • Updated namespace declarations in 552+ files across multiple projects
  • Affected projects:
    • McpServer.Director
    • McpServer.UI.Core
    • McpServer.Web
    • McpServerManager.Android
    • McpServerManager.Core
    • McpServerManager.Desktop
    • McpServerManager
    • Test projects (Director.Tests, UI.Core.Tests, Web.Tests)
  • Updated NuSpec package files
  • Updated project references and imports in Razor and XAML files
  • Updated test infrastructure and configurations

Impact

  • Internal namespace consistency improved
  • No breaking API changes (public interfaces unchanged)
  • All internal class references updated to match new namespace structure

Rename all internal project namespaces to match the solution name:
- McpServer.Director.* → McpServerManager.Director.*
- McpServer.UI.Core.* → McpServerManager.UI.Core.*
- McpServer.Web.* → McpServerManager.Web.*
- McpServer.VsExtension.* → McpServerManager.VsExtension.*

Updated across 652 .cs files, AXAML, Razor, XAML, and .csproj configs.
External lib submodule namespaces (McpServer.Client, McpServer.Cqrs,
McpServer.McpAgent) are preserved unchanged.

Also updates OIDC development config to use discovery from running
MCP server rather than hardcoded Keycloak values.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sharpninja sharpninja merged commit d4826a2 into main Apr 1, 2026
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.

1 participant