Catch rate limit errors and raise them so that callers can response appropriately.

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ghx (0.2.1)
+    ghx (0.3.0)
       faraday (~> 2.9.0)
       faraday-retry (~> 2.2.1)
       octokit

lib/ghx.rb

@@ -13,6 +13,7 @@ def symbolize_keys!
 end
 
 require_relative "version"
+require_relative "ghx/errors"
 require_relative "ghx/graphql_client"
 require_relative "ghx/rest_client"
 require_relative "ghx/dependabot"

lib/ghx/errors.rb

@@ -0,0 +1,7 @@
+module GHX
+  class GHXError < StandardError; end
+
+  class RateLimitExceededError < GHXError; end
+
+  class OtherApiError < GHXError; end
+end

lib/ghx/issue.rb

@@ -13,10 +13,6 @@ def self.search(owner:, repo:, query:)
       data.fetch("items").to_a.map do |issue_data|
         new(owner: owner, repo: repo, **issue_data)
       end
-    rescue => e
-      GHX.logger.error "Error searching for issues with query: #{e.message}"
-      GHX.logger.error "Received data: #{data}"
-      []
     end
 
     # Find an issue by its number

lib/ghx/rest_client.rb

@@ -18,17 +18,8 @@ def initialize(api_key)
     def get(path)
       uri = URI.parse("https://api.github.com/#{path}")
       request = Net::HTTP::Get.new(uri)
-      request["Accept"] = "application/vnd.github+json"
-      request["Authorization"] = "Bearer #{@api_key}"
-      request["X-Github-Api-Version"] = "2022-11-28"
-
-      req_options = {
-        use_ssl: uri.scheme == "https"
-      }
 
-      response = Net::HTTP.start(uri.hostname, uri.port, req_options) do |http|
-        http.request(request)
-      end
+      response = _http_request(uri: uri, request: request)
 
       JSON.parse(response.body)
     end
@@ -40,19 +31,10 @@ def get(path)
     def post(path, params)
       uri = URI.parse("https://api.github.com/#{path}")
       request = Net::HTTP::Post.new(uri)
-      request["Accept"] = "application/vnd.github+json"
-      request["Authorization"] = "Bearer #{@api_key}"
-      request["X-Github-Api-Version"] = "2022-11-28"
-
-      req_options = {
-        use_ssl: uri.scheme == "https"
-      }
 
       request.body = params.to_json
 
-      response = Net::HTTP.start(uri.hostname, uri.port, req_options) do |http|
-        http.request(request)
-      end
+      response = _http_request(uri: uri, request: request)
 
       JSON.parse(response.body)
     end
@@ -64,6 +46,17 @@ def post(path, params)
     def patch(path, params)
       uri = URI.parse("https://api.github.com/#{path}")
       request = Net::HTTP::Patch.new(uri)
+
+      request.body = params.to_json
+
+      response = _http_request(uri: uri, request: request)
+
+      JSON.parse(response.body)
+    end
+
+    private
+
+    def _http_request(uri:, request:)
       request["Accept"] = "application/vnd.github+json"
       request["Authorization"] = "Bearer #{@api_key}"
       request["X-Github-Api-Version"] = "2022-11-28"
@@ -72,13 +65,22 @@ def patch(path, params)
         use_ssl: uri.scheme == "https"
       }
 
-      request.body = params.to_json
-
       response = Net::HTTP.start(uri.hostname, uri.port, req_options) do |http|
         http.request(request)
       end
 
-      JSON.parse(response.body)
+      if response.code.to_i < 400
+        response
+      elsif [403, 429].include?(response.code.to_i)
+        if response["X-RateLimit-Remaining"].to_i == 0
+          reset_time = Time.at(response["X-RateLimit-Reset"].to_i)
+          raise GHX::RateLimitExceededError, "GitHub API rate limit exceeded. Try again after #{reset_time}"
+        else
+          raise GHX::RateLimitExceededError, "GitHub API rate limit exceeded. Try again later."
+        end
+      else
+        raise GHX::OtherApiError, "GitHub API returned an error: #{response.code} #{response.body}"
+      end
     end
   end
 end

lib/version.rb

@@ -1,3 +1,3 @@
 module GHX
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end