Content
63%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.
The body is highly actionable, with concrete commands, deterministic scoring rules, and well-validated edge-case handling, and its section structure is clear. Its weaknesses are density and monolithic structure: substantial design-rationale prose inflates the token budget, and ~310 lines of reference-grade detail live inline with no bundle files to offload them.
Suggestions
Move the stable reference material — 'This repo's shape', the sweep session-forensics in 'The three call sites', and the worktree/submodule mechanics — into a references/ file (e.g. references/repo-shape.md), keeping SKILL.md to the invocation rules, the caps table, and the core workflow.
Compress the design-history justification sections ('It used to be narrated…', the Stop-gate disarm rationale, 'Why not attribute automatically?') into one-line rationale notes attached to the rules they justify, cutting roughly a third of the body.
Give each cap in the 'What caps the score' table its remedy as an explicit command (e.g. the exact push and stash-apply forms already stated in prose), so remediation is copy-paste ready rather than described.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body never explains concepts Claude already knows and is information-dense, but several sections are design-history justification rather than operational guidance: "It used to be narrated", "Why not attribute automatically? It was tried and it does not work", the Stop-gate disarm rationale, and the dravr-platform/mirroir-mcp sweep comparison. These could be cut or heavily compressed without affecting execution, fitting the 3 anchor (mostly efficient, some unnecessary explanation that could be tightened) rather than 4 (only minor trims needed). | 3 / 5 |
Actionability | Concrete, executable commands appear throughout: the four bilan.sh invocations with flags, defined exit codes, `bilan.sh ack --why "..."`, `git push origin <branch>:main`, `git submodule update --init .registre`, and `carnet.sh status <n>`, plus the precise core rule "Run it before you give a number. Report the score it prints." Minor gaps keep it below 5: most caps' remedies are described in prose rather than given as commands, and the limitation-label procedure names subcommands without a full invocation. | 4 / 5 |
Workflow Clarity | The core loop — run the script, report the number it prints, apply each cap's printed evidence and remedy, re-run — is unambiguous and deterministic (exit codes, the min() rule), and the limitation exemption has an explicit two-half validation. It sits below 5 because the sequence is implicit across prose sections rather than a stated workflow, and the sweep/worktree subsections describe conditions and edge cases rather than ordered steps. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/ directories), and all deep reference material — repo shape, sweep session-forensics, worktree/submodule mechanics, CI policy — is inline in one ~310-line file. Section headers are clear, so it is above the 2 anchor's minimal structure, but content that belongs in separate reference files is inlined and there are no one-level-deep references at all, matching the 3 anchor. | 3 / 5 |
Total | 14 / 20 Passed |