Method statement_needs_graph_context

graphlite/src/exec/executor.rs:2724–2788

Single consolidated method to check if a statement requires graph context This replaces all the scattered *_needs_graph_context methods for a clean PostgreSQL-style approach

(&self, statement: &crate::ast::Statement)

    /// Single consolidated method to check if a statement requires graph context
    /// This replaces all the scattered *_needs_graph_context methods for a clean PostgreSQL-style approach
    fn statement_needs_graph_context(&self, statement: &crate::ast::Statement) -> bool {
        use crate::ast::Statement;

        match statement {
            Statement::Query(_) => {
                // Query graph context requirements are now determined by the validator
                // This method should not be used for Query statements - use the validator's flag instead
                // Returning true here for backward compatibility, but caller should use validator flag
                true
            }
            Statement::DataStatement(_) => {
                // Data statements (INSERT, DELETE, etc.) always need graph context
                true
            }
            Statement::Call(call_stmt) => {
                // Check if the specific procedure needs graph context
                self.procedure_needs_graph_context(&call_stmt.procedure_name)
            }
            Statement::Select(select_stmt) => {
                // SELECT statements may or may not need graph context
                self.select_statement_needs_graph_context(select_stmt)
            }
            Statement::SessionStatement(_) => {
                // Session statements (SET SESSION, etc.) don't need graph context
                false
            }
            Statement::CatalogStatement(_) => {
                // Catalog statements don't need graph context
                false
            }
            Statement::Declare(_) => {
                // Cursor declarations don't need graph context
                false
            }
            Statement::Next(_) => {
                // NEXT statements may need graph context if they reference graph data
                true
            }
            Statement::AtLocation(_) => {
                // AT statements need graph context for location resolution
                true
            }
            Statement::TransactionStatement(_) => {
                // Transaction statements don't need graph context
                false
            }
            Statement::ProcedureBody(procedure_body) => {
                // Procedure body needs graph context if any of its statements need it
                self.statement_needs_graph_context(&procedure_body.initial_statement)
                    || procedure_body
                        .chained_statements
                        .iter()
                        .any(|chained| self.statement_needs_graph_context(&chained.statement))
            }
            Statement::IndexStatement(_) => {
                // Index DDL statements don't need graph context
                false
            }