honeymaker 0.11.0 → 0.11.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c8386a3ce395f193a10d210d48e62f48ffb5f2c2cf6ab2dbf77cd8a6466700c9
-  data.tar.gz: 9a4bd23725e93deb9a4b8e6aad1f9e86f6b27f13ecf0507031a9fbd53b2f65ba
+  metadata.gz: a33b3f43a40a17e8f8655683b4349901db4222a6ec5db4d4a8f82585f0dacdd1
+  data.tar.gz: dde27e04419cb12327dfee7c3ecf69c61f36ddf28c1e9b7e2872c785a726106b
 SHA512:
-  metadata.gz: 76f414d30b7ae364453bdb8b0522e2c1be01adc6db1126addfc2d0f3fc7111df3d3de41714dab6ec98b25dba8116605f9a9c064a58fe907aba2a4eb24f3f81f5
-  data.tar.gz: b6d38bc14208cadd407faa789607deddba96a9bd0a5c63d0ed5efbbc6fc86bef6fadda21f30814051e0cae5c1ded2a1bba8a8f5e2a99f06074dffb9c545f9229
+  metadata.gz: f6b7a44553dbc9a413c6df817f4ffbff0f263c077f163e10bce5389d7aba60c108d956c68f9f713ac2b7fef69c699540e3759a1fb5470162146d746ace18a624
+  data.tar.gz: e53261fd199b590b1d23318d5b2689a59a5b6dc2a402e09b6fb3d63010b1bbb79281e4b3cafd8401964315e51f8a7fbe39e45aa250217e0ed9851c35bc7c494d

data/lib/honeymaker/client.rb

@@ -70,8 +70,19 @@ module Honeymaker
       status = e.respond_to?(:response_status) ? e.response_status : nil
       Result::Failure.new(error_message, data: { status: status })
     rescue StandardError => e
+      # NOT an exchange error — a bug here, in a vendor gem, or in the caller, and it arrives in the
+      # same Result::Failure as a genuine venue rejection. Callers classify that text to decide "out
+      # of funds" / "bad key" / "safe to retry", so an unlabelled TypeError gets attributed to the
+      # exchange: a hyperliquid-rb signing crash sat in production order history for three months
+      # reading exactly like a venue rejection. Name the class so the true source is unmistakable,
+      # and flag it so a caller can refuse to classify it as the venue's at all.
+      #
+      # The class is a PREFIX, not a replacement: the network failures that also land here
+      # (Net::ReadTimeout, execution expired, connection refused) are matched by substring for
+      # transient retry, and those matches must keep working.
       msg = e.message
-      Result::Failure.new((msg && !msg.empty?) ? msg : "Unknown error")
+      labelled = (msg && !msg.empty?) ? "#{e.class}: #{msg}" : e.class.to_s
+      Result::Failure.new(labelled, data: { client_error: true })
     end
 
     # A business error the exchange returned inside an HTTP-200 envelope (KuCoin's non-"200000"

data/lib/honeymaker/exchanges/mexc.rb

@@ -24,7 +24,10 @@ module Honeymaker
               quote_decimals: product["quoteAssetPrecision"],
               price_decimals: f[:price] ? Utils.decimals(f[:price]["tickSize"]) : product["quotePrecision"],
               available: true,
-              trading_enabled: product["status"] == "TRADING"
+              # MEXC encodes status as "1", not Binance's "TRADING" — every symbol it returns is "1".
+              # isSpotTradingAllowed is the real gate: ~5% of listed symbols reject a spot order.
+              # `!= false` keeps the fail-open default if the key is ever absent, matching the siblings.
+              trading_enabled: product["status"] == "1" && product["isSpotTradingAllowed"] != false
             }
           end
         end

data/lib/honeymaker/version.rb

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

data/test/fixtures/mexc_exchange_info.json

