-
Notifications
You must be signed in to change notification settings - Fork 45
Fix/tool annotations mapping #91
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
galatanovidiu
wants to merge
52
commits into
trunk
Choose a base branch
from
fix/tool-annotations-mapping
base: trunk
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
Changes from 36 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
8d4a7ca
feat(annotations): Add support for MCP annotations on prompts and res…
galatanovidiu d9b7b73
feat(annotations): Add support for MCP annotations on tools
galatanovidiu 1284837
docs: Update MCP annotations documentation to clarify type-specific b…
galatanovidiu ea1f25a
Add test fixtures for MCP annotation mapping
galatanovidiu 1a53442
Update McpAdapterConfigTest for new annotation test fixtures
galatanovidiu e6c6f13
Update validator tests to use MCP-compliant annotation format
galatanovidiu 84036d3
Add comprehensive MCP annotation validation tests for prompts
galatanovidiu a67eff3
Add comprehensive MCP annotation validation tests for resources
galatanovidiu 1338f82
Add comprehensive MCP annotation validation tests for tools
galatanovidiu fd03768
Add tests for MCP annotation mapping in prompts
galatanovidiu 4bd5906
Add tests for MCP annotation mapping in resources
galatanovidiu e406505
Add tests for MCP annotation mapping in tools
galatanovidiu 410478f
Removes redundant field type checks in annotation processing
galatanovidiu 5c62821
Refactors priority validation logic for clarity
galatanovidiu 4279271
Update includes/Domain/Resources/RegisterAbilityAsMcpResource.php
galatanovidiu bdfd6f9
Update includes/Domain/Resources/McpResourceValidator.php
galatanovidiu 94c7456
Update includes/Domain/Prompts/RegisterAbilityAsMcpPrompt.php
galatanovidiu 00a6423
Add McpValidator utility class with shared validation methods
galatanovidiu 26dff6f
Refactor ISO 8601 validation to use shared McpValidator utility
galatanovidiu baee55a
Update wp-env config to use WordPress master branch
galatanovidiu 06b44eb
Extracts annotation mapping logic into shared utility class
galatanovidiu 18f4bc4
Merge remote-tracking branch 'origin/fix/tool-annotations-mapping' in…
galatanovidiu e90ac38
Remove trailing whitespace in McpAnnotationMapper.php
galatanovidiu a324e79
Fix null annotation values being converted to false
galatanovidiu 6c1452c
Remove unnecessary assertion in test_non_mcp_fields_are_filtered_out
galatanovidiu ea8a12c
test: strengthen invalid annotations test assertions
galatanovidiu fd72336
test: strengthen priority clamping assertions
galatanovidiu f7a961a
test: assert priority exists in clamping test
galatanovidiu 578c6b7
test: strengthen invalid annotations filtering assertions
galatanovidiu b6bf7d7
test: adds test assertion for boolean type validation
galatanovidiu e5a8284
Adds explicit priority field validation check
galatanovidiu f0d3e7f
Update comment in McpAdapterConfigTest to clarify resource discovery …
galatanovidiu beac29c
Refactor priority field validation in McpAnnotationMapper
galatanovidiu e23f669
Add whitespace validation to base64 validator
galatanovidiu 17b57ca
Add comprehensive test suite for McpValidator
galatanovidiu d3e00ee
Add comprehensive test suite for McpAnnotationMapper
galatanovidiu b3c3f4e
Refactor McpPromptValidator and McpResourceValidator to use centraliz…
galatanovidiu 670e382
Add McpValidator utility class with shared validation methods
galatanovidiu 19a9f97
Refactor McpPromptValidator to use shared McpValidator methods
galatanovidiu 75bb8f4
Refactor McpResourceValidator to use shared McpValidator methods
galatanovidiu 7f0519f
Refactor McpToolValidator to use shared McpValidator methods
galatanovidiu 13e50e8
Update tests to use refactored validator methods
galatanovidiu 5c639d3
Improve MCP annotation handling and validation
galatanovidiu 44c49e6
Refactor RegisterAbilityAsMcpTool to streamline title assignment
galatanovidiu 5964afc
Refactor ability metadata annotations for consistency
galatanovidiu 94fec28
Add title assignment for annotations in RegisterAbilityAsMcpTool
galatanovidiu 8e10f8b
Update ability tests to reflect annotation key changes
galatanovidiu 0fa7819
Refactor validation logic in McpValidator and improve ability metadat…
galatanovidiu 0b99a5b
Use null instead of empty string for ability_property in 1:1 mappings
galatanovidiu 0bb6289
Use switch statement for annotation field validation
galatanovidiu b0df6c6
Use switch statements for annotation field validation
galatanovidiu 08f972d
Simplify resolve_annotation_value return type
galatanovidiu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.