support using containers with ctx.exports

[emily-shen]

Fixes CC-6386.

The containers control plane needs a DO namespace id (sort of*) when creating a container app in order to link the container app and DO. This is helpfully returned by EWC when the script has a binding to a DO, but not when the DO is simply exported from the script and unbound. For now, we have to list and filter all DO namespace ids, which is not ideal, but there is a PR in flight that should enable search params on the DO list endpoint.

*The container create API can actually accept a script and class name instead, and this is what (future) direct API users would be using.

However, wrangler is in an awkward place because of how we automatically provision containers. Wrangler deploy needs to know whether a particular container + durable object combination already exists, and thus whether we should create a new container app, modify an existing one, or error because you've accidentally tried to reuse the name of an existing container app.

So wrangler will have to get the container's DO's namespace id to make sure it hasn't changed, and at that point we might as well just pass that to the container create API.

(possibly this is something that should be solved EWC side but i think we could follow up with that if we want?)

---

• Tests
  • Tests included/updated
  • Tests not necessary because:
• Public documentation
  • Cloudflare docs PR(s):
  • Documentation not necessary because: current docs are still correct with this change, need to work out with product how we want to use this.
• Wrangler V3 Backport
  • Wrangler PR:
  • Not necessary because: containers

[changeset-bot]

🦋 Changeset detected

Latest commit: 1743847

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

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

[pkg-pr-new]

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: 1743847

[emily-shen]

All of this code is in the cloudchamber folder but isn't actually used by any cloudchamber paths, so i've moved these functions into the containers folder. (under build.ts and deploy.ts)

[emily-shen]

not strictly related to this PR but dry run will have already exited by the point this is called so this was unnecessary

[petebacondarwin]

LGTM with a few questions