DrRataplan / xq-lsp / 30455340044

93%

push

github

fix: stop retaining full ASTs for every glob-matched file to prevent OOM

Cross-file features (find-references, rename, call-hierarchy, code-lens
counts) previously parsed and permanently cached a full AST for every file
matched by lsp-config.xq's glob. xq-parser's operator-precedence grammar
wraps even a bare variable reference in ~30 layers of wrapper nodes, so the
retained trees run ~2 orders of magnitude larger than the source text —
enough to OOM the server on a workspace with a few hundred glob-matched
files.

Glob files are now cached forever only as lightweight, ast-free summaries
(functions/variables/imports/namespace decls + text) for cheap candidate
discovery. Cross-file occurrence search pre-filters candidates with a plain
text substring check, then parses the full AST on demand for the (usually
small) subset that survives the filter, discarding it right after instead
of retaining it.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

1456 of 1725 branches covered (84.41%)

Branch coverage included in aggregate %.

64 of 64 new or added lines in 3 files covered. (100.0%)

5424 of 5651 relevant lines covered (95.98%)

319266.46 hits per line