Remove process exit signal handler

[miguelg719]

why

process.exit() calls and signal handlers in BaseCache.ts were causing immediate process termination, interfering with applications' graceful shutdown logic that waits for active jobs to complete.

what changed

• Removed SIGINT/SIGTERM handlers from BaseCache.ts
• Removed process.exit() calls from BaseCache.ts and lib/index.ts
• Kept passive 'exit' event handler for lock cleanup

test plan

[changeset-bot]

🦋 Changeset detected

Latest commit: 45ee10c

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[greptile-apps]

Greptile Summary

• Removed process exit handlers that were interfering with application-level graceful shutdown logic
• Applications using Stagehand as a library can now manage their own signal handlers and shutdown sequence

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• Changes correctly remove problematic process exit handlers while preserving cleanup functionality, following best practices for libraries
• No files require special attention

Important Files Changed

Filename	Overview
lib/cache/BaseCache.ts	Removed SIGINT/SIGTERM handlers and uncaughtException handler, kept passive exit event handler for lock cleanup
lib/index.ts	Removed process.exit(0) call from signal handler cleanup finally block

Sequence Diagram

sequenceDiagram
    participant App as "Application"
    participant Stagehand as "Stagehand"
    participant BaseCache as "BaseCache"
    participant Process as "Process"
    
    App->>Stagehand: new Stagehand()
    Stagehand->>BaseCache: new BaseCache()
    BaseCache->>Process: on("exit", releaseLockOnExit)
    Note over BaseCache,Process: Only passive exit handler<br/>No SIGINT/SIGTERM handlers
    
    App->>Process: SIGINT/SIGTERM
    Note over App: Application handles signal<br/>Controls shutdown timing
    App->>Stagehand: close()
    Stagehand->>BaseCache: releaseLock()
    BaseCache->>Process: Remove lock file
    Process->>Process: "exit" event fires
    Process->>BaseCache: releaseLockOnExit()
    Note over Process: Application controls process.exit()

Loading

[greptile-apps]

_{2 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format}