Исправить алгоритм автокурса: GAP, защита от обгона лидера, обработка одинаковых курсов

[dapi]

Описание проблемы

Текущая реализация auto_comission_by_external_comissions в RateComissionCalculator не соответствует бизнес-требованиям.

Бизнес-правила (требуемое поведение)

Занимать наивысшую позицию от позиции X до позиции Y, соблюдая диапазон курса от A до B.
Если в целевом диапазоне все обменники имеют одинаковый курс (мы не можем занять позицию), мы делаем курс на 1 тысячную меньше, при условии что это нас не выкинет из беста (то есть мы не должны обогнать лучший обменник).

Проблемы текущей реализации

Требование	Текущий код	Статус
GAP = 0.001 (1 тысячная)	AUTO_COMISSION_GAP = 0.01 (1 сотая)	❌ Неверно
Не обгонять позицию 1	Не реализовано	❌ Отсутствует
Если все одинаковые → ставим такой же	Ставит first - GAP всегда	❌ Неверно

Примеры (Use Cases)

Пример #1 (AS-IS) — текущее поведение ✅

Условие: позиции 2-5, курс 1%-2%

BestChange:
  Поз 1: обменник1 = 1%
  Поз 2: обмен2    = 1.1%
  Поз 3: обменник3 = 2%

Результат: 1.099% (1.1% - 0.001)
Ожидаемая позиция: 2

Пример #2 (TO-BE) — требует доработки ❌

Условие: позиции 2-5, курс 1%-2%

BestChange:
  Поз 1: обменник1 = 0.9%  ← лучший
  Поз 2-7: все по 1%

Текущий результат: 0.99% (неверно)
Ожидаемый результат: 1% (все одинаковые, встаём в очередь)

Пример #3 (TO-BE) — требует доработки ❌

Условие: позиции 2-5, курс 1%-2%

BestChange:
  Поз 1-7: все по 1%

Текущий результат: 0.99% (обгоняем лидера → выкинет из беста!)
Ожидаемый результат: 1% (не обгоняем лидера)

Предлагаемое решение

AUTO_COMISSION_GAP = 0.001  # Изменить с 0.01 на 0.001

def auto_comission_by_external_comissions
  return 0 unless could_be_calculated?

  external_rates_in_target_position = external_rates[(position_from - 1)..(position_to - 1)]
  return autorate_from unless external_rates_in_target_position.present?

  external_rates_in_target_comission = external_rates_in_target_position.select do |rate|
    (autorate_from..autorate_to).include?(rate.target_rate_percent)
  end
  return autorate_from if external_rates_in_target_comission.empty?

  best_in_target = external_rates_in_target_comission.first.target_rate_percent
  best_overall = external_rates.first&.target_rate_percent

  # Проверяем: все ли в целевом диапазоне имеют одинаковый курс?
  all_same = external_rates_in_target_comission.all? { |r| r.target_rate_percent == best_in_target }

  if all_same
    # Все одинаковые — ставим такой же курс
    target_comission = best_in_target
  else
    # Есть разные — ставим чуть лучше
    target_comission = best_in_target - AUTO_COMISSION_GAP
  end

  # НЕ обгоняем лидера (позиция 1)
  if best_overall && target_comission < best_overall
    target_comission = best_in_target
  end

  target_comission
end

Файлы для изменения

• app/services/gera/rate_comission_calculator.rb

Чеклист

• Изменить AUTO_COMISSION_GAP с 0.01 на 0.001
• Добавить проверку "все одинаковые курсы"
• Добавить защиту от обгона лидера (позиция 1)
• Написать тесты для всех use cases
• Обновить документацию

[dapi]

AUTO_COMISSION_GAP нужно вообще вытащить в ENV

[dapi]

Уточнение требований: GAP должен быть абсолютным, не процентным

Проблема с процентным GAP

Текущий AUTO_COMISSION_GAP = 0.01 — это процент. Для направлений крипто→фиат это создаёт проблему:

Направление: USDT → RUB
Курс: ~78.77 RUB/USDT

Позиции в BestChange:
  Поз 1:  78.7760
  Поз 2:  78.7758
  Поз 3:  78.7755
  Поз 4:  78.7753
  Поз 5:  78.7752  ← целевая позиция
  ...
  Поз 11: 78.7750

Разница между позициями: ~0.0002-0.0003 RUB (доли копейки)

❌ Текущий GAP (0.01% от 78.77) = 0.007877 RUB ≈ 0.79 копейки
   → Перепрыгивает ВСЕ 11 позиций
   → Курс становится "неадекватным"
   → BestChange выкидывает из рейтинга!

✅ Нужный GAP: 0.0001 RUB (0.01 копейки)
   → Встаём ровно на целевую позицию

Новое требование

GAP должен быть абсолютным значением в фиатной валюте направления (валюта TO):

Направление	Валюта GAP	Рекомендуемое значение
USDT → RUB	RUB	0.0001 (0.01 коп)
USDT → USD	USD	0.0001
BTC → EUR	EUR	0.0001
RUB → USDT	USDT	0.0001

Изменения в алгоритме

# Было (процент):
target_comission = best_rate_percent - AUTO_COMISSION_GAP

# Должно быть (абсолют в валюте TO):
gap_in_fiat = 0.0001  # Настраиваемый параметр
target_rate = best_rate - gap_in_fiat  # Применяем к курсу, не к проценту

Вопросы для реализации

1. Где хранить GAP?

   • Глобальная константа?
   • Настройка на уровне ExchangeRate?
   • Настройка на уровне PaymentSystem?

2. Разный GAP для разных валют?

   • RUB: 0.0001 (0.01 коп)
   • USD/EUR: 0.0001
   • Или единый для всех?

3. Применять к курсу или к комиссии?

   • К курсу напрямую (рекомендуется)
   • Конвертировать в процент комиссии

[dapi]

Уточнение: Scope ограничен направлениями USDT↔RUB

Новые правила (абсолютный GAP) применяются только для:

• USDT → RUB
• RUB → USDT

Для всех остальных направлений — текущая логика без изменений.

Предлагаемая реализация

ABSOLUTE_GAP_DIRECTIONS = [
  ['USDT', 'RUB'],
  ['RUB', 'USDT']
].freeze

ABSOLUTE_GAP_VALUE = 0.0001  # В фиатной валюте (RUB)

def auto_comission_by_external_comissions
  return 0 unless could_be_calculated?

  external_rates_in_target_position = external_rates[(position_from - 1)..(position_to - 1)]
  return autorate_from unless external_rates_in_target_position.present?

  external_rates_in_target_comission = external_rates_in_target_position.select do |rate|
    (autorate_from..autorate_to).include?(rate.target_rate_percent)
  end
  return autorate_from if external_rates_in_target_comission.empty?

  best_in_target = external_rates_in_target_comission.first
  best_overall = external_rates.first

  # Проверяем: все ли одинаковые?
  all_same = external_rates_in_target_comission.all? do |r| 
    r.target_rate_percent == best_in_target.target_rate_percent 
  end

  if all_same
    target_comission = best_in_target.target_rate_percent
  elsif use_absolute_gap?
    # Для USDT↔RUB: абсолютный GAP
    target_comission = calculate_with_absolute_gap(best_in_target)
  else
    # Для остальных: процентный GAP (текущая логика)
    target_comission = best_in_target.target_rate_percent - AUTO_COMISSION_GAP
  end

  # Не обгоняем лидера
  if best_overall && target_comission < best_overall.target_rate_percent
    target_comission = best_in_target.target_rate_percent
  end

  target_comission
end

private

def use_absolute_gap?
  ABSOLUTE_GAP_DIRECTIONS.include?([in_currency.to_s, out_currency.to_s])
end

def calculate_with_absolute_gap(best_rate)
  # Конвертируем абсолютный GAP (0.0001 RUB) в процент комиссии
  base_rate = current_base_rate  # Базовый курс валютной пары
  gap_percent = (ABSOLUTE_GAP_VALUE / base_rate) * 100
  best_rate.target_rate_percent - gap_percent
end

Чеклист обновлён

• Добавить константу ABSOLUTE_GAP_DIRECTIONS для USDT↔RUB
• Добавить константу ABSOLUTE_GAP_VALUE = 0.0001
• Реализовать метод use_absolute_gap?
• Реализовать метод calculate_with_absolute_gap
• Сохранить текущую логику для остальных направлений
• Добавить защиту от обгона лидера (для всех направлений)
• Добавить обработку "все одинаковые" (для всех направлений)
• Тесты для USDT→RUB, RUB→USDT
• Тесты для других направлений (без изменений)