Uh oh!
There was an error while loading. Please reload this page.
docs(standards): the PII gate no longer exists, so stop promising it (backend#1630) - #185
Merged
Merged
Conversation
…(backend#1630) Caught by Bugbot on the model-zoo staging mirror (model-zoo#131, held): the same promotion that deletes public-pii-gate-caller.yml also carries a CLAUDE.md asserting "gitleaks and the PII gate will catch it". The gate was retired earlier today by Lukas's decision (backend#1409); this line was not updated with it. That is a violation of this file's own rule, which I wrote: "If your change makes a statement in any CLAUDE.md, BUGBOT.md, or runbook false, update that file in the same PR." I retired the gate and left the canon claiming it protects - and this canon syncs into every repo's CLAUDE.md, so the false claim was queued to propagate to 19 repos. The replacement text does not just delete the reference; it names the resulting gap. gitleaks still covers secrets in CODE. Nothing now scans PR titles, descriptions or commit messages, which is exactly what the retired gate did and exactly where a customer name is easiest to leak on a public repo. A reader who only saw "gitleaks catches it" would reasonably assume the prose is covered too. Refs backend#1630, backend#1409.
saadqbal
approved these changes
Aug 10, 2026
Uh oh!
There was an error while loading. Please reload this page.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Caught by Bugbot on the model-zoo staging mirror (model-zoo#131, correctly held): the same promotion that deletes
public-pii-gate-caller.ymlalso carries a CLAUDE.md asserting "gitleaks and the PII gate will catch it". The gate was retired earlier today (backend#1409, Lukas's decision); this line was not updated with it.That breaks this file's own rule, which I wrote:"If your change makes a statement in any CLAUDE.md, BUGBOT.md, or runbook false, update that file in the same PR." I retired the gate and left the canon promising it — and this canon syncs into every repo's CLAUDE.md, so the false claim was queued to propagate to 19 repos.
The new text names the gap rather than hiding it
Deleting the reference alone would have been worse than the false claim in one specific way: a reader who saw only "gitleaks catches it" would reasonably assume PR prose was covered too. gitleaks reads code. The retired gate read titles, descriptions and commit messages — which is exactly where a customer name is easiest to leak on a public repo, and now the only control there is review.
Sequencing note for #1602
If any of the ~19
docs/1602-org-standards-syncPRs have already been opened with the old block, they carry the false sentence and need a re-sync after this merges. The sync workflow'screate-prsdispatch regenerates from the canon, so re-running it is the fix — not hand-editing 19 PRs.Unblocks model-zoo#131's held thread: the removal was intentional, and this is the "fix the CLAUDE.md claim" half of the two options that ticket offered.
Parent backend#1405.
Note
Low Risk
Documentation-only change to org standards prose; no runtime, auth, or CI behavior is modified in this diff.
Overview
Fixes stale quality-bar text in the org-wide
org-standards.mdcanon that still said gitleaks and the PII gate will catch customer data in commits and PR metadata after the public PII gate was retired (backend#1409).The bullet now states that gitleaks only scans code, that nothing automated scans PR titles, descriptions, or commit messages anymore, cites the retirement date and ticket, and puts keeping customer names out of public PR prose on reviewers instead of implying a check still exists.
Because this file syncs into every repo’s
CLAUDE.md, the change stops the false claim from propagating on the next org-standards sync.Reviewed by Cursor Bugbot for commit 4c8a410. Bugbot is set up for automated code reviews on this repo. Configure here.