honeymaker 0.8.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 329432e2569d62afa1bd99352d9f9075dbfb93004b27856f3ae535e03aba2a00
-  data.tar.gz: fe3f83098b328ad88e41c385efd8fe855ba8386c838beae26caab2d02a479a45
+  metadata.gz: 12080b0cbcae6a52a2680efde8f4131b30f4ebcf70647c1c297ac88217c459ba
+  data.tar.gz: c475a4e5da6e8f7ac2e0b5f3eaa84e12192eec504876871028935a6b3be05cae
 SHA512:
-  metadata.gz: 1f38749e06bf4b3fbf03da5830bcc68bfe581b8f901f566894ee51d81a0b6883cd59bd8646ed98ca2fa119db70beee055b10dce2068387b85377b1b1e6cef6bc
-  data.tar.gz: b83aa96da78eab87f3004321175b470ade7caa34b7843f326c72a3b479d548cedf223328a70700afadc46e96bca17844002ab6467fc9cfd31313bd7ea5118825
+  metadata.gz: c6c9bffcfa55f7f096a4128c2b09119e06d11e5be58bc04dde162a693600595e7662334034d38a049b7faaa2f9535976668b22890564bf6ddcf82f8f1c1ade2e
+  data.tar.gz: eedbdfab0c9fa12e8b4ae86362eafcd90b4d8319caace49bdb642405ca99a90836af69a6c8a50d9a13e555f6f1be4008175fb7e019927d8198060d7835a3754c

data/Rakefile

@@ -29,26 +29,38 @@ def bump_version(segment)
   puts "Bumped version to #{new_version}"
 end
 
-task :bump_version do
-  bump_version(@bump_segment || :patch)
-end
+# Bundler's `release` task is defined when this Rakefile loads — it captures
+# the gemspec version at that moment in `Bundler::GemHelper`. If we bump the
+# version mid-run (as a build prerequisite), the gem is published correctly
+# but the final "Pushed <gem> <ver>" log line still prints the old cached
+# version. Fix: bump first, then invoke bundler's release in a subprocess so
+# it re-reads the gemspec fresh.
+
+Rake::Task[:release].clear
 
-task release_preflight: :bump_version do
+def do_release(segment)
+  bump_version(segment)
   sh "bundle install"
   Rake::Task[:test].invoke
-  sh 'git add -A && git diff --cached --quiet || git commit -m "$(ruby -e "require_relative \'lib/honeymaker/version\'; puts Honeymaker::VERSION")"'
+  sh %(git add -A && git diff --cached --quiet || git commit -m "v#{current_version}")
+  sh "bundle exec rake _bundler_release"
 end
 
-Rake::Task[:build].enhance([:release_preflight])
+task _bundler_release: %w[build release:guard_clean release:source_control_push release:rubygem_push]
+
+desc "Bump patch, run tests, push to rubygems"
+task :release do
+  do_release(:patch)
+end
 
 namespace :release do
+  desc "Bump minor, run tests, push to rubygems"
   task :minor do
-    @bump_segment = :minor
-    Rake::Task[:release].invoke
+    do_release(:minor)
   end
 
+  desc "Bump major, run tests, push to rubygems"
   task :major do
-    @bump_segment = :major
-    Rake::Task[:release].invoke
+    do_release(:major)
   end
 end

data/lib/honeymaker/exchanges/binance.rb

@@ -25,7 +25,8 @@ module Honeymaker
               base_decimals: Utils.decimals(f[:lot_size]["stepSize"]),
               quote_decimals: product["quoteAssetPrecision"],
               price_decimals: Utils.decimals(f[:price]["tickSize"]),
-              available: product["status"] == "TRADING"
+              available: true,
+              trading_enabled: product["status"] == "TRADING"
             }
           end
         end

data/lib/honeymaker/exchanges/bingx.rb

@@ -25,7 +25,8 @@ module Honeymaker
               base_decimals: Utils.decimals(product["stepSize"]),
               quote_decimals: Utils.decimals(product["tickSize"]),
               price_decimals: Utils.decimals(product["tickSize"]),
-              available: product["status"].to_i == 1
+              available: true,
+              trading_enabled: product["status"].to_i == 1
             }
           end
         end

data/lib/honeymaker/exchanges/bitget.rb

@@ -21,7 +21,8 @@ module Honeymaker
               base_decimals: product["quantityPrecision"].to_i,
               quote_decimals: product["quotePrecision"].to_i,
               price_decimals: product["pricePrecision"].to_i,
