refactor: split changelog helpers

Separate changelog generation from output emission so release-note collection and writing logic can evolve independently.

Made-with: Cursor

Author: haasonsaas

TODO.md

@@ -89,7 +89,7 @@
 - [x] `src/commands/misc/feedback/apply.rs`: split acceptance/rejection counters from store mutation helpers.
 - [x] `src/commands/misc/discussion/command.rs`: separate the interactive loop from single-shot execution.
 - [x] `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.
+- [x] `src/commands/misc/changelog.rs`: evaluate splitting changelog collection from output formatting.
 - [x] `src/commands/eval/command.rs`: separate CLI option prep, fixture execution, and report lifecycle.
 - [x] `src/commands/feedback_eval/command.rs`: separate input loading from report/output orchestration.
 

src/commands/misc/changelog.rs

@@ -1,8 +1,14 @@
+#[path = "changelog/generate.rs"]
+mod generate;
+#[path = "changelog/output.rs"]
+mod output;
+
 use anyhow::Result;
 use std::path::PathBuf;
 use tracing::info;
 
-use crate::core;
+use generate::generate_changelog_output;
+use output::emit_changelog_output;
 
 pub async fn changelog_command(
     from: Option<String>,
@@ -11,24 +17,6 @@ pub async fn changelog_command(
     output_path: Option<PathBuf>,
 ) -> Result<()> {
     info!("Generating changelog/release notes");
-
-    let generator = core::ChangelogGenerator::new(".")?;
-
-    let output = if let Some(version) = release {
-        info!("Generating release notes for version {}", version);
-        generator.generate_release_notes(&version, from.as_deref())?
-    } else {
-        let to_ref = to.as_deref().unwrap_or("HEAD");
-        info!("Generating changelog from {:?} to {}", from, to_ref);
-        generator.generate_changelog(from.as_deref(), to_ref)?
-    };
-
-    if let Some(path) = output_path {
-        tokio::fs::write(path, output).await?;
-        info!("Changelog written to file");
-    } else {
-        println!("{}", output);
-    }
-
-    Ok(())
+    let output = generate_changelog_output(from.as_deref(), to.as_deref(), release.as_deref())?;
+    emit_changelog_output(output_path.as_deref(), &output).await
 }

src/commands/misc/changelog/generate.rs

@@ -0,0 +1,21 @@
+use anyhow::Result;
+use tracing::info;
+
+use crate::core;
+
+pub(super) fn generate_changelog_output(
+    from: Option<&str>,
+    to: Option<&str>,
+    release: Option<&str>,
+) -> Result<String> {
+    let generator = core::ChangelogGenerator::new(".")?;
+
+    if let Some(version) = release {
+        info!("Generating release notes for version {}", version);
+        generator.generate_release_notes(version, from)
+    } else {
+        let to_ref = to.unwrap_or("HEAD");
+        info!("Generating changelog from {:?} to {}", from, to_ref);
+        generator.generate_changelog(from, to_ref)
+    }
+}

src/commands/misc/changelog/output.rs

@@ -0,0 +1,14 @@
+use anyhow::Result;
+use std::path::Path;
+use tracing::info;
+
+pub(super) async fn emit_changelog_output(output_path: Option<&Path>, output: &str) -> Result<()> {
+    if let Some(path) = output_path {
+        tokio::fs::write(path, output).await?;
+        info!("Changelog written to file");
+    } else {
+        println!("{}", output);
+    }
+
+    Ok(())
+}