Improve Error Handling and Prevent Recursive Calls in Currency Conversion

[sweep-ai]

This pull request was created by Sweep to resolve the following request by @curtisdelicata:

Fix possible recursion loop as stated called by In Container.php line 1048 could be any file

Continue chatting at https://sweep-chat-demo.vercel.app/c/279998f2-27e5-4887-9874-732b688944cd.

Purpose

Enhance system stability by implementing robust error prevention and handling mechanisms for recursive calls in currency conversion and exception handling.

Description

This pull request introduces critical improvements to prevent potential infinite recursion and stack overflow scenarios:

1. In the CurrencyService, we've implemented:

   • A static flag to prevent recursive currency conversion attempts
   • Reduced maximum recursion depth for rate calculations
   • Added a try-finally block to ensure the processing flag is always reset
   • Throw a RuntimeException if a recursive conversion is detected

2. In the Exceptions/Handler, we've enhanced error reporting by:

   • Adding a flag to prevent recursive error handling
   • Expanding error detection to include additional recursion-related error messages
   • Implementing a try-finally block to ensure the handling flag is reset
   • Improving logging for stack overflow and recursion-related errors

Summary

• Improved recursion prevention in CurrencyService
• Enhanced error handling in Exceptions/Handler
• Added safeguards against potential infinite loops
• Reduced maximum recursion depth
• Implemented robust error logging mechanisms

Key files modified:

• app/Services/CurrencyService.php
• app/Exceptions/Handler.php