Content
82%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 triage skill with copy-paste commands and canned queries, clear sequencing, and sensible validation guidance. Minor conciseness trims and slightly stronger access-failure recovery would push it higher.
Suggestions
Trim the Chrome Safe Storage/browser_cookie3 aside and the 'real incident' anecdote, or move them to a short footnote, to tighten conciseness toward a 5.
Add a brief error-recovery note for the access path (e.g., what to do if `hogli metabase:login` fails or the database id is not found) to strengthen the workflow's feedback loop.
Consider extracting the canned SQL queries into a references/ file with a one-line pointer from SKILL.md, which would improve progressive_disclosure and reduce token load when only discovery is needed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense and domain-specific with little padding of concepts Claude already knows, but a few asides could be trimmed (the Chrome Safe Storage/browser_cookie3 explanation and the 'a real incident: the top-cost actor was invisible' anecdote), keeping it just short of fully lean. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready guidance: concrete `hogli metabase:login/query` commands with --region flags and a database-discovery command, plus canned SQL queries covering the top-actors, partition-scoped, merge-storm, and generic top-10 cases. | 5 / 5 |
Workflow Clarity | Clear numbered login -> discover database id -> run queries sequence with validation checkpoints ('Always start with discovery' and the partition-dimension check query); read-only triage so the destructive-cap does not apply, but error-recovery feedback loops for access failures are only implicit. | 4 / 5 |
Progressive Disclosure | Well-organized into clearly headed sections (Access, Schema, Metric inventory, Lens, Canned queries, Cautions, Related) with clear sibling-skill pointers; no bundle files exist to split further, and at ~200 lines it does not qualify for the simple-skill exception to 5. | 4 / 5 |
Total | 17 / 20 Passed |