From 8af810cdd26854c35b015f70eb20479c4d1f9643 Mon Sep 17 00:00:00 2001 From: Sharang Parnerkar Date: Wed, 18 Mar 2026 14:21:09 +0100 Subject: [PATCH] fix: stop storing code review findings in dashboard, use PR comments only MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Code review findings are not actionable in the findings dashboard — they lack PR context and clutter the list. The PR review pipeline already posts inline comments directly on PRs (GitHub, Gitea, GitLab), which is the appropriate place for code review feedback. - Remove LLM code review stage from the scan pipeline (orchestrator) - Remove "Code Review" option from the findings type filter dropdown Co-Authored-By: Claude Opus 4.6 (1M context) --- compliance-agent/src/pipeline/orchestrator.rs | 16 ---------------- compliance-dashboard/src/pages/findings.rs | 1 - 2 files changed, 17 deletions(-) diff --git a/compliance-agent/src/pipeline/orchestrator.rs b/compliance-agent/src/pipeline/orchestrator.rs index b8c1314..b02dc7a 100644 --- a/compliance-agent/src/pipeline/orchestrator.rs +++ b/compliance-agent/src/pipeline/orchestrator.rs @@ -10,7 +10,6 @@ use compliance_core::AgentConfig; use crate::database::Database; use crate::error::AgentError; use crate::llm::LlmClient; -use crate::pipeline::code_review::CodeReviewScanner; use crate::pipeline::cve::CveScanner; use crate::pipeline::git::GitOps; use crate::pipeline::gitleaks::GitleaksScanner; @@ -241,21 +240,6 @@ impl PipelineOrchestrator { Err(e) => tracing::warn!("[{repo_id}] Lint scanning failed: {e}"), } - // Stage 4c: LLM Code Review (only on incremental scans) - if let Some(old_sha) = &repo.last_scanned_commit { - tracing::info!("[{repo_id}] Stage 4c: LLM Code Review"); - self.update_phase(scan_run_id, "code_review").await; - let review_output = async { - let reviewer = CodeReviewScanner::new(self.llm.clone()); - reviewer - .review_diff(&repo_path, &repo_id, old_sha, ¤t_sha) - .await - } - .instrument(tracing::info_span!("stage_code_review")) - .await; - all_findings.extend(review_output.findings); - } - // Stage 4.5: Graph Building tracing::info!("[{repo_id}] Stage 4.5: Graph Building"); self.update_phase(scan_run_id, "graph_building").await; diff --git a/compliance-dashboard/src/pages/findings.rs b/compliance-dashboard/src/pages/findings.rs index 8784fe2..b7b546b 100644 --- a/compliance-dashboard/src/pages/findings.rs +++ b/compliance-dashboard/src/pages/findings.rs @@ -123,7 +123,6 @@ pub fn FindingsPage() -> Element { option { value: "oauth", "OAuth" } option { value: "secret_detection", "Secrets" } option { value: "lint", "Lint" } - option { value: "code_review", "Code Review" } } select { onchange: move |e| { status_filter.set(e.value()); page.set(1); }, -- 2.49.1