refactor: split feedback eval command helpers

Separate feedback input loading from report emission so the command entrypoint stays focused on wiring the two phases together.

Made-with: Cursor

Author: haasonsaas

TODO.md

@@ -91,7 +91,7 @@
 - [ ] `src/commands/misc/discussion/selection.rs`: split file loading/ID repair from selection rules.
 - [ ] `src/commands/misc/changelog.rs`: evaluate splitting changelog collection from output formatting.
 - [ ] `src/commands/eval/command.rs`: separate CLI option prep, fixture execution, and report lifecycle.
-- [ ] `src/commands/feedback_eval/command.rs`: separate input loading from report/output orchestration.
+- [x] `src/commands/feedback_eval/command.rs`: separate input loading from report/output orchestration.
 
 ### Review pipeline backlog
 

src/commands/feedback_eval/command.rs

@@ -1,30 +1,19 @@
+#[path = "command/load.rs"]
+mod load;
+#[path = "command/report.rs"]
+mod report;
+
 use anyhow::Result;
 use std::path::PathBuf;
 
-use super::input::load_feedback_eval_input;
-use super::report::{
-    build_feedback_eval_report, print_feedback_eval_report, write_feedback_eval_report,
-};
+use load::load_feedback_eval_or_bail;
+use report::emit_feedback_eval_report;
 
 pub async fn feedback_eval_command(
     input_path: PathBuf,
     output_path: Option<PathBuf>,
     confidence_threshold: f32,
 ) -> Result<()> {
-    let loaded = load_feedback_eval_input(&input_path).await?;
-    if loaded.comments.is_empty() {
-        anyhow::bail!(
-            "No accepted/rejected feedback examples found in {}",
-            input_path.display()
-        );
-    }
-
-    let report = build_feedback_eval_report(&loaded, confidence_threshold.clamp(0.0, 1.0));
-    print_feedback_eval_report(&report);
-
-    if let Some(path) = output_path.as_deref() {
-        write_feedback_eval_report(&report, path).await?;
-    }
-
-    Ok(())
+    let loaded = load_feedback_eval_or_bail(&input_path).await?;
+    emit_feedback_eval_report(&loaded, output_path.as_deref(), confidence_threshold).await
 }

src/commands/feedback_eval/command/load.rs

@@ -0,0 +1,17 @@
+use anyhow::Result;
+use std::path::Path;
+
+use super::super::input::load_feedback_eval_input;
+use super::super::LoadedFeedbackEvalInput;
+
+pub(super) async fn load_feedback_eval_or_bail(path: &Path) -> Result<LoadedFeedbackEvalInput> {
+    let loaded = load_feedback_eval_input(path).await?;
+    if loaded.comments.is_empty() {
+        anyhow::bail!(
+            "No accepted/rejected feedback examples found in {}",
+            path.display()
+        );
+    }
+
+    Ok(loaded)
+}

src/commands/feedback_eval/command/report.rs

@@ -0,0 +1,22 @@
+use anyhow::Result;
+use std::path::Path;
+
+use super::super::report::{
+    build_feedback_eval_report, print_feedback_eval_report, write_feedback_eval_report,
+};
+use super::super::LoadedFeedbackEvalInput;
+
+pub(super) async fn emit_feedback_eval_report(
+    loaded: &LoadedFeedbackEvalInput,
+    output_path: Option<&Path>,
+    confidence_threshold: f32,
+) -> Result<()> {
+    let report = build_feedback_eval_report(loaded, confidence_threshold.clamp(0.0, 1.0));
+    print_feedback_eval_report(&report);
+
+    if let Some(path) = output_path {
+        write_feedback_eval_report(&report, path).await?;
+    }
+
+    Ok(())
+}