From a9d039dad3c266c0bfbb04601d763cfc1fff2121 Mon Sep 17 00:00:00 2001 From: Sharang Parnerkar Date: Wed, 18 Mar 2026 15:18:07 +0000 Subject: [PATCH] fix: stop storing code review findings in dashboard (#22) --- 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); },