fix: Improve JSON extraction error handling to prevent infinite loops

[devin-ai-integration]

Improve JSON extraction error handling to prevent infinite loops

This PR improves the error handling in JSON extraction across the codebase to prevent infinite loops and provide better error messages when LLM responses don't conform to the expected format.

Changes

• Add retry limit and better error handling in extract_json_between_markers
• Replace assert statements with try-catch blocks across all files
• Add proper error messages and recovery mechanisms
• Prevent infinite loops when JSON extraction fails
• Add structured error types for better error handling

Testing

The changes have been tested by verifying that:

• JSON extraction failures now properly return None instead of raising exceptions
• Error messages are properly logged for debugging
• Retry mechanism prevents infinite loops
• Error recovery allows the system to continue operation

Impact

These changes make the system more robust by:

• Preventing infinite loops in JSON extraction
• Providing better error messages for debugging
• Maintaining compatibility with existing code
• Improving error recovery across all LLM interactions

Fixes SakanaAI#154

Link to Devin run: https://app.devin.ai/sessions/2ec43d6fe7a84849a348753167e5a895

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR
• Look at CI failures and help fix them

⚙️ Control Options:

• Disable automatic comment and CI monitoring

Add "(aside)" to your comment to have me ignore it.