fix up titles and get users supporting bookmarks

lib/bluesky_hose.ex

···
 
   # Get a demo user for posting bookmarks
   defp get_or_create_demo_user do
-    case Accounts.get_user_by_username("bluesky_bot") do
-      nil ->
-        {:ok, user} = Accounts.create_user(%{
-          username: "bluesky_bot",
-          email: "[e-mail]",
-          display_name: "Bluesky Bot"
-        })
-        user
-      user ->
-        user
-    end
+    alias Bookmarker.Accounts.DemoUsers
+    DemoUsers.get_random_demo_user()
   end
 
   def handle_disconnect(%{reason: {:local, reason}}, state) do

lib/bookmarker/accounts/demo_users.ex

···
+defmodule Bookmarker.Accounts.DemoUsers do
+  @moduledoc """
+  Module for handling demo user creation and selection.
+  """
+
+  alias Bookmarker.Accounts
+  alias Bookmarker.Accounts.User
+  alias Bookmarker.Repo
+  import Ecto.Query
+
+  @demo_user_count 100
+  @demo_user_prefix "demo_user_"
+
+  @doc """
+  Returns a random demo user from the pool of demo users.
+  If demo users don't exist yet, creates them first.
+  """
+  def get_random_demo_user do
+    ensure_demo_users_exist()
+
+    User
+    |> where([u], like(u.username, ^"#{@demo_user_prefix}%"))
+    |> Repo.all()
+    |> Enum.random()
+  end
+
+  @doc """
+  Creates all demo users if they don't exist already.
+  """
+  def ensure_demo_users_exist do
+    # Check if we already have demo users
+    demo_user_count =
+      User
+      |> where([u], like(u.username, ^"#{@demo_user_prefix}%"))
+      |> Repo.aggregate(:count, :id)
+
+    if demo_user_count < @demo_user_count do
+      # Create the remaining demo users
+      Enum.each(demo_user_count..(@demo_user_count - 1), fn index ->
+        username = "#{@demo_user_prefix}#{index + 1}"
+
+        case Accounts.get_user_by_username(username) do
+          nil ->
+            {:ok, _user} = Accounts.create_user(%{
+              username: username,
+              email: "#{username}@example.com",
+              display_name: generate_display_name(index)
+            })
+          _user ->
+            :ok
+        end
+      end)
+    end
+  end
+
+  # Generate a realistic display name
+  defp generate_display_name(index) do
+    first_names = [
+      "Alex", "Jamie", "Jordan", "Taylor", "Casey", "Riley", "Avery", "Quinn",
+      "Morgan", "Reese", "Charlie", "Blake", "Parker", "Dakota", "Peyton", "Elliot",
+      "Rowan", "Cameron", "Sam", "Ashley", "Robin", "Skyler", "Sage", "Jesse"
+    ]
+
+    last_names = [
+      "Smith", "Jones", "Brown", "Johnson", "Williams", "Davis", "Miller", "Wilson",
+      "Taylor", "Moore", "Anderson", "Thomas", "Jackson", "White", "Harris", "Martin",
+      "Thompson", "Garcia", "Martinez", "Robinson", "Clark", "Rodriguez", "Lewis", "Lee"
+    ]
+
+    first_name = Enum.at(first_names, rem(index, length(first_names)))
+    last_name = Enum.at(last_names, rem(div(index, length(first_names)), length(last_names)))
+
+    "#{first_name} #{last_name}"
+  end
+end

lib/bookmarker/bookmarks/bookmark.ex

···
     bookmark
     |> cast(attrs, [:url, :title, :description, :user_id])
     |> validate_required([:url, :title, :user_id])
-    |> validate_length(:title, min: 1, max: 255)
+    |> validate_length(:title, min: 1)
     |> validate_length(:url, min: 5, max: 2048)
     |> foreign_key_constraint(:user_id)
     |> put_assoc(:tags, tags)

lib/bookmarker_web/live/bookmark_live/new.ex

···
   end
 
   defp get_or_create_demo_user do
-    case Accounts.get_user_by_username("demo_user") do
-      nil ->
-        {:ok, user} = Accounts.create_user(%{
-          username: "demo_user",
-          email: "[e-mail]",
-          display_name: "Demo User"
-        })
-        user
-      user ->
-        user
-    end
+    alias Bookmarker.Accounts.DemoUsers
+    DemoUsers.get_random_demo_user()
   end
 end

lib/bookmarker_web/live/bookmark_live/show.ex

···
   end
 
   defp get_or_create_demo_user do
-    case Accounts.get_user_by_username("demo_user") do
-      nil ->
-        {:ok, user} = Accounts.create_user(%{
-          username: "demo_user",
-          email: "[e-mail]",
-          display_name: "Demo User"
-        })
-        user
-      user ->
-        user
-    end
+    alias Bookmarker.Accounts.DemoUsers
+    DemoUsers.get_random_demo_user()
   end
 end

lib/bookmarker_web/live/home_live.ex

···
   end
 
   defp get_or_create_demo_user do
-    case Accounts.get_user_by_username("demo_user") do
-      nil ->
-        {:ok, user} = Accounts.create_user(%{
-          username: "demo_user",
-          email: "[e-mail]",
-          display_name: "Demo User"
-        })
-        user
-      user ->
-        user
-    end
+    alias Bookmarker.Accounts.DemoUsers
+    DemoUsers.get_random_demo_user()
   end
 end

lib/bookmarker_web/live/tag_live/show.ex

···
   end
 
   defp get_or_create_demo_user do
-    case Accounts.get_user_by_username("demo_user") do
-      nil ->
-        {:ok, user} = Accounts.create_user(%{
-          username: "demo_user",
-          email: "[e-mail]",
-          display_name: "Demo User"
-        })
-        user
-      user ->
-        user
-    end
+    alias Bookmarker.Accounts.DemoUsers
+    DemoUsers.get_random_demo_user()
   end
 end

priv/repo/migrations/20250321001052_alter_bookmark_title_to_text.exs

···
+defmodule Bookmarker.Repo.Migrations.AlterBookmarkTitleToText do
+  use Ecto.Migration
+
+  def change do
+    alter table(:bookmarks) do
+      modify :title, :text, null: false
+    end
+  end
+end