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.
A well-organized, concise workflow-oriented skill body with concrete best practices and good example invocations. Weaknesses are the missing validation step for generated SQL, the dangling reference to an external `sql-queries` skill in place of a real bundle reference, and mild redundancy between the Tips section and the workflow.
Suggestions
Add a validation checkpoint after writing the query — e.g., 'If a warehouse is connected, dry-run the query (or EXPLAIN it) and fix errors before presenting' — which would also address the deferred-correctness gap in step 6.
Replace '(see `sql-queries` skill for details)' with an actual reference path in this skill's bundle (e.g., references/dialects.md) or inline the key dialect differences, so dialect-specific guidance is verifiable and one level deep.
Trim the Tips section or fold it into the workflow (it repeats step 2's dialect guidance), and drop the cloud-provider parentheticals from the dialect list to save tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean bullet-style guidance that assumes Claude's competence ("Never use `SELECT *`", "Prefer `EXISTS` over `IN`") without explaining what SQL or a CTE is. Not 5 because the 10-item dialect list with cloud-provider parentheticals and the Tips section partially restate workflow guidance ("Mention your SQL dialect upfront" duplicates step 2), adding tokens that could be trimmed. Not 3 because padding is minor and there is no concept-teaching. | 4 / 5 |
Actionability | Concrete, executable guidance throughout: a 6-step workflow, specific best-practice rules, and three realistic example invocations ("Count of orders by status for the last 30 days", cohort retention, 500M-row partitioned table). Not 5 because dialect-specific syntax — a core promise — is deferred without a path ("see `sql-queries` skill for details", which is not a bundle file), and no illustrative SQL snippet grounds the expected output format. Not 3 because the guidance is specific and directly executable, not high-level hints. | 4 / 5 |
Workflow Clarity | A clear, well-sequenced 6-step process (understand request → determine dialect → discover schema → write → present → offer to execute) with a user-interaction checkpoint ("ask which they use") and an execution offer. Not 5 because there is no validation checkpoint on the produced SQL (e.g., dry-run/EXPLAIN against a connected warehouse, or re-reading the request against output columns) — validation is only implicit. Not 3 because the sequence is complete and explicit; the skill is also non-destructive, so the batch/destruction cap does not apply. | 4 / 5 |
Progressive Disclosure | Good structure with clear section headers (Usage, Workflow, Examples, Tips) and appropriate inline length (~120 lines of overview-level guidance, no monolithic reference dump). Not 5 because no bundle files exist: dialect-specific detail is deferred to an unverifiable external `sql-queries` skill and a `../../CONNECTORS.md` path outside the bundle, rather than well-signaled one-level-deep references. Not 3 because the inline content is appropriately overview-level and navigation is easy. | 4 / 5 |
Total | 16 / 20 Passed |