-              available: product["status"] == "online"
+              available: true,
+              trading_enabled: product["status"] == "online"
             }
           end
         end

data/lib/honeymaker/exchanges/bitmart.rb

@@ -21,7 +21,8 @@ module Honeymaker
               base_decimals: Utils.decimals(product["base_min_size"]),
               quote_decimals: Utils.decimals(product["quote_increment"]),
               price_decimals: product["price_max_precision"].to_i,
-              available: product["trade_status"] == "trading"
+              available: true,
+              trading_enabled: product["trade_status"] == "trading"
             }
           end
         end

data/lib/honeymaker/exchanges/bitrue.rb

@@ -23,7 +23,8 @@ module Honeymaker
               base_decimals: f[:lot_size] ? Utils.decimals(f[:lot_size]["stepSize"]) : product["baseAssetPrecision"],
               quote_decimals: product["quotePrecision"],
               price_decimals: f[:price] ? Utils.decimals(f[:price]["tickSize"]) : product["quotePrecision"],
-              available: product["status"] == "TRADING"
+              available: true,
+              trading_enabled: product["status"] == "TRADING"
             }
           end
         end

data/lib/honeymaker/exchanges/bitvavo.rb

@@ -24,7 +24,8 @@ module Honeymaker
               base_decimals: product["pricePrecision"] || 8,
               quote_decimals: product["pricePrecision"] || 8,
               price_decimals: product["pricePrecision"] || 8,
-              available: product["status"] == "trading"
+              available: true,
+              trading_enabled: product["status"] == "trading"
             }
           end
         end

data/lib/honeymaker/exchanges/bybit.rb

@@ -26,7 +26,8 @@ module Honeymaker
               base_decimals: Utils.decimals(lot_size_filter["basePrecision"]),
               quote_decimals: Utils.decimals(lot_size_filter["quotePrecision"]),
               price_decimals: Utils.decimals(price_filter["tickSize"]),
-              available: product["status"] == "Trading"
+              available: true,
+              trading_enabled: product["status"] == "Trading"
             }
           end
         end

data/lib/honeymaker/exchanges/coinbase.rb

@@ -31,7 +31,8 @@ module Honeymaker
               base_decimals: Utils.decimals(product["base_increment"]),
               quote_decimals: Utils.decimals(product["quote_increment"]),
               price_decimals: Utils.decimals(product["price_increment"]),
-              available: true
+              available: true,
+              trading_enabled: trading_enabled?(product)
             }
           end
         end
@@ -50,6 +51,15 @@ module Honeymaker
 
       private
 
+      # Coinbase only signals problems explicitly: a `trading_disabled` flag or a non-"online"
+      # `status`. Absent either signal, treat the pair as enabled.
+      def trading_enabled?(product)
+        return false if product["trading_disabled"] == true
+        return false if product.key?("status") && product["status"] != "online"
+
+        true
+      end
+
       def connection
         @connection ||= build_connection(BASE_URL)
       end

data/lib/honeymaker/exchanges/gemini.rb

@@ -27,7 +27,8 @@ module Honeymaker
               base_decimals: Utils.decimals(tick_size),
               quote_decimals: Utils.decimals(quote_increment),
               price_decimals: Utils.decimals(quote_increment),
-              available: detail["status"] == "open"
+              available: true,
+              trading_enabled: detail["status"] == "open"
             }
           end
         end

data/lib/honeymaker/exchanges/hyperliquid.rb

@@ -31,7 +31,8 @@ module Honeymaker
               base_decimals: base_token["szDecimals"] || 0,
               quote_decimals: 2,
               price_decimals: 5,
-              available: true
+              available: true,
+              trading_enabled: true
             }
           end
         end

data/lib/honeymaker/exchanges/kraken.rb

@@ -59,7 +59,8 @@ module Honeymaker
               base_decimals: info["lot_decimals"],
               quote_decimals: info["cost_decimals"],
               price_decimals: info["pair_decimals"],
-              available: true
+              available: true,
+              trading_enabled: info.key?("status") ? info["status"] == "online" : true
             }
           end
         end

data/lib/honeymaker/exchanges/kucoin.rb

@@ -21,7 +21,8 @@ module Honeymaker
               base_decimals: Utils.decimals(product["baseIncrement"]),
               quote_decimals: Utils.decimals(product["quoteIncrement"]),
               price_decimals: Utils.decimals(product["priceIncrement"]),
