Skip to main content

Production Review

This section is a self-guided review for a senior engineer evaluating whether Terra is ready for real user data in production.
All current users and data are test/internal. This review is the gate before handling real PII and public traffic.

How to Use This Guide

Each page explains a system area with real code, points you to tests you can run, and suggests things to try to break. You don’t need to read every line — the guide tells you where to focus.
# Run the current Terra unit/integration suite
pnpm --dir apps/terra test

# Enforced no-skip policy for Playwright specs
node apps/terra/scripts/check-terra-skipped-tests.mjs

# Launch-critical smoke (public status page)
pnpm --dir apps/terra exec playwright test tests/status-lookup.spec.ts --project=chromium --grep "Smoke Tests"

# Type check
pnpm --dir apps/terra tsc --noEmit
Current snapshot from a local verification run:
  • Vitest: 634 passed, 0 skipped.
  • Playwright inventory: 93 executions across 3 browsers (31 test specs), with no static test.skip/describe.skip in tests/.

Architecture at a Glance

Review Categories

Security

Auth, RBAC, input validation, rate limiting

Data Protection

PII encryption, data isolation, audit logging

Functional Logic

Submission flow, notifications, integrations

Test Coverage

Every test case and how to run it

Known Gaps

What’s missing and the remediation plan

PR Review Guide

How to review ongoing PRs efficiently