feat(gateway): web admin version-probe parity and drop unused mcpmux-mcp

Un-stub check_server_version / check_all_server_versions onto ServerVersionProbeService. Delete the orphaned mcpmux-mcp crate; ClientPool had no callers.

Signed-off-by: crimsonsunset <jsangio1@gmail.com>

Author: crimsonsunset

AGENTS.md

@@ -16,8 +16,7 @@ mcp-mux/
 ├── crates/
 │   ├── mcpmux-core/       # Domain entities, repository traits, service layer, EventBus
 │   ├── mcpmux-gateway/    # Axum gateway — routing, OAuth refresh, FeatureSet filtering
-│   ├── mcpmux-storage/    # SQLite + AES-256-GCM field encryption + OS keychain
-│   └── mcpmux-mcp/        # MCP protocol client wrapper (rmcp SDK)
+│   └── mcpmux-storage/    # SQLite + AES-256-GCM field encryption + OS keychain
 ├── packages/ui/           # Shared UI components (`@mcpmux/ui`)
 ├── schemas/               # JSON Schemas surfaced in the Monaco editor
 └── tests/                 # Rust integration, TS unit (vitest), desktop E2E (WDIO), web E2E (playwright)

CLAUDE.md

@@ -66,12 +66,11 @@ pnpm typecheck          # TypeScript type checking (recursive)
 
 ### Rust Crate Architecture
 
-The Cargo workspace has 4 library crates + 1 app crate + 1 test crate:
+The Cargo workspace has 3 library crates + 1 app crate + 1 test crate:
 
 - **mcpmux-core** (`crates/mcpmux-core/`) - Domain layer: entities (Space, InstalledServer, FeatureSet, Client), repository traits, domain services, application services with event emission, and the central EventBus
 - **mcpmux-gateway** (`crates/mcpmux-gateway/`) - Axum HTTP gateway: routes MCP calls to correct servers, manages OAuth 2.1+PKCE token refresh, filters tools/resources/prompts based on FeatureSets, per-client access key auth, server connection pooling
 - **mcpmux-storage** (`crates/mcpmux-storage/`) - SQLite persistence with AES-256-GCM field-level encryption via ring, typed credential rows (per-token encryption), DPAPI key storage on Windows (`keychain_dpapi.rs`), OS keychain on macOS/Linux via keyring crate, zeroize for secure memory clearing
-- **mcpmux-mcp** (`crates/mcpmux-mcp/`) - MCP protocol client management using rmcp SDK
 - **apps/desktop/src-tauri** - Tauri 2 app shell, Tauri commands, system tray, deep-link handler (`mcpmux://`)
 - **tests/rust** - Integration test crate
 

Cargo.lock

@@ -4,7 +4,6 @@ members = [
     "apps/desktop/src-tauri",
     "crates/mcpmux-core",
     "crates/mcpmux-gateway",
-    "crates/mcpmux-mcp",
     "crates/mcpmux-storage",
     "tests/rust",
 ]
@@ -78,7 +77,6 @@ http-body-util = "0.1"
 # Internal crates (path-only for workspace)
 mcpmux-core = { path = "crates/mcpmux-core" }
 mcpmux-gateway = { path = "crates/mcpmux-gateway" }
-mcpmux-mcp = { path = "crates/mcpmux-mcp" }
 mcpmux-storage = { path = "crates/mcpmux-storage" }
 
 [profile.release]

Cargo.toml

@@ -242,8 +242,7 @@ mcp-mux/
 ├── crates/
 │   ├── mcpmux-core/       # Domain logic
 │   ├── mcpmux-gateway/    # HTTP gateway, OAuth, routing
