Skip to content

WC-5936 Honor Workers Builds worker name overrides in wrangler preview - #15499

Merged
WillTaylorDev merged 1 commit into
mainfrom
willtaylor/wc-5936-wrangler-preview-ci-override-name
Sep 4, 2026
Merged

WC-5936 Honor Workers Builds worker name overrides in wrangler preview#15499
WillTaylorDev merged 1 commit into
mainfrom
willtaylor/wc-5936-wrangler-preview-ci-override-name

Conversation

@WillTaylorDev

@WillTaylorDev WillTaylorDev commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Fixes WC-5936

Workers Builds sets an override so a Worker keeps its dashboard name even when a project's Wrangler config uses a different one. wrangler deploy already honored that override, but wrangler preview and its secrets, settings, and delete subcommands read the configured name directly and failed with a name mismatch error.

Preview commands now check the override first, matching how deploy resolves the worker name.


  • 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 restores existing deploy behavior for preview and doesn't add anything new for users.

A picture of a cute animal (not mandatory, but encouraged)
image


Devin Review

Workers Builds sets an override so a Worker keeps its dashboard name
even when a project's Wrangler config uses a different one. wrangler
deploy already honored that override, but wrangler preview and its
secrets, settings, and delete subcommands read the configured name
directly and failed with a name mismatch error.

Preview commands now check the override first, matching how deploy
resolves the worker name.
@changeset-bot

changeset-bot Bot commented Sep 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: cf3624e

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

This PR includes changesets to release 3 packages
Name Type
wrangler Patch
@cloudflare/vite-plugin Patch
@cloudflare/vitest-plugin 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 Sep 3, 2026
@workers-devprod
workers-devprod requested review from a team and edmundhung and removed request for a team September 3, 2026 18:07
@workers-devprod

workers-devprod commented Sep 3, 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.

Devin Review

@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

@pkg-pr-new

pkg-pr-new Bot commented Sep 3, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

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

@cloudflare/build-output-utils

npm i https://pkg.pr.new/@cloudflare/build-output-utils@15499

@cloudflare/codemods

npm i https://pkg.pr.new/@cloudflare/codemods@15499

@cloudflare/config

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

create-cloudflare

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

@cloudflare/deploy-helpers

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-functions

npm i https://pkg.pr.new/@cloudflare/pages-functions@15499

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-plugin

npm i https://pkg.pr.new/@cloudflare/vitest-plugin@15499

@cloudflare/workers-auth

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: cf3624e

@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 Sep 4, 2026
@WillTaylorDev
WillTaylorDev merged commit ffc7efd into main Sep 4, 2026
71 of 75 checks passed
@WillTaylorDev
WillTaylorDev deleted the willtaylor/wc-5936-wrangler-preview-ci-override-name branch September 4, 2026 15:12
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Sep 4, 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.

4 participants