honeymaker 0.10.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bbfab70631b45882e86a1f4af9ea69b9c32c98840a383183b69ebae133b2f3bf
-  data.tar.gz: 3d65a28ae18b85f5de60959aac90e327471ff617488956538442b6841ddc4bd9
+  metadata.gz: c8386a3ce395f193a10d210d48e62f48ffb5f2c2cf6ab2dbf77cd8a6466700c9
+  data.tar.gz: 9a4bd23725e93deb9a4b8e6aad1f9e86f6b27f13ecf0507031a9fbd53b2f65ba
 SHA512:
-  metadata.gz: 00f3d5d33b5c1d231be4f1bafb5815083bc26cafae31a8cfe2c4403be217ece9120f7195bc6f5fa15d14286950c9f8817762767a4824f0c29396a6b679ca483e
-  data.tar.gz: 3209e5756f1aab9c2af0537c533b3b2dbadf468c635c5f882d4f85362679597495d78f9ec77a2f7ca1b10a8c1c534687d19b95be73989ce88fe23fbf5f03120a
+  metadata.gz: 76f414d30b7ae364453bdb8b0522e2c1be01adc6db1126addfc2d0f3fc7111df3d3de41714dab6ec98b25dba8116605f9a9c064a58fe907aba2a4eb24f3f81f5
+  data.tar.gz: b6d38bc14208cadd407faa789607deddba96a9bd0a5c63d0ed5efbbc6fc86bef6fadda21f30814051e0cae5c1ded2a1bba8a8f5e2a99f06074dffb9c545f9229

data/lib/honeymaker/client.rb

@@ -74,6 +74,17 @@ module Honeymaker
       Result::Failure.new((msg && !msg.empty?) ? msg : "Unknown error")
     end
 
+    # A business error the exchange returned inside an HTTP-200 envelope (KuCoin's non-"200000"
+    # code, Bitget's non-"00000"). Keep the exchange's own code and message: callers classify on
+    # this text to tell insufficient funds from a bad key from a stale timestamp, so collapsing it
+    # to a bare constant silently disables every one of those checks and leaves the operator with
+    # an unactionable log line. Falls back to the constant only when the body carries no detail.
+    def api_error(exchange, body)
+      detail = body.is_a?(Hash) ? [body["code"], body["msg"] || body["message"]] : []
+      detail = detail.compact.map(&:to_s).reject(&:empty?).join(": ")
+      Result::Failure.new(detail.empty? ? "#{exchange} API error" : "#{exchange} API error #{detail}")
+    end
+
     def connection
       @connection ||= build_client_connection(self.class::URL)
     end

data/lib/honeymaker/clients/bingx.rb

@@ -57,6 +57,11 @@ module Honeymaker
         return result if result.failure?
 
         raw = result.data
+        # BingX rejects with HTTP 200 and a non-zero `code`. Without this the rejection fell
+        # through as a Success carrying order_id "#{symbol}-" (nil id): the bot recorded a trade
+        # that was never placed, then polled a malformed id forever.
+        return api_error("BingX", raw) unless raw.is_a?(Hash) && raw["code"].to_i.zero?
+
         order_id = raw.dig("data", "orderId") || raw.dig("data", "data", "orderId")
         Result::Success.new({ order_id: "#{symbol}-#{order_id}", raw: raw })
       end
@@ -66,8 +71,9 @@ module Honeymaker
           symbol: symbol, orderId: order_id, clientOrderID: client_order_id
         })
         return result if result.failure?
+        return api_error("BingX", result.data) unless result.data.is_a?(Hash) && result.data["code"].to_i.zero?
 
-        raw = result.data.is_a?(Hash) && result.data.key?("data") ? result.data["data"] : result.data
+        raw = result.data.key?("data") ? result.data["data"] : result.data
         Result::Success.new(normalize_order("#{symbol}-#{raw['orderId']}", raw))
       end
 

data/lib/honeymaker/clients/bitget.rb

@@ -44,7 +44,7 @@ module Honeymaker
         result = get_account_assets
         return result if result.failure?
 
