feat(lsp): implement get_hover and get_definitions for LSP runtime #1240

Merged
freemo merged 1 commits from feature/lsp-epic-completion into master 2026-04-02 16:52:34 +00:00

1 Commits

Author SHA1 Message Date
hamza.khyari b330958605 feat(lsp): implement get_hover and get_definitions for LSP runtime
CI / build (pull_request) Successful in 17s
CI / helm (pull_request) Successful in 22s
CI / lint (pull_request) Successful in 3m19s
CI / quality (pull_request) Successful in 3m42s
CI / typecheck (pull_request) Successful in 3m55s
CI / security (pull_request) Successful in 4m5s
CI / unit_tests (pull_request) Successful in 8m54s
CI / docker (pull_request) Successful in 1m18s
CI / e2e_tests (pull_request) Failing after 12m12s
CI / coverage (pull_request) Successful in 12m23s
CI / integration_tests (pull_request) Successful in 24m27s
CI / status-check (pull_request) Failing after 1s
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Successful in 54m54s
Add hover and definition support to LspClient and LspRuntime.

- LspClient.get_hover(): sends textDocument/hover, returns Hover dict
- LspClient.get_definitions(): sends textDocument/definition, handles
  Location, Location[], and LocationLink[] responses
- LspRuntime wrappers: input validation, file reading, language
  detection, 1-based to 0-based line/column conversion
- try/finally for did_close() safety on both new methods
- Tool adapter: HOVER and DEFINITIONS dispatch to runtime instead of
  raising LspNotAvailableError
- Updated lsp_tool_adapter_coverage test (HOVER -> REFERENCES)
- 21 Behave BDD scenarios with full path coverage

ISSUES CLOSED: #1243
2026-04-02 12:40:29 +00:00