Tweaks

Author: PederHP

samples/AspNetCoreMcpPerSessionTools/Program.cs

@@ -15,7 +15,6 @@
         {
             // Determine tool category from route parameters
             var toolCategory = GetToolCategoryFromRoute(httpContext);
-            var sessionInfo = GetSessionInfo(httpContext);
 
             // Get the tool collection that we can modify per session
             var toolCollection = mcpOptions.Capabilities?.Tools?.ToolCollection;
@@ -51,11 +50,6 @@
                         break;
                 }
             }
-
-            // Optional: Log the session configuration for debugging
-            var logger = httpContext.RequestServices.GetRequiredService<ILogger<Program>>();
-            logger.LogInformation("Configured MCP session for category '{ToolCategory}' from {SessionInfo}, {ToolCount} tools available", 
-                toolCategory, sessionInfo, toolCollection?.Count ?? 0);
         };
     })
     .WithTools<ClockTool>()
@@ -75,47 +69,9 @@
 
 var app = builder.Build();
 
-// Add middleware to log requests for demo purposes
-app.Use(async (context, next) =>
-{
-    var logger = context.RequestServices.GetRequiredService<ILogger<Program>>();
-    var toolCategory = GetToolCategoryFromRoute(context);
-    var sessionInfo = GetSessionInfo(context);
-    
-    logger.LogInformation("Request for category '{ToolCategory}' from {SessionInfo}: {Method} {Path}", 
-        toolCategory, sessionInfo, context.Request.Method, context.Request.Path);
-    
-    await next();
-});
-
 // Map MCP with route parameter for tool category filtering
 app.MapMcp("/{toolCategory?}");
 
-// Add endpoints to test different tool categories
-app.MapGet("/", () => Results.Text(
-    "MCP Per-Session Tools Demo\n" +
-    "=========================\n" +
-    "Available endpoints:\n" +
-    "- /clock - MCP server with clock/time tools\n" +
-    "- /calculator - MCP server with calculation tools\n" +
-    "- /userinfo - MCP server with session/system info tools\n" +
-    "- /all - MCP server with all tools (default)\n" +
-    "\n" +
-    "Test routes:\n" +
-    "- /test-category/{category} - Test category detection\n"
-));
-
-app.MapGet("/test-category/{toolCategory?}", (string? toolCategory, HttpContext context) =>
-{
-    var detectedCategory = GetToolCategoryFromRoute(context);
-    var sessionInfo = GetSessionInfo(context);
-    
-    return Results.Text($"Tool Category: {detectedCategory ?? "all (default)"}\n" +
-                       $"Session Info: {sessionInfo}\n" +
-                       $"Route Parameter: {toolCategory ?? "none"}\n" +
-                       $"Message: MCP session would be configured for '{detectedCategory ?? "all"}' tools");
-});
-
 app.Run();
 
 // Helper methods for route-based tool category detection
@@ -126,33 +82,9 @@
     {
         return string.IsNullOrEmpty(category) ? "all" : category;
     }
-    
-    // Fallback: try to extract from path
-    var path = context.Request.Path.Value?.Trim('/');
-    if (!string.IsNullOrEmpty(path))
-    {
-        var segments = path.Split('/');
-        if (segments.Length > 0)
-        {
-            var firstSegment = segments[0].ToLower();
-            if (firstSegment is "clock" or "calculator" or "userinfo" or "all")
-            {
-                return firstSegment;
-            }
-        }
-    }
-    
-    // Default to "all" if no category specified
-    return "all";
-}
 
