#412 CONSOLIDATED: Low Priority Fixes and Minor Improvements
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 AllLoading audit history...