-              available: product["enableTrading"]
+              available: true,
+              trading_enabled: product["enableTrading"]
             }
           end
         end

data/lib/honeymaker/exchanges/mexc.rb

@@ -23,7 +23,8 @@ module Honeymaker
               base_decimals: f[:lot_size] ? Utils.decimals(f[:lot_size]["stepSize"]) : product["baseAssetPrecision"],
               quote_decimals: product["quoteAssetPrecision"],
               price_decimals: f[:price] ? Utils.decimals(f[:price]["tickSize"]) : product["quotePrecision"],
-              available: product["status"] == "TRADING"
+              available: true,
+              trading_enabled: product["status"] == "TRADING"
             }
           end
         end

data/lib/honeymaker/version.rb

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

data/test/honeymaker/exchanges/binance_test.rb

@@ -26,6 +26,19 @@ class Honeymaker::Exchanges::BinanceTest < Minitest::Test
     assert_equal 8, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
+  end
+
+  def test_get_tickers_info_marks_non_trading_pairs_disabled
+    body = load_fixture("binance_exchange_info.json")
+    body["symbols"].first["status"] = "BREAK"
+    stub_connection(body)
+
+    result = @exchange.get_tickers_info
+
+    ticker = result.data.first
+    assert ticker[:available]        # still listed
+    refute ticker[:trading_enabled]  # but not trading
   end
 
   def test_get_tickers_info_handles_api_error

data/test/honeymaker/exchanges/binance_us_test.rb

@@ -27,6 +27,8 @@ class Honeymaker::Exchanges::BinanceUsTest < Minitest::Test
     ticker = result.data.first
     assert_equal "BTCUSDT", ticker[:ticker]
     assert_equal "BTC", ticker[:base]
+    assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
   def test_get_bid_ask_parses_response

data/test/honeymaker/exchanges/bingx_test.rb

@@ -30,6 +30,7 @@ class Honeymaker::Exchanges::BingXTest < Minitest::Test
     assert_equal 2, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
   def test_skips_symbols_without_dash

data/test/honeymaker/exchanges/bitget_test.rb

@@ -29,16 +29,18 @@ class Honeymaker::Exchanges::BitgetTest < Minitest::Test
     assert_equal 8, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
-  def test_offline_symbol_not_available
+  def test_offline_symbol_listed_but_not_trading_enabled
     body = load_fixture("bitget_symbols.json")
     stub_connection(body)
 
     result = @exchange.get_tickers_info
 
     eth = result.data.find { |t| t[:ticker] == "ETHUSDT" }
-    refute eth[:available]
+    assert eth[:available]          # still listed
+    refute eth[:trading_enabled]    # but not trading
   end
 
   def test_get_bid_ask_parses_response

data/test/honeymaker/exchanges/bitmart_test.rb

@@ -29,16 +29,18 @@ class Honeymaker::Exchanges::BitMartTest < Minitest::Test
     assert_equal 2, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
-  def test_pre_trade_not_available
+  def test_pre_trade_listed_but_not_trading_enabled
     body = load_fixture("bitmart_symbols.json")
     stub_connection(body)
 
     result = @exchange.get_tickers_info
 
     eth = result.data.find { |t| t[:ticker] == "ETH_USDT" }
-    refute eth[:available]
+    assert eth[:available]          # still listed
+    refute eth[:trading_enabled]    # but not trading
   end
 
   def test_get_bid_ask_parses_response

data/test/honeymaker/exchanges/bitrue_test.rb

@@ -29,6 +29,7 @@ class Honeymaker::Exchanges::BitrueTest < Minitest::Test
     assert_equal 2, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
   def test_falls_back_to_precision_fields_when_no_filters

data/test/honeymaker/exchanges/bitvavo_test.rb

@@ -29,16 +29,18 @@ class Honeymaker::Exchanges::BitvavoTest < Minitest::Test
     assert_equal 5, ticker[:quote_decimals]
     assert_equal 5, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
-  def test_halted_market_not_available
+  def test_halted_market_listed_but_not_trading_enabled
     body = load_fixture("bitvavo_markets.json")
     stub_connection(body)
 
     result = @exchange.get_tickers_info
 
     eth = result.data.find { |t| t[:ticker] == "ETH-EUR" }
