fix(lsp): restore explicit _process cleanup in OSError handler
CI / push-validation (pull_request) Successful in 34s
CI / helm (pull_request) Successful in 59s
CI / lint (pull_request) Failing after 1m9s
CI / build (pull_request) Successful in 1m10s
CI / quality (pull_request) Successful in 1m46s
CI / security (pull_request) Successful in 2m3s
CI / typecheck (pull_request) Successful in 2m8s
CI / integration_tests (pull_request) Successful in 4m39s
CI / unit_tests (pull_request) Failing after 6m29s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 4s

PR branch HEAD was using try-except-pass around stop() without an
explicit self._process = None assignment in the OSError exception
handler.  This corrected it to match master and the FileNotFoundError
pattern: direct stop() followed by explicit _process = None.

Closes #10597
This commit is contained in:
2026-05-15 03:51:56 +00:00
parent 90f6f7087c
commit ca189e1b67
+2 -4
View File
@@ -128,10 +128,8 @@ class StdioTransport:
# raised (e.g. exec in a child fails), so we must terminate # raised (e.g. exec in a child fails), so we must terminate
# and wait to avoid leaking a zombie or orphan. # and wait to avoid leaking a zombie or orphan.
if self._process is not None: if self._process is not None:
try: self.stop()
self.stop() self._process = None
except Exception: # noqa: BLE001 -- best-effort cleanup
pass
from cleveragents.lsp.errors import LspError from cleveragents.lsp.errors import LspError