fix(build): drop COPY of deleted views/public from Dockerfiles #65

Merged

David merged 1 commit from fix/dockerfile-drop-deleted-spa-copies into main

2026-06-21 03:44:27 +02:00

David commented

2026-06-21 03:38:59 +02:00

Owner

Summary

Fixes the OCI image build, which broke after the rebrand (#64) removed the vendored legacy SPA (public/ and views/). oci-build/Dockerfile still copied those directories, so the build failed:

COPY views ./views   -> "/views": not found
COPY public ./public -> "/public": not found

Changes

oci-build/Dockerfile: remove the dead COPY views ./views and COPY public ./public lines. The release binary no longer needs those trees - templates serve a built-in default and embedded_assets is disk-only after #64.
Dockerfile (dev): drop the now-stale --ignore views/** / --ignore public/** flags from the cargo-watch command.
compose.dev.yml: drop the ./views and ./public bind mounts so a dev run does not recreate the deleted directories.

Verification

cargo fmt, clippy, and the full test suite pass (pre-commit hook, CI parity). A local docker buildx build -f oci-build/Dockerfile gets past the previously-failing COPY steps and builds the release binary.

🤖 Generated with Claude Code

## Summary Fixes the OCI image build, which broke after the rebrand (#64) removed the vendored legacy SPA (`public/` and `views/`). `oci-build/Dockerfile` still copied those directories, so the build failed: ``` COPY views ./views -> "/views": not found COPY public ./public -> "/public": not found ``` ## Changes - `oci-build/Dockerfile`: remove the dead `COPY views ./views` and `COPY public ./public` lines. The release binary no longer needs those trees - templates serve a built-in default and `embedded_assets` is disk-only after #64. - `Dockerfile` (dev): drop the now-stale `--ignore views/**` / `--ignore public/**` flags from the cargo-watch command. - `compose.dev.yml`: drop the `./views` and `./public` bind mounts so a dev run does not recreate the deleted directories. ## Verification `cargo fmt`, `clippy`, and the full test suite pass (pre-commit hook, CI parity). A local `docker buildx build -f oci-build/Dockerfile` gets past the previously-failing COPY steps and builds the release binary. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

David added 1 commit

2026-06-21 03:38:59 +02:00

fix(build): drop COPY of deleted views/public from Dockerfiles

Check / fmt + clippy + build + tests (pull_request) Successful in 59s

Details

Create release / Create release from merged PR (pull_request) Has been skipped

Details

6382b9a887

The rebrand (#64) removed the vendored legacy SPA (public/ and views/) and excised its compile-time embed, but the production image Dockerfile still copied those directories, so the OCI build failed at `COPY views ./views` / `COPY public ./public` with "not found".

Remove the dead COPY lines from oci-build/Dockerfile. The release binary no longer needs those trees: templates serve a built-in default and embedded_assets is disk-only. Also drop the now-stale `--ignore views/** --ignore public/**` flags from the dev Dockerfile's cargo-watch command and the `./views` / `./public` bind mounts from compose.dev.yml so a dev run does not recreate the deleted dirs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>