Skip to content

fix(ci): allowlist tickets.localhost + theirdomain.com in customer-PII guard - #2062

Merged
ToddHebebrand merged 1 commit into
mainfrom
ToddHebebrand/issue-2046-ci-red-main
Jun 29, 2026
Merged

fix(ci): allowlist tickets.localhost + theirdomain.com in customer-PII guard#2062
ToddHebebrand merged 1 commit into
mainfrom
ToddHebebrand/issue-2046-ci-red-main

Conversation

@ToddHebebrand

Copy link
Copy Markdown
Collaborator

What

#2060 (editable inbound email alias) introduced two placeholder email domains in its planning/spec docs that the customer-PII guard (Security Audit job) doesn't recognise, turning main red again:

  • support@tickets.localhostdocs/superpowers/plans/2026-06-29-inbound-alias.md
  • partner-1@theirdomain.comdocs/superpowers/specs/2026-06-29-inbound-alias-design.md

Fix

  • Add localhost to the reserved-TLD suffix group (RFC 6761 reserves .localhost).
  • Add theirdomain.com to the exact placeholder-domain allowlist, alongside the existing yourdomain.com / theirmsp.com entries.

Mechanical follow-up to #2059 (same recurring class of failure). Guard passes locally (exit=0).

Refs #2046

🤖 Generated with Claude Code

…I guard

#2060 added placeholder emails in its planning/spec docs whose domains
weren't recognised by the customer-PII guard, reddening main:

  • support@tickets.localhost  (docs/superpowers/plans/2026-06-29-inbound-alias.md)
  • partner-1@theirdomain.com   (docs/superpowers/specs/2026-06-29-inbound-alias-design.md)

Add `localhost` to the reserved-TLD suffix group (RFC 6761) and
`theirdomain.com` to the exact placeholder-domain list, mirroring the
existing `yourdomain.com` / `theirmsp.com` entries.

Refs #2046

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ToddHebebrand
ToddHebebrand merged commit 8b7a2e0 into main Jun 29, 2026
24 of 26 checks passed
@ToddHebebrand
ToddHebebrand deleted the ToddHebebrand/issue-2046-ci-red-main branch June 29, 2026 23:52
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.

1 participant