Content
75%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 content is a well-structured, mostly executable guide to driving Metabase through the Membrane CLI, with a strong connection-state workflow. Its main weaknesses are a padded intro, a broken empty overview section, dangling step references, and the absence of any Metabase-specific action example or output verification.
Suggestions
Remove or fill the empty 'Metabase Overview' section (the bare Question/Result/Dashboard bullets and 'Use action names and parameters as needed.' add noise) and trim the opening paragraph that explains what Metabase is.
Fix the step numbering: replace the dangling 'skip to Step 2' and lone '1b' with a consistent labeled sequence so the connection flow is unambiguous.
Add one concrete Metabase action example (e.g., listing questions or fetching a dashboard result) with its expected output, plus a verification note after running actions that mutate data.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean executable CLI commands with a flag table, but it pads a few spots Claude likely already knows (the opening 'Metabase is a business intelligence...' paragraph) and includes a broken, near-empty 'Metabase Overview' section, fitting 'efficient; minor instances of over-explanation that could be trimmed' rather than the spotless 5. | 4 / 5 |
Actionability | It provides copy-paste-ready commands for install, login, connection, action search/run, and proxy with a concrete flag table, but the examples are generic Membrane CLI patterns with placeholders (CONNECTION_ID, --input '{"key": "value"}') and no Metabase-specific action example, matching 'mostly executable guidance; minor gaps'. | 4 / 5 |
Workflow Clarity | The connection flow has a clear sequence with explicit state checkpoints and a feedback loop (poll --wait, handle READY/BUILDING/CLIENT_ACTION_REQUIRED, re-poll), but it references a non-existent 'Step 2' and '1b' without a 1a, and running actions lacks any result verification, fitting 'clear sequence with most checkpoints present; minor validation gaps'. | 4 / 5 |
Progressive Disclosure | With no bundle files, the single ~154-line SKILL.md is organized into clear sections (Install, Authentication, Connecting, Searching, Running, Proxy, Best practices) that are easy to navigate, matching 'good structure; most content appropriately placed; minor organization gaps'; the empty Overview section and thin 'Popular actions' pointer are the minor gaps. | 4 / 5 |
Total | 16 / 20 Passed |