Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
Highly actionable with complete executable code and a clear sequenced workflow including validation and rollback, but it is verbose and fails to leverage its reference file: implementation.md is neither linked from the body nor used to offload the duplicated detailed code.
Suggestions
Link references/implementation.md from the body (e.g. a '## Implementation details' section pointing to it) and move the duplicated detailed code there, keeping SKILL.md as a lean overview.
Add an explicit validate->fix->retry loop to the batch migration path (when migrate_table reports MISMATCH, diagnose via the error-handling table and re-run) to satisfy the feedback-loop expectation for destructive/batch operations.
Tighten the six code blocks to the essential, copy-paste-ready snippets and drop the duplicated discovery/schema/cutover implementations already present in the reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly executable code that earns its place with little concept-explanation padding, but at ~340 lines across six full code blocks it is verbose, and much of the content duplicates references/implementation.md rather than being kept lean. | 2 / 3 |
Actionability | Every step ships complete, copy-paste-ready Python/SQL (e.g. assess_hive_metastore, migrate_table, the Auto Loader streaming block) with concrete Spark SQL and DDL rather than pseudocode. | 3 / 3 |
Workflow Clarity | The six steps are clearly sequenced and validation exists (row-count checks, cutover rollback, error-handling table), but the batch/destructive migration only records OK/MISMATCH without an explicit validate->fix->retry feedback loop, which caps this dimension at 2 per the rubric. | 2 / 3 |
Progressive Disclosure | A bundle reference (references/implementation.md) exists but is never signaled or linked from the body, and the body inlines full code for all six steps that overlaps that reference — content that should be split out is inline and the reference path is not clearly navigated. | 2 / 3 |
Total | 9 / 12 Passed |