Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The reviewer previously selected changed filenames and then read whole files, which could include unrelated legacy or unstaged code while missing regressions visible only in deleted lines, renames, or workflow patches. Git-based scopes now review the patch first, with an initial-commit fallback and an explicit zero-pass response for an empty staged diff. Aggregate JSON also has a documented, consistent
issuesschema.The integration suite now covers those review scopes alongside auto-fix behavior and tidy cleanup. It checks the final behavior of both repaired fixture bugs, recognizes equivalent finding descriptions without requiring a line number, and stops immediately with the underlying Claude CLI error when a live call fails. The contributor guide and offline validation checks reflect the expanded coverage.
Validation:
make test— 69 passed.make integrationwas reported passing before the final harness cleanup; the final revision could not be rerun here because this environment has no Claude login or API key.