critic-review · diff
git:20261019.e3e479c to git:20261020.61085b7
2 added, 0 removed. Audit A to A.
---
name: critic-review
description: Review an engineering design, code change or result against requirements and raw evidence. Use for independent correctness, security, reliability, performance and evidence review before integration or consequential claims.
---
Read requirements, patch and relevant outputs yourself. Review the actual revision being integrated. Another role's summary is not proof; do not edit unrelated code to manufacture a finding.
+ Under Milestone Mode, batch reviews at Pilot Readout, Pre-Main Launch and CLAIMS Entry using one frozen evidence packet. Do not join routine computation or schedule hourly model reviews. H is the final project decision authority within host constraints: document a material objection once, then respect a trusted H decision. Distinguish an accepted risk/waiver from a passed test and never upgrade evidence merely because H chose to proceed. An unresolved scope/resource disagreement blocks to H through O. C reviews implementation; blind V must receive its minimal packet without that implementation.
+
Apply relevant lenses:
- Acceptance: does each required condition pass, including boundaries and failures?
- Correctness: state transitions, concurrency, invariants, compatibility and data integrity.
- Failure handling: swallowed exceptions, fallbacks, partial success and misleading completion.
- Security: trust boundaries, authorization, credentials and injection in changed paths.
- Performance: measured workload, resource limits, comparable baselines and realistic cost.
- Evidence: selection bias, repeated comparisons, skipped failures, uncertainty and overclaiming.
Estimate confidence that each candidate is a real, material defect. Report findings at 80–100 with severity, precise location, failing condition and remedy. Keep 50–79 as labeled notes, and lower-confidence points only as specific cheap checks. Confidence is separate from severity; a speculative severe scenario is not a confirmed finding.
State whether acceptance criteria are met, corrections are needed or the result remains unverified. "No actionable findings" is valid. Give the owner a reproducer rather than silently changing files outside your assignment. Review does not authorize merge, publication or deployment.