Skip to content

Conversation

@zjwu0522
Copy link
Collaborator

@zjwu0522 zjwu0522 commented Dec 2, 2025

Summary

  • Add _obfuscate_mentions() method that replaces @username with @username_XXXX (random suffix) to prevent notifications to real GitHub users
  • Add safety check that prevents importing templates to public repositories
  • Apply obfuscation to issue bodies, PR bodies, and all comment content
  • Add README notice explaining the privacy requirement and changes

This addresses feedback from original GitHub authors who were inadvertently notified when evaluation repositories were created as public.

Test plan

  • Verify regex correctly handles simple usernames: @user@user_x7k2
  • Verify regex correctly handles hyphenated usernames: @user-name@user-name_a3f9
  • Verify regex correctly excludes emails: [email protected] should NOT be modified
  • Verify public repo import raises RuntimeError with clear error message

🤖 Generated with Claude Code

- Add _obfuscate_mentions() method that replaces @username with @username_XXXX
  (random suffix) to prevent notifications to real GitHub users
- Add safety check that prevents importing templates to public repositories
- Apply obfuscation to issue bodies, PR bodies, and all comment content
- Add README notice explaining the privacy requirement and changes

This addresses feedback from original GitHub authors who were inadvertently
notified when evaluation repositories were created as public.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Copy link
Collaborator

@xyliugo xyliugo left a comment

Choose a reason for hiding this comment

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

good fix

@xyliugo xyliugo merged commit 26f1bbb into main Dec 3, 2025
@xyliugo xyliugo deleted the feat/obfuscate-github-mentions branch December 3, 2025 06:46
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