-
Notifications
You must be signed in to change notification settings - Fork 25
[Good First Issue] [Bug] Fix memory leak in conversation historyΒ #86
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomershacktoberfest
Description
π― Description
The conversation history in StandardAgent might grow indefinitely if window size is set incorrectly.
π Problem/Motivation
Memory leaks can cause performance issues and crashes in long-running agents.
β Acceptance Criteria
- Add bounds checking for history window
- Ensure history is properly trimmed
- Add tests verifying memory cleanup
- Handle edge cases (window size 0, negative)
π Step-by-Step Instructions
Making Changes:
6. Open agents/standard_agent.py
7. Review _record_interaction method
8. Add bounds checking and validation
9. Add tests that verify memory doesn't grow indefinitely
π Estimated Effort
Time: 45 minutes | Difficulty: Beginner | Skills: Python, debugging
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomershacktoberfest