Skip to content

fix fixer triggering on reviewer's own comment, full document review, single PR review, comment, clear footer#301

Merged
jth-nw merged 2 commits intomainfrom
dev
Feb 24, 2026
Merged

fix fixer triggering on reviewer's own comment, full document review, single PR review, comment, clear footer#301
jth-nw merged 2 commits intomainfrom
dev

Conversation

@jth-nw
Copy link
Copy Markdown
Collaborator

@jth-nw jth-nw commented Feb 24, 2026

No description provided.

james-haytko_nwx and others added 2 commits February 24, 2026 12:08
The reviewer posts a comment ending with `@claude apply all fixes`.
The fixer's trigger checked for @claude in any comment body, so it
was firing on the reviewer's own comment and applying all fixes
immediately — before the user could review inline suggestions.

Added user.login != 'github-actions[bot]' guard to prevent this.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
- Claude now reads each file in full and categorizes issues as
  'Issues in PR changes' vs 'Preexisting issues'
- Review summary is written to /tmp/review-summary.md and combined
  with inline suggestions into a single PR review (no separate
  issue comment)
- Removed cleanup of issue comments (no longer posted)
- Added footer directing users to inline suggestions for diff-area
  fixes and @claude for preexisting issues

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@jth-nw jth-nw merged commit d61958f into main Feb 24, 2026
6 of 8 checks passed
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