Compare commits
3 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 613f0e9795 | |||
| f1162ed4a4 | |||
| 8039ada222 |
@@ -10,7 +10,7 @@
|
|||||||
"plugins": [
|
"plugins": [
|
||||||
{
|
{
|
||||||
"name": "claude-mem",
|
"name": "claude-mem",
|
||||||
"version": "10.2.1",
|
"version": "10.2.2",
|
||||||
"source": "./plugin",
|
"source": "./plugin",
|
||||||
"description": "Persistent memory system for Claude Code - context compression across sessions"
|
"description": "Persistent memory system for Claude Code - context compression across sessions"
|
||||||
}
|
}
|
||||||
|
|||||||
+12
-65
@@ -2,6 +2,18 @@
|
|||||||
|
|
||||||
All notable changes to claude-mem.
|
All notable changes to claude-mem.
|
||||||
|
|
||||||
|
## [v10.2.1] - 2026-02-16
|
||||||
|
|
||||||
|
## Bug Fixes
|
||||||
|
|
||||||
|
- **Bun install & sharp native modules**: Fixed stale native module cache issues on Bun updates, added `node-addon-api` as a dev dependency required by sharp (#1140)
|
||||||
|
- **PendingMessageStore consolidation**: Deduplicated PendingMessageStore initialization in worker-service; added session-scoped filtering to `resetStaleProcessingMessages` to prevent cross-session message resets (#1140)
|
||||||
|
- **Gemini empty response handling**: Fixed silent message deletion when Gemini returns empty summary responses — now logs a warning and preserves the original message (#1138)
|
||||||
|
- **Idle timeout session scoping**: Fixed idle timeout handler to only reset messages for the timed-out session instead of globally resetting all sessions (#1138)
|
||||||
|
- **Shell injection in sync-marketplace**: Replaced `execSync` with `spawnSync` for rsync calls to eliminate command injection via gitignore patterns (#1138)
|
||||||
|
- **Sharp cache invalidation**: Added cache clearing for sharp's native bindings when Bun version changes (#1138)
|
||||||
|
- **Marketplace install**: Switched marketplace sync from npm to bun for package installation consistency (#1140)
|
||||||
|
|
||||||
## [v10.1.0] - 2026-02-16
|
## [v10.1.0] - 2026-02-16
|
||||||
|
|
||||||
## SessionStart System Message & Cleaner Defaults
|
## SessionStart System Message & Cleaner Defaults
|
||||||
@@ -1411,68 +1423,3 @@ Patch release v8.2.4
|
|||||||
- Update hooks.json to use worker-service.cjs CLI
|
- Update hooks.json to use worker-service.cjs CLI
|
||||||
- Add comprehensive tests for hook constants and worker spawn functionality
|
- Add comprehensive tests for hook constants and worker spawn functionality
|
||||||
|
|
||||||
## [v8.2.1] - 2025-12-27
|
|
||||||
|
|
||||||
## 🔧 Worker Lifecycle Hardening
|
|
||||||
|
|
||||||
This patch release addresses critical bugs discovered during PR review of the self-spawn pattern introduced in 8.2.0. The worker daemon now handles edge cases robustly across both Unix and Windows platforms.
|
|
||||||
|
|
||||||
### 🐛 Critical Bug Fixes
|
|
||||||
|
|
||||||
#### Process Exit Detection Fixed
|
|
||||||
The `waitForProcessesExit` function was crashing when processes exited during monitoring. The `process.kill(pid, 0)` call throws when a process no longer exists, which was not being caught. Now wrapped in try/catch to correctly identify exited processes.
|
|
||||||
|
|
||||||
#### Spawn PID Validation
|
|
||||||
The worker daemon now validates that `spawn()` actually returned a valid PID before writing to the PID file. Previously, spawn failures could leave invalid PID files that broke subsequent lifecycle operations.
|
|
||||||
|
|
||||||
#### Cross-Platform Orphan Cleanup
|
|
||||||
- **Unix**: Replaced single `kill` command with individual `process.kill()` calls wrapped in try/catch, so one already-exited process doesn't abort cleanup of remaining orphans
|
|
||||||
- **Windows**: Wrapped `taskkill` calls in try/catch for the same reason
|
|
||||||
|
|
||||||
#### Health Check Reliability
|
|
||||||
Changed `waitForHealth` to use the `/api/readiness` endpoint (returns 503 until fully initialized) instead of just checking if the port is in use. Callers now wait for *actual* worker readiness, not just network availability.
|
|
||||||
|
|
||||||
### 🔄 Refactoring
|
|
||||||
|
|
||||||
#### Code Consolidation (-580 lines)
|
|
||||||
Deleted obsolete process management infrastructure that was replaced by the self-spawn pattern:
|
|
||||||
- `src/services/process/ProcessManager.ts` (433 lines) - PID management now in worker-service
|
|
||||||
- `src/cli/worker-cli.ts` (81 lines) - CLI handling now in worker-service
|
|
||||||
- `src/services/worker-wrapper.ts` (157 lines) - Replaced by `--daemon` flag
|
|
||||||
|
|
||||||
#### Updated Hook Commands
|
|
||||||
All hooks now use `worker-service.cjs` CLI directly instead of the deleted `worker-cli.js`.
|
|
||||||
|
|
||||||
### ⏱️ Timeout Adjustments
|
|
||||||
|
|
||||||
Increased timeouts throughout for compatibility with slow systems:
|
|
||||||
|
|
||||||
| Component | Before | After |
|
|
||||||
|-----------|--------|-------|
|
|
||||||
| Default hook timeout | 120s | 300s |
|
|
||||||
| Health check timeout | 1s | 30s |
|
|
||||||
| Health check retries | 15 | 300 |
|
|
||||||
| Context initialization | 30s | 300s |
|
|
||||||
| MCP connection | 15s | 300s |
|
|
||||||
| PowerShell commands | 5s | 60s |
|
|
||||||
| Git commands | 30s | 300s |
|
|
||||||
| NPM install | 120s | 600s |
|
|
||||||
| Hook worker commands | 30s | 180s |
|
|
||||||
|
|
||||||
### 🧪 Testing
|
|
||||||
|
|
||||||
Added comprehensive test suites:
|
|
||||||
- `tests/hook-constants.test.ts` - Validates timeout configurations
|
|
||||||
- `tests/worker-spawn.test.ts` - Tests worker CLI and health endpoints
|
|
||||||
|
|
||||||
### 🛡️ Additional Robustness
|
|
||||||
|
|
||||||
- PID validation in restart command (matches start command behavior)
|
|
||||||
- Try/catch around `forceKillProcess()` for graceful shutdown
|
|
||||||
- Try/catch around `getChildProcesses()` for Windows failures
|
|
||||||
- Improved logging for PID file operations and HTTP shutdown
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
**Full Changelog**: https://github.com/thedotmack/claude-mem/compare/v8.2.0...v8.2.1
|
|
||||||
|
|
||||||
|
|||||||
+1
-2
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "claude-mem",
|
"name": "claude-mem",
|
||||||
"version": "10.2.1",
|
"version": "10.2.2",
|
||||||
"description": "Memory compression system for Claude Code - persist context across sessions",
|
"description": "Memory compression system for Claude Code - persist context across sessions",
|
||||||
"keywords": [
|
"keywords": [
|
||||||
"claude",
|
"claude",
|
||||||
@@ -118,7 +118,6 @@
|
|||||||
"@types/react": "^18.3.5",
|
"@types/react": "^18.3.5",
|
||||||
"@types/react-dom": "^18.3.0",
|
"@types/react-dom": "^18.3.0",
|
||||||
"esbuild": "^0.27.2",
|
"esbuild": "^0.27.2",
|
||||||
"node-addon-api": "^8.5.0",
|
|
||||||
"np": "^11.0.2",
|
"np": "^11.0.2",
|
||||||
"tsx": "^4.20.6",
|
"tsx": "^4.20.6",
|
||||||
"typescript": "^5.3.0"
|
"typescript": "^5.3.0"
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "claude-mem",
|
"name": "claude-mem",
|
||||||
"version": "10.2.1",
|
"version": "10.2.2",
|
||||||
"description": "Persistent memory system for Claude Code - seamlessly preserve context across sessions",
|
"description": "Persistent memory system for Claude Code - seamlessly preserve context across sessions",
|
||||||
"author": {
|
"author": {
|
||||||
"name": "Alex Newman"
|
"name": "Alex Newman"
|
||||||
|
|||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "claude-mem-plugin",
|
"name": "claude-mem-plugin",
|
||||||
"version": "10.2.1",
|
"version": "10.2.2",
|
||||||
"private": true,
|
"private": true,
|
||||||
"description": "Runtime dependencies for claude-mem bundled hooks",
|
"description": "Runtime dependencies for claude-mem bundled hooks",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
|
|||||||
File diff suppressed because one or more lines are too long
File diff suppressed because one or more lines are too long
@@ -5,7 +5,7 @@
|
|||||||
* Ensures Bun runtime and uv (Python package manager) are installed
|
* Ensures Bun runtime and uv (Python package manager) are installed
|
||||||
* (auto-installs if missing) and handles dependency installation when needed.
|
* (auto-installs if missing) and handles dependency installation when needed.
|
||||||
*/
|
*/
|
||||||
import { existsSync, readFileSync, writeFileSync, rmSync } from 'fs';
|
import { existsSync, readFileSync, writeFileSync } from 'fs';
|
||||||
import { execSync, spawnSync } from 'child_process';
|
import { execSync, spawnSync } from 'child_process';
|
||||||
import { join } from 'path';
|
import { join } from 'path';
|
||||||
import { homedir } from 'os';
|
import { homedir } from 'os';
|
||||||
@@ -260,18 +260,17 @@ function installDeps() {
|
|||||||
|
|
||||||
console.error('📦 Installing dependencies with Bun...');
|
console.error('📦 Installing dependencies with Bun...');
|
||||||
|
|
||||||
// Clear stale native module cache (sharp/libvips) to prevent broken dylib references.
|
|
||||||
// Bun's cache can retain native binaries that reference companion libraries at
|
|
||||||
// broken relative paths after version upgrades.
|
|
||||||
const bunCacheImgDir = join(homedir(), '.bun', 'install', 'cache', '@img');
|
|
||||||
if (existsSync(bunCacheImgDir)) {
|
|
||||||
console.error(' Clearing stale native module cache (@img/sharp)...');
|
|
||||||
rmSync(bunCacheImgDir, { recursive: true, force: true });
|
|
||||||
}
|
|
||||||
|
|
||||||
// Quote path for Windows paths with spaces
|
// Quote path for Windows paths with spaces
|
||||||
const bunCmd = IS_WINDOWS && bunPath.includes(' ') ? `"${bunPath}"` : bunPath;
|
const bunCmd = IS_WINDOWS && bunPath.includes(' ') ? `"${bunPath}"` : bunPath;
|
||||||
|
|
||||||
|
// Clear Bun's package cache to prevent stale native module artifacts
|
||||||
|
try {
|
||||||
|
execSync(`${bunCmd} pm cache rm`, { cwd: ROOT, stdio: 'pipe', shell: IS_WINDOWS });
|
||||||
|
console.error(' Cleared Bun package cache');
|
||||||
|
} catch {
|
||||||
|
// Cache may not exist yet on first install
|
||||||
|
}
|
||||||
|
|
||||||
execSync(`${bunCmd} install`, { cwd: ROOT, stdio: 'inherit', shell: IS_WINDOWS });
|
execSync(`${bunCmd} install`, { cwd: ROOT, stdio: 'inherit', shell: IS_WINDOWS });
|
||||||
|
|
||||||
// Write version marker
|
// Write version marker
|
||||||
|
|||||||
@@ -80,22 +80,12 @@ try {
|
|||||||
{ stdio: 'inherit' }
|
{ stdio: 'inherit' }
|
||||||
);
|
);
|
||||||
|
|
||||||
// Remove stale lockfiles before install — they pin old native dep versions
|
// Clear Bun's package cache to prevent stale native module artifacts
|
||||||
const { unlinkSync, rmSync } = require('fs');
|
try {
|
||||||
for (const lockfile of ['package-lock.json', 'bun.lock']) {
|
execSync('bun pm cache rm', { cwd: INSTALLED_PATH, stdio: 'pipe' });
|
||||||
const lockpath = path.join(INSTALLED_PATH, lockfile);
|
console.log('Cleared Bun package cache');
|
||||||
if (existsSync(lockpath)) {
|
} catch {
|
||||||
unlinkSync(lockpath);
|
// Cache may not exist yet on first install
|
||||||
console.log(`Removed stale ${lockfile}`);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Clear stale native module cache (sharp/libvips) — Bun's cache can retain
|
|
||||||
// native binaries that reference companion libraries at broken relative paths
|
|
||||||
const bunCacheImgDir = path.join(os.homedir(), '.bun', 'install', 'cache', '@img');
|
|
||||||
if (existsSync(bunCacheImgDir)) {
|
|
||||||
rmSync(bunCacheImgDir, { recursive: true, force: true });
|
|
||||||
console.log('Cleared stale native module cache (@img/sharp)');
|
|
||||||
}
|
}
|
||||||
|
|
||||||
console.log('Running bun install in marketplace...');
|
console.log('Running bun install in marketplace...');
|
||||||
|
|||||||
Reference in New Issue
Block a user