@@ -1,40 +1,64 @@
 {
+  "_provenance": "Real capture: GET https://api.mexc.com/api/v3/exchangeInfo (2026-09-05), trimmed to two symbols and to the fields this parser reads. MEXC returns status \"1\" for every symbol and PERCENT_PRICE_BY_SIDE as its only filter \u2014 it emits neither \"TRADING\" nor LOT_SIZE/PRICE_FILTER/MIN_NOTIONAL. Re-capture and diff rather than hand-editing; mexc_test.rb asserts this file's SHA-256.",
+  "timezone": "CST",
   "symbols": [
     {
-      "symbol": "BTCUSDT",
-      "status": "TRADING",
-      "baseAsset": "BTC",
+      "symbol": "METALUSDT",
+      "status": "1",
+      "baseAsset": "METAL",
+      "baseAssetPrecision": 2,
       "quoteAsset": "USDT",
-      "baseAssetPrecision": 8,
-      "quoteAssetPrecision": 8,
-      "quotePrecision": 2,
+      "quotePrecision": 5,
+      "quoteAssetPrecision": 5,
+      "orderTypes": [
+        "LIMIT",
+        "MARKET",
+        "LIMIT_MAKER"
+      ],
+      "isSpotTradingAllowed": true,
+      "isMarginTradingAllowed": false,
+      "quoteAmountPrecision": "1",
+      "baseSizePrecision": "0.1",
+      "permissions": [
+        "SPOT"
+      ],
       "filters": [
         {
-          "filterType": "PRICE_FILTER",
-          "tickSize": "0.01"
-        },
-        {
-          "filterType": "LOT_SIZE",
-          "minQty": "0.00001",
-          "maxQty": "9000",
-          "stepSize": "0.00001"
-        },
-        {
-          "filterType": "MIN_NOTIONAL",
-          "minNotional": "5",
-          "maxNotional": "9000000"
+          "filterType": "PERCENT_PRICE_BY_SIDE",
+          "bidMultiplierUp": "0.2",
+          "askMultiplierDown": "0.2"
         }
-      ]
+      ],
+      "maxQuoteAmount": "2000000"
     },
     {
-      "symbol": "ETHUSDT",
-      "status": "HALT",
-      "baseAsset": "ETH",
+      "symbol": "PALMAIUSDT",
+      "status": "1",
+      "baseAsset": "PALMAI",
+      "baseAssetPrecision": 2,
       "quoteAsset": "USDT",
-      "baseAssetPrecision": 6,
-      "quoteAssetPrecision": 8,
-      "quotePrecision": 2,
-      "filters": []
+      "quotePrecision": 4,
+      "quoteAssetPrecision": 4,
+      "orderTypes": [
+        "LIMIT",
+        "MARKET",
+        "LIMIT_MAKER"
+      ],
+      "isSpotTradingAllowed": false,
+      "isMarginTradingAllowed": false,
+      "quoteAmountPrecision": "1",
+      "baseSizePrecision": "0",
+      "permissions": [
+        "SPOT"
+      ],
+      "filters": [
+        {
+          "filterType": "PERCENT_PRICE_BY_SIDE",
+          "bidMultiplierUp": "0.2",
+          "askMultiplierDown": "0.2"
+        }
+      ],
+      "maxQuoteAmount": "2000000"
     }
   ]
 }

data/test/honeymaker/client_test.rb

@@ -41,7 +41,44 @@ class Honeymaker::ClientTest < Minitest::Test
     client = Honeymaker::Client.new
     result = client.send(:with_rescue) { raise StandardError, "boom" }
     assert result.failure?
-    assert_equal ["boom"], result.errors
+    assert_equal ["StandardError: boom"], result.errors
+  end
+
+  # A bug in this gem, in a vendor gem, or in the caller is NOT an exchange error, but it lands in
+  # the same Result::Failure. Callers classify failure text to decide "out of funds" / "bad key" /
+  # "retry me", so an unlabelled TypeError reads as something the venue said and gets filed and
+  # reported as one. Naming the class makes the true source unmistakable, and the flag lets a
+  # caller refuse to classify it at all.
+  def test_with_rescue_names_the_exception_class_for_non_api_errors
+    client = Honeymaker::Client.new
+    result = client.send(:with_rescue) { raise TypeError, "String can't be coerced into Float" }
+    assert result.failure?
+    assert_equal ["TypeError: String can't be coerced into Float"], result.errors
+    assert result.data[:client_error], "a local exception must be flagged, not passed off as the venue's"
+  end
+
+  def test_with_rescue_labels_an_empty_message_with_its_class
+    client = Honeymaker::Client.new
+    result = client.send(:with_rescue) { raise NoMethodError, "" }
+    assert result.failure?
+    assert_equal ["NoMethodError"], result.errors
+  end
+
+  # Network failures reach callers through this branch too, and the transient-retry matchers key
+  # off substrings of them. Prefixing must not break that.
+  def test_with_rescue_keeps_network_substrings_matchable
+    client = Honeymaker::Client.new
+    result = client.send(:with_rescue) { raise Net::ReadTimeout, "Net::ReadTimeout with #<TCPSocket:(closed)>" }
+    assert_includes result.errors.first, "Net::ReadTimeout"
+  end
+
+  # An exchange error is not ours: it must stay unflagged and unprefixed so the venue's own text
+  # reaches the classifiers verbatim.
+  def test_with_rescue_does_not_flag_api_errors
+    client = Honeymaker::Client.new
+    result = client.send(:with_rescue) { raise Faraday::TimeoutError, "timeout" }
+    assert result.failure?
+    assert_nil result.data[:client_error]
   end
 
   def test_hmac_sha256

