Content
60%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, domain-aware skill body with genuine validation discipline for risky database operations and clearly signaled one-level-deep references. Its weaknesses are verbosity from duplicated workflow/rule statements and abstract deliverable descriptions that defer all concrete how-to to resource files whose presence cannot be confirmed in this bundle.
Suggestions
Consolidate the three overlapping workflow schemes (Scenes, Structural Flow, Default Workflow) into a single canonical sequence to remove repetition and free tokens.
Deduplicate the vector-DB ownership rule and the anti-pattern §5 "existing usage" sentence, which each appear twice; state once and reference resources/anti-patterns.md §5.
Add one or two inline concrete artifacts (e.g. a sample expand-contract migration sequence or a minimal deliverable template snippet) so the guidance is executable without relying solely on unverified resource files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body does not teach concepts Claude already knows, but it is noticeably redundant: three overlapping workflow schemes (Scenes, Structural Flow, Default Workflow), the vector-DB-as-retrieval-infrastructure rule stated twice (Control-flow features and guardrail 12), and the anti-pattern §5 "existing usage" sentence repeated nearly verbatim in the workflow path and guardrail 18; the SSL-primitive table adds tokens without actionable value. | 3 / 5 |
Actionability | Some guidance is concrete (expand-contract steps, "measure, explain, nominate the dominant cost node, then optimize and re-measure"), but most of the body describes deliverables abstractly ("Produce external/conceptual/internal schema documentation") with no inline commands, sample DDL, or example artifacts; all how-to detail is deferred to resources/ files. | 3 / 5 |
Workflow Clarity | Sequencing is clear (Entry → PREPARE/ACQUIRE/REASON/ACT/VERIFY/FINALIZE → Exit, plus Explore/Design/Optimize) and validation checkpoints exist for destructive operations (VERIFY scene, pre-handoff checklist, expand-contract with soak window and separate deploy, "should not execute risky migrations without explicit user intent and verification"), so the destructive-ops cap does not apply; the three overlapping workflow schemes leave minor gaps that keep it below 5. | 4 / 5 |
Progressive Disclosure | A well-signaled References section lists one-level-deep pointers each labeled with its purpose ("Self-check (run before handoff)", "Live migration playbook", etc.) and the body appropriately stays at overview level per domain; however, the 18-guardrail policy block is heavy for the main file and none of the referenced resources/*.md or ../_shared files exist in the provided bundle, so navigation cannot be fully verified. | 4 / 5 |
Total | 14 / 20 Passed |