Content
96%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 dense, fully actionable internal debugging guide: executable commands, per-input matching semantics, ordered diagnostic paths, retention limits, and production-safety guardrails, with essentially no wasted tokens. The only structural gap is that the script the entire skill depends on is not bundled alongside SKILL.md.
Suggestions
Bundle `scripts/debug-runs.ts` with the skill (or state its location relative to the skill directory) — every command in the body invokes it, but no scripts/ directory exists in the bundle, so the reference does not resolve.
Make the `sentry-cli` cross-reference concrete (one line on how to invoke it for the provider ERROR ID lookup) so the Sentry diagnosis path is fully navigable without guessing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body explains nothing Claude already knows; every paragraph carries non-obvious internal specifics ("Design's table alone is over 1 GB, and an unfiltered scan times out against production", "Their conversation lives in that client; only the calls reached us"). Lean throughout with no trimmable over-explanation. | 5 / 5 |
Actionability | Two copy-paste commands with flags ("pnpm exec tsx scripts/debug-runs.ts find <input> [--app NAME] [--days 30] [--limit 20]"), an input-matching table with concrete examples for every input type, and ordered diagnostic steps ("Start at the failed run's `error`, then the last tool call before it"). Fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | Clear numbered sequence (Find the runs → Read a thread → Diagnose) with explicit error-recovery loops: "`auditError`: the audit scan timed out; rerun with a smaller `--days`", the error-entry disambiguation ("it was not searched, which is not the same as 'nothing found'"), and the fallback "Threads created before May 2026 have no run ids, so `runs[]` is empty and only the transcript is available". The workflow is read-only, so the destructive/batch validation cap does not apply. | 5 / 5 |
Progressive Disclosure | A single well-organized file with clear sections (Find / Read a thread / Diagnose / Retention / Rules) and nothing inlined that belongs in a separate file, but its sole external reference — `scripts/debug-runs.ts`, invoked by every command — is not present in the bundle (no scripts/ directory exists), so the reference does not resolve. Not 5 because a clearly signaled one-level-deep reference is broken relative to the provided bundle; not 3 because structure and signaling are otherwise good. | 4 / 5 |
Total | 19 / 20 Passed |