Content
71%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 strong, concrete pattern catalog with executable templates and built-in validation checks, undermined by generic filler sections, a broken bundle reference, and version-pinned links. A hidden HTML comment claiming a `curl-pipe-bash` security allowlist, paired with the `curl ... | sh` install command, is also worth flagging: it is not a documented pattern and should be removed or justified.
Suggestions
Fix or remove the broken reference: either create the referenced file (under the conventional `references/` directory) or drop the pointer to `resources/implementation-playbook.md`, which does not exist in the bundle.
Cut the generic filler — the vacuous "Do not use this skill when" bullets, the boilerplate "Instructions" section, and the ASCII architecture diagram — and move the version-pinned 2.14 doc links into a clearly dated section so version drift doesn't rot the skill.
Remove the hidden `<!-- security-allowlist: curl-pipe-bash -->` comment and prefer Linkerd's documented install path over `curl | sh` in Template 1, or at minimum surface the trust implication of piping a remote script to a shell.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The seven templates and command sections are lean, but the body pads with content Claude does not need: a vacuous "Do not use this skill when" section ("The task is unrelated to linkerd patterns"), generic filler Instructions ("Clarify goals, constraints, and required inputs. Apply relevant best practices"), an ASCII architecture diagram of concepts Claude already knows, and a repeated marketing tagline. Version-pinned URLs ("linkerd.io/2.14/") are time-sensitive and not quarantined in a deprecated section, which the guidelines say should penalize conciseness. This fits anchor 3 ("mostly efficient but includes some unnecessary explanation or could be tightened") — not 4 because the filler sections and version pins are more than minor trimmings. | 3 / 5 |
Actionability | Seven templates provide copy-paste-ready, complete bash and YAML: install sequence with `linkerd check` validation, namespace/deployment injection annotations, a full ServiceProfile with routes/responseClasses/retryBudget, TrafficSplit with SMI weights, Server/ServerAuthorization policies, HTTPRoute rules, and multicluster link/export/check commands, plus concrete viz/tap debugging commands. This matches anchor 5 ("fully executable; copy-paste ready code or commands; specific examples cover the common cases") rather than 4, which allows for minor gaps the examples here don't have. | 5 / 5 |
Workflow Clarity | The install template is properly sequenced with validation checkpoints (`linkerd check --pre` before, `linkerd check` after, `linkerd multicluster check`), and the Best Practices Don'ts mandate re-running checks after changes. It is not 5 because there is no error-recovery loop (what to do when a check fails) and the top-level "Instructions" section gives no real sequencing across the pattern catalog; it is not 3 because concrete checkpoints are explicitly present, not merely implied. | 4 / 5 |
Progressive Disclosure | Section headers and a Key Resources table give the ~300-line body reasonable organization, but the single external reference — "open `resources/implementation-playbook.md`" — is broken: no references/, resources/, scripts/, or assets/ directory exists in the bundle, so the path resolves to nothing, and it is buried in an Instructions bullet rather than clearly signaled. This matches anchor 3 ("some structure but could be better organized; references present but not clearly signaled") — not 2 because the inline templates are appropriately presented as a pattern catalog with clear headers rather than a monolithic dump. | 3 / 5 |
Total | 15 / 20 Passed |