Skip to content

ci(security): add public-repo PII gate caller#81

Merged
shujaatTracebloc merged 2 commits into
developfrom
ci/public-pii-gate-caller
Jun 17, 2026
Merged

ci(security): add public-repo PII gate caller#81
shujaatTracebloc merged 2 commits into
developfrom
ci/public-pii-gate-caller

Conversation

@LukasWodka

Copy link
Copy Markdown
Contributor

Adds the per-repo caller for the org-wide public-repo PII gate (tracebloc/.github#60). Blocks PRs whose title/body/commits contain a denylisted customer name or secret. Inactive (warn-only) until the PII_DENYLIST org secret is set, so this won't block existing PRs on merge.

Part of the public-repo PII cleanup (customer names were found leaked in public PR bodies).

Blocks PRs that leak customer/partner names or secrets in title/body/commits.
Calls the reusable gate in tracebloc/.github. Inactive until the org
PII_DENYLIST secret is set (warns, doesn't block, until then).
@LukasWodka LukasWodka self-assigned this Jun 17, 2026
@LukasWodka

Copy link
Copy Markdown
Contributor Author

👋 Heads-up — Code review queue is at 32 / 30

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

The vendored copy at internal/schema/ingest.v1.json had drifted from
upstream, failing the `scripts/sync-schema.sh --check` CI gate on every
PR. Upstream replaced `instance_segmentation` with `token_classification`
across the category enums, updated the texts/resolution descriptions
([width, height] order + PIL note), and added the token_classification
`texts` and masked_language_modeling no-`label` conditional rules.

Regenerated via `scripts/sync-schema.sh`; `--check` is now clean.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants