feat: Enhance RAG incremental update notifications with detailed status messages

[rostilos]

Summary by CodeRabbit

• Improvements
  • Added status notifications for search index updates, providing visibility when operations are skipped due to module unavailability, disabled features, or initialization in progress, as well as confirmation on successful completion.

[codecrow-local]

⚠️ Code Analysis Results

Quality Gate Default Quality Gate: ✅ PASSED

Summary

Pull Request Review: feat: Enhance RAG incremental update notifications with detailed status messages

Status	PASS WITH WARNINGS
Risk Level	LOW
Review Coverage	1 file analyzed in depth
Confidence	HIGH

---

Executive Summary

This PR enhances the RAG incremental update process by introducing detailed status messages within the BranchAnalysisProcessor. While the changes successfully improve visibility into the analysis loop, they introduce minor architectural redundancy by duplicating string construction logic found in other modules. The overall risk is low, but the implementation creates a small maintenance burden regarding message consistency.

Recommendation

Decision: PASS WITH WARNINGS

The PR is functionally sound and safe to merge; however, it is recommended to consolidate RAG notification templates into a shared service or constant file to prevent UI-facing string divergence across the analysis and RAG engines.

---

Issues Overview

Severity	Count
🔵 Low	1	Minor issues and improvements

---

Analysis completed on 2026-03-01 23:16:12 | View Full Report | Pull Request

---

📋 Detailed Issues (1)

🔵 Low Severity Issues

Id on Platform: 3600

Category: 🏗️ Architecture

File: .../analysis/BranchAnalysisProcessor.java:1

Redundant Status Message Construction Logic

Redundant Status Message Construction Logic
The PR introduces manual string concatenation for RAG update status messages in BranchAnalysisProcessor.java, while RagOperationsServiceImpl.java already contains similar logic for triggering incremental updates and reporting status. This creates a maintenance burden where UI-facing status strings are scattered across different modules (analysis-engine vs rag-engine).
Evidence: BranchAnalysisProcessor.java (Line 684) constructs a message using 'rag_update_complete' with concatenated counts. RagOperationsServiceImpl.java (Line 135) uses a similar pattern: 'Updating RAG index with ' + (addedOrModifiedSize + deletedFiles.size()) + ' changed files'.
Business impact: Inconsistent user experience if status message formats diverge between different parts of the RAG lifecycle. Localization or branding changes would require updates in multiple disconnected modules.
Also affects: java-ecosystem/libs/rag-engine/src/main/java/org/rostilos/codecrow/ragengine/service/RagOperationsServiceImpl.java

💡 Suggested Fix

Consolidate RAG notification string templates into a shared constant file or a dedicated NotificationService within a common library to ensure uniform messaging across the analysis and RAG engines.

View Issue Details

---

Files Affected

• .../analysis/BranchAnalysisProcessor.java: 1 issue

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6239553 and 1a4247e.

📒 Files selected for processing (1)

• java-ecosystem/libs/analysis-engine/src/main/java/org/rostilos/codecrow/analysisengine/processor/analysis/BranchAnalysisProcessor.java

---

📝 Walkthrough

Walkthrough

The BranchAnalysisProcessor.performIncrementalRagUpdate method now emits status notifications for various RAG processing scenarios: unavailable service, disabled project settings, unready index, and successful updates. Eight lines were added to track and communicate incremental RAG update progress.

Changes

Cohort / File(s)	Summary
RAG Status Emission Updates java-ecosystem/libs/analysis-engine/src/main/java/org/rostilos/codecrow/analysisengine/processor/analysis/BranchAnalysisProcessor.java	Added status emission calls to performIncrementalRagUpdate method to emit rag_skipped for unavailable service, disabled projects, and unready indexes; emit rag_update_complete on successful RAG updates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A rabbit hops through status streams,
Emitting signals, chasing dreams,
RAG updates flow with careful care,
Each skip and success declared with flair! ✨

✨ 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.3-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.}