fix: make Docker build and PostgreSQL migrations work end-to-end

- Update Dockerfile base image to bookworm-20260202 (old tag missing)
- Reorder Dockerfile: run mix compile before assets.deploy so Phoenix
  colocated hooks are generated before esbuild needs them
- Add :phoenix_live_view compiler to mix.exs for colocated JS generation
- Guard Oban SQLite migration with adapter check to skip on PostgreSQL
- Rewrite discovery_sources migration to use ALTER TABLE on PostgreSQL
  instead of SQLite-specific table recreation with TEXT foreign keys

Author: raffaelschneider

Dockerfile

@@ -1,7 +1,7 @@
 # ---- Build Stage ----
 ARG ELIXIR_VERSION=1.19.5
 ARG OTP_VERSION=28.3.1
-ARG DEBIAN_VERSION=bookworm-20241202-slim
+ARG DEBIAN_VERSION=bookworm-20260202-slim
 ARG BUILDER_IMAGE="hexpm/elixir:${ELIXIR_VERSION}-erlang-${OTP_VERSION}-debian-${DEBIAN_VERSION}"
 ARG RUNNER_IMAGE="debian:${DEBIAN_VERSION}"
 
@@ -28,18 +28,18 @@ RUN mkdir config
 COPY config/config.exs config/prod.exs config/
 RUN mix deps.compile
 
-# Copy application code
+# Copy application code and runtime config
 COPY priv priv
 COPY lib lib
 COPY assets assets
-
-# Compile assets
-RUN mix assets.deploy
-
-# Compile the release
 COPY config/runtime.exs config/
+
+# Compile the application (generates phoenix-colocated hooks needed by esbuild)
 RUN mix compile
 
+# Compile assets (must run after mix compile for colocated hooks)
+RUN mix assets.deploy
+
 # Build the release
 RUN mix release
 

mix.exs

@@ -10,6 +10,7 @@ defmodule SentinelCp.MixProject do
       start_permanent: Mix.env() == :prod,
       aliases: aliases(),
       deps: deps(),
+      compilers: [:phoenix_live_view] ++ Mix.compilers(),
       listeners: [Phoenix.CodeReloader]
     ]
   end

priv/repo/migrations/20260116063800_create_oban_jobs.exs

@@ -2,10 +2,19 @@ defmodule SentinelCp.Repo.Migrations.CreateObanJobs do
   use Ecto.Migration
 
   def up do
-    Oban.Migrations.SQLite.up(version: 1)
+    if Application.get_env(:sentinel_cp, :ecto_adapter) == Ecto.Adapters.Postgres do
+      # PostgreSQL Oban tables are handled by the earlier migration
+      :ok
+    else
+      Oban.Migrations.SQLite.up(version: 1)
+    end
   end
 
   def down do
-    Oban.Migrations.SQLite.down(version: 1)
+    if Application.get_env(:sentinel_cp, :ecto_adapter) == Ecto.Adapters.Postgres do
+      :ok
+    else
+      Oban.Migrations.SQLite.down(version: 1)
+    end
   end
 end

priv/repo/migrations/20260213140003_add_discovery_source_config.exs

@@ -2,74 +2,87 @@ defmodule SentinelCp.Repo.Migrations.AddDiscoverySourceConfig do
   use Ecto.Migration
 
   def up do
-    # SQLite doesn't support ALTER COLUMN, so we recreate the table
-    # to make hostname nullable (K8s sources don't use it)
-    execute("""
-    CREATE TABLE discovery_sources_new (
-      id TEXT PRIMARY KEY,
-      source_type TEXT NOT NULL DEFAULT 'dns_srv',
-      hostname TEXT,
-      config TEXT DEFAULT '{}',
-      sync_interval_seconds INTEGER DEFAULT 60,
-      auto_sync INTEGER DEFAULT 1,
-      last_synced_at TEXT,
-      last_sync_status TEXT DEFAULT 'pending',
-      last_sync_error TEXT,
-      last_sync_targets_count INTEGER DEFAULT 0,
-      upstream_group_id TEXT NOT NULL REFERENCES upstream_groups(id) ON DELETE CASCADE,
-      project_id TEXT NOT NULL REFERENCES projects(id) ON DELETE CASCADE,
-      inserted_at TEXT NOT NULL,
-      updated_at TEXT NOT NULL
-    )
-    """)
+    if Application.get_env(:sentinel_cp, :ecto_adapter) == Ecto.Adapters.Postgres do
+      # PostgreSQL supports ALTER COLUMN directly
+      alter table(:discovery_sources) do
+        modify :hostname, :string, null: true
+        add_if_not_exists :config, :map, default: %{}
+      end
+    else
+      # SQLite doesn't support ALTER COLUMN, so we recreate the table
+      execute("""
+      CREATE TABLE discovery_sources_new (
+        id TEXT PRIMARY KEY,
+        source_type TEXT NOT NULL DEFAULT 'dns_srv',
+        hostname TEXT,
+        config TEXT DEFAULT '{}',
+        sync_interval_seconds INTEGER DEFAULT 60,
+        auto_sync INTEGER DEFAULT 1,
+        last_synced_at TEXT,
+        last_sync_status TEXT DEFAULT 'pending',
+        last_sync_error TEXT,
+        last_sync_targets_count INTEGER DEFAULT 0,
+        upstream_group_id TEXT NOT NULL REFERENCES upstream_groups(id) ON DELETE CASCADE,
+        project_id TEXT NOT NULL REFERENCES projects(id) ON DELETE CASCADE,
+        inserted_at TEXT NOT NULL,
+        updated_at TEXT NOT NULL
+      )
+      """)
 
