Add retry limit to ActiveRecord::SerializationFailure rescue

[Copilot]

Addresses feedback on PR #63 to prevent infinite retry loops when handling serialization failures.

Changes

• Retry limiting: Cap retries at 3 attempts maximum using a counter variable
• Enhanced logging: Include attempt number in warnings ("attempt 1/3") and log error on exhaustion
• Exception propagation: Re-raise the exception after max retries to surface persistent failures

Implementation

def execute_query
  retry_count ||= 0
  begin
    # ... query execution ...
  rescue ActiveRecord::SerializationFailure => e
    retry_count += 1
    if retry_count < 3
      Rails.logger.warn("Retrying after serialization failure (attempt #{retry_count}/3): #{e.message}")
      retry
    else
      Rails.logger.error("Max retries reached after serialization failure: #{e.message}")
      raise
    end
  end
end

This prevents unbounded retries while maintaining automatic recovery for transient failures.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.