fix: Updated both job files 4 sidekiq (#13)

* feat: implement datadragon and player sync

  1. Data Dragon Service - Busca dados estáticos da Riot
  2. Região Configurável - Não mais hardcoded, usa player.region
  3. Champion Mapping - Funcionando corretamente agora
  4. 8 Novos API Endpoints - Para dados da Riot
  5. 5 Rake Tasks - Gerenciamento de cache e sync
  6. Suite de Testes - Criada para SyncPlayerFromRiotJob

* chore: add admin bypass ruleset

* feat(players): extract riot sync service

- Create Players::Services::RiotSyncService
- Extract Riot API logic from controller
- Support import, sync, search operations
- Add retry logic for better success

* feat(players): extract stats service

- Create Players::Services::StatsService (90 lines)
- Extract statistics calculation
- Calculate win rate, KDA, recent form
- Make stats reusable and testable

* refactor(players): migrate controller to module

- Move to Players::Controllers namespace
- Reduce from 750 to 350 lines (-53%)
- Use services for business logic
- Create proxy for backwards compatibility

* refactor(players): migrate serializers and jobs

- Move serializers to Players module
- Move jobs to Players module
- Organize all players domain code

* refactor(auth): migrate authentication module

- Move AuthController to module
- Move User and Organization serializers
- JwtService already in module
- Create proxy controller

* refactor: migrate matches, schedules, team goals

- Migrate 3 core business modules
- Organize by domain functionality
- Create proxies for all

* refactor: migrate analytics module

- Migrate 7 analytics controllers
- Organize performance analysis
- Maintain namespace structure

* refactor: migrate scouting module

- Migrate scouting controllers
- Move serializers and jobs
- Organize talent discovery

* refactor: migrate riot integration

- Migrate 2 Riot controllers
- Move 2 services to module
- Consolidate Riot API functionality

* refactor: migrate vod reviews and dashboard

- Migrate VOD Reviews (2 controllers)
- Migrate Dashboard module
- Organize video review features

* refactor: migrate dashboard  to modular arc

- Migrate Dashboard module

* docs: finalize migration and update gitignore

- Update gitignore for backups
- Complete modular architecture migration

* fix: Updated both job files 4 sidekiq (#12)

1. Use /lol/league/v4/entries/by-puuid/{puuid} instead of /by-summoner/{id}
  2. Added new method fetch_ranked_stats_by_puuid as a workaround

Author: Bulletdev

app/jobs/sync_player_from_riot_job.rb

@@ -4,14 +4,12 @@ class SyncPlayerFromRiotJob < ApplicationJob
   def perform(player_id)
     player = Player.find(player_id)
 
-    # Check if player has necessary data
     unless player.riot_puuid.present? || player.summoner_name.present?
       player.update(sync_status: 'error', last_sync_at: Time.current)
       Rails.logger.error "Player #{player_id} missing Riot info"
       return
     end
 
-    # Get Riot API key
     riot_api_key = ENV['RIOT_API_KEY']
     unless riot_api_key.present?
       player.update(sync_status: 'error', last_sync_at: Time.current)
@@ -20,20 +18,18 @@ def perform(player_id)
     end
 
     begin
-      # Use player's region or default to BR1
       region = player.region.presence&.downcase || 'br1'
 
-      # Fetch summoner data
       if player.riot_puuid.present?
         summoner_data = fetch_summoner_by_puuid(player.riot_puuid, region, riot_api_key)
       else
         summoner_data = fetch_summoner_by_name(player.summoner_name, region, riot_api_key)
       end
 
-      # Fetch ranked stats
-      ranked_data = fetch_ranked_stats(summoner_data['id'], region, riot_api_key)
+      # Use PUUID for league endpoint (workaround for Riot API bug where summoner_data['id'] is nil)
+      # See: https://github.com/RiotGames/developer-relations/issues/1092
+      ranked_data = fetch_ranked_stats_by_puuid(player.riot_puuid, region, riot_api_key)
 
-      # Update player data
       update_data = {
         riot_puuid: summoner_data['puuid'],
         riot_summoner_id: summoner_data['id'],
@@ -43,7 +39,6 @@ def perform(player_id)
         last_sync_at: Time.current
       }
 
-      # Update ranked stats if available
       solo_queue = ranked_data.find { |q| q['queueType'] == 'RANKED_SOLO_5x5' }
       if solo_queue
         update_data.merge!({
@@ -85,7 +80,6 @@ def fetch_summoner_by_name(summoner_name, region, api_key)
     game_name, tag_line = summoner_name.split('#')
     tag_line ||= region.upcase
 
-    # Get PUUID from Riot ID
     account_url = "https://americas.api.riotgames.com/riot/account/v1/accounts/by-riot-id/#{URI.encode_www_form_component(game_name)}/#{URI.encode_www_form_component(tag_line)}"
     account_uri = URI(account_url)
     account_request = Net::HTTP::Get.new(account_uri)
@@ -144,4 +138,24 @@ def fetch_ranked_stats(summoner_id, region, api_key)
 
     JSON.parse(response.body)
   end
+
+  def fetch_ranked_stats_by_puuid(puuid, region, api_key)
+    require 'net/http'
+    require 'json'
+
+    url = "https://#{region}.api.riotgames.com/lol/league/v4/entries/by-puuid/#{puuid}"
+    uri = URI(url)
+    request = Net::HTTP::Get.new(uri)
+    request['X-Riot-Token'] = api_key
+
+    response = Net::HTTP.start(uri.hostname, uri.port, use_ssl: true) do |http|
+      http.request(request)
+    end
+
+    unless response.is_a?(Net::HTTPSuccess)
+      raise "Riot API Error: #{response.code} - #{response.body}"
+    end
+
+    JSON.parse(response.body)
+  end
 end

app/modules/players/jobs/sync_player_from_riot_job.rb

@@ -26,7 +26,9 @@ def perform(player_id)
         summoner_data = fetch_summoner_by_name(player.summoner_name, region, riot_api_key)
       end
 
-      ranked_data = fetch_ranked_stats(summoner_data['id'], region, riot_api_key)
+      # Use PUUID for league endpoint (workaround for Riot API bug where summoner_data['id'] is nil)
+      # See: https://github.com/RiotGames/developer-relations/issues/1092
+      ranked_data = fetch_ranked_stats_by_puuid(player.riot_puuid, region, riot_api_key)
 
       update_data = {
         riot_puuid: summoner_data['puuid'],
@@ -136,4 +138,24 @@ def fetch_ranked_stats(summoner_id, region, api_key)
 
     JSON.parse(response.body)
   end
+
+  def fetch_ranked_stats_by_puuid(puuid, region, api_key)
+    require 'net/http'
+    require 'json'
+
+    url = "https://#{region}.api.riotgames.com/lol/league/v4/entries/by-puuid/#{puuid}"
+    uri = URI(url)
+    request = Net::HTTP::Get.new(uri)
+    request['X-Riot-Token'] = api_key
+
+    response = Net::HTTP.start(uri.hostname, uri.port, use_ssl: true) do |http|
+      http.request(request)
+    end
+
+    unless response.is_a?(Net::HTTPSuccess)
+      raise "Riot API Error: #{response.code} - #{response.body}"
+    end
+
+    JSON.parse(response.body)
+  end
 end

docker-compose.yml

@@ -67,7 +67,7 @@ services:
         condition: service_healthy
       redis:
         condition: service_healthy
-    command: bundle exec sidekiq
+    command: bundle exec sidekiq -C config/sidekiq.yml
 
 volumes:
   postgres_data: