fix(memory-core): include 'distances' in query — restore re-ranker semantic score (#13222) (#13223)

query_summaries and query_raw_memories passed a chroma `include` that omitted
'distances', so the response carried no distance values. The Dual-Pass re-ranker
(StorageRouter) read an empty distances array -> vectorDist 0 -> semanticScore a
constant 1, silently degrading ranking to topology-only and reporting
distance:0 / relevanceScore:1 on every result.

Add 'distances' to the include in MemoryService.queryMemories and
SummaryService.querySummaries. Restores real vector distances end-to-end:
Pass-1 distances -> real semanticScore -> composite rank -> relevanceScore.

Regression: two substrate-free guards capture the include each service sends and
assert it contains 'distances' (the prior relevanceScore > 0 assertion could not
catch a constant 1). Split from #12450 (bug-b, the score-reporting half).

Author: neo-opus-grace

ai/services/memory-core/MemoryService.mjs

@@ -1102,7 +1102,8 @@ class MemoryService extends Base {
             const queryArgs = {
                 queryTexts: [query],
                 nResults,
-                include   : ['metadatas']
+                // 'distances' is load-bearing — the Dual-Pass re-ranker's semantic score needs it (else topology-only).
+                include   : ['metadatas', 'distances']
             };
 
             // Tenant-scoped where clause with additive shared-commons access.

ai/services/memory-core/SummaryService.mjs

@@ -371,7 +371,8 @@ class SummaryService extends Base {
             const queryArgs = {
                 queryTexts: [query],
                 nResults,
-                include   : ['metadatas', 'documents']
+                // 'distances' is load-bearing — the Dual-Pass re-ranker's semantic score needs it (else topology-only).
+                include   : ['metadatas', 'documents', 'distances']
             };
 
             // Tenant-scoped where clause with additive shared-commons access.

test/playwright/unit/ai/services/memory-core/QueryReRanker.spec.mjs

@@ -148,6 +148,58 @@ test.describe('StorageRouter Query Re-Ranker Defensive Handling', () => {
         expect(result.results[0].relevanceScore).toBeGreaterThan(0);
     });
 
+    test('#13222: queryMemories requests distances in the chroma include (re-ranker semantic-score input)', async () => {
+        // Regression guard for the include that omitted 'distances'. The Dual-Pass re-ranker reads the
+        // vector distance as its Pass-1 semantic score; without 'distances' in the include, chroma
+        // returns none, the score collapses to a constant 1, and ranking silently degrades to
+        // topology-only (the pre-existing `relevanceScore > 0` assertion cannot catch a constant 1).
+        // Substrate-free: stub the collection to capture the include the service actually sends, so the
+        // guard runs without the seeded chroma data the behavioral path needs.
+        const StorageRouter = (await import('../../../../../../ai/services/memory-core/managers/StorageRouter.mjs')).default;
+        await StorageRouter.ready();
+
+        const originalGetCollection = StorageRouter.getMemoryCollection;
+        let capturedInclude;
+
+        StorageRouter.getMemoryCollection = async () => ({
+            query: async (args) => {
+                capturedInclude = args.include;
+                return {ids: [[]], distances: [[]], metadatas: [[]]};
+            }
+        });
+
+        try {
+            await SDK.Memory_Service.queryMemories({query: 'include probe', nResults: 3, sessionId: 'include-probe'});
+            expect(capturedInclude).toContain('distances');
+        } finally {
+            StorageRouter.getMemoryCollection = originalGetCollection;
+        }
+    });
+
+    test('#13222: querySummaries requests distances in the chroma include (summary-path parity)', async () => {
+        const SummaryService = (await import('../../../../../../ai/services/memory-core/SummaryService.mjs')).default;
+        await SummaryService.ready();
+        const StorageRouter = (await import('../../../../../../ai/services/memory-core/managers/StorageRouter.mjs')).default;
+        await StorageRouter.ready();
+
+        const originalGetCollection = StorageRouter.getSummaryCollection;
+        let capturedInclude;
+
+        StorageRouter.getSummaryCollection = async () => ({
+            query: async (args) => {
+                capturedInclude = args.include;
+                return {ids: [[]], distances: [[]], metadatas: [[]], documents: [[]]};
+            }
+        });
+
+        try {
+            await SummaryService.querySummaries({query: 'include probe', nResults: 3});
+            expect(capturedInclude).toContain('distances');
+        } finally {
+            StorageRouter.getSummaryCollection = originalGetCollection;
+        }
+    });
+
     test('SummaryService.querySummaries should not crash on empty collections', async () => {
         // The summary collection is empty (no summarization run)
         // This should return gracefully, not crash