Content
78%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 strong, highly actionable skill body: executable commands for every mode, a clear six-step workflow, and correct use of one-level-deep bundle references. The main weaknesses are redundancy around fractional-share mode and dated regulatory text inline, which cost token efficiency without adding proportionate guidance.
Suggestions
Collapse the five near-duplicate bash blocks into one canonical invocation plus a short table of mode-specific flags, and consolidate the fractional-share guidance (currently in Overview, Step 1, Step 2, and Key Principles) into a single place.
Move the dated FINRA intraday-margin detail (effective 2026-06-04, phase-in through 2027-10-20) into references/sizing_methodologies.md and keep only a short 'check your broker's current intraday margin rules' pointer in the body.
Add a brief validation checkpoint after running the script (e.g., confirm the output JSON's final_risk_pct is within the requested risk budget and re-check inputs on error) to close the workflow's feedback-loop gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and skill-specific, but it repeats the fractional-share guidance in at least four places (Overview, Step 1, a full extra bash example, Key Principles 3-4) and the five near-duplicate bash blocks could be collapsed into one command plus a flag table. It also embeds time-sensitive regulatory detail with specific dates ("effective 2026-06-04, with broker phase-in allowed through 2027-10-20") outside any 'old patterns' section, which the guidelines penalize. Not score 4 because these are more than minor trimmable instances; not score 2 because there is no padding that explains concepts Claude already knows. | 3 / 5 |
Actionability | Every workflow step is backed by copy-paste-ready commands: five complete bash invocations covering fixed-fractional, fractional, ATR, and both Kelly modes, plus a constraints example and a concrete JSON output schema. The examples cover the common cases exactly as the top anchor requires. Not score 4 because the commands are complete and executable as written with all flags and realistic values. | 5 / 5 |
Workflow Clarity | The six steps are clearly sequenced with concrete commands and explicit decision points (mode selection in Step 1, scenario comparison in Step 4, binding-constraint explanation in Step 5). This is a read-only calculation skill, so the destructive/batch cap does not apply. Not score 5 because there are no explicit validation checkpoints or error-recovery loops (e.g., what to do if the script errors or inputs are inconsistent); not score 3 because the sequence itself is complete and checkpoints are only implicit, not missing. | 4 / 5 |
Progressive Disclosure | The body is a workflow-focused overview that delegates methodology depth to a well-signaled, one-level-deep reference ("Read `references/sizing_methodologies.md`", listed again in Resources with a description), and the calculation logic to `scripts/position_sizer.py`; both files exist in the bundle. Not score 4 because there is no inline content that clearly belongs in a separate file and all references are clearly signaled in both the workflow and a Resources section. | 5 / 5 |
Total | 17 / 20 Passed |