[wrangler] Prioritize CLOUDFLARE_ACCOUNT_ID over cached account id for Pages commands

[emily-shen]

Fixes #13590.

Most pages commands previously read the account id from their cached pages.json before checking the CLOUDFLARE_ACCOUNT_ID environment variable.

This is inconsistent with other commands, including the pages project list/create/delete commands which already do prioritise CLOUDFLARE_ACCOUNT_ID.

This PR applies the same pattern to the remaining Pages commands:

• wrangler pages deploy

• wrangler pages deployment list / deployment delete

• wrangler pages deployment tail

• wrangler pages download config

• wrangler pages secret (put/bulk/delete/list)

• 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: bugfix

Devin PR requested by @emily-shen (eshen@cloudflare.com)

Link to Devin session: https://app.devin.ai/sessions/c2f96415cbf44d12a22dec27fad422b4

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR that start with 'DevinAI' or '@devin'.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring

[changeset-bot]

🦋 Changeset detected

Latest commit: 875f5a2

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

[ask-bonk]

UnknownError: ProviderInitError

github run

[ask-bonk]

@emily-shen Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

[pkg-pr-new]

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: 875f5a2

[github-actions]

Changeset Review

Reviewed Changesets

.changeset/pages-deploy-env-account-id.md

Package: wrangler
Version Type: patch ✅

Validation Results:

Criteria	Status	Notes
Version Type	✅	patch is appropriate for this bug fix
Changelog Quality	✅	Clear description of the fix, affected commands listed, previous behavior explained
Markdown Headers	✅	No h1/h2/h3 headers found
Analytics	✅	N/A - no analytics collection
Dependabot	✅	N/A - not a dependency update
Experimental Features	✅	N/A - bug fix for existing functionality

Summary:
This changeset correctly describes a bug fix where the CLOUDFLARE_ACCOUNT_ID environment variable was not being prioritized over cached account IDs in certain Pages commands. The description clearly lists the affected commands and notes that pages project commands were already working correctly. Good level of detail for users to understand what was fixed.

---

Overall Result

✅ All changesets look good

[workers-devprod]

Codeowners approval required for this PR:

• ✅ @cloudflare/wrangler

Show detailed file reviewers

[devin-ai-integration]

✅ Devin Review: No Issues Found

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

View in Devin Review to see 2 additional findings.

[workers-devprod]

Codeowners reviews satisfied