-
Notifications
You must be signed in to change notification settings - Fork 1
1.x Resources Management #23
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
Merged
Merged
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
Introduced a resource handling system including ResourceRepository, ResourceInterface, and example implementations. Updated service definitions, configuration, and tests to register and validate resources dynamically. Enhanced MCP server with resource management capabilities.
The filesystem resource provider configuration is now commented out in the KlpMcpServerExtensionTest. This change reflects that the resource provider is not currently needed for the test, improving clarity and maintaining focus on relevant configurations.
Added a "Usage" section in the README with an example for creating custom resources. Removed outdated and commented configuration code related to resources in `Configuration.php`.
Added a "Usage" section in the README with an example for creating custom resources. Removed outdated and commented configuration code related to resources in `Configuration.php`.
Introduced resource templates and handling in the server, added tests for resource listing, reading, and repository logic. Enhanced `VersionCheckTool` and updated descriptors while refining the resource handling logic and configuration options.
Implemented the `supportsResources` feature in `ServerCapabilities` to enable resource registration and exposure to clients. Updated related methods, serialization logic, and tests to account for resources configuration. Extended `MCPServer` to integrate resources functionality seamlessly.
Added PHPDoc comments to MCPServerTest for better documentation of test cases. Introduced comprehensive tests for McpDocumentationResource ensuring accurate URI handling, MIME type guessing, and resource management. Cleaned up unnecessary verbosity in KlpMcpServerExtensionTest comments for clarity.
The `guidelines.md` file was renamed to `development_guidelines.md` for improved clarity. All corresponding references in the documentation were updated to reflect this change. This ensures consistency and avoids potential broken links.
Cleaned up unnecessary imports to improve code readability and maintainability. This reduces potential confusion and ensures that only relevant dependencies are included.
Enhanced resource filtering logic to target .md files in Finder. Added detailed documentation on building resources and templates with comprehensive examples.
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.
No description provided.