Content
78%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: executable CLI commands, explicit decision gates with user confirmation, phase-based reading routing to real one-level-deep reference files, and validation checkpoints. The main weakness is token efficiency — repeated rules, a duplicated disambiguation paragraph, and unquarantined version numbers.
Suggestions
State the app-name rule (lowercase/hyphens, ≤26 chars) once in Generic Guidelines and drop the repeats in the scaffolding section and Directory Naming section.
Trim the "Is this even a Databricks App?" paragraph, which restates the description's aibi-dashboards disambiguation at length; one or two sentences suffice.
Consolidate version requirements (CLI >= v0.294.0, >= v1.20.0 for auth-mode) into the frontmatter compatibility field or a single compatibility note instead of scattering them inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense and actionable and assumes framework knowledge ("LLMs already know these frameworks — the challenge is Databricks platform integration"), but it could be tightened: the dashboard-vs-app disambiguation repeats the description at length, the ≤26-char app-name rule is stated three times (Generic Guidelines, scaffolding section, Directory Naming), and raw version requirements (CLI >= v0.294.0, >= v1.20.0) sit outside any compatibility/deprecated section. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout: `databricks apps manifest --profile <PROFILE>`, `databricks apps init --name <NAME> --features <plugin1> --set <plugin1>.<resourceKey>.<field>=<value>`, `npx @databricks/appkit docs`, `databricks apps validate --profile <PROFILE>`, plus wrong/right scaffolding examples and a concrete routing table — matches the anchor for fully executable guidance covering common cases. | 5 / 5 |
Workflow Clarity | "Development Workflow (FOLLOW THIS ORDER)" gives clear sequencing with validation checkpoints (verify typegen ✓, update smoke selectors BEFORE validation, `apps validate` before deploy, post-deploy `apps get` status check) and explicit decision gates, but there is no explicit fix-and-revalidate error-recovery loop, so it fits anchor 4 rather than 5. | 4 / 5 |
Progressive Disclosure | The "Required Reading by Phase" table routes each task to a specific one-level-deep reference (e.g. [SQL Queries Guide](references/appkit/sql-queries.md), [Platform Guide](references/platform-guide.md)), all of which exist in the bundle, while SKILL.md stays an overview with routing and decision gates — matching the anchor for a clear overview with well-signaled one-level-deep references. | 5 / 5 |
Total | 17 / 20 Passed |