Agent farm converter

[csmangum]

This pull request introduces a new "simple" tiering strategy, updates memory ID generation for uniqueness, enhances memory system configuration with Redis STM settings, and adds a new debugging script. It also includes updates to tests and other minor improvements. Below is a summary of the changes grouped by theme:

Tiering Strategy Enhancements:

• Introduced a SimpleTieringStrategy class that places all memories in the STM tier. Updated the create_tiering_strategy factory function and configuration defaults to use this new strategy. (converter/tiering.py: [1] [2] [3]; converter/config.py: [4] [5] [6]

Memory System Improvements:

• Updated memory ID generation to include agent ID, step number, and timestamp for uniqueness. (memory/agent_memory.py: memory/agent_memory.pyL223-R224)
• Enhanced add_memory to include tier information in metadata and pass it to appropriate memory storage methods. (memory/core.py: memory/core.pyR363-R383)

Configuration and Debugging:

• Added Redis STM configuration to MemoryConfig, including a memory limit, TTL, and namespace. (converter/converter.py: [1] [2]
• Introduced a new debug_converter.py script for running and validating the AgentFarm to Memory System conversion process. (debug_converter.py: debug_converter.pyR1-R82)

Testing Updates:

• Updated tests to validate the new "simple" tiering strategy and ensure compatibility with the updated configurations. (tests/converter/test_config.py: [1] [2]; tests/test_memory_agent.py: [3] [4] [5] [6]

Minor Fixes and Enhancements:

• Ensured memory entries have required fields like id during similarity searches. (memory/search/strategies/similarity.py: memory/search/strategies/similarity.pyR227-R230)

[Copilot]

Pull Request Overview

This PR adds a “simple” tiering strategy as the default, improves memory ID uniqueness and tier metadata handling, extends the converter configuration with Redis STM settings, and supplies a debug script along with updated tests.

• Introduce SimpleTieringStrategy and make it the default in config and factory
• Update memory ID format and pass tier info through add_memory
• Add Redis STM settings to converter config and include debug_converter.py for validation
• Align tests with new tiering defaults and engine imports

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
converter/tiering.py	Added SimpleTieringStrategy, updated factory default to “simple”
converter/config.py	Set tiering_strategy_type default to “simple” and updated validation list
memory/agent_memory.py	Revised _create_memory_entry to use timestamp-based IDs and imported uuid (unused)
memory/core.py	Extended add_memory to pull tier from metadata and pass it to store calls
memory/search/strategies/similarity.py	Added handling for AgentMemorySystem and ensure each memory has an “id”
converter/converter.py	Imported and configured RedisSTMConfig in converter flow
debug_converter.py	New script for running and validating the conversion process
tests/converter/test_config.py	Updated assertions to expect simple tiering strategy
tests/test_memory_agent.py	Removed deprecated imports and fixed TextEmbeddingEngine patch target

Comments suppressed due to low confidence (2)

memory/search/strategies/similarity.py:102

• AgentMemorySystem is not imported in this module, which will cause a NameError. Add from memory.core import AgentMemorySystem at the top of the file.

if isinstance(self.memory_system, AgentMemorySystem):

memory/agent_memory.py:6

• [nitpick] The uuid module is imported but never used; consider removing this import to keep the code clean.

import uuid