Skip to content

diagnostic: Add lowering/undef-global-var for on-change feedback#450

Merged
aviatesk merged 1 commit intomasterfrom
diagnostic-onchange-undefvar
Jan 6, 2026
Merged

diagnostic: Add lowering/undef-global-var for on-change feedback#450
aviatesk merged 1 commit intomasterfrom
diagnostic-onchange-undefvar

Conversation

@aviatesk
Copy link
Owner

@aviatesk aviatesk commented Jan 6, 2026

Screen.Recording.2026-01-06.at.19.12.37.mov

Add a new lowering/undef-global-var diagnostic that reports undefined global variable references on document change. This provides faster feedback compared to inference/undef-global-var, which runs on save.

The on-change diagnostic uses JuliaLowering's variable analysis to detect simple undefined references with accurate position information (exact character range). The on-save version detects a superset of cases including qualified references like Base.undefvar, but reports position on a line basis.

Add a new `lowering/undef-global-var` diagnostic that reports undefined
global variable references on document change. This provides faster
feedback compared to `inference/undef-global-var`, which runs on save.

The on-change diagnostic uses JuliaLowering's variable analysis to
detect simple undefined references with accurate position information
(exact character range). The on-save version detects a superset of
cases including qualified references like `Base.undefvar`, but reports
position on a line basis.
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

❌ Patch coverage is 92.85714% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.63%. Comparing base (23d007b) to head (8dacd57).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
src/diagnostic.jl 92.85% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #450      +/-   ##
==========================================
+ Coverage   65.53%   65.63%   +0.10%     
==========================================
  Files          44       44              
  Lines        5724     5742      +18     
==========================================
+ Hits         3751     3769      +18     
  Misses       1973     1973              
Flag Coverage Δ
JETLS.jl 65.63% <92.85%> (+0.10%) ⬆️

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 merged commit 4070a90 into master Jan 6, 2026
12 of 13 checks passed
@aviatesk aviatesk deleted the diagnostic-onchange-undefvar branch January 6, 2026 10:15
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