ci: auto-sync skills to duyet/skills on push to master#67
Conversation
Triggers on push to master when any skill-type plugin changes. Flattens each plugin's skills/ subdir (or root SKILL.md for good-html) into the flat layout that duyet/skills uses, opens a PR there with the diff. Requires SKILLS_SYNC_TOKEN repo secret: a fine-grained PAT scoped to duyet/skills with contents:write + pull-requests:write.
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
Warning Review limit reached
Your plan currently allows 1 review/hour. Refill in 25 minutes and 56 seconds. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more review capacity refills, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Welcome! 👋Thank you for your first contribution to this project! We appreciate you taking the time to improve our codebase. Next Steps
Contribution Guidelines
Thanks again for contributing! 🎉 |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
- add --- document start - disable truthy rule on `on:` (matches lint.yml convention) - wrap long lines to fit 80-col default
The printf format string is deliberately single-quoted to keep \n and backticks literal; the arguments do the interpolation.
Satisfies zizmor unpinned-uses (error) and artipacked (warning) audits that super-linter runs against changed workflow files.
Prettier (default super-linter formatter) was rewriting quotes/spaces. Quoted `on:` sidesteps the truthy/comment-spacing tug-of-war between yamllint and prettier.
1 participant
Summary
Adds
.github/workflows/sync-skills.yml— when any skill-type plugin (ormarketplace.json) changes onmaster, the workflow:<plugin>/skills/<skill>/(or rootSKILL.mdforgood-html) into<skill>/layoutduyet/skills(so deletions propagate)duyet/skillstitledSync skills from codex-claude-plugins@<sha>If there are no skill changes after flattening, exits cleanly without opening a PR.
Required secret
Before merging, create a fine-grained PAT in the repo settings:
duyet/skillsContents: read & write,Pull requests: read & writeStore as
SKILLS_SYNC_TOKENunder repo Secrets. The defaultGITHUB_TOKENcan't push to a different repo.Allow-list
The plugins to sync are listed once at the top of the workflow as
SOURCE_PLUGINS. Editing that line is the only step needed to add or remove a synced skill.Companion PR on the target repo: duyet/skills#6 (initial bulk sync + adds the matching
npx skills addCLI).Test plan
SKILLS_SYNC_TOKENto repo secretsworkflow_dispatchonce — confirm it opens (or no-ops cleanly against) duyet/skillsskills/.../SKILL.mdon master and confirm a fresh PR opens on duyet/skillsGenerated by Claude Code