Skip to content

fix: add Kotlin call extraction#107

Merged
tirth8205 merged 2 commits intotirth8205:mainfrom
gzenz:fix/kotlin-calls
Apr 8, 2026
Merged

fix: add Kotlin call extraction#107
tirth8205 merged 2 commits intotirth8205:mainfrom
gzenz:fix/kotlin-calls

Conversation

@gzenz
Copy link
Copy Markdown
Contributor

@gzenz gzenz commented Apr 5, 2026

Summary

  • Kotlin files produced zero CALLS edges because _get_call_name() didn't handle Kotlin's tree-sitter node types
  • Added simple_identifier to the simple call check (Kotlin uses this instead of identifier)
  • Added navigation_expression to member types with navigation_suffix traversal for method calls

Test plan

  • TestKotlinParsing::test_finds_calls -- println() detected, repo.save() detected
  • Full suite: 566 passed, 0 failed

…ssion)

Kotlin files produced zero CALLS edges because _get_call_name() didn't
handle Kotlin's tree-sitter node types: simple_identifier for plain
function calls and navigation_expression for method calls (obj.method).
- Add _TEST_ANNOTATIONS set for JUnit @test, @ParameterizedTest, etc.
- Extract annotations from modifiers child (Java/Kotlin/C#) and
  decorated_definition parent (Python) in _extract_functions()
- Pass decorators to _is_test_function() for annotation-based detection
- Add *Test.kt and *Test.java to _TEST_FILE_PATTERNS
- Add tests for Java @test, Kotlin @test, and Python test detection
@gzenz gzenz marked this pull request as draft April 5, 2026 21:59
@tirth8205 tirth8205 marked this pull request as ready for review April 8, 2026 19:30
@tirth8205 tirth8205 merged commit 3710f19 into tirth8205:main Apr 8, 2026
9 checks passed
loc4atnt pushed a commit to loc4atnt/code-review-graph that referenced this pull request Apr 9, 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.

2 participants