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.
An expert-oriented, highly actionable router skill with strong trap coverage and current-vs-deprecated API mapping, weakened by broken reference-link paths in its API tables and some duplicated/vestigial material. Fixing the link prefix and consolidating the reference index would lift it substantially.
Suggestions
Fix the API table links to include the references/ prefix (e.g. references/streaming-table/streaming-table-python.md → references/streaming-table-python.md); as written, dozens of table links resolve to non-existent paths.
Remove the vestigial '## Language-specific guides' section, which contains only one definitional sentence, and merge the duplicate ST-vs-MV aggregation guidance in 'Common Traps' into a single bullet.
Consider moving the six API comparison tables into a single reference file and having SKILL.md link to it, eliminating the duplicate reference index (tables vs. 'Pipeline API Reference' section).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and expert-facing — decision tree, traps, current-vs-deprecated API tables — assuming Claude already knows Spark and Databricks. Minor trimmable instances remain: the vestigial '## Language-specific guides' heading holds only one definitional sentence ("is a framework for building batch or streaming data pipelines") explaining a concept Claude doesn't need, and the ST-vs-MV aggregation guidance is restated across two adjacent Common Traps bullets. | 4 / 5 |
Actionability | Fully executable throughout: a copy-paste `databricks bundle init` command with a concrete --config-file payload, an exact `resources/*.job.yml` scheduling snippet, exact decorators/SQL for every feature (e.g. `@dp.append_flow(once=True)`, `CREATE OR REFRESH PRIVATE STREAMING TABLE`), and a four-step numbered workflow with literal commands. Specific examples cover the common cases. | 5 / 5 |
Workflow Clarity | The Development Workflow is a clear sequence (Validate → Deploy → Run → Check status) with a critical checkpoint ("You must deploy before running") and destructive-operation guarding (full refresh is "the most expensive and dangerous option, and can lead to data loss" and must be explicitly user-selected). Missing an explicit error-recovery loop (what to do when `bundle validate` fails), so it sits at anchor 4 rather than 5. | 4 / 5 |
Progressive Disclosure | Bundle structure is real and the final 'Pipeline API Reference' section links correctly one level deep into references/, but every link inside the six API comparison tables (e.g. `[streaming-table-python](streaming-table/streaming-table-python.md)`) omits the `references/` prefix and points to non-existent subdirectories — broken navigation for a large portion of the file. Those tables also duplicate the reference index inline rather than pointing to it, fitting anchor 3 ('references present but not clearly signaled; content that should be separate is inline') better than anchor 4. | 3 / 5 |
Total | 16 / 20 Passed |