Content
65%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a minimal auto-generated redirect stub: maximally token-efficient, but it delegates 100% of the skill's substance to an external file (~/.agents/skills/pr-style/SKILL.md) that is not part of the bundle and does not exist in this environment, leaving the skill non-functional as shipped. Without the target file or any inline fallback content, actionability, workflow clarity, and progressive disclosure all bottom out at the pointer level.
Suggestions
Ship the referenced formatting content inside the skill bundle (e.g., a references/ file with the title format, body template, label mapping, assignment rules, and draft policy) instead of relying solely on the external absolute path ~/.agents/skills/pr-style/SKILL.md, which resolves only on machines where that shared skill happens to be installed.
Add a fallback/validation checkpoint to the one-step workflow: state what to do if the shared skill file is missing (e.g., fall back to the bundled reference, or fail with a clear message) so the delegation step has an error-recovery path.
Inline a minimal summary of the core rules (a title-format example and a body-template skeleton) in SKILL.md so the skill remains actionable when read in isolation, keeping the full detail in the external or referenced file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is two lines: a provenance comment and one directive ("Load and follow the shared skill at ~/.agents/skills/pr-style/SKILL.md"). Nothing pads or over-explains; every token serves the redirect, matching anchor 5 ("lean and efficient; every token earns its place"). It is clearly above anchor 4, which still expects some trimmable over-explanation. | 5 / 5 |
Actionability | The single directive is concrete and unambiguous (an exact action and path), so it clears anchor 2's "high-level hints". But it stops at a pointer: none of the actual formatting guidance (title format, body template, label mapping, assignment, draft policy) exists in the body or the bundle, and the referenced file "~/.agents/skills/pr-style/SKILL.md" is not present, so as shipped the skill delivers no executable value — below anchor 4's "concrete code or commands with minor gaps". | 3 / 5 |
Workflow Clarity | The single action ("Load and follow the shared skill") is unambiguous, which under the simple-skill exception could score 5. However the one-step workflow dead-ends: the delegation target is missing from the bundle, and there is no checkpoint or fallback for that case (e.g., what to do if the file cannot be found), matching anchor 3's "sequence present but checkpoints missing or implicit". | 3 / 5 |
Progressive Disclosure | The reference is one level deep and clearly signaled — the entire body is the pointer — but it points to an absolute home-directory path outside the skill bundle (no references/, scripts/, or assets/ directories exist), and that path does not resolve in this environment. The body also offers no overview of what the referenced material contains, fitting anchor 3 ("some structure but could be better organized") rather than anchor 4's "most content is appropriately placed". | 3 / 5 |
Total | 14 / 20 Passed |