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 spec: an ordered step checklist, concrete command templates, strict result-integrity rules, and cleanly split one-level-deep references. Its weaknesses are redundant restatement of the delegation/gateway/token contract across sections and the absence of an explicit error-recovery feedback loop.
Suggestions
State the ontology-core delegation, gateway/base-url, and no-token contract once in a single 委托契约 section and reference it elsewhere instead of restating it in 调用方式, 子技能依赖, 与 smart-data-analysis 的关系, and 配置 — this trims the conciseness duplication.
Add an explicit error-recovery step for failed dataview queries (e.g., on schema/SQL error: re-run schema discovery, regenerate SQL, retry once) to turn the implicit schema-first validation into a closed validate→fix→retry loop and lift workflow_clarity.
Provide at least one fully-resolved example command (with a concrete kn-id / dataview-id / accountId value) in tool-examples.md so the command templates read as copy-paste ready rather than placeholder-only.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is table- and checklist-driven with no over-explanation of known concepts, but the ontology-core delegation contract, the gateway/base-url note, and the no-token note are each restated across 4–5 sections (调用方式, 子技能依赖, 与 smart-data-analysis 的关系, 配置), which is unnecessary duplication that could be tightened — matching the 3 'mostly efficient but could be tightened' anchor rather than the 4 'minor instances'. | 3 / 5 |
Actionability | Concrete command templates are given — 'bkn get <kn-id>', 'bkn object-type list/get', 'dataview query <dataview-id> --sql "…"' — with key options and ordering, but they use placeholders and the real execution entry is intentionally omitted ('不出现真实执行入口'), so they are mostly-but-not-fully executable, fitting the 4 anchor. | 4 / 5 |
Workflow Clarity | A clearly ordered 5-step 主流程 checklist plus per-step 约束 (schema-first to prevent SQL hallucination, SELECT/WITH-only enforcement, empty-result handling) provide most checkpoints, but no explicit validate→fix→retry feedback loop for failed queries keeps it at 4 rather than 5. | 4 / 5 |
Progressive Disclosure | The body is a clear overview that points to a '必读 references(按步骤)' table linking four one-level-deep reference files (kn-resolve.md, schema-discovery.md, sql-execute.md, tool-examples.md), all of which exist in ./references/, with links clearly signaled and organized by step — matching the 5 anchor. | 5 / 5 |
Total | 16 / 20 Passed |