Content
57%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-structured, largely actionable body undermined by changelog-tagged version noise repeated five times, a conformance-test stub section, and four references to files that do not exist anywhere in the bundle — one of which carries the skill's only validation guidance for a mutating batch operation. Cleaning the version tags and either shipping or removing the dangling references would materially raise conciseness and workflow clarity.
Suggestions
Strip the repeated "(v0.25.1)" markers (they also contradict frontmatter `version: 1.1.0`) and delete the trailing '## Output Format' conformance-test stub — both are pure token overhead.
Add an explicit post-patch validation step (e.g. re-scan the patched page for citation-format compliance and re-run the format check on a sample before batch-committing) so the batch/mutating workflow has a real feedback loop rather than a dangling `conventions/test-before-bulk.md` pointer.
Create a references/ directory containing the cited files (quality.md citation format, tweet-resolution pipeline detail, X API recipe usage) or rewrite the inlined detail directly, so no navigation target in the body is broken.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient, but "**(v0.25.1)**" is stamped five times across Contract, Phases, and step headings (and conflicts with the frontmatter's `version: 1.1.0`), and the trailing "## Output Format" stub whose only content is "The literal section header here exists for the conformance test (`test/skills-conformance.test.ts`)" is pure padding. Version-tagged time-sensitive info outside a deprecated section and meta-noise for a test harness are exactly the unnecessary tokens the rubric penalizes. | 3 / 5 |
Actionability | Concrete, near-executable guidance throughout: a runnable bash candidate-finder, exact search query patterns (`from:<handle> "<exact quote fragment>"`, `"<exact quote>" -is:retweet`), a before/after patch example, a JSON state-file schema, and a report template. Not 5 because the core integration point is hand-waved — "goes through whatever X integration the host has configured (typical shape: a recipe under `recipes/x-api/` ...)" — leaving the actual API call non-executable. | 4 / 5 |
Workflow Clarity | The 5-phase sequence and 5-step tweet pipeline are clearly laid out, and Step 4 verifies candidates ("Confirm the text matches the quoted fragment"), but this is a mutating batch operation and the validation guidance is a dangling pointer ("see `conventions/test-before-bulk.md`" — the file does not exist in the bundle) with no post-patch re-validation loop. Per the rubric's cap, a batch/destructive skill without a working validation step cannot score above 3. | 3 / 5 |
Progressive Disclosure | Section headers organize the ~190-line body, but everything is inlined (the full tweet-resolution pipeline and batch-mode detail would fit a reference file), and every external pointer — "[conventions/quality.md](../conventions/quality.md)", "[_output-rules.md](../_output-rules.md)", "conventions/test-before-bulk.md", "recipes/x-api/" — is dangling: the skill bundle contains no references/, scripts/, or assets/ directories at all. Structure exists, but navigation targets are unreliable, matching the 'some structure, could be better organized' anchor. | 3 / 5 |
Total | 13 / 20 Passed |