claude-code - 💡(How to fix) Fix /ultrareview returns empty findings [] on large diff instead of error [1 comments, 2 participants]

Either return actual findings, or surface a clear error/partial-result indicator when the review cannot be completed (e.g. diff too large, timeout, internal failure). No error message or partial-result warning was shown. 3. Observe [] findings with completed status and no error.

TL;DR

The issue can be mitigated by implementing a check for large diffs and returning a partial-result indicator or error when the review cannot be completed.

Guidance

• Investigate the /ultrareview command to determine if there is a threshold for diff size that triggers a partial-result indicator or error.
• Modify the command to return a clear error or warning when the diff size exceeds a certain threshold (e.g., 100 files or 10k lines).
• Consider adding a timeout mechanism to handle large diffs that take too long to process.
• Review the task notification status to ensure it accurately reflects the outcome of the review process.

Example

No code snippet is provided as the issue does not contain sufficient information about the implementation details of the /ultrareview command.

Notes

The solution may require modifications to the Claude Code CLI and the /ultrareview command. The exact implementation details are not provided in the issue.

Recommendation

Apply a workaround by modifying the /ultrareview command to return a clear error or warning when the diff size exceeds a certain threshold. This will provide a clear signal to the user that the review did not run due to the large diff size.