Content
88%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 instruction-only skill: terse, concrete, and validation-heavy, with an unambiguous prioritized situation tree, exact commands for every write, and strong feedback loops around failure and interruption. The only improvement areas are minor: provide a literal PR body template and consider splitting the invariants or PR-body spec into a reference file to slim the main file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body teaches only skill-specific protocol: "Read the issue again before every write", "You push after every commit. A run can end at any moment", and exact commands like "git ls-remote --exit-code --heads origin tessl/<id>". No concept Claude already knows is explained, no section is padded, and every token carries operational weight, matching the lean anchor 5 rather than anchor 4 (which would still contain trimmable over-explanation). | 5 / 5 |
Actionability | Nearly everything is copy-paste executable: "gh issue view <n> --json number,title,body,url,state,labels,assignees,author,comments", "gh pr create --base <base> --label tessl --title \"#<n>: <what the change does>\"", "git merge --no-ff origin/<base>", the literal started-work comment block, and the "tessl launch attach-link" invocation. It sits at anchor 4 rather than 5 only because a few pieces are described rather than templated: the PR body is specified as a contents list ("why the change exists and what it does; what you ran and saw") instead of a literal block, and test commands are deferred to AGENTS.md/CLAUDE.md/README.md. | 4 / 5 |
Workflow Clarity | The "Situations" section is an ordered decision tree ("Read the record and take the first situation that applies") with explicit validation checkpoints and feedback loops: re-read the issue before every write, check the stop condition before every write, run tests again after merging the base, and on PR-creation failure "run `gh pr list --head <branch> --state open` before trying again". This matches anchor 5 (clear sequence, explicit validation, error-recovery loops) and exceeds anchor 4, where checkpoints would have gaps; the risky force-push case is also guarded with an explicit `--force-with-lease` recipe. | 5 / 5 |
Progressive Disclosure | The skill is a single self-contained file with clear, well-ordered sections (inputs, GitHub, done-state, situations, invariants, doing the work, judgment) and no nested or buried references; the paths it names (`.cloud-launch/trigger-events.json`, `setup.sh`, `AGENTS.md`) are runtime inputs, clearly signaled. It fits anchor 4 (good structure, content appropriately placed, minor organization gaps) rather than 5 because at ~130 dense lines the one-level-deep reference pattern of anchor 5 is absent and sections such as the 11-item invariants list could plausibly live in a companion file for a leaner overview. | 4 / 5 |
Total | 18 / 20 Passed |