docs: remove operator personal email (PII) from the docs#42
Merged
Conversation
…tructions The owner's personal Gmail appeared next to the Cloudflare account id in two docs — in ROADMAP as the wrangler-whoami account name, and in the project-instructions as an account label. It was convenience metadata, never required: the non-secret account id alone identifies the account. Strip the email from both (keep the id), and note in the instructions that the owner email is intentionally omitted. Note: the email still exists in already-merged git history / past PR descriptions; scrubbing that would require rewriting shared history and is out of scope here. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LVPyHAtQ1cdK76RpFasafP
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
wecanjustbuildthings | 33f2fcd | Commit Preview URL Branch Preview URL |
Jul 03 2026, 03:21 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Removes the operator's personal Gmail from the two docs that carried it, keeping the non-secret Cloudflare account id.
Why it was there: convenience metadata, not a requirement. Cloudflare names an account
"<owner-email>'s Account", so the email rode along next to the account id in two places —ROADMAP.md(thewrangler whoamiaccount name) anddocs/claude-project-instructions.md(an account label). The prior review flagged the account id as non-secret and correct to keep; the email is personal PII with no reason to be in a public repo. The account id alone identifies the account, so the email is safe to strip.Changes (docs only):
ROADMAP.md— "These3remain@gmail.com's Account" → "the project's Cloudflare account" (id unchanged).docs/claude-project-instructions.md— dropped the email; kept the account id and added a one-line note that the owner email is intentionally omitted.A repo-wide scan confirms no personal email remains in any tracked file.
Note: the email still exists in already-merged git history and past PR descriptions. Scrubbing that would require rewriting shared history (and force-pushing
main) — invasive and out of scope for this change; say the word if you want to pursue it separately.Type of change
Required checks
npm run check— N/A (no schema touched)npm run enforce— N/A (no catalog/recipe/provider strings touched)npm test— N/A (no code touched)npm run build— N/A (Markdown-only)For catalog entries
N/A — no catalog entries in this PR.
For recipes
N/A — no recipes in this PR.
🤖 Generated with Claude Code
Generated by Claude Code