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
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:
@@ -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
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user