Content
60%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 delivers solid executable CLI guidance with a clear connection/readiness workflow, but it is undermined by a massively padded entity list that bloats token usage. Moving that catalog to a reference file and trimming the generated "Acceptable..." entries would materially improve quality.
Suggestions
Move the ~280-item entity catalog out of SKILL.md into a references file (e.g. references/entities.md) and keep only the top-level entity groups inline.
Delete the auto-generated "Acceptable Data Security ..." proliferation, which appears to be padding rather than real Drata entities.
Add one fully-resolved example command (real connection id and a concrete --input payload) for running an action so guidance is copy-paste ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The "Drata Overview" section is a ~280-item bullet list, the bulk of which are auto-generated padding entries (dozens of "Acceptable Data Security ..." variants), making the body noticeably verbose and heavily padded even though the CLI instruction sections are reasonably tight; not 1 because it avoids explaining concepts Claude already knows, and not 3 because the entity-list padding wastes substantial tokens. | 2 / 5 |
Actionability | Provides concrete, executable commands (npm install, membrane login, connection ensure, action run, request) with real flags and a useful popular-actions table, but uses placeholders (CONNECTION_ID, actionId) and a generic --input example, leaving minor gaps; not 5 because common-case examples rely on placeholders rather than fully copy-paste-ready invocations. | 4 / 5 |
Workflow Clarity | A clear sequence (install → authenticate → connect → search → run → proxy) with a readiness feedback loop (poll --wait, handle CLIENT_ACTION_REQUIRED, re-poll) and error-state guidance, though some validation checkpoints are implicit rather than explicit; not 5 because recovery steps are not framed as an explicit checklist. | 4 / 5 |
Progressive Disclosure | Single SKILL.md with well-organized headers and no nested references, but the giant inline entity list is content that arguably belongs in a separate reference file rather than the overview; not 5 because the oversized inline list prevents the overview from staying lean and navigable. | 4 / 5 |
Total | 14 / 20 Passed |