skinbaron_api_client 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bac140e69332695d80e8a5f324a8becc9fb88052e80fddd0774d849bba2d1ee5
-  data.tar.gz: cb4c3b9386949e5cd4ca9b05136a02a7015ca722e48b5fec0d71c44da3c3de8b
+  metadata.gz: a8196ed40a109dfdcc450596dfc8a558f034766ddd02bdf99d910262fcf9281e
+  data.tar.gz: 8b2e857ad1ca9969810f75346875ca0bef84587dec657bfc157fa10087c3a97a
 SHA512:
-  metadata.gz: f79f5e0ffcfd24a8ffec58da35ffa995e12da53e2e57abfb01e5f51aef6b44d3fb61419a45ecb12bba292da22ee062b0806db911af84e45e2c4f2339b8b5403c
-  data.tar.gz: 50b1432feb1bb58928a27fd68fb59e22351b3dcb2db8e492e4baf694876ab24abe7152c10b0844bc206a1e0f1d9546cb3577604f76d7c806ac5b007fe40ce738
+  metadata.gz: b00aac24f670697d06a4c89189d1a73e6be9a1c8f0502695168e713a9b51fa386c7715b8f1c5220106e45785da7832d614898054ae53fbcab636d7ea36db3db3
+  data.tar.gz: b206725aa8fc3031cd2fa0552173eab730d7c5be96f1ffd5ead2f4aebf874b8c4fc9cda9c9b89eecb687d1ffa78b94446d83d3bbc3a40b0ccd559a438205988d

data/lib/skinbaron_api_client/client.rb

@@ -4,14 +4,11 @@
 
 require_relative "configuration"
 require_relative "http_client"
-require_relative "error_handling"
 require_relative "logger"
 require_relative "endpoints/search"
 
 module SkinbaronApiClient
   class Client
-    include ErrorHandling
-
     attr_reader :config, :http_client, :search_endpoint
 
     def initialize(**options)

data/lib/skinbaron_api_client/http_client.rb

@@ -1,13 +1,10 @@
 require "http"
 require "json"
-require_relative "error_handling"
 require_relative "logger"
 
 module SkinbaronApiClient
   # HTTP client for making requests to the Skinbaron API
   class HttpClient
-    include ErrorHandling
-
     attr_reader :base_url, :headers, :debug
 
     def initialize(base_url:, headers: {}, debug: false)
@@ -16,7 +13,7 @@ module SkinbaronApiClient
       @debug = debug
     end
 
-    with_error_handling def post(endpoint:, body: {})
+    def post(endpoint:, body: {})
       url = "#{base_url}/#{endpoint}"
       debug_log "Making POST request to: #{url}"
       debug_log "Request body: #{body.to_json}"
@@ -30,6 +27,7 @@ module SkinbaronApiClient
                           })
 
       http_response = HTTP.headers(headers).post(url, json: body)
+      raise AuthenticationError, "Authentication failed" if http_response.body.to_s.include? "wrong or unauthenticated"
 
       Logger.instance.log({
                             type: "RESPONSE",

data/lib/skinbaron_api_client/logger.rb

@@ -54,9 +54,6 @@ module SkinbaronApiClient
         @request_logger.info(JSON.pretty_generate(log_data))
         @request_logger.info("#{"=" * 80}\n")
       end
-    rescue JSON::ParserError => e
-      @logger.error("LOGGER: failed to parse search response", error: e.message)
-      @logger.error("LOGGER: Response: #{JSON.pretty_generate(log_data)}")
     end
 
     LEVELS.each do |level|

data/lib/skinbaron_api_client/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SkinbaronApiClient
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 end

data/lib/skinbaron_api_client.rb

@@ -7,7 +7,6 @@ require "json"
 require_relative "skinbaron_api_client/version"
 require_relative "skinbaron_api_client/client"
 require_relative "skinbaron_api_client/logger"
-require_relative "skinbaron_api_client/error_handling"
 
 # Example usage:
 #

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: skinbaron_api_client
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Sam Schams
@@ -64,7 +64,6 @@ files:
 - lib/skinbaron_api_client/client.rb
 - lib/skinbaron_api_client/configuration.rb
 - lib/skinbaron_api_client/endpoints/search.rb
-- lib/skinbaron_api_client/error_handling.rb
 - lib/skinbaron_api_client/http_client.rb
 - lib/skinbaron_api_client/logger.rb
 - lib/skinbaron_api_client/version.rb

data/lib/skinbaron_api_client/error_handling.rb

@@ -1,64 +0,0 @@
-# frozen_string_literal: true
-
-module SkinbaronApiClient
-  # Mixin for adding error handling capabilities to API client classes.
-  # Provides methods for response validation and error logging.
-  module ErrorHandling
-    def self.included(base)
-      base.extend(ClassMethods)
-    end
-
-    # Class methods for decorating instance methods with error handling.
-    # Provides the with_error_handling decorator for wrapping methods.
-    module ClassMethods
-      def with_error_handling(method_name)
-        original_method = instance_method(method_name)
-
-        define_method(method_name) do |*args, **kwargs|
-          response = original_method.bind(self).call(*args, **kwargs)
-
-          check_response_success(response)
-          check_response_authentication(response)
-
-          response
-        rescue HTTP::Error => e
-          log_error(message: "HTTP request failed", error: e)
-          raise RequestError, "HTTP request failed: #{e.message}"
-        rescue JSON::ParserError => e
-          log_error(message: "JSON parsing failed", error: e)
-          raise ResponseError, "Invalid JSON response: #{e.message}"
-        end
-      end
-    end
-
-    private
-
-    def logger
-      SkinbaronApiClient::Logger.instance
-    end
-
-    def log_error(message:, error:)
-      logger.error(
-        message,
-        {
-          error_class: error.class.name,
-          error_message: error.message,
-          backtrace: error.backtrace&.first(5)
-        }
-      )
-    end
-
-    def check_response_success(response)
-      return if response[:status]&.success?
-
-      raise ResponseError, "Request failed with status #{response[:status]}. Response: #{response[:body]}"
-    end
-
-    def check_response_authentication(response)
-      return unless response[:body].to_s.include? "wrong or unauthenticated request"
-
-      logger.error("Authentication failed", { body: response[:body] })
-      raise AuthenticationError, "Authentication failed: #{response[:body]}"
-    end
-  end
-end