Refactor: Architecture cleanup (Phases 1-2)#49

Open

vl3c wants to merge 28 commits intomainfrom

refactor/architecture-cleanup

Commits on Mar 24, 2026

Add architectural review document
vl3c
committed
Remove debug code left in production paths
vl3c
committed
Centralize scattered constants into constants.py and config.py
vl3c
committed
Extract shared env loading into static/env_config.py
vl3c
committed
Standardize error handling and extract route helper functions
vl3c
committed
Pin all dependencies to exact versions
vl3c
committed
Fix client test imports for moved constants
vl3c
committed
Extract BaseRendererTelemetry from duplicated SVG/Canvas2D telemetry
vl3c
committed
Extract BaseDrawableManager from shared drawable manager patterns
vl3c
committed
Add tests for new server modules from refactor
vl3c
committed
Add client tests for BaseDrawableManager and BaseRendererTelemetry
vl3c
committed
Update error recovery tests for removed test trigger
vl3c
committed
Remove side effect from Segment.get_state()
vl3c
committed
Migrate 7 more managers to BaseDrawableManager
vl3c
committed
Extract VisibilityManager from Canvas
vl3c
committed
Fix angle manager test mock for BaseDrawableManager migration
vl3c
committed
Extract ToolCallLogManager from AIInterface
vl3c
committed
Extract MessageMenuManager from AIInterface
vl3c
committed
Extract ImageAttachmentManager from AIInterface
vl3c
committed
Extract TTSUIManager from AIInterface
vl3c
committed
Extract ChatUIManager from AIInterface
vl3c
committed
Update architectural review with completion status and Phase 3 roadmap
vl3c
committed
Update documentation for architecture refactoring
vl3c
committed
Complete env_config migration and fix hardcoded snapshot path
vl3c
committed
Make ResultProcessor.generate_result_key public
vl3c
committed
Address medium and low priority review feedback
vl3c
committed
Remove redundant test and convert bare asserts to unittest style
vl3c
committed

Commits on Mar 25, 2026

Address remaining review feedback
vl3c
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refactor: Architecture cleanup (Phases 1-2)#49

Refactor: Architecture cleanup (Phases 1-2)#49
vl3c wants to merge 28 commits intomainfrom
refactor/architecture-cleanup

Commits on Mar 24, 2026

Add architectural review document

Remove debug code left in production paths

Centralize scattered constants into constants.py and config.py

Extract shared env loading into static/env_config.py

Standardize error handling and extract route helper functions

Pin all dependencies to exact versions

Fix client test imports for moved constants

Extract BaseRendererTelemetry from duplicated SVG/Canvas2D telemetry

Extract BaseDrawableManager from shared drawable manager patterns

Add tests for new server modules from refactor

Add client tests for BaseDrawableManager and BaseRendererTelemetry

Update error recovery tests for removed test trigger

Remove side effect from Segment.get_state()

Migrate 7 more managers to BaseDrawableManager

Extract VisibilityManager from Canvas

Fix angle manager test mock for BaseDrawableManager migration

Extract ToolCallLogManager from AIInterface

Extract MessageMenuManager from AIInterface

Extract ImageAttachmentManager from AIInterface

Extract TTSUIManager from AIInterface

Extract ChatUIManager from AIInterface

Update architectural review with completion status and Phase 3 roadmap

Update documentation for architecture refactoring

Complete env_config migration and fix hardcoded snapshot path

Make ResultProcessor.generate_result_key public

Address medium and low priority review feedback

Remove redundant test and convert bare asserts to unittest style

Commits on Mar 25, 2026

Address remaining review feedback