fix: adjust array symbol syntax issues

Bulletdev · Bulletdev · commit cfbdadf3b01b · 2026-02-15T20:31:37.000-03:00
diff --git a/app/controllers/api/v1/strategy/draft_plans_controller.rb b/app/controllers/api/v1/strategy/draft_plans_controller.rb
@@ -180,10 +180,10 @@ def draft_plan_params
             our_bans: [],
             opponent_bans: [],
             priority_picks: {},
-            if_then_scenarios: [
-              :trigger,
-              :action,
-              :note
+            if_then_scenarios: %i[
+              trigger
+              action
+              note
             ]
           )
         end
diff --git a/app/modules/scouting/controllers/watchlist_controller.rb b/app/modules/scouting/controllers/watchlist_controller.rb
@@ -6,7 +6,7 @@ module Scouting
       # Watchlist Controller
       # Manages organization-specific player scouting watchlists
       class WatchlistController < Api::V1::BaseController
-        before_action :set_authorized_target, only: [:create, :destroy]
+        before_action :set_authorized_target, only: %i[create destroy]
         # GET /api/v1/scouting/watchlist
         # Returns high-priority scouting targets in org's watchlist
         def index
diff --git a/app/modules/strategy/controllers/draft_plans_controller.rb b/app/modules/strategy/controllers/draft_plans_controller.rb
@@ -180,10 +180,10 @@ def draft_plan_params
             our_bans: [],
             opponent_bans: [],
             priority_picks: {},
-            if_then_scenarios: [
-              :trigger,
-              :action,
-              :note
+            if_then_scenarios: %i[
+              trigger
+              action
+              note
             ]
           )
         end
diff --git a/config/initializers/rack_attack.rb b/config/initializers/rack_attack.rb
@@ -98,7 +98,7 @@ class Attack
       req = payload[:request]
 
       # Only log if request was actually blocked or throttled
-      if [:throttle, :blocklist].include?(req.env['rack.attack.match_type'])
+      if %i[throttle blocklist].include?(req.env['rack.attack.match_type'])
         discriminator = req.env['rack.attack.matched']
         Rails.logger.warn "[Rack::Attack] #{req.env['rack.attack.match_type'].to_s.capitalize} #{discriminator}: #{req.env['REQUEST_METHOD']} #{req.url} from #{req.ip}"
       end
diff --git a/config/routes.rb b/config/routes.rb
@@ -106,7 +106,7 @@
         end
 
         # FAQ
-        resources :faq, only: [:index, :show], param: :slug, controller: 'faqs' do
+        resources :faq, only: %i[index show], param: :slug, controller: 'faqs' do
           member do
             post :helpful, to: 'faqs#mark_helpful'
             post 'not-helpful', to: 'faqs#mark_not_helpful'
diff --git a/db/migrate/20260111120000_create_support_tickets.rb b/db/migrate/20260111120000_create_support_tickets.rb
@@ -34,7 +34,7 @@ def change
     add_index :support_tickets, :category
     add_index :support_tickets, :priority
     add_index :support_tickets, :deleted_at
-    add_index :support_tickets, [:organization_id, :status]
-    add_index :support_tickets, [:assigned_to_id, :status]
+    add_index :support_tickets, %i[organization_id status]
+    add_index :support_tickets, %i[assigned_to_id status]
   end
 end
diff --git a/db/migrate/20260111120001_create_support_ticket_messages.rb b/db/migrate/20260111120001_create_support_ticket_messages.rb
@@ -16,7 +16,7 @@ def change
       t.timestamps
     end
 
-    add_index :support_ticket_messages, [:support_ticket_id, :created_at]
+    add_index :support_ticket_messages, %i[support_ticket_id created_at]
     add_index :support_ticket_messages, :message_type
   end
 end
diff --git a/db/migrate/20260111120002_create_support_faqs.rb b/db/migrate/20260111120002_create_support_faqs.rb
@@ -27,6 +27,6 @@ def change
     add_index :support_faqs, :slug, unique: true
     add_index :support_faqs, :category
     add_index :support_faqs, :locale
-    add_index :support_faqs, [:published, :position]
+    add_index :support_faqs, %i[published position]
   end
 end
diff --git a/db/migrate/20260118173507_create_draft_plans.rb b/db/migrate/20260118173507_create_draft_plans.rb
@@ -22,8 +22,8 @@ def change
     add_foreign_key :draft_plans, :users, column: :updated_by_id
 
     add_index :draft_plans, :organization_id
