Conversation
- Removed version history from README.md (kept reference to Changelog) - Updated Changelog.md with MCP integration as new feature in v6.1.0 - Simplified documentation to focus on MCP feature addition - All version references updated to 6.1.0 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The XAML file had invalid CDATA wrapper tags that were causing build errors. Removed the wrapper to fix XML parsing. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Added C# 11 language version support for MCP features - Fixed XAML file: removed invalid CDATA wrapper from McpServerWindow.xaml - Fixed McpServer.cs: - Removed unnecessary async from StartAsync method - Changed tools array to explicitly typed object[] - Added generic type parameter to InvokeAsync calls - Fixed using directives ordering (System.Linq before System.Net) - Replaced empty strings with string.Empty - Fixed SnoopMcpTools.cs: - Updated McpServerTool attributes to use parameterless constructor - Replaced empty strings with string.Empty - Added System.Text.Json 10.0.0 package dependency for MCP support Note: Additional work needed for full .NET Framework 4.6.2 and .NET 8.0 compatibility - .NET 6.0 build succeeds - .NET Framework 4.6.2 needs System.Range/Index polyfills - .NET 8.0 has MCP SDK API compatibility issues 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix .NET Framework 4.6.2 compatibility: * Replace C# 8+ range operators with Substring() * Fix StreamWriter constructor signature * Fix WriteAsync signature for .NET Framework * Add null-forgiving operators where needed - Fix .NET 8 MCP SDK compatibility: * Use webApplication.Urls.Add() instead of UseUrls/ConfigureKestrel * Simplify server startup with RunAsync() - Fix StyleCop violations: * Separate multiple attributes onto different lines * Add blank lines between properties * Use string.Empty instead of empty string literals - Update System.Text.Json to 10.0.0 for MCP SDK compatibility All target frameworks now build successfully: - .NET Framework 4.6.2 ✓ - .NET 6.0 ✓ - .NET 8.0 ✓ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixed build errors across all target frameworks (.NET Framework 4.6.2, .NET 6.0, .NET 8.0) for the MCP integration feature.
Changes Made
.NET Framework 4.6.2 Compatibility Fixes
[..n]withSubstring(0, n)StreamWriterconstructor to include bufferSize parameterWriteAsyncto use offset/length parameters.NET 8.0 MCP SDK Compatibility Fixes
webApplication.Urls.Add()instead ofUseUrls/ConfigureKestrelRunAsync()instead ofStartAsync+WaitForShutdownAsyncStyleCop Compliance
""withstring.EmptyPackage Updates
Build Status
All target frameworks now build successfully:
Files Modified
Snoop.Core/Infrastructure/MCP/McpServer.cs- Cross-framework API compatibility fixesSnoop.Core/Infrastructure/MCP/SnoopMcpTools.cs- StyleCop compliance fixesDirectory.packages.props- System.Text.Json version update (already in previous commit)Testing
🤖 Generated with Claude Code