13 lines
1.1 KiB
Markdown
13 lines
1.1 KiB
Markdown
---
|
|
name: review
|
|
description: Read-only review of requested repository changes for relevant FEM contracts, correctness, tests, and scope.
|
|
---
|
|
|
|
# Repository Review
|
|
|
|
Review the requested diff, or current working-tree changes when no range is supplied. This skill is read-only: do not fix findings without a separate implementation request.
|
|
|
|
Read only the contracts and architecture sections affected by the diff; read `docs/TOLERANCE.md` for numerical/comparison changes. Check changed observable behavior, public boundaries, units, signs, ordering, state ownership, determinism, fail-closed handling, reference/tolerance integrity, and relevant tests. Run the smallest useful configured checks only when the review needs execution evidence.
|
|
|
|
Report actionable findings first, ordered by severity. Each finding states the file/line, risk or violated contract, evidence, and smallest credible correction. Then summarize checks run, gaps, and residual risks. Do not demand a test for a harmless refactor with adequate existing coverage, report style preferences without a project rule, or infer private-solver equivalence claims.
|