-    refute eth[:available]
+    assert eth[:available]          # still listed
+    refute eth[:trading_enabled]    # but not trading
   end
 
   def test_get_bid_ask_parses_response

data/test/honeymaker/exchanges/bybit_test.rb

@@ -30,6 +30,7 @@ class Honeymaker::Exchanges::BybitTest < Minitest::Test
     assert_equal 8, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
   def test_get_bid_ask_parses_response

data/test/honeymaker/exchanges/coinbase_test.rb

@@ -29,6 +29,29 @@ class Honeymaker::Exchanges::CoinbaseTest < Minitest::Test
     assert_equal 2, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled] # no disabled signal -> defaults to true
+  end
+
+  def test_get_tickers_info_disables_when_trading_disabled
+    body = load_fixture("coinbase_products.json")
+    body["products"].first["trading_disabled"] = true
+    stub_connection(body)
+
+    result = @exchange.get_tickers_info
+
+    ticker = result.data.first
+    assert ticker[:available]        # still listed
+    refute ticker[:trading_enabled]  # but not trading
+  end
+
+  def test_get_tickers_info_disables_when_status_not_online
+    body = load_fixture("coinbase_products.json")
+    body["products"].first["status"] = "delisted"
+    stub_connection(body)
+
+    result = @exchange.get_tickers_info
+
+    refute result.data.first[:trading_enabled]
   end
 
   def test_filters_blacklisted_assets

data/test/honeymaker/exchanges/gemini_test.rb

@@ -35,6 +35,7 @@ class Honeymaker::Exchanges::GeminiTest < Minitest::Test
     assert_equal 2, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
   def test_get_bid_ask_parses_response

data/test/honeymaker/exchanges/hyperliquid_test.rb

@@ -28,6 +28,7 @@ class Honeymaker::Exchanges::HyperliquidTest < Minitest::Test
     assert_equal 2, ticker[:quote_decimals]
     assert_equal 5, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled] # no native status -> always true
   end
 
   def test_skips_pairs_with_missing_tokens

data/test/honeymaker/exchanges/kraken_test.rb

@@ -26,6 +26,38 @@ class Honeymaker::Exchanges::KrakenTest < Minitest::Test
     assert_equal 5, ticker[:quote_decimals]
     assert_equal 1, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled] # fixture has no status -> defaults to true
+  end
+
+  def test_get_tickers_info_disables_non_online_status
+    body = load_fixture("kraken_asset_pairs.json")
+    body["result"]["XBTUSDT"]["status"] = "cancel_only"
+    stub_request(body)
+
+    result = @exchange.get_tickers_info
+
+    ticker = result.data.first
+    assert ticker[:available]        # still listed
+    refute ticker[:trading_enabled]  # but not trading
+  end
+
+  def test_get_tickers_info_enables_online_status
+    body = load_fixture("kraken_asset_pairs.json")
+    body["result"]["XBTUSDT"]["status"] = "online"
+    stub_request(body)
+
+    result = @exchange.get_tickers_info
+
+    assert result.data.first[:trading_enabled]
+  end
+
+  def test_get_tickers_info_defaults_trading_enabled_when_status_absent
+    body = load_fixture("kraken_asset_pairs.json") # no status key
+    stub_request(body)
+
+    result = @exchange.get_tickers_info
+
+    assert result.data.first[:trading_enabled]
   end
 
   def test_get_tickers_info_skips_pairs_without_wsname

data/test/honeymaker/exchanges/kucoin_test.rb

@@ -30,6 +30,7 @@ class Honeymaker::Exchanges::KucoinTest < Minitest::Test
     assert_equal 6, ticker[:quote_decimals]
     assert_equal 1, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
   def test_get_bid_ask_parses_response

data/test/honeymaker/exchanges/mexc_test.rb

@@ -29,16 +29,18 @@ class Honeymaker::Exchanges::MexcTest < Minitest::Test
     assert_equal 8, ticker[:quote_decimals]
     assert_equal 2, ticker[:price_decimals]
     assert ticker[:available]
+    assert ticker[:trading_enabled]
   end
 
-  def test_halted_symbol_not_available
+  def test_halted_symbol_listed_but_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" }
-    refute halted[:available]
+    assert halted[:available]        # still listed
+    refute halted[:trading_enabled]  # but not trading
   end
 
   def test_falls_back_to_precision_fields_when_no_filters

metadata

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