020-quality-gates

Quality Gates

Four quality gate variants for different session phases. Commands are read from Session Config in CLAUDE.md (field defaults shown below).

Config Fields

Field	Default	Notes
test-command	pnpm test --run	Set to skip to disable
typecheck-command	tsgo --noEmit	Set to skip to disable
lint-command	pnpm lint	Set to skip to disable

Variant 1: Baseline

When: Session start (Phase 3) Purpose: Quick health check — non-blocking.

{typecheck-command} 2>&1 | tail -5
{test-command} 2>&1 | tail -5

• Report results but do NOT block the session
• Store error counts as the session baseline for later comparison

Variant 2: Incremental

When: After implementation waves (Impl-Core, Impl-Polish) Purpose: Verify implementation did not break anything.

# Run tests on changed files only
{test-command} -- <changed-test-files>

# Full typecheck (always full — partial typecheck is unreliable)
{typecheck-command}

• Report failures but do NOT block wave progression
• If issues found, add fix tasks to the next wave automatically

Variant 3: Full Gate

When: Session end (Phase 2) — before committing Purpose: Final quality gate — BLOCKING.

# 1. Typecheck — must produce 0 errors
{typecheck-command}

# 2. Tests — must pass (exit code 0)
{test-command}

# 3. Lint — errors NOT OK, warnings OK
{lint-command}

# 4. Debug artifact scan
grep -rn 'console\.log\|debugger\|TODO: remove' \
  --include='*.ts' --include='*.tsx' --include='*.js' \
  src/ lib/ 2>/dev/null || true

Rules:

• Do NOT commit if any check fails
• Fix quick issues (<2 min) inline
• For longer fixes: create a priority::high issue and exclude affected files from commit

Variant 4: Per-File

When: Session review (reviewer agent) Purpose: Targeted check on specific changed files.

# Run tests for specific files
{test-command} -- <specific-file-paths>

# Full typecheck
{typecheck-command}