1. AST is a recursive data structure, so `astChunk` is redundant. 
2. I agree, `extractedErrors` is a better name.
3. The intent was to not modify `error`, but I'm thinking now a better convention might be to have one copy of `extractedErrors` and modify as we go along. It's a much less complicated structure than the AST.