@smallironman/mcp-memory-keeper 0.12.2-fork1

package/dist/__tests__/integration/issue33-array-items-schema.test.js

@@ -0,0 +1,165 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+const globals_1 = require("@jest/globals");
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+/**
+ * Issue #33: full tool profile exposes a schema that breaks some OpenAI-compatible providers
+ *
+ * Some providers reject tool schemas where an array property lacks an `items` declaration.
+ * Per JSON Schema spec, `items` is required for `type: 'array'` to fully describe the schema.
+ *
+ * This test scans src/index.ts and verifies that every property with `type: 'array'`
+ * has an `items` declaration within the same schema block.
+ *
+ * @see https://github.com/mkreyman/mcp-memory-keeper/issues/33
+ */
+/**
+ * Scan the source for tool definitions and find array properties missing `items`.
+ * Skips block comments to avoid false positives from commented-out schemas.
+ * Line numbers refer to the original source file for accurate debugging.
+ */
+function findArrayPropertiesMissingItems(src) {
+    const violations = [];
+    const lines = src.split('\n');
+    let currentTool = '';
+    let inBlockComment = false;
+    for (let i = 0; i < lines.length; i++) {
+        const line = lines[i];
+        // Track block comment state
+        if (inBlockComment) {
+            if (line.includes('*/')) {
+                inBlockComment = false;
+            }
+            continue;
+        }
+        if (line.includes('/*')) {
+            inBlockComment = true;
+            if (line.includes('*/')) {
+                inBlockComment = false;
+            }
+            continue;
+        }
+        // Track which tool we're inside
+        const toolMatch = line.match(/name:\s*'(context_[a-z_]+)'/);
+        if (toolMatch) {
+            currentTool = toolMatch[1];
+        }
+        // Find array type declarations
+        if (!line.match(/type:\s*'array'/))
+            continue;
+        if (!currentTool)
+            continue;
+        // Find the property name (check current line first for single-line declarations)
+        let propertyName = '(unknown)';
+        const currentLinePropMatch = line.match(/(\w+)\s*:\s*\{/);
+        if (currentLinePropMatch) {
+            propertyName = currentLinePropMatch[1];
+        }
+        else {
+            for (let j = i - 1; j >= Math.max(0, i - 5); j--) {
+                const propMatch = lines[j].match(/(\w+)\s*:\s*\{/);
+                if (propMatch) {
+                    propertyName = propMatch[1];
+                    break;
+                }
+            }
+        }
+        // Check current line first (handles single-line array declarations)
+        let foundItems = line.includes('items');
+        let depth = 0;
+        for (let j = i + 1; !foundItems && j < Math.min(lines.length, i + 50); j++) {
+            const fwdLine = lines[j];
+            for (const ch of fwdLine) {
+                if (ch === '{')
+                    depth++;
+                if (ch === '}')
+                    depth--;
+            }
+            if (fwdLine.match(/^\s*items\s*:/)) {
+                foundItems = true;
+                break;
+            }
+            if (depth < 0)
+                break;
+        }
+        if (!foundItems) {
+            violations.push({
+                tool: currentTool,
+                property: propertyName,
+                line: i + 1,
+            });
+        }
+    }
+    return violations;
+}
+(0, globals_1.describe)('Issue #33: Array properties must declare items', () => {
+    const indexPath = path.join(__dirname, '..', '..', 'index.ts');
+    const src = fs.readFileSync(indexPath, 'utf-8');
+    (0, globals_1.it)('should find tool definitions in source', () => {
+        const toolNames = src.match(/name:\s*'context_[a-z_]+'/g);
+        (0, globals_1.expect)(toolNames).not.toBeNull();
+        (0, globals_1.expect)(toolNames.length).toBeGreaterThan(0);
+    });
+    (0, globals_1.it)('every array property in every tool schema must have an items declaration', () => {
+        const violations = findArrayPropertiesMissingItems(src);
+        (0, globals_1.expect)(violations).toHaveLength(0);
+    });
+    (0, globals_1.it)('context_delegate.input.insights specifically must have items', () => {
+        const lines = src.split('\n');
+        let inDelegate = false;
+        let insightsLine = -1;
+        for (let i = 0; i < lines.length; i++) {
+            if (lines[i].match(/name:\s*'context_delegate'/)) {
+                inDelegate = true;
+            }
+            if (inDelegate &&
+                i > 0 &&
+                lines[i].match(/name:\s*'context_/) &&
+                !lines[i].match(/context_delegate/)) {
+                break;
+            }
+            if (inDelegate && lines[i].match(/insights\s*:\s*\{/)) {
+                insightsLine = i;
+                break;
+            }
+        }
+        (0, globals_1.expect)(insightsLine).toBeGreaterThan(-1);
+        const insightsBlock = lines.slice(insightsLine, insightsLine + 5).join('\n');
+        (0, globals_1.expect)(insightsBlock).toContain("type: 'array'");
+        (0, globals_1.expect)(insightsBlock).toMatch(/items\s*:/);
+    });
+});

package/dist/__tests__/integration/knowledge-graph.test.js

@@ -0,0 +1,338 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+const database_1 = require("../../utils/database");
+const knowledge_graph_1 = require("../../utils/knowledge-graph");
+const os = __importStar(require("os"));
+const path = __importStar(require("path"));
+const fs = __importStar(require("fs"));
+const uuid_1 = require("uuid");
+describe('Knowledge Graph Integration Tests', () => {
+    let dbManager;
+    let knowledgeGraph;
+    let tempDbPath;
+    let db;
+    let testSessionId;
+    beforeEach(() => {
+        tempDbPath = path.join(os.tmpdir(), `test-kg-integration-${Date.now()}.db`);
+        dbManager = new database_1.DatabaseManager({
+            filename: tempDbPath,
+            maxSize: 10 * 1024 * 1024,
+            walMode: true,
+        });
+        db = dbManager.getDatabase();
+        knowledgeGraph = new knowledge_graph_1.KnowledgeGraphManager(db);
+        // Create test session
+        testSessionId = (0, uuid_1.v4)();
+        db.prepare('INSERT INTO sessions (id, name, description) VALUES (?, ?, ?)').run(testSessionId, 'KG Integration Test', 'Testing knowledge graph integration');
+    });
+    afterEach(() => {
+        dbManager.close();
+        try {
+            fs.unlinkSync(tempDbPath);
+            fs.unlinkSync(`${tempDbPath}-wal`);
+            fs.unlinkSync(`${tempDbPath}-shm`);
+        }
+        catch (_e) {
+            // Ignore
+        }
+    });
+    describe('context_analyze', () => {
+        beforeEach(() => {
+            // Add some context items to analyze
+            const items = [
+                {
+                    key: 'task_auth',
+                    value: 'Working on AuthService class that implements authentication using JWT tokens',
+                    category: 'task',
+                },
+                {
+                    key: 'decision_db',
+                    value: 'UserModel extends BaseModel and uses PostgreSQL database',
+                    category: 'decision',
+                },
+                {
+                    key: 'progress_api',
+                    value: 'The function validateToken calls checkExpiry and getUserById',
+                    category: 'progress',
+                },
+                {
+                    key: 'note_files',
+                    value: 'Modified files: auth.service.ts, user.model.ts, and token.utils.ts',
+                    category: 'note',
+                },
+            ];
+            items.forEach(item => {
+                db.prepare('INSERT INTO context_items (id, session_id, key, value, category) VALUES (?, ?, ?, ?, ?)').run((0, uuid_1.v4)(), testSessionId, item.key, item.value, item.category);
+            });
+        });
+        it('should analyze all context items and extract entities', () => {
+            // Simulate context_analyze tool call
+            const items = db
+                .prepare('SELECT * FROM context_items WHERE session_id = ?')
+                .all(testSessionId);
+            let entitiesCreated = 0;
+            let _relationsCreated = 0;
+            for (const item of items) {
+                const analysis = knowledgeGraph.analyzeContext(testSessionId, item.value);
+                for (const entityData of analysis.entities) {
+                    const existing = knowledgeGraph.findEntity(testSessionId, entityData.name, entityData.type);
+                    if (!existing) {
+                        knowledgeGraph.createEntity(testSessionId, entityData.type, entityData.name, {
+                            confidence: entityData.confidence,
+                            source: item.key,
+                        });
+                        entitiesCreated++;
+                    }
+                }
+                for (const relationData of analysis.relations) {
+                    const subject = knowledgeGraph.findEntity(testSessionId, relationData.subject);
+                    const object = knowledgeGraph.findEntity(testSessionId, relationData.object);
+                    if (subject && object) {
+                        knowledgeGraph.createRelation(testSessionId, subject.id, relationData.predicate, object.id, relationData.confidence);
+                        _relationsCreated++;
+                    }
+                }
+            }
+            expect(entitiesCreated).toBeGreaterThan(0);
+            // Verify specific entities were created
+            expect(knowledgeGraph.findEntity(testSessionId, 'AuthService')).toBeDefined();
+            expect(knowledgeGraph.findEntity(testSessionId, 'auth.service.ts')).toBeDefined();
+            expect(knowledgeGraph.findEntity(testSessionId, 'validateToken')).toBeDefined();
+        });
+        it('should analyze specific categories only', () => {
+            const categories = ['task', 'decision'];
+            const items = db
+                .prepare(`SELECT * FROM context_items WHERE session_id = ? AND category IN (${categories.map(() => '?').join(',')})`)
+                .all(testSessionId, ...categories);
+            expect(items).toHaveLength(2);
+            for (const item of items) {
+                const analysis = knowledgeGraph.analyzeContext(testSessionId, item.value);
+                expect(analysis.entities.length).toBeGreaterThan(0);
+            }
+        });
+    });
+    describe('context_find_related', () => {
+        beforeEach(() => {
+            // Create a network of entities
+            const authService = knowledgeGraph.createEntity(testSessionId, 'class', 'AuthService');
+            const userModel = knowledgeGraph.createEntity(testSessionId, 'class', 'UserModel');
+            const tokenUtil = knowledgeGraph.createEntity(testSessionId, 'module', 'TokenUtil');
+            const database = knowledgeGraph.createEntity(testSessionId, 'database', 'PostgreSQL');
+            const validateFunc = knowledgeGraph.createEntity(testSessionId, 'function', 'validateToken');
+            // Create relations
+            knowledgeGraph.createRelation(testSessionId, authService.id, 'uses', userModel.id);
+            knowledgeGraph.createRelation(testSessionId, authService.id, 'imports', tokenUtil.id);
+            knowledgeGraph.createRelation(testSessionId, userModel.id, 'stores_in', database.id);
+            knowledgeGraph.createRelation(testSessionId, authService.id, 'contains', validateFunc.id);
+            // Add a context item that references AuthService
+            db.prepare('INSERT INTO context_items (id, session_id, key, value) VALUES (?, ?, ?, ?)').run((0, uuid_1.v4)(), testSessionId, 'auth_task', 'Working on AuthService authentication');
+        });
+        it('should find related entities by entity name', () => {
+            const entity = knowledgeGraph.findEntity(testSessionId, 'AuthService');
+            expect(entity).toBeDefined();
+            const connected = knowledgeGraph.getConnectedEntities(entity.id, 2);
+            expect(connected.size).toBeGreaterThan(1);
+            // Should include AuthService and its connections
+            const entities = Array.from(connected).map(id => {
+                const e = db.prepare('SELECT * FROM entities WHERE id = ?').get(id);
+                return e.name;
+            });
+            expect(entities).toContain('AuthService');
+            expect(entities).toContain('UserModel');
+            expect(entities).toContain('TokenUtil');
+            expect(entities).toContain('validateToken');
+        });
+        it('should find related entities by context key', () => {
+            const contextItem = db
+                .prepare('SELECT * FROM context_items WHERE session_id = ? AND key = ?')
+                .get(testSessionId, 'auth_task');
+            expect(contextItem).toBeDefined();
+            // Extract entity from context
+            const analysis = knowledgeGraph.analyzeContext(testSessionId, contextItem.value);
+            expect(analysis.entities.length).toBeGreaterThan(0);
+            const entity = knowledgeGraph.findEntity(testSessionId, 'AuthService');
+            expect(entity).toBeDefined();
+        });
+        it('should filter by relation types', () => {
+            const entity = knowledgeGraph.findEntity(testSessionId, 'AuthService');
+            const relations = knowledgeGraph.getRelations(entity.id);
+            const usesRelations = relations.filter(r => r.predicate === 'uses');
+            const importsRelations = relations.filter(r => r.predicate === 'imports');
+            expect(usesRelations.length).toBeGreaterThan(0);
+            expect(importsRelations.length).toBeGreaterThan(0);
+        });
+    });
+    describe('context_visualize', () => {
+        beforeEach(() => {
+            // Create entities for visualization
+            knowledgeGraph.createEntity(testSessionId, 'class', 'User');
+            knowledgeGraph.createEntity(testSessionId, 'class', 'Product');
+            knowledgeGraph.createEntity(testSessionId, 'service', 'AuthService');
+            // Create context items with different categories and times
+            const now = new Date();
+            for (let i = 0; i < 10; i++) {
+                const category = ['task', 'decision', 'progress'][i % 3];
+                const priority = ['high', 'normal', 'low'][i % 3];
+                db.prepare(`INSERT INTO context_items 
+           (id, session_id, key, value, category, priority, created_at) 
+           VALUES (?, ?, ?, ?, ?, ?, ?)`).run((0, uuid_1.v4)(), testSessionId, `item_${i}`, `Test item ${i}`, category, priority, new Date(now.getTime() - i * 3600000).toISOString() // 1 hour apart
+                );
+            }
+        });
+        it('should generate graph visualization data', () => {
+            const user = knowledgeGraph.findEntity(testSessionId, 'User');
+            const auth = knowledgeGraph.findEntity(testSessionId, 'AuthService');
+            knowledgeGraph.createRelation(testSessionId, auth.id, 'validates', user.id);
+            const graphData = knowledgeGraph.getGraphData(testSessionId);
+            expect(graphData.nodes.length).toBeGreaterThan(0);
+            expect(graphData.edges.length).toBeGreaterThan(0);
+            // Verify structure
+            expect(graphData.nodes[0]).toHaveProperty('id');
+            expect(graphData.nodes[0]).toHaveProperty('type');
+            expect(graphData.nodes[0]).toHaveProperty('name');
+            expect(graphData.edges[0]).toHaveProperty('source');
+            expect(graphData.edges[0]).toHaveProperty('target');
+            expect(graphData.edges[0]).toHaveProperty('predicate');
+        });
+        it('should generate timeline visualization data', () => {
+            const timeline = db
+                .prepare(`
+        SELECT 
+          strftime('%Y-%m-%d %H:00', created_at) as hour,
+          COUNT(*) as events,
+          GROUP_CONCAT(DISTINCT category) as categories
+        FROM context_items
+        WHERE session_id = ?
+        GROUP BY hour
+        ORDER BY hour DESC
+        LIMIT 24
+      `)
+                .all(testSessionId);
+            expect(timeline.length).toBeGreaterThan(0);
+            expect(timeline[0]).toHaveProperty('hour');
+            expect(timeline[0]).toHaveProperty('events');
+            expect(timeline[0]).toHaveProperty('categories');
+        });
+        it('should generate heatmap visualization data', () => {
+            const heatmap = db
+                .prepare(`
+        SELECT 
+          category,
+          priority,
+          COUNT(*) as count
+        FROM context_items
+        WHERE session_id = ?
+        GROUP BY category, priority
+      `)
+                .all(testSessionId);
+            expect(heatmap.length).toBeGreaterThan(0);
+            expect(heatmap[0]).toHaveProperty('category');
+            expect(heatmap[0]).toHaveProperty('priority');
+            expect(heatmap[0]).toHaveProperty('count');
+            // Verify we have data for different category/priority combinations
+            const categories = new Set(heatmap.map(h => h.category));
+            const priorities = new Set(heatmap.map(h => h.priority));
+            expect(categories.size).toBeGreaterThanOrEqual(2);
+            expect(priorities.size).toBeGreaterThanOrEqual(2);
+        });
+    });
+    describe('Complex scenarios', () => {
+        it('should handle code analysis workflow', () => {
+            // Simulate analyzing a codebase
+            const codeContext = [
+                'The AuthController class handles authentication endpoints',
+                'It uses AuthService which implements IAuthService interface',
+                'AuthService calls UserRepository to fetch user data',
+                'The validateToken function uses jwt.verify from jsonwebtoken library',
+                'Files involved: auth.controller.ts, auth.service.ts, user.repository.ts',
+            ];
+            // Analyze each piece of context
+            const allEntities = new Set();
+            const allRelations = [];
+            codeContext.forEach(text => {
+                const analysis = knowledgeGraph.analyzeContext(testSessionId, text);
+                analysis.entities.forEach(e => {
+                    if (!knowledgeGraph.findEntity(testSessionId, e.name, e.type)) {
+                        knowledgeGraph.createEntity(testSessionId, e.type, e.name);
+                        allEntities.add(e.name);
+                    }
+                });
+                analysis.relations.forEach(r => {
+                    const subject = knowledgeGraph.findEntity(testSessionId, r.subject);
+                    const object = knowledgeGraph.findEntity(testSessionId, r.object);
+                    if (subject && object) {
+                        knowledgeGraph.createRelation(testSessionId, subject.id, r.predicate, object.id, r.confidence);
+                        allRelations.push(r);
+                    }
+                });
+            });
+            // Verify entities were extracted
+            expect(allEntities.size).toBeGreaterThan(5);
+            expect(allEntities).toContain('AuthController');
+            expect(allEntities).toContain('AuthService');
+            expect(allEntities).toContain('auth.controller.ts');
+            // Verify relationships
+            expect(allRelations.length).toBeGreaterThan(0);
+            // Test finding related entities
+            const authService = knowledgeGraph.findEntity(testSessionId, 'AuthService');
+            if (authService) {
+                const connected = knowledgeGraph.getConnectedEntities(authService.id, 2);
+                expect(connected.size).toBeGreaterThan(1);
+            }
+        });
+        it('should maintain graph consistency across operations', () => {
+            // Create initial graph
+            const module1 = knowledgeGraph.createEntity(testSessionId, 'module', 'CoreModule');
+            const module2 = knowledgeGraph.createEntity(testSessionId, 'module', 'UtilsModule');
+            const service = knowledgeGraph.createEntity(testSessionId, 'service', 'DataService');
+            knowledgeGraph.createRelation(testSessionId, module1.id, 'imports', module2.id);
+            knowledgeGraph.createRelation(testSessionId, module1.id, 'provides', service.id);
+            // Add observations
+            knowledgeGraph.addObservation(service.id, 'Handles data transformation');
+            // Verify graph integrity
+            const graphData = knowledgeGraph.getGraphData(testSessionId);
+            expect(graphData.nodes).toHaveLength(3);
+            expect(graphData.edges).toHaveLength(2);
+            // Test pruning doesn't remove entities with relations
+            const pruned = knowledgeGraph.pruneOrphanedEntities(testSessionId);
+            expect(pruned).toBe(0);
+            // All entities should still exist
+            expect(knowledgeGraph.findEntity(testSessionId, 'CoreModule')).toBeDefined();
+            expect(knowledgeGraph.findEntity(testSessionId, 'UtilsModule')).toBeDefined();
+            expect(knowledgeGraph.findEntity(testSessionId, 'DataService')).toBeDefined();
+        });
+    });
+});