🔥 Remove legacy GeneratedReview structured review system

Delete the backwards-compatibility code path for dimension-based code reviews
in favor of the simpler MarkdownReview approach.

- Remove GeneratedReview, CodeIssue, DimensionAnalysis, and QualityDimension
  types along with their formatting implementations (~600 lines)
- Remove Review variant from StructuredResponse enum in iris.rs
- Clean up unused imports (textwrap, SUCCESS_GREEN) and constants
- Update public exports in lib.rs and types/mod.rs
- Remove obsolete tests for legacy review formatting

Author: hyperb1iss

src/agents/iris.rs

@@ -59,10 +59,8 @@ pub enum StructuredResponse {
     PullRequest(crate::types::GeneratedPullRequest),
     Changelog(crate::types::ChangelogResponse),
     ReleaseNotes(crate::types::ReleaseNotesResponse),
-    /// New markdown-based review (LLM-driven structure)
+    /// Markdown-based review (LLM-driven structure)
     MarkdownReview(crate::types::MarkdownReview),
-    /// Legacy structured review (kept for backwards compatibility)
-    Review(Box<crate::types::GeneratedReview>),
     PlainText(String),
 }
 
@@ -84,9 +82,6 @@ impl fmt::Display for StructuredResponse {
             StructuredResponse::MarkdownReview(review) => {
                 write!(f, "{}", review.format())
             }
-            StructuredResponse::Review(review) => {
-                write!(f, "{}", review.format())
-            }
             StructuredResponse::PlainText(text) => {
                 write!(f, "{text}")
             }
@@ -740,16 +735,6 @@ Guidelines:
                     .await?;
                 Ok(StructuredResponse::MarkdownReview(response))
             }
-            // Legacy structured review (kept for backwards compatibility)
-            "GeneratedReview" => {
-                let response = self
-                    .execute_with_agent::<crate::types::GeneratedReview>(
-                        &system_prompt,
-                        user_prompt,
-                    )
-                    .await?;
-                Ok(StructuredResponse::Review(Box::new(response)))
-            }
             _ => {
                 // Fallback to regular agent for unknown types
                 let agent = self.build_agent()?;

src/lib.rs

@@ -38,6 +38,6 @@ pub use providers::{Provider, ProviderConfig};
 
 // Re-exports from types module
 pub use types::{
-    CodeIssue, DimensionAnalysis, GeneratedMessage, GeneratedPullRequest, GeneratedReview,
-    MarkdownReview, QualityDimension, format_commit_message, format_pull_request,
+    GeneratedMessage, GeneratedPullRequest, MarkdownReview, format_commit_message,
+    format_pull_request,
 };

src/types/mod.rs

@@ -20,10 +20,7 @@ pub use self::commit::{GeneratedMessage, format_commit_message};
 pub use pr::{GeneratedPullRequest, format_pull_request};
 
 // Review types
-pub use review::{
-    CodeIssue, DimensionAnalysis, GeneratedReview, MarkdownReview, QualityDimension,
-    render_markdown_for_terminal,
-};
+pub use review::{MarkdownReview, render_markdown_for_terminal};
 
 // Changelog types
 pub use changelog::{BreakingChange, ChangeEntry, ChangeMetrics, ChangelogResponse, ChangelogType};