fix(memory-core): add boundary guards to CollectionProxy and fix config key (#10504) (#10505)

* fix(memory-core): add boundary guards to CollectionProxy and fix config key (#10504)

* fix(memory-core): address PR review feedback (#10504)

* test(memory-core): fix environment variable names in SessionSummarization spec (#10504)

---------

Co-authored-by: tobiu <tobiasuhlig78@gmail.com>

Author: neo-gemini-pro

ai/mcp/server/memory-core/managers/CollectionProxy.mjs

@@ -19,7 +19,7 @@ class CollectionProxy extends Base {
     }
 
     async getManagers() {
-        const architecture = aiConfig.architecture || 'hybrid';
+        const architecture = aiConfig.engine || 'hybrid';
         const managers = [];
 
         // In Hybrid RAG, vectors exclusively live in ChromaDB
@@ -57,16 +57,25 @@ class CollectionProxy extends Base {
 
     async get(args) {
         const collections = await this.getCollections();
+        if (!collections || collections.length === 0 || !collections[0]) {
+            throw new Error(`[CollectionProxy] get() failed: No underlying collection available for type '${this.collectionType}'`);
+        }
         return collections[0].get(args);
     }
 
     async query(args) {
         const collections = await this.getCollections();
+        if (!collections || collections.length === 0 || !collections[0]) {
+            throw new Error(`[CollectionProxy] query() failed: No underlying collection available for type '${this.collectionType}'`);
+        }
         return collections[0].query(args);
     }
 
     async count() {
         const collections = await this.getCollections();
+        if (!collections || collections.length === 0 || !collections[0]) {
+            throw new Error(`[CollectionProxy] count() failed: No underlying collection available for type '${this.collectionType}'`);
+        }
         return collections[0].count();
     }
 

test/playwright/unit/ai/mcp/server/memory-core/services/SessionSummarization.spec.mjs

@@ -2,8 +2,8 @@ import {setup} from '../../../../../../setup.mjs';
 
 const appName = 'MemoryCoreTest';
 
-process.env.MODEL_PROVIDER = 'openAiCompatible';
-process.env.OPENAI_COMPATIBLE_MODEL   = 'gemma4';
+process.env.NEO_MODEL_PROVIDER = 'openAiCompatible';
+process.env.NEO_OPENAI_COMPATIBLE_MODEL = 'gemma4';
 
 setup({
     neoConfig: {
@@ -50,8 +50,7 @@ test.describe('Memory Core Offline Summarization', () => {
 
         const testDbName              = `memory-core-session-test-${process.pid}-${Date.now()}.sqlite`;
         const testDbPath              = path.join(tmpDir, testDbName);
-        aiConfig.engines.neo.dataDir  = tmpDir;
-        aiConfig.engines.neo.filename = testDbName;
+        aiConfig.storagePaths.graph   = testDbPath;
 
         // Remove existing test db
         if (fs.existsSync(testDbPath)) {