Skip to content

fix: restrict account name redaction to CI environments only#12671

Merged
emily-shen merged 1 commit intomainfrom
mattietk/fix/redacted-accounts-in-noninteractive
Feb 25, 2026
Merged

fix: restrict account name redaction to CI environments only#12671
emily-shen merged 1 commit intomainfrom
mattietk/fix/redacted-accounts-in-noninteractive

Conversation

@MattieTK
Copy link
Copy Markdown
Member

@MattieTK MattieTK commented Feb 25, 2026 •
edited by devin-ai-integration bot
Loading

Describe your change...

Fixes internal discussion around redaction only in CI environments, and non-interactive environments (eg agents) having this redaction removed.

  • 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: fixing a regression to local experience
Open with Devin

@MattieTK MattieTK requested review from a team as code owners February 25, 2026 06:28
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 25, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: b24fb1d

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

@github-project-automation github-project-automation bot moved this to Untriaged in workers-sdk Feb 25, 2026
@claude
Copy link
Copy Markdown
Contributor

claude bot commented Feb 25, 2026 •
edited
Loading

Claude encountered an error —— View job

Changeset Review

Reviewing changeset files...

Todo List:

  • Read changeset guidelines
  • Review changeset file
  • Validate against criteria
  • Post findings

devin-ai-integration[bot]
devin-ai-integration bot reviewed Feb 25, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

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

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 potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Feb 25, 2026 •
edited
Loading

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: b24fb1d

@MattieTK MattieTK force-pushed the mattietk/fix/redacted-accounts-in-noninteractive branch from 15b5221 to b24fb1d Compare February 25, 2026 06:50
petebacondarwin
petebacondarwin approved these changes Feb 25, 2026
View reviewed changes
@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Feb 25, 2026
@emily-shen emily-shen merged commit f765244 into main Feb 25, 2026
40 of 45 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Feb 25, 2026
@emily-shen emily-shen deleted the mattietk/fix/redacted-accounts-in-noninteractive branch February 25, 2026 09:57
@workers-devprod workers-devprod mentioned this pull request Feb 25, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

@petebacondarwin petebacondarwin petebacondarwin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

workers-sdk
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@MattieTK @petebacondarwin @emily-shen @workers-devprod