feat(api): persist fix-loop telemetry

Author: haasonsaas

TODO.md

@@ -53,7 +53,7 @@ This roadmap is derived from deep research into Greptile's public docs, blog, MC
 23. [x] Add a max-iteration policy and loop budget controls for autonomous review convergence.
 24. [x] Add "issue replay" prompts that hand unresolved findings back to a coding agent with file-local context.
 25. [x] Add a handoff contract from reviewer findings to fix agents with rule IDs, evidence, and suggested diffs.
-26. [ ] Persist loop-level telemetry: iterations, fixes attempted, findings cleared, findings reopened.
+26. [x] Persist loop-level telemetry: iterations, fixes attempted, findings cleared, findings reopened.
 27. [x] Add "challenge the finding" verification loops where a validator tries to falsify a suspected issue before keeping it.
 28. [ ] Add caching between iterations so repeated codebase retrieval and verification runs are cheaper.
 29. [ ] Allow loop policies to differ by profile: conservative auditor, high-autonomy fixer, or report-only.

src/core/comment.rs

@@ -36,9 +36,9 @@ use tags::extract_tags;
 
 pub use identity::compute_comment_id;
 pub use types::{
-    Category, CodeSuggestion, Comment, CommentStatus, FixEffort, MergeReadiness, RawComment,
-    ReviewCompletenessSummary, ReviewSummary, ReviewVerificationState, ReviewVerificationSummary,
-    Severity,
+    Category, CodeSuggestion, Comment, CommentStatus, FixEffort, FixLoopTelemetry, MergeReadiness,
+    RawComment, ReviewCompletenessSummary, ReviewSummary, ReviewVerificationState,
+    ReviewVerificationSummary, Severity,
 };
 
 pub struct CommentSynthesizer;

src/core/comment/summary.rs

@@ -74,6 +74,7 @@ pub(super) fn generate_summary(comments: &[Comment]) -> ReviewSummary {
         merge_readiness: default_merge_readiness(open_blockers),
         verification: ReviewVerificationSummary::default(),
         readiness_reasons: Vec::new(),
+        loop_telemetry: None,
     }
 }
 
@@ -83,6 +84,7 @@ pub(super) fn inherit_review_state(
 ) -> ReviewSummary {
     if let Some(previous) = previous {
         summary.verification = previous.verification.clone();
+        summary.loop_telemetry = previous.loop_telemetry.clone();
     }
     apply_review_runtime_state(summary, false)
 }
@@ -211,7 +213,7 @@ mod tests {
     use std::path::PathBuf;
 
     use super::*;
-    use crate::core::comment::{Category, FixEffort};
+    use crate::core::comment::{Category, FixEffort, FixLoopTelemetry};
 
     fn make_comment(
         id: &str,
@@ -409,4 +411,26 @@ mod tests {
         assert_eq!(summary.completeness.fixed_findings, 1);
         assert_eq!(summary.completeness.stale_findings, 1);
     }
+
+    #[test]
+    fn inherit_review_state_preserves_fix_loop_telemetry() {
+        let previous = ReviewSummary {
+            loop_telemetry: Some(FixLoopTelemetry {
+                iterations: 3,
+                fixes_attempted: 2,
+                findings_cleared: 4,
+                findings_reopened: 1,
+            }),
+            ..generate_summary(&[make_comment(
+                "open-warning",
+                Severity::Warning,
+                Category::Bug,
+                CommentStatus::Open,
+            )])
+        };
+
+        let inherited = inherit_review_state(generate_summary(&[]), Some(&previous));
+
+        assert_eq!(inherited.loop_telemetry, previous.loop_telemetry);
+    }
 }

src/core/comment/types.rs

@@ -65,6 +65,20 @@ pub struct ReviewSummary {
     pub verification: ReviewVerificationSummary,
     #[serde(default, skip_serializing_if = "Vec::is_empty")]
     pub readiness_reasons: Vec<String>,
+    #[serde(default, skip_serializing_if = "Option::is_none")]
+    pub loop_telemetry: Option<FixLoopTelemetry>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq, Default)]
+pub struct FixLoopTelemetry {
+    #[serde(default)]
+    pub iterations: usize,
+    #[serde(default)]
+    pub fixes_attempted: usize,
+    #[serde(default)]
+    pub findings_cleared: usize,
+    #[serde(default)]
+    pub findings_reopened: usize,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq, Default)]

src/output/format.rs

@@ -943,6 +943,7 @@ mod tests {
             merge_readiness: crate::core::comment::MergeReadiness::NeedsAttention,
             verification: crate::core::comment::ReviewVerificationSummary::default(),
             readiness_reasons: Vec::new(),
+            loop_telemetry: None,
         };
         let comments = vec![
             build_test_comment(