-
Notifications
You must be signed in to change notification settings - Fork 146
Merge .net CLI to main #166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
asklar
wants to merge
72
commits into
modelcontextprotocol:main
Choose a base branch
from
asklar:dotnet
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
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
- Complete .NET 8 CLI port with feature parity to TypeScript version - Interactive init command with server type auto-detection - Pack/unpack with identical archive format and output formatting - Validate command with comprehensive manifest validation - Sign/verify/unsign using .NET SignedCms with PKCS#7 detached signatures - Info command showing bundle details and signature status - Self-signed certificate generation for testing - Comprehensive test suite with CLI integration tests - Modern .slnx solution format - Updated .gitignore with .NET build artifact patterns - Documentation in dotnet/README.md and main README.md
- Updated author information - Added NuGet package metadata (license, URLs, tags) - Removed unused Spectre.Console dependency - Configured README.md to be included in package - Enhanced package description and tags for discoverability
- Disabled debug symbols generation (DebugType=none, DebugSymbols=false) - Excluded satellite resource assemblies (SatelliteResourceLanguages=en) - Bumped version to 0.1.1 - Reduced package size from ~507KB to ~367KB (28% reduction)
…and manifest validation improvements - Added options for dynamic tool discovery in PackCommand: --force, --update, and --no-discover. - Implemented validation of referenced files in the manifest before packing. - Improved handling of tool and prompt mismatches during packing, with options to update the manifest. - Enhanced ValidateCommand to support directory-based validation and update capabilities. - Introduced ValidationSeverity enum to categorize validation issues as errors or warnings. - Updated ManifestValidator to provide more detailed validation feedback, including warnings for missing prompt texts. - Added JsonSerializerOptions customization for better JSON handling in McpbJsonContext. - Updated project dependencies to include ModelContextProtocol for enhanced functionality.
…ltiple files - Updated formatting and indentation in CLI documentation and test files for better readability. - Enhanced the structure of manifest-related classes and validation logic to ensure clarity. - Adjusted JSON serialization settings for better compatibility and maintainability. - Fixed minor issues in project file references and ensured consistent use of naming conventions.
…nd improve error handling
Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…on from client Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
… trigger Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…d document file-system-node rich schemas Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…ified manifest args Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…ation Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…gnment time Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…ation Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…Text.Json source generator Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
…n-theming Add localization and icons support to .NET MCPB CLI (manifest v0.3)
Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
Co-authored-by: asklar <[email protected]>
Add NuGet publishing workflow with trusted publishing
This workflow publishes the .NET tool to NuGet using Trusted Publishing (OIDC). It includes steps for checkout, setup, restoring dependencies, building, testing, packing, and publishing.
Removed API key from NuGet push command for security.
Removed API key from NuGet push command for security.
- Added `--discover` option to validate that discovered tools/prompts match the manifest without updating. - Introduced `--verbose` option to print detailed validation steps. - Implemented checks to prevent simultaneous use of `--update` and `--discover`. - Enhanced logging for asset path normalization and discovery violations. - Updated manifest name and version based on server responses when using `--update`. - Improved error handling and reporting for validation mismatches. - Incremented version number to 0.3.4 in the project file.
update filesystem server version
Add the ability to pass user_config to discovery
app exec alias support
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Motivation and Context
How Has This Been Tested?
Breaking Changes
Types of changes
Checklist
Additional context