Content
76%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 high-signal reference document: every line carries repo-specific policy that Claude could not infer, with exact commands, tables, and failure consequences. Its weaknesses are structural rather than informational — the 'add a feature' workflow is never sequenced, key content is duplicated between prose and tables, and the single file is starting to accumulate content that could live in reference files.
Suggestions
Add a short numbered 'Adding or changing a feature' workflow near the top that sequences the existing guidance: check WASM/Android compatibility against the ORT-incompatible list, decide on type-only/tract companions, update every relevant aggregate (with the full→windows-target parity consequence), then run `task verify:feature-parity` before finishing.
Remove the duplication between the inline wasm-target/android-target composition in the ORT-incompatible-targets section and the Aggregate Sets table — state each set's membership exactly once and cross-reference it.
Move the CI parity-guard detail (guarded-pairs table, UNGUARDED rationale, script mechanics) into a reference file (e.g. references/feature-parity.md) and keep a two-line summary plus the command in SKILL.md, restoring a lean overview page.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious repo facts ("no pyke prebuilt" for the x86_64 emulator, the jsDelivr 50 MB cap, "opt-level=2 still SIGBUS'd in go:e2e", "pdf_oxide" rejected not aliased) and explains nothing Claude already knows — matching the score-5 anchor's assumption of competence. It falls one notch below 5 due to verbatim duplication: the wasm-target composition ("no-ort-target + excel-wasm + ocr-wasm + layout-tract + auto-rotate-tract + ner-candle-wasm ... no tree-sitter") appears both inline and again in the aggregate table, and the long android-target member list is inlined in two places. | 4 / 5 |
Actionability | Guidance is fully concrete and executable: the exact CI command (`task verify:feature-parity` / `python3 scripts/ci/check-feature-parity.py crates/xberg/Cargo.toml`), exact feature algebra (`pdf = ["pdf-native"]`), exact config location (`core/config/pdf.rs`), a table of guarded pairs with allowed deltas, and explicit consequences ("adding a feature to `full` without adding it to `windows-target` fails CI"). Per the rubric's instruction-only note, the absence of code is not penalized when guidance is this actionable. | 5 / 5 |
Workflow Clarity | The primary use case promised by the description — adding or wiring a feature — is a multi-step process (assess WASM/Android safety, pick type-only/tract companions, update the relevant aggregates, run the parity guard, check profile implications), but the body never assembles these into a sequence. All the pieces, including a real validation mechanism, are present yet scattered across sections, matching the score-3 anchor ('sequence present but checkpoints missing or implicit') rather than score 4's clear sequence. | 3 / 5 |
Progressive Disclosure | No bundle files exist, so this is a single-file skill; its section headers (ORT-Incompatible Targets, PDF Backend, ORT Variants, Platform-Conditional, Aggregate Sets, CI parity guard, Build Profiles) make it navigable and every repo-file pointer (crates/xberg/Cargo.toml, scripts/ci/check-feature-parity.py) is real and clearly signaled. It sits at score 4 rather than 5 because at 115 lines some self-contained blocks — the CI parity-guard tables and the long aggregate member lists — are inlined where a one-level-deep reference file would keep the overview leaner, and the under-50-line simple-skill exception does not apply. | 4 / 5 |
Total | 16 / 20 Passed |