Teamwork / kommentaar / 28527927887

55%

master: 55%

Build Type

Pull #115

github

Committed by shane-tw

Commit Message

docparse: reuse FindComments ASTs in type resolution

Two optimisations to reduce redundant file parsing:

1. Cross-phase AST reuse: FindComments already parses every controller
   file to extract endpoint comments. Those ASTs were discarded, causing
   getDecls to re-parse the same files from disk when resolving types.
   A sync.Map (astFileCache) now caches *ast.File by absolute path during
   FindComments; getDecls checks this cache before calling parser.ParseFile.

2. Parallel parsing in getDecls: files not already in astFileCache are now
   parsed concurrently (same WaitGroup pattern used in FindComments) rather
   than sequentially via goutil.ParseFiles. This halves the wall-clock cost
   of first-time lookups into large dependency packages.

Combined, these changes eliminate the dominant double-parse bottleneck: all
files scanned by FindComments are resolved in getDecls at zero I/O cost,
and uncached dependency packages are parsed in parallel instead of serially.

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

Pull Request Pull Request #115: docparse: parse source files in parallel

Coverage Stats

100 of 115 new or added lines in 1 file covered. (86.96%)

1 existing line in 1 file now uncovered.

1255 of 2296 relevant lines covered (54.66%)

40.79 hits per line