Fix 23 todo tests: BETWEEN transformation and float literal parsing

- Transform NOT BETWEEN into or(less, greater) to match ClickHouse EXPLAIN output
- Transform BETWEEN into and(greaterOrEquals, lessOrEquals)
- Fix float literal parsing for trailing dot syntax (e.g., 1. is now Float64)
- Update ast.json golden file for float literal change
- Remove todo flag from 23 now-passing tests

Fixed tests include polygon operations, geohash constants, decimal/float
handling, and BETWEEN expression tests.

Author: claude

internal/explain/functions.go

@@ -229,17 +229,37 @@ func explainLikeExpr(sb *strings.Builder, n *ast.LikeExpr, indent string, depth
 }
 
 func explainBetweenExpr(sb *strings.Builder, n *ast.BetweenExpr, indent string, depth int) {
-	// BETWEEN is represented as Function and with two comparisons
-	// But for explain, we can use a simpler form
-	fnName := "between"
 	if n.Not {
-		fnName = "notBetween"
+		// NOT BETWEEN is transformed to: expr < low OR expr > high
+		// Represented as: Function or with two comparisons: less and greater
+		fmt.Fprintf(sb, "%sFunction or (children %d)\n", indent, 1)
+		fmt.Fprintf(sb, "%s ExpressionList (children %d)\n", indent, 2)
+		// less(expr, low)
+		fmt.Fprintf(sb, "%s  Function less (children %d)\n", indent, 1)
+		fmt.Fprintf(sb, "%s   ExpressionList (children %d)\n", indent, 2)
+		Node(sb, n.Expr, depth+4)
+		Node(sb, n.Low, depth+4)
+		// greater(expr, high)
+		fmt.Fprintf(sb, "%s  Function greater (children %d)\n", indent, 1)
+		fmt.Fprintf(sb, "%s   ExpressionList (children %d)\n", indent, 2)
+		Node(sb, n.Expr, depth+4)
+		Node(sb, n.High, depth+4)
+	} else {
+		// BETWEEN is represented as Function and with two comparisons
+		// expr >= low AND expr <= high
+		fmt.Fprintf(sb, "%sFunction and (children %d)\n", indent, 1)
+		fmt.Fprintf(sb, "%s ExpressionList (children %d)\n", indent, 2)
+		// greaterOrEquals(expr, low)
+		fmt.Fprintf(sb, "%s  Function greaterOrEquals (children %d)\n", indent, 1)
+		fmt.Fprintf(sb, "%s   ExpressionList (children %d)\n", indent, 2)
+		Node(sb, n.Expr, depth+4)
+		Node(sb, n.Low, depth+4)
+		// lessOrEquals(expr, high)
+		fmt.Fprintf(sb, "%s  Function lessOrEquals (children %d)\n", indent, 1)
+		fmt.Fprintf(sb, "%s   ExpressionList (children %d)\n", indent, 2)
+		Node(sb, n.Expr, depth+4)
+		Node(sb, n.High, depth+4)
 	}
-	fmt.Fprintf(sb, "%sFunction %s (children %d)\n", indent, fnName, 1)
-	fmt.Fprintf(sb, "%s ExpressionList (children %d)\n", indent, 3)
-	Node(sb, n.Expr, depth+2)
-	Node(sb, n.Low, depth+2)
-	Node(sb, n.High, depth+2)
 }
 
 func explainIsNullExpr(sb *strings.Builder, n *ast.IsNullExpr, indent string, depth int) {

lexer/lexer.go

@@ -594,16 +594,21 @@ func (l *Lexer) readNumber() Item {
 		}
 	}
 
-	// Check for decimal point
-	if l.ch == '.' && unicode.IsDigit(l.peekChar()) {
-		sb.WriteRune(l.ch)
-		l.readChar()
-		for unicode.IsDigit(l.ch) {
+	// Check for decimal point (either followed by digit or end of number like 1.)
+	if l.ch == '.' {
+		nextCh := l.peekChar()
+		// Allow 1. (trailing dot with no digits) and 1.5 (dot with digits)
+		// But not 1.something (identifier-like)
+		if unicode.IsDigit(nextCh) || (!isIdentStart(nextCh) && nextCh != '.') {
 			sb.WriteRune(l.ch)
 			l.readChar()
-			// Handle underscore separators
-			for l.ch == '_' && unicode.IsDigit(l.peekChar()) {
+			for unicode.IsDigit(l.ch) {
+				sb.WriteRune(l.ch)
 				l.readChar()
+				// Handle underscore separators
+				for l.ch == '_' && unicode.IsDigit(l.peekChar()) {
+					l.readChar()
+				}
 			}
 		}
 	}
@@ -675,15 +680,20 @@ func (l *Lexer) readNumberOrIdent() Item {
 		}
 	}
 
-	// Check for decimal point
-	if l.ch == '.' && unicode.IsDigit(l.peekChar()) {
-		sb.WriteRune(l.ch)
-		l.readChar()
-		for unicode.IsDigit(l.ch) {
+	// Check for decimal point (either followed by digit or end of number like 1.)
+	if l.ch == '.' {
+		nextCh := l.peekChar()
+		// Allow 1. (trailing dot with no digits) and 1.5 (dot with digits)
+		// But not 1.something (identifier-like)
+		if unicode.IsDigit(nextCh) || (!isIdentStart(nextCh) && nextCh != '.') {
 			sb.WriteRune(l.ch)
 			l.readChar()
-			for l.ch == '_' && unicode.IsDigit(l.peekChar()) {
+			for unicode.IsDigit(l.ch) {
+				sb.WriteRune(l.ch)
 				l.readChar()
+				for l.ch == '_' && unicode.IsDigit(l.peekChar()) {
+					l.readChar()
+				}
 			}
 		}
 	}

parser/testdata/00320_between/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/00500_point_in_polygon_nan/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/00834_not_between/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/01300_polygon_convex_hull/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/01300_svg/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/01300_wkt/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/01301_polygons_within/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/01302_polygons_distance/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}