Content
72%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 lean, highly specific feature-flag reference with concrete commands and config values, but it functions as a single inline catalog rather than an overview with sequenced workflows or one-level-deep references.
Suggestions
Add a short numbered workflow for the primary task (e.g. 1. decide platform targets, 2. edit crates/xberg/Cargo.toml, 3. add to the relevant aggregate sets, 4. run `task verify:feature-parity`, 5. fix any flagged deltas) to lift workflow_clarity above 3.
Split the larger reference blocks — the aggregate-sets table, CI parity guard, and build profiles — into separate files under references/ and link to them from SKILL.md so the body serves as a navigable overview (improves progressive_disclosure).
Add a brief 'adding a new feature' checklist that names the aggregate sets and guarded supersets a new feature must be added to, turning the implicit CI rule into an explicit pre-commit step.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and information-packed with no padding — it never explains what Cargo features, ORT, or tract are, and every line states a specific fact, feature name, config value, or command, assuming Claude's competence. | 5 / 5 |
Actionability | It gives concrete, executable guidance — the exact parity command `python3 scripts/ci/check-feature-parity.py crates/xberg/Cargo.toml`, exact config values `PdfConfig.backend` parses ("native", "pdfium"), and explicit directives like "Read the manifest before assuming a member" — but it is reference-catalog oriented rather than fully step-by-step, leaving minor gaps. | 4 / 5 |
Workflow Clarity | The primary tasks named in the description (adding, wiring, debugging a feature) are not sequenced into steps; the CI parity-guard section does provide one explicit validation checkpoint with clear "fails CI" consequences and a guarded-pairs table, but the overall workflow sequencing is implicit rather than laid out. | 3 / 5 |
Progressive Disclosure | The document is well-sectioned with clear `##` headers and a table, but it is a ~110-line monolithic inline reference with no pointers to separate reference files; substantial sections like the aggregate-sets table and CI parity guard could be split out so SKILL.md acts as an overview. | 3 / 5 |
Total | 15 / 20 Passed |