-│   ├── mcpmux-storage/    # SQLite + encryption + OS keychain
-│   └── mcpmux-mcp/        # MCP protocol
+│   └── mcpmux-storage/    # SQLite + encryption + OS keychain
 ├── packages/ui/           # Shared UI components
 └── tests/                 # Unit, integration, E2E tests
 ```

README.md

@@ -28,7 +28,7 @@ Generated via `cargo license`
 - openssl
 - sync_wrapper
 - tao
-- mcpmux-core, mcpmux-gateway, mcpmux-mcp, mcpmux-storage (internal)
+- mcpmux-core, mcpmux-gateway, mcpmux-storage (internal)
 
 ### Apache-2.0 OR MIT (Dual Licensed)
 - tokio, tokio-macros, tokio-util (async runtime)

THIRD_PARTY_LICENSES

@@ -515,6 +515,7 @@ pub async fn start_admin_server_if_enabled(
         gateway_port_service: app_state.gateway_port_service.clone(),
         server_discovery: app_state.server_discovery.clone(),
         settings_repository: app_state.settings_repository.clone(),
+        installed_server_repository: app_state.installed_server_repository.clone(),
         workspace_binding_repository: app_state.workspace_binding_repository.clone(),
         machine_repository: Arc::new(SqliteMachineRepository::new(app_state.database())),
         inbound_client_repository: Arc::new(mcpmux_storage::InboundClientRepository::new(

apps/desktop/src-tauri/src/services/admin_server.rs

@@ -5,8 +5,8 @@ use std::sync::Arc;
 
 use mcpmux_core::{
     AppSettingsRepository, ApplicationServices, FeatureSetRepository, GatewayPortService,
-    MachineRepository, ServerDiscoveryService, ServerFeatureRepository, ServerLogManager,
-    SpaceBaseDirRepository, SpaceBuiltinConfigRepository, SpaceService,
+    InstalledServerRepository, MachineRepository, ServerDiscoveryService, ServerFeatureRepository,
+    ServerLogManager, SpaceBaseDirRepository, SpaceBuiltinConfigRepository, SpaceService,
     WorkspaceAppearanceRepository, WorkspaceBindingRepository,
 };
 use mcpmux_storage::InboundClientRepository;
@@ -35,6 +35,7 @@ pub struct AdminBridgeCtx {
     pub gateway_port_service: Arc<GatewayPortService>,
     pub server_discovery: Arc<ServerDiscoveryService>,
     pub settings_repository: Arc<dyn AppSettingsRepository>,
+    pub installed_server_repository: Arc<dyn InstalledServerRepository>,
     pub workspace_binding_repository: Arc<dyn WorkspaceBindingRepository>,
     pub machine_repository: Arc<dyn MachineRepository>,
     pub inbound_client_repository: Arc<InboundClientRepository>,

crates/mcpmux-gateway/src/admin/bridge_context.rs

@@ -21,6 +21,7 @@ use crate::admin::command_bridge::read::{
     to_workspace_appearance_response, to_workspace_binding_response,
 };
 use crate::admin::command_bridge::space::{self, UpdateSpaceInput};
+use crate::services::ServerVersionProbeService;
 
 const LOCAL_ICON_PREFIX: &str = "local:workspace-icons/";
 const WORKSPACE_ICON_DIR: &str = "workspace-icons";
@@ -1246,19 +1247,40 @@ pub async fn revoke_oauth_client_feature_set(
         .await
 }
 
+fn version_probe(ctx: &AdminBridgeCtx) -> ServerVersionProbeService {
+    ServerVersionProbeService::new(
+        ctx.installed_server_repository.clone(),
+        ctx.settings_repository.clone(),
+        ctx.services.event_bus.clone(),
+    )
+}
+
 /// Probe npm/PyPI for a single installed server package update.
 pub async fn check_server_version(
-    _ctx: &AdminBridgeCtx,
-    _body: ServerConnectionBody,
+    ctx: &AdminBridgeCtx,
+    body: ServerConnectionBody,
 ) -> Result<Value> {
-    // ponytail: version probing lands in Phase 5
-    Err(anyhow!("Server version checking not yet available"))
+    let result = version_probe(ctx)
+        .probe_server(&body.space_id, &body.server_id)
+        .await?;
+    Ok(json!({
+        "spaceId": result.space_id,
+        "serverId": result.server_id,
+        "currentVersion": result.current_version,
+        "latestVersion": result.latest_version,
+        "updateAvailable": result.update_available,
+        "checkedAt": result.checked_at.to_rfc3339(),
+    }))
 }
 
 /// Probe all notify/auto package-managed servers for available updates.
-pub async fn check_all_server_versions(_ctx: &AdminBridgeCtx) -> Result<Value> {
-    // ponytail: version probing lands in Phase 5
-    Err(anyhow!("Server version checking not yet available"))
+pub async fn check_all_server_versions(ctx: &AdminBridgeCtx) -> Result<Value> {
+    let summary = version_probe(ctx).probe_all().await?;
+    Ok(json!({
+        "checked": summary.checked,
+        "updatesAvailable": summary.updates_available,
+        "checkedAt": summary.checked_at.to_rfc3339(),
+    }))
 }
 
 const META_TOOLS_REQUIRE_APPROVAL_KEY: &str = "meta_tools.require_approval";