Content
58%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 content is a dense, unusually rigorous operations manual with an excellent gated workflow (G1–G5, steps 1–7, validation feedback loops, output checklist), but it pays for that with heavy redundancy — the same three core rules are repeated four to five times across sections. Its bigger structural problem is that the whole detail layer is delegated to a reference.md that is missing from the bundle, so every deferred lookup fails.
Suggestions
Add the missing reference.md (MoeTracing templates, 打点密度与均匀性要求, 常见陷阱, 编译与打包门禁, point_map 契约) or inline the essentials — currently all 12+ links to it dead-end.
Deduplicate the G2 output-position rule, the xxx_profiling prohibition, and the G5 resolve() rule by stating each once (e.g., in the gate table) and cross-referencing from other sections; this could cut the body substantially without losing information.
Move the 探讨结论 maintenance-log table (e.g., MIX_AIC_1_2_SLOTS_PER_GROUP, MAX_INFER_GETBLOCKNUM_UB = 128) into reference.md or a CHANGELOG-style file — it is retrieval-oriented history, not execution guidance.
Include one concrete MoeTracing code snippet and one real sample-script CLI invocation in the body so the common cases are executable without the missing reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body restates the same core rules many times: the G2 output-position rule appears in the G1–G5 table, again in 默认交付标准, again in 全链路操作性定义, and again in step 7's checklist; 「禁止注册 xxx_profiling / 保持原算子名不变」 is repeated in at least five places (红线, 目标边界, 探讨结论表, 步骤 7 模式 B 与推荐); the G5 resolve() rule is stated three times. This is 'noticeably verbose; several unnecessary… padded sections' (2) rather than level 3's 'some… could be tightened', since entire sections duplicate one another wholesale, though the content is domain-specific rather than concepts Claude already knows. | 2 / 5 |
Actionability | Guidance is mostly executable: named scripts with real paths (scripts/validate_trace_points.py, check_compile_safety.py), a copy-paste command 「python3 <skill_root>/scripts/bootstrap_trace_toolchain.py --build-dir umdk/build/cam/comm_operator」, a concrete grep self-check 「grep -E 'MoeTracing|TRACE_POINT' .../op_kernel」, and per-layer checklists with exact file patterns (op_host/<op>.cpp, pybind/<op>.cpp). It falls short of 5 because there are no actual code templates (MoeTracing call form, sample CLI invocations) — those are delegated to reference.md, leaving minor gaps. | 4 / 5 |
Workflow Clarity | The 必须执行的流程 is a clearly numbered 1–7 sequence with explicit validation checkpoints and feedback loops: step 5 校验 with 「如果校验失败,修正问题后重新运行」, a mandatory full-compile gate, the G1–G5 gate table, and a mandatory 输出约定 checklist requiring per-gate satisfied/unsatisfied declaration. This matches the 'clear sequence with explicit validation steps; feedback loops for error recovery; checklists' anchor. | 5 / 5 |
Progressive Disclosure | The body is structured around one-level-deep references, but the primary target [reference.md](reference.md) — linked ~12 times for MoeTracing templates, 打点密度要求, 常见陷阱, 编译与打包门禁, and point_map 契约 — does not exist in the bundle (only scripts/ is present). Per the guideline to score against the actual bundle structure, navigation dead-ends for all deferred detail, which is at least as bad as 'references are buried' (2); it is not 3, because the split that the structure promises is not actually realized for the bulk of the detailed material. | 2 / 5 |
Total | 13 / 20 Passed |