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.
A highly actionable, well-sequenced scout skill with executable SQL and explicit validation gates; its main weakness is verbosity from restating the same trusted-source and header-corroboration rules across sections.
Suggestions
Consolidate the trusted-source / corroborated-header rules into one canonical block in Decide and reference it from Explore and Reconstruct policy instead of restating each time.
Trim editorializing ('Internalize that shape', 'This is the scout's biggest blind spot') and fold redundant push-path explanations into the single 'How this relates to the push-based CSP source' section.
Consider extracting the four detection SQL queries into a `references/queries.md` (own bundle) so the main SKILL.md reads as an overview, strengthening progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly specialized CSP-specific knowledge Claude would not already know (count/users shape table, nonce-normalization rationale, standing-block blind spot), but the Decide section restates the trusted-source / corroborated-header requirement several times and editorializing like 'Internalize that shape' could be trimmed. | 3 / 5 |
Actionability | Provides four copy-paste-ready ClickHouse SQL queries (JSONExtractString, domain(), uniq(), argMax(), groupArray) plus executable `gh search code` / `gh api ... | base64 -d` commands and concrete scratchpad key formats, covering the main detection cases. | 5 / 5 |
Workflow Clarity | Clear run sequence (Get oriented -> Profile shape -> Explore -> Save memory -> Decide -> Close out -> When to stop) with explicit validation gates (corroborate header against code, check in-flight PRs, own-surface check, push-path dedupe) and feedback loops (the code-vs-reports reconcile table). | 5 / 5 |
Progressive Disclosure | Well-signaled one-level-deep deferral to `authoring-scouts` -> `references/report-contract.md` and code paths (`posthog/api/csp.py`, `posthog/tasks/csp_signal.py`) with clear section navigation, but no own bundle files exist and the lengthy SQL/Decide content is fully inlined rather than split into this skill's own references. | 4 / 5 |
Total | 17 / 20 Passed |