-
Notifications
You must be signed in to change notification settings - Fork 0
1.0.0 rc #29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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
…pic + openai + google providers compatibility for MCP client. MCP-client error handling
…ling ( pipeline-agent side )
…upport for new Gemini-3 preview models
…nhance issue count updates
…nd PR author settings
…rove analysis mode handling
…ting in repository. Fix issue with non-resolving issues duplicates ( branch reconciliation )
…ed issue reconciliation
…status handling for previous issues
…ure correct handling of duplicate issues
… tokens and enhance connection handling
… issue deduplication; enhance prompt builder with detailed instructions
feat: Add AST-based code chunking support with new splitter and confi…
…revious issues and updating prompt instructions for unresolved issues
feat: Enhance branch reconciliation by restoring missing diffs from p…
… compiled Python files
…apabilities in branch issues
Enhanced Filtering and Query Capabilities, Issues API improvements
…for repository interactions
feat: Add GitLab support with configuration, client, and API actions
…ality ( + re-apply webhook + small bugfix on branch issues count )
feat: Implement webhook management and VCS connection change functionalyty + bugfix
|
Important Review skippedToo many files! 148 files out of 298 files are above the max files limit of 150. You can disable this status message by setting the Comment |
…l repo path handling
feat: Enhance webhook setup for REPOSITORY_TOKEN connections with ful…
feat: Add GitLab support to VcsMcpClientFactory for client creation
feat: Add GitLab personal/repository token support in VCS connection …
…uest and branch events - Added GitHubPullRequestWebhookHandler to handle pull request events from GitHub. - Introduced GitHubWebhookParser for parsing GitHub webhook payloads. - Created GitLabBranchWebhookHandler for handling branch push events in GitLab. - Developed GitLabMergeRequestWebhookHandler to manage merge request events from GitLab. - Implemented GitLabWebhookParser for parsing GitLab webhook payloads. - Refactored AllowedCommandUserService and ProjectService to utilize unified VCS repo bindings. - Updated ReviewRequestDto to include GitLab as a supported VCS provider.
feat: Implement GitHub and GitLab webhook handlers for pull/merge req…
incremental RAG updates ( fixes )
fix: Ensure proper categorization of files in incremental RAG updates
…incremental updates
fix: Update file loading logic to handle relative paths correctly in …
… logging in RAGIndexManager
fix: Enhance collection existence check to support aliases in Qdrant
fix: Enhance GitLabReportingService to support inline comments and im…
fix: Enhance GitLab OAuth handling for repository onboarding and webh…
changes -> diffs ( gitlab MR diff retrieval API changes )
…d analysis updates - Added target branch name and current commit hash to AI client services for GitHub and GitLab. - Implemented findPullRequestForCommit method in GitHubOperationsService to retrieve PRs associated with commits. - Implemented findPullRequestForCommit method in GitLabOperationsService to retrieve MRs associated with commits. - Created GitHubPrMergeWebhookHandler to handle PR merge events and trigger branch reconciliation. - Created GitLabMrMergeWebhookHandler to handle MR merge events and trigger branch reconciliation. - Updated AnalysisIssueController and ProjectAnalyticsController to support new issue resolution context (resolvedByPr and resolvedCommitHash). - Enhanced IssueStatusUpdateRequest to include resolvedByPr and resolvedCommitHash fields. - Updated AnalysisService to handle new resolution metadata when updating issue status. - Improved prompt builder documentation for resolved issues. - Removed outdated IMPLEMENTATION_SUMMARY.md file.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This pull request introduces several improvements and refactorings to the analysis engine and deployment configuration, focusing on enhanced support for incremental code analysis, improved PR diff handling, and codebase organization. The most important changes are grouped below.
Incremental Analysis and PR Diff Handling:
AiAnalysisRequestand its implementation, including new fields and builder methods forAnalysisMode,deltaDiff, and commit hashes. This enables more precise and efficient code analysis workflows. [1] [2] [3] [4] [5] [6]Codebase Refactoring and Organization:
clientpackage toaiclientin the analysis engine, updating all relevant exports, opens, and import statements for consistency and clarity. [1] [2] [3] [4] [5]BranchAnalysisProcessorto use the unifiedgetEffectiveVcsRepoInfo()accessor, simplifying and consolidating logic for obtaining repository information.Branch Issue Management:
Configuration and Integration Enhancements:
application.properties.sample, along with documentation for setting up GitLab OAuth applications. [1] [2].env.sample, supporting improved document chunking for retrieval-augmented generation.Submodule Update:
frontendsubmodule to a newer commit.This pull request introduces several enhancements and fixes across the codebase, mainly focused on improving incremental analysis support, enriching DTOs with detection metadata, and refining VCS diff handling. It also adds configuration options for GitLab OAuth and updates the frontend submodule.Key changes include:
Incremental Analysis & DTO Enhancements
AiAnalysisRequestinterface and its implementation to support incremental analysis by adding fields foranalysisMode,deltaDiff,previousCommitHash, andcurrentCommitHash, along with corresponding builder methods. [1] [2] [3] [4] [5] [6]IssueDTOrecord to include detection metadata such asanalysisId,prNumber,commitHash, anddetectedAt, and modified thefromEntitymethod to populate these fields. [1] [2]VCS Diff Handling Improvements
VcsOperationsServiceinterface for fetching pull request diffs (getPullRequestDiff) and commit range diffs (getCommitRangeDiff), enabling more precise diff retrieval for incremental and PR-based analyses.BranchAnalysisProcessorto use PR diffs when a source PR number is provided, ensuring all files from the original PR are analyzed, and improved branch issue count updates after mapping. [1] [2] [3] [4]idinstead ofissueId) for AI issue mapping.Configuration & Integration Updates
application.properties.sample, including instructions and new properties for client ID, secret, and base URL.Module & Utility Updates
DiffContentFilterto reference the correct module (vcs-mcp).Frontend