AfterShip / clickhouse-sql-parser / 28775814529

Pull #281

github

Fail ParseStmts when a recorded lexer error survives to the success path

Review follow-up: lastError was only consulted through wrapError, which
only runs when the parser itself fails. A failed lex can leave the
offset advanced - consumeIdent skips the opening backtick before
discovering the quote is unclosed - so with `SELECT 1 `bad` the
retained input re-lexes cleanly to EOF, parsing reaches the success
path, and the recorded error was silently dropped.

- ParseStmts now fails if lexerState still carries an error when
  parsing reaches end of input.
- peekToken restores the saved state on the error path too, so a failed
  lookahead neither leaves the offset advanced nor records an error for
  input the parser has not actually reached.

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

30 of 37 new or added lines in 3 files covered. (81.08%)

8953 of 16478 relevant lines covered (54.33%)

3025.41 hits per line