[P0] Add .dockerignore for frontend to exclude node_modules, .git, dist #92

New issue

Closed

opened 2026-07-07 16:21:51 +12:00 by fastie81 · 2 comments

fastie81 commented

2026-07-07 16:21:51 +12:00

Owner

Problem

Dockerfile copies entire frontend/ directory (line 6: COPY frontend/ ./) including:

node_modules/ (~100MB+)
.git/
dist/ (built artifacts)
IDE files

This dramatically slows down Docker builds.

Expected Behavior

Add .dockerignore in frontend/ or root with:

node_modules
.git
dist
.env
*.log
.DS_Store

Dockerfile (lines 4-7)
Need to create: frontend/.dockerignore or root .dockerignore

### Problem Dockerfile copies entire `frontend/` directory (line 6: `COPY frontend/ ./`) including: - `node_modules/` (~100MB+) - `.git/` - `dist/` (built artifacts) - IDE files This dramatically slows down Docker builds. ### Expected Behavior Add `.dockerignore` in `frontend/` or root with: ``` node_modules .git dist .env *.log .DS_Store ``` ### Related Files - `Dockerfile` (lines 4-7) - Need to create: `frontend/.dockerignore` or root `.dockerignore`

fastie81 added the

labels

2026-07-07 16:21:51 +12:00

fastie81 referenced this issue from a commit

2026-09-14 11:24:07 +12:00

fix(security): remove hardcoded secrets, add coverage gate, tighten dockerignore

fastie81 referenced this issue

2026-09-14 11:24:10 +12:00

fix(security): remove hardcoded secrets, add coverage gate, tighten dockerignore #142

fastie81 closed this issue

2026-10-21 18:43:56 +13:00

fastie81 commented

2026-10-21 18:44:05 +13:00

Author

Owner

Merged #142: .dockerignore now excludes **/node_modules, **/.vite, **/dist, **/.env*, .DS_Store. Dockerfile only COPYs app/, main.py, entrypoint.sh, scripts/, requirements.txt and builds the frontend in its own stage, so nothing it needs is excluded. Safe to close once main CI is green.

Merged #142: `.dockerignore` now excludes `**/node_modules`, `**/.vite`, `**/dist`, `**/.env*`, `.DS_Store`. Dockerfile only COPYs app/, main.py, entrypoint.sh, scripts/, requirements.txt and builds the frontend in its own stage, so nothing it needs is excluded. Safe to close once main CI is green.

fastie81 commented

2026-10-21 19:11:44 +13:00

Author

Owner

Live in production as of v3.4.10 (image ci-168f17c). Closing.

Live in production as of v3.4.10 (image `ci-168f17c`). Closing.