fix: use /episodes/{actorId}/{sessionId} and /reflections/{actorId} for EPISODIC namespaces

Author: tejaskash

src/cli/commands/add/__tests__/add-memory.test.ts

@@ -139,8 +139,8 @@ describe('add memory command', () => {
       expect(summarization?.namespaces).toEqual(['/summaries/{actorId}/{sessionId}']);
 
       const episodic = memory?.strategies?.find((s: { type: string }) => s.type === 'EPISODIC');
-      expect(episodic?.namespaces).toEqual(['/strategy/{memoryStrategyId}/actor/{actorId}/']);
-      expect(episodic?.reflectionNamespaces).toEqual(['/strategy/{memoryStrategyId}/actor/{actorId}/']);
+      expect(episodic?.namespaces).toEqual(['/episodes/{actorId}/{sessionId}']);
+      expect(episodic?.reflectionNamespaces).toEqual(['/reflections/{actorId}']);
     });
 
     it('creates memory with EPISODIC strategy including default namespaces and reflectionNamespaces', async () => {
@@ -154,8 +154,8 @@ describe('add memory command', () => {
       const memory = projectSpec.memories.find((m: { name: string }) => m.name === memoryName);
       const episodic = memory?.strategies?.find((s: { type: string }) => s.type === 'EPISODIC');
       expect(episodic).toBeTruthy();
-      expect(episodic?.namespaces).toEqual(['/strategy/{memoryStrategyId}/actor/{actorId}/']);
-      expect(episodic?.reflectionNamespaces).toEqual(['/strategy/{memoryStrategyId}/actor/{actorId}/']);
+      expect(episodic?.namespaces).toEqual(['/episodes/{actorId}/{sessionId}']);
+      expect(episodic?.reflectionNamespaces).toEqual(['/reflections/{actorId}']);
     });
   });
 });

src/cli/commands/create/__tests__/create.test.ts

@@ -141,8 +141,8 @@ describe('create command', () => {
 
       const episodic = memory?.strategies?.find((s: { type: string }) => s.type === 'EPISODIC');
       expect(episodic, 'EPISODIC strategy should exist in longAndShortTerm').toBeTruthy();
-      expect(episodic?.namespaces).toEqual(['/strategy/{memoryStrategyId}/actor/{actorId}/']);
-      expect(episodic?.reflectionNamespaces).toEqual(['/strategy/{memoryStrategyId}/actor/{actorId}/']);
+      expect(episodic?.namespaces).toEqual(['/episodes/{actorId}/{sessionId}']);
+      expect(episodic?.reflectionNamespaces).toEqual(['/reflections/{actorId}']);
     });
   });
 

src/schema/schemas/primitives/__tests__/memory.test.ts

@@ -76,24 +76,24 @@ describe('MemoryStrategySchema', () => {
   it('accepts EPISODIC strategy with reflectionNamespaces', () => {
     const result = MemoryStrategySchema.safeParse({
       type: 'EPISODIC',
-      namespaces: ['/strategy/{memoryStrategyId}/actor/{actorId}/'],
-      reflectionNamespaces: ['/strategy/{memoryStrategyId}/actor/{actorId}/'],
+      namespaces: ['/episodes/{actorId}/{sessionId}'],
+      reflectionNamespaces: ['/reflections/{actorId}'],
     });
     expect(result.success).toBe(true);
   });
 
   it('rejects EPISODIC strategy without reflectionNamespaces', () => {
     const result = MemoryStrategySchema.safeParse({
       type: 'EPISODIC',
-      namespaces: ['/strategy/{memoryStrategyId}/actor/{actorId}/'],
+      namespaces: ['/episodes/{actorId}/{sessionId}'],
     });
     expect(result.success).toBe(false);
   });
 
   it('rejects EPISODIC strategy with empty reflectionNamespaces', () => {
     const result = MemoryStrategySchema.safeParse({
       type: 'EPISODIC',
-      namespaces: ['/strategy/{memoryStrategyId}/actor/{actorId}/'],
+      namespaces: ['/episodes/{actorId}/{sessionId}'],
       reflectionNamespaces: [],
     });
     expect(result.success).toBe(false);
@@ -119,7 +119,7 @@ describe('DEFAULT_STRATEGY_NAMESPACES', () => {
   });
 
   it('has default namespaces for EPISODIC', () => {
-    expect(DEFAULT_STRATEGY_NAMESPACES.EPISODIC).toEqual(['/strategy/{memoryStrategyId}/actor/{actorId}/']);
+    expect(DEFAULT_STRATEGY_NAMESPACES.EPISODIC).toEqual(['/episodes/{actorId}/{sessionId}']);
   });
 
   it('does not have default namespaces for CUSTOM (removed)', () => {

src/schema/schemas/primitives/memory.ts

@@ -24,14 +24,14 @@ export const DEFAULT_STRATEGY_NAMESPACES: Partial<Record<MemoryStrategyType, str
   SEMANTIC: ['/users/{actorId}/facts'],
   USER_PREFERENCE: ['/users/{actorId}/preferences'],
   SUMMARIZATION: ['/summaries/{actorId}/{sessionId}'],
-  EPISODIC: ['/strategy/{memoryStrategyId}/actor/{actorId}/'],
+  EPISODIC: ['/episodes/{actorId}/{sessionId}'],
 };
 
 /**
  * Default reflection namespaces for the EPISODIC strategy.
  * The service requires reflection namespaces to be the same as or a prefix of episode namespaces.
  */
-export const DEFAULT_EPISODIC_REFLECTION_NAMESPACES: string[] = ['/strategy/{memoryStrategyId}/actor/{actorId}/'];
+export const DEFAULT_EPISODIC_REFLECTION_NAMESPACES: string[] = ['/reflections/{actorId}'];
 
 /**
  * Memory strategy name validation.