-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
architectureArchitecture changesArchitecture changesjulesTask for Jules AITask for Jules AImediumMedium priorityMedium priority
Description
Task Type
refactor - Architecture consolidation
Component
vertice_cli session management
Problem Description
Session management is scattered across 3 different locations with overlapping functionality.
Duplicate Locations
vertice_cli/core/session_manager/(package) - Current canonicalvertice_cli/managers/session_manager.py- Duplicatevertice_cli/session/manager.py- Duplicate
Files to Modify
vertice_cli/managers/session_manager.py- Convert to re-exportvertice_cli/session/manager.py- Convert to re-export- Update imports in dependent files
Current Code (Problematic)
Three separate implementations exist.
Expected Code (Fix)
# vertice_cli/managers/session_manager.py
"""DEPRECATED: Use vertice_cli.core.session_manager instead."""
import warnings
warnings.warn(
"vertice_cli.managers.session_manager is deprecated. "
"Use vertice_cli.core.session_manager instead.",
DeprecationWarning,
stacklevel=2,
)
from vertice_cli.core.session_manager import *
from vertice_cli.core.session_manager import SessionManager
# vertice_cli/session/manager.py
"""DEPRECATED: Use vertice_cli.core.session_manager instead."""
import warnings
warnings.warn(
"vertice_cli.session.manager is deprecated. "
"Use vertice_cli.core.session_manager instead.",
DeprecationWarning,
stacklevel=2,
)
from vertice_cli.core.session_manager import *
from vertice_cli.core.session_manager import SessionManagerValidation Commands
# Verify all import paths work
python3 -c "from vertice_cli.core.session_manager import SessionManager; print('OK')"
python3 -c "from vertice_cli.managers.session_manager import SessionManager; print('OK')"
python3 -c "from vertice_cli.session.manager import SessionManager; print('OK')"
# Check for duplicate class definitions (should be 1)
grep -rn "class SessionManager" vertice_cli/ --include="*.py" | grep -v "DEPRECATED" | wc -l
# Run tests
pytest tests/ -v -k sessionPriority
medium
References
- Single source of truth principle
- Audit ID: ARCH-005
Metadata
Metadata
Assignees
Labels
architectureArchitecture changesArchitecture changesjulesTask for Jules AITask for Jules AImediumMedium priorityMedium priority