refactor(binder): consolidate binder alias semantics and split type_check (#19750)

* refactor(ast): add leave hooks to visitor traversal

* test(sql): add binder coverage for prebind edge cases

* refactor(sql): streamline binder clause analysis

* refactor(sql): add name-resolution normalize helper

* refactor(sql): split semantic type checker internals

* fix(sql): preserve GROUP BY alias resolution

* fix(sql): narrow GROUP BY alias resolution

* refactor(sql): consolidate binder alias facts

* fix(sql): restore result cache state calls after merge

* refine

Author: forsaken628

src/query/ast/src/visit.rs

@@ -90,7 +90,7 @@ pub enum VisitControl<B = ()> {
 
 pub type VisitResult = Result<VisitControl, !>;
 
-/// Pre-order visitor hooks for immutable AST traversal.
+/// Visitor hooks for immutable AST traversal.
 ///
 /// Returning `Continue` lets the walker descend into children automatically.
 /// Returning `SkipChildren` stops descent for the current node.
@@ -113,6 +113,10 @@ pub trait Visitor {
         Ok(VisitControl::Continue)
     }
 
+    fn leave_query(&mut self, _query: &Query) -> Result<(), Self::Error> {
+        Ok(())
+    }
+
     fn visit_set_expr(
         &mut self,
         _set_expr: &SetExpr,
@@ -138,6 +142,10 @@ pub trait Visitor {
         Ok(VisitControl::Continue)
     }
 
+    fn leave_expr(&mut self, _expr: &Expr) -> Result<(), Self::Error> {
+        Ok(())
+    }
+
     fn visit_function_call(
         &mut self,
         _call: &FunctionCall,
@@ -160,7 +168,7 @@ pub trait Visitor {
     }
 }
 
-/// Pre-order visitor hooks for mutable AST traversal.
+/// Visitor hooks for mutable AST traversal.
 ///
 /// Returning `Continue` lets the walker descend into children automatically.
 /// Returning `SkipChildren` stops descent for the current node.
@@ -186,6 +194,10 @@ pub trait VisitorMut {
         Ok(VisitControl::Continue)
     }
 
+    fn leave_query(&mut self, _query: &mut Query) -> Result<(), Self::Error> {
+        Ok(())
+    }
+
     fn visit_set_expr(
         &mut self,
         _set_expr: &mut SetExpr,
@@ -211,6 +223,10 @@ pub trait VisitorMut {
         Ok(VisitControl::Continue)
     }
 
+    fn leave_expr(&mut self, _expr: &mut Expr) -> Result<(), Self::Error> {
+        Ok(())
+    }
+
     fn visit_function_call(
         &mut self,
         _call: &mut FunctionCall,