#412 CONSOLIDATED: Low Priority Fixes and Minor Improvements

closed low Created 2025-12-12 14:48 · Updated 2025-12-12 17:01

Description

Edit
## Consolidated Low Priority Issues ### Code Quality 1. ~~**Logger f-string** (ex #371, #396)~~: FIXED - orchestrator.py:585 now has f-prefix 2. ~~**Magic numbers** (ex #372)~~: FIXED - activity_threshold, max_iterations, ttl_seconds now configurable in [worker] section 3. ~~**Connection leak docstring** (ex #367)~~: FIXED - get_raw_db_connection() docstring updated with correct pattern ### Testing 4. ~~**Flaky tests** (ex #340)~~: FIXED - test_concurrent_logging uses retry mechanism, test_async_deferral_multiple_jobs timeout increased to 90s ### Documentation 5. ~~**_save_state automation** (ex #370)~~: FIXED - Enhanced docstring with usage guidance ### Deferred (Requires Separate Issue) 6. **Sherlock Phase 4** (ex #334): Moved back to parent issue #330 - too substantial for low-priority consolidation ## Status 5/6 items completed. Item 6 deferred to #330. ## Original Issues Consolidated from: #334, #340, #367, #370, #371, #372

Comments

Loading comments...

Context

Loading context...

Audit History

View All
Loading audit history...