ci(memory_leaks): Fix CI valgrind check and resolve function memory leaks

[SIGMazer]

Description

This pull request includes several important changes to memory management and testing procedures in the codebase. The changes focus on improving the handling of function parameters and enhancing the reliability of the test suite.

Memory management improvements:

• ast.c: Added a call to free_parameters to ensure all function parameters are properly freed in free_ast.
• ast.c: Added a call to SAFE_FREE for variables in enter_function_scope to prevent memory leaks.
• ast.c: Added a call to reverse_parameter_list in enter_function_scope to ensure the parameter list is in the correct order.

Testing enhancements:

• run_valgrind_tests.sh: Added a check for Valgrind errors and an exit command to stop the script if Valgrind detects any issues. This ensures that memory leaks and errors are caught during testing.

Related Issue

Fixes #<issue_number>

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update
• Performance improvement
• Refactor

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have documented my changes in the code or documentation
• I have added tests that prove my changes work (if applicable)
• I have run the unit tests locally
• I have run the valgrind memory tests locally
• All new and existing tests pass