kortirso · kortirso · Dec 7, 2024 · Dec 7, 2024
diff --git a/app/models/access_token.rb b/app/models/access_token.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class AccessToken < ApplicationRecord
-  include Uuidable
-
   FORMAT_BY_PROVIDER = {
     Providerable::GITHUB => 'github_pat_',
     Providerable::GITLAB => 'glpat-'

diff --git a/app/models/api_access_token.rb b/app/models/api_access_token.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class ApiAccessToken < ApplicationRecord
-  include Uuidable
-
   encrypts :value, deterministic: true
 
   belongs_to :user

diff --git a/app/models/companies/user.rb b/app/models/companies/user.rb
@@ -4,8 +4,6 @@ module Companies
   class User < ApplicationRecord
     self.table_name = :companies_users
 
-    include Uuidable
-
     READ = 'read'
     WRITE = 'write'
 

diff --git a/app/models/company.rb b/app/models/company.rb
@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 class Company < ApplicationRecord
-  include Uuidable
   include Tokenable
   include Insightable
   include Configurable

diff --git a/app/models/concerns/uuidable.rb b/app/models/concerns/uuidable.rb
diff --git a/app/models/entity.rb b/app/models/entity.rb
@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 class Entity < ApplicationRecord
-  include Uuidable
   include Providerable
 
   EMPTY_PAYLOAD = {

diff --git a/app/models/entity/ignore.rb b/app/models/entity/ignore.rb
@@ -4,8 +4,6 @@ class Entity
   class Ignore < ApplicationRecord
     self.table_name = :ignores
 
-    include Uuidable
-
     belongs_to :insightable, polymorphic: true
   end
 end
diff --git a/app/models/excludes/group.rb b/app/models/excludes/group.rb
@@ -4,8 +4,6 @@ module Excludes
   class Group < ApplicationRecord
     self.table_name = :excludes_groups
 
-    include Uuidable
-
     FREE_GROUPS_AMOUNT = 2
 
     belongs_to :insightable, polymorphic: true

diff --git a/app/models/excludes/rule.rb b/app/models/excludes/rule.rb
@@ -4,8 +4,6 @@ module Excludes
   class Rule < ApplicationRecord
     self.table_name = :excludes_rules
 
-    include Uuidable
-
     FREE_RULES_AMOUNT = 3
 
     TITLE_TARGET = 'title'

diff --git a/app/models/invite.rb b/app/models/invite.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class Invite < ApplicationRecord
-  include Uuidable
-
   INVITEABLE_TYPES = %w[
     User
     Company

diff --git a/app/models/issue.rb b/app/models/issue.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class Issue < ApplicationRecord
-  include Uuidable
-
   belongs_to :repository
 
   has_many :comments, class_name: 'Issue::Comment', dependent: :destroy

diff --git a/app/models/issue/comment.rb b/app/models/issue/comment.rb
@@ -2,8 +2,6 @@
 
 class Issue
   class Comment < ApplicationRecord
-    include Uuidable
-
     belongs_to :issue
   end
 end
diff --git a/app/models/kudos/achievement.rb b/app/models/kudos/achievement.rb
@@ -5,7 +5,6 @@ class Achievement < Kudos::ApplicationRecord
     self.table_name = :kudos_achievements
 
     include Kudos
-    include Uuidable
 
     belongs_to :kudos_achievement_group, class_name: 'Kudos::AchievementGroup'
 

diff --git a/app/models/kudos/achievement_group.rb b/app/models/kudos/achievement_group.rb
@@ -4,8 +4,6 @@ module Kudos
   class AchievementGroup < ApplicationRecord
     self.table_name = :kudos_achievement_groups
 
-    include Uuidable
-
     belongs_to :parent,
                class_name: 'Kudos::AchievementGroup',
                foreign_key: :parent_id,

diff --git a/app/models/notification.rb b/app/models/notification.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class Notification < ApplicationRecord
-  include Uuidable
-
   NOTIFYABLE_TYPES = %w[
     Company
     Repository

diff --git a/app/models/pull_request.rb b/app/models/pull_request.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class PullRequest < ApplicationRecord
-  include Uuidable
-
   belongs_to :repository
   belongs_to :entity
 

diff --git a/app/models/repository.rb b/app/models/repository.rb
@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 class Repository < ApplicationRecord
-  include Uuidable
   include Tokenable
   include Insightable
   include Providerable

diff --git a/app/models/user.rb b/app/models/user.rb
@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 class User < ApplicationRecord
-  include Uuidable
   include Kudos::Achievementable
   include Notifyable
   include Inviteable

diff --git a/app/models/user/session.rb b/app/models/user/session.rb
@@ -4,8 +4,6 @@ class User
   class Session < ApplicationRecord
     self.table_name = :users_sessions
 
-    include Uuidable
-
     belongs_to :user
   end
 end
diff --git a/app/models/webhook.rb b/app/models/webhook.rb
@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 class Webhook < ApplicationRecord
-  include Uuidable
-
   CUSTOM = 'custom'
   SLACK = 'slack'
   DISCORD = 'discord'

diff --git a/db/migrate/20241207103004_add_uuids_to_tables.rb b/db/migrate/20241207103004_add_uuids_to_tables.rb
@@ -0,0 +1,38 @@
+class AddUuidsToTables < ActiveRecord::Migration[7.2]
+  disable_ddl_transaction!
+
+  TABLES_WITH_MODELS = {
+    work_times: WorkTime,
+    vacations: User::Vacation,
+    subscriptions: User::Subscription,
+    subscribers: Subscriber,
+    repositories_insights: Repositories::Insight,
+    pull_requests_reviews: PullRequest::Review,
+    pull_requests_comments: PullRequest::Comment,
+    insights: Insight,
+    identities: Identity,
+    feedbacks: User::Feedback
+  }
+
+  def up
+    TABLES_WITH_MODELS.each do |table, model|
+      add_column table, :uuid, :uuid
+      change_column_default table, :uuid, 'gen_random_uuid()'
+      add_index table, :uuid, unique: true, algorithm: :concurrently
+
+      model.unscoped.in_batches do |relation|
+        relation.where(uuid: nil).update_all('uuid = gen_random_uuid()')
+      end
+
+      safety_assured { change_column_null table, :uuid, false }
+    end
+  end
+
+  def down
+    safety_assured do
+      TABLES_WITH_MODELS.each_key do |table|
+        remove_column table, :uuid
+      end
+    end
+  end
+end
diff --git a/db/migrate/20241207105035_change_uuid_defaults.rb b/db/migrate/20241207105035_change_uuid_defaults.rb
@@ -0,0 +1,15 @@
+class ChangeUuidDefaults < ActiveRecord::Migration[7.2]
+  TABLES = %i[
+    webhooks users_sessions users repositories pull_requests
+    notifications issues issue_comments invites ignores excludes_rules
+    excludes_groups entities companies_users companies api_access_tokens access_tokens
+  ]
+
+  def change
+    safety_assured do
+      TABLES.each do |table|
+        change_column_default table, :uuid, 'gen_random_uuid()'
+      end
+    end
+  end
+end
diff --git a/db/migrate/20241207105036_change_uuid_defaults_kudos.rb b/db/migrate/20241207105036_change_uuid_defaults_kudos.rb
@@ -0,0 +1,13 @@
+class ChangeUuidDefaultsKudos < ActiveRecord::Migration[7.2]
+  TABLES = %i[
+    kudos_achievements kudos_achievement_groups
+  ]
+
+  def change
+    safety_assured do
+      TABLES.each do |table|
+        change_column_default table, :uuid, 'gen_random_uuid()'
+      end
+    end
+  end
+end