Disable warnings for invalid config in wrangler login & logout

[WalshyDev]

What this PR solves / how to test

Fixes WO-184

Author has addressed the following

• Tests
  • TODO (before merge)
  • Tests included
  • Tests not necessary because:
• E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)
  • I don't know
  • Required
  • Not required because: no changes
• Changeset (Changeset guidelines)
  • TODO (before merge)
  • Changeset included
  • Changeset not necessary because:
• Public documentation
  • TODO (before merge)
  • Cloudflare docs PR(s):
  • Documentation not necessary because: no public changes

[changeset-bot]

🦋 Changeset detected

Latest commit: 8e521b3

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

This PR includes changesets to release 2 packages

Name	Type
wrangler	Patch
@cloudflare/vitest-pool-workers	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-actions]

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-wrangler-6756

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/6756/npm-package-wrangler-6756

Or you can use npx with this latest build directly:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-wrangler-6756 dev path/to/script.js

Additional artifacts:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-create-cloudflare-6756 --no-auto-update

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-cloudflare-kv-asset-handler-6756

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-miniflare-6756

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-cloudflare-pages-shared-6756

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-cloudflare-vitest-pool-workers-6756

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-cloudflare-workers-editor-shared-6756

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-cloudflare-workers-shared-6756

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/11576974580/npm-package-cloudflare-workflows-shared-6756

Note that these links will no longer work once the GitHub Actions artifact expires.

---

[email protected] includes the following runtime dependencies:

Package	Constraint	Resolved
miniflare	workspace:*	3.20241022.0
workerd	1.20241022.0	1.20241022.0
workerd --version	1.20241022.0	2024-10-22

Please ensure constraints are pinned, and miniflare/workerd minor versions match.

[penalosa]

I'm not sure I understand the intention here—in this case there is an invalid wrangler.toml file, and I think we do want to warn. It's not a hard error, so this shouldn't be causing user issues

[WalshyDev]

Yeah but warning on an invalid wrangler.toml during login/logout just doesn't make much sense - we read it just for the metrics value but we don't use anything else.

A user seeing a warning after logging in is confusing.

[threepointone]

Actually a good idea, I like it

[penalosa]

For context, I had originally blocked this because I had thought that config being invalid could potentially cause weird behaviour in login, primarily around the account_id field. Having read the PR more closely, since we're only reading config after actually logging in or out that concern is removed. Thanks @WalshyDev for this!