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-structured, highly actionable skill body with a strong decision framework, real executable commands, and clearly signaled one-level-deep references that all exist. The main weaknesses are inline duplication of the serverless reference content, a muddled Databricks Connect quick-start snippet, and missing validation around irreversible delete operations.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational — decision matrix, gotchas ("'client': '4' is required... '1' silently ignores them", task run_id vs parent run_id), and copy-paste commands with almost no explanation of concepts Claude already knows. It is not a 5 because the Serverless Job section duplicates nearly all of references/2-serverless-job.md inline (same four steps, commands, and gotchas), and the mode-preference guidance is stated three times (decision matrix, decision flow, prose). | 4 / 5 |
Actionability | Mostly executable, copy-paste-ready guidance: full `databricks workspace import`/`jobs submit` commands with complete JSON bodies, jq trimming commands, and warehouse create/edit JSON. It falls short of 5 because the Databricks Connect example in the body is muddled — Python code inside a ```bash block with a `...` ellipsis placeholder — leaving the preferred mode's quick-start not fully executable from the body alone. | 4 / 5 |
Workflow Clarity | The serverless flow is clearly sequenced (upload → submit → poll → fetch output) with explicit validation checkpoints: "Only read .state.result_state... once life-cycle is TERMINATED", and the decision flow covers failure branches ("Install fails? → Ask user (don't auto-switch modes)"). Minor gaps keep it at 4 rather than 5: no explicit recovery loop when a run FAILED (fetch error is described, but not what to do next), and irreversible operations (warehouses delete, clusters permanent-delete) have no verification/confirmation step. | 4 / 5 |
Progressive Disclosure | Good structure: a decision matrix and per-mode sections that each link to a real one-level-deep reference file (all three references/ files exist), with clear signaling ("Read the reference file for your chosen mode before proceeding") and well-organized references containing 'Use when' / 'When NOT to Use' guidance. It is not a 5 because the body's serverless section substantially duplicates its reference file's content inline, and the ~40-line SQL Warehouses block lives entirely in SKILL.md with no reference file of its own. | 4 / 5 |
Total | 16 / 20 Passed |