Content
68%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.
This is a minimal generated shim: perfectly lean and unambiguous as a pointer, but it carries none of the skill's actual operational content — everything depends on the external shared file resolving. The description frontmatter does the heavy lifting for triggering; the body's sole job is delegation, which it does cleanly but with an environment-dependent absolute reference. Suggestions focus on making the delegation robust and self-sufficient.
Suggestions
Make the delegation robust: state what to do if `~/.agents/skills/ci-fix/SKILL.md` is missing (e.g., fall back to bundled instructions or surface an explicit error), since the body is otherwise empty and the skill fails silently without that file.
Add one line of orientation (e.g., "Full workflow: enumerate failures → diagnose root cause → fix → commit/push → watch runs → repeat") so the body is not entirely dependent on the external file for the skill's core loop and validation checkpoints.
If the bundle is meant to be portable, ship the shared SKILL.md (or a copy) in a references/ directory and reference it bundle-relative instead of via an absolute home-directory path.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is two lines — an HTML generation comment and "Load and follow the shared skill at `~/.agents/skills/ci-fix/SKILL.md`" — with zero padding and no explanation of things Claude already knows. Not 4: there is literally nothing that could be trimmed; every token earns its place. | 5 / 5 |
Actionability | The single directive is concrete (a specific path and a specific action), but all task-executable guidance — how to enumerate failures, apply fixes, commit, push, watch runs — is delegated to the referenced file, so the body itself is incomplete guidance for the task. Not 4: an executable pointer is not executable task guidance; the body contains no steps, commands, or examples of its own. | 3 / 5 |
Workflow Clarity | The delegation action itself is unambiguous, but the body exposes no sequence and no validation checkpoints for the multi-step fix→commit→push→watch→repeat loop it fronts; the simple-skill exception doesn't apply because the underlying task is a complex, risky (push-triggering) workflow whose checkpoints are invisible here. Not 1: the one step written is coherent and unambiguous rather than missing. | 3 / 5 |
Progressive Disclosure | One clearly-signaled, one-level-deep reference — exactly the right shape for a wrapper — and no inlined content that belongs elsewhere. Not 5: the pointer targets an absolute machine-local path (`~/.agents/skills/ci-fix/SKILL.md`) outside this bundle (no references/ directory exists, and the path does not resolve in this environment), so navigation depends on environment setup rather than bundled files. | 4 / 5 |
Total | 15 / 20 Passed |