fiddler_script/

parser.rs

//! Parser for FiddlerScript.
//!
//! This module implements a recursive descent parser with Pratt parsing
//! for expression precedence handling.
//!
//! The parser transforms a token stream into an Abstract Syntax Tree (AST).

use crate::ast::{BinaryOp, Block, ElseClause, Expression, Program, Statement, UnaryOp};
use crate::error::ParseError;
use crate::lexer::{Token, TokenKind};

/// The parser for FiddlerScript.
pub struct Parser {
    /// The tokens to parse
    tokens: Vec<Token>,
    /// Current position in the token stream
    current: usize,
}

impl Parser {
    /// Create a new parser for the given tokens.
    pub fn new(tokens: Vec<Token>) -> Self {
        Self { tokens, current: 0 }
    }

    /// Parse a complete program.
    pub fn parse(&mut self) -> Result<Program, ParseError> {
        let mut statements = Vec::new();

        while !self.is_at_end() {
            statements.push(self.parse_statement()?);
        }

        Ok(Program::new(statements))
    }

    // === Helper methods ===

    /// Check if we've reached the end of the token stream.
    fn is_at_end(&self) -> bool {
        self.peek().kind == TokenKind::Eof
    }

    /// Get the current token without consuming it.
    fn peek(&self) -> &Token {
        self.tokens.get(self.current).unwrap_or_else(|| {
            self.tokens
                .last()
                .expect("Token stream should have at least EOF")
        })
    }

    /// Get the previous token.
    fn previous(&self) -> &Token {
        &self.tokens[self.current.saturating_sub(1)]
    }

    /// Advance to the next token and return the current one.
    fn advance(&mut self) -> &Token {
        if !self.is_at_end() {
            self.current += 1;
        }
        self.previous()
    }

    /// Check if the current token matches the given kind.
    fn check(&self, kind: &TokenKind) -> bool {
        if self.is_at_end() {
            return false;
        }
        std::mem::discriminant(&self.peek().kind) == std::mem::discriminant(kind)
    }

    /// Consume the current token if it matches the given kind.
    fn match_token(&mut self, kind: &TokenKind) -> bool {
        if self.check(kind) {
            self.advance();
            true
        } else {
            false
        }
    }

    /// Expect the current token to match the given kind, or return an error.
    fn expect(&mut self, kind: &TokenKind, expected: &str) -> Result<&Token, ParseError> {
        if self.check(kind) {
            Ok(self.advance())
        } else {
            Err(ParseError::UnexpectedToken(
                self.peek().kind.to_string(),
                self.peek().position,
                expected.to_string(),
            ))
        }
    }

    // === Statement parsing ===

    /// Parse a statement.
    fn parse_statement(&mut self) -> Result<Statement, ParseError> {
        match &self.peek().kind {
            TokenKind::Let => self.parse_let_statement(),
            TokenKind::If => self.parse_if_statement(),
            TokenKind::For => self.parse_for_statement(),
            TokenKind::Return => self.parse_return_statement(),
            TokenKind::Fn => self.parse_function_definition(),
            TokenKind::LeftBrace => {
                // Check if this is a dictionary literal (expression) or a block statement
                if self.is_dictionary_literal() {
                    self.parse_expression_statement()
                } else {
                    let block = self.parse_block()?;
                    Ok(Statement::Block(block))
                }
            }
            _ => self.parse_expression_statement(),
        }
    }

    /// Parse a let statement: `let x = expr;`
    fn parse_let_statement(&mut self) -> Result<Statement, ParseError> {
        let position = self.peek().position;
        self.advance(); // consume 'let'

        let name = match &self.peek().kind {
            TokenKind::Identifier(name) => name.clone(),
            _ => return Err(ParseError::ExpectedIdentifier(self.peek().position)),
        };
        self.advance();

        self.expect(&TokenKind::Assign, "=")?;
        let value = self.parse_expression()?;
        self.expect(&TokenKind::Semicolon, ";")?;

        Ok(Statement::Let {
            name,
            value,
            position,
        })
    }