-    add_index :draft_plans, [:organization_id, :opponent_team]
-    add_index :draft_plans, [:organization_id, :is_active]
+    add_index :draft_plans, %i[organization_id opponent_team]
+    add_index :draft_plans, %i[organization_id is_active]
     add_index :draft_plans, :patch_version
     add_index :draft_plans, :created_by_id
     add_index :draft_plans, :updated_by_id
diff --git a/db/migrate/20260118173516_create_tactical_boards.rb b/db/migrate/20260118173516_create_tactical_boards.rb
@@ -21,7 +21,7 @@ def change
     add_foreign_key :tactical_boards, :users, column: :updated_by_id
 
     add_index :tactical_boards, :organization_id
-    add_index :tactical_boards, [:organization_id, :created_at]
+    add_index :tactical_boards, %i[organization_id created_at]
     add_index :tactical_boards, :match_id
     add_index :tactical_boards, :scrim_id
     add_index :tactical_boards, :created_by_id
diff --git a/db/migrate/20260125000003_add_unique_index_to_scouting_targets_riot_puuid.rb b/db/migrate/20260125000003_add_unique_index_to_scouting_targets_riot_puuid.rb
@@ -7,7 +7,7 @@ def change
 
     # Add unique composite index scoped to organization
     # This allows the same player (PUUID) to be a scouting target in multiple organizations
-    add_index :scouting_targets, [:organization_id, :riot_puuid],
+    add_index :scouting_targets, %i[organization_id riot_puuid],
               unique: true,
               name: 'index_scouting_targets_on_org_and_puuid',
               where: "riot_puuid IS NOT NULL"
diff --git a/db/migrate/20260203030000_add_query_performance_indexes.rb b/db/migrate/20260203030000_add_query_performance_indexes.rb
@@ -3,37 +3,37 @@
 class AddQueryPerformanceIndexes < ActiveRecord::Migration[7.0]
   def change
     # Índices para otimizar queries de players com organization_id + deleted_at
-    add_index :players, [:organization_id, :deleted_at],
+    add_index :players, %i[organization_id deleted_at],
               name: 'idx_players_org_deleted',
               if_not_exists: true
 
     # Índice composto para queries de players ativos por organização
-    add_index :players, [:organization_id, :deleted_at, :status],
+    add_index :players, %i[organization_id deleted_at status],
               name: 'idx_players_org_deleted_status',
               if_not_exists: true
 
     # Índices para sync_status
-    add_index :players, [:organization_id, :sync_status],
+    add_index :players, %i[organization_id sync_status],
               name: 'idx_players_org_sync_status',
               if_not_exists: true
 
     # Índice para last_sync_at (usado em verificações de sync)
-    add_index :players, [:organization_id, :last_sync_at],
+    add_index :players, %i[organization_id last_sync_at],
               name: 'idx_players_org_last_sync',
               if_not_exists: true
 
     # Índice para contract_end_date (usado em alertas de contrato)
-    add_index :players, [:organization_id, :contract_end_date],
+    add_index :players, %i[organization_id contract_end_date],
               name: 'idx_players_org_contract_end',
               if_not_exists: true
 
     # Índice para queries de matches por created_at (usado em tier_limits)
-    add_index :matches, [:organization_id, :created_at],
+    add_index :matches, %i[organization_id created_at],
               name: 'idx_matches_org_created',
               if_not_exists: true
 
     # Índice para schedules por event_type
-    add_index :schedules, [:organization_id, :event_type],
+    add_index :schedules, %i[organization_id event_type],
               name: 'idx_schedules_org_event_type',
               if_not_exists: true
   end
diff --git a/db/migrate/20260203040000_add_advanced_performance_indexes.rb b/db/migrate/20260203040000_add_advanced_performance_indexes.rb
@@ -5,33 +5,33 @@ def change
 
     # 1. Índice parcial para players ativos (WHERE deleted_at IS NULL)
     # Complementa idx_players_org_deleted com filtro parcial
-    add_index :players, [:organization_id, :deleted_at],
+    add_index :players, %i[organization_id deleted_at],
       name: 'idx_players_org_deleted_active',
       where: "deleted_at IS NULL",
       if_not_exists: true,
       comment: 'Índice parcial para COUNT de players ativos'
 
     # 2. matches com game_start e victory (usado em analytics de winrate)
-    add_index :matches, [:organization_id, :game_start, :victory],
+    add_index :matches, %i[organization_id game_start victory],
       name: 'idx_matches_org_game_start_victory',
       if_not_exists: true,
       comment: 'Otimiza queries de winrate por período'
 
     # 3. team_goals por status (dashboard de metas)
