Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a dense, highly actionable operator manual with strong workflow sequencing and validation, though slightly repetitive. Progressive disclosure is well designed but its referenced resource files are absent from the bundle, so the navigation cannot be completed.
Suggestions
Deduplicate the calibration and agent-choice guidance: state the --limit 50 --concurrency 5 recommendation and the codex-vs-claude trade-off once and cross-reference them elsewhere instead of restating in Scenes, Transitions, Canonical workflow, and Guardrails.
Add the referenced resources/ files (setup.md, scanning.md, pr-review.md, matchers.md, triage.md, config.md) and the ../_shared/* files to the bundle so the progressive-disclosure navigation is actually reachable, or inline the essential parts of the missing files into SKILL.md.
Tighten the Scheduling section's Intent signature / When to use / When NOT to use lists, which overlap with the description and the Scenes list, to reduce token cost without losing trigger coverage.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and dense with deepsec-specific knowledge Claude would not know (cost bands, resume semantics, two-job CI pattern), but recurring repetition (e.g. the --limit 50 --concurrency 5 calibration advice restated ~4 times, the agent-choice trade-off in both Scenes and Guardrails) could be tightened; not score 3 because some tokens do not earn their place twice, not score 1 because it does not explain generic concepts or pad. | 2 / 3 |
Actionability | Provides fully executable bash blocks (e.g. "bunx deepsec init", "bunx deepsec process --diff origin/${BASE_REF} --comment-out comment.md"), specific flags, file paths, and quantified matcher hit-rate targets (1-20 hits / 1k files for precise); not score 2 because the guidance is copy-paste ready rather than pseudocode. | 3 / 3 |
Workflow Clarity | Clear sequenced workflow (Bootstrap -> Calibrate -> Full investigation -> PR mode -> Matchers -> Resume) with explicit validation checkpoints (deepsec status, RunMeta, exit codes 0/1/other) and feedback loops for quota, refusal, and high-FP recovery; not score 2 because checkpoints are explicit, not implicit. | 3 / 3 |
Progressive Disclosure | The body is an overview with one-level-deep, clearly signaled references to resources/*.md and ../_shared/* files, but none of those referenced files (nor references/, scripts/, assets/ directories) are present in the actual bundle, so the offloaded detail cannot be navigated or verified; not score 3 because the promised detail files are missing, not score 1 because references are one level deep and well organized rather than deeply nested. | 2 / 3 |
Total | 10 / 12 Passed |