    /// Parse an if statement: `if (condition) { ... } else { ... }`
    fn parse_if_statement(&mut self) -> Result<Statement, ParseError> {
        let position = self.peek().position;
        self.advance(); // consume 'if'

        self.expect(&TokenKind::LeftParen, "(")?;
        let condition = self.parse_expression()?;
        self.expect(&TokenKind::RightParen, ")")?;

        let then_block = self.parse_block()?;

        let else_block = if self.match_token(&TokenKind::Else) {
            if self.check(&TokenKind::If) {
                // else if
                Some(ElseClause::ElseIf(Box::new(self.parse_if_statement()?)))
            } else {
                // else block
                Some(ElseClause::Block(self.parse_block()?))
            }
        } else {
            None
        };

        Ok(Statement::If {
            condition,
            then_block,
            else_block,
            position,
        })
    }

    /// Parse a for statement: `for (init; condition; update) { ... }`
    fn parse_for_statement(&mut self) -> Result<Statement, ParseError> {
        let position = self.peek().position;
        self.advance(); // consume 'for'

        self.expect(&TokenKind::LeftParen, "(")?;

        // Init clause
        let init = if self.match_token(&TokenKind::Semicolon) {
            None
        } else if self.check(&TokenKind::Let) {
            Some(Box::new(self.parse_let_statement()?))
        } else {
            let expr = self.parse_expression()?;
            let pos = expr.position();
            self.expect(&TokenKind::Semicolon, ";")?;
            Some(Box::new(Statement::Expression {
                expression: expr,
                position: pos,
            }))
        };

        // Condition clause
        let condition = if self.check(&TokenKind::Semicolon) {
            None
        } else {
            Some(self.parse_expression()?)
        };
        self.expect(&TokenKind::Semicolon, ";")?;

        // Update clause
        let update = if self.check(&TokenKind::RightParen) {
            None
        } else {
            Some(self.parse_expression()?)
        };
        self.expect(&TokenKind::RightParen, ")")?;

        let body = self.parse_block()?;

        Ok(Statement::For {
            init,
            condition,
            update,
            body,
            position,
        })
    }

    /// Parse a return statement: `return expr;`
    fn parse_return_statement(&mut self) -> Result<Statement, ParseError> {
        let position = self.peek().position;
        self.advance(); // consume 'return'

        let value = if self.check(&TokenKind::Semicolon) {
            None
        } else {
            Some(self.parse_expression()?)
        };

        self.expect(&TokenKind::Semicolon, ";")?;

        Ok(Statement::Return { value, position })
    }

    /// Parse a function definition: `fn name(params) { ... }`
    fn parse_function_definition(&mut self) -> Result<Statement, ParseError> {
        let position = self.peek().position;
        self.advance(); // consume 'fn'

        let name = match &self.peek().kind {
            TokenKind::Identifier(name) => name.clone(),
            _ => return Err(ParseError::ExpectedIdentifier(self.peek().position)),
        };
        self.advance();

        self.expect(&TokenKind::LeftParen, "(")?;
        let params = self.parse_parameters()?;
        self.expect(&TokenKind::RightParen, ")")?;

        let body = self.parse_block()?;

        Ok(Statement::Function {
            name,
            params,
            body,
            position,
        })
    }

    /// Parse function parameters.
    fn parse_parameters(&mut self) -> Result<Vec<String>, ParseError> {
        let mut params = Vec::new();

        if !self.check(&TokenKind::RightParen) {
            loop {
                let pos = self.peek().position;
                let name = match &self.peek().kind {
                    TokenKind::Identifier(name) => name.clone(),
                    _ => return Err(ParseError::ExpectedIdentifier(self.peek().position)),
                };
                self.advance();

                // Check for duplicate parameter
                if params.contains(&name) {
                    return Err(ParseError::UnexpectedToken(
                        name,
                        pos,
                        "unique parameter name (duplicate parameter)".to_string(),
                    ));
                }

                params.push(name);

                if !self.match_token(&TokenKind::Comma) {
                    break;
                }
            }
        }

        Ok(params)
    }

