Skip to content

fix(ci): use installed web Wrangler - #26

Merged
caio-pizzol merged 1 commit into
mainfrom
caio/fix-web-deploy-wrangler
Aug 13, 2026
Merged

fix(ci): use installed web Wrangler#26
caio-pizzol merged 1 commit into
mainfrom
caio/fix-web-deploy-wrangler

Conversation

@caio-pizzol

@caio-pizzol caio-pizzol commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

What changed

Run the Cloudflare Pages action from apps/web, where Wrangler is already installed, and deploy the local dist directory.

Why

The action ran from the repository root, could not find the workspace Wrangler binary, and tried to install Wrangler during deployment. That temporary Bun install failed while extracting the package.

Verified

  • bun run check
  • bun run wrangler --version from apps/web returns 4.61.0
  • npx --no-install wrangler --version from apps/web returns 4.61.0
  • Full deployment run 31689220019 passed, including database migrations, MCP deployment, the production web build, and Cloudflare Pages deployment

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review
Tip of the day
💡 Did you know, you can type 'qodo, fix this' on a finding and the fix lands right on your PR

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗


Powered by Qodo

@caio-pizzol
caio-pizzol enabled auto-merge (squash) August 13, 2026 10:04

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Re-trigger cubic

@caio-pizzol
caio-pizzol merged commit e704b4c into main Aug 13, 2026
5 checks passed
@caio-pizzol
caio-pizzol deleted the caio/fix-web-deploy-wrangler branch August 13, 2026 10:05
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