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
34 lines
1.4 KiB
TypeScript
34 lines
1.4 KiB
TypeScript
export const HOOK_TIMEOUTS = {
|
|
DEFAULT: 300000, // Standard HTTP timeout (5 min for slow systems)
|
|
HEALTH_CHECK: 3000, // Worker health check (3s — healthy worker responds in <100ms)
|
|
POST_SPAWN_WAIT: 5000, // Wait for daemon to start after spawn (starts in <1s on Linux)
|
|
PORT_IN_USE_WAIT: 3000, // Wait when port occupied but health failing
|
|
WORKER_STARTUP_WAIT: 1000,
|
|
PRE_RESTART_SETTLE_DELAY: 2000, // Give files time to sync before restart
|
|
POWERSHELL_COMMAND: 10000, // PowerShell process enumeration (10s - typically completes in <1s)
|
|
WINDOWS_MULTIPLIER: 1.5 // Platform-specific adjustment for hook-side operations
|
|
} as const;
|
|
|
|
/**
|
|
* Hook exit codes for Claude Code
|
|
*
|
|
* Exit code behavior per Claude Code docs:
|
|
* - 0: Success. For SessionStart/UserPromptSubmit, stdout added to context.
|
|
* - 2: Blocking error. For SessionStart, stderr shown to user only.
|
|
* - Other non-zero: stderr shown in verbose mode only.
|
|
*/
|
|
export const HOOK_EXIT_CODES = {
|
|
SUCCESS: 0,
|
|
FAILURE: 1,
|
|
/** Blocking error - for SessionStart, shows stderr to user only */
|
|
BLOCKING_ERROR: 2,
|
|
/** Show stderr to user only, don't inject into context. Used by user-message handler (Cursor). */
|
|
USER_MESSAGE_ONLY: 3,
|
|
} as const;
|
|
|
|
export function getTimeout(baseTimeout: number): number {
|
|
return process.platform === 'win32'
|
|
? Math.round(baseTimeout * HOOK_TIMEOUTS.WINDOWS_MULTIPLIER)
|
|
: baseTimeout;
|
|
}
|