AfterShip / clickhouse-sql-parser

54%

master: 54%

LAST BUILD ON BRANCH fix/surface-lexer-errors

branch: SELECT

CHANGE BRANCH

x

Sync Branches

• No branch selected
• 14-2
• add-benchmarks
• add-coverall
• add-modify-ttl
• add-more-badges
• add-valid-until-clause
• ajruckman_alter_table_delete_where
• ajruckman_placeholder_and_select_fix
• alert-autofix-1
• allow-string-as-tablename
• allow-var-start-with-dollar
• alter-role-settings
• alter-table-drop-detached
• case-statement-when-spacing
• change-sql-string-to-compact
• claude/frosty-wiles-784a80
• claude/issue-184-20250827-0341
• claude/romantic-kowalevski-5a7ec1
• codec-delta-level
• codex/fix-semantic-traversal
• copilot/add-beautify-format-alter-role
• copilot/add-beautify-format-for-create-user
• copilot/add-beautify-format-instruction
• copilot/add-beautify-format-support
• copilot/add-ident-break-for-insert
• copilot/add-ident-break-for-settings
• copilot/add-ident-break-from-expression
• copilot/add-ident-break-on-cluster
• copilot/add-support-in-partition-expression
• copilot/fix-12
• copilot/fix-140
• copilot/fix-179
• copilot/fix-181
• copilot/fix-187
• copilot/fix-75186bce-1dd8-4e67-bcd7-685469aa2574
• copilot/fix-e5e3e050-4a0c-4226-8f60-806cafedacba
• copilot/fix-stderr-routing-issue
• copilot/refactor-formatter-chaining-methods
• copilot/update-tool-version-to-0417
• dev
• dev1
• distinct-on
• feat/add-order-by-with-fill-support
• feat/alter_modify_query_support
• feat_support_db_comment
• feature/access-tuple-with-index
• feature/add-projection
• feature/add-union-distinct
• feature/adjust_engineExpr_String_printing_order
• feature/allow-select-modifier
• feature/alter-table-replace
• feature/check-expression
• feature/clear-projection-statement
• feature/column-def-alias-materialized
• feature/create-function
• feature/create-or-replace
• feature/create-role
• feature/delete-from
• feature/drop-projection
• feature/drop-user-role
• feature/grant-privilege
• feature/grouping-sets
• feature/insert-expr
• feature/insert-with-type-column
• feature/materialize-index-and-projection
• feature/optimize-expression
• feature/query-parameter
• feature/rename-table
• feature/select-in-function
• feature/support-json-type-skip
• feature/support-placeholder
• feature/support-ttl-policy
• feature/support-tuple
• feature/timestamp-interval
• fix-array-join-take-2
• fix-bugs-1
• fix-cannot-parse-join-only
• fix-database-column-name
• fix-describe-table
• fix-materialized-bug
• fix-no-as-alias
• fix-order-by-panic
• fix-panic-in-eof-error
• fix-select-limit-add-select-final
• fix/allow-parameters-for-null-engine
• fix/ast-walker
• fix/between-expr
• fix/between-visitor
• fix/cast-expr
• fix/column-alias-string
• fix/distinct-keyword-preservation
• fix/distinct-on-dot-notation
• fix/dollar-sign
• fix/dont_stop_walk_nil
• fix/double-quote-as-ident
• fix/expression-precedence
• fix/group-by-column-with-keyworld
• fix/insert-into
• fix/issue-118
• fix/issue-176
• fix/issue-187
• fix/issue-222
• fix/issues-108
• fix/keyword-as-column-in-select
• fix/keyword_alias
• fix/lexer-numbers-comments-utf8
• fix/lexer-unterminated-comment-hang
• fix/modify-ttl
• fix/multi-join
• fix/multi-line-comments
• fix/multi_distinct_except
• fix/nil-deref-error-formatting
• fix/npe-unexpected-token
• fix/panic-invalid-sql
• fix/parse-comment-literal
• fix/position-tracking
• fix/readme
• fix/remove-extra-leading-space
• fix/reserved-keyword-split
• fix/select-as-query
• fix/select-issues
• fix/string-as-table-ident
• fix/surface-lexer-errors
• fix/table-visitor
• fix/traversal-drift
• fix/union-all
• fix/walk_when_node
• fix/when_case_then
• fix/window_clause
• fix/wraperror-index-out-of-range
• fix/wrong-unary-expr-string
• ft-enhance-json-options
• improve-lexer
• improve-rename-syntax
• improve/parse-expr
• issue-204
• issue-23
• json-type-hints
• lexer-naming
• lexer-string-escapes
• master
• minor-refactor-column-type
• project-group-or-order
• public-api-additions
• refactor/allow-to-use-precedence
• refactor/expect-token-kind
• refactor/operation-name
• replicated-database-fix
• reserved-keyword-split
• role/alter
• role/on-cluster-expr
• select-format
• select-with-left-join
• settings-bool-parser
• sharad/add-comment-support-to-create-view
• sharad/bug_fix
• sharad/codec_parser
• sharad/codec_parser_v3
• sharad/delta_number_literal
• sharad/dictionary_v1
• sharad/fix-mv-comment-before-as
• sharad/fix-rmv-column-list-engine
• sharad/mv_definer
• sharad/named_parameter_v1
• sharad/table_comment
• structured-parse-errors
• support-concat-operator
• support-multiple-array-joins
• support-nested-tuple-definitions
• support-offset-without-limit
• support-refresh-in-mv
• support/table-alias-without-as
• swgillespie/alter-table-segv
• swgillespie/clickhouse-remotesecure
• test-automation
• traversal-drift-test
• unify-sql-terms
• update-readme

Build # 28775814529

Build Type

Pull #281

github

Committed by git-hulk

Commit Message

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>

Pull Request Pull Request #281: Surface lexer errors instead of misreporting them as EOF

Coverage Stats

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

2912 existing lines in 10 files now uncovered.

8953 of 16478 relevant lines covered (54.33%)

3025.41 hits per line