Skip to content

Fix Kotlin class references without a language server - #52

Merged
rcosteira79 merged 1 commit into
mainfrom
fix/kotlin-class-references
Nov 13, 2026
Merged

rcosteira79 merged 1 commit into
mainfrom
fix/kotlin-class-references

Conversation

@rcosteira79

Copy link
Copy Markdown
Collaborator

Clicking a Kotlin class such as LeaveAllowance still fell back to text search without an external language server: the existing built-in resolver only understood same-file properties, functions and parameters.

Add bounded Kotlin type lookup across repository source files. It follows packages, explicit and aliased imports, qualified names and a single wildcard scope. It keeps unrelated same-named types and lexical shadows separate, respects file-private declarations, and leaves duplicate declarations (including KMP expect/actual pairs) ambiguous. PNGs and other non-Kotlin files do not consume the source budget. Fallback messages now also explain the built-in lookup's limits.

This provides syntax candidates, not compiler or Gradle resolution. Dependency sources, receiver inference, nested/local types and KMP source-set selection still need a project-aware provider; incomplete parsing and scan limits remain visible.

Validation:

  • Full local verification: 1,203 passed, 13 skipped, zero failures.
  • Focused Kotlin and symbol-peek regression tests: 25 passed, plus typecheck.
  • Compiled CLI: isolated review API resolved a class and its aliased import without a language server.
  • Read-only check against the reported repository: found LeaveAllowance at its declaration and 30 reference locations. The live tour was not changed or restarted.

@rcosteira79
rcosteira79 merged commit c9329ec into main Nov 13, 2026
14 checks passed
@rcosteira79
rcosteira79 deleted the fix/kotlin-class-references branch November 13, 2026 11:52
@rcosteira79 rcosteira79 mentioned this pull request Nov 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant