Skip to content

Fix Deploy: Production wrangler install failure#720

Merged
NullVoxPopuli merged 1 commit into
universal-ember:mainfrom
NullVoxPopuli-ai-agent:fix-deploy-production-wrangler
Apr 6, 2026
Merged

Fix Deploy: Production wrangler install failure#720
NullVoxPopuli merged 1 commit into
universal-ember:mainfrom
NullVoxPopuli-ai-agent:fix-deploy-production-wrangler

Conversation

@NullVoxPopuli-ai-agent
Copy link
Copy Markdown

Summary

  • Split DeployProduction into separate BuildProduction and DeployProduction jobs
  • Build job checks out code, builds docs-app, and uploads the dist as an artifact
  • Deploy job only downloads the artifact and runs wrangler — no repo checkout means no pnpm-lock.yaml, so wrangler-action falls back to npm (matching the existing deploy-preview pattern)

Problem

cloudflare/wrangler-action@v3 detects pnpm from the lockfile and runs pnpm add wrangler, which fails in a workspace with ERR_PNPM_ADDING_TO_ROOT (requires -w flag the action doesn't pass).

Test plan

  • CI passes on this PR
  • Merge to main triggers a successful production deploy

🤖 Generated with Claude Code

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 6, 2026

Project Preview URL
Docs https://a67ee880.ember-primitives.pages.dev

Logs

The wrangler-action detects pnpm from the lockfile and runs
`pnpm add wrangler` which fails in a workspace without `-w`.
By separating the build (with repo checkout) from the deploy
(artifact-only, no checkout), the deploy job has no lockfile
present so wrangler-action falls back to npm — matching the
existing deploy-preview pattern.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@NullVoxPopuli-ai-agent NullVoxPopuli-ai-agent force-pushed the fix-deploy-production-wrangler branch from fb59f7d to 6350326 Compare April 6, 2026 03:23
@NullVoxPopuli NullVoxPopuli merged commit a5e18fe into universal-ember:main Apr 6, 2026
19 checks passed
@github-actions github-actions Bot mentioned this pull request Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants