Code Review

Kind: activity

Layer: Evolution Principles

Record: architecture:code-review

Severity: mandatory

Scope: code change

Canonical: Ontology

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

Listed in Architecture principles, after Design Review and before Impact Analysis.

Requires

• Review Standards

Reinforces

• Quality
• Security
• Consistency

Enables

• Defect Detection

Conflicts with

• Direct-to-main Unreviewed Change

In tension with

• Throughput

Tensions

• Code Review / Throughput

Violated by

• Direct-to-main Unreviewed Change

Severity

• mandatory

Category

• Architecture Review / Evolution / Governance Artifacts

Enforced by

• runtime/entrypoints/writing.entrypoint.ts

Refactors

• Architecture Review
• Model Safety
• Agentic Architecture

Linked from

• Correctness / Determinism / Verification
• SOLID / Object-Oriented Design
• Architecture Review / Evolution / Governance Artifacts
• Model Architecture
• Scalability / Performance / Optimization
• Streaming / Pipeline / Dataflow Processing
• Transactions / State / Concurrency
• Architecture Review Evolution Governance
• Quality Attributes
• Severity levels
• The resolutions