fix: re-adapt skills to skill-creator standards#5
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, 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 the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
There was a problem hiding this comment.
Pull request overview
This PR aligns the repository’s Claude skills with the current skill-creator metadata standards and tightens the pre-commit zizmor invocation to use the repo’s configuration.
Changes:
- Update the local pre-commit
zizmorhook to usezizmor.ymlwhen scanning.github/workflows/. - Rework skill frontmatter descriptions to be trigger-focused (“Use this skill whenever…”).
- Remove
allowed-toolsfrom skill frontmatter and adddisable-model-invocation: trueto/ship.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.pre-commit-config.yaml |
Updates zizmor hook entry to apply the repo’s zizmor.yml config when linting workflows. |
.claude/skills/ship/SKILL.md |
Updates frontmatter description, removes allowed-tools, adds disable-model-invocation: true. |
.claude/skills/new-lab/SKILL.md |
Updates frontmatter description and removes allowed-tools. |
You can also share your feedback on Copilot code review. Take the survey.
Summary
allowed-toolsfrom skill frontmatter (skills inherit all capabilities)disable-model-invocation: trueto/ship(side-effect workflow)Test plan
/autocomplete