Content
78%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 highly actionable, information-dense skill body with runnable TypeScript for every workflow step and a genuinely valuable catalog of backend pitfalls. Its weaknesses are systematic repetition (the typed-rows point, the duplicated Quick Example) and — most seriously — a Rules section that points at nine rule files that do not exist in the bundle, breaking the skill's own progressive-disclosure structure.
Suggestions
Ship the rules/ directory (workflow-find-errors, workflow-get-context, workflow-correlate-logs, workflow-check-metrics, workflow-identify-cause, pattern-http-errors, pattern-slow-requests, pattern-distributed, pattern-log-driven) or remove the 'Read rules/<rule-name>.md' pointers — nine referenced files are missing from the bundle.
Delete or differentiate the Quick Example section: it repeats the RCA Workflow step-1 query almost verbatim, adding only the avg/max Duration measures that step 4 already shows.
State the typed-rows guarantee once (e.g., in Code mode) and drop the repeats in the code comment, the aggregate explanation, and Tip 3.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and expert-level — the gotchas list contains only non-obvious backend knowledge and it never explains concepts Claude already knows. However, the typed-rows guarantee is stated four times ('fully typed — rows autocomplete and typos are compile errors', 'Result rows are fully typed — iterate data directly and never cast', the '// typed rows' comment, and Tip 3), and the Quick Example duplicates the Workflow step-1 query almost verbatim. This is minor over-explanation that could be trimmed — above anchor 3's midpoint, but not anchor 5's 'every token earns its place'. | 4 / 5 |
Actionability | Fully executable throughout: a connect snippet with config-override args, the run command 'npx tsx rca.mts' with a '.mts' rationale and a 'node --experimental-strip-types' fallback, complete built queries for every workflow step, a copy-paste error-handling pattern for KopaiQueryBuildError with e.issues, and concrete CLI commands. Specific examples cover the common cases (error triage, trace context, log correlation, metric time-series). | 5 / 5 |
Workflow Clarity | The RCA Workflow is a clearly sequenced 5-step process (find failing work → get trace context → correlate logs → quantify impact → present findings), each with runnable code and pointers to detail rules. Validation exists — the 'empty result on a busy system ≈ a wrong column/value' re-check, the retention-floor caution, and build-error try/catch — but these checkpoints live in the gotchas section rather than being embedded as explicit workflow steps, which fits anchor 4's 'most checkpoints present; minor validation gaps' rather than anchor 5's explicit in-workflow validation with feedback loops. | 4 / 5 |
Progressive Disclosure | Section structure is good and the three references/*.md files exist and are one level deep, but 9 of the 12 referenced paths are dangling: 'Read rules/<rule-name>.md for details' and every 'See workflow-find-errors' / 'pattern-http-errors' pointer targets a rules/ directory that is absent from the bundle. Navigation to the promised detail material is impossible, which falls below anchor 4's 'references mostly clear' despite the otherwise good organization. | 3 / 5 |
Total | 16 / 20 Passed |