karellen / karellen-jdtls-kotlin / 23573584772

80%

push

github

Fix call hierarchy and bidirectional Java↔Kotlin property/getter interop (#12) (#13)

Three related fixes for cross-language call hierarchy and property search:

1. **Null declaring type NPE**: top-level Kotlin functions/properties now get
   a synthetic file-facade declaring type (`<FileName>Kt`), matching Kotlin's
   JVM compilation convention. Fixes outgoing call hierarchy NPE and incoming
   calls returning empty results.

2. **Forward: Java getter search → Kotlin property access**: `indexPropertyAccess()`
   emits METHOD_REF for `getX`/`isX`/`setX` when indexing property navigation.
   `derivePropertyName()` handles JDT's lowercase selectors. For non-standard
   getter capitalization (e.g., `getPPInsertTimer`), queries JDT METHOD_DECL
   index for actual getter names instead of guessing.

3. **Reverse: Kotlin property search → Java getter/setter calls**: when searching
   for FieldPattern references, runs supplementary `SearchEngine.search()` with
   MethodPattern for derived getter/setter names through the default Java
   participant only (avoids recursion).

Verified against real mixed Java/Kotlin projects.
275 integration tests, 87% instruction / 68% branch coverage.

1907 of 2767 branches covered (68.92%)

Branch coverage included in aggregate %.

262 of 291 new or added lines in 5 files covered. (90.03%)

4091 of 4817 relevant lines covered (84.93%)

3.05 hits per line