ci: skip dependency lifecycle scripts during MCP sync#4839
Open
timelis wants to merge 1 commit into
Open
Conversation
timelis
marked this pull request as ready for review
July 16, 2026 16:15
Author
|
The options available within this repository are either to downgrade and pin the pnpm version, as Dosu mentioned in #4837, or to run However, I personally believe that merging the pull request opened in Dokploy/mcp is the best solution: |
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
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.
Summary
pnpm install --ignore-scriptsin thesync-versionjob.Root cause
The release workflow installs the latest pnpm and then runs
pnpm installin a freshDokploy/mcpclone. With pnpm v11, the current MCP configuration causes installation to exit withERR_PNPM_IGNORED_BUILDSforesbuild@0.27.7. The workflow stops before updating the MCP package version or generated OpenAPI tools.The sync step only needs the explicitly invoked
fetch-openapiandgeneratescripts. Dependency lifecycle scripts are not required for those commands, so this repository-local change disables them during installation.Related MCP fix
The preferred project-level fix is Dokploy/mcp#62. It migrates MCP from the deprecated
onlyBuiltDependenciesconfiguration to pnpm v11'sallowBuilds, includes the required configuration in the Docker build, and fixes Dokploy/mcp#59 without pinning pnpm.Merging Dokploy/mcp#62 is the better long-term fix for MCP's normal install and Docker workflows. This PR remains intentionally limited to
Dokploy/dokploy, where--ignore-scriptsis the self-contained way to restore the release sync and avoid executing unnecessary dependency lifecycle scripts in a job that pushes to another repository.Validation
Tested from a clean, unmodified
Dokploy/mcpmain checkout in a Linux container using the Node version recommended byCONTRIBUTING.md:24.4.011.13.1pnpm install --ignore-scripts: passedpnpm run fetch-openapi: fetched 541 endpointspnpm run generate: generated 541 tools with 0 errorsgit diff --check: passedCloses #4837
Related: Dokploy/mcp#62, Dokploy/mcp#59, Dokploy/mcp#60
Supersedes #4838, which was closed and recreated from
canaryaccording toCONTRIBUTING.md.