mirror of
https://github.com/alibaba/open-code-review.git
synced 2026-07-10 01:39:12 +00:00
482de7ac52
* fix(agent): skip binary files during code review Previously, binary files were passed through the review pipeline because shouldReview only checked path/extension rules. This caused the agent to incorrectly attempt reviewing binary content. - Add ExcludeBinary reason to the exclusion filter - Change shouldReview/whyExcluded to accept model.Diff instead of path string, enabling access to the IsBinary flag - Check IsBinary as the first gate in whyExcluded - Differentiate skip log messages for binary vs path-filtered files * test: add unit tests for preview.go * fix: normalize dash to em-dash in binary file skip log message * test(agent): strengthen UserIncludePattern tests to cover additive semantics - Document that include patterns are additive, not exclusive - Add cases verifying include bypasses default-path exclusion - Add cases for extension filter precedence over include logic - Add TestWhyExcluded_IncludeBypassesDefaultPath for rescue semantics - Add TestWhyExcluded_IncludeAndExcludeInteraction for priority order |
||
|---|---|---|
| .. | ||
| agent | ||
| config | ||
| diff | ||
| llm | ||
| model | ||
| release | ||
| session | ||
| stdout | ||
| suggestdiff | ||
| telemetry | ||
| tool | ||
| viewer | ||