Repository navigation
[MEDIUM] Consolidate _get_ssot_config Functions (Phase 2 of #751) #763
Description
Activity
- added 6 commits that reference this issue
on Feb 2, 2026 Implementation Complete ✅
Phase 2 migration has been completed. All 6 files with duplicate
_get_ssot_config()functions have been migrated to useConfigRegistry.Summary of Changes:
File Usages Migrated Type compat.py7 Module-level constants defaults.py4 Function-local config redis_constants.py4 Class properties model_constants.py10 Class attributes + method network_constants.py45 Class attributes + URLs manager.py2 Instance variables Commits:
17a20e46- migrate compat.py30b65040- migrate defaults.py02eabeeb- migrate redis_constants.py73654181- migrate model_constants.py9b0e8098- migrate network_constants.py2d9a92b0- migrate manager.py
Test Results:
- 45/45 config-related tests passing
- All migrated modules import and function correctly
PR: #764
- added a commit that references this issue
on Jul 24, 2026 - added a commit that references this issue
on Aug 1, 2026
Overview
Phase 2 continuation of #751. Migrate 6 duplicate
_get_ssot_config()functions to use the newConfigRegistry.Files to Migrate
src/constants/network_constants.py(~400 lines)src/constants/redis_constants.pysrc/constants/model_constants.pysrc/config/compat.pysrc/config/manager.pysrc/config/defaults.pyChallenge
These files use attribute access patterns like
_ssot.redis.hostwhich need conversion toConfigRegistry.get("redis.host"). Type conversions may also be needed (strings → typed values).Acceptance Criteria
_get_ssot_config()definitions removedRelated
src/config/registry.pysrc/config/registry_defaults.py