-    add_index :team_goals, [:organization_id, :status],
+    add_index :team_goals, %i[organization_id status],
       name: 'idx_team_goals_org_status',
       if_not_exists: true,
       comment: 'Otimiza COUNT de goals por status'
 
     # 4. schedules com start_time e event_type (calendário)
-    add_index :schedules, [:organization_id, :start_time, :event_type],
+    add_index :schedules, %i[organization_id start_time event_type],
       name: 'idx_schedules_org_time_type',
       if_not_exists: true,
       comment: 'Otimiza queries de próximos eventos'
 
     # 5. player_match_stats para agregações (SUM/AVG de estatísticas)
     # Complementa os índices existentes com ordem otimizada para aggregations
-    add_index :player_match_stats, [:match_id, :player_id],
+    add_index :player_match_stats, %i[match_id player_id],
       name: 'idx_player_stats_match_player_agg',
       if_not_exists: true,
       comment: 'Otimiza agregações de estatísticas (SUM kills/deaths/assists)'
diff --git a/db/migrate/20260208122755_create_scouting_watchlists.rb b/db/migrate/20260208122755_create_scouting_watchlists.rb
@@ -17,7 +17,7 @@ def change
       t.timestamps
 
       # Ensure one watchlist entry per org per target
-      t.index [:organization_id, :scouting_target_id], unique: true, name: 'index_watchlists_on_org_and_target'
+      t.index %i[organization_id scouting_target_id], unique: true, name: 'index_watchlists_on_org_and_target'
       t.index :priority
       t.index :status
       t.index :last_reviewed
diff --git a/db/migrate/20260212000000_remove_duplicate_indexes.rb b/db/migrate/20260212000000_remove_duplicate_indexes.rb
@@ -22,16 +22,16 @@ def up
 
   def down
     # Re-create the removed indexes if rollback is needed
-    add_index :matches, [:organization_id, :game_start], name: "index_matches_on_org_and_game_start", if_not_exists: true
-    add_index :matches, [:organization_id, :victory], name: "index_matches_on_org_and_victory", if_not_exists: true
+    add_index :matches, %i[organization_id game_start], name: "index_matches_on_org_and_game_start", if_not_exists: true
+    add_index :matches, %i[organization_id victory], name: "index_matches_on_org_and_victory", if_not_exists: true
 
     add_index :player_match_stats, :match_id, name: "index_player_match_stats_on_match", if_not_exists: true
     add_index :player_match_stats, :match_id, name: "index_player_match_stats_on_match_id", if_not_exists: true
 
-    add_index :players, [:organization_id, :status], name: "index_players_on_org_and_status", if_not_exists: true
+    add_index :players, %i[organization_id status], name: "index_players_on_org_and_status", if_not_exists: true
 
-    add_index :schedules, [:organization_id, :scheduled_time, :schedule_type], name: "index_schedules_on_org_time_type", if_not_exists: true
+    add_index :schedules, %i[organization_id scheduled_time schedule_type], name: "index_schedules_on_org_time_type", if_not_exists: true
 
-    add_index :team_goals, [:organization_id, :status], name: "index_team_goals_on_org_and_status", if_not_exists: true
+    add_index :team_goals, %i[organization_id status], name: "index_team_goals_on_org_and_status", if_not_exists: true
   end
 end
diff --git a/db/migrate/20260214171659_add_performance_indexes.rb b/db/migrate/20260214171659_add_performance_indexes.rb
@@ -6,13 +6,13 @@ class AddPerformanceIndexes < ActiveRecord::Migration[7.2]
   def up
     # Index for matches COUNT query (26,444 calls, 8.9s total)
     # Query: SELECT COUNT(*) FROM matches WHERE organization_id = ? AND created_at > ?
-    add_index :matches, [:organization_id, :created_at],
+    add_index :matches, %i[organization_id created_at],
               name: 'idx_matches_org_created',
               algorithm: :concurrently,
               if_not_exists: true
 
     # Additional helpful index for matches ordering
-    add_index :matches, [:organization_id, :id],
+    add_index :matches, %i[organization_id id],
               name: 'idx_matches_org_id',
               algorithm: :concurrently,
               if_not_exists: true
diff --git a/lib/tasks/bot_stats.rake b/lib/tasks/bot_stats.rake
@@ -51,7 +51,7 @@ namespace :bot_stats do
   end
 
   desc 'Show bot statistics for date range'
-  task :range, [:start_date, :end_date] => :environment do |_t, args|
+  task :range, %i[start_date end_date] => :environment do |_t, args|
     unless ENV['REDIS_URL'] && ENV['TRACK_BOT_STATS'] == 'true'
       puts '  Bot stats tracking is not enabled.'
       exit
