1.5.5 rc

[rostilos]

Summary by CodeRabbit

• New Features

  • Added support for filtering code analysis issues to display only the latest version in PR views
  • Implemented exclusion of resolved issues from inline PR file analysis displays

• Bug Fixes

  • Improved PR file viewers to show relevant analysis results by filtering to latest versions and hiding resolved issues

[codecrow-local]

⚠️ Code Analysis Results

Quality Gate Default Quality Gate: 🔴 FAILED

• MEDIUM Issues by Severity > 0 (actual: 1) - FAILED

Summary

Pull Request Review: 1.5.5 rc

Status	PASS WITH WARNINGS
Risk Level	LOW
Review Coverage	3 files analyzed in depth
Confidence	HIGH

---

Executive Summary

This PR prepares the 1.5.5 release candidate by updating repository methods and service layers for code analysis retrieval. The changes focus on data persistence and aggregation across the core library and web-server services. While the logic is sound, the introduction of complex JPA subqueries presents a potential performance bottleneck for the UI as the dataset grows.

Recommendation

Decision: PASS WITH WARNINGS

The PR is functionally complete, but it is recommended to address the query efficiency in CodeAnalysisIssueRepository to prevent UI latency. Consider denormalizing the version metadata or optimizing the subquery logic before the final release.

Issues Overview

Severity	Count
🟡 Medium	1	Issues that should be addressed

---

Analysis completed on 2026-03-06 12:15:10 | View Full Report | Pull Request

---

📋 Detailed Issues (1)

🟡 Medium Severity Issues

Id on Platform: 3794

Category: 🏗️ Architecture

File: .../codeanalysis/CodeAnalysisIssueRepository.java:115

Potential Performance Degradation in Code Analysis Retrieval

Potential Performance Degradation in Code Analysis Retrieval
The introduction of complex JPA queries with nested MAX subqueries in the repository layer creates a systemic performance risk. These queries are consumed by both the core CodeAnalysisService and the web-server's FileViewService to provide real-time UI data (file tree counts and inline annotations).
Evidence: The repository defines a query using SELECT MAX(a.prVersion) FROM CodeAnalysis a which is called by FileViewService to aggregate issue counts. As the number of analysis runs per project/PR grows, this O(N) or O(log N) lookup per row will degrade UI responsiveness.
Business impact: Slow page loads in the code review UI, leading to developer frustration and increased database CPU utilization during peak review periods.
Also affects: java-ecosystem/services/web-server/src/main/java/org/rostilos/codecrow/webserver/analysis/service/FileViewService.java, java-ecosystem/libs/core/src/main/java/org/rostilos/codecrow/core/service/CodeAnalysisService.java

💡 Suggested Fix

Consider denormalizing the 'latest version' flag into the CodeAnalysis table or using a specialized native query with a Window Function (RANK/DENSE_RANK) if the database supports it, rather than a correlated subquery.

View Issue Details

---

Files Affected

• .../codeanalysis/CodeAnalysisIssueRepository.java: 1 issue

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Added two new repository query methods to filter code analysis issues by latest PR version, corresponding service methods to expose this functionality, and updated the file view service to use these latest-version queries while filtering out resolved issues.

Changes

Cohort / File(s)	Summary
Repository Layer java-ecosystem/libs/core/src/main/java/org/rostilos/codecrow/core/persistence/repository/codeanalysis/CodeAnalysisIssueRepository.java	Added two new query methods: findByProjectIdAndPrNumberLatestVersion and findByProjectIdAndPrNumberAndFilePathLatestVersion to retrieve issues scoped to the latest PR version using subqueries for MAX(prVersion).
Service Layer java-ecosystem/libs/core/src/main/java/org/rostilos/codecrow/core/service/CodeAnalysisService.java	Added two public methods: findIssuesByPrNumberLatestVersion and findIssuesByPrNumberAndFilePathLatestVersion that delegate to the new repository methods.
Consumer Layer java-ecosystem/services/web-server/src/main/java/org/rostilos/codecrow/webserver/analysis/service/FileViewService.java	Updated method calls in listPrFiles, getPrFileView, getPrFileSnippet, and getPrFileSnippetByRange to use latest-version service methods and added filtering to exclude resolved issues from inline concerns.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Hops through PRs with grace so fine,
Latest versions now align,
Resolved issues fade away,
Fresh analysis leads the way! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title '1.5.5 rc' is a version designation rather than a descriptive summary of the actual code changes made in the pull request.	Use a descriptive title that summarizes the main changes, such as 'Add latest-version filtering for PR code analysis issues' or 'Support latest PR version in code analysis queries and viewers'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch 1.5.5-rc

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}