Skip to content

Improve review pipeline resilience and output behavior#54

Merged
cosmocoder merged 5 commits intomainfrom
fix-issues
Mar 8, 2026
Merged

Improve review pipeline resilience and output behavior#54
cosmocoder merged 5 commits intomainfrom
fix-issues

Conversation

@cosmocoder
Copy link
Copy Markdown
Owner

@cosmocoder cosmocoder commented Mar 8, 2026

The following fixes/changes were made:

  • make large PR chunking resilient to partial failures
  • normalize multi-file review result aggregation
  • remove the stale positives output contract
  • ensure Markdown output respects --output-file
  • preserve partial context when one context retrieval step fails instead of dropping all context.

- Updated `reviewLargePRInChunks` to use `Promise.allSettled` for better error handling during chunk processing.
- Implemented logic to preserve successful chunk results even when some chunks fail, ensuring comprehensive results are returned.
- Flatten file-level results when aggregating multi-file reviews so output formatters receive consistent entries.
- Also wrap skipped single-file results with file metadata.
- Handle context retrieval failures per source so one rejected lookup does not discard successful PR comments, guidelines, code examples, or custom document results.
@cosmocoder cosmocoder merged commit 1d82427 into main Mar 8, 2026
24 checks passed
@cosmocoder cosmocoder deleted the fix-issues branch March 8, 2026 19:10
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 8, 2026

🎉 This PR is included in version 1.2.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant