Feature/ca 20 source code based issues relations#136
Merged
Conversation
- Added support for retrieving file snippets by explicit line range in FileViewController, allowing users to specify start and end lines. - Implemented the `getBranchFileSnippetByRange` method in FileViewService to handle the new snippet retrieval logic with inline issue annotations. - Introduced a new endpoint in PullRequestController for bulk updating the status of branch issues, enabling users to resolve or reopen multiple issues at once. - Enhanced error handling and response structure for the bulk update operation, providing detailed feedback on success and failure counts.
… enhance prompt handling
…diffs when range diff fails
…onException during graph sync
…h detailed processing steps
…ith processing logic
… adjust ReconcileTaskScheduler timeout
…revent concurrent calls
…contents - Added `getReconciliationFileContents` method to `AiAnalysisRequest` interface for handling pre-fetched file contents. - Updated `AiAnalysisRequestImpl` to store and return reconciliation file contents. - Modified `BranchAnalysisProcessor` to collect and utilize fetched file contents for AI reconciliation, avoiding redundant VCS calls. - Enhanced `VcsAiClientService` interface and implementations to support passing pre-fetched file contents during branch reconciliation requests. - Introduced new prompt template for direct LLM reconciliation using inlined file contents. - Updated orchestrator and review service to handle direct reconciliation path, improving performance by skipping MCP agent calls when file contents are available. - Added new output schemas for reconciliation results, ensuring structured responses from the LLM.
…s, reducing VCS API calls
…e in Docker Compose
…hAnalysisProcessor tests - Introduced DiffParsingUtils class with methods to parse file paths, split diffs by file, parse hunks, and map line numbers. - Enhanced BranchAnalysisProcessorTest to utilize new branch file operations services, improving test coverage and structure. - Added EventNotificationEmitter for emitting status notifications.
- Introduced DetectionSource enum to differentiate between issues detected via PR analysis and direct push analysis. - Updated CodeAnalysisIssue and BranchIssue models to include detectionSource field. - Modified IssueDTO to accommodate detectionSource in issue representation. - Enhanced CodeAnalysisService to create direct push analyses from AI responses. - Updated repositories to support querying for direct push analyses. - Added state management for PullRequest to track lifecycle states (OPEN, MERGED, DECLINED). - Implemented direct push analysis request building in VCS client services for Bitbucket, GitHub, and GitLab. - Updated database schema to include detection_source and state columns in relevant tables. - Enhanced tests to cover new functionality and ensure backward compatibility. - Adjusted ReviewService logic to handle fresh branch analyses without prior issues.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
No description provided.