Content
63%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.
The body is well-structured and highly actionable, with concrete commands, snippets, and a clear staged workflow. Its main weaknesses are duplicated content (validator commands and the subclassing prohibition each appear twice) and a large set of referenced bundle files (stages/, checklists/, subagent-prompts/, artifacts/templates.md) that are missing, which undermines both conciseness and navigation.
Suggestions
Deduplicate: keep the validator commands in one place (the Validation section) and state the direct-inheritance rule once, linking from Common Mistakes instead of restating it
Ship the missing bundle files (stages/*.md, checklists/compatibility.md, subagent-prompts/*.md, artifacts/templates.md) or inline their essential content — the workflow currently dead-ends at nine dangling references
Add an explicit validate-fail → fix → re-validate loop inline (e.g., "If the validator reports errors, fix them and re-run before registering") so error recovery does not depend on the missing stages/validate.md
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and largely avoids teaching known concepts, but it duplicates content: the validator commands appear verbatim in both the Quick Reference table and the Validation section, and the "never subclass an existing agent" prohibition is stated at length in both Base Class and the Common Mistakes table. Not 4 because these are more than minor trims — whole repeated blocks could be deleted; not 2 because there is no padding or explanation of concepts Claude already knows. | 3 / 5 |
Actionability | Guidance is mostly executable: concrete validator commands ($PYTHON_PATH .agentsociety/bin/ags.py create-agent --file ...), a copy-paste extra_skill_paths JSON snippet, a full SKILL.md frontmatter example, named required methods, and a mistakes-to-fixes table. Not 5 because there is no inline agent-class code template — the pointer to artifacts/templates.md is the only source of scaffolding, and that file is absent from the bundle. | 4 / 5 |
Workflow Clarity | The dot-graph pipeline (intake → design → generate → validate → register), stage notes, and the explicit final validator step give a clear sequence with a concrete checkpoint. Not 5 because the error-recovery loop is not explicit in the body — what to do when the validator fails is deferred to stages/validate.md and checklists/compatibility.md, neither of which is present in the bundle. | 4 / 5 |
Progressive Disclosure | Sections are well organized and the five references/*.md citations resolve to real, one-level-deep files. However, roughly nine referenced paths do not exist in the bundle: stages/intake.md, stages/design.md, stages/generate.md, stages/validate.md, checklists/compatibility.md, subagent-prompts/planner.md, subagent-prompts/implementer.md, subagent-prompts/reviewer.md, and artifacts/templates.md. Not 4 because navigation genuinely breaks at those dangling references; not 2 because the structure that exists is good and the present references are clearly signaled. | 3 / 5 |
Total | 14 / 20 Passed |