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 tight, information-dense diagnostic skill that respects the token budget and gives a concrete routing workflow with real domain knowledge. The gaps are minor: no example search commands for the correlate-by-ID step, and a non-relative path to the bundled extractor script.
Suggestions
Add one concrete search example for the core correlation step, e.g. `grep '<turnId>' events.jsonl` or a jq filter over ahp/*.jsonl using _ahpLog.ts, so the 'search by the known time or ID' instruction is copy-paste ready.
Reference the bundled extractor relative to the skill directory ("python3 scripts/extract.py \"<archive>.zip\"") so the command works regardless of the current working directory.
Fold the causality check into the workflow as an explicit validation step (e.g., step 5: 'Confirm any error you find has a matching failed response, missing completion, or user-visible consequence before treating it as the cause').
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence: it never explains what a log or timezone is, and every line delivers non-obvious domain knowledge (rotation naming ".1.jsonl", "_ahpLog.truncated: true", UTC vs local time, ephemeral vs persisted events). This matches anchor 5 (every token earns its place) rather than anchor 4, which requires identifiable over-explanation to trim. | 5 / 5 |
Actionability | Concrete, executable guidance dominates: a copy-paste extractor command ("python3 .github/skills/agent-host-logs/scripts/extract.py \"<archive>.zip\""), a symptom-to-file routing list, and named correlation fields (turnId, serverSeq, _ahpLog.ts). It falls short of anchor 5 only because the central analysis move — "Search by the known time or ID, then follow the same operation into the adjacent layer" — has no concrete command example (e.g., a grep/jq pattern against events.jsonl). | 4 / 5 |
Workflow Clarity | "How to Start" gives a clear numbered sequence (list files, identify symptom, pick the closest file, search by time/ID and follow into the adjacent layer) with a decision routing list — matching anchor 4's clear sequence with most checkpoints present. It is not anchor 5 because the verification guidance ("Warning or error severity alone does not prove causality. Look for the matching failed response...") is a tip rather than an explicit validation step inside the workflow; the read-only, non-batch nature of the task keeps it above anchor 3. | 4 / 5 |
Progressive Disclosure | The ~80-line body is well organized into purposeful sections with the one bundled script (scripts/extract.py, verified to exist) referenced inline and a best-effort file inventory — good structure with content appropriately placed (anchor 4). It misses anchor 5 because the script is referenced by its repo-root path ".github/skills/agent-host-logs/scripts/extract.py" rather than a bundle-relative "scripts/extract.py", leaving navigation dependent on the working directory. | 4 / 5 |
Total | 17 / 20 Passed |