    /// Parse an expression statement: `expr;`
    fn parse_expression_statement(&mut self) -> Result<Statement, ParseError> {
        let expression = self.parse_expression()?;
        let position = expression.position();
        self.expect(&TokenKind::Semicolon, ";")?;
        Ok(Statement::Expression {
            expression,
            position,
        })
    }

    /// Parse a block: `{ statements }`
    fn parse_block(&mut self) -> Result<Block, ParseError> {
        let position = self.peek().position;
        self.expect(&TokenKind::LeftBrace, "{")?;

        let mut statements = Vec::new();
        while !self.check(&TokenKind::RightBrace) && !self.is_at_end() {
            statements.push(self.parse_statement()?);
        }

        self.expect(&TokenKind::RightBrace, "}")?;
        Ok(Block::new(statements, position))
    }

    // === Expression parsing (Pratt parsing) ===

    /// Parse an expression.
    pub fn parse_expression(&mut self) -> Result<Expression, ParseError> {
        self.parse_assignment()
    }

    /// Parse assignment expression.
    fn parse_assignment(&mut self) -> Result<Expression, ParseError> {
        let expr = self.parse_or()?;

        if self.match_token(&TokenKind::Assign) {
            let position = self.previous().position;
            let value = self.parse_assignment()?;

            match expr {
                Expression::Identifier { name, .. } => {
                    return Ok(Expression::Assignment {
                        name,
                        value: Box::new(value),
                        position,
                    });
                }
                _ => return Err(ParseError::InvalidAssignmentTarget(position)),
            }
        }

        Ok(expr)
    }

    /// Parse logical OR expression.
    fn parse_or(&mut self) -> Result<Expression, ParseError> {
        let mut left = self.parse_and()?;

        while self.match_token(&TokenKind::Or) {
            let position = self.previous().position;
            let right = self.parse_and()?;
            left = Expression::Binary {
                left: Box::new(left),
                operator: BinaryOp::Or,
                right: Box::new(right),
                position,
            };
        }

        Ok(left)
    }

    /// Parse logical AND expression.
    fn parse_and(&mut self) -> Result<Expression, ParseError> {
        let mut left = self.parse_equality()?;

        while self.match_token(&TokenKind::And) {
            let position = self.previous().position;
            let right = self.parse_equality()?;
            left = Expression::Binary {
                left: Box::new(left),
                operator: BinaryOp::And,
                right: Box::new(right),
                position,
            };
        }

        Ok(left)
    }

    /// Parse equality expression.
    fn parse_equality(&mut self) -> Result<Expression, ParseError> {
        let mut left = self.parse_comparison()?;

        loop {
            let op = if self.match_token(&TokenKind::Equal) {
                BinaryOp::Equal
            } else if self.match_token(&TokenKind::NotEqual) {
                BinaryOp::NotEqual
            } else {
                break;
            };

            let position = self.previous().position;
            let right = self.parse_comparison()?;
            left = Expression::Binary {
                left: Box::new(left),
                operator: op,
                right: Box::new(right),
                position,
            };
        }

        Ok(left)
    }

    /// Parse comparison expression.
    fn parse_comparison(&mut self) -> Result<Expression, ParseError> {
        let mut left = self.parse_addition()?;

        loop {
            let op = if self.match_token(&TokenKind::LessThan) {
                BinaryOp::LessThan
            } else if self.match_token(&TokenKind::LessEqual) {
                BinaryOp::LessEqual
            } else if self.match_token(&TokenKind::GreaterThan) {
                BinaryOp::GreaterThan
            } else if self.match_token(&TokenKind::GreaterEqual) {
                BinaryOp::GreaterEqual
            } else {
                break;
            };

            let position = self.previous().position;
            let right = self.parse_addition()?;
            left = Expression::Binary {
                left: Box::new(left),
                operator: op,
                right: Box::new(right),
                position,
            };
        }

        Ok(left)
    }

