mcp-memory-keeper 0.10.0

package/bin/mcp-memory-keeper

@@ -0,0 +1,52 @@
+#!/usr/bin/env node
+
+/**
+ * MCP Memory Keeper CLI
+ * This wrapper ensures the server runs correctly when invoked via npx
+ */
+
+const { spawn } = require('child_process');
+const path = require('path');
+const fs = require('fs');
+const os = require('os');
+
+// Determine the data directory
+const DATA_DIR = process.env.DATA_DIR || path.join(os.homedir(), 'mcp-data', 'memory-keeper');
+
+// Ensure data directory exists
+if (!fs.existsSync(DATA_DIR)) {
+  fs.mkdirSync(DATA_DIR, { recursive: true });
+}
+
+// Set environment variable for the server
+process.env.DATA_DIR = DATA_DIR;
+
+// Get the path to the actual server
+const serverPath = path.join(__dirname, '..', 'dist', 'index.js');
+
+// Check if the server is built
+if (!fs.existsSync(serverPath)) {
+  console.error('Error: Server not built. This should not happen with the npm package.');
+  console.error('Please report this issue at: https://github.com/mkreyman/mcp-memory-keeper/issues');
+  process.exit(1);
+}
+
+// Change to data directory (where context.db will be created)
+process.chdir(DATA_DIR);
+
+// Spawn the server
+const child = spawn(process.execPath, [serverPath, ...process.argv.slice(2)], {
+  stdio: 'inherit',
+  env: process.env
+});
+
+// Handle exit
+child.on('exit', (code) => {
+  process.exit(code);
+});
+
+// Handle errors
+child.on('error', (err) => {
+  console.error('Failed to start memory-keeper server:', err);
+  process.exit(1);
+});

package/dist/__tests__/helpers/database-test-helper.js