-    execute("""
-    INSERT INTO discovery_sources_new
-    SELECT id, source_type, hostname, '{}', sync_interval_seconds, auto_sync,
-           last_synced_at, last_sync_status, last_sync_error, last_sync_targets_count,
-           upstream_group_id, project_id, inserted_at, updated_at
-    FROM discovery_sources
-    """)
+      execute("""
+      INSERT INTO discovery_sources_new
+      SELECT id, source_type, hostname, '{}', sync_interval_seconds, auto_sync,
+             last_synced_at, last_sync_status, last_sync_error, last_sync_targets_count,
+             upstream_group_id, project_id, inserted_at, updated_at
+      FROM discovery_sources
+      """)
 
-    execute("DROP TABLE discovery_sources")
-    execute("ALTER TABLE discovery_sources_new RENAME TO discovery_sources")
+      execute("DROP TABLE discovery_sources")
+      execute("ALTER TABLE discovery_sources_new RENAME TO discovery_sources")
 
-    # Recreate indexes
-    create index(:discovery_sources, [:project_id])
-    create unique_index(:discovery_sources, [:upstream_group_id])
+      create index(:discovery_sources, [:project_id])
+      create unique_index(:discovery_sources, [:upstream_group_id])
+    end
   end
 
   def down do
-    execute("""
-    CREATE TABLE discovery_sources_old (
-      id TEXT PRIMARY KEY,
-      source_type TEXT NOT NULL DEFAULT 'dns_srv',
-      hostname TEXT NOT NULL,
-      sync_interval_seconds INTEGER DEFAULT 60,
-      auto_sync INTEGER DEFAULT 1,
-      last_synced_at TEXT,
-      last_sync_status TEXT DEFAULT 'pending',
-      last_sync_error TEXT,
-      last_sync_targets_count INTEGER DEFAULT 0,
-      upstream_group_id TEXT NOT NULL REFERENCES upstream_groups(id) ON DELETE CASCADE,
-      project_id TEXT NOT NULL REFERENCES projects(id) ON DELETE CASCADE,
-      inserted_at TEXT NOT NULL,
-      updated_at TEXT NOT NULL
-    )
-    """)
+    if Application.get_env(:sentinel_cp, :ecto_adapter) == Ecto.Adapters.Postgres do
+      alter table(:discovery_sources) do
+        modify :hostname, :string, null: false
+        remove :config
+      end
+    else
+      execute("""
+      CREATE TABLE discovery_sources_old (
+        id TEXT PRIMARY KEY,
+        source_type TEXT NOT NULL DEFAULT 'dns_srv',
+        hostname TEXT NOT NULL,
+        sync_interval_seconds INTEGER DEFAULT 60,
+        auto_sync INTEGER DEFAULT 1,
+        last_synced_at TEXT,
+        last_sync_status TEXT DEFAULT 'pending',
+        last_sync_error TEXT,
+        last_sync_targets_count INTEGER DEFAULT 0,
+        upstream_group_id TEXT NOT NULL REFERENCES upstream_groups(id) ON DELETE CASCADE,
+        project_id TEXT NOT NULL REFERENCES projects(id) ON DELETE CASCADE,
+        inserted_at TEXT NOT NULL,
+        updated_at TEXT NOT NULL
+      )
+      """)
 
-    execute("""
-    INSERT INTO discovery_sources_old
-    SELECT id, source_type, COALESCE(hostname, ''), sync_interval_seconds, auto_sync,
-           last_synced_at, last_sync_status, last_sync_error, last_sync_targets_count,
-           upstream_group_id, project_id, inserted_at, updated_at
-    FROM discovery_sources
-    """)
+      execute("""
+      INSERT INTO discovery_sources_old
+      SELECT id, source_type, COALESCE(hostname, ''), sync_interval_seconds, auto_sync,
+             last_synced_at, last_sync_status, last_sync_error, last_sync_targets_count,
+             upstream_group_id, project_id, inserted_at, updated_at
+      FROM discovery_sources
+      """)
 
-    execute("DROP TABLE discovery_sources")
-    execute("ALTER TABLE discovery_sources_old RENAME TO discovery_sources")
+      execute("DROP TABLE discovery_sources")
+      execute("ALTER TABLE discovery_sources_old RENAME TO discovery_sources")
 
-    create index(:discovery_sources, [:project_id])
-    create unique_index(:discovery_sources, [:upstream_group_id])
+      create index(:discovery_sources, [:project_id])
+      create unique_index(:discovery_sources, [:upstream_group_id])
+    end
   end
 end