Media page fixes, and removal of timex

config/config.exs

@@ -19,6 +19,9 @@ config :phoenix_kit, PhoenixKit.Mailer, adapter: Swoosh.Adapters.Local
 
 # Note: Hammer rate limiting configuration is automatically added to parent
 # applications via mix phoenix_kit.install/update tasks
+# For standalone development, configure Hammer here:
+config :hammer,
+  backend: {Hammer.Backend.ETS, [expiry_ms: 60_000 * 60 * 4, cleanup_interval_ms: 60_000 * 10]}
 
 # Configure Ueberauth (minimal configuration for compilation)
 # Applications using PhoenixKit should configure their own providers

lib/mix/tasks/phoenix_kit.install.ex

@@ -57,6 +57,7 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
       MailerConfig,
       MigrationStrategy,
       OAuthConfig,
+      ObanConfig,
       RateLimiterConfig,
       RepoDetection,
       RouterIntegration
@@ -95,7 +96,9 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
       |> MailerConfig.add_mailer_configuration()
       |> RateLimiterConfig.add_rate_limiter_configuration()
       |> OAuthConfig.add_oauth_configuration()
+      |> ObanConfig.add_oban_configuration()
       |> ApplicationSupervisor.add_supervisor()
+      |> ObanConfig.add_oban_supervisor()
       |> LayoutConfig.add_layout_integration_configuration()
       |> CssIntegration.add_automatic_css_integration()
       |> DemoFiles.copy_test_demo_files()
@@ -274,6 +277,7 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
       PhoenixKit requires configuration for:
       - Ueberauth (OAuth authentication)
       - Hammer (rate limiting)
+      - Oban (background jobs for file processing)
 
       This configuration will be added now.
 
@@ -316,6 +320,10 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
           !has_active_hammer_config?(lines) ->
             :missing
 
+          # Missing Oban configuration (check for active, non-commented config)
+          !has_active_oban_config?(lines) ->
+            :missing
+
           # All required configuration present
           true ->
             :ok
@@ -348,6 +356,26 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
       has_hammer_config and has_expiry_ms
     end
 
+    # Check if active (non-commented) Oban configuration exists
+    defp has_active_oban_config?(lines) do
+      has_oban_config =
+        Enum.any?(lines, fn line ->
+          trimmed = String.trim(line)
+          # Not a comment and contains config :phoenix_kit, Oban
+          !String.starts_with?(trimmed, "#") and
+            String.contains?(line, "config :phoenix_kit, Oban")
+        end)
+
+      has_queues =
+        Enum.any?(lines, fn line ->
+          trimmed = String.trim(line)
+          # Not a comment and contains queues:
+          !String.starts_with?(trimmed, "#") and String.contains?(line, "queues:")
+        end)
+
+      has_oban_config and has_queues
+    end
+
     # Add completion notice with essential next steps (reduced duplication)
     defp add_completion_notice(igniter) do
       notice = """

lib/mix/tasks/phoenix_kit.update.ex

@@ -88,6 +88,7 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
       BasicConfiguration,
       Common,
       CssIntegration,
+      ObanConfig,
       RateLimiterConfig
     }
 
@@ -242,6 +243,10 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
           !has_active_hammer_config?(lines) ->
             :missing
 
+          # Missing Oban configuration (check for active, non-commented config)
+          !has_active_oban_config?(lines) ->
+            :missing
+
           # All required configuration present
           true ->
             :ok
@@ -274,6 +279,26 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
       has_hammer_config and has_expiry_ms
     end
 
+    # Check if active (non-commented) Oban configuration exists
+    defp has_active_oban_config?(lines) do
+      has_oban_config =
+        Enum.any?(lines, fn line ->
+          trimmed = String.trim(line)
+          # Not a comment and contains config :phoenix_kit, Oban
+          !String.starts_with?(trimmed, "#") and
+            String.contains?(line, "config :phoenix_kit, Oban")
+        end)
+
+      has_queues =
+        Enum.any?(lines, fn line ->
+          trimmed = String.trim(line)
+          # Not a comment and contains queues:
+          !String.starts_with?(trimmed, "#") and String.contains?(line, "queues:")
+        end)
+
+      has_oban_config and has_queues
+    end
+
     # Perform the igniter-based update logic
     defp perform_igniter_update(igniter, opts) do
       prefix = opts[:prefix] || "public"
@@ -285,6 +310,9 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
       # Ensure Hammer rate limiter configuration exists
       igniter = validate_and_add_hammer_config(igniter)
 
+      # Ensure Oban configuration exists
+      igniter = validate_and_add_oban_config(igniter)
+
       # Check if this is the first pass (config missing) or second pass (config exists)
       config_status = Process.get(:phoenix_kit_config_status, :ok)
 
@@ -860,6 +888,53 @@ if Code.ensure_loaded?(Igniter.Mix.Task) do
 
       Igniter.add_notice(igniter, String.trim(notice))
     end
+
+    # Validate and add Oban configuration if missing
+    defp validate_and_add_oban_config(igniter) do
+      config_exists = ObanConfig.oban_config_exists?(igniter)
+      supervisor_exists = ObanConfig.oban_supervisor_exists?(igniter)
+
+      igniter =
+        if config_exists do
+          igniter
+        else
+          # Configuration missing, add it
+          igniter
+          |> ObanConfig.add_oban_configuration()
+          |> add_oban_config_added_notice()
+        end
+
+      # Check and add supervisor separately
+      if supervisor_exists do
+        igniter
+      else
+        igniter
+        |> ObanConfig.add_oban_supervisor()
+        |> add_oban_supervisor_added_notice()
+      end
+    end
+
+    # Add notice about Oban configuration being added
+    defp add_oban_config_added_notice(igniter) do
+      notice = """
+      ⚠️  Added missing Oban configuration to config.exs
+         IMPORTANT: Restart your server if it's currently running.
+         Without Oban, the storage system cannot process uploaded files.
+      """
+
+      Igniter.add_notice(igniter, String.trim(notice))
+    end
+
+    # Add notice about Oban supervisor being added
+    defp add_oban_supervisor_added_notice(igniter) do
+      notice = """
+      ⚠️  Added Oban to application supervisor tree in application.ex
+         IMPORTANT: Restart your server if it's currently running.
+         Oban will now start automatically with your application.
+      """
+
+      Igniter.add_notice(igniter, String.trim(notice))
+    end
   end
 
   # Fallback module for when Igniter is not available