Move GitHub integration to backend, add live progress & inline PR reviews

- Move all GitHub API calls from client-side @octokit/rest to backend
  /api/gh/* endpoints (repos, PRs, status, PR review)
- Remove @octokit/rest dependency and web/src/lib/github.ts
- Upgrade post_pr_review_comments to post inline comments with the
  GitHub Reviews API (severity icons, suggestions, code blocks)
- Add avatar_url to GhStatusResponse, stargazers_count/private to GhRepo
- Add React Query hooks for GitHub operations (useGhStatus, useGhRepos,
  useGhPrs, useStartPrReview)
- Add ProgressCallback/ProgressUpdate to review pipeline for live
  per-file progress tracking
- Add ProgressBanner and EventPanel components to ReviewView
- Frontend shows partial results during scan (comments stream in)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: haasonsaas

src/review/mod.rs

@@ -7,8 +7,10 @@ mod rule_helpers;
 pub use filters::apply_review_filters;
 pub use pipeline::{
     review_diff_content, review_diff_content_with_repo, review_diff_content_raw,
+    review_diff_content_raw_with_progress,
     extract_symbols_from_diff, filter_comments_for_diff,
     build_symbol_index, build_review_guidance,
+    ProgressCallback, ProgressUpdate,
 };
 pub use feedback::{
     load_feedback_store, load_feedback_store_from_path, save_feedback_store,

src/review/pipeline.rs

@@ -8,13 +8,28 @@ use std::path::PathBuf;
 use std::time::Instant;
 use tracing::{info, warn};
 
+use std::sync::Arc;
+
 use crate::adapters;
 use crate::config;
 use crate::core;
 use crate::core::offline::optimize_prompt_for_local;
 use crate::output::OutputFormat;
 use crate::parsing::parse_llm_response;
 use crate::plugins;
+
+/// Progress update emitted per file during review.
+pub struct ProgressUpdate {
+    pub current_file: String,
+    pub files_total: usize,
+    pub files_completed: usize,
+    pub files_skipped: usize,
+    /// Comments found so far (accumulated from all completed files).
+    pub comments_so_far: Vec<core::Comment>,
+}
+
+/// Callback invoked before each file's LLM call and after completion.
+pub type ProgressCallback = Arc<dyn Fn(ProgressUpdate) + Send + Sync>;
 use super::context_helpers::{
     inject_custom_context, inject_pattern_repository_context,
     rank_and_trim_context_chunks, resolve_pattern_repositories,
@@ -46,6 +61,16 @@ pub async fn review_diff_content_raw(
     diff_content: &str,
     config: config::Config,
     repo_path: &Path,
+) -> Result<Vec<core::Comment>> {
+    review_diff_content_raw_with_progress(diff_content, config, repo_path, None).await
+}
+
+/// Like `review_diff_content_raw` but with an optional progress callback.
+pub async fn review_diff_content_raw_with_progress(
+    diff_content: &str,
+    config: config::Config,
+    repo_path: &Path,
+    on_progress: Option<ProgressCallback>,
 ) -> Result<Vec<core::Comment>> {
     // For local models, chunk oversized diffs instead of truncating
     if should_optimize_for_local(&config) {
@@ -61,7 +86,7 @@ pub async fn review_diff_content_raw(
             let mut all_comments = Vec::new();
             for (i, chunk) in chunks.iter().enumerate() {
                 eprintln!("Processing chunk {}/{}...", i + 1, chunks.len());
-                match review_diff_content_raw_inner(chunk, config.clone(), repo_path).await {
+                match review_diff_content_raw_inner(chunk, config.clone(), repo_path, on_progress.clone()).await {
                     Ok(comments) => all_comments.extend(comments),
                     Err(e) => {
                         eprintln!("Warning: chunk {} failed: {}", i + 1, e);
@@ -72,17 +97,23 @@ pub async fn review_diff_content_raw(
         }
     }
 
-    review_diff_content_raw_inner(diff_content, config, repo_path).await
+    review_diff_content_raw_inner(diff_content, config, repo_path, on_progress).await
 }
 
 async fn review_diff_content_raw_inner(
     diff_content: &str,
     config: config::Config,
     repo_path: &Path,
+    on_progress: Option<ProgressCallback>,
 ) -> Result<Vec<core::Comment>> {
     let diffs = core::DiffParser::parse_unified_diff(diff_content)?;
     info!("Parsed {} file diffs", diffs.len());
 
+    // Pre-count reviewable files for progress tracking
+    let files_total = diffs.len();
+    let mut files_completed: usize = 0;
+    let mut files_skipped: usize = 0;
+
     // Check file change limit
     if let Some(limit) = config.file_change_limit {
         if limit > 0 && diffs.len() > limit {
@@ -151,17 +182,31 @@ async fn review_diff_content_raw_inner(
         // Check if file should be excluded
         if config.should_exclude(&diff.file_path) {
             info!("Skipping excluded file: {}", diff.file_path.display());
+            files_skipped += 1;
             continue;
         }
         if diff.is_deleted {
             info!("Skipping deleted file: {}", diff.file_path.display());
+            files_skipped += 1;
             continue;
         }
         if diff.is_binary || diff.hunks.is_empty() {
             info!("Skipping non-text diff: {}", diff.file_path.display());
+            files_skipped += 1;
             continue;
         }
 
+        // Emit progress: about to review this file
+        if let Some(ref cb) = on_progress {
+            cb(ProgressUpdate {
+                current_file: diff.file_path.display().to_string(),
+                files_total,
+                files_completed,
+                files_skipped,
+                comments_so_far: all_comments.clone(),
+            });
+        }
+
         let mut context_chunks = context_fetcher
             .fetch_context_for_file(
                 &diff.file_path,
@@ -337,6 +382,8 @@ async fn review_diff_content_raw_inner(
             let comments = filter_comments_for_diff(diff, comments);
             all_comments.extend(comments);
         }
+
+        files_completed += 1;
     }
 
     // Run post-processors to filter and refine comments