Merge pull request #2 from CapulusCodeNinja/bugfix/1-Compatibility-Issue-with-Google-Antigravity-MCP-Integration

Bugfix 1 compatibility issue with google antigravity mcp integration

Author: CapulusCodeNinja

Directory.Build.props

@@ -38,8 +38,8 @@
 
     <PropertyGroup Label="Default Versioning">
         <!-- Default version for all projects (can be overridden per project) -->
-        <VersionPrefix>1.1.2</VersionPrefix>
-        <VersionSuffix>5</VersionSuffix>
+        <VersionPrefix>1.1.3</VersionPrefix>
+        <VersionSuffix>1</VersionSuffix>
 
         <!-- Calculated version -->
         <Version Condition="'$(Version)' == ''">$(VersionPrefix).$(VersionSuffix)</Version>

README.md

@@ -4,8 +4,8 @@
 
 ![MCP Nexus Icon](https://github.com/CapulusCodeNinja/mcp_nexus/blob/main/images/mcp_nexus_small.png?raw=true)
 
-![Tests](https://img.shields.io/badge/tests-1249%20passing-brightgreen)
-![Coverage](https://img.shields.io/badge/coverage-86.9%25%20lines-green)
+![Tests](https://img.shields.io/badge/tests-1247%20passing-brightgreen)
+![Coverage](https://img.shields.io/badge/coverage-87.1%25%20lines-green)
 ![Build](https://img.shields.io/badge/build-passing-brightgreen)
 ![License](https://img.shields.io/badge/license-Apache%202.0-blue)
 
@@ -66,50 +66,83 @@ dotnet run --project nexus/nexus.csproj
 
 ### Cursor IDE Integration
 
-Add to `.cursor/mcp.json`:
+#### STDIO Integration
+
+![Cursor Stdio](https://github.com/CapulusCodeNinja/mcp_nexus/blob/main/images/integrations/Cursor_stdio.gif?raw=true)
 
 ```json
 {
   "mcpServers": {
-    "mcp-nexus": {
+    "mcp_nexus": {
+      "type": "stdio",
       "command": "dotnet",
-      "args": ["run", "--project", "C:\\path\\to\\mcp_nexus\\nexus\\nexus.csproj"],
-      "type": "stdio"
+      "args": [
+        "run",
+        "--project",
+        "C:\\Sources\\Github\\CapulusCodeNinja\\mcp_nexus\\nexus\\nexus.csproj",
+        "--",
+        "--stdio"
+      ]
     }
   }
 }
 ```
 
-### Claude Desktop Integration
+#### HTTP Integration
 
-Add to `claude_desktop_config.json`:
+![Cursor Http](https://github.com/CapulusCodeNinja/mcp_nexus/blob/main/images/integrations/Cursor_http.gif?raw=true)
 
 ```json
 {
   "mcpServers": {
     "mcp-nexus": {
-      "command": "dotnet",
-      "args": ["run", "--project", "C:\\path\\to\\mcp_nexus\\nexus\\nexus.csproj"],
-      "cwd": "C:\\path\\to\\mcp_nexus"
+      "url": "http://0.0.0.0:5511/",
+      "headers": {
+        "Content-Type": "application/json"
+      }
     }
   }
 }
 ```
 
+### Google Antigravity Integration
+
+#### STDIO Integration
 
-### Integration using HTTP
+![Antigravity Stdio](https://github.com/CapulusCodeNinja/mcp_nexus/blob/main/images/integrations/Antigravity_stdio.gif?raw=true)
 
 ```json
 {
-  "mcpServers": {
-    "mcp-nexus": {
-      "type": "http",
-      "url": "http://0.0.0.0:5511/",
-      "headers": {
-        "Content-Type": "application/json"
-      }
+    "mcpServers": {
+        "mcp_nexus": {
+            "command": "C:\\Program Files\\dotnet\\dotnet.exe",
+            "args": [
+                "run",
+                "--project",
+                "C:\\Sources\\Github\\CapulusCodeNinja\\mcp_nexus\\nexus\\nexus.csproj",
+                "--",
+                "--stdio"
+            ]
+        }
+    },
+    "inputs": []
+}
+```
+
+#### HTTP Integration
+
+![Antigravity Http](https://github.com/CapulusCodeNinja/mcp_nexus/blob/main/images/integrations/Antigravity_http.gif?raw=true)
+
+```json
+{
+    "mcpServers": {
+        "mcp-nexus": {
+            "serverUrl": "http://0.0.0.0:5511/",
+            "headers": {
+                "Content-Type": "application/json"
+            }
+        }
     }
-  }
 }
 ```
 

images/integrations/Antigravity_http.gif

@@ -67,12 +67,34 @@ public void DisplayBanner()
     /// </summary>
     private void DisplayStartupHeader()
     {
+        var isStdioMode = m_CommandLineContext.IsStdioMode;
         var version = Assembly.GetExecutingAssembly().GetName().Version?.ToString() ?? "Unknown";
         var processId = Environment.ProcessId;
         var startTime = DateTime.Now;
-        var host = Environment.GetEnvironmentVariable("ASPNETCORE_URLS")?.Split(':').LastOrDefault() ?? "0.0.0.0";
-        var port = Environment.GetEnvironmentVariable("ASPNETCORE_URLS")?.Split(':').LastOrDefault() ?? "5511";
-        var transportMode = "http";
+        string host;
+        string port;
+        string transportMode;
+
+        if (isStdioMode)
+        {
+            transportMode = "stdio";
+            host = "stdio";
+            port = "n/a";
+        }
+        else
+        {
+            transportMode = "http";
+
+            // Prefer configuration for host/port; fall back to ASPNETCORE_URLS when available.
+            var sharedConfiguration = m_Settings.Get();
+            var configuredHost = sharedConfiguration.McpNexus.Server.Host;
+            var configuredPort = sharedConfiguration.McpNexus.Server.Port;
+
+            host = configuredHost ?? "0.0.0.0";
+            port = configuredPort > 0
+                ? configuredPort.ToString(System.Globalization.CultureInfo.InvariantCulture)
+                : (Environment.GetEnvironmentVariable("ASPNETCORE_URLS")?.Split(':').LastOrDefault() ?? "5511");
+        }
 
         m_Logger.Info("╔═══════════════════════════════════════════════════════════════════╗");
         m_Logger.Info("                            MCP NEXUS STARTUP");

images/integrations/Antigravity_stdio.gif

@@ -24,6 +24,13 @@ public virtual void ConfigureLogging(ILoggingBuilder logging, bool isServiceMode
         var logLevel = GetLogLevelFromConfiguration();
         ConfigureNLogDynamically(logLevel, isServiceMode);
         ConfigureNLogProvider(logging, logLevel);
+
+        // Prevent any console-based providers from emitting noisy logs that would interfere
+        // with MCP stdio communication. These filters ensure that even if a console logger
+        // is added by hosting infrastructure, the most verbose categories are suppressed.
+        _ = logging.AddFilter("ModelContextProtocol.Server.StdioServerTransport", Microsoft.Extensions.Logging.LogLevel.None);
+        _ = logging.AddFilter("ModelContextProtocol.Server.McpServer", Microsoft.Extensions.Logging.LogLevel.None);
+        _ = logging.AddFilter("Microsoft.Hosting.Lifetime", Microsoft.Extensions.Logging.LogLevel.None);
     }
 
     /// <summary>
@@ -68,19 +75,6 @@ protected virtual void ConfigureNLogDynamically(Microsoft.Extensions.Logging.Log
             nlogConfig.LoggingRules.Add(new NLog.Config.LoggingRule("*", NLog.LogLevel.Info, NLog.LogLevel.Fatal, fileTarget));
         }
 
-        // Ensure stderr console target exists
-        if (nlogConfig.FindTargetByName("stderr") is not NLog.Targets.ConsoleTarget)
-        {
-            var stderrTarget = new NLog.Targets.ConsoleTarget("stderr")
-            {
-                StdErr = true,
-                Layout = "${longdate} [${level:uppercase=true}] ${message} ${exception:format=ToString}",
-                Encoding = System.Text.Encoding.UTF8,
-            };
-            nlogConfig.AddTarget(stderrTarget);
-            nlogConfig.LoggingRules.Add(new NLog.Config.LoggingRule("*", NLog.LogLevel.Info, NLog.LogLevel.Fatal, stderrTarget));
-        }
-
         // Apply service-mode paths (ProgramData vs app dir)
         ConfigureLogPaths(nlogConfig, isServiceMode);
 
@@ -182,18 +176,22 @@ protected virtual void SetInternalLogFile(bool isServiceMode)
 
     /// <summary>
     /// Configures the NLog provider for Microsoft.Extensions.Logging.
+    /// Ensures that all framework logging is routed through NLog so that no logs
+    /// are written directly to stdout. This is critical for MCP stdio mode where
+    /// stdout must be reserved exclusively for JSON-RPC messages and all human-readable
+    /// logging must go to file targets (or explicitly configured sinks).
     /// </summary>
     /// <param name="logging">The logging builder to configure.</param>
     /// <param name="logLevel">The log level to set.</param>
     protected virtual void ConfigureNLogProvider(ILoggingBuilder logging, Microsoft.Extensions.Logging.LogLevel logLevel)
     {
-        // Only configure NLog for Microsoft.Extensions.Logging when Trace is enabled
-        if (logLevel != Microsoft.Extensions.Logging.LogLevel.Trace)
-        {
-            return;
-        }
+        ArgumentNullException.ThrowIfNull(logging);
 
+        // Remove default console/debug providers so nothing writes directly to stdout/stderr.
         _ = logging.ClearProviders();
+
+        // Route Microsoft.Extensions.Logging through NLog (using NLog.Web for compatibility
+        // with both generic hosts and ASP.NET Core).
         _ = logging.AddNLogWeb();
         _ = logging.SetMinimumLevel(logLevel);
     }

images/integrations/Cursor_http.gif

@@ -1,5 +1,6 @@
 using FluentAssertions;
 
+using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
 
 using Moq;
@@ -181,7 +182,9 @@ public void ConfigureLogging_WithServiceModeFalse_ConfiguresLogging()
         // Arrange
         m_Settings = new Settings();
         m_Settings.LoadConfiguration();
+        var services = new ServiceCollection();
         var mockLoggingBuilder = new Mock<ILoggingBuilder>();
+        _ = mockLoggingBuilder.SetupGet(builder => builder.Services).Returns(services);
         var isServiceMode = false;
 
         // Act
@@ -200,7 +203,9 @@ public void ConfigureLogging_WithServiceModeTrue_ConfiguresLogging()
         // Arrange
         m_Settings = new Settings();
         m_Settings.LoadConfiguration();
+        var services = new ServiceCollection();
         var mockLoggingBuilder = new Mock<ILoggingBuilder>();
+        _ = mockLoggingBuilder.SetupGet(builder => builder.Services).Returns(services);
         var isServiceMode = true;
 
         // Act
@@ -218,7 +223,9 @@ public void ConfigureLogging_AfterConstructor_ConfiguresLogging()
     {
         // Arrange
         m_Settings = new Settings();
+        var services = new ServiceCollection();
         var mockLoggingBuilder = new Mock<ILoggingBuilder>();
+        _ = mockLoggingBuilder.SetupGet(builder => builder.Services).Returns(services);
         var isServiceMode = false;
 
         // Act - should work since constructor creates ConfigurationLoader