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.
An exemplary rule-based skill body: token-efficient, organized by scenario, with precise API/path-level guidance and a well-structured reference table pointing to real, one-level-deep bundle files. The residual gaps are minor — a couple of directives ('verify with browser tooling', 'preserve URL state') remain high-level without a concrete example.
Suggestions
In the Verification section, name a concrete verification action (e.g., which browser tool command or what to check) instead of the generic 'use the available browser tooling'.
Add a 3-4 line example of a handle.agentRoute metadata object in Routes.tsx so the Route Metadata rule is copy-paste-ready rather than field-name-only.
Show one concrete URL-state pattern (e.g., a route/search param mapping for a selected tab) to make the URL State section executable rather than directional.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and efficient throughout: short sections of project-specific rules with no explanation of concepts Claude already knows (no React/Relay primers, no padding). Every token carries a project-specific instruction, matching the 5 anchor; it is clearly above the 4 anchor's 'minor instances of over-explanation'. | 5 / 5 |
Actionability | Names exact, executable targets: 'useAgentDataChangeFetchKey from @phoenix/hooks in the query owner', 'handle.agentRoute metadata in js/app/src/Routes.tsx' with field-level detail ('label', 'description'), and 'encode it in route params or search params'. It sits between anchors: mostly executable with minor gaps (4) rather than fully copy-paste-ready (5) — phrases like 'use the available browser tooling to verify' and 'preserve the relevant URL state' give direction without a concrete example or command. | 4 / 5 |
Workflow Clarity | An explicit starting step ('Before starting work, explore js/app/src/components/ and js/app/package.json… then follow these rules') plus scenario-conditioned rules and a verification section give a clear structure with most checkpoints present (4). Not a 5 because it is a rule collection rather than a sequenced workflow — there are no explicit feedback/recovery checkpoints — and not a 3 because the per-section when/what pairs are unambiguous and validation guidance ('use the available browser tooling to verify the UI looks correct', 'check its usages across the app') is present. | 4 / 5 |
Progressive Disclosure | A concise overview body with a 'when to read' table cleanly mapping all five real reference files (components.md, relay.md, accessibility.md, test-ids.md, resize-svg-logo-assets.md) to task conditions, references exactly one level deep (verified: reference files point only to the bundled script, not to further .md files), and out-of-scope areas delegated to sibling skills (phoenix-storybook, phoenix-design). Matches the 5 anchor's clear overview with well-signaled one-level-deep references. | 5 / 5 |
Total | 18 / 20 Passed |