Content
81%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 high-quality, actionable CI triage skill with concrete commands, explicit validation/feedback loops, and a clear escalation protocol. Its main weakness is repetition of the same directives across several sections, which could be consolidated to improve token efficiency.
Suggestions
Consolidate the repeated directives ("do not hand-edit generated files", "check remote-testing rules", "do not answer from stale CI state") into a single stated-once rules block referenced where needed, to remove 3-5 restatements.
Consider moving the detailed mermaid dependency graph and Key Chains into a separate reference file linked from the Dependency Order section, since they are reference material rather than triage steps.
Tighten the Whack-a-Mole Prevention and Escalation Protocol sections, which overlap heavily on the two-similar-sync cutoff rule.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is substantive and project-specific (no generic concept explanations), but several directives are restated 3-5 times — "do not hand-edit generated files", "check remote-testing rules", "do not answer from stale CI state", and the two-similar-sync escalation cutoff appear across multiple sections — so it is mostly efficient but could be tightened. | 3 / 5 |
Actionability | Concrete copy-paste commands cover the common cases — `gh run rerun --failed`, `pbpaste | git apply`, `./frb_internal precommit-generate`, `git reset --hard HEAD && git clean -fdx`, `./frb_internal lint --fix` — with the Quick Reference table mapping symptoms to exact fixes. | 5 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced (Triage Order 1-5, the 7-step Escalation Protocol) with explicit validation checkpoints and feedback loops — rerun-then-check-flaky, clean-then-reproduce, accept generated outputs only after confirming no new regressions — plus a Common Mistakes checklist. | 5 / 5 |
Progressive Disclosure | The single file is well-organized with clear section headers and well-signaled one-level-deep pointers to sibling skills (frb-fix-merge-conflict, frb-ci-filter, frb-code-generation, frb-debugging), but at ~320 lines everything is inlined with no bundle files to split into, leaving minor organization gaps versus an ideal split. | 4 / 5 |
Total | 17 / 20 Passed |