fix(ci): copy skills to skills/ directory instead of repo root#4065
fix(ci): copy skills to skills/ directory instead of repo root#4065NathanFlurry wants to merge 1 commit intomainfrom
Conversation
|
🚅 Deployed to the rivet-pr-4065 environment in rivet-frontend
|
PR Review: Fix CI skills directory structureSummaryThis PR correctly fixes the skills publishing workflow to copy generated skills to the Code Quality ✅The changes are clean and well-commented:
Implementation Review ✅Before: Skills were being copied to The fix properly:
Potential IssuesNone identified. The changes are straightforward and correct. Testing VerificationThe PR description mentions testing instructions in the workflow file comments. You can verify the fix works by:
Minor ObservationsGood practices followed:
Recommendation✅ LGTM - This is a clean bugfix that resolves the directory structure issue. The code is clear, correct, and maintains consistency with the repository's patterns. |
The skills workflow was copying generated skills to the root of rivet-dev/skills repo instead of the skills/ subdirectory.
Changes
skills-repo/skills/$skill_nameinstead ofskills-repo/$skill_nameskills/directory exists before copyingrivetkit-typescriptcleanup (not needed)