-
Notifications
You must be signed in to change notification settings - Fork 74
Copy over techdocs-mcp-tool plugin from redhat-ai-dev fork #1491
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
Copy over techdocs-mcp-tool plugin from redhat-ai-dev fork #1491
Conversation
Co-authored-by: Theofanis Petkos <thepetk@gmail.com> Signed-off-by: John Collier <jcollier@redhat.com>
Signed-off-by: John Collier <jcollier@redhat.com>
Changed Packages
|
Signed-off-by: John Collier <jcollier@redhat.com>
The specific test failure in techdocs plugin:
|
Failing test cases should be addressed by: redhat-ai-dev/community-plugins#6 |
54ad293
to
69c45f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm based also on my tests locally. Regarding the sonar cloud error I'm not sure if is a good idea to remove the regex mentioned.
Rebasing on main before merging |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apologies did not get to this before you all wrapped up but fwiw I saw no issues / had no comments post merge
Hey, I just made a Pull Request!
This PR copies over @thepetk's techdocs-mcp-tool plugin originally present on the redhat-ai-dev fork. No changes to the plugin have been made.
I've also updated the app-config in the workspace to reference the new plugin, and Cursor shows all of the MCP tools when connected to the MCP server:
However, the tests from the techdocs tool are failing and need investigation
CC @thepetk
✔️ Checklist