[deploy-helpers] Containers support for Worker Preview deployments in the Build Output

[GregBrimble]

Fixes WC-5979.

Adds Containers support for Preview deployments in the Build Config.

---

• 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: helper package change

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

---

[changeset-bot]

🦋 Changeset detected

Latest commit: d686cad

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

This PR includes changesets to release 10 packages

Name	Type
@cloudflare/workers-utils	Minor
@cloudflare/deploy-helpers	Minor
wrangler	Minor
@cloudflare/autoconfig	Patch
@cloudflare/cli-shared-helpers	Patch
@cloudflare/containers-shared	Patch
@cloudflare/remote-bindings	Patch
@cloudflare/vite-plugin	Patch
@cloudflare/vitest-plugin	Patch
@cloudflare/workers-auth	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

[workers-devprod]

Codeowners approval required for this PR:

• @cloudflare/cloudchamber
• ✅ @cloudflare/wrangler

Show detailed file reviewers

• packages/containers-shared/src/build.ts: [@cloudflare/cloudchamber]
• packages/containers-shared/tests/build-and-push.test.ts: [@cloudflare/cloudchamber]

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Adds callback-driven Container deployment support to Build Output previews.

1. High: Posted 1 inline suggestion.

github run

[pkg-pr-new]

@cloudflare/autoconfig

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

@cloudflare/build-output-utils

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

@cloudflare/codemods

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

@cloudflare/config

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

@cloudflare/containers-shared

npm i https://pkg.pr.new/@cloudflare/containers-shared@15822

create-cloudflare

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

@cloudflare/deploy-helpers

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-functions

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

@cloudflare/pages-shared

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

@cloudflare/runtime-types

npm i https://pkg.pr.new/@cloudflare/runtime-types@15822

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-plugin

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

@cloudflare/workers-auth

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: d686cad

[petebacondarwin]

There appears to be a Bonk review worth looking into. But otherwise LGTM.

[GregBrimble]

Bonk's review is stale. I added validation here:

workers-sdk/packages/deploy-helpers/src/preview/preview.ts

Lines 1360 to 1371 in 1503ca3

	if (
	previewContainers.length > 0 &&
	(callbacks?.getNormalizedContainerOptions === undefined ||
	callbacks.deployPreviewContainers === undefined)
	) {
	throw new UserError(
	"Preview uploads from Build Output with Containers require container deployment callbacks.",
	{
	telemetryMessage: "preview build output container callbacks missing",
	}
	);
	}

[petebacondarwin]

Codeowners bypass

[workers-devprod]

Codeowners reviews satisfied

[devin-ai-integration]

Devin Review found 2 new potential issues.