Content
85%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 tightly written, highly operational instruction skill with excellent routing, an explicit output schema, and a built-in self-check — but the bundle is broken: every referenced reference file and script is absent, so the skill's core citation-driven workflow cannot execute as shipped. Fixing the bundle and showing script invocations would lift this to near-top marks across the board.
Suggestions
Ship the referenced bundle files: references/01-asset-class-norms.md through 05-benchmark-returns.md and scripts/benchmark_comparator.py and scripts/fee_drag_calculator.py are cited as the factual foundation (every flag, norm, question, and benchmark is cited from them) but none exist in the bundle, leaving the routing table and ID-citation rules pointing at nothing.
Add one-line invocation examples for both scripts (command syntax, inputs, expected output) — e.g. how benchmark_comparator.py is called and what it returns — so the §2 and §4 steps are executable rather than inferential.
Include a small example of the JSON artifact object (or per-field types) so the 'Output as JSON' mode has an unambiguous target shape instead of just a field name list.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 118-line body is dense and telegraphic throughout — it assumes Claude knows what a syndication, waterfall, J-curve, or cap rate is, uses routing tables instead of prose, and devotes essentially every line to operational rules ('Never gross-to-gross', 'pass 0 for undisclosed fees'). It matches the 5 anchor (lean, assumes competence, every token earns its place) rather than 4, since there is no over-explanation of known concepts, only scope-driven length. | 5 / 5 |
Actionability | Highly concrete guidance — a per-deal-type routing table (02 section, 03 flag prefixes, 05 comparator), a 10-section output schema, ID-citation rules, and named helper scripts. It falls short of the 5 anchor because script usage is never shown: 'scripts/benchmark_comparator.py' gets no invocation syntax at all, 'scripts/fee_drag_calculator.py' only hints at args ('pass 0 for undisclosed fees'), and the JSON artifact is a bare field list with no example object. | 4 / 5 |
Workflow Clarity | Clear sequenced workflow (Classify & parse → Route by deal type → Analyze & assemble) with explicit branching that reconverges on the same 10 sections, mode handling, an artifacts table mapping asks to outputs, and an explicit pre-emission validation step: 'Self-check before emitting: every flag family touched in §§2–3 appears by ID in §5, and every stated pref / waterfall / debt term has its 04 probe cited by ID in §8.' This analysis skill is neither destructive nor batch, so the cap-3 rule doesn't apply; the checklist satisfies the 5 anchor. | 5 / 5 |
Progressive Disclosure | The in-file design is strong — a load-condition table ('Always' vs 'GP identified'), one-level-deep references, and clear pointers — but scored against the actual bundle, all five 'references/*.md' files and both 'scripts/*.py' files cited as the skill's 'factual foundation' are missing from the bundle entirely. The routing leads nowhere as shipped, and the mandate that 'every norm, fee, flag, question, benchmark comes from references/' cannot be satisfied. This sits at 3: better organized than the 2 anchor's buried/inlined failure, but the broken pointers preclude 4. | 3 / 5 |
Total | 17 / 20 Passed |