Claude reviewer may flag already-fixed issues from previous commits

Resolved 💬 2 comments Opened Jul 27, 2025 by CharlonTank Closed Jul 27, 2025

Description

Claude reviewer sometimes continues to flag issues that were already addressed in subsequent commits within the same PR. This can lead to confusion when the reviewer mentions problems that have already been fixed.

Example Scenario

  1. Initial commit includes a temporary file (e.g., a migration script)
  2. Claude reviewer correctly identifies this file should be removed
  3. Developer pushes a follow-up commit removing the file
  4. However, Claude reviewer might still reference the file as if it's present

Expected Behavior

Claude reviewer should analyze the final state of the PR (all commits combined) rather than individual commits, or clearly indicate which commit it's reviewing.

Suggested Improvements

  1. Make Claude reviewer analyze the cumulative diff of all commits in a PR
  2. Or clearly label which specific commit is being reviewed
  3. Add ability to re-run the review after fixes are pushed

Additional Context

This issue can cause developers to spend time investigating problems that have already been resolved, especially in PRs with multiple commits addressing review feedback.

View original on GitHub ↗

This issue has 2 comments on GitHub. Read the full discussion on GitHub ↗