karellen / karellen-lsp-mcp / 23701943211

72%

Build Type

push

github

Committed by web-flow

Commit Message

[release] Convert jdtls jdt:// URIs to standard jar:file:// URIs (#26)

## Summary

- Add `normalize_uri()` hook to `LspNormalizer` base class, overridden
in `JdtlsNormalizer` to convert proprietary `jdt://` URIs to standard
`jar:file://` URIs
- Parse JAR file path and class reference from jdt:// query string
(backslash-escaped paths, `<`/`(` encoded package/class separators)
- Thread normalizer through all daemon parse functions:
`_parse_locations`, `_parse_call_hierarchy`, `_parse_type_hierarchy`,
`_make_tree_node`, `_walk_tree`, `_parse_workspace_symbols`
- Expose `normalizer` property on `LspClient`
- Example:
`jdt://contents/yavi-0.9.1.jar/...?=...%5C/path%5C/to%5C/yavi-0.9.1.jar=.../%3Cam.ik.yavi.core%28ConstraintViolations.class`
→
`jar:file:///path/to/yavi-0.9.1.jar!/am/ik/yavi/core/ConstraintViolations.class`

## Test plan

- [x] 6 new unit tests for `_jdt_uri_to_jar_uri` and `normalize_uri`
(291 total, all passing)
- [x] flake8 clean
- [x] CI passes
- [x] Integration test with jdtls confirms jar:file:// URIs in call
hierarchy results

Coverage Stats

681 of 1038 branches covered (65.61%)

Branch coverage included in aggregate %.

47 of 52 new or added lines in 3 files covered. (90.38%)

8 existing lines in 1 file now uncovered.

2260 of 2902 relevant lines covered (77.88%)

6.88 hits per line