Add skills support - #52
Conversation
Signed-off-by: Stephanie <yangcao@redhat.com>
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
The invalid_question_response replacement regex only looked ahead to ^mcp_servers:, so it swallowed the new skills: block (inserted between invalid_question_response and mcp_servers) and deleted it whenever the sync script regenerated the file, causing validate-configs to fail. Co-authored-by: Cursor <cursoragent@cursor.com>
Jdubrick
left a comment
Jdubrick
left a comment
There was a problem hiding this comment.
Changes themselves lgtm. Do we want to update CONTRIBUTING.md with the new make command? and do we need to update the gitops sync script in this repo to sync this to the dev environment, or are those follow ups?
Signed-off-by: Stephanie <yangcao@redhat.com>
updated CONTRIBUTING.md. |
2 participants
What does this PR do?:
Description:
Summary
skillsconfig inlightspeed-stack.yaml, compose volume mount,fetch-skills.shscript, and
make get-skillstargetTest plan
make get-skillsfetches skills intoskills/directorymake local-upstarts successfully with skills volume mounted🤖 Generated with Claude Code
Which issue(s) this PR fixes:
https://redhat.atlassian.net/browse/RHIDP-14224
PR acceptance criteria:
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.
Tested and Verified
Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)
How to test changes / Special notes to the reviewer:
skills are available under
/v1/skills, and also consumable in query