eliott/claude-mem
1
0
Fork 0
mirror of https://github.com/thedotmack/claude-mem synced 2026-04-25 17:15:04 +02:00
Code Issues Packages Projects Releases Wiki Activity

refactor: remove startWorker function and adjust installation flow

Browse Source
This commit is contained in:
Alex Newman
2025-11-07 18:00:42 -05:00
parent 30a42036aa
commit 6204fe9b9d
1 changed files with 5 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
scripts/smart-install.js
View File

@@ -209,39 +209,6 @@ function runNpmInstall() {
} }
} }
function startWorker() {
const ECOSYSTEM_CONFIG = join(PLUGIN_ROOT, 'ecosystem.config.cjs');
const PM2_PATH = join(PLUGIN_ROOT, 'node_modules', '.bin', 'pm2');
log('🚀 Starting worker service...', colors.dim);
try {
// Use the full path to PM2 to avoid PATH issues on Windows
// PM2 will either start it or report it's already running (both are success cases)
execSync(`"${PM2_PATH}" start "${ECOSYSTEM_CONFIG}"`, {
cwd: PLUGIN_ROOT,
stdio: 'pipe', // Capture output to avoid clutter
encoding: 'utf-8',
});
log('✓ Worker service ready', colors.dim);
return true;
} catch (error) {
// PM2 errors are often non-critical (e.g., "already running")
// Don't fail the entire setup if worker start has issues
log(`⚠️ Worker startup issue (non-critical): ${error.message}`, colors.yellow);
// Check if it's just because worker is already running
if (error.message && (error.message.includes('already') || error.message.includes('exist'))) {
log('✓ Worker was already running', colors.dim);
return true;
}
return false;
}
}
async function main() { async function main() {
try { try {
// Check if we need to install dependencies // Check if we need to install dependencies
@@ -253,17 +220,16 @@ async function main() {
if (!installSuccess) { if (!installSuccess) {
log('', colors.red); log('', colors.red);
log('⚠️ Installation failed - worker startup may fail', colors.yellow); log('⚠️ Installation failed', colors.yellow);
log('', colors.reset); log('', colors.reset);
// Don't exit - still try to start worker with existing deps process.exit(1);
} }
} }
// Always start/ensure worker is running // Worker will be started lazily when needed (e.g., when save-hook sends data)
// This runs whether we installed deps or not // Context hook only needs database access, not the worker service
startWorker();
// Success - dependencies installed (if needed) and worker running // Success - dependencies installed (if needed)
process.exit(0); process.exit(0);
} catch (error) { } catch (error) {