Skip to content

ci: skip dependency lifecycle scripts during MCP sync#4839

Open
timelis wants to merge 1 commit into
Dokploy:canaryfrom
timelis:codex/ci-mcp-sync-ignore-scripts
Open

ci: skip dependency lifecycle scripts during MCP sync#4839
timelis wants to merge 1 commit into
Dokploy:canaryfrom
timelis:codex/ci-mcp-sync-ignore-scripts

Conversation

@timelis

@timelis timelis commented Jul 16, 2026

Copy link
Copy Markdown

Summary

  • Install MCP dependencies with pnpm install --ignore-scripts in the sync-version job.
  • Unblock Dokploy-to-MCP version and generated tool synchronization without pinning pnpm to an older release.
  • Keep the core change focused to one CI command.

Root cause

The release workflow installs the latest pnpm and then runs pnpm install in a fresh Dokploy/mcp clone. With pnpm v11, the current MCP configuration causes installation to exit with ERR_PNPM_IGNORED_BUILDS for esbuild@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-openapi and generate scripts. 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 onlyBuiltDependencies configuration to pnpm v11's allowBuilds, 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-scripts is 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/mcp main checkout in a Linux container using the Node version recommended by CONTRIBUTING.md:

  • Node 24.4.0
  • pnpm 11.13.1
  • pnpm install --ignore-scripts: passed
  • pnpm run fetch-openapi: fetched 541 endpoints
  • pnpm run generate: generated 541 tools with 0 errors
  • git diff --check: passed

Closes #4837

Related: Dokploy/mcp#62, Dokploy/mcp#59, Dokploy/mcp#60

Supersedes #4838, which was closed and recreated from canary according to CONTRIBUTING.md.

@timelis
timelis marked this pull request as ready for review July 16, 2026 16:15
@timelis
timelis requested a review from Siumauricio as a code owner July 16, 2026 16:15
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 16, 2026

timelis commented Jul 16, 2026

Copy link
Copy Markdown
Author

The options available within this repository are either to downgrade and pin the pnpm version, as Dosu mentioned in #4837, or to run pnpm install --ignore-scripts.

However, I personally believe that merging the pull request opened in Dokploy/mcp is the best solution:

Dokploy/mcp#62

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sync-version job fails while installing MCP dependencies with pnpm 11 Docker compose build fails: lockfile excluded and pnpm latest breaks install

1 participant