fix: use memoryStrategyId instead of strategyId in search_long_term_memories

The boto3 retrieve_memory_records API requires memoryStrategyId in
searchCriteria, but MemorySessionManager was sending strategyId,
causing a parameter validation error.

Fixes #303

Author: tejaskash

src/bedrock_agentcore/memory/session.py

@@ -905,7 +905,7 @@ def search_long_term_memories(
         logger.info("  -> Querying long-term memory in namespace '%s' with query: '%s'...", namespace_prefix, query)
         search_criteria = {"searchQuery": query, "topK": top_k}
         if strategy_id:
-            search_criteria["strategyId"] = strategy_id
+            search_criteria["memoryStrategyId"] = strategy_id
 
         namespace = namespace_prefix
         params = {

tests/bedrock_agentcore/memory/test_session.py

@@ -1213,7 +1213,7 @@ def test_search_long_term_memories_with_strategy(self):
 
             # Verify strategy_id was passed
             call_args = mock_client_instance.retrieve_memory_records.call_args[1]
-            assert call_args["searchCriteria"]["strategyId"] == "strategy-123"
+            assert call_args["searchCriteria"]["memoryStrategyId"] == "strategy-123"
 
     def test_search_long_term_memories_client_error(self):
         """Test search_long_term_memories with ClientError."""
@@ -2182,7 +2182,7 @@ def test_search_long_term_memories_without_strategy_id(self):
 
             # Verify strategy_id was not passed
             call_args = mock_client_instance.retrieve_memory_records.call_args[1]
-            assert "strategyId" not in call_args["searchCriteria"]
+            assert "memoryStrategyId" not in call_args["searchCriteria"]
 
     def test_list_long_term_memory_records_without_strategy_id(self):
         """Test list_long_term_memory_records without strategy_id to cover missing branch."""