architecture-review-checklistlisted

# Architecture Review Checklist ## 1. TASK Compliance - [ ] **Coverage:** All Use Cases mapped to components? - [ ] **Constraints:** All non-functional requirements met? ## 2. Data Model (CRITICAL) - [ ] **Completeness:** All entities, attributes, relationships defined? - [ ] **Types:** Correct data types chosen? (e.g., TIMESTAMP vs VARCHAR) - [ ] **Indexes:** Defined for frequent queries? - [ ] **Migrations:** Plan for existing data exists? - [ ] **Business Rules:** Constraints enforced (UNIQUE, NOT NULL)? ## 3. System Design - [ ] **Simplicity:** Least moving parts? (No overengineering). - [ ] **Style:** Pattern matches problem (Monolith vs Microservices). - [ ] **Boundaries:** Clear segregation of duties (SRP). - [ ] **Document Size:** `docs/ARCHITECTURE.md` is ≤1500 lines, OR is an INDEX (~≤200 lines) with section chunks in `docs/architectures/` and all links resolving. - [ ] **No Per-Task Drift:** ARCHITECTURE.md is a single living document — no `architecture-NNN-*.md` snapshots, nothing moved into `docs/archives/`. ## 4. Security - [ ] **Auth:** Authentication & Authorization defined? - [ ] **Protection:** OWASP Top 10 considered? - [ ] **Secrets:** No hardcoded keys? ## 5. Scalability & Reliability - [ ] **Scaling:** Horizontal/Vertical strategy? - [ ] **Faults:** Error handling, retries, backups? ## 6. Register (`documentation-standards` §5.5) - [ ] **Scan attached:** `artifact-formalizer/scripts/scan_register.py docs/ARCHITECTURE.md --sections` was run; `