Content
66%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 well-orchestrated thin-index SKILL.md with outstanding mode-based workflow design, explicit gates, and a closing checklist — but the bundle is incomplete: 12 of the 14 referenced rule/template files are absent, so the on-demand loading the whole skill depends on cannot actually happen. The description and structure are strong; the bundle contents are the critical gap.
Suggestions
Ship the referenced bundle files: all ten `rules/*.md` files (what-to-track, what-not-to-track, pii-and-compliance, event-design, otel-conventions, tracking-plan, implementation-web, implementation-mobile, audit-checklist, update-and-remove) and both `templates/*` files are referenced but missing from the bundle — every mode's step 1 currently fails.
De-duplicate the file lists: the Workflow-by-Mode sections and the Required Reading by Mode table repeat the same file references; keep the per-mode steps as actions and let the table carry the file loading.
Inline the minimal wrapper contract (the `track()` signature and event-name union) in SKILL.md so implement mode has executable guidance even before `templates/analytics-wrapper.template.ts` is loaded.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean — mode tables, numbered steps, one-line anti-patterns — and explains nothing Claude already knows. The one trim opportunity is duplication: the Workflow-by-Mode steps restate the exact file lists that the Required Reading by Mode table already carries. Efficient with minor instances that could be tightened, so 4 rather than 5. | 4 / 5 |
Actionability | The in-body guidance is concrete (explicit gates like "gate the user before writing instrumentation", "PII gate... on every property before the diff is final", "Mark `deprecated` first, set a sunset date, then remove"), but nearly all executable detail is deferred to `rules/*.md` and `templates/*.md`, and 12 of the 14 referenced paths do not exist in the bundle. Every mode's first step ("Load rules/what-to-track.md") points at a missing file, leaving key implementation details unavailable — some concrete guidance but incomplete. | 3 / 5 |
Workflow Clarity | Five modes each get a clear numbered sequence with explicit validation checkpoints: the plan-exists gate before instrumentation, the PII gate before the diff is final, downstream-consumer identification before deletion, and a Definition of Done checklist closing the loop. Remove mode's destructive risk is explicitly safeguarded (deprecate first, sunset date, update plan in same PR), so the destructive-operation cap does not apply. | 5 / 5 |
Progressive Disclosure | The index design itself is excellent — one level deep, clearly signaled, organized by mode — but scored against the actual bundle: only `references/event-catalog.md` and `references/platforms.md` exist. All ten `rules/*.md` files and both `templates/*` files referenced throughout the body are dangling, so navigation leads to dead ends for 12 of 14 paths. The claimed structure is not realized on disk, which is as broken as buried or missing references. | 2 / 5 |
Total | 14 / 20 Passed |