data/test/honeymaker/exchanges/mexc_test.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "test_helper"
+require "digest"
 
 class Honeymaker::Exchanges::MexcTest < Minitest::Test
   include FixtureHelper
@@ -9,6 +10,23 @@ class Honeymaker::Exchanges::MexcTest < Minitest::Test
     @exchange = Honeymaker::Exchanges::Mexc.new
   end
 
+  # H1. MEXC's exchangeInfo has never carried Binance's status encoding. The fixture this file
+  # drives was a hand-edited clone of the Binance one ("TRADING"/"HALT", LOT_SIZE/PRICE_FILTER/
+  # MIN_NOTIONAL filters) written from the same wrong assumption as the parser it tested, so both
+  # agreed and both were wrong. Asserting the raw bytes is the only guard that survives an author
+  # who believes the wrong thing twice; the digest also binds this file to deltabadger's copy, which
+  # drives the same capture through the other implementation of this contract.
+  def test_fixture_carries_mexcs_own_status_encoding
+    body = load_fixture("mexc_exchange_info.json")
+
+    assert_equal "1", body["symbols"].first["status"]
+    refute body["symbols"].any? { |s| s["status"] == "TRADING" }
+    refute body["symbols"].any? { |s| s["filters"].any? { |f| f["filterType"] == "LOT_SIZE" } }
+
+    assert_equal "13a65c0c8ebde52fac4cdca9a5be9d9a18ba7b5438c496b1b30606cf52fd60d3",
+                 Digest::SHA256.hexdigest(File.read(fixture_path("mexc_exchange_info.json")))
+  end
+
   def test_get_tickers_info_parses_response
     body = load_fixture("mexc_exchange_info.json")
     stub_connection(body)
@@ -19,40 +37,42 @@ class Honeymaker::Exchanges::MexcTest < Minitest::Test
     assert_equal 2, result.data.size
 
     ticker = result.data.first
-    assert_equal "BTCUSDT", ticker[:ticker]
-    assert_equal "BTC", ticker[:base]
+    assert_equal "METALUSDT", ticker[:ticker]
+    assert_equal "METAL", ticker[:base]
     assert_equal "USDT", ticker[:quote]
-    assert_equal "0.00001", ticker[:minimum_base_size]
-    assert_equal "5", ticker[:minimum_quote_size]
-    assert_equal "9000", ticker[:maximum_base_size]
-    assert_equal 5, ticker[:base_decimals]
-    assert_equal 8, ticker[:quote_decimals]
-    assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
     assert ticker[:trading_enabled]
   end
 
-  def test_halted_symbol_listed_but_not_trading_enabled
+  # H2. status is "1" for every MEXC symbol, including the 103 the venue will not accept a spot
+  # order for. The status check alone re-enables all of them; isSpotTradingAllowed is what keeps
+  # them out of the picker.
+  def test_a_symbol_mexc_does_not_allow_for_spot_is_not_trading_enabled
     body = load_fixture("mexc_exchange_info.json")
     stub_connection(body)
 
     result = @exchange.get_tickers_info
 
-    halted = result.data.find { |t| t[:ticker] == "ETHUSDT" }
-    assert halted[:available]        # still listed
-    refute halted[:trading_enabled]  # but not trading
+    disallowed = result.data.find { |t| t[:ticker] == "PALMAIUSDT" }
+    assert_equal "1", body["symbols"].last["status"]
+    assert disallowed[:available]        # still listed
+    refute disallowed[:trading_enabled]  # but not spot-tradable
   end
 
-  def test_falls_back_to_precision_fields_when_no_filters
+  # H5. Real MEXC sends only PERCENT_PRICE_BY_SIDE, so the precision fallback is the ONLY path —
+  # there is no filter-bearing symbol to contrast it against.
+  def test_falls_back_to_precision_fields_when_no_size_filters
     body = load_fixture("mexc_exchange_info.json")
     stub_connection(body)
 
     result = @exchange.get_tickers_info
 
-    # ETHUSDT has empty filters, should use baseAssetPrecision/quotePrecision
-    eth = result.data.find { |t| t[:ticker] == "ETHUSDT" }
-    assert_equal 6, eth[:base_decimals]
-    assert_equal 2, eth[:price_decimals]
+    metal = result.data.find { |t| t[:ticker] == "METALUSDT" }
+    assert_equal 2, metal[:base_decimals]   # baseAssetPrecision
+    assert_equal 5, metal[:quote_decimals]  # quoteAssetPrecision
+    assert_equal 5, metal[:price_decimals]  # quotePrecision
+    assert_nil metal[:minimum_base_size]
+    assert_nil metal[:minimum_quote_size]
   end
 
   def test_get_bid_ask_parses_response

metadata

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