Address code review feedback: implicit usings, SDK sorting, and error messages

Improvements based on review:
- Remove System.IO using statement (implicitly included via ImplicitUsings)
- Add explicit sorting of SDKs by version to ensure latest is always correct, regardless of CLI output order
- Improve error message in DotNetCommandExecutor to include failed command arguments for better troubleshooting

Note: Path construction for SDK/runtime paths is correct as-is. The CLI output format is "version [directory]" where directory does NOT include the version, so Path.Combine(path, version) correctly builds the full path.

Co-authored-by: jongalloway <[email protected]>

Author: Copilot

DotNetMcp/DotNetCommandExecutor.cs

@@ -126,7 +126,7 @@ public static async Task<string> ExecuteCommandForResourceAsync(string arguments
         using var process = Process.Start(startInfo);
         if (process == null)
         {
-            throw new InvalidOperationException("Failed to start dotnet process");
+            throw new InvalidOperationException($"Failed to start dotnet process with arguments: {arguments}");
         }
 
         var output = await process.StandardOutput.ReadToEndAsync();

DotNetMcp/DotNetResources.cs

@@ -1,4 +1,3 @@
-using System.IO;
 using System.Text.Json;
 using Microsoft.Extensions.Logging;
 using ModelContextProtocol.Server;
@@ -57,10 +56,15 @@ public async Task<string> GetSdkInfo()
                 }
             }
 
+            // Sort SDKs by version to ensure we always return the latest, regardless of CLI output order
+            var sortedSdks = sdks
+                .OrderBy(sdk => Version.TryParse(sdk.Version, out var v) ? v : new Version(0, 0))
+                .ToList();
+
             var response = new
             {
-                sdks,
-                latestSdk = sdks.LastOrDefault()?.Version
+                sdks = sortedSdks,
+                latestSdk = sortedSdks.LastOrDefault()?.Version
             };
 
             return JsonSerializer.Serialize(response, new JsonSerializerOptions { WriteIndented = true });