Content
86%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.
A strong, executable skill body: copy-paste-ready config and CI workflow, a clear setup sequence, and an appropriately delegated budgets reference. The main gaps are minor: some repeated "per [lhci]" padding, pinned versions that will age, and no explicit local pre-CI validation step.
Suggestions
Trim the repeated '(per [lhci][lhci])' / 'canonical' qualifiers and collapse the lab-vs-field section to its table to cut tokens without losing information.
Add an explicit validation checkpoint in the workflow, e.g. "run `lhci autorun` locally before committing the workflow; fix assertions that error, then re-run" before the CI-integration step.
Move version-sensitive details (the `@lhci/cli@0.15.x` reference, action versions) into a pinned-version note or old-patterns section so they can age without misleading.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely operational (config, commands, workflow YAML) rather than padded, but includes some trimming opportunities: repeated "per [lhci][lhci]" / "canonical" phrasing, a lab-vs-field explainer that partially restates known concepts, and pinned time-sensitive versions ("@lhci/cli@0.15.x", checkout@v5). Not anchor 3 because the CWV table and threshold facts carry operational value rather than being generic explanation; not anchor 5 because of the noted redundancy and version pins outside any deprecated/old-patterns section. | 4 / 5 |
Actionability | Fully executable throughout: `npm install --save-dev @lhci/cli`, a complete copy-paste-ready `.lighthouserc.js`, `lhci autorun` with per-phase and per-URL variants, and a full GitHub Actions workflow including artifact upload. This matches the anchor-5 example of copy-paste ready commands covering the common cases. | 5 / 5 |
Workflow Clarity | Clear Install → Configure → Running → CI-integration sequence with the autorun phases (collect/assert/upload) enumerated, and warn-vs-error levels explained. It falls short of anchor 5 because there is no explicit local validation step (e.g., run `lhci autorun` locally to verify the config before committing the workflow) and no error-recovery guidance for what to do when a gate fails beyond the anti-pattern table's advice. | 4 / 5 |
Progressive Disclosure | The deep design-time material (per-route budgets, budget.json, assertMatrix, validation) is split into a real one-level-deep reference — references/budgets.md exists, is well-signaled in the "Authoring budgets" section with "Start there when..." guidance, and contains no nested .md references — while the body keeps the operational quick path. Navigation is easy with a consistent section structure and a References section. | 5 / 5 |
Total | 18 / 20 Passed |