Skip to content

test: public PII gate smoke check (tracebloc/.github#62)#87

Closed
saadqbal wants to merge 4 commits into
developfrom
test/pii-gate-smoke-62
Closed

test: public PII gate smoke check (tracebloc/.github#62)#87
saadqbal wants to merge 4 commits into
developfrom
test/pii-gate-smoke-62

Conversation

@saadqbal

@saadqbal saadqbal commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Smoke test: confirms the public-pii-gate.yml@main reusable workflow now resolves after publishing to tracebloc/.github@main (was failing 0s with no jobs). No-op doc file; will be closed once the Public PII gate check goes green. Tracking: tracebloc/.github#62


Note

Low Risk
CI-only wiring and a disposable doc file; the workflow ref change is intentional and temporary for validation, with no application or data-path impact.

Overview
Temporary smoke test for the Public PII gate CI check after publishing fixes in tracebloc/.github.

The caller workflow public-pii-gate-caller.yml now pins the reusable workflow to @fix/pii-gate-token-perms instead of @main, so this PR can confirm the gate actually runs (previously it failed in ~0s with no jobs). A throwaway PII_GATE_SMOKE.md was added to document the check and force a fresh synchronize run; both are meant to be reverted or closed once the gate is green on @main (tracebloc/.github#62).

Reviewed by Cursor Bugbot for commit b27747b. Bugbot is set up for automated code reviews on this repo. Configure here.

@saadqbal saadqbal self-assigned this Jun 18, 2026
@LukasWodka

Copy link
Copy Markdown
Contributor

👋 Heads-up — Code review queue is at 31 / 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.)

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit b27747b. Configure here.

jobs:
pii-gate:
uses: tracebloc/.github/.github/workflows/public-pii-gate.yml@main
uses: tracebloc/.github/.github/workflows/public-pii-gate.yml@fix/pii-gate-token-perms

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caller pins unstable workflow ref

High Severity

The reusable workflow uses ref was changed from @main to @fix/pii-gate-token-perms, while the new smoke doc and PR goal say the check validates resolution from @main. Every other caller workflow in this repo pins @main. Merging this leaves the Public PII gate on a short-lived branch ref that can break all PR checks when that branch is removed.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit b27747b. Configure here.

@saadqbal

Copy link
Copy Markdown
Collaborator Author

Test PR for tracebloc/.github#62 — confirmed the Public PII gate resolves and runs green once the reusable workflow's token-perms are fixed (run 27748465192, minimal caller). Closing; fix tracked in tracebloc/.github#63.

@saadqbal saadqbal closed this Jun 18, 2026
@saadqbal saadqbal deleted the test/pii-gate-smoke-62 branch June 18, 2026 09:06
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.

2 participants