🟢 Track: Architecture Modernization Epic

[arkavo-com]

Overview

This epic tracks the architectural modernization of the Arkavo iOS app. The codebase contains ~31,800 lines across 83+ Swift files with significant technical debt.

Assessment Summary

Category	Status
~40%	✅ Salvageable as-is
~35%	⚠️ Needs refactoring
~25%	❌ Needs rewrite

Phase 1: Stabilize (Critical)

• 🔴 CRITICAL: Replace force unwraps in security-critical code #206 - Replace force unwraps in security code
• 🔴 CRITICAL: Fix memory leaks in NotificationCenter observers #207 - Fix memory leaks in observers
• 🔴 CRITICAL: Replace unsafe ServiceLocator with proper DI #208 - Replace unsafe ServiceLocator

Phase 2: Architect (High Priority)

• 🟠 HIGH: Refactor GroupViewModel (1,886 lines) into smaller services #209 - Refactor GroupViewModel (1,886 lines)
• 🟠 HIGH: Redesign ArkavoMessageRouter with clear pipeline #210 - Redesign ArkavoMessageRouter
• 🟠 HIGH: Replace SharedState god object with feature-specific state #211 - Replace SharedState god object
• 🟠 HIGH: Modularize AuthenticationManager (653 lines) #212 - Modularize AuthenticationManager
• 🟠 HIGH: Split VideoCreateView (1,303 lines) - extract business logic #213 - Split VideoCreateView (1,303 lines)

Phase 3: Clean Up (Medium Priority)

• 🟡 MEDIUM: Remove duplicate StreamManager files #214 - Remove duplicate StreamManager files
• 🟡 MEDIUM: Standardize on async/await throughout codebase #215 - Standardize on async/await
• 🟡 MEDIUM: Replace print statements with structured logging #216 - Replace print with structured logging
• 🟡 MEDIUM: Address 30+ TODO/FIXME comments #217 - Address 30+ TODO/FIXME comments
• 🟡 MEDIUM: Modularize RemoteCameraStreamer (994 lines) #218 - Modularize RemoteCameraStreamer
• 🟡 MEDIUM: Extract business logic from RegistrationView #219 - Extract logic from RegistrationView

Salvageable Components (Keep)

• ✅ PersistenceController.swift - Clean repository pattern
• ✅ Account & Profile models - Well-structured SwiftData
• ✅ CertificatePinningDelegate - Security-focused
• ✅ AgentService - Good abstraction

Success Criteria

• Zero force unwraps in security code
• All ViewModels < 400 lines
• Proper dependency injection framework
• 80%+ unit test coverage on services
• No god objects remaining