Rod Boev
418e38ee46
Reduce timeouts to eliminate 10-30s startup delay when worker is dead (common on WSL2 after hibernate). Add stale PID detection, graceful error handling across all handlers, and error classification that distinguishes worker unavailability from handler bugs. - HEALTH_CHECK 30s→3s, new POST_SPAWN_WAIT (5s), PORT_IN_USE_WAIT (3s) - isProcessAlive() with EPERM handling, cleanStalePidFile() - getPluginVersion() try-catch for shutdown race (#1042) - isWorkerUnavailableError: transport+5xx+429→exit 0, 4xx→exit 2 - No-op handler for unknown event types (#984) - Wrap all handler fetch calls in try-catch for graceful degradation - CLAUDE_MEM_HEALTH_TIMEOUT_MS env var override with validation
67 lines
2.5 KiB
TypeScript
67 lines
2.5 KiB
TypeScript
/**
|
|
* Event Handler Factory
|
|
*
|
|
* Returns the appropriate handler for a given event type.
|
|
*/
|
|
|
|
import type { EventHandler } from '../types.js';
|
|
import { HOOK_EXIT_CODES } from '../../shared/hook-constants.js';
|
|
import { contextHandler } from './context.js';
|
|
import { sessionInitHandler } from './session-init.js';
|
|
import { observationHandler } from './observation.js';
|
|
import { summarizeHandler } from './summarize.js';
|
|
import { userMessageHandler } from './user-message.js';
|
|
import { fileEditHandler } from './file-edit.js';
|
|
import { sessionCompleteHandler } from './session-complete.js';
|
|
|
|
export type EventType =
|
|
| 'context' // SessionStart - inject context
|
|
| 'session-init' // UserPromptSubmit - initialize session
|
|
| 'observation' // PostToolUse - save observation
|
|
| 'summarize' // Stop - generate summary (phase 1)
|
|
| 'session-complete' // Stop - complete session (phase 2) - fixes #842
|
|
| 'user-message' // SessionStart (parallel) - display to user
|
|
| 'file-edit'; // Cursor afterFileEdit
|
|
|
|
const handlers: Record<EventType, EventHandler> = {
|
|
'context': contextHandler,
|
|
'session-init': sessionInitHandler,
|
|
'observation': observationHandler,
|
|
'summarize': summarizeHandler,
|
|
'session-complete': sessionCompleteHandler,
|
|
'user-message': userMessageHandler,
|
|
'file-edit': fileEditHandler
|
|
};
|
|
|
|
/**
|
|
* Get the event handler for a given event type.
|
|
*
|
|
* Returns a no-op handler for unknown event types instead of throwing (fix #984).
|
|
* Claude Code may send new event types that the plugin doesn't handle yet —
|
|
* throwing would surface as a BLOCKING_ERROR to the user.
|
|
*
|
|
* @param eventType The type of event to handle
|
|
* @returns The appropriate EventHandler, or a no-op handler for unknown types
|
|
*/
|
|
export function getEventHandler(eventType: string): EventHandler {
|
|
const handler = handlers[eventType as EventType];
|
|
if (!handler) {
|
|
console.error(`[claude-mem] Unknown event type: ${eventType}, returning no-op`);
|
|
return {
|
|
async execute() {
|
|
return { continue: true, suppressOutput: true, exitCode: HOOK_EXIT_CODES.SUCCESS };
|
|
}
|
|
};
|
|
}
|
|
return handler;
|
|
}
|
|
|
|
// Re-export individual handlers for direct access if needed
|
|
export { contextHandler } from './context.js';
|
|
export { sessionInitHandler } from './session-init.js';
|
|
export { observationHandler } from './observation.js';
|
|
export { summarizeHandler } from './summarize.js';
|
|
export { userMessageHandler } from './user-message.js';
|
|
export { fileEditHandler } from './file-edit.js';
|
|
export { sessionCompleteHandler } from './session-complete.js';
|