-
Notifications
You must be signed in to change notification settings - Fork 51
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
Refactor SessionManager into SessionTracker
Currently, we have two components that manage browser sessions:
SessionManager
in/packages/agent/src/tools/session/lib/SessionManager.ts
SessionTracker
in/packages/agent/src/tools/session/SessionTracker.ts
These components have overlapping functionality but are implemented separately. We need to merge the functionality of SessionManager into SessionTracker to reduce duplication and simplify the codebase.
Tasks
- Merge the functionality from SessionManager into SessionTracker
- Update any references to SessionManager to use SessionTracker instead
- Ensure proper browser session cleanup is maintained
- Remove the SessionManager class once the refactoring is complete
- Update any tests that might be affected
Technical Details
- Both classes work with browser sessions but in slightly different ways
- SessionTracker is focused on tracking session state
- SessionManager handles creating and closing sessions
- We need to combine these responsibilities into a single class
Impact
This refactoring will simplify our codebase and reduce maintenance overhead by eliminating duplicate functionality.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request