Content
82%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, actionable skill body that leans on a clean reference split and concrete commands. Main weaknesses are mild internal repetition and a drift between the body's 'unavailable command groups' claim and content inside build-kn-from-db.md.
Suggestions
Consolidate the jq projection rules and the unavailable-command-group (auth/config/agent/skill/toolbox/dataflow/context-loader) list so each appears once; cross-reference instead of repeating.
Reconcile references/build-kn-from-db.md with the body: it documents `ontology agent` and `context-loader` flows that the body says are unavailable — either remove those sections or note they require a deployment the body excludes.
Surface an inline validation/confirmation checkpoint for destructive/batch operations (e.g. `bkn delete`, `action-type execute`, `ds delete`) in the body rather than relying solely on references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Largely lean and reference-style with dense tables, but the jq projection rules and the unavailable-command-group list each recur 2-3 times across sections, which could be consolidated. | 4 / 5 |
Actionability | Provides concrete executable commands throughout — real `ontology` invocations, jq projections, and copy-paste-ready examples covering common cases like list/get/query/health. | 5 / 5 |
Workflow Clarity | The body routes to sequenced workflows with validation (e.g. build-kn-from-db has numbered steps plus a '验证' checkpoint) and offers a '先裸跑 jq keys/type' pre-check, but validation for destructive/batch ops is mostly delegated to references rather than surfaced inline. | 4 / 5 |
Progressive Disclosure | Clear overview with one-level-deep references signaled via a command-group table and inline links, well split across references/*.md; however references/build-kn-from-db.md documents `ontology agent`/`context-loader` that the body declares unavailable, a minor navigation inconsistency. | 4 / 5 |
Total | 17 / 20 Passed |