mcp-memory-keeper 0.12.0 → 0.12.1

package/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.12.1] - 2026-03-24
+
+### Fixed
+
+- **Server crashes with SQLITE_CANTOPEN when parent process CWD changes** (#31)
+  - Server now resolves database path to an absolute location (`$DATA_DIR` or `~/mcp-data/memory-keeper/`) instead of relying on CWD
+  - Added try/catch around data directory creation with actionable error message
+  - Startup warning with exact `cp` command when legacy `context.db` detected in CWD
+  - README "from source" install command now points to `bin/mcp-memory-keeper` instead of `node dist/index.js`
+  - Added Upgrading section documenting database path change and migration steps
+
+### Technical
+
+- Fixed integration tests to use `DATA_DIR` instead of dead `MCP_DB_PATH` environment variable
+- Fixed `git.init()` in tests to use `--initial-branch=master` for deterministic behavior
+
 ## [0.12.0] - 2026-02-06
 
 ### Added
@@ -491,7 +507,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Security**: Security updates
 - **Technical**: Internal improvements
 
-[Unreleased]: https://github.com/mkreyman/mcp-memory-keeper/compare/v0.12.0...HEAD
+[Unreleased]: https://github.com/mkreyman/mcp-memory-keeper/compare/v0.12.1...HEAD
+[0.12.1]: https://github.com/mkreyman/mcp-memory-keeper/compare/v0.12.0...v0.12.1
 [0.12.0]: https://github.com/mkreyman/mcp-memory-keeper/compare/v0.11.0...v0.12.0
 [0.11.0]: https://github.com/mkreyman/mcp-memory-keeper/compare/v0.10.2...v0.11.0
 [0.10.2]: https://github.com/mkreyman/mcp-memory-keeper/compare/v0.10.1...v0.10.2

package/README.md

@@ -174,7 +174,7 @@ npm install
 npm run build
 
 # 4. Add to Claude
-claude mcp add memory-keeper node /absolute/path/to/mcp-memory-keeper/dist/index.js
+claude mcp add memory-keeper /absolute/path/to/mcp-memory-keeper/bin/mcp-memory-keeper
 ```
 
 </details>
@@ -1212,6 +1212,38 @@ Test categories:
 - [ ] Custom context templates
 - [ ] Automatic retention policies
 
+## Upgrading
+
+### Database path change (v0.12.x+)
+
+Prior to this release, the server resolved `context.db` relative to the process's current working directory. The database now lives at an absolute path:
+
+- **Default:** `~/mcp-data/memory-keeper/context.db`
+- **Custom:** set `DATA_DIR=/your/path` — the server will use `$DATA_DIR/context.db`
+
+If you have existing data in a `context.db` in your old working directory, move it to the new location before restarting the server:
+
+```bash
+mkdir -p ~/mcp-data/memory-keeper
+cp /path/to/old/context.db ~/mcp-data/memory-keeper/context.db
+```
+
+If `DATA_DIR` is set, use that path as the destination instead of `~/mcp-data/memory-keeper/`.
+
+The server will print a warning to stderr if it detects a `context.db` in the current directory that differs from the configured data directory, including the exact `cp` command to run.
+
+### From-source install command change
+
+If you registered memory-keeper using `node dist/index.js` directly, update your MCP config to use the bin wrapper instead:
+
+```bash
+# remove the old entry
+claude mcp remove memory-keeper
+
+# add the updated entry
+claude mcp add memory-keeper /absolute/path/to/mcp-memory-keeper/bin/mcp-memory-keeper
+```
+
 ## Contributing
 
 Contributions are welcome! Please feel free to submit a Pull Request.

package/dist/__tests__/integration/git-integration.test.js

@@ -55,7 +55,7 @@ describe('Git Integration Tests', () => {
         // Create and initialize a real git repo for testing
         fs.mkdirSync(tempRepoPath, { recursive: true });
         git = (0, simple_git_1.simpleGit)(tempRepoPath);
-        await git.init();
+        await git.init(['--initial-branch=master']);
         await git.addConfig('user.name', 'Test User');
         await git.addConfig('user.email', 'test@example.com');
         // Use repo-local hooks directory to prevent global hooks from interfering

package/dist/__tests__/integration/server-initialization.test.js

@@ -86,12 +86,11 @@ const os = __importStar(require("os"));
         }
     });
     (0, globals_1.it)('should start server and respond to initialize request', done => {
-        const dbPath = path.join(tempDir, 'test.db');
         // Start the server
         serverProcess = (0, child_process_1.spawn)('node', [path.join(__dirname, '../../../dist/index.js')], {
             env: {
                 ...process.env,
-                MCP_DB_PATH: dbPath,
+                DATA_DIR: tempDir,
             },
             stdio: ['pipe', 'pipe', 'pipe'],
         });
@@ -161,6 +160,7 @@ const os = __importStar(require("os"));
             // Start the server
             serverProcess = (0, child_process_1.spawn)('node', [path.join(__dirname, '../../../dist/index.js')], {
                 stdio: ['pipe', 'pipe', 'pipe'],
+                env: { ...process.env, DATA_DIR: tempDir },
             });
             // Track for global cleanup
             if (!global.testProcesses) {
@@ -190,11 +190,10 @@ const os = __importStar(require("os"));
         }
     });
     (0, globals_1.it)('should handle invalid requests gracefully', done => {
-        const dbPath = path.join(tempDir, 'test.db');
         serverProcess = (0, child_process_1.spawn)('node', [path.join(__dirname, '../../../dist/index.js')], {
             env: {
                 ...process.env,
-                MCP_DB_PATH: dbPath,
+                DATA_DIR: tempDir,
             },
             stdio: ['pipe', 'pipe', 'pipe'],
         });
@@ -256,11 +255,10 @@ const os = __importStar(require("os"));
         }, 5000);
     });
     (0, globals_1.it)('should handle server shutdown gracefully', done => {
-        const dbPath = path.join(tempDir, 'test.db');
         serverProcess = (0, child_process_1.spawn)('node', [path.join(__dirname, '../../../dist/index.js')], {
             env: {
                 ...process.env,
-                MCP_DB_PATH: dbPath,
+                DATA_DIR: tempDir,
             },
             stdio: ['pipe', 'pipe', 'pipe'],
         });

package/dist/index.js

@@ -58,7 +58,26 @@ const token_limits_js_1 = require("./utils/token-limits.js");
 const contextWatchHandlers_js_1 = require("./handlers/contextWatchHandlers.js");
 const tool_profiles_js_1 = require("./utils/tool-profiles.js");
 // Initialize database with migrations
-const dbManager = new database_js_1.DatabaseManager({ filename: 'context.db' });
+const dataDir = process.env.DATA_DIR
+    ? path.resolve(process.env.DATA_DIR)
+    : path.join(os.homedir(), 'mcp-data', 'memory-keeper');
+try {
+    fs.mkdirSync(dataDir, { recursive: true });
+}
+catch (err) {
+    console.error(`[memory-keeper] FATAL: Cannot create data directory "${dataDir}": ${err.message}\n` +
+        `Set DATA_DIR to a writable location or create the directory manually.`);
+    process.exit(1);
+}
+// Warn users whose legacy DB is sitting in CWD
+const legacyDb = path.join(process.cwd(), 'context.db');
+if (process.cwd() !== dataDir && fs.existsSync(legacyDb)) {
+    console.error(`[memory-keeper] WARNING: context.db found in current directory but the ` +
+        `database now lives in ${dataDir}. ` +
+        `To preserve your data, run:\n` +
+        `  cp "${legacyDb}" "${path.join(dataDir, 'context.db')}"`);
+}
+const dbManager = new database_js_1.DatabaseManager({ filename: path.join(dataDir, 'context.db') });
 exports.dbManager = dbManager;
 const db = dbManager.getDatabase();
 // Initialize repository manager

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-memory-keeper",
-  "version": "0.12.0",
+  "version": "0.12.1",
   "description": "MCP server for persistent context management in AI coding assistants",
   "main": "dist/index.js",
   "bin": {

package/dist/__tests__/integration/issue-token-limit-channel-query.test.js

@@ -1,128 +0,0 @@
-"use strict";
-/**
- * Test for token limit issue with channel queries
- *
- * Reproduces the bug where context_get with limit: 50 still exceeds token limits
- * when querying a channel with large items.
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-const globals_1 = require("@jest/globals");
-const database_js_1 = require("../../utils/database.js");
-const RepositoryManager_js_1 = require("../../repositories/RepositoryManager.js");
-const token_limits_js_1 = require("../../utils/token-limits.js");
-(0, globals_1.describe)('Token Limit with Channel Query Bug', () => {
-    let dbManager;
-    let repositories;
-    let sessionId;
-    (0, globals_1.beforeEach)(() => {
-        dbManager = new database_js_1.DatabaseManager({ filename: ':memory:' });
-        repositories = new RepositoryManager_js_1.RepositoryManager(dbManager);
-        const session = repositories.sessions.create({
-            name: 'Test Session',
-            defaultChannel: 'test-channel',
-        });
-        sessionId = session.id;
-    });
-    (0, globals_1.afterEach)(() => {
-        dbManager.close();
-    });
-    (0, globals_1.it)('should enforce token limits even when limit parameter is provided', () => {
-        // Create 50 large context items (each ~1000 chars)
-        const largeValue = 'x'.repeat(1000);
-        for (let i = 0; i < 50; i++) {
-            repositories.contexts.save(sessionId, {
-                key: `large-item-${i}`,
-                value: largeValue,
-                channel: 'outbound-call-center',
-                priority: 'normal',
-            });
-        }
-        // Query with limit: 50
-        const result = repositories.contexts.queryEnhanced({
-            sessionId,
-            channel: 'outbound-call-center',
-            limit: 50,
-            sort: 'updated_desc',
-            includeMetadata: false,
-        });
-        (0, globals_1.expect)(result.items.length).toBe(50);
-        // Check if response would exceed token limit
-        const tokenConfig = (0, token_limits_js_1.getTokenConfig)();
-        const { exceedsLimit, safeItemCount } = (0, token_limits_js_1.checkTokenLimit)(result.items, false, tokenConfig);
-        // Build the actual response structure
-        const response = {
-            items: result.items,
-            pagination: {
-                total: result.totalCount,
-                returned: result.items.length,
-                offset: 0,
-                hasMore: false,
-                nextOffset: null,
-                truncated: false,
-                truncatedCount: 0,
-            },
-        };
-        const responseJson = JSON.stringify(response, null, 2);
-        const actualTokens = Math.ceil(responseJson.length / tokenConfig.charsPerToken);
-        // The bug: even with limit: 50, the response can exceed token limits
-        if (actualTokens > tokenConfig.mcpMaxTokens) {
-            // BUG REPRODUCED: The response exceeds token limits
-            // Verify that checkTokenLimit correctly detected the issue
-            (0, globals_1.expect)(exceedsLimit).toBe(true);
-            (0, globals_1.expect)(safeItemCount).toBeLessThan(50);
-            (0, globals_1.expect)(actualTokens).toBeGreaterThan(tokenConfig.mcpMaxTokens);
-        }
-    });
-    (0, globals_1.it)('should respect token limits over user-provided limit parameter', () => {
-        // Create 100 large context items (each ~800 chars)
-        const largeValue = 'y'.repeat(800);
-        for (let i = 0; i < 100; i++) {
-            repositories.contexts.save(sessionId, {
-                key: `item-${i}`,
-                value: largeValue,
-                channel: 'test-channel',
-                priority: 'normal',
-            });
-        }
-        // Query with limit: 50 (user expectation)
-        const result = repositories.contexts.queryEnhanced({
-            sessionId,
-            channel: 'test-channel',
-            limit: 50,
-            sort: 'created_desc',
-            includeMetadata: false,
-        });
-        // Simulate the context_get handler logic
-        const tokenConfig = (0, token_limits_js_1.getTokenConfig)();
-        const { exceedsLimit, safeItemCount } = (0, token_limits_js_1.checkTokenLimit)(result.items, false, tokenConfig);
-        let actualItems = result.items;
-        let wasTruncated = false;
-        if (exceedsLimit && safeItemCount < result.items.length) {
-            actualItems = result.items.slice(0, safeItemCount);
-            wasTruncated = true;
-        }
-        // Build response
-        const response = {
-            items: actualItems,
-            pagination: {
-                total: result.totalCount,
-                returned: actualItems.length,
-                offset: 0,
-                hasMore: wasTruncated || actualItems.length < result.totalCount,
-                nextOffset: wasTruncated ? actualItems.length : null,
-                truncated: wasTruncated,
-                truncatedCount: wasTruncated ? result.items.length - actualItems.length : 0,
-            },
-        };
-        const responseJson = JSON.stringify(response, null, 2);
-        const actualTokens = Math.ceil(responseJson.length / tokenConfig.charsPerToken);
-        // Verify token limit is not exceeded
-        (0, globals_1.expect)(actualTokens).toBeLessThanOrEqual(tokenConfig.mcpMaxTokens);
-        // Verify truncation occurred if needed
-        if (exceedsLimit) {
-            (0, globals_1.expect)(wasTruncated).toBe(true);
-            (0, globals_1.expect)(actualItems.length).toBeLessThan(50);
-            (0, globals_1.expect)(actualItems.length).toBe(safeItemCount);
-        }
-    });
-});