docs(comfyui-plugin): add a skills index, installation, and the finishing-pass gate - #2229
Merged
Conversation
…hing-pass gate Three gaps from a /configure:readme audit against the house plugin-README shape (sampled from git-, blueprint-, taskwarrior-, agent-patterns-plugin): - **No skills index.** 17 skills documented in prose only, with no scannable entry point; 4 of 5 sibling plugin READMEs carry a table. Added one grouped by job — build/ship a pack, author-verify-publish, work on a graph, pick a node — with each row linking to its existing detail section. Descriptions are taken from each skill's own frontmatter rather than re-invented. - **No installation section.** 3 of 5 siblings have one. Added the marketplace add/install pair plus the external tools the pack-building skills expect (python3, uv, bun, just, rsvg-convert, Docker). - **Stale content.** The README described the `just assets` placeholder guard but not the finishing-pass gate, so it documented the state of the world before the previous commit. Added the pack-CI assertion and the re-runnable `--verify` verdict, and noted that /comfy-node reads STATUS= before handing back. Badges, logo, and tech-stack sections from the skill's generic project template are deliberately skipped: no plugin README in this repo uses them, and a plugin README is an index of skills, not a project landing page. Verified: all 10 anchor targets resolve to real headings, and all 17 skills on disk are named in the README.
laurigates
added
docs
Contributor
Plugin Compliance Review
Recommendations
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
A
/configure:readmeaudit ofcomfyui-plugin/README.md, applied against the house plugin-README shape rather than the skill's generic project template.Audit
Sampled the shape from
git-plugin,blueprint-plugin,taskwarrior-plugin,agent-patterns-plugin,foundryvtt-plugin:just assetsplaceholder guard but not the finishing-pass gate from #2226Changes
descriptionrather than re-invented, so the index cannot drift from the skill it names.python3,uv,bun,just,rsvg-convert, Docker).scaffold.py --verifyverdict, and notes that/comfy-nodereadsSTATUS=before handing back. This is the README update thenudge-plugin-readme-currencyhook asked for on that PR; fix(comfyui-plugin): gate the scaffold finishing pass instead of printing it #2226 merged before it was applied, so it lands here.Deliberately not applied
Badges, a logo block, and a tech-stack table are in the skill's standard template but are skipped: no plugin README in this repo carries them, and a plugin README is an index of skills, not a project landing page. Importing them would have made this file the outlier instead of fixing the way it already was one.
Verification
skills/.