Describe the bug
There is a critical logical flaw in the STLTMemory class where short-term memories are permanently deleted instead of being consolidated into long-term memory. When the short-term memory exceeds its capacity, the oldest items are popped from the queue but are never actually passed to the LLM for summarization.
Instead, the LLM is currently being prompted to summarize the remaining active short-term memories rather than the ones that were popped off, leading to redundant summarization.
Expected Behaviour
When the Short-term capacity+ consolidation capacity is exceeded, the code is supposed to pop off the oldest k entries, where k is the consolidation capacity, it is then supposed to summarize these k entries and integrate it into the long-term memory.