@@ -0,0 +1,160 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.DatabaseTestHelper = void 0;
+/**
+ * Helper utilities for managing database behavior in tests
+ */
+class DatabaseTestHelper {
+    db;
+    originalTriggers = new Map();
+    constructor(db) {
+        this.db = db;
+    }
+    /**
+     * Disable timestamp update triggers to prevent unexpected timestamp updates
+     * during tests that are sensitive to update counts.
+     */
+    disableTimestampTriggers() {
+        const triggerNames = [
+            'update_context_items_timestamp',
+            'update_sessions_timestamp',
+            'update_retention_policies_timestamp',
+            'update_feature_flags_timestamp',
+            'increment_sequence_update',
+        ];
+        for (const triggerName of triggerNames) {
+            // Get the trigger definition before dropping it
+            const triggerDef = this.db
+                .prepare(`SELECT sql FROM sqlite_master WHERE type = 'trigger' AND name = ?`)
+                .get(triggerName);
+            if (triggerDef && triggerDef.sql) {
+                this.originalTriggers.set(triggerName, triggerDef.sql);
+                // Drop the trigger
+                this.db.exec(`DROP TRIGGER IF EXISTS ${triggerName}`);
+            }
+        }
+    }
+    /**
+     * Re-enable timestamp update triggers after tests
+     */
+    enableTimestampTriggers() {
+        for (const [triggerName, sql] of this.originalTriggers) {
+            if (sql) {
+                try {
+                    this.db.exec(sql);
+                }
+                catch (error) {
+                    console.warn(`Failed to recreate trigger ${triggerName}:`, error);
+                }
+            }
+        }
+        this.originalTriggers.clear();
+    }
+    /**
+     * Normalize timestamps for comparison in tests.
+     * Sets all timestamps to a fixed value to avoid timestamp-based failures.
+     */
+    normalizeTimestamps(items) {
+        const fixedTimestamp = '2024-01-01T00:00:00.000Z';
+        return items.map(item => ({
+            ...item,
+            created_at: fixedTimestamp,
+            updated_at: fixedTimestamp,
+        }));
+    }
+    /**
+     * Update an item without triggering timestamp updates.
+     * Useful for tests that need to control timestamps precisely.
+     */
+    updateWithoutTimestamp(table, updates, where) {
+        // Build the UPDATE statement
+        const setClauses = Object.keys(updates)
+            .map(key => `${key} = ?`)
+            .join(', ');
+        const whereClauses = Object.keys(where)
+            .map(key => `${key} = ?`)
+            .join(' AND ');
+        const sql = `UPDATE ${table} SET ${setClauses} WHERE ${whereClauses}`;
+        const values = [...Object.values(updates), ...Object.values(where)];
+        // Temporarily disable triggers
+        this.disableTimestampTriggers();
+        try {
+            this.db.prepare(sql).run(...values);
+        }
+        finally {
+            // Re-enable triggers
+            this.enableTimestampTriggers();
+        }
+    }
+    /**
+     * Get counts of items by their modification status
+     * Useful for tests that check added vs modified items
+     */
+    getModificationCounts(sessionId, sinceTimestamp) {
+        const added = this.db
+            .prepare(`SELECT COUNT(*) as count FROM context_items 
+         WHERE session_id = ? AND created_at > ?`)
+            .get(sessionId, sinceTimestamp);
+        const modified = this.db
+            .prepare(`SELECT COUNT(*) as count FROM context_items 
+         WHERE session_id = ? 
+         AND created_at <= ? 
+         AND updated_at > ?`)
+            .get(sessionId, sinceTimestamp, sinceTimestamp);
+        return {
+            added: added.count,
+            modified: modified.count,
+        };
+    }
+    /**
+     * Create a test item with controlled timestamps
+     */
+    createTestItem(params) {
+        const { id, sessionId, key, value, createdAt = new Date().toISOString(), updatedAt = createdAt, category = null, priority = 'normal', channel = 'general', } = params;
+        // Temporarily disable triggers to control timestamps
+        this.disableTimestampTriggers();
+        try {
+            this.db
+                .prepare(`INSERT INTO context_items 
+           (id, session_id, key, value, category, priority, channel, created_at, updated_at) 
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run(id, sessionId, key, value, category, priority, channel, createdAt, updatedAt);
+        }
+        finally {
+            this.enableTimestampTriggers();
+        }
+    }
+    /**
+     * Helper to check if change tracking triggers are affecting counts
+     */
+    getChangeTrackingCount(sessionId, sinceSequence = 0) {
+        try {
+            const result = this.db
+                .prepare(`SELECT COUNT(*) as count FROM context_changes 
+           WHERE session_id = ? AND sequence_id > ?`)
+                .get(sessionId, sinceSequence);
+            return result.count;
+        }
+        catch {
+            // Table might not exist in all test environments
+            return 0;
+        }
+    }
+    /**
+     * Clear all change tracking data for a clean test state
+     */
+    clearChangeTracking(sessionId) {
+        try {
+            if (sessionId) {
+                this.db.prepare('DELETE FROM context_changes WHERE session_id = ?').run(sessionId);
+            }
+            else {
+                this.db.prepare('DELETE FROM context_changes').run();
+            }
+        }
+        catch {
+            // Table might not exist in all test environments
+        }
+    }
+}
+exports.DatabaseTestHelper = DatabaseTestHelper;

package/dist/__tests__/helpers/test-server.js

@@ -0,0 +1,92 @@
+"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 });
+exports.TestServer = void 0;
+const index_js_1 = require("@modelcontextprotocol/sdk/server/index.js");
+const database_js_1 = require("../../utils/database.js");
+const git_js_1 = require("../../utils/git.js");
+const os = __importStar(require("os"));
+const path = __importStar(require("path"));
+const fs = __importStar(require("fs"));
+class TestServer {
+    server;
+    dbManager;
+    gitOps;
+    tempDbPath;
+    constructor() {
+        // Create temporary database for testing
+        this.tempDbPath = path.join(os.tmpdir(), `test-mcp-${Date.now()}.db`);
+        this.dbManager = new database_js_1.DatabaseManager({
+            filename: this.tempDbPath,
+            maxSize: 10 * 1024 * 1024, // 10MB
+            walMode: true,
+        });
+        this.gitOps = new git_js_1.GitOperations(os.tmpdir());
+        // Import and configure the server logic here
+        // For now, we'll create a minimal server for testing
+        this.server = new index_js_1.Server({
+            name: 'test-memory-keeper',
+            version: '1.0.0',
+        }, {
+            capabilities: {
+                tools: {},
+            },
+        });
+    }
+    async callTool(name, args) {
+        // This would call the actual tool handler
+        // For now, returning a mock response
+        return {
+            content: [
+                {
+                    type: 'text',
+                    text: `Called ${name} with ${JSON.stringify(args)}`,
+                },
+            ],
+        };
+    }
+    cleanup() {
+        this.dbManager.close();
+        try {
+            fs.unlinkSync(this.tempDbPath);
+            fs.unlinkSync(`${this.tempDbPath}-wal`);
+            fs.unlinkSync(`${this.tempDbPath}-shm`);
+        }
+        catch (_e) {
+            // Ignore
+        }
+    }
+}
+exports.TestServer = TestServer;