BMH Review Work
Workflow
- Review as a code reviewer first: findings before summary, ordered by severity, with file and line references.
- Read
references/review-gates.mdbefore judging completeness. - Check whether the change was driven by tests. New behavior without tests is a finding.
- Verify hexagonal boundaries: domain must not import adapters, CLI, filesystem, process APIs, or provider-specific schemas.
- Verify v1 scope: only Codex and Claude Code adapters are production scope.
- Verify observability integrity: tokens, context, and cost must carry source and confidence.
- Verify security: secrets are redacted before reports; raw payload handling is explicit.
- Run or report verification commands. If commands fail, include the exact failure category.
Review Rules
- Lead with bugs, risks, missing tests, and spec violations.
- Treat missing acceptance coverage as a blocker for new behavior.
- Do not accept "works manually" as benchmark evidence.
- Do not accept global user config mutation for hook installation.
- Do not accept metrics without
measurement_source,capture_source, andconfidence. - Do not accept provider-specific data in domain models.
- Do not accept Cursor/OpenCode/Pi implementation as v1 production scope without a new ADR.
Required Output
Use this structure:
- Findings
- Open questions or assumptions
- Verification run
- Change summary
If there are no findings, say that clearly and identify any residual risk or unrun verification.