karellen / karellen-lsp-mcp / 23716738629

72%

push

github

[release] Apply per-tool timeout to LSP request timeout (#27)

## Summary

- Per-tool `timeout` parameter now also controls the LSP JSON-RPC
request timeout, not just the readiness/indexing wait
- Previously, a user specifying `timeout: 300` would wait up to 300s for
indexing but only 60s for the actual LSP call to return
- Uses `_request_timeout_override` on the client, set for the duration
of the dispatch and cleared in a `finally` block
- Refactored `_handle_lsp_request` to separate timeout setup from
dispatch logic (`_dispatch_lsp`)

## Test plan

- [x] 291 unit tests pass
- [x] flake8 clean
- [x] CI passes

683 of 1040 branches covered (65.67%)

Branch coverage included in aggregate %.

13 of 13 new or added lines in 2 files covered. (100.0%)

2271 of 2913 relevant lines covered (77.96%)

6.89 hits per line