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 well-structured, highly actionable skill body with excellent executable examples and thoughtful error-recovery guidance. Its main weaknesses are inlined implementation internals that inflate token cost and a bundle file (references/asm-exec) that is never referenced by path, leaving a gap in how the agent locates and invokes the wrapper.
Suggestions
Reference the script explicitly, e.g. 'The wrapper lives at references/asm-exec; copy it to PATH or invoke it as `python references/asm-exec -- <command>`', so the agent can actually run it.
Move the 'SigV4 signing' internals and 'How It Works' resolution mechanics into a short reference beside the script (the script's docstring already covers much of it) and keep only the resolution order and security note in SKILL.md.
Add a pre-flight validation step before first use (e.g. check SMA on localhost:2773 or 'aws sts get-caller-identity') to complete the workflow's checkpoint sequence.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The rules, syntax table, and usage examples are lean, but sections like 'SigV4 signing' (credential resolution order, service/region inference from hostname) and 'How It Works' (re.sub callable mechanics, subprocess internals) explain implementation details the agent never acts on — these belong in or alongside the bundled script. Not 4 because two full paragraphs of signing internals and resolution-order mechanics are clearly trimmable; not 2 because there is no padding explaining concepts Claude already knows. | 3 / 5 |
Actionability | Concrete, copy-paste-ready bash examples covering psql, curl, mysql, docker, and config templating, plus a precise syntax table and actionable troubleshooting (specific stderr formats, 'aws sts get-caller-identity', ASM_EXEC_MCP_TIMEOUT). Not 5 because the skill never states where the asm-exec script lives or how to make it invokable — the references/asm-exec bundle file is not mentioned by path, so the agent cannot fully execute the guidance unaided. | 4 / 5 |
Workflow Clarity | The usage pattern is unambiguous (MUST NOT fetch directly; MUST use {{resolve:...}} via asm-exec), and the troubleshooting section provides a genuine feedback loop (read the stderr cause line first, check credentials/region/permissions, interpret 401 vs. missing secret, raise timeout). Not 5 because there is no pre-flight checkpoint (verify SMA is running or credentials are resolvable before first use) and no guidance on obtaining/invoking the script itself; well above 3 since error recovery is explicit and the single action is clear. | 4 / 5 |
Progressive Disclosure | Sections are well-organized, but the bundle contains references/asm-exec and the body never signals that path — 'How It Works' and 'SigV4 signing' implementation detail is inlined in SKILL.md where a one-level-deep reference to the script (or a short README for it) belongs. Fits anchor 3 ('references present but not clearly signaled; content that should be separate is inline'); not 4 because the sole bundle file is unreferenced and navigable discovery of it depends on guesswork. | 3 / 5 |
Total | 14 / 20 Passed |