Fix CTE double-processing - only register names, process in processIndirect

The CTE's inner SELECT was being processed twice:
1. In WITH clause processing (visiting CTE's SELECT)
2. In processIndirect when CTE is referenced

This caused issues because the annotated AST would have stale/conflicting
data from the first pass when processIndirect tried to analyze it.

Subqueries don't have this issue - they're processed only once in
processIndirect. CTEs should follow the same pattern: just register
the CTE name in the registry, and let processIndirect handle all the
actual SELECT processing when the CTE is referenced.

Author: claude

internal/stackql/astanalysis/earlyanalysis/ast_expand.go

@@ -216,18 +216,14 @@ func (v *indirectExpandAstVisitor) Visit(node sqlparser.SQLNode) error {
 		addIf(node.StraightJoinHint, sqlparser.StraightJoinHint)
 		addIf(node.SQLCalcFoundRows, sqlparser.SQLCalcFoundRowsStr)
 
-		// Process CTEs (Common Table Expressions) if present
+		// Register CTEs (Common Table Expressions) if present.
+		// We only register CTE names here - the actual SELECT processing
+		// happens in processIndirect when the CTE is referenced.
+		// This mirrors how subqueries work (processed once in processIndirect).
 		if node.With != nil {
 			for _, cte := range node.With.CTEs {
 				cteName := cte.Name.GetRawVal()
 				v.cteRegistry[cteName] = cte
-				// Process the CTE's select statement
-				if cte.Select != nil {
-					err := cte.Select.Accept(v)
-					if err != nil {
-						return err
-					}
-				}
 			}
 		}