cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.0k Pull Requests 115 Actions Packages Projects Releases Wiki Activity

fix(lsp): cleanup subprocess on failed initialization in StdioTransport.start() #11070

Closed
HAL9000 wants to merge 2 commits from fix/pr-11050-subprocess-cleanup into master
pull from: fix/pr-11050-subprocess-cleanup
merge into: cleveragents:master
Conversation 6 Commits 2 Files Changed 6
+78 -7

2 Commits

Author SHA1 Message Date
CleverAgents Bot 06f7a6928e ci: stop master workflow on PR updates
CI / lint (pull_request) Has been cancelled
Details
CI / typecheck (pull_request) Has been cancelled
Details
CI / security (pull_request) Has been cancelled
Details
CI / quality (pull_request) Has been cancelled
Details
CI / unit_tests (pull_request) Has been cancelled
Details
CI / integration_tests (pull_request) Has been cancelled
Details
CI / e2e_tests (pull_request) Has been cancelled
Details
CI / coverage (pull_request) Has been cancelled
Details
CI / build (pull_request) Has been cancelled
Details
CI / docker (pull_request) Has been cancelled
Details
CI / helm (pull_request) Has been cancelled
Details
CI / push-validation (pull_request) Has been cancelled
Details
CI / status-check (pull_request) Has been cancelled
Details 
Remove the stale pull_request trigger from master.yml so PR branch commits do not launch the master workflow.

Maintenance patch for PR #11070.
 2026-06-10 20:19:11 -04:00
HAL9000 5167d47ce6 fix(lsp): cleanup subprocess on failed initialization in StdioTransport.start()
CI / lint (pull_request) Failing after 1m2s
Details
CI / quality (pull_request) Successful in 1m19s
Details
CI / typecheck (pull_request) Successful in 1m32s
Details
CI / security (pull_request) Successful in 1m49s
Details
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / push-validation (pull_request) Successful in 29s
Details
CI / helm (pull_request) Successful in 40s
Details
CI / build (pull_request) Successful in 47s
Details
CI / benchmark-regression (pull_request) Failing after 1m10s
Details
CI / e2e_tests (pull_request) Failing after 4m6s
Details
CI / integration_tests (pull_request) Successful in 7m11s
Details
CI / unit_tests (pull_request) Failing after 9m34s
Details
CI / coverage (pull_request) Has been skipped
Details
CI / docker (pull_request) Has been skipped
Details
CI / status-check (pull_request) Failing after 4s
Details 
Ensure spawned processes are terminated if any exception occurs between
successful subprocess.Popen() and the end of Start(), preventing orphaned
LSP server processes.

The fix wraps post-Popen success logging in a try-except that calls self.stop()
to clean up the process before re-raising the exception.

ISSUES CLOSED: #11050
 2026-05-09 02:57:05 +00:00