# Code Review

> The activity of having a second party read a code change against review standards before it merges.

Record: `architecture:code-review`
Kind: activity
Layer: [Evolution Principles](https://banes-lab.com/records/layer/evolution-principles.md)
Severity: mandatory
Scope: code change
Canonical: https://banes-lab.com/ontology#architecture-code-review

Listed in [Architecture principles](https://banes-lab.com/api/records/architecture.md), after [Design Review](https://banes-lab.com/records/architecture/design-review.md) and before [Impact Analysis](https://banes-lab.com/records/architecture/impact-analysis.md).

## Repair

- Refactored by: Apply Review Feedback
- Detected by: missing approval/review
- Violated by: unreviewed production code changes
- Measured by: review coverage, defect escape rate
- Enforced by: branch protection

## Requires

- [Review Standards](https://banes-lab.com/records/lexicon/review-standards.md)

## Reinforces

- [Quality](https://banes-lab.com/records/lexicon/quality.md)
- [Security](https://banes-lab.com/records/lexicon/security.md)
- [Consistency](https://banes-lab.com/records/architecture/consistency.md)

## Enables

- [Defect Detection](https://banes-lab.com/records/lexicon/defect-detection.md)

## Conflicts with

- [Direct-to-main Unreviewed Change](https://banes-lab.com/records/lexicon/direct-to-main-unreviewed-change.md)

## In tension with

- [Throughput](https://banes-lab.com/records/architecture/throughput.md)

## Tensions

- [Code Review / Throughput](https://banes-lab.com/records/tension/code-review-throughput.md)

## Severity

- [mandatory](https://banes-lab.com/records/vocabulary/severity-mandatory.md)

## Category

- [Architecture Review / Evolution / Governance Artifacts](https://banes-lab.com/records/architecture-category/architecture-review-evolution-governance-artifacts.md)

## Enforced by

- [runtime/entrypoints/writing.entrypoint.ts](https://banes-lab.com/source/build/runtime/entrypoints/writing.entrypoint.ts.md)

## Linked from

- [Correctness / Determinism / Verification](https://banes-lab.com/ontology/principles/architecture-category-correctness-determinism-verification.md)
- [SOLID / Object-Oriented Design](https://banes-lab.com/ontology/principles/architecture-category-solid-object-oriented-design.md)
- [Scalability / Performance / Optimization](https://banes-lab.com/ontology/principles/architecture-category-scalability-performance-optimization.md)
- [Streaming / Pipeline / Dataflow Processing](https://banes-lab.com/ontology/principles/architecture-category-streaming-pipeline-dataflow-processing.md)
- [Transactions / State / Concurrency](https://banes-lab.com/ontology/principles/architecture-category-transactions-state-concurrency.md)
- [Architecture Review Evolution Governance](https://banes-lab.com/ontology/lexicon/lexicon-category-architecture-review-evolution-governance.md)
- [Quality Attributes](https://banes-lab.com/ontology/lexicon/lexicon-category-quality-attributes.md)
- [Severity levels](https://banes-lab.com/ontology/schema/the-vocabulary-severity.md)
- [The resolutions](https://banes-lab.com/ontology/schema/the-resolutions.md)