Open
Conversation
- Replaced @PreAuthorize annotations with @IsWorkspaceMember and @HasOwnerOrAdminRights in JobController, AllowedCommandUserController, ProjectController, QualityGateController, and VCS controllers (BitbucketCloud, GitHub, GitLab). - Improved readability and maintainability of security checks by utilizing custom annotations. - Removed redundant security checks in various endpoints to streamline access control. - Deleted unused _extract_archive function in web_server.py to clean up codebase.
- Updated import paths for IssueDTO in prompt_builder.py. - Modified analysis_summary prompt in prompt_constants.py for clarity. - Enhanced Dockerfile with CPU threading optimizations. - Improved environment validation in main.py to support Ollama and OpenRouter embedding providers. - Added new endpoints for parsing files and batch parsing in api.py, including AST metadata extraction. - Introduced embedding_factory.py to manage embedding model creation for Ollama and OpenRouter. - Implemented Ollama embedding wrapper in ollama_embedding.py for local model support. - Updated index_manager to check vector dimensions before copying branches. - Refactored RAGConfig to validate embedding provider configurations and auto-detect embedding dimensions. - Adjusted query_service to utilize the new embedding factory for model instantiation.
…action management in JobService and WebhookAsyncProcessor
|
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
✨ Finishing touches🧪 Generate unit tests (beta)
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.
This pull request introduces several enhancements across environment configuration, security, PR analysis enrichment, and concurrency handling. The most notable changes are the addition of new DTOs and builder support for PR enrichment data, improvements to job logging transactional behavior, and new security annotations for workspace-based access control.
Pull Request Enrichment and DTOs:
FileContentDto,FileRelationshipDto,ParsedFileMetadataDto, andPrEnrichmentDataDto, which provide structured file contents, dependency relationships, and parsing metadata for more intelligent and context-aware PR analysis. [1] [2] [3] [4]AiAnalysisRequestImplto include enrichment data, with builder and getter support forPrEnrichmentDataDto, enabling the analysis engine to consume enriched file and relationship information. [1] [2] [3] [4] [5] [6]module-info.javafor analysis engine.Security and Access Control:
IsWorkspaceOwner,IsWorkspaceMember, andHasOwnerOrAdminRights, enabling declarative workspace-based access control for Spring-managed beans and methods. [1] [2] [3] [4]Job Logging and Concurrency:
JobServiceto useREQUIRES_NEWtransaction propagation, ensuring logs are committed immediately and not blocked by parent transactions. This improves reliability and responsiveness of job progress logging. [1] [2] [3]versionfield toVcsConnectionentity for optimistic locking, preventing concurrent token refresh operations from overwriting each other.Environment Configuration:
.env.samplefiles for both RAG pipeline and web frontend, adding configuration options for Ollama/OpenRouter embedding provider, performance tuning, and new blog URL. [1] [2]