-        return Result::Failure.new("Bitget API error") unless result.data["code"] == "00000"
+        return api_error("Bitget", result.data) unless result.data["code"] == "00000"
 
         balances = {}
         (result.data["data"] || []).each do |asset|
@@ -64,7 +64,7 @@ module Honeymaker
           size: size, quoteSize: quote_size, price: price, force: force, clientOid: client_oid
         })
         return result if result.failure?
-        return Result::Failure.new("Bitget API error") unless result.data["code"] == "00000"
+        return api_error("Bitget", result.data) unless result.data["code"] == "00000"
 
         order_id = result.data.dig("data", "orderId")
         Result::Success.new({ order_id: "#{symbol}-#{order_id}", raw: result.data })
@@ -73,7 +73,7 @@ module Honeymaker
       def get_order(order_id: nil, client_oid: nil)
         result = get_signed("/api/v2/spot/trade/orderInfo", { orderId: order_id, clientOid: client_oid })
         return result if result.failure?
-        return Result::Failure.new("Bitget API error") unless result.data["code"] == "00000"
+        return api_error("Bitget", result.data) unless result.data["code"] == "00000"
 
         order_list = result.data["data"]
         raw = order_list.is_a?(Array) ? order_list.first : order_list

data/lib/honeymaker/clients/kucoin.rb

@@ -39,7 +39,7 @@ module Honeymaker
         result = get_accounts(type: type)
         return result if result.failure?
 
-        return Result::Failure.new("KuCoin API error") unless result.data["code"] == "200000"
+        return api_error("KuCoin", result.data) unless result.data["code"] == "200000"
 
         balances = {}
         (result.data["data"] || []).each do |account|
@@ -61,7 +61,7 @@ module Honeymaker
           timeInForce: time_in_force, stp: stp
         })
         return result if result.failure?
-        return Result::Failure.new("KuCoin API error") unless result.data["code"] == "200000"
+        return api_error("KuCoin", result.data) unless result.data["code"] == "200000"
 
         order_id = result.data.dig("data", "orderId")
         Result::Success.new({ order_id: order_id, raw: result.data })
@@ -70,7 +70,7 @@ module Honeymaker
       def get_order(order_id:)
         result = get_signed("/api/v1/orders/#{order_id}")
         return result if result.failure?
-        return Result::Failure.new("KuCoin API error") unless result.data["code"] == "200000"
+        return api_error("KuCoin", result.data) unless result.data["code"] == "200000"
 
         raw = result.data["data"]
         return Result::Failure.new("Order not found") unless raw

data/lib/honeymaker/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Honeymaker
-  VERSION = "0.10.0"
+  VERSION = "0.11.0"
 end

data/test/honeymaker/clients/bingx_client_test.rb

@@ -41,6 +41,30 @@ class Honeymaker::Clients::BingXTest < Minitest::Test
     assert result.success?
   end
 
+  # BingX rejects with HTTP 200 and a non-zero `code`. Without a guard the rejection was read as
+  # a placed order and handed back as order_id "BTC-USDT-" (nil id), so a bot recorded a trade
+  # that does not exist and then polled a malformed id forever.
+  def test_place_order_fails_on_nonzero_code
+    stub_connection(:post, { "code" => 100004, "msg" => "insufficient balance" })
+    result = @client.place_order(symbol: "BTC-USDT", side: "BUY", type: "MARKET", quantity: "0.001")
+    assert result.failure?
+    assert_includes result.errors.first, "insufficient balance"
+  end
+
+  def test_get_order_fails_on_nonzero_code
+    stub_connection(:get, { "code" => 80016, "msg" => "Order does not exist" })
+    result = @client.get_order(symbol: "BTC-USDT", order_id: "123")
+    assert result.failure?
+    assert_includes result.errors.first, "Order does not exist"
+  end
+
+  def test_place_order_succeeds_on_zero_code
+    stub_connection(:post, { "code" => 0, "data" => { "orderId" => "123" } })
+    result = @client.place_order(symbol: "BTC-USDT", side: "BUY", type: "MARKET", quantity: "0.001")
+    assert result.success?
+    assert_equal "BTC-USDT-123", result.data[:order_id]
+  end
+
   def test_cancel_order
     stub_connection(:post, { "data" => { "orderId" => "123" } })
     result = @client.cancel_order(symbol: "BTC-USDT", order_id: "123")

