Merge pull request #152 from joyofrails/feat/user-ulids

Use ULID for user primary key

Author: rossta

Gemfile

@@ -8,6 +8,7 @@ gem "rails", "~> 7.1", group: [:default, :wasm] # Bundle edge Rails instead: gem
 gem "puma", ">= 5.0" # Use the Puma web server [https://github.com/puma/puma]
 gem "sqlite3", force_ruby_platform: true # Use sqlite3 as the database for Active Record [https://github.com/sparklemotion/sqlite3-ruby]
 gem "activerecord-enhancedsqlite3-adapter" # Enhanced SQLite3 adapter for Active Record [https://github.com/fractaledmind/activerecord-enhancedsqlite3-adapter]
+gem "sqlite-ulid" # A SQLite extension for generating and working with ULIDs [https://github.com/asg017/sqlite-ulid]
 
 gem "solid_cache" # A database-backed ActiveSupport::Cache::Store [https://github.com/rails/solid_cache]
 gem "solid_queue" # A database-backed ActiveJob backend [https://github.com/basecamp/solid_queue]

Gemfile.lock

@@ -457,6 +457,8 @@ GEM
       actionpack (>= 5.2)
       activesupport (>= 5.2)
       sprockets (>= 3.0.0)
+    sqlite-ulid (0.2.1-arm64-darwin)
+    sqlite-ulid (0.2.1-x86_64-linux)
     sqlite3 (1.7.3)
       mini_portile2 (~> 2.8.0)
     standard (1.35.1)
@@ -565,6 +567,7 @@ DEPENDENCIES
   sitepress-rails
   solid_cache
   solid_queue
+  sqlite-ulid
   sqlite3
   standard
   stimulus-rails

config/database.yml

@@ -8,6 +8,8 @@ default: &default
   adapter: sqlite3
   pool: <%= ENV.fetch("RAILS_MAX_THREADS") { 3 } %>
   timeout: 5000
+  extensions:
+    - sqlite_ulid
 
 development:
   primary:

db/migrate/20240605030240_create_users.rb

@@ -1,6 +1,7 @@
 class CreateUsers < ActiveRecord::Migration[7.1]
   def change
-    create_table :users do |t|
+    create_table :users, force: true, id: false do |t|
+      t.primary_key :id, :string, default: -> { "ULID()" }
       t.string :email, null: false
       t.string :password_digest, null: false
 

db/migrate/20240607120410_create_email_exchanges.rb

@@ -2,7 +2,7 @@ class CreateEmailExchanges < ActiveRecord::Migration[7.1]
   def change
     create_table :email_exchanges do |t|
       t.string :email, null: false
-      t.references :user, null: false, foreign_key: true
+      t.references :user, null: false, foreign_key: true, type: :string
       t.string :status, null: false, default: 0 # pending
 
       t.timestamps

db/migrate/20240609132723_create_notifications.rb

@@ -3,7 +3,7 @@ def change
     create_table :notifications do |t|
       t.string :type
       t.references :notification_event, null: false, foreign_key: true
-      t.references :recipient, polymorphic: true, null: false, type: :bigint
+      t.references :recipient, polymorphic: true, null: false, type: :string
       t.datetime :read_at
       t.datetime :seen_at
 

db/schema.rb

@@ -33,12 +33,14 @@ def render_page(*, **)
   end
 
   it "renders contents of file by heterogeneous line number collection" do
-    page = render_page(CodeBlock::AppFile.new("config/database.yml", lines: [7..8, 58]))
+    page = render_page(CodeBlock::AppFile.new("config/database.yml", lines: [7..8, 60]))
     expect(page).to have_content(<<~YAML.strip)
       default: &default
         adapter: sqlite3
     YAML
 
+    puts page
+
     expect(page).to have_content("database: storage/production/data.sqlite3")
     expect(page).not_to have_content("database: storage/development/data.sqlite3")
   end