Skip to content

[wrangler] Mention temporary preview accounts in whoami when unauthenticated#14328

Open
edevil wants to merge 1 commit into
mainfrom
whoami-temporary-account-hint
Open

[wrangler] Mention temporary preview accounts in whoami when unauthenticated#14328
edevil wants to merge 1 commit into
mainfrom
whoami-temporary-account-hint

Conversation

@edevil

@edevil edevil commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

When wrangler whoami is run without credentials, it previously printed only:

You are not authenticated. Please run wrangler login.

This didn't surface the temporary-preview-account feature (the hidden --temporary flag). Since models and users frequently run wrangler whoami to check auth status, this PR adds a second line nudging them toward deploying without logging in:

To deploy without logging in, run a command like wrangler deploy --temporary to use a temporary preview account.

The wording matches the existing nudge already shown on non-interactive auth errors in register-yargs-command.ts. The original first line is left intact so existing references/tests don't break. A test asserting both lines has been added (there was previously no coverage of the non-JSON unauthenticated output).


  • 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: this only adds an informational hint to existing CLI output for the already-documented --temporary flag; no public API or behavior changes.

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


Open in Devin Review

…ticated

When `wrangler whoami` runs without credentials, it now also tells the
user they can deploy without logging in via a temporary preview account
(`wrangler deploy --temporary`), matching the existing nudge shown on
non-interactive auth errors.
@edevil edevil requested a review from workers-devprod as a code owner June 16, 2026 17:20
@changeset-bot

changeset-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c417773

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

This PR includes changesets to release 3 packages
Name Type
wrangler Patch
@cloudflare/vite-plugin 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-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk Jun 16, 2026
@workers-devprod workers-devprod requested review from a team and jamesopstad and removed request for a team June 16, 2026 17:20
@workers-devprod

workers-devprod commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@ask-bonk

ask-bonk Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

The change is minimal, correct, and well-tested:

  • The new hint references wrangler deploy --temporary, and deploy does support --temporary — accurate.
  • It's informational output only, no behavior/API change.
  • The original first line is preserved, so existing tests don't break.
  • The new test correctly uses test-context expect, follows the repo's testing conventions, and asserts both lines.
  • The changeset is present, correctly scoped as a wrangler patch, uses no banned headers.

No logic bugs, security concerns, or backward-compat issues.

LGTM

github run

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

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

Open in Devin Review

@github-actions

Copy link
Copy Markdown
Contributor

✅ All changesets look good

@workers-devprod workers-devprod left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codeowners reviews satisfied

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Approved

Development

Successfully merging this pull request may close these issues.

3 participants