Repository navigation
fix(server): keep running after Cursor shell spawn failures - #13862
Yash-Singh1 merged 4 commits into
Conversation
…aught-shell-spawn
Co-authored-by: Cursor <cursoragent@cursor.com>
ApprovabilityVerdict: Not approved Macroscope's review found this PR not approvable — The PR changes process-wide unhandled-rejection behavior in the server and adds diagnostic-suppression directives in both production and test files. The production guard is narrowly targeted and covered by subprocess tests, but these changes require human review. You can add or adjust custom eligibility rules. Learn more. |
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: unavailable · PR result: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
The guard runs before the SDK loads, outside an Effect runtime, so its stderr print and the probe deadline stay on Node's own APIs. Co-authored-by: Cursor <cursoragent@cursor.com>
What Changed
Catch shell spawn failures from the Cursor Agent SDK so they do not terminate the server process.
Why
A failed shell spawn could otherwise bring down the server while handling a Cursor session.
UI Changes
Not applicable.
Checklist