-
Notifications
You must be signed in to change notification settings - Fork 44
Support tagging Claude coauthor #752
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Orca Security Scan Summary
| Status | Check | Issues by priority | |
|---|---|---|---|
| Infrastructure as Code | View in Orca | ||
| SAST | View in Orca | ||
| Secrets | View in Orca | ||
| Vulnerabilities | View in Orca |
|
Please mark which AI tools you used for this PR by checking the appropriate boxes:
Tip: If you want to avoid this comment in the future, you can add a label of the format |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✨ PR Review
The PR adds a new automation to label PRs authored by Claude or with Claude as a co-author. The implementation looks mostly sound but has regex pattern issues that could affect matching accuracy.
2 issues detected:
🐞 Bug - Applying case-insensitive regex patterns to already lowercased strings is redundant and potentially error-prone.
Details: The regex pattern uses case-insensitive character classes
[Cc]laudeafter already converting the string to lowercase. This redundancy could cause unexpected matching behavior or inefficiency.
File:docs/downloads/automation-library/integrations/claude_code/label_claude_code_by_co_author.cm (14-14)🐞 Bug - The current pattern may not match all legitimate co-author line formats used in practice.
Details: The regex pattern for detecting co-author lines may be too restrictive and could miss legitimate variations of the "Co-authored-by" format, such as different casing or spacing patterns commonly used in Git commits.
File:docs/downloads/automation-library/integrations/claude_code/label_claude_code_by_co_author.cm (15-15)
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀
docs/downloads/automation-library/integrations/claude_code/label_claude_code_by_co_author.cm
Outdated
Show resolved
Hide resolved
docs/downloads/automation-library/integrations/claude_code/label_claude_code_by_co_author.cm
Show resolved
Hide resolved
…el_claude_code_by_co_author.cm Co-authored-by: gitstream-cm[bot] <111687743+gitstream-cm[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✨ PR Review
The PR adds automation to detect Claude-authored or co-authored PRs and apply labels. The implementation has improved from previous versions, but still contains a pattern matching issue that could result in false positives.
1 issues detected:
🐞 Bug - Substring matching without word boundaries will match unintended usernames containing "claude".
Details: The author matching pattern will match any username containing "claude" as a substring, potentially causing false positives for users like "claudesmith", "mcclaude", or "claudenova".
File:docs/downloads/automation-library/integrations/claude_code/label_claude_code_by_co_author.cm (14-14)
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀
docs/downloads/automation-library/integrations/claude_code/label_claude_code_by_co_author.cm
Show resolved
Hide resolved
… coauthor-tagging
MishaKav
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✨ PR Review
LGTM
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀
✨ PR Description
Purpose: Add tagging system for PRs with Claude as co-author to track and label AI-assisted code contributions.
Main changes:
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