-static string GetSessionInfo(HttpContext context)
-{
-    var userAgent = context.Request.Headers.UserAgent.ToString();
-    var clientInfo = !string.IsNullOrEmpty(userAgent) ? userAgent[..Math.Min(userAgent.Length, 20)] + "..." : "Unknown";
-    var remoteIp = context.Connection.RemoteIpAddress?.ToString() ?? "unknown";
-    
-    return $"{clientInfo} ({remoteIp})";
+    // Default to "all" if no category specified or empty
+    return "all";
 }
 
 static void AddToolsForType<[System.Diagnostics.CodeAnalysis.DynamicallyAccessedMembers(

samples/AspNetCoreMcpPerSessionTools/README.md

@@ -147,38 +147,3 @@ static string? GetToolCategoryFromRoute(HttpContext context)
     return "all"; // Default
 }
 ```
-
-### 4. Dynamic Tool Loading
-The `AddToolsForType<T>` helper method uses reflection to discover and add all tools from a specific tool type to the session's tool collection.
-
-## Key Benefits
-
-- **Easy Testing**: No need to manage authentication tokens or headers
-- **Clear Separation**: Each tool category is isolated and can be tested independently  
-- **Flexible Architecture**: Easy to add new tool categories or modify existing ones
-- **Production Ready**: The same technique can be extended for production scenarios with proper routing logic
-- **Observable**: Built-in logging shows exactly which tools are configured for each session
-
-## Adapting for Production
-
-For production use, you might want to:
-
-1. **Add Authentication**: Combine route-based filtering with proper authentication
-2. **Database-Driven Categories**: Load tool categories and permissions from a database
-3. **User-Specific Routing**: Use user information to determine allowed categories
-4. **Advanced Routing**: Support nested categories or query parameters
-5. **Rate Limiting**: Add rate limiting per tool category
-6. **Caching**: Cache tool collections for better performance
-
-## Related Issues
-
-- [#714](https://github.com/modelcontextprotocol/csharp-sdk/issues/714) - Support varying tools/resources per user
-- [#237](https://github.com/modelcontextprotocol/csharp-sdk/issues/237) - Session-specific tool configuration  
-- [#476](https://github.com/modelcontextprotocol/csharp-sdk/issues/476) - Dynamic tool management
-- [#612](https://github.com/modelcontextprotocol/csharp-sdk/issues/612) - Per-session resource filtering
-
-## Learn More
-
-- [Model Context Protocol Specification](https://modelcontextprotocol.io/)
-- [ASP.NET Core MCP Integration](../../src/ModelContextProtocol.AspNetCore/README.md)
-- [MCP C# SDK Documentation](https://modelcontextprotocol.github.io/csharp-sdk/)

samples/AspNetCoreMcpPerSessionTools/Tools/UserInfoTool.cs

@@ -4,47 +4,20 @@
 namespace AspNetCoreMcpPerSessionTools.Tools;
 
 /// <summary>
-/// User information and session-related tools
+/// User information tools
 /// </summary>
 [McpServerToolType]
 public sealed class UserInfoTool
 {
-    [McpServerTool, Description("Gets information about the current MCP session.")]
-    public static string GetSessionInfo()
+    [McpServerTool, Description("Gets information about the current user in the MCP session.")]
+    public static string GetUserInfo()
     {
-        return $"MCP Session Information:\n" +
-               $"- Session ID: {Guid.NewGuid():N}[..8] (simulated)\n" +
-               $"- Session Start: {DateTime.Now.AddMinutes(-new Random().Next(1, 60)):HH:mm:ss}\n" +
-               $"- Protocol Version: MCP 2025-06-18\n" +
-               $"- Transport: HTTP";
-    }
-
-    [McpServerTool, Description("Gets system information about the server environment.")]
-    public static string GetSystemInfo()
-    {
-        return $"System Information:\n" +
-               $"- Server Time: {DateTime.Now:yyyy-MM-dd HH:mm:ss} UTC\n" +
-               $"- Platform: {Environment.OSVersion.Platform}\n" +
-               $"- Runtime: .NET {Environment.Version}\n" +
-               $"- Processor Count: {Environment.ProcessorCount}\n" +
-               $"- Working Set: {Environment.WorkingSet / (1024 * 1024):F1} MB";
-    }
-
-    [McpServerTool, Description("Echoes back the provided message with session context.")]
-    public static string EchoWithContext(
-        [Description("Message to echo back")] string message)
-    {
-        return $"[Session Echo] {DateTime.Now:HH:mm:ss}: {message}";
-    }
-
-    [Description("Gets basic connection information about the client.")]
-    [McpServerTool]
-    public static string GetConnectionInfo()
-    {
-        return $"Connection Information:\n" +
-               $"- Connection Type: HTTP MCP Transport\n" +
-               $"- Connected At: {DateTime.Now.AddMinutes(-new Random().Next(1, 30)):HH:mm:ss}\n" +
-               $"- Status: Active\n" +
-               $"- Messages Exchanged: {new Random().Next(1, 100)} (simulated)";
+        // Dummy user information for demonstration purposes
+        return $"User Information:\n" +
+               $"- User ID: {Guid.NewGuid():N}[..8] (simulated)\n" +
+               $"- Username: User{new Random().Next(1, 1000)}\n" +
+               $"- Roles: User, Guest\n" +
+               $"- Last Login: {DateTime.Now.AddMinutes(-new Random().Next(1, 60)):HH:mm:ss}\n" +
+               $"- Account Status: Active";
     }
 }