AI docker changes

Dockerfile

@@ -54,16 +54,11 @@ RUN chmod +x bin/*
 # -j 1 disable parallel compilation to avoid a QEMU bug: https://github.com/rails/bootsnap/issues/495
 RUN bundle exec bootsnap precompile -j 1 app/ lib/
 
-# Precompiling assets for production without requiring secret RAILS_MASTER_KEY
-# Użycie BuildKit secret dla dummy secret_key_base
-
-ENV ASSETS_PRECOMPILE=true
-
-RUN --mount=type=secret,id=SECRET_KEY_BASE,mode=0444,required=true \
-    --mount=type=secret,id=RAILS_MASTER_KEY,mode=0444,required=true \
-    SECRET_KEY_BASE=$(cat /run/secrets/SECRET_KEY_BASE) \
-    RAILS_MASTER_KEY=$(cat /run/secrets/RAILS_MASTER_KEY) \
-    ./bin/rails assets:precompile
+# NOTE: BuildKit secret mounts are not available in all build environments
+# (for example, Hugging Face Spaces). To keep builds portable we skip
+# precompiling assets at build-time here and instead perform a best-effort
+# assets precompile at container start. This makes the image buildable
+# on Spaces without requiring BuildKit secret support.
 
 # Final stage for app image
 FROM base
@@ -80,6 +75,7 @@ COPY --chown=rails:rails --from=build /rails /rails
 # Entrypoint prepares the database.
 ENTRYPOINT ["/rails/bin/docker-entrypoint"]
 
-# Start server via Thruster by default, this can be overwritten at runtime
-EXPOSE 3000
-CMD ["./bin/rails", "server", "-b", "0.0.0.0", "-p", "3000"]
+# Start server via Thruster by default; listen on port from $PORT (default 8080)
+# Use shell CMD so the runtime can run database prep and a best-effort assets precompile
+EXPOSE 8080
+CMD ["./bin/start.sh"]

README-spaces.md

@@ -0,0 +1,20 @@
+Hugging Face Spaces Docker notes
+
+- Required port: Spaces will set `PORT`; the container should listen on `$PORT`. This project defaults to 8080.
+- Provide `RAILS_MASTER_KEY` via the Spaces secrets or environment variables if you use encrypted credentials.
+- For production DB use, set `DATABASE_URL` to a hosted Postgres/MySQL instance; otherwise the image uses SQLite (not persistent across container restarts).
+
+Build & run (local example):
+
+```bash
+docker build -t antique_analyzer .
+# run on port 8080 and pass a master key if needed
+docker run -e RAILS_MASTER_KEY="$RAILS_MASTER_KEY" -e PORT=8080 -p 8080:8080 antique_analyzer
+```
+
+Notes for Spaces:
+- Push this repository to a Space that uses a `Dockerfile`.
+- Add any required env vars (`RAILS_MASTER_KEY`, `DATABASE_URL`) in the Space settings.
+- The container will attempt to run `rails db:prepare` and `rails assets:precompile` on startup; ensure migrations and assets are compatible with that flow.
+
+If you want a faster startup, consider precompiling assets during build in CI and providing them in the image (requires secure handling of `RAILS_MASTER_KEY`).

bin/start.sh

@@ -0,0 +1,15 @@
+#!/bin/bash -e
+
+# Start script for container environments (Hugging Face Spaces compatible)
+#  - prepare or migrate DB
+#  - precompile assets (best-effort)
+#  - start Rails server on $PORT (default 8080)
+
+echo "[container] Running db:prepare..."
+bundle exec rails db:prepare
+
+echo "[container] Precompiling assets (best-effort)..."
+bundle exec rails assets:precompile || true
+
+echo "[container] Starting Rails server on 0.0.0.0:${PORT:-8080}"
+exec bundle exec rails server -b 0.0.0.0 -p "${PORT:-8080}"

config/puma.rb

@@ -28,8 +28,11 @@
 threads_count = ENV.fetch("RAILS_MAX_THREADS", 3)
 threads threads_count, threads_count
 
-# Specifies the `port` that Puma will listen on to receive requests; default is 3000.
-port ENV.fetch("PORT", 3000)
+# Specifies the `port` that Puma will listen on to receive requests; default is 8080 to match container environments.
+port ENV.fetch("PORT", 8080)
+
+# Bind explicitly to all interfaces in container environments.
+bind "tcp://0.0.0.0:#{ENV.fetch('PORT', 8080)}"
 
 # Allow puma to be restarted by `bin/rails restart` command.
 plugin :tmp_restart