fix(docker): n'installe plus/ne build plus Cypress dans l'image de prod

- README : le dev local cible désormais `docker compose up -d postgres`
  explicitement. `docker-compose.yml` définit aussi le service `app`
  (celui que Portainer build en prod) — sans nom de service,
  `docker compose up -d` construisait et démarrait aussi ce conteneur en
  local, déclenchant un `pnpm install` sur tout le monorepo (donc le
  `cypress` d'apps/web, avec son téléchargement de binaire) pour une image
  dont le dev local n'a pas besoin.
- Dockerfile (apps/api) : `CYPRESS_INSTALL_BINARY=0` sur le stage `build`
  — évite le téléchargement du binaire Cypress (~200 Mo, Electron) à
  chaque build de l'image de prod, qu'il vienne de `docker compose` ou du
  pipeline de release. Le package `cypress` lui-même reste installé (vraie
  entrée du lockfile) ; `pnpm prune --prod` puis un wipe+reinstall complet
  ont été testés pour l'éliminer aussi, mais se sont révélés dangereux
  dans ce monorepo pnpm (prune ignore les node_modules des autres
  packages du workspace ; le wipe+reinstall a fait sauter le binding
  natif compilé d'argon2, crash silencieux au démarrage du conteneur) —
  pas justifié pour ~10 Mo de JS inerte une fois le téléchargement du
  binaire évité.
- apps/api/package.json : déplace `prisma` (le CLI, invoqué directement
  par le `CMD` du conteneur pour `migrate deploy`) de devDependencies
  vers dependencies — classification plus correcte indépendamment du
  point ci-dessus.

Vérifié par un vrai `docker build` + `docker run` contre un Postgres réel
(migrations, seed, démarrage du serveur, signup avec hash argon2 réel).
This commit is contained in:
Nicolas 2026-08-19 09:19:29 +02:00
parent c34eaa89d0
commit ee6fb0dc05
4 changed files with 31 additions and 5 deletions

View file

@ -75,7 +75,7 @@ hors du repo).
```bash ```bash
# Base de données Postgres locale # Base de données Postgres locale
docker compose up -d docker compose up -d postgres
# Applique le schéma (première fois / après un changement de prisma/schema.prisma) # Applique le schéma (première fois / après un changement de prisma/schema.prisma)
pnpm --filter api exec prisma migrate dev pnpm --filter api exec prisma migrate dev
@ -94,6 +94,14 @@ pnpm dev:web
> Dans ce cas, mets `POSTGRES_PORT=5433` (ou autre) dans ton `.env` **et** adapte le > Dans ce cas, mets `POSTGRES_PORT=5433` (ou autre) dans ton `.env` **et** adapte le
> port dans la `DATABASE_URL` de `apps/api/.env`. > port dans la `DATABASE_URL` de `apps/api/.env`.
> **Toujours cibler `postgres`, jamais `docker compose up -d` tout court.** Le même
> `docker-compose.yml` définit aussi le service `app` (voir [Déploiement](#déploiement)) —
> celui que Portainer construit en production. Sans nom de service, `docker compose up -d`
> démarre les deux : ça déclenche un `pnpm install` sur tout le monorepo (donc aussi le
> `cypress` d'`apps/web`, avec son téléchargement de binaire) rien que pour builder une
> image dont le dev local n'a pas besoin (on sert le front/back directement via
> `pnpm dev:web`/`pnpm dev:api`, pas ce conteneur).
## Qualité / Tests ## Qualité / Tests
```bash ```bash

View file

@ -14,6 +14,24 @@ WORKDIR /repo
# process, one container, no separate nginx/static host. Builds both so the # process, one container, no separate nginx/static host. Builds both so the
# runtime stage below can copy each app's build output independently. # runtime stage below can copy each app's build output independently.
FROM base AS build FROM base AS build
# `pnpm install` (below) installs every workspace's dependencies, including
# apps/web's `cypress` devDependency — this image never runs it, so skip its
# (large, Electron-bundled) binary download: saves build time/bandwidth and
# removes a network dependency on Cypress's CDN from every production build.
# The `cypress` npm *package* itself still ends up in `node_modules` (it's a
# real lockfile entry) and gets copied into `runtime` below — a deliberate
# tradeoff. Tried to prune it out too (`pnpm prune --prod`, then a full
# wipe-and-reinstall with `--prod`), but in this pnpm workspace both proved
# actively unsafe rather than just ineffective: `prune` doesn't cascade into
# sibling workspace projects' `node_modules` at all, and even a *scoped*
# `pnpm --filter web prune --prod` emptied out apps/api's entire
# `node_modules` as a side effect; the wipe-and-reinstall variant silently
# dropped argon2's compiled native binding (no postinstall re-run to rebuild
# it), which only surfaced as a crash *after* deploy
# (`Cannot find module '.../argon2.node'`) — not something to risk on the
# real production image for what's ~10MB of inert JS once the binary
# download above is already skipped.
ENV CYPRESS_INSTALL_BINARY=0
COPY . . COPY . .
RUN pnpm install --frozen-lockfile RUN pnpm install --frozen-lockfile
RUN pnpm --filter api build RUN pnpm --filter api build

View file

@ -27,6 +27,7 @@
"dotenv": "^16.4.5", "dotenv": "^16.4.5",
"express": "^4.21.1", "express": "^4.21.1",
"jsonwebtoken": "^9.0.3", "jsonwebtoken": "^9.0.3",
"prisma": "^5.22.0",
"zod": "^3.23.8" "zod": "^3.23.8"
}, },
"devDependencies": { "devDependencies": {
@ -39,7 +40,6 @@
"chai": "^5.1.2", "chai": "^5.1.2",
"cross-env": "^10.1.0", "cross-env": "^10.1.0",
"mocha": "^10.8.2", "mocha": "^10.8.2",
"prisma": "^5.22.0",
"supertest": "^7.0.0", "supertest": "^7.0.0",
"tsx": "^4.19.2", "tsx": "^4.19.2",
"typescript": "^5.7.2" "typescript": "^5.7.2"

View file

@ -44,6 +44,9 @@ importers:
jsonwebtoken: jsonwebtoken:
specifier: ^9.0.3 specifier: ^9.0.3
version: 9.0.3 version: 9.0.3
prisma:
specifier: ^5.22.0
version: 5.22.0
zod: zod:
specifier: ^3.23.8 specifier: ^3.23.8
version: 3.25.76 version: 3.25.76
@ -75,9 +78,6 @@ importers:
mocha: mocha:
specifier: ^10.8.2 specifier: ^10.8.2
version: 10.8.2 version: 10.8.2
prisma:
specifier: ^5.22.0
version: 5.22.0
supertest: supertest:
specifier: ^7.0.0 specifier: ^7.0.0
version: 7.2.2 version: 7.2.2