Content
67%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 is action-dense with executable CLI commands and a logical, checkpointed connection workflow, but it is padded with conceptual explanation and overview filler, uses inconsistent step numbering, and keeps all content inline rather than splitting reference material into separate files.
Suggestions
Cut conceptual filler and the 'MindsDB Overview' bullet list, leaving only what Claude needs to execute; move the MindsDB product description to a single line or remove it.
Normalize the workflow into explicit numbered steps (1. Install, 2. Authenticate, 3. Connect, 4. Wait for READY, 5. Discover/run actions) so the 'skip to Step 2' references resolve, and add a verify/retry checkpoint after running actions.
Move the proxy flag table and detailed connection-state handling into a reference file (e.g., references/PROXY.md) and link to it, keeping SKILL.md a concise overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is concrete CLI commands, but it pads with explanations Claude largely doesn't need ('MindsDB is an open-source AI layer... making AI accessible to data stored in SQL databases', 'so you can focus on the integration logic rather than auth plumbing') and a low-value 'MindsDB Overview' bullet list, matching the 'mostly efficient but includes some unnecessary explanation' anchor. | 3 / 5 |
Actionability | It provides copy-paste-ready commands for install, login, connection ensure, polling, action list/run, and proxy requests plus a flag table, but the examples are generic Membrane-CLI workflow steps rather than MindsDB-specific cases, leaving minor gaps versus the fully-covered level. | 4 / 5 |
Workflow Clarity | A clear install → authenticate → connect → wait-for-ready → search → run → proxy sequence is present with state-based checkpoints (poll until not BUILDING, handle CLIENT_ACTION_REQUIRED), but the step numbering is inconsistent ('skip to Step 2' and '1b' without labeled Step 1/2 headers) and there is no explicit verify/retry loop for action-run outcomes, keeping it at 'clear sequence with minor validation gaps'. | 4 / 5 |
Progressive Disclosure | The body is well-organized into clear section headers with no buried or nested references and no bundle files, but at ~140 lines it exceeds the simple-skill threshold and inlines bulk (the proxy flag table, detailed connection-state handling) that could live in separate reference files, fitting 'good structure; minor organization gaps'. | 4 / 5 |
Total | 15 / 20 Passed |