data/test/honeymaker/clients/bitget_client_test.rb

@@ -104,6 +104,37 @@ class Honeymaker::Clients::BitgetTest < Minitest::Test
     assert headers.key?(:"ACCESS-SIGN")
   end
 
+  # Bitget signals business errors in an HTTP-200 envelope. Callers classify on this text
+  # (insufficient funds, bad key, stale timestamp), so the code and msg must survive.
+  def test_place_order_keeps_api_code_and_message
+    stub_connection(:post, { "code" => "43012", "msg" => "Insufficient balance" })
+    result = @client.place_order(symbol: "BTCUSDT", side: "buy", order_type: "market", quote_size: "100")
+    assert result.failure?
+    assert_includes result.errors.first, "Insufficient balance"
+    assert_includes result.errors.first, "43012"
+  end
+
+  def test_get_order_keeps_api_code_and_message
+    stub_connection(:get, { "code" => "40008", "msg" => "Request timestamp expired" })
+    result = @client.get_order(order_id: "BTCUSDT-123")
+    assert result.failure?
+    assert_includes result.errors.first, "Request timestamp expired"
+  end
+
+  def test_get_balances_keeps_api_code_and_message
+    stub_connection(:get, { "code" => "40012", "msg" => "Apikey does not exist" })
+    result = @client.get_balances
+    assert result.failure?
+    assert_includes result.errors.first, "Apikey does not exist"
+  end
+
+  def test_api_error_falls_back_when_body_carries_no_detail
+    stub_connection(:post, { "code" => nil })
+    result = @client.place_order(symbol: "BTCUSDT", side: "buy", order_type: "market", quote_size: "100")
+    assert result.failure?
+    assert_equal "Bitget API error", result.errors.first
+  end
+
   private
 
   def stub_connection(method, body)

data/test/honeymaker/clients/kucoin_client_test.rb

@@ -107,6 +107,37 @@ class Honeymaker::Clients::KucoinTest < Minitest::Test
     assert_equal expected, headers[:"KC-API-PASSPHRASE"]
   end
 
+  # KuCoin signals business errors in an HTTP-200 envelope. Callers classify on this text
+  # (insufficient funds, bad key, stale timestamp), so the code and msg must survive.
+  def test_place_order_keeps_api_code_and_message
+    stub_connection(:post, { "code" => "200004", "msg" => "Balance insufficient!" })
+    result = @client.place_order(client_oid: "c1", side: "buy", symbol: "BTC-USDT", type: "market", funds: "100")
+    assert result.failure?
+    assert_includes result.errors.first, "Balance insufficient!"
+    assert_includes result.errors.first, "200004"
+  end
+
+  def test_get_order_keeps_api_code_and_message
+    stub_connection(:get, { "code" => "400002", "msg" => "Invalid KC-API-TIMESTAMP." })
+    result = @client.get_order(order_id: "123")
+    assert result.failure?
+    assert_includes result.errors.first, "Invalid KC-API-TIMESTAMP."
+  end
+
+  def test_get_balances_keeps_api_code_and_message
+    stub_connection(:get, { "code" => "400003", "msg" => "KC-API-KEY not exists" })
+    result = @client.get_balances
+    assert result.failure?
+    assert_includes result.errors.first, "KC-API-KEY not exists"
+  end
+
+  def test_api_error_falls_back_when_body_carries_no_detail
+    stub_connection(:post, { "code" => nil })
+    result = @client.place_order(client_oid: "c1", side: "buy", symbol: "BTC-USDT", type: "market", funds: "100")
+    assert result.failure?
+    assert_equal "KuCoin API error", result.errors.first
+  end
+
   private
 
   def stub_connection(method, body)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: honeymaker
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Deltabadger