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.
A dense, expert-level body with genuinely concrete commands, data schemas, and expected outputs, plus a well-sequenced workflow with a lint gate. The weaknesses are repetition (boundary and precedent content stated 3-4 times across sections), an inlined forcing-question library that belongs in a reference file, referenced bundle files that do not exist in the skill as shipped, and a missing fix→re-lint→publish loop.
Suggestions
Consolidate the repeated boundary/precedent content: the deal-desk/pricing-strategist distinction appears in Purpose, When-to-use, Assumptions, and the Distinct-from table, and the 'Maria approved 28% on Acme' precedent anecdote appears three times — state each once and cross-reference.
Move the 8-question forcing-question library (recommended answers + canon citations) into a references/ file, keeping one-line question summaries in SKILL.md, and ship the referenced scripts/, references/, and assets/ files so every path in the body resolves.
Add an explicit re-validation loop to the workflow: after resolving BLOCKER findings, re-run policy_linter.py and publish only on PASS or PASS_WITH_WARNINGS.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Prose is expert-density and assumes Claude's competence (no explaining ARR, NRR, or discounting basics), but the same content repeats across sections: the precedent-loop anecdote ('Reps are citing "Maria approved 28% on Acme last quarter"' in When-to-use, 'Letting precedent set policy... Maria approved 28%' in Anti-patterns, and forcing-question 6) and the deal-desk/pricing-strategist boundary appears four times (Purpose, 'Do NOT use this skill', Assumptions, and the Distinct-from table). Consolidating these would cut 20-30% of tokens — more than the 'minor trim instances' of the anchor at 4, but not padded with concepts Claude already knows, so not a 2. | 3 / 5 |
Actionability | Copy-paste-ready commands with exact flags ('python3 scripts/discount_matrix_builder.py --input policy_intake.json --profile saas --output json > matrix.json'), exact intake fields ('arr, discount_pct, term_months, payment_terms_days, strategic_value, win_lost, nrr_12mo per deal'), and concrete expected outputs ('lints to FAIL with 4 BLOCKERs + 6 MAJORs + 2 MINORs'). Falls short of fully-executable because the referenced scripts/ and assets/ files are not present in the bundle, so the commands cannot be run as shipped, and the input JSON schema is only discoverable via --sample. | 4 / 5 |
Workflow Clarity | A clear 5-step sequence with an explicit validation gate ('Resolve every BLOCKER before publishing the matrix to AEs'), THIN-data flags ('Cells with n < 5 observed deals are flagged THIN'), NRR review flags, and a quarterly re-run loop ('Re-run the builder and the linter every quarter'). Not a 5 because the fix→re-validate loop is implied rather than stated — after resolving BLOCKERs there is no explicit instruction to re-run policy_linter.py and confirm PASS before publishing, which is the exact 'minor validation gap' of the anchor at 4. | 4 / 5 |
Progressive Disclosure | Section structure is good and references are clearly signaled (a References section with per-file descriptions and source counts), but the actual bundle contains no references/, scripts/, or assets/ directories — every referenced path (discount_governance_canon.md, policy_design_template.md, the three .py scripts) is absent, so the disclosed structure cannot be verified. Additionally, substantial content that belongs in a reference file — the 8-question forcing-question library with recommended answers and canon citations (~35 lines) — is inlined in SKILL.md, fitting the 'could be better organized / content that should be separate is inline' anchor at 3 rather than the 'appropriately placed' anchor at 4. | 3 / 5 |
Total | 14 / 20 Passed |