fix: align memory retrieval strategies with agentcore.json config

- Add actor_id=user_id to all retrieve_memories calls (was missing in
  Strands translator)
- Pass memoryOption through to translators so generated Python code
  matches the strategies written to agentcore.json
- shortTerm: no namespace-specific retrieval (no strategies in schema)
- longAndShortTerm: retrieve from all three strategy namespaces
  (/users/{user_id}/facts, /users/{user_id}/preferences,
  /summaries/{user_id}/) matching SEMANTIC, USER_PREFERENCE, and
  SUMMARIZATION strategies

Author: tejaskash

src/cli/operations/agent/import/__tests__/translator.test.ts

@@ -43,6 +43,7 @@ describe('StrandsTranslator', () => {
     const translator = new StrandsTranslator(config, {
       agentConfig: config,
       enableMemory: false,
+      memoryOption: 'none',
       enableObservability: true,
     });
     const result = translator.translate();
@@ -68,6 +69,7 @@ describe('StrandsTranslator', () => {
     const translator = new StrandsTranslator(config, {
       agentConfig: config,
       enableMemory: true,
+      memoryOption: 'longAndShortTerm',
       enableObservability: false,
     });
     const result = translator.translate();
@@ -101,6 +103,7 @@ describe('StrandsTranslator', () => {
     const translator = new StrandsTranslator(config, {
       agentConfig: config,
       enableMemory: false,
+      memoryOption: 'none',
       enableObservability: false,
     });
     const result = translator.translate();
@@ -125,6 +128,7 @@ describe('StrandsTranslator', () => {
     const translator = new StrandsTranslator(config, {
       agentConfig: config,
       enableMemory: false,
+      memoryOption: 'none',
       enableObservability: false,
     });
     const result = translator.translate();
@@ -142,6 +146,7 @@ describe('LangGraphTranslator', () => {
     const translator = new LangGraphTranslator(config, {
       agentConfig: config,
       enableMemory: false,
+      memoryOption: 'none',
       enableObservability: true,
     });
     const result = translator.translate();
@@ -171,6 +176,7 @@ describe('LangGraphTranslator', () => {
     const translator = new LangGraphTranslator(config, {
       agentConfig: config,
       enableMemory: false,
+      memoryOption: 'none',
       enableObservability: false,
     });
     const result = translator.translate();
@@ -193,6 +199,7 @@ describe('LangGraphTranslator', () => {
     const translator = new LangGraphTranslator(config, {
       agentConfig: config,
       enableMemory: false,
+      memoryOption: 'none',
       enableObservability: false,
     });
     const result = translator.translate();

src/cli/operations/agent/import/base-translator.ts

@@ -14,10 +14,12 @@ import type {
   KnowledgeBaseInfo,
   PromptConfiguration,
 } from '../../../aws/bedrock-import-types';
+import type { MemoryOption } from '../../../tui/screens/generate/types';
 
 export interface TranslatorOptions {
   agentConfig: BedrockAgentConfig;
   enableMemory: boolean;
+  memoryOption: MemoryOption;
   enableObservability: boolean;
 }
 
@@ -52,6 +54,7 @@ export abstract class BaseBedrockTranslator {
   protected readonly promptConfigs: PromptConfiguration[];
   protected readonly memoryEnabled: boolean;
   protected readonly agentcoreMemoryEnabled: boolean;
+  protected readonly hasLongTermStrategies: boolean;
   protected readonly codeInterpreterEnabled: boolean;
   protected readonly userInputEnabled: boolean;
   protected readonly multiAgentEnabled: boolean;
@@ -90,6 +93,7 @@ export abstract class BaseBedrockTranslator {
     const memoryConfig = this.agentInfo.memoryConfiguration;
     this.memoryEnabled = !!memoryConfig?.enabledMemoryTypes?.length;
     this.agentcoreMemoryEnabled = options.enableMemory && this.memoryEnabled;
+    this.hasLongTermStrategies = options.memoryOption === 'longAndShortTerm';
 
     // Features
     this.codeInterpreterEnabled = this.actionGroups.some(

src/cli/operations/agent/import/index.ts

@@ -43,6 +43,7 @@ export async function executeImportAgent(
     const translatorOptions = {
       agentConfig,
       enableMemory,
+      memoryOption: memory,
       enableObservability: true,
     };
 

src/cli/operations/agent/import/langgraph-translator.ts

@@ -200,11 +200,16 @@ def invoke_${collabName}(query: str, state: Annotated[dict, InjectedState]) -> s
       code += 'tools += action_group_tools\n';
     }
 
-    // Memory retrieval code
-    const memoryRetrieveCode = this.agentcoreMemoryEnabled
+    // Memory retrieval code — must match the strategies written to agentcore.json:
+    // shortTerm has no strategies (no namespace-specific retrieval)
+    // longAndShortTerm has SEMANTIC, USER_PREFERENCE, and SUMMARIZATION strategies
+    const memoryRetrieveCode = this.agentcoreMemoryEnabled && this.hasLongTermStrategies
       ? `
-            memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f'/summaries/{user_id}/', query="Retrieve the most recent session summaries.", actor_id=user_id, top_k=20)
-            memory_synopsis = "\\n".join([m.get("content", {}).get("text", "") for m in memories])`
+            semantic_memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f'/users/{user_id}/facts', query="Retrieve relevant facts.", actor_id=user_id, top_k=3)
+            pref_memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f'/users/{user_id}/preferences', query="Retrieve user preferences.", actor_id=user_id, top_k=3)
+            summary_memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f'/summaries/{user_id}/', query="Retrieve the most recent session summaries.", actor_id=user_id, top_k=3)
+            all_memories = semantic_memories + pref_memories + summary_memories
+            memory_synopsis = "\\n".join([m.get("content", {}).get("text", "") for m in all_memories])`
       : this.memoryEnabled
         ? '    memory_synopsis = memory_manager.get_memory_synopsis()'
         : '';

src/cli/operations/agent/import/strands-translator.ts

@@ -170,11 +170,16 @@ def invoke_${collabName}(query: str) -> str:
       code += 'tools += action_group_tools\n';
     }
 
-    // Memory retrieval code
-    const memoryRetrieveLines = this.agentcoreMemoryEnabled
+    // Memory retrieval code — must match the strategies written to agentcore.json:
+    // shortTerm has no strategies (no namespace-specific retrieval)
+    // longAndShortTerm has SEMANTIC, USER_PREFERENCE, and SUMMARIZATION strategies
+    const memoryRetrieveLines = this.agentcoreMemoryEnabled && this.hasLongTermStrategies
       ? [
-          '        memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f\'/summaries/{user_id}/\', query="Retrieve the most recent session summaries.", top_k=20)',
-          '        memory_synopsis = "\\n".join([m.get("content", {}).get("text", "") for m in memories])',
+          '        semantic_memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f\'/users/{user_id}/facts\', query="Retrieve relevant facts.", actor_id=user_id, top_k=3)',
+          '        pref_memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f\'/users/{user_id}/preferences\', query="Retrieve user preferences.", actor_id=user_id, top_k=3)',
+          '        summary_memories = memory_client.retrieve_memories(memory_id=memory_id, namespace=f\'/summaries/{user_id}/\', query="Retrieve the most recent session summaries.", actor_id=user_id, top_k=3)',
+          '        all_memories = semantic_memories + pref_memories + summary_memories',
+          '        memory_synopsis = "\\n".join([m.get("content", {}).get("text", "") for m in all_memories])',
         ]
       : this.memoryEnabled
         ? ['        memory_synopsis = memory_manager.get_memory_synopsis()']