fix: remove map frontend check and optimize docker image

Author: lkstrp

Dockerfile

@@ -1,3 +1,5 @@
+# syntax=docker/dockerfile:1
+
 # Stage 1: Builder stage
 FROM python:3.13-slim AS builder
 
@@ -35,19 +37,18 @@ RUN apt-get update && apt-get install -y \
     libnetcdf22 \
     && rm -rf /var/lib/apt/lists/*
 
-# Copy from builder stage
-COPY --from=builder /root/.local/bin/uv /usr/local/bin/uv
-COPY --from=builder /app/.venv /app/.venv
-COPY --from=builder /app/src /app/src
-
-COPY pyproject.toml uv.lock MANIFEST.in ./
-
 # Create non-root user for running the application
 RUN groupadd -r appuser -g 1000 && \
-    useradd -r -u 1000 -g appuser -m -s /bin/bash appuser
+    useradd -r -u 1000 -g appuser -m -s /bin/bash appuser && \
+    mkdir -p /data/networks && \
+    chown -R appuser:appuser /data
+
+# Copy from builder stage with correct ownership
+COPY --from=builder --chown=appuser:appuser /root/.local/bin/uv /usr/local/bin/uv
+COPY --from=builder --chown=appuser:appuser /app/.venv /app/.venv
+COPY --from=builder --chown=appuser:appuser /app/src /app/src
 
-RUN mkdir -p /data/networks && \
-    chown -R appuser:appuser /app /data
+COPY --chown=appuser:appuser pyproject.toml uv.lock MANIFEST.in ./
 
 USER appuser
 
@@ -77,7 +78,7 @@ RUN npm ci && \
 # Stage 4: Full stack (adds built frontend to backend base)
 FROM backend AS full
 
-COPY --from=app-builder /frontend/build/ src/pypsa_app/backend/static/app/
+COPY --from=app-builder --chown=appuser:appuser /frontend/build/ src/pypsa_app/backend/static/app/
 
 # Copy package.json for version detection
-COPY --from=app-builder /frontend/package.json frontend/app/package.json
+COPY --from=app-builder --chown=appuser:appuser /frontend/package.json frontend/app/package.json

src/pypsa_app/cli.py

@@ -60,29 +60,15 @@ def serve(
         click.echo(f"   Application: http://{host}:{port}")
         click.echo(f"   API docs: http://{host}:{port}/api/docs")
 
-    # Check if frontends are built (prod mode)
+    # Check if frontend is built (prod mode)
     if not dev:
         static_dir = Path(__file__).parent / "backend" / "static"
         app_dir = static_dir / "app"
-        map_dir = static_dir / "map"
 
-        missing = []
         if not app_dir.exists() or not (app_dir / "index.html").exists():
-            missing.append(
-                ("App frontend", "cd frontend/app && npm ci && npm run build")
-            )
-        if not map_dir.exists() or not (map_dir / "index.html").exists():
-            missing.append(
-                ("Map frontend", "cd frontend/map && npm ci && npm run build")
-            )
-
-        if missing:
-            click.echo(
-                f"Warning: {', '.join(m[0] for m in missing)} not built!", err=True
-            )
+            click.echo("Error: App frontend not built!", err=True)
             click.echo("   Build with:", err=True)
-            for _, cmd in missing:
-                click.echo(f"     {cmd}", err=True)
+            click.echo("     cd frontend/app && npm ci && npm run build", err=True)
             click.echo("   Or use --dev flag for API-only mode", err=True)
             sys.exit(1)
 
@@ -106,30 +92,16 @@ def info() -> None:
     click.echo(f"Python: {sys.version.split()[0]}")
     click.echo(f"Python executable: {sys.executable}")
 
-    # Check frontends
+    # Check frontend
     static_dir = Path(__file__).parent / "backend" / "static"
-    frontends = [
-        ("App", static_dir / "app"),
-        ("Map", static_dir / "map"),
-    ]
+    app_dir = static_dir / "app"
+    app_built = app_dir.exists() and (app_dir / "index.html").exists()
 
-    built = [
-        (name, d.exists() and (d / "index.html").exists()) for name, d in frontends
-    ]
-
-    if all(status for _, status in built):
-        click.echo("\nFrontends: Both built and ready")
+    if app_built:
+        click.echo("\nFrontend: Built and ready")
     else:
-        click.echo("\nFrontends:")
-        for (name, _), is_built in zip(frontends, built, strict=True):
-            if is_built:
-                click.echo(f"   {name}: Built")
-            else:
-                click.echo(
-                    f"   {name}: Not built"
-                    f" (run: cd frontend/{name.lower()}"
-                    " && npm ci && npm run build)"
-                )
+        click.echo("\nFrontend: Not built")
+        click.echo("   Run: cd frontend/app && npm ci && npm run build")
 
     click.echo("\nEnvironment variables:")
     for key in [