Skip to content

WIP: analysis: Add ASTTypeAnnotator.jl#620

Open
aviatesk wants to merge 1 commit intomasterfrom
avi/ASTTypeAnnotator
Open

WIP: analysis: Add ASTTypeAnnotator.jl#620
aviatesk wants to merge 1 commit intomasterfrom
avi/ASTTypeAnnotator

Conversation

@aviatesk
Copy link
Copy Markdown
Owner

@aviatesk aviatesk commented Apr 4, 2026

Implement the ASTTypeAnnotator pipeline, which takes a SyntaxTree, generates CodeInfo from it, and performs type annotation via abstract interpretation.
By maintaining a reverse mapping from CodeInfo back to SyntaxTree, we can trace from a specific statement in CodeInfo to its source representation.

This is needed for implementing type on hover, inlay type hints, and property completions, etc.

Marked as WIP since there are still many edge cases and inference result caching is not yet implemented.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 4, 2026

Codecov Report

❌ Patch coverage is 0% with 179 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.72%. Comparing base (2323784) to head (518d4a6).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/analysis/ASTTypeAnnotator.jl 0.00% 157 Missing ⚠️
src/utils/inference.jl 0.00% 22 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #620      +/-   ##
==========================================
- Coverage   70.08%   68.72%   -1.36%     
==========================================
  Files          52       54       +2     
  Lines        9111     9290     +179     
==========================================
  Hits         6385     6385              
- Misses       2726     2905     +179     
Flag Coverage Δ
JETLS.jl 68.72% <0.00%> (-1.36%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aviatesk aviatesk force-pushed the avi/ASTTypeAnnotator branch 3 times, most recently from 1058286 to 609a5c4 Compare April 19, 2026 04:41
Implement the `ASTTypeAnnotator` pipeline, which takes a `SyntaxTree`,
generates `CodeInfo` from it, and performs type annotation via abstract
interpretation.
By maintaining a reverse mapping from `CodeInfo` back to `SyntaxTree`,
we can trace from a specific statement in `CodeInfo` to its source
representation.

This is needed for implementing type on hover, inlay type hints, and
property completions, etc.

Marked as WIP since there are still many edge cases and inference
result caching is not yet implemented.
@aviatesk aviatesk force-pushed the avi/ASTTypeAnnotator branch from 609a5c4 to 518d4a6 Compare April 21, 2026 12:29
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