Skip to content

fix: corriger l'échec du build docker lié aux scripts de build ignorés par pnpm - #108

Merged
KepoParis merged 1 commit into
mainfrom
fix/pnpm-ignored-builds
Sep 4, 2026
Merged

fix: corriger l'échec du build docker lié aux scripts de build ignorés par pnpm#108
KepoParis merged 1 commit into
mainfrom
fix/pnpm-ignored-builds

Conversation

@KepoParis

Copy link
Copy Markdown
Contributor

Issues liées

Aucune.


Quel est le comportement actuel ?

Le build de l'image docker échoue à l'étape pnpm install :

[ERR_PNPM_IGNORED_BUILDS] Ignored build scripts: [email protected]
Run "pnpm approve-builds" to pick which dependencies should be allowed to run scripts.

Le Dockerfile installe pnpm sans version fixée (npm install --location=global pnpm),
chaque build récupère donc la dernière version publiée. pnpm 10 bloque par défaut les
scripts de cycle de vie des dépendances et, contrairement à pnpm 9, sort en erreur au
lieu d'émettre un avertissement. Le script postinstall d'esbuild, qui télécharge son
binaire natif, est concerné. Le code et le lockfile n'ont pas changé, seule la version
de pnpm a bougé.

Quel est le nouveau comportement ?

  • package.json : ajout de pnpm.onlyBuiltDependencies: ["esbuild"] pour autoriser
    explicitement le script postinstall d'esbuild. Sans lui, le binaire natif est absent
    et pnpm run build échouerait à l'étape suivante.
  • Dockerfile : pnpm est épinglé sur pnpm@10 pour que la version ne dérive plus
    d'un build à l'autre.

Vérifié en local : docker build --target build passe, pnpm install puis
vitepress build docs se terminent sans erreur.

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

Aucune.

@KepoParis
KepoParis requested a review from iliesmrf September 4, 2026 15:24
@KepoParis
KepoParis merged commit 6d649bb into main Sep 4, 2026
1 check passed
@KepoParis
KepoParis deleted the fix/pnpm-ignored-builds branch September 4, 2026 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants