Fetch script metadata directly instead of listing all scripts - #15132

🦋 Changeset detected

Latest commit: dc757b4

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

All changesets look good

So mockSubDomainRequest("test-sub-domain", true, false) just sets once=false (allowing the subdomain handler to respond multiple times). This is a test-mock adjustment in the "no previous script" test cases, likely because the deploy flow now retries/re-requests. This is a test-only concern and not a logic bug.

My analysis is complete. The change is well-implemented:

1. Consolidates to a single endpoint — the CI-tag flow (match-tag.ts) already fetches /workers/services/{name} in the same deploy, so this reuses an established path and reduces payload.
2. Error handling is consistent — isWorkerNotFoundError + codes 10090/10007 match the established pattern used in match-tag.ts and other tests.
3. Type usage is correct — ServiceMetadataRes.default_environment.script is always present per the type.
4. Mock changes are appropriate — dropping { once: true } correctly allows the shared endpoint to answer both the pre-upload check and the migration flow.

I found no logic bugs, security issues, backward-compatibility violations, or incorrect API behavior. The behavior for the "script not found" case (apply all migrations) is preserved: previously .find() returned undefined; now the not-found error is suppressed leaving script undefined — same downstream result.

LGTM

github run

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

commit: dc757b4

Codeowners approval required for this PR:

• @cloudflare/cloudchamber
• ✅ @cloudflare/wrangler