Skip to content

chore: teach reviews to catch untagged PII logs - #6923

Merged
chenghao-mou merged 3 commits into
mainfrom
chenghao/chore/AGT-3074-pii-log-review
Aug 21, 2026
Merged

chore: teach reviews to catch untagged PII logs#6923
chenghao-mou merged 3 commits into
mainfrom
chenghao/chore/AGT-3074-pii-log-review

Conversation

@chenghao-mou

@chenghao-mou chenghao-mou commented Aug 20, 2026

Copy link
Copy Markdown
Member

PII-bearing log and telemetry values can bypass redaction when they appear in message or name fields, use untagged structured keys, or surface through exception messages, tracebacks, cause chains, and unretrieved tasks.

Teach repository reviews to flag these paths across nested metadata, provider payloads, and duplicate emissions, even when static tests miss the field.

Addresses AGT-3074

Initial prompt and agent context

Model: GPT-5.6

can we create a new stacked PR on top of the 6356 with a new REVIEW.md such that we check for PII sensitive logs and whether they are properly tagged or not?

we don't have to stack the PR actually.

we should add exception logging into the scope as well. We recently had a leak in log #6740

@chenghao-mou
chenghao-mou requested a review from a team as a code owner August 20, 2026 13:43

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@chenghao-mou
chenghao-mou force-pushed the chenghao/chore/AGT-3074-pii-log-review branch from 74da01c to 3d196e2 Compare August 20, 2026 13:49
@chenghao-mou
chenghao-mou changed the base branch from chenghao/feat/redaction-support to main August 20, 2026 13:50
@chenghao-mou
chenghao-mou merged commit 89c7104 into main Aug 21, 2026
8 checks passed
@chenghao-mou
chenghao-mou deleted the chenghao/chore/AGT-3074-pii-log-review branch August 21, 2026 09:23
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