Content
71%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.
The body is a well-structured orchestration playbook with explicit sequenced workflows, validation checkpoints, executable CLI templates, and clear one-level-deep references to sibling skills. Its main weakness is repetition: several governance rules (no direct CLI, no cross-fallback) are restated many times, adding tokens without new information.
Suggestions
Consolidate the repeated 'Never execute ontology CLI directly — delegate to ontology-core' rule to a single authoritative statement; the 5+ restatements across sections are redundant token cost.
State the '禁止交叉兜底' rule once in its dedicated section and reference it by name elsewhere rather than re-explaining the full rationale in the 总入口原则 list.
Move the detailed JSON condition-wrapper example and the SQL-vs-query-vs-properties comparison table toward a referenced reference file (e.g. ontology-core/bkn.md) so the SKILL.md body stays a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~270-line body is densely packed with routing rules, tables, and checklists that are largely necessary, but it repeats the same constraints many times (the 'Never execute ontology CLI directly / delegate to ontology-core' rule is restated 5+ times, and the no-cross-fallback rule is stated then re-stated in a dedicated section), which is noticeable padding for an orchestrator skill. | 3 / 5 |
Actionability | It provides concrete, executable guidance — explicit CLI commands (bkn object-type query/get, dataview query --sql), a correct JSON body template with the required 'condition' wrapper, an explicit wrong-example error message, and concrete handoff field tables — with only minor gaps (no full copy-paste SQL example, ot-id selection left to LLM judgment). | 4 / 5 |
Workflow Clarity | Multi-step processes are explicitly sequenced with copy-and-tick checklists for both 找表 and 问数 branches, a top-level 5-step orchestration flow, explicit validation/feedback checkpoints (占位未填 → 告知用户并停止; schema discovery before SQL; re-clarify on ambiguity), and explicit error-recovery guidance for when a branch fails. | 5 / 5 |
Progressive Disclosure | Structure is good with clear section headers and one-level-deep references to sibling skills ([smart-search-tables/SKILL.md], [smart-ask-data/SKILL.md], [ontology-core/SKILL.md]) and to ontology-core's references/*.md (尤其 bkn.md); no references/ scripts/ assets/ bundle files exist in this skill's own directory, so there is no deep nesting, though some reference-target detail is inlined that could live in those referenced files. | 4 / 5 |
Total | 16 / 20 Passed |