Skip to content

[wrangler] Improve Email Routing plan and apply output - #14833

Merged
jamesopstad merged 2 commits into
cloudflare:mainfrom
DiogoSantoss:dsantos/email-routing-plan-colors
Jul 27, 2026
Merged

[wrangler] Improve Email Routing plan and apply output#14833
jamesopstad merged 2 commits into
cloudflare:mainfrom
DiogoSantoss:dsantos/email-routing-plan-colors

Conversation

@DiogoSantoss

@DiogoSantoss DiogoSantoss commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Improve declarative Email Routing deploy output and apply performance.

  • Color plan additions green, updates yellow, and deletions and conflicts red. Chalk continues to emit plain markers when color is disabled.
  • Apply independent zones concurrently with a limit of 10 active zones. Changes within each zone remain serial in server-provided plan order so capacity-sensitive delete-before-add transitions remain safe.
  • Keep failure reporting deterministic in plan order and count every attempted change in progress output.

  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: this changes terminal presentation and apply performance without changing configuration or API behavior.

@changeset-bot

changeset-bot Bot commented Jul 24, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4db8c0d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
wrangler Patch
@cloudflare/deploy-helpers Patch
@cloudflare/vite-plugin Patch
@cloudflare/vitest-pool-workers Patch
@cloudflare/remote-bindings Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk Jul 24, 2026
@workers-devprod
workers-devprod requested review from a team and jamesopstad and removed request for a team July 24, 2026 10:20
@workers-devprod

workers-devprod commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@pkg-pr-new

pkg-pr-new Bot commented Jul 24, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@14833

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@14833

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@14833

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@14833

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@14833

miniflare

npm i https://pkg.pr.new/miniflare@14833

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@14833

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@14833

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@14833

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@14833

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@14833

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@14833

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@14833

wrangler

npm i https://pkg.pr.new/wrangler@14833

commit: 4db8c0d

@workers-devprod workers-devprod left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Codeowners reviews satisfied

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Jul 24, 2026
@DiogoSantoss DiogoSantoss changed the title [wrangler] Color Email Routing plan markers [wrangler] Improve Email Routing plan and apply output Jul 24, 2026
@DiogoSantoss

Copy link
Copy Markdown
Contributor Author

Can we merge ?

@jamesopstad

Copy link
Copy Markdown
Contributor

Can we merge ?

You widened the scope of the PR after my approval so I've dismissed it. I can review again but currently lots of tests are failing. It probably needs a rebase.

@DiogoSantoss
DiogoSantoss force-pushed the dsantos/email-routing-plan-colors branch from 462b54f to 4db8c0d Compare July 24, 2026 14:34
@DiogoSantoss

Copy link
Copy Markdown
Contributor Author

You widened the scope of the PR after my approval so I've dismissed it. I can review again but currently lots of tests are failing. It probably needs a rebase.

Rebased. I think the remaining failures are not related to my changes, is there a chance they are flaky ?

@jamesopstad
jamesopstad merged commit 773ead4 into cloudflare:main Jul 27, 2026
74 of 77 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Jul 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants