From 265458b94f52ef6b0570af4c17b2d959f22386df Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 26 Jan 2026 07:08:00 +0000
Subject: [PATCH] Fix error transformer to return instead of raise

The default_error_transformer should return the error object,
not raise it. The with_provider_response wrapper handles the
error and wraps it in a Response object.

Co-authored-by: jjmata <187772+jjmata@users.noreply.github.com>
---
 app/models/provider/twelve_data.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/models/provider/twelve_data.rb b/app/models/provider/twelve_data.rb
index bfc6fef1d..c68ef89cd 100644
--- a/app/models/provider/twelve_data.rb
+++ b/app/models/provider/twelve_data.rb
@@ -239,10 +239,10 @@ class Provider::TwelveData < Provider
         response_body = error.response&.dig(:body)
         status_code = error.response&.dig(:status)
 
-        # Detect 429 rate limit errors
+        # Detect 429 rate limit errors and return RateLimitError
         if status_code == 429
           message = extract_error_message(response_body) || error.message
-          raise RateLimitError.new(
+          return RateLimitError.new(
             "TwelveData rate limit exceeded: #{message}",
             details: response_body
           )