chore: upgrade API Platform to 4.3.5#638
Merged
Merged
Conversation
vincentchalamon
added
the
dependencies
Latest pnpm versions fail with "global bin directory is not in PATH" when running `pnpm config -g set store-dir` during the image build. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Setting PNPM_HOME to the default `/root/.local/share/pnpm` left pnpm still looking for `$PNPM_HOME/bin` in PATH. Use a dedicated directory as recommended by pnpm Docker docs. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
pnpm computes its global bin directory as `$PNPM_HOME/bin`, not `$PNPM_HOME` itself. Add the `/bin` suffix to PATH so the check passes. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
pnpm 10.x aborts node_modules purge in absence of a TTY unless CI is declared. Without it, `pnpm install` fails with ERR_PNPM_ABORTED_REMOVE_MODULES_DIR_NO_TTY in Docker builds. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
`pnpm fetch --prod` misses some packages (e.g. @api-platform/admin) that are required by `pnpm install --offline --prod`, causing ERR_PNPM_NO_OFFLINE_TARBALL during the build. Fetching everything ensures the offline install has all tarballs available. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Without package.json present, corepack runs `pnpm fetch` with the @latest pnpm from the base stage, while `pnpm install` later uses the version pinned in package.json (10.33.0). The two versions can have incompatible store layouts, leading to ERR_PNPM_NO_OFFLINE_TARBALL. Copy package.json alongside the lockfile so corepack uses the same pnpm version for both steps. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Next.js 16 build fails when TypeScript is not installed, but the project keeps typescript and @types/* in devDependencies. The final prod image is built from the Next.js standalone output, so installing devDeps in the builder stage does not bloat the runtime image. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See https://github.com/api-platform/core/releases/tag/v4.3.5.