New Skill: wp-plugin-directory-guidelines#22
New Skill: wp-plugin-directory-guidelines#22davidperezgar wants to merge 16 commits intoWordPress:trunkfrom
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Summary
Testing
|
justlevine
left a comment
There was a problem hiding this comment.
Thanks for submitting this @davidperezgar 🙌
We should do our best to align with Skills best practices, particularly around progressive disclosure. For example, at most a skill should have 5K tokens, whereas this in its current form is around 8500-10,000 tokens depending on the model.
I left some more specific feedback inline on the PR, but if you havn't yet, I recommend you read both that Claude doc and the best practices from here: https://agentskills.io/skill-creation/best-practices .
|
I can recommend using the skill-creator skill to help with this. In general, is the intent for this to be an all-encompassing plugin review skill, or more focussed on just guidelines and license? I assume there's more to a review? But I'm not sure. |
|
Hello, thanks for your suggestions. I'll take care of them. |
Co-authored-by: Dovid Levine <justlevine@gmail.com>
Co-authored-by: Dovid Levine <justlevine@gmail.com>
|
@obenland and @justlevine all changes have been done. Could you check again? Thanks. |
skills/wp-plugin-directory-guidelines/references/guideline-review-checklist.md
Outdated
Show resolved
Hide resolved
skills/wp-plugin-directory-guidelines/references/guideline-review-checklist.md
Show resolved
Hide resolved
Co-authored-by: Konstantin Obenland <obenland@gmx.de>
Co-authored-by: Konstantin Obenland <obenland@gmx.de>
Co-authored-by: Konstantin Obenland <obenland@gmx.de>
|
Thanks for all suggestions @obenland . I've fixed, and added more examples in the rest of guidelines. |
justlevine
left a comment
There was a problem hiding this comment.
This skill looks much closer to the recommended best practices and skills that have been evaled for effectiveness, and resembles what I would expect a well-authored skill to look like. 🚀
We're still unable to return home, so I cannot eval them myself. looking forward to seeing the results.
Summary
Adds the
wp-plugin-directory-guidelinesskill — a structured review checklist for AI assistants evaluating WordPress plugins against the Detailed Plugin Guidelines and GPL licensing requirements.Fixes #21
What's included
SKILL.md— structured as actionable review steps, not a reference doc:License_Utilstrait, common violation patterns, and key concepts (distribution, derivative works, source availability)README.md— addedwp-plugin-directory-guidelinesto the Available Skills tableDesign decisions
is_license_gpl_compatible()regex inplugin-check, not a hand-curated listTest plan
SKILL.mdfrontmatter parses correctly (name,description,compatibility)references/links inSKILL.mdresolve to existing fileswp-project-triage)License_Utils.phpinplugin-check