    /// Parse addition/subtraction expression.
    fn parse_addition(&mut self) -> Result<Expression, ParseError> {
        let mut left = self.parse_multiplication()?;

        loop {
            let op = if self.match_token(&TokenKind::Plus) {
                BinaryOp::Add
            } else if self.match_token(&TokenKind::Minus) {
                BinaryOp::Subtract
            } else {
                break;
            };

            let position = self.previous().position;
            let right = self.parse_multiplication()?;
            left = Expression::Binary {
                left: Box::new(left),
                operator: op,
                right: Box::new(right),
                position,
            };
        }

        Ok(left)
    }

    /// Parse multiplication/division/modulo expression.
    fn parse_multiplication(&mut self) -> Result<Expression, ParseError> {
        let mut left = self.parse_unary()?;

        loop {
            let op = if self.match_token(&TokenKind::Star) {
                BinaryOp::Multiply
            } else if self.match_token(&TokenKind::Slash) {
                BinaryOp::Divide
            } else if self.match_token(&TokenKind::Percent) {
                BinaryOp::Modulo
            } else {
                break;
            };

            let position = self.previous().position;
            let right = self.parse_unary()?;
            left = Expression::Binary {
                left: Box::new(left),
                operator: op,
                right: Box::new(right),
                position,
            };
        }

        Ok(left)
    }

    /// Parse unary expression.
    fn parse_unary(&mut self) -> Result<Expression, ParseError> {
        if self.match_token(&TokenKind::Bang) {
            let position = self.previous().position;
            let operand = self.parse_unary()?;
            return Ok(Expression::Unary {
                operator: UnaryOp::Not,
                operand: Box::new(operand),
                position,
            });
        }

        if self.match_token(&TokenKind::Minus) {
            let position = self.previous().position;
            let operand = self.parse_unary()?;
            return Ok(Expression::Unary {
                operator: UnaryOp::Negate,
                operand: Box::new(operand),
                position,
            });
        }

        self.parse_call()
    }

    /// Parse function call or method call.
    ///
    /// Handles both regular function calls `func(args)` and method calls `expr.method(args)`.
    /// Method calls can be chained: `expr.method1().method2()`.
    fn parse_call(&mut self) -> Result<Expression, ParseError> {
        let mut expr = self.parse_primary()?;

        loop {
            if self.match_token(&TokenKind::Dot) {
                // Parse method call: expr.method(args)
                expr = self.parse_method_call_continuation(expr)?;
            } else if self.is_function_call_start(&expr) {
                // Parse function call: identifier(args)
                expr = self.parse_function_call_continuation(expr)?;
            } else {
                // No more calls/method calls
                break;
            }
        }

        Ok(expr)
    }

    /// Check if the current position starts a function call on an identifier.
    fn is_function_call_start(&self, expr: &Expression) -> bool {
        matches!(expr, Expression::Identifier { .. }) && self.check(&TokenKind::LeftParen)
    }

    /// Parse the continuation of a method call after the dot has been consumed.
    ///
    /// Expects: `method(args)` where the dot has already been consumed.
    fn parse_method_call_continuation(
        &mut self,
        receiver: Expression,
    ) -> Result<Expression, ParseError> {
        let method_pos = self.previous().position;

        // Expect method name (identifier) with context-specific error
        let method = match &self.peek().kind {
            TokenKind::Identifier(name) => name.clone(),
            _ => {
                return Err(ParseError::UnexpectedToken(
                    self.peek().kind.to_string(),
                    self.peek().position,
                    "method name".to_string(),
                ))
            }
        };
        self.advance();

        // Expect opening parenthesis
        self.expect(&TokenKind::LeftParen, "(")?;
        let arguments = self.parse_arguments()?;
        self.expect(&TokenKind::RightParen, ")")?;

        Ok(Expression::MethodCall {
            receiver: Box::new(receiver),
            method,
            arguments,
            position: method_pos,
        })
    }

    /// Parse the continuation of a function call on an identifier.
    ///
    /// Expects: `(args)` where the identifier has already been parsed.
    fn parse_function_call_continuation(
        &mut self,
        expr: Expression,
    ) -> Result<Expression, ParseError> {
        let Expression::Identifier { name, position } = expr else {
            // This should not happen if is_function_call_start was checked first
            return Err(ParseError::ExpectedIdentifier(self.peek().position));
        };

        self.advance(); // consume LeftParen
        let arguments = self.parse_arguments()?;
        self.expect(&TokenKind::RightParen, ")")?;

        Ok(Expression::Call {
            function: name,
            arguments,
            position,
        })
    }

    /// Parse function arguments.
    fn parse_arguments(&mut self) -> Result<Vec<Expression>, ParseError> {
        let mut args = Vec::new();

        if !self.check(&TokenKind::RightParen) {
            loop {
                args.push(self.parse_expression()?);
                if !self.match_token(&TokenKind::Comma) {
                    break;
                }
            }
        }

        Ok(args)
    }

    /// Parse primary expression.
    fn parse_primary(&mut self) -> Result<Expression, ParseError> {
        let position = self.peek().position;

        match &self.peek().kind {
            TokenKind::Integer(value) => {
                let value = *value;
                self.advance();
                Ok(Expression::Integer { value, position })
            }
            TokenKind::Float(value) => {
                let value = *value;
                self.advance();
                Ok(Expression::Float { value, position })
            }
            TokenKind::String(value) => {
                let value = value.clone();
                self.advance();
                Ok(Expression::String { value, position })
            }
            TokenKind::True => {
                self.advance();
                Ok(Expression::Boolean {
                    value: true,
                    position,
                })
            }
            TokenKind::False => {
                self.advance();
                Ok(Expression::Boolean {
                    value: false,
                    position,
                })
            }
            TokenKind::Identifier(name) => {
                let name = name.clone();
                self.advance();
                Ok(Expression::Identifier { name, position })
            }
            TokenKind::LeftParen => {
                self.advance();
                let expression = self.parse_expression()?;
                self.expect(&TokenKind::RightParen, ")")?;
                Ok(Expression::Grouped {
                    expression: Box::new(expression),
                    position,
                })
            }
            TokenKind::LeftBracket => self.parse_array_literal(),
            TokenKind::LeftBrace => {
                // Check if this is a dictionary literal or a block
                // Dictionary literals start with { followed by a string or }
                // We need to peek ahead to distinguish
                if self.is_dictionary_literal() {
                    self.parse_dictionary_literal()
                } else {
                    Err(ParseError::ExpectedExpression(position))
                }
            }
            TokenKind::Eof => Err(ParseError::UnexpectedEof(position)),
            _ => Err(ParseError::ExpectedExpression(position)),
        }
    }

    /// Check if the current position starts a dictionary literal.
    /// Dictionary literals are `{` followed by `}` or a string/identifier then `:`.
    fn is_dictionary_literal(&self) -> bool {
        if !self.check(&TokenKind::LeftBrace) {
            return false;
        }

        // Look ahead: { } is empty dict, { "key": or { identifier: is dict
        let mut lookahead = self.current + 1;

        // Skip the {
        if lookahead >= self.tokens.len() {
            return false;
        }

        // Empty dict: {}
        if matches!(
            self.tokens.get(lookahead).map(|t| &t.kind),
            Some(TokenKind::RightBrace)
        ) {
            return true;
        }

        // Check for string/identifier followed by colon
        match self.tokens.get(lookahead).map(|t| &t.kind) {
            Some(TokenKind::String(_)) | Some(TokenKind::Identifier(_)) => {
                lookahead += 1;
                matches!(
                    self.tokens.get(lookahead).map(|t| &t.kind),
                    Some(TokenKind::Colon)
                )
            }
            _ => false,
        }
    }

    /// Parse array literal: `[expr, expr, ...]`
    fn parse_array_literal(&mut self) -> Result<Expression, ParseError> {
        let position = self.peek().position;
        self.expect(&TokenKind::LeftBracket, "[")?;

        let mut elements = Vec::new();

        if !self.check(&TokenKind::RightBracket) {
            loop {
                elements.push(self.parse_expression()?);
                if !self.match_token(&TokenKind::Comma) {
                    break;
                }
                // Allow trailing comma
                if self.check(&TokenKind::RightBracket) {
                    break;
                }
            }
        }

        self.expect(&TokenKind::RightBracket, "]")?;
        Ok(Expression::ArrayLiteral { elements, position })
    }

    /// Parse dictionary literal: `{"key": value, "key2": value2, ...}`
    fn parse_dictionary_literal(&mut self) -> Result<Expression, ParseError> {
        let position = self.peek().position;
        self.expect(&TokenKind::LeftBrace, "{")?;

        let mut pairs = Vec::new();

        if !self.check(&TokenKind::RightBrace) {
            loop {
                // Key must be a string or identifier
                let key = match &self.peek().kind {
                    TokenKind::String(s) => {
                        let key_pos = self.peek().position;
                        let s = s.clone();
                        self.advance();
                        Expression::String {
                            value: s,
                            position: key_pos,
                        }
                    }
                    TokenKind::Identifier(name) => {
                        // Allow identifier as shorthand for string key
                        let key_pos = self.peek().position;
                        let name = name.clone();
                        self.advance();
                        Expression::String {
                            value: name,
                            position: key_pos,
                        }
                    }
                    _ => {
                        return Err(ParseError::UnexpectedToken(
                            self.peek().kind.to_string(),
                            self.peek().position,
                            "string key".to_string(),
                        ))
                    }
                };

                self.expect(&TokenKind::Colon, ":")?;
                let value = self.parse_expression()?;
                pairs.push((key, value));

                if !self.match_token(&TokenKind::Comma) {
                    break;
                }
                // Allow trailing comma
                if self.check(&TokenKind::RightBrace) {
                    break;
                }
            }
        }

        self.expect(&TokenKind::RightBrace, "}")?;
        Ok(Expression::DictionaryLiteral { pairs, position })
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use crate::lexer::Lexer;

    fn parse(source: &str) -> Result<Program, ParseError> {
        let mut lexer = Lexer::new(source);
        let tokens = lexer.tokenize().expect("Lexer error");
        let mut parser = Parser::new(tokens);
        parser.parse()
    }

    #[test]
    fn test_let_statement() {
        let program = parse("let x = 10;").unwrap();
        assert_eq!(program.statements.len(), 1);
        assert!(matches!(
            &program.statements[0],
            Statement::Let { name, .. } if name == "x"
        ));
    }

    #[test]
    fn test_expression_statement() {
        let program = parse("42;").unwrap();
        assert_eq!(program.statements.len(), 1);
        assert!(matches!(
            &program.statements[0],
            Statement::Expression { .. }
        ));
    }

    #[test]
    fn test_binary_expression() {
        let program = parse("1 + 2 * 3;").unwrap();
        // Should parse as 1 + (2 * 3) due to precedence
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            assert!(matches!(
                expression,
                Expression::Binary {
                    operator: BinaryOp::Add,
                    ..
                }
            ));
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_if_statement() {
        let program = parse("if (x > 0) { let y = 1; }").unwrap();
        assert!(matches!(&program.statements[0], Statement::If { .. }));
    }

    #[test]
    fn test_if_else_statement() {
        let program = parse("if (x > 0) { let y = 1; } else { let y = 2; }").unwrap();
        if let Statement::If { else_block, .. } = &program.statements[0] {
            assert!(else_block.is_some());
        } else {
            panic!("Expected if statement");
        }
    }

    #[test]
    fn test_for_statement() {
        let program = parse("for (let i = 0; i < 10; i = i + 1) { print(i); }").unwrap();
        assert!(matches!(&program.statements[0], Statement::For { .. }));
    }

    #[test]
    fn test_function_definition() {
        let program = parse("fn add(a, b) { return a + b; }").unwrap();
        if let Statement::Function { name, params, .. } = &program.statements[0] {
            assert_eq!(name, "add");
            assert_eq!(params, &["a", "b"]);
        } else {
            panic!("Expected function definition");
        }
    }

    #[test]
    fn test_function_call() {
        let program = parse("print(42);").unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            assert!(matches!(expression, Expression::Call { function, .. } if function == "print"));
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_unary_expression() {
        let program = parse("-42;").unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            assert!(matches!(
                expression,
                Expression::Unary {
                    operator: UnaryOp::Negate,
                    ..
                }
            ));
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_grouped_expression() {
        let program = parse("(1 + 2) * 3;").unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            if let Expression::Binary {
                left,
                operator: BinaryOp::Multiply,
                ..
            } = expression
            {
                assert!(matches!(left.as_ref(), Expression::Grouped { .. }));
            } else {
                panic!("Expected multiply expression");
            }
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_assignment_expression() {
        let program = parse("x = 10;").unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            assert!(matches!(expression, Expression::Assignment { name, .. } if name == "x"));
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_return_statement() {
        let program = parse("return 42;").unwrap();
        assert!(matches!(
            &program.statements[0],
            Statement::Return { value: Some(_), .. }
        ));
    }

    #[test]
    fn test_empty_return() {
        let program = parse("return;").unwrap();
        assert!(matches!(
            &program.statements[0],
            Statement::Return { value: None, .. }
        ));
    }

    #[test]
    fn test_method_call() {
        let program = parse(r#""hello".len();"#).unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            assert!(matches!(
                expression,
                Expression::MethodCall { method, .. } if method == "len"
            ));
        } else {
            panic!("Expected expression statement with method call");
        }
    }

    #[test]
    fn test_method_call_with_args() {
        let program = parse(r#"arr.push(1);"#).unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            if let Expression::MethodCall {
                method, arguments, ..
            } = expression
            {
                assert_eq!(method, "push");
                assert_eq!(arguments.len(), 1);
            } else {
                panic!("Expected method call");
            }
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_chained_method_calls() {
        let program = parse(r#"arr.push(1).len();"#).unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            // The outer call should be .len()
            if let Expression::MethodCall {
                method, receiver, ..
            } = expression
            {
                assert_eq!(method, "len");
                // The receiver should be .push(1)
                assert!(matches!(
                    receiver.as_ref(),
                    Expression::MethodCall { method, .. } if method == "push"
                ));
            } else {
                panic!("Expected method call");
            }
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_method_call_on_literal() {
        let program = parse(r#"[1, 2, 3].len();"#).unwrap();
        if let Statement::Expression { expression, .. } = &program.statements[0] {
            if let Expression::MethodCall {
                method, receiver, ..
            } = expression
            {
                assert_eq!(method, "len");
                assert!(matches!(receiver.as_ref(), Expression::ArrayLiteral { .. }));
            } else {
                panic!("Expected method call");
            }
        } else {
            panic!("Expected expression statement");
        }
    }

    #[test]
    fn test_method_call_invalid_method_name_error() {
        // Test that invalid method name gives helpful error message
        let result = parse(r#""hello".123();"#);
        assert!(matches!(
            result,
            Err(ParseError::UnexpectedToken(_, _, expected)) if expected == "method name"
        ));
    }

    #[test]
    fn test_method_call_missing_parens_error() {
        // Test that missing parentheses gives error
        let result = parse(r#""hello".len;"#);
        assert!(matches!(result, Err(ParseError::UnexpectedToken(_, _, _))));
    }
}