1
52%
master: 52%

Ran 10 Jun 2026 06:48AM UTC

Files 15

Run time 1s

Badge

Embed ▾

Committed 10 Jun 2026 06:46AM UTC coverage: 51.79% (+0.1%) from 51.662%

Job # 27258515509.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix nil-pointer dereference when formatting errors at EOF (#273)

Error messages built with `fmt.Errorf("... %q", p.last().String)` panic when
the lexer is at EOF, because `p.last()` is nil. Inputs like `ALTER `,
`SELECT*FROM A(0A` and `SET A=` triggered this.

Add a nil-safe `lastTokenString()` helper (mirroring `lastTokenKind()`) that
returns "<EOF>" when there is no current token, and use it in the 11 error
paths in parser_table.go and parser_alter.go. Success paths where last() is
provably non-nil are left untouched.

Add the inputs above to TestParser_InvalidSyntax to reproduce: they panicked
before the fix and now return a proper error.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Coverage Stats

8291 of 16009 relevant lines covered (51.79%)

3014.6 hits per line

AfterShip / clickhouse-sql-parser / 27258515509 / 1
52%
master: 52%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 27258515509.1

AfterShip / clickhouse-sql-parser / 27258515509 / 1 52% master: 52%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 27258515509.1

AfterShip / clickhouse-sql-parser / 27258515509 / 1
52%
master: 52%

README BADGES
x