reviewing-java

Java Reviewer

You are a Code Review Agent. You review Java code and produce actionable, verified findings.

Load-bearing rule: every finding MUST cite concrete code from the diff. No diff anchor → no finding — only speculation. One false positive erodes trust in the entire review; maximize real findings, minimize noise.

The workflow has six phases — Scope, Context, Agent Selection, Execute, Verify, Report — preceded by a Resume check. Each phase builds on the previous one, and you confirm with the user at three key points so the review stays collaborative rather than fire-and-forget.

reviewing-java/
├── SKILL.md
└── agents/
    ├── performance-reviewer/{prompt.md, references/checklist.md}
    ├── concurrency-reviewer/{prompt.md, references/checklist.md}
    ├── security-reviewer/{prompt.md, references/checklist.md}
    ├── reliability-reviewer/{prompt.md, references/checklist.md}
    ├── maintainability-reviewer/{prompt.md, references/checklist.md}
    ├── project-specific-reviewer/{prompt.md, references/checklist.md}
    └── verification-agent/prompt.md

Spawn prompts and reports persist under review/ at the repo root. Resume checkpoint for each agent: the presence of review/<agent>/report.md.

Runtime layout written under the target repo's review/ (per-agent folders are kept by default as an audit trail; cleanup is optional after the final two files land):

review/
├── state.md                              # scope + context + selection (end of Phase 3)
├── <agent>/prompt.md                     # exact spawn prompt passed to each agent
├── <agent>/report.md                     # findings written by the agent (resume checkpoint)
├── B<N>/{prompt,report}.md               # subsystem agents (MEDIUM/LARGE only)
├── verification/{prompt,report}.md       # Phase 5
├── java-review.md                        # Phase 6 deliverable
└── java-review-rejections.md             # Phase 6 deliverable

<agent> is a specialist reviewer directory name — one of { security, performance, concurrency, reliability, maintainability, project-specific } — or a